-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[promtail] Add automount option for service account and pod #2990
Conversation
Signed-off-by: emilkor1 <emilkordahl@gmail.com>
Signed-off-by: emilkor1 <emilkordahl@gmail.com>
Duplicate PR: #2991 |
@emilkor1 my PR covers grafana only. Not promtail, this is PR is still valid. |
Thanks for letting me know.. was too quick there |
Can someone look at this? @unguiculus @Whyeasy |
For security hardening we also need this. Any reason why this does not get merged? |
Signed-off-by: Jan-Otto Kröpke <mail@jkroepke.de>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@zanhsieh - I'm not an official promtail maintainer, I point this PR to you back. Feel to merge. |
This PR contains the following updates: | Package | Update | Change | |---|---|---| | [promtail](https://grafana.com/loki) ([source](https://github.com/grafana/helm-charts)) | patch | `6.16.4` -> `6.16.5` | --- ### Release Notes <details> <summary>grafana/helm-charts (promtail)</summary> ### [`v6.16.5`](https://github.com/grafana/helm-charts/releases/tag/promtail-6.16.5) [Compare Source](https://github.com/grafana/helm-charts/compare/promtail-6.16.4...promtail-6.16.5) Promtail is an agent which ships the contents of local logs to a Loki instance #### What's Changed - \[promtail] Add automount option for service account and pod by [@​emilkor1](https://github.com/emilkor1) in [https://github.com/grafana/helm-charts/pull/2990](https://github.com/grafana/helm-charts/pull/2990) **Full Changelog**: grafana/helm-charts@grafana-sampling-1.0.1...promtail-6.16.5 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/defenseunicorns/uds-core). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC4yNi4xIiwidXBkYXRlZEluVmVyIjoiMzguNTYuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Micah Nagel <micah.nagel@defenseunicorns.com>
This PR contains the following updates: | Package | Update | Change | |---|---|---| | [promtail](https://grafana.com/loki) ([source](https://github.com/grafana/helm-charts)) | patch | `6.16.4` -> `6.16.5` | --- ### Release Notes <details> <summary>grafana/helm-charts (promtail)</summary> ### [`v6.16.5`](https://github.com/grafana/helm-charts/releases/tag/promtail-6.16.5) [Compare Source](https://github.com/grafana/helm-charts/compare/promtail-6.16.4...promtail-6.16.5) Promtail is an agent which ships the contents of local logs to a Loki instance #### What's Changed - \[promtail] Add automount option for service account and pod by [@​emilkor1](https://github.com/emilkor1) in [https://github.com/grafana/helm-charts/pull/2990](https://github.com/grafana/helm-charts/pull/2990) **Full Changelog**: grafana/helm-charts@grafana-sampling-1.0.1...promtail-6.16.5 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "before 4am on the first day of the month" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40NDAuNyIsInVwZGF0ZWRJblZlciI6IjM3LjQ0MC43IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJhdXRvbWVyZ2UiXX0=-->
Azure recommended actions suggests disabling automount of API credentials.