Bug 1431546
Summary: | Container Review Request: python3 - Platform for building and running Python 3.5 applications | ||
---|---|---|---|
Product: | [Fedora] Fedora Container Images | Reporter: | Honza Horak <hhorak> |
Component: | Container Review | Assignee: | Marek Skalický <mskalick> |
Status: | CLOSED NOTABUG | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | 26 | CC: | container-review, hhorak, mskalick, package-review, panemade, torsava |
Target Milestone: | --- | Flags: | mskalick:
fedora-review+
|
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-08-20 22:42:32 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: | |||
Bug Depends On: | |||
Bug Blocks: | 1436796 |
Description
Honza Horak
2017-03-13 08:19:32 UTC
* Few labels are missing: [1] BZComponent, RUN or Usage, Help * Version label should be set to 0 [2] * help file missing [3] [1] https://fedoraproject.org/wiki/Container:Guidelines#LABELS [2] https://fedoraproject.org/wiki/Container:Guidelines#VERSIONING [3] https://fedoraproject.org/wiki/Container:Guidelines#Help_File Thanks for the review, it's now fixed: ... ENV NAME=python \ VERSION=0 \ RELEASE=1 \ ARCH=x86_64 LABEL BZComponent="$NAME" \ Name="$FGC/$NAME" \ Version="$VERSION" \ Release="$RELEASE.$DISTTAG" \ Architecture="$ARCH" \ Usage="s2i build file:///your/app 25/python your-app" \ Help="help.1" ... Dockerfile URL: https://hhorak.fedorapeople.org/python-docker/Dockerfile Other files URL: https://hhorak.fedorapeople.org/python-docker/ Today I read the guidelines again and found this sentence: "The FROM instruction must be fully-qualified with a registry name, image name, and tag as shown in this example: FROM registry.example.com/imagename:tag " It says that image tag must be there. On the other hand ":latest" is the default in Docker. So I don't know what to thing about this rule. Correct, using :latest as it is also used in the examples: FROM registry.fedoraproject.org/f25/kubernetes-master:latest (https://fedoraproject.org/wiki/Container:Guidelines#FROM) Should be fixed now. I also asked on #fedora-cloud and they told me, that :latest is required. Thanks, for updating it. Actually I've realized we should probably keep the version in the name, similar as it is done for the module names, so we should eventually have python2 and python3 containers. I've changed the subject to reflect this. Tomas, does it make sense to you this way? (In reply to Honza Horak from comment #6) > Actually I've realized we should probably keep the version in the name, > similar as it is done for the module names, so we should eventually have > python2 and python3 containers. I've changed the subject to reflect this. > > Tomas, does it make sense to you this way? Indeed it does! Food for thought: It might even be beneficial to go one step further and name it `python35`. Python minor versions (3.5 v 3.6) are not API/ABI compatible and thus instead of updating the container from python 3.5 to 3.6 it might make more sense to release a new `python36` container. However, the maintenance burden would of course be increased. My apologies, I skimmed the Dockerfile and mistakenly assumed it was packaging the rh-python35 collection due to the openshift tag. Upon closer examination it's packaging the python3 ecosystem straight from fedora. In this case I believe the name `python3` is indeed the best choice for the container. Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/container/python3 |