-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Validate that any change to default metrics defined in metadata.yaml are accompanied by a "breaking" changelog #10902
Comments
Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
Removing To add justification, backends often charge users based on how many metrics are being ingested. New metrics being sent to backends result in billing changes. From this, I believe customer impact is high enough to warrant the breaking change label. |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
@atoulme Since mdatagen is moving to |
yes please |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping |
transferred issue as mdatagen lives in the core repo now |
Component(s)
cmd/mdatagen
Is your feature request related to a problem? Please describe.
I want to make sure that every time we change the default metrics associated with a receiver, we can validate that this results in a breaking change called out in a changelog message.
Describe the solution you'd like
Add to the generated files created by mdatagen the hash of the default metrics, as a base64 encoded string.
Add a warning above the hash explaining that the hash is here to explicitly need to be changed when a change to default metrics is made, and that this qualifies the component to report a "breaking" changelog.
Add a github action build with the following algorithm:
record the changelog type - if breaking, exit early
runs
git diff
, greps for any occurrence of the hash change.if any change found, check that the changelog type is "breaking".
If not exit 1.
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: