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

[dev-v2.6] Metrics server fix #977

Merged

Conversation

rayandas
Copy link
Contributor

Related: rancher/rancher#39078
Depends on: #967

@rayandas rayandas changed the title [dev v2.6] Metric server fix [dev-v2.6] Metric server fix Sep 22, 2022
@rayandas rayandas changed the title [dev-v2.6] Metric server fix [dev-v2.6] Metrics server fix Sep 22, 2022
@rayandas rayandas force-pushed the metric-server-fix-dev-v2.6 branch from 9671320 to 4733f28 Compare September 27, 2022 09:59
@rayandas rayandas marked this pull request as ready for review September 27, 2022 10:00
@rayandas rayandas force-pushed the metric-server-fix-dev-v2.6 branch from 4733f28 to d7c37b4 Compare September 27, 2022 17:01
@rayandas rayandas force-pushed the metric-server-fix-dev-v2.6 branch 2 times, most recently from 2941aaf to ef62559 Compare October 13, 2022 06:04
@rayandas
Copy link
Contributor Author

@kinarashah I added the changes from #984 to this PR. It should be good to go.

Copy link
Member

@kinarashah kinarashah left a comment

Choose a reason for hiding this comment

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

@rayandas this is updating v1.22.13-rancher1-1 with new metrics-server image, but we need to add a new v1.22.15-rancher1-1 for this - same as v2.7.0 PR. Also, this is why drone is failing right now because an already released k8s version v1.22.13 got updated.

@rayandas rayandas force-pushed the metric-server-fix-dev-v2.6 branch from ef62559 to d70b9dd Compare October 14, 2022 12:13
@rayandas rayandas requested a review from a team October 21, 2022 03:58
@rayandas rayandas merged commit 9e080eb into rancher:dev-v2.6-0922-patches Oct 27, 2022
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.

2 participants