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

feat: add podLables to Sentry hooks #659

Conversation

nishaero
Copy link
Contributor

@nishaero nishaero commented Jul 6, 2022

The init jobs do not have unique podLables
The lables can be used for mapping these init jobs with services
Use case: To allow access to services inside
Consul mesh, a service is needed.

The init jobs do not have unique podLables
The lables can be used for mapping these init jobs with services
Use case: To allow access to services inside
Consul mesh, a service is needed.
@nishaero nishaero changed the title feat: add podLables to sentry Hooks feat: add podLables to Sentry hooks Jul 6, 2022
@@ -456,6 +459,8 @@ hooks:
affinity: {}
nodeSelector: {}
# tolerations: []
snubaMigrate:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't you need to use snubaMigrate: {}?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't you need to use snubaMigrate: {}?

Yes, I have added it. Thank You :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have bumped the chartversion to 15.0.1 this should fix the lint issue

@Mokto Mokto merged commit 50b8f1e into sentry-kubernetes:develop Jul 27, 2022
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
Development

Successfully merging this pull request may close these issues.

3 participants