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

Upgrade helmchart for signoz #46

Merged
merged 7 commits into from
Nov 6, 2024

Conversation

BryanFauble
Copy link
Contributor

@BryanFauble BryanFauble commented Nov 6, 2024

Problem:

  1. SigNoz had a few version updates which include bug fixes/features.

Solution:

  1. Upgrade the helm chart version
  2. Port over the values file into the new values file.

Testing:

  1. This was deployed to the dpe sandbox cluster to verify deployment

Upgrade process:

  1. Update local helm repo helm repo update (See https://signoz.io/docs/install/kubernetes/others/ for setting up helm to begin with)
  2. Show new latest version of signoz/signoz chart: helm search repo signoz
  3. Create new values.yaml file that is used in the new chart version helm show values signoz/signoz > values-new.yaml
  4. Perform a diff of the existing values.yaml file and the values-new.yaml
  5. Port over any configuration that is needed, and adjust anything that is new
  6. Commit/deploy the changes through the regular CI/CD process (ie: Pull request, spacelift deployment, argocd deployment)

@spacelift-int-sagebionetworks spacelift-int-sagebionetworks bot temporarily deployed to spacelift/dpe-dev-kubernetes-infrastructure November 6, 2024 18:29 Inactive
@BryanFauble BryanFauble marked this pull request as ready for review November 6, 2024 18:55
@BryanFauble BryanFauble requested a review from a team as a code owner November 6, 2024 18:55
modules/signoz/templates/values.yaml Show resolved Hide resolved
@BryanFauble BryanFauble merged commit fc53860 into signoz-testing Nov 6, 2024
3 checks passed
@BryanFauble BryanFauble deleted the upgrade-signoz-helm-chart branch November 6, 2024 19:02
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