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

uber.go/automaxprocs package upgrade for cgroups2 support #10584

Merged
merged 1 commit into from
Apr 13, 2022

Conversation

TomasKohout
Copy link
Contributor

Signed-off-by: Tomas Kohout tomas.kohout1995@gmail.com

Hi,

I'm here only to bump version of automaxprocs to 1.5.1 since it now supports v2 cgroups. 🙂

@TomasKohout
Copy link
Contributor Author

I'll rebase as soon as fix for ui test is in main. 🙂

@roidelapluie
Copy link
Member

@Nexucis this ui tests seem to be flappy. do you know why?

@roidelapluie
Copy link
Member

Could you change base branch to release-2.35? we are cutting a new rc anyway

@TomasKohout TomasKohout changed the base branch from main to release-2.35 April 13, 2022 11:20
@TomasKohout TomasKohout force-pushed the automaxprocs_upgrade branch 2 times, most recently from cda5e74 to db81f01 Compare April 13, 2022 11:23
@roidelapluie
Copy link
Member

Can you cherry pick only your commit?

Signed-off-by: Tomas Kohout <tomas.kohout1995@gmail.com>
@TomasKohout
Copy link
Contributor Author

Done. 👍

@Nexucis
Copy link
Member

Nexucis commented Apr 13, 2022

@Nexucis this ui tests seem to be flappy. do you know why?

Not yet :/. Another weird things with the UI environment. You fixed one thing, another one doesn't work as expected lol.

Copy link
Member

@Nexucis Nexucis left a comment

Choose a reason for hiding this comment

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

thanks for the changes @TomasKohout

@Nexucis Nexucis enabled auto-merge (squash) April 13, 2022 12:49
@Nexucis Nexucis merged commit a1aa372 into prometheus:release-2.35 Apr 13, 2022
roidelapluie pushed a commit to roidelapluie/prometheus that referenced this pull request Jun 22, 2022
…#10584)

Signed-off-by: Tomas Kohout <tomas.kohout1995@gmail.com>
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