Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[telemetry_chargeback] Price needs to be calculated for each synthetic dataframe#420

Draft
ayefimov-1 wants to merge 3 commits into
masterinfrawatch/feature-verification-tests:masterfrom
alexy_ck_job_8einfrawatch/feature-verification-tests:alexy_ck_job_8eCopy head branch name to clipboard
Draft

[telemetry_chargeback] Price needs to be calculated for each synthetic dataframe#420
ayefimov-1 wants to merge 3 commits into
masterinfrawatch/feature-verification-tests:masterfrom
alexy_ck_job_8einfrawatch/feature-verification-tests:alexy_ck_job_8eCopy head branch name to clipboard

Conversation

@ayefimov-1

@ayefimov-1 ayefimov-1 commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

gen_synth_loki_data.py:

  • Compute price at generation time: price = unit_cost * qty_mutated.
  • Add _apply_mutate() for NUMBOOL/CEIL/FLOOR transforms.
  • Removed 2 hour offset

Update test scenarios:

  • 1-hour window, 300s steps, realistic costs.
  • Remove factor/offset from all files
  • Remove "price" from test scenario

Authored-by: @ayefimov-1
AI Assisted by: Claude

ayefimov-1 and others added 2 commits July 25, 2026 08:04
Compute price at generation time: price = unit_cost * qty_mutated.
Simplify gen_db_summary.py to sum price directly (matching CK method).
Add _apply_mutate() for NUMBOOL/CEIL/FLOOR transforms.
Update test scenarios: 1-hour window, 300s steps, realistic costs.
Remove factor/offset from all files.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
gen_synth_loki_data.py:

Compute price at generation time: price = unit_cost * qty_mutated.
Add _apply_mutate() for NUMBOOL/CEIL/FLOOR transforms.
Removed 2 hour offset
gen_db_summary.py

Simplify to sum "price" directly (matching CK method)
Use price to generate rating
Test Scenarios files:

Update test scenarios: 1-hour window, 300s steps, realistic costs.
Remove factor/offset from all files
Remove "price" from test scenario

Authored-by: @ayefimov-1
AI Assisted by: Claude
@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 420,4e43ff27dabd81ffb1b563870271a071a6e5e5fd

@ayefimov-1
ayefimov-1 marked this pull request as draft July 27, 2026 00:02
@ayefimov-1 ayefimov-1 changed the title [telemetry_chargeback] Fix rating calculation for synthetic data [telemetry_chargeback] Price needs to be calculated for each dataframe for synthetic data Jul 27, 2026
@ayefimov-1 ayefimov-1 changed the title [telemetry_chargeback] Price needs to be calculated for each dataframe for synthetic data [telemetry_chargeback] Price needs to be calculated for each synthetic dataframe Jul 27, 2026
@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/0f88d4a56b9444cd8e2c45a632cc595f

✔️ telemetry-openstack-meta-content-provider-master SUCCESS in 2h 38m 49s
functional-chargeback-tests-osp18 FAILURE in 2h 14m 05s
functional-tests-osp18 FAILURE in 2h 11m 35s

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.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.