Bug 1420947

Summary: Container Review Request: nginx - F25 docker container with nginx
Product: [Fedora] Fedora Reporter: Mike Ruckman <mruckman>
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: CLOSED DUPLICATE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: bperkins, hhorak, mruckman, package-review
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-02-22 16:29:08 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Mike Ruckman 2017-02-09 23:16:03 UTC
Dockerfile URL: https://roshi.fedorapeople.org/containers/nginx/Dockerfile
Source URL: https://roshi.fedorapeople.org/containers/nginx/
Description: This builds a container with nginx.
Fedora Account System Username: roshi

Comment 1 Honza Horak 2017-02-15 13:39:34 UTC
Hi Mike,

It's funny coincidence, that I've created a similar container review request for nginx:
https://bugzilla.redhat.com/show_bug.cgi?id=1421605

That one is a bit more complicated, as you see can see from the Dockerfile, but it will work fine with OpenShift, which is IMO very important.

What do you think? Would you be in favor to give way to the BZ#1421605?

Comment 2 Mike Ruckman 2017-02-15 14:21:38 UTC
I saw the comment on the pagure ticket first [0], so I'll just copy my response here:

I'm fine with that provided the BZ#1421605 Dockerfile gets updated to meet the same specifications as far FROM being the official fedora registry and keeping the label conventions in BZ#1420947. Those come from the guidelines on the wiki here: https://fedoraproject.org/wiki/Container:Guidelines

Does that work Honza? Thanks for pointing out the duplicate! I guess I shoulda looked around for those before...

[0] https://pagure.io/atomic-wg/issue/206

Comment 3 Mike Ruckman 2017-02-22 16:29:08 UTC

*** This bug has been marked as a duplicate of bug 1421605 ***