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: allow service monitor without authentication or with auth data from existing secret #181

Conversation

maxnitze
Copy link
Contributor

@maxnitze maxnitze commented Aug 8, 2024

Allow to disable authentication of the service monitor by settings .Values.serviceAccount.authentication.enabled=false. Also allow to provide an existing secret for the service monitor authentication.

@maxnitze maxnitze force-pushed the feat/allow-service-monitor-without-or-with-custom-authentication branch 2 times, most recently from 06ec3fe to 4f06e99 Compare August 8, 2024 14:43
@maxnitze maxnitze force-pushed the feat/allow-service-monitor-without-or-with-custom-authentication branch from 4f06e99 to cf9da31 Compare August 8, 2024 15:13
@k3rnelpan1c-dev k3rnelpan1c-dev added documentation Improvements or additions to documentation enhancement New feature or request labels Sep 2, 2024
@k3rnelpan1c-dev
Copy link
Owner

Hey, sorry for the late response, been rather busy IRL lately and haven't had the time nor motivation to really give this project the attention it deserves.
I will see to review both of your PRs tomorrow.

Copy link
Owner

@k3rnelpan1c-dev k3rnelpan1c-dev left a comment

Choose a reason for hiding this comment

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

Besides that one comment that I would like a response for, your changes look good to me. Thank you for this contribution and your patience!

@maxnitze
Copy link
Contributor Author

Hey! I answered your comment. I can switch it, of course. Makes more sense to me as well. Your call :)

@k3rnelpan1c-dev k3rnelpan1c-dev merged commit 6169848 into k3rnelpan1c-dev:main Jan 1, 2025
@k3rnelpan1c-dev
Copy link
Owner

Thank you for your contribution!

@maxnitze maxnitze deleted the feat/allow-service-monitor-without-or-with-custom-authentication branch January 2, 2025 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants