Skip to content

[telemetry_chargeback] add the ability to pass test scenario names via extra-vars#375

Open
ayefimov-1 wants to merge 26 commits into
masterfrom
alexy_ck_job_6
Open

[telemetry_chargeback] add the ability to pass test scenario names via extra-vars#375
ayefimov-1 wants to merge 26 commits into
masterfrom
alexy_ck_job_6

Conversation

@ayefimov-1
Copy link
Copy Markdown
Contributor

@ayefimov-1 ayefimov-1 commented May 4, 2026

add the ability to pass test scenario names via extra-vars

  • adding cloudkitty_test_scenarios variable to overwrite default list of test scenarios
  • debug_dir option for python scripts added for consistency

Authored-by: @myadla
Co-authored-by: @ayefimov-1
AI Assisted by: Claude

ayefimov-1 and others added 22 commits April 20, 2026 11:27
- uses synth data to calculate total cost via script
- run "openstack rating summary get" to get total cost from loki
- compares script_totals and Loki_Totals if same then job passes
- Used Gemini and Cursor AI
push and retriieved from loki

- uses synth data to calculate total cost via script
- run "openstack rating summary get" to get total cost from loki
- compares script_totals and Loki_Totals if same then job passes
- Used Gemini and Cursor AI
* Validates chargeback data is generated and then
push and retriieved from loki

- uses synth data to calculate total cost via script
- run "openstack rating summary get" to get total cost from loki
- compares script_totals and Loki_Totals if same then job passes
- Used Gemini and Cursor AI

* Review Changes

* Delete roles/telemetry_chargeback/.gitignore

---------

Co-authored-by: ayefimov <ayefimov@redhat.com>
push and retriieved from loki

- uses synth data to calculate total cost via script
- run "openstack rating summary get" to get total cost from loki
- compares script_totals and Loki_Totals if same then job passes
- Used Gemini and Cursor AI
Address review feedback: consolidate repeated when conditions into
blocks and skip the entire workflow when no scenarios are found.
Adding a method to selecting telemetry_chargeback scenarios to run which do not require changing the role
@centosinfra-prod-github-app
Copy link
Copy Markdown

Unable to freeze job graph: Unable to modify final job <Job noop explicit: None implied: None source: None#0> attribute parent=noop with variant <Job feature-verification-tests-noop explicit: None implied: {MatchAny:{ImpliedBranchMatcher:master}} source: infrawatch/feature-verification-tests/.zuul.yaml@master#139>

@ayefimov-1 ayefimov-1 changed the base branch from master to alexy_ck_job_5 May 4, 2026 21:22
extra-vars

Written by @alexy
Assisted by Claude
@ayefimov-1 ayefimov-1 force-pushed the alexy_ck_job_5 branch 2 times, most recently from cc028a1 to 92234d4 Compare May 13, 2026 20:29
@ayefimov-1 ayefimov-1 changed the base branch from alexy_ck_job_5 to alexy_ck_job_5a May 15, 2026 14:11
@ayefimov-1 ayefimov-1 force-pushed the alexy_ck_job_5a branch 3 times, most recently from a48c3a8 to 3a395e0 Compare May 19, 2026 12:52
@ayefimov-1 ayefimov-1 changed the base branch from alexy_ck_job_5a to master May 19, 2026 15:34
Added missing cloudkitty_debug_dir variable documentation that exists
in defaults/main.yml but was not documented in the README.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@centosinfra-prod-github-app
Copy link
Copy Markdown

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/infrawatch/feature-verification-tests for 375,8c0ad607f040b1e5e4be351dd9889db9a7bf5f58

@centosinfra-prod-github-app
Copy link
Copy Markdown

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/infrawatch/feature-verification-tests for 375,1cf9456df73b054ca50ac5a070a6835de7383d65

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants