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

[kube-prometheus-stack] Allow autoTagVersion for hyperkube image #81

Closed
wants to merge 1 commit into from
Closed

[kube-prometheus-stack] Allow autoTagVersion for hyperkube image #81

wants to merge 1 commit into from

Conversation

VLZZZ
Copy link
Contributor

@VLZZZ VLZZZ commented Sep 10, 2020

Valentin Zayash valioozz@gmail.com

What this PR does / why we need it:

This PR will allow to conditionally set image tag for hyperkube depending on installed Kubernetes version using Helm's built-in Capabilities.

https://v2.helm.sh/docs/chart_template_guide/#built-in-objects

Capabilities.KubeVersion provides a way to look up the Kubernetes version. It has the following values: Major, Minor, GitVersion, GitCommit, GitTreeState, BuildDate, GoVersion, Compiler, and Platform.

It will be really useful to avoid manual reconfiguration across clusters with different versions.
It won't affect existing installation until autoTagVersion will be set to true

Special notes for your reviewer:

This PR was previously opened to the stable/chart - #23590

Checklist

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [prometheus-couchdb-exporter])

@gkarthiks
Copy link
Member

gkarthiks commented Sep 10, 2020

@VLZZZ Thanks for you PR, please add your signoff to get DCO check passed. Click on the Details link on the DCO failed run to see how to do that.

@gkarthiks
Copy link
Member

@VLZZZ not sure whether you want to do this, as I vaguely remember that the build of hyperkube is no longer supported in k/k

@gkarthiks
Copy link
Member

got the links for you.

ref:

/cc @scottrigby @Xtigyro @bismarck

/hold

@VLZZZ
Copy link
Contributor Author

VLZZZ commented Sep 18, 2020

@gkarthiks Thank you for providing all of the details!
I'm going to close this PR.

@VLZZZ VLZZZ closed this Sep 18, 2020
@VLZZZ VLZZZ deleted the feat/conditional_hyperkube branch September 18, 2020 13:46
nf-npieros pushed a commit to nf-npieros/helm-charts that referenced this pull request Jan 17, 2022
nf-npieros pushed a commit to nf-npieros/helm-charts that referenced this pull request Jan 17, 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.

None yet

2 participants