https://bugzilla.redhat.com/show_bug.cgi?id=1822652

Nathan Scott nathans@redhat.com changed:

What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+

--- Comment #1 from Nathan Scott nathans@redhat.com --- Minor things to double check below, all similar to previous review - Ack'd.

[x] Naming [x] matches main service it delivers [x] follows general fedora naming guidelines

[ ] Dockerfile Creation [x] FROM instruction is first line [?] FROM instruction is a fully-qualifies fedora registry/image:tag name eg FROM registry.fedoraproject.org/fedora:latest

(container using rawhide - should it use latest?)

[x] required labels present: [x] com.redhat.component [x] name [x] version (usually 0) [?] architecture (usually 86_64)

(double check - missing? needed? may be fine without)

[x] maintainer

(should be an external list I guess - pcp@groups.io? pcp-masters?)

[x] run or usage [x] summary [x] optional labels present: [-] url [-] help

(double check - missing? wanted? fine without)

[x] labels defined in a single line [x] CMD or ENTRYPOINT present [x] Volumes used for persistent data [x] Volumes used for application data [x] Volumes listed in the help file [x] example run command contains volume with a persistent name [x] Volumes defined as narrowly as possible [x] Volumes in help file have full path of the volume [x] Volumes in help file show why its marked as a volume [x] Volumes in help file should describe space, permissions and performance needs [-] readme may contain suggested additional volumes that aren’t made mandatory

[x] Image Contents [x] contains no net new code (ie must use rpms) [x] each container image should provide only one "service"

[x] Help File Creation [x] must have a help.1 or README.md [?] must also be COPYed into the container, to live in the base directory

(check - is this missing? or handled via the 'COPY root /' line?)

[x] help file should contain [x] brief description of what service/software the image contains [x] the purpose it fulfills in a larger infrastructure [x] contain directions on how to configure the contained service [x] detail dependencies on any dependent services [x] detail what each of the VOLUMES is for, if any [x] explanation of each PORT the image listens on (protocol, purpose) [x] links to any external documentation if any exist [x] any special requirements of the container must be listed [x] information about major variants on how it can be built

https://bugzilla.redhat.com/show_bug.cgi?id=1822652

Andreas Gerstmayr agerstmayr@redhat.com changed:

What |Removed |Added ---------------------------------------------------------------------------- Summary|Container Review Request - |Container Review Request: |Grafana - metrics dashboard |grafana - metrics dashboard |and graph editor |and graph editor

https://bugzilla.redhat.com/show_bug.cgi?id=1822652

Andreas Gerstmayr agerstmayr@redhat.com changed:

What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution|--- |NEXTRELEASE Last Closed| |2020-04-15 18:47:51