-
Notifications
You must be signed in to change notification settings - Fork 245
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
Bump Beyla to version 1.5.1 #645
Conversation
CHANGELOG.md
Outdated
@@ -39,6 +39,7 @@ Main (unreleased) | |||
- Update `alloy-mixin` to use more specific alert group names (for example, | |||
`alloy_clustering` instead of `clustering`) to avoid collision with installs | |||
of `agent-flow-mixin`. (@rfratto) | |||
- Bump Beyla to v1.5.1. (@marctc) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ideally, it'd be good to mention how this affects Alloy users. Would that be very difficult to do? It seems there are lots of changes in 1.5.0, so maybe it's not very easy.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it doesn't affect users in a notable way, then maybe we don't need to mention this upgrade in the changelog.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm also not quite sure which version we're upgrading from.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would linking to Beyla's changelog be enough? There aren't changes to Beyla component itself, just internal changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IIRC, for exporters when bumping to a newer version we rarely explained what changed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does every change in Beyla's changelog impact Alloy users? It would be most helpful if you just describe in one sentence what the sorts of improvements are noteworthy from an Alloy user point of view. I'm still not sure which version we're upgrading from, so not sure which changelog to link to also.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does every change in Beyla's changelog impact Alloy users?
No, only the users that use Beyla's componet in Alloy.
PR Description
Bumps Beyla to version 1.5.1, which includes a bunch of bugfixes.
PR Checklist