-
Notifications
You must be signed in to change notification settings - Fork 216
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add status reconcile tests #2416
Conversation
383f119
to
6b878d7
Compare
7ccb214
to
157aa9f
Compare
b08c007
to
c857009
Compare
5f12e60
to
27674fa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added a few comments and questions.
In general I think we can check for more Status fields and verify that all fields make sense after updating dependant objects.
The tests make sense to verify that updating a bundledeployment
triggers an update in bundle
or gitrepo
(for example), but I think we could also use the tests for verifying that the whole Status is what we expect to be.
40cd41c
to
b184137
Compare
* BundleDeployment triggers Bundle * Bundle triggers GitRepo * BundleDeployment triggers Cluster * uses BundleDeployment and GitRepo for update * Cluster triggers ClusterGroup
Triggers by status change on bundleDeployments Triggers by a change in cluster
b184137
to
8222ef2
Compare
Refers to #2400
Add integration tests to check status reconciles