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

[bitnami/grafana] don't error out on read-only config dir #77131

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

enkore
Copy link

@enkore enkore commented Feb 10, 2025

Description of the change

Errors when copying the example configuration files to the Grafana config dir are now ignored.

Benefits

The Grafana config dir as a whole or subdirectories may become read-only to the container, e.g. when mounting configmaps or secrets. Currently, these configurations require overwriting the entrypoint, which is undesirable.

Possible drawbacks

@enkore enkore force-pushed the grafana-configmap-support branch from 007675d to 75d67c5 Compare February 10, 2025 07:46
@carrodher carrodher added verify Execute verification workflow for these changes in-progress labels Feb 10, 2025
@github-actions github-actions bot removed the triage Triage is needed label Feb 10, 2025
@github-actions github-actions bot removed the request for review from carrodher February 10, 2025 13:15
@github-actions github-actions bot requested a review from gongomgra February 10, 2025 13:15
@gongomgra
Copy link
Contributor

gongomgra commented Feb 11, 2025

Hi @enkore,

Thanks for your contribution. I'm afraid I'm not fully understanding your changes. Can you provide us with an scenario in which this is required? Wouldn't this error prevent the initialization of Grafana due to missing configuration?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
grafana in-progress verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants