-
Notifications
You must be signed in to change notification settings - Fork 25
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
Move Grafana setting files to Persistent Storage #84
Conversation
Codecov Report
@@ Coverage Diff @@
## main #84 +/- ##
=======================================
Coverage ? 38.17%
=======================================
Files ? 13
Lines ? 2174
Branches ? 0
=======================================
Hits ? 830
Misses ? 1322
Partials ? 22
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
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.
Thanks Yanjun. Some comments on the documentation.
3f23c73
to
da1f9ba
Compare
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
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.
Minor comments
docs/network-flow-visibility.md
Outdated
Deployment restart. Other changes in settings like passwords and preferences will | ||
also be kept. | ||
|
||
In Theia 0.1, the dashboards will be lost after restarting the Grafana Deployment. |
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.
In Theia 0.1, the dashboards will be lost after restarting the Grafana Deployment. | |
In Theia 0.1, the changes to dashboards and settings will be lost after restarting the Grafana Deployment. |
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.
Updated. Thanks!
da1f9ba
to
4f8401e
Compare
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!
857aec0
to
634f01b
Compare
/theia-test-e2e |
634f01b
to
4c657fb
Compare
Signed-off-by: Yanjun Zhou <zhouya@vmware.com>
4c657fb
to
883666b
Compare
Previously the Grafana settings will be lost if pod is restarted.
This PR modifies the Grafana volumes to keep the settings persistent.
The changes are as follows.
/data
from PersistentVolume as it is not used/var/lib/grafana
to PersistentVolume to make the settingspersistent
/var/lib/grafana/dashboards
to/opt/grafana/dashboards
as K8s does not allow loading directoryto PV through configmap
Signed-off-by: Yanjun Zhou zhouya@vmware.com