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

[grafana] feat: add shareProcessNamespace option to restart Grafana on LDAP config changes #3569

Merged
merged 2 commits into from
Feb 14, 2025

Conversation

jiayuchen888
Copy link
Contributor

This parameter is added because the ldap reload api is not working https://grafana.com/docs/grafana/latest/developers/http_api/admin/#reload-ldap-configuration

To allow an extraContainer to restart the Grafana container

@CLAassistant
Copy link

CLAassistant commented Feb 13, 2025

CLA assistant check
All committers have signed the CLA.

@jkroepke
Copy link
Collaborator

jkroepke commented Feb 13, 2025

  • please bump the chart version
  • please sign the CLA and sign-off your commits to accept the DCO.

However, since there is already a merge commit on this PR, do a force push will mess-up the PR.

Unless your are a git pro, I fully recommend to start from scratch here.

Verified

This commit was signed with the committer’s verified signature.
jiayuchen888 Jiayu CHEN
…fig changes

Signed-off-by: jiaychen <jiayu.chen@camptocamp.com>
@jkroepke jkroepke changed the title feat: add shareProcessNamespace option to restart Grafana on LDAP config changes [grafana] feat: add shareProcessNamespace option to restart Grafana on LDAP config changes Feb 14, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Signed-off-by: MH <zanhsieh@gmail.com>
@zanhsieh zanhsieh merged commit 2bb91de into grafana:main Feb 14, 2025
6 checks passed
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.

None yet

4 participants