-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add 1.19.0 changelog #1451
Add 1.19.0 changelog #1451
Conversation
Signed-off-by: Arthur Silva Sens <arthur.sens@coralogix.com>
Once this is done, I can mark the Version function in |
@@ -1,5 +1,12 @@ | |||
## Unreleased | |||
|
|||
## 1.19.0 / 2023-02-27 | |||
|
|||
The module `prometheus/common v0.48.0` introduced a bug when used together with client_golang. If your project uses client_golang and you want to use `prometheus/common v0.48.0` or higher, please update client_golang to v1.19.0. |
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.
Personally I find this frustrating, because I can't figure out what the bug is,
even after reading the full list of commits between 1.18 and 1.19.
Could you give a hint?
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.
A few constants ceased to exist (or were made private) in common 0.48.0, client_golang would break when trying to access those constants.
The fix was done here: #1448
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.
and now that you mention I totally agree that this text could be clearer, I'm just not sure if users are interested in such explanation "constant values could not be accessed anymore" 🤔
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.
Thanks, linking to the fix is enough information to trace it through.
I would say the root of the thing is prometheus/common#576; some constants were deliberately hidden. I would use the word "incompatibility" rather than "bug".
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.
We still haven't merged release-1.19
back to main
, so I've raised #1454 and we can get it rephrased :)
No description provided.