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

Helm Charts Clean-Up #3822

Closed
3 tasks done
Pothulapati opened this issue Dec 12, 2019 · 2 comments
Closed
3 tasks done

Helm Charts Clean-Up #3822

Pothulapati opened this issue Dec 12, 2019 · 2 comments

Comments

@Pothulapati
Copy link
Contributor

Pothulapati commented Dec 12, 2019

As the plan is to use helm charts as the core-parts for Linkerd Add-Ons per #3794. It was also planned to make changes to follow best-practices as a pre-requisite.

The following tasks have been taken up for the same:

  • Make linkerd2 chart use smaller-case values, support the same for cli and inject`
  • Make linkerd2-cni use smaller-case
  • Move common values out to global (in preparation for Linkerd Add-Ons #3794 )

@ihcsim @grampelberg

@christyjacob4
Copy link
Contributor

I think this can be closed in light of the merged PRs

@Pothulapati
Copy link
Contributor Author

Yes, Thanks!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants