Skip to content

[Backport perf-v17] feat(tags): add JenkinsJob tag for billing aggregation#14121

Draft
scylladbbot wants to merge 1 commit intoscylladb:branch-perf-v17from
scylladbbot:backport/14118/to-perf-v17
Draft

[Backport perf-v17] feat(tags): add JenkinsJob tag for billing aggregation#14121
scylladbbot wants to merge 1 commit intoscylladb:branch-perf-v17from
scylladbbot:backport/14118/to-perf-v17

Conversation

@scylladbbot
Copy link

Add a new 'JenkinsJob' tag derived from BUILD_TAG that strips the trailing dash and build number. This enables aggregating costs per job in billing reports, since JenkinsJobTag (which includes the full BUILD_TAG with build number) is split across multiple runs and can't be natively aggregated in billing systems.

Example:
BUILD_TAG=jenkins-scylla-master-tier1-longevity-12h-test-128
JenkinsJobTag=jenkins-scylla-master-tier1-longevity-12h-test-128
JenkinsJob=jenkins-scylla-master-tier1-longevity-12h-test

Testing

Screenshot 2026-03-18 at 18 36 56

PR pre-checks (self review)

  • I added the relevant backport labels
  • I didn't leave commented-out/debugging code

Reminders

  • Add New configuration option and document them (in sdcm/sct_config.py)

  • Add unit tests to cover my changes (under unit-test/ folder)

  • Update the Readme/doc folder relevant to this change (if needed)

  • (cherry picked from commit c088ec5)

Parent PR: #14118

Add a new 'JenkinsJob' tag derived from BUILD_TAG that strips the trailing
dash and build number. This enables aggregating costs per job in billing
reports, since JenkinsJobTag (which includes the full BUILD_TAG with build
number) is split across multiple runs and can't be natively aggregated in
billing systems.

Example:
  BUILD_TAG=jenkins-scylla-master-tier1-longevity-12h-test-128
  JenkinsJobTag=jenkins-scylla-master-tier1-longevity-12h-test-128
  JenkinsJob=jenkins-scylla-master-tier1-longevity-12h-test
(cherry picked from commit c088ec5)
@scylladbbot
Copy link
Author

@roydahan - This PR has conflicts, therefore it was moved to draft
Please resolve them and mark this PR as ready for review

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants