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

Fix go_collector_latest_test Fail on go1.19 #1136

Merged
merged 1 commit into from Nov 7, 2022

Conversation

rogerogers
Copy link
Contributor

Fix go_collector_latest_test Fail on go1.19
#1135

Signed-off-by: rogerogers rogers@rogerogers.com

Signed-off-by: rogerogers <rogers@rogerogers.com>
@dswarbrick
Copy link
Contributor

When this is merged, it would be nice to also update the circleci config to actually test with Go 1.19 (as the release notes state).

@kakkoyun
Copy link
Member

kakkoyun commented Nov 7, 2022

When this is merged, it would be nice to also update the circleci config to actually test with Go 1.19 (as the release notes state).

When the latest minor release was published, Go 1.19 wasn't officially released, thus the metrics are not the latest file and the CI hasn't added Go 1.19 stage. We will add the support with the upcoming release.

Copy link
Member

@kakkoyun kakkoyun 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 for adding these regression tests. I really like them.
For completeness' sake, could you add a test for 1.18?

And I will go ahead and land support for Go 1.19 in a separate PR. Could rebase this PR after it's merged?

@kakkoyun
Copy link
Member

kakkoyun commented Nov 7, 2022

ref: #1160

@kakkoyun kakkoyun merged commit b785d0c into prometheus:main Nov 7, 2022
@kakkoyun
Copy link
Member

kakkoyun commented Nov 7, 2022

I decided to merge this and base the other PR on these changes.

@rogerogers rogerogers deleted the feature/test branch November 10, 2022 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants