-
Notifications
You must be signed in to change notification settings - Fork 14
[roles/common] allow using a dict to configure pod tests #281
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
base: master
Are you sure you want to change the base?
Conversation
8d32b41 to
07cd96c
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/62b33ff373e4457db11f5976dd7b9ed6 ✔️ feature-verification-tests-noop SUCCESS in 5s |
The pod tests are currently configured by setting a common_pod_list var that specifies the names of the pods. These pods must all be in the same namespace and share the same status_str and test_id. This means that the user needs to run the role multiple times in order to check a list of pods that are in different namespaces or have different status_str values. This change allow the configuration of the tests to either remain the same or to use a dictionary of values per pod, allowing to check for pods across multiple namespaces or with multiple statuses and run the role once.
f45e510 to
d995322
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/bfa4716dbb4b41bd8451e61d880db5a7 ✔️ feature-verification-tests-noop SUCCESS in 4s |
The pod tests are currently configured by setting a common_pod_list var that specifies the names of the pods. These pods must all be in the same namespace and share the same status_str and test_id. This means that the user needs to run the role multiple times in order to check a list of pods that are in different namespaces or have different status_str values.
This change allow the configuration of the tests to either remain the same or to use a dictionary of values per pod, allowing to check for pods across multiple namespaces or with multiple statuses and run the role once.
Depends-On: openstack-k8s-operators/ci-framework#3063