Merge ~sergiodj/ubuntu-docker-images/+git/grafana:8.1-21.10 into ~ubuntu-docker-images/ubuntu-docker-images/+git/grafana:7.4-21.04
Proposed by
Sergio Durigan Junior
Status: | Merged |
---|---|
Merge reported by: | Sergio Durigan Junior |
Merged at revision: | 1a1e4ffaf026f16c9e1a57b62fd92e9b2cb96d45 |
Proposed branch: | ~sergiodj/ubuntu-docker-images/+git/grafana:8.1-21.10 |
Merge into: | ~ubuntu-docker-images/ubuntu-docker-images/+git/grafana:7.4-21.04 |
Diff against target: |
170 lines (+43/-42) 4 files modified
Dockerfile (+36/-35) HACKING.md (+2/-2) README.md (+3/-3) data/grafana.yaml (+2/-2) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Athos Ribeiro | Approve | ||
Bryce Harrington | Pending | ||
Canonical Server | Pending | ||
Review via email:
|
Description of the change
This MP updates the Dockerfile to 21.10. I took this opportunity to also bring the Dockerfile up to our current standards.
This has been built locally and tested using our unittest. The MP is targeting the 21.04 branch to facilitate the review process, but a new 21.10 branch will be created from it.
To post a comment you must log in.
Hi Sergio,
The changes LGTM.
I did build and run the image, and can confirm Grafana starts and is available in port 3000.
I did see one odd thing during startup though: the logs show the following error:
EROR[08- 26|12:33: 37] Failed to read plugin provisioning files from directory logger= provisioning. plugins path=/etc/ grafana/ provisioning/ plugins error="open /etc/grafana/ provisioning/ plugins: no such file or directory"
While I understand this could (should?) be set as a volume by users in case they want to load their custom configurations, should we create that directory for a clean startup otherwise?
Also, see https:/ /github. com/grafana/ grafana/ pull/29570