-
Notifications
You must be signed in to change notification settings - Fork 11
OSPRH-16753: Converge tests based on enabled features #275
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
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/bc6f82e76d23449e9466de93d4537f53 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 17m 14s |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/3785f304595a4596939682f1b0621d35 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 16m 36s |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/975fcf27071f481387790d8187b30e84 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 15m 34s |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/56d2d6b1c80647a5baf5bd38bb65a456 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 20m 02s |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/c9283d8c0ac8407fa6d0fe58af253e36 ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 27m 54s |
ff4c733
to
76eae50
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/ed8bb4f450374d91922c3828853b4566 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 52m 29s |
f0c084d
to
c03852a
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/05999691d7c84e5b9f4801eaa621c95f ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 20m 00s |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/0823578f8ac040fd8b1b5852f23abd12 ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 31m 55s |
ci/logging_tests_computes.yml
Outdated
@@ -18,10 +18,12 @@ | |||
- name: "Run Telemetry Logging tests" | |||
ansible.builtin.import_role: | |||
name: telemetry_logging | |||
when: hostvars['localhost'].enable_logging_role | bool |
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.
Instead of guarding each task, you could use some pre_tasks to skip this play if the variable is false.
Here's an example:
feature-verification-tests/ci/report_result.yml
Lines 62 to 66 in e855781
pre_tasks: | |
- name: Stop play early if we do not want to check the failures | |
when: | |
- not (check_failures | default(true) | bool) | |
ansible.builtin.meta: end_play |
ci/logging_tests_computes.yml
Outdated
@@ -18,10 +18,12 @@ | |||
- name: "Run Telemetry Logging tests" | |||
ansible.builtin.import_role: | |||
name: telemetry_logging | |||
when: hostvars['localhost'].enable_logging_role | bool |
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 would prefer some other name, like logging_enabled or logging_features_enabled, but that's up for debate
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.
Same feedback here for the pre_tasks. It will reduce the size of the change, and reduce the possibility of errors later, if new tasks are needed for the autoscaling tests, for example.
ci/run_cloudops_tests_osp18.yml
Outdated
@@ -13,10 +13,12 @@ | |||
- name: Include vars from the extra_vars files | |||
ansible.builtin.include_vars: | |||
dir: "{{ cifmw_basedir }}/artifacts/parameters" | |||
when: hostvars['localhost'].enable_autoscaling_role | bool |
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.
Guarding the block is sufficient here. The individual task don't need to be guarded.
ci/run_fvt_tests.yml
Outdated
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.
If going with the approach of using pre_tasks (as suggested above), then the check for each set of tests could happen in the pre_task section of the play.
There's an alternative approach where the tests are in one large playbook, with multiple plays that will detect whether the particular features are enabled, then try to run all the tests, with guarding in the appropriate place.
cafbf53
to
b17f5eb
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/878a9bbe4f5d4ed2b70c374414688cc7 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 56m 40s |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/a314b27fc5814757b28ca74ccd4601a4 ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 32m 22s |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/1b790b56ab794be3a32f8c9af7b0d295 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 49m 29s |
11751b1
to
703620c
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/a8d56a905d5642bbad01aa3aa42daed5 ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 01m 43s |
703620c
to
e7b26ba
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/44b8d8ead7f446f6b6204cd90e4330ae ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 56m 58s |
e7b26ba
to
106945a
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/df73bca68c0b46d6a463b9341ccec9ea ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 00m 55s |
19dd5ab
to
b83b6c7
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/02117aaf34fc4483be6ebbdd62bc1803 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 56m 15s |
b83b6c7
to
f2d40c7
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/3acaa7b1a2804525bce5e11bc79eeb52 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 57m 43s |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/171d52d383204608a719a1b72fbb0446 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 59m 44s |
Converge the following tests based on enabled features
Depends-On: openstack-k8s-operators/telemetry-operator#696