Skip to content

Add serviceaccount for hooks in sentry chart #1704

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

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

PaulFarver
Copy link

Add a serviceaccount for the helm hooks in the sentry chart.

Our clusters do not allow use of the default serviceaccounts, so we would love to get this implemented :)

@PaulFarver PaulFarver force-pushed the sentry-service-account-for-hooks branch from d2144fc to 4fa8988 Compare February 14, 2025 00:03
@Mokto
Copy link
Contributor

Mokto commented Feb 26, 2025

👋 Hi, @PaulFarver,
I detected conflicts against the base branch 🙊
You'll want to sync 🔄 your branch with upstream!

@PaulFarver PaulFarver force-pushed the sentry-service-account-for-hooks branch from 4fa8988 to d9153c7 Compare February 26, 2025 14:18
@PaulFarver
Copy link
Author

Hi @Mokto
thanks for taking a look at this. I've rebased it now :)

@Mokto Mokto removed the conflicts label Mar 3, 2025
@Mokto
Copy link
Contributor

Mokto commented Mar 6, 2025

👋 Hi, @PaulFarver,
I detected conflicts against the base branch 🙊
You'll want to sync 🔄 your branch with upstream!

@Mokto Mokto added the conflicts label Mar 6, 2025
@jmealo jmealo mentioned this pull request Apr 24, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants