refactor sentry section #145

egabancho · 2024-12-13T10:24:31Z

invenio: refactor sentry block

* Deprecates non-camelcase sentry variables.

* Addresses secret handling as described in #117.

charts/invenio/templates/NOTES.txt

charts/invenio/templates/sentry-secret.yaml

charts/invenio/values.yaml

lindhe

Since I don't use Sentry, I cannot test these changes. But as far as I can tell, it looks good and as long as you test it before merging you have my approval. 👍

egabancho · 2024-12-16T13:52:37Z

I'm testing all the changes on our test instance. I will have to spin up a new one from scratch after the holiday break and test it there as well 😄

* Deprecates non-camelcase sentry variables. * Addresses secret handling as described in inveniosoftware#117.

egabancho force-pushed the sentry branch 2 times, most recently from a11c33a to 930141c Compare December 13, 2024 14:31

egabancho changed the title ~~invenio: refactor sentry block~~ invenio: refactor Dec 13, 2024

egabancho changed the title ~~invenio: refactor~~ invenio section refactor Dec 13, 2024

egabancho marked this pull request as draft December 15, 2024 14:30

egabancho force-pushed the sentry branch 2 times, most recently from 72ebfc6 to bf38b3f Compare December 16, 2024 10:40

egabancho changed the title ~~invenio section refactor~~ refactor sentry section Dec 16, 2024

egabancho force-pushed the sentry branch 2 times, most recently from aefb719 to 95af940 Compare December 16, 2024 10:44

egabancho marked this pull request as ready for review December 16, 2024 10:45

egabancho requested a review from lindhe December 16, 2024 10:45