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

Extra volume mounts #49

Merged
merged 5 commits into from
Oct 7, 2019
Merged

Extra volume mounts #49

merged 5 commits into from
Oct 7, 2019

Conversation

pb82
Copy link
Collaborator

@pb82 pb82 commented Sep 15, 2019

Allow mounting extra config files and make them available as volume mounts to all containers in the Grafana pod. This allows configuring e.g. LDAP.

Fixes #47

Example documentation for LDAP is provided here: https://github.com/pb82/grafana-operator/blob/79f2ef0a8d2ef29e650e9126f505f579ef560741/documentation/extra_files.md

Can be verified using the latest tag.

@@ -94,7 +94,9 @@ func (r *ReconcileGrafanaDashboard) Reconcile(request reconcile.Request) (reconc
// Return and don't requeue
return reconcile.Result{}, nil
}
// Error reading the object - requeue the request.
// Error reading the object - requeue the r

Choose a reason for hiding this comment

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

Seems like formatting messed up this comment.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

good spot, updated

@pb82 pb82 merged commit a251bb1 into grafana:master Oct 7, 2019
@pb82 pb82 deleted the extra-volume-mounts branch October 7, 2019 11:04
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.

ldap.toml currently not created
2 participants