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

fix(sentry): remove duplicate annotation #3213

Merged
merged 1 commit into from
Jan 5, 2024
Merged

Conversation

themightychris
Copy link
Contributor

@themightychris themightychris commented Jan 5, 2024

The Sentry helm chart's default configuration already adds this annotation, so including it here too results in a duplicate key and invalid YAML

I'm going to merge and deploy this right away as it is part of my work to reconcile the state of the k8s cluster and I need to push the change through the GitHub Action workflow to verify

The Sentry helm chart's default configuration already adds this annotation, so including it here too results in a duplicate key and invalid YAML
Copy link

github-actions bot commented Jan 5, 2024

The following changes will be applied to the production Kubernetes cluster upon merge.

BE AWARE this may not reveal changes that have been manually applied to the cluster getting undone—applying manual changes to the cluster should be avoided.

sentry, sentry-sentry-postgresql, StatefulSet (apps) has changed:
...
          fsGroup: 1001
        hostNetwork: false
        hostIPC: false
        containers:
          - name: postgresql
-           image: docker.io/bitnami/postgresql:15.3.0-debian-11-r0
+           image: docker.io/bitnami/postgresql:11.14.0-debian-10-r28
            imagePullPolicy: "IfNotPresent"
            securityContext:
              runAsUser: 1001
            env:
              - name: BITNAMI_DEBUG
...

@themightychris themightychris merged commit a1f50d0 into main Jan 5, 2024
2 checks passed
@themightychris themightychris deleted the minor-k8s-fixes branch January 5, 2024 14:45
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.

1 participant