Skip to content

Fix benchmark version and release date in tailoring files#120

Merged
mpurg merged 3 commits intomasterfrom
fix_tailoring_description
Feb 27, 2026
Merged

Fix benchmark version and release date in tailoring files#120
mpurg merged 3 commits intomasterfrom
fix_tailoring_description

Conversation

@mpurg
Copy link
Copy Markdown
Contributor

@mpurg mpurg commented Feb 27, 2026

The benchmark version and release date are now parametrized in the tailoring file template and replaced at build time.

This fixes the issue of the tailoring files (and thus report) having the same description, version, and release_date regardless of which benchmark they are associated with.

Related bug report: https://bugs.launchpad.net/usg/+bug/2142286

Other changes:

  • tailoring file templates use string.Template placeholders
  • introduced 'release_date' to benchmark release metadata
  • introduced 'latest_benchmark_id' to channel data
  • switched to YAML BaseLoader and cast fields explicitly
  • minor fix in error logging in build tools

This fixes the issue of the tailoring files (and thus report) having the
same description, version, and release_date regardless of which
benchmark they are associated with.

The benchmark version and release date are now parametrized in the
tailoring file template and replaced at build time.

Other changes:
- tailoring file templates use string.Template placeholders
- introduced 'release_date' to benchmark release metadata
- introduced 'latest_benchmark_id' to channel data
- switched to YAML BaseLoader and cast fields explicitly
@mpurg mpurg added the bug Something isn't working label Feb 27, 2026
Copy link
Copy Markdown
Contributor

@dodys dodys left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks!

@mpurg mpurg merged commit bce4b37 into master Feb 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants