[bitnami/grafana] Ensure bitnami/grafana folder is created with usable permissions

[bpfoster]

Description of the change

Ensures that /bitnami/grafana folder is created with usable permissions. I believe this follows the examples of other bitnami images.

Without this folder being created, adding user init scripts leads to an error on startup:
mkdir: cannot create directory '/bitnami': Permission denied

Benefits

Necessary folders are created with proper permissions so that the image can run as the non-root user.

Possible drawbacks

None

Applicable issues

• fixes [bitnami/grafana] Running grafana image with user init script causes error #52238

Additional information

[github-actions]

This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution.

[bpfoster]

Un-stale

[CeliaGMqrz]

Hi @bpfoster,

Sorry for my late response. Thanks for your contribution!

Maybe we could maintain consistency by using grafana_env_var_value as well?

[bpfoster]

Hi @CeliaGMqrz no problem for the delay!

If we use the grafana_env_var_value function, it would require the variable being prefixed with GF_ (or GRAFANA_CFG_).
This would move it from the "Grafana paths" to the "Grafana configurations" section in the env file.

Happy to make that change if you'd prefer it that way, just want to confirm the change to the env variable as well as I'm not familiar with the reasoning for separate naming conventions.

[github-actions]

This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution.

[bpfoster]

Un-stale

[CeliaGMqrz]

Hi @bpfoster,

Thanks for your patience and taking care of this issue.

In effect, this change would require to set the GF_ prefix and move it to the Grafana configuration section in the env file.

[bpfoster]

@CeliaGMqrz sounds good. Variable is now named GF_VOLUME_DIR and is reference by the grafana_env_var_value function per your recommendation.

[github-actions]

This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution.

[bpfoster]

Un-stale

[CeliaGMqrz]

Hi @bpfoster, thanks for your feedback.

Sorry for the delay after the vacation.

The changes have not been applied correctly. Could you fix the DCO issue?

[bpfoster]

@CeliaGMqrz my apologies for missing the commit signing. Amended the commit and DCO check is now passing.

[github-actions]

This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution.

[bpfoster]

Un-stale

[CeliaGMqrz]

Hi, @bpfoster

Sorry to keep you waiting with this PR.

I have consulted with the team and we are going to merge these changes for a temporary workaround. I will create an internal task to review the persistence logic of this container in accordance with grafana configuration to adapt this feature in a cleaner way.

Thanks for your contribution!