-
Notifications
You must be signed in to change notification settings - Fork 14
Validate all applicable pods, services and containers are running for chargeback #324
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
Conversation
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/2a5c4fcdfd814d0b807abd853c67e071 ❌ telemetry-openstack-meta-content-provider-master FAILURE in 8m 14s |
|
recheck |
elfiesmelfie
left a comment
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.
Please rebase this change onto the updated main, and remove references to the observe_chargeback role for now.
#319 can cover adding the observe_chargeback role and adding it to the run_chargeback_tests playbook
ci/chargeback_tests.yml
Outdated
| @@ -0,0 +1,58 @@ | |||
| --- | |||
| - name: "Verify all the applicable projects, endpoints, pods & services for cloudkitty" | |||
| hosts: controller | |||
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.
Please update this to be consistent with the other playbooks.
| hosts: controller | |
| hosts: "{{ cifmw_target_hook_host | default('localhost') }}" |
ci/chargeback_tests.yml
Outdated
| - [cloudkitty,rating,public] | ||
| - [cloudkitty,rating,internal] | ||
|
|
||
| common_service_test_id: "" |
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.
We don't need the test_ids anymore, please remove them.
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.
done
2720487 to
51d9914
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/fadc639283664f76aabae9febc0e64d5 ❌ telemetry-openstack-meta-content-provider-master FAILURE in 35m 51s |
51d9914 to
de9f6f3
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/257d276bc3904da3830f6ad780a92efe ✔️ telemetry-openstack-meta-content-provider-master SUCCESS in 2h 54m 32s |
Jira: OSPRH-21447 Validate all applicable pods, services and containers are running
de9f6f3 to
54f8454
Compare
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/3c2d773805f141ae8ca19cdbe28e80a8 ❌ telemetry-openstack-meta-content-provider-master FAILURE in 5m 03s |
elfiesmelfie
left a comment
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 made some minor updates to the PR:
- removed the dependency on the chargeback role.
- rebased against master
- renamed the playbook to be consistent with other playbooks in repo
We discussed merging this into the https://github.com/elfiesmelfie/feature-verification-tests/tree/test_ck_job branch for now, until the CI job PR is in a state to be merged.
Can you re-target this PR to the elfiesmelfie/test_ck_job branch instead of origin/master?
|
Thank you Muneesha! Once the dependent PRs are merged, it'll be merged into master with the rest of the cloudkitty job |
|
Thank you |
Validate all applicable pods, services and containers are running for chargeback