-
Notifications
You must be signed in to change notification settings - Fork 144
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
waiter.py Add ClusterOperator Test #879
waiter.py Add ClusterOperator Test #879
Conversation
Build succeeded. ✔️ ansible-galaxy-importer SUCCESS in 4m 49s |
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.
could you please also add a changelog fragment? thanks!
Co-authored-by: Mandar Kulkarni <[email protected]>
Co-authored-by: Mandar Kulkarni <[email protected]>
Build succeeded. ✔️ ansible-galaxy-importer SUCCESS in 4m 11s |
Build succeeded. ✔️ ansible-galaxy-importer SUCCESS in 7m 51s |
It should be there now |
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.
The code looks good; you need to fix the CI issues
Co-authored-by: Bikouo Aubin <[email protected]>
Co-authored-by: Bikouo Aubin <[email protected]>
I am having a small problem with the linter. I was copying the deployments.yml fixture which seems to have multiple documents. However, when I run the linter it's getting tripped up
I'm not exactly sure how to fix this except by removing usecases |
Build succeeded. ✔️ ansible-galaxy-importer SUCCESS in 2m 47s |
Build succeeded. ✔️ ansible-galaxy-importer SUCCESS in 3m 55s |
Hi @stratus-ss , for this, could you please add line below to files in https://github.com/stratus-ss/kubernetes.core/tree/ClusterOperator/tests/sanity
similar to kubernetes.core/tests/sanity/ignore-2.19.txt Lines 14 to 16 in 91df2f1
|
Build succeeded. ✔️ ansible-galaxy-importer SUCCESS in 2m 51s |
@stratus-ss you'll need to add the change to all ignore files here https://github.com/stratus-ss/kubernetes.core/tree/ClusterOperator/tests/sanity ignore-2.14.txt |
Build succeeded. ✔️ ansible-galaxy-importer SUCCESS in 3m 57s |
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.
LGTM!
Build succeeded (gate pipeline). ✔️ ansible-galaxy-importer SUCCESS in 4m 20s |
7cdf0d0
into
ansible-collections:main
SUMMARY
Fixes #869
During an OpenShift installation, one of the checks to see that the cluster is ready to proceed with configuration is to check to ensure that the Cluster Operators are in an Available: True Degraded: False Progressing: False state. While you can currently use the k8s_info module to get a json response, the resulting json needs to be iterated over several times to get the appropriate status.
This PR adds functionality into
waiter.py
which loops over all resource instances of the cluster operators. If any of them is not ready, waiter returnsFalse
and the task false. If the task returns, you can assume that all the cluster operators are healthy.ISSUE TYPE
COMPONENT NAME
waiter.py
ADDITIONAL INFORMATION
A simple playbook will trigger the
waiter.py
to watch theClusterOperator
objectThis will produce the simple response if everything is functioning properly:
If the timeout is reached:
UNSOLVED: How to know which Operators are failing