Skip to content
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

Subset of metadata fields #113

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

rsevilla87
Copy link
Member

@rsevilla87 rsevilla87 commented Sep 26, 2024

Type of change

  • Refactor
  • New feature
  • Bug fix
  • Optimization
  • Documentation Update

Description

Follow-up of kube-burner/kube-burner#702
As agreed with the team (cc @mukrishn), let's add a small subset of metadata fields to all metrics (ocpVersion and ocpMajorVersion) generated by kube-burner.

Related Tickets & Documents

  • Related Issue #
  • Closes #

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.

go.mod Outdated Show resolved Hide resolved
@rsevilla87 rsevilla87 force-pushed the minimal-metadata branch 3 times, most recently from cbe6ed4 to ef52eef Compare September 26, 2024 13:12
go.mod Outdated
@@ -85,3 +85,5 @@ require (
sigs.k8s.io/structured-merge-diff/v4 v4.4.1 // indirect
sigs.k8s.io/yaml v1.3.0 // indirect
)

replace github.com/kube-burner/kube-burner => github.com/rsevilla87/kube-burner v1.7.6-0.20240926090502-2e59036445b9
Copy link
Contributor

Choose a reason for hiding this comment

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

upstream is merged, will wait for this to be updated as well.

Signed-off-by: Raul Sevilla <[email protected]>
Signed-off-by: Raul Sevilla <[email protected]>
Signed-off-by: Raul Sevilla <[email protected]>
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