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

[5.6.0] Sentry relay crashing with default values.yaml #202

Closed
dnlsndr opened this issue Oct 11, 2020 · 0 comments · Fixed by #203, #1216 or #1217
Closed

[5.6.0] Sentry relay crashing with default values.yaml #202

dnlsndr opened this issue Oct 11, 2020 · 0 comments · Fixed by #203, #1216 or #1217

Comments

@dnlsndr
Copy link
Contributor

dnlsndr commented Oct 11, 2020

When trying to install the sentry chart, the relay pod always crashes. The default values have not been touched except for these fields:

user.email: "<hidden-email>"
user.password "<hidden-password>"

mail.backend: smtp
mail.useTls: true
mail.username: "<hidden>"
mail.password: "<hidden>"
mail.port: 465
mail.host: smtp.gmail.com
mail.from: "<hidden>"

slack.clientId: <hidden>
slack.clientSecret: <hidden>
slack.verificationToken: <hidden>

I can confirm that these values have worked before with the 5.2.0 version of the chart.
When trying to get some log output from the relay pod, the k8s server only responds with a BadRequest - even with the
--previous flag

When looking at the pod through kubectl describe I also can't find any hints as to what's going on. The pod just keeps crashing out of nowhere.

Can somebody confirm this behaviour, since I'm not all to sure that it's not to do with something in my cluster configuration?
Note: The Chart is installed via a wrapper Helm Chart in a sentry namespace.

Mokto pushed a commit that referenced this issue Oct 11, 2020
Due to trailing minuses in the conditionals, the yaml is geneated in an erroneous way such that there are no newlines before and after the relay.mode yaml key.
This leads to invalid yaml syntax causing the relay pod to crash.

fixes: #202
Signed-off-by: Daniel Schneider <daniel.schneider@eramux.com>
This was referenced Apr 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant