Bug 1032670

Should this be just a subpackage of docker?

might be simpler to keep them separate :), but I'm ok with whatever. cc'ing Vincent and Matt.

(In reply to Daniel Walsh from comment #1)
> Should this be just a subpackage of docker?

I don't think so. While it is from the dotcloud team, it is a separate project, subject to its own releases/fixes/etc.

/me starts to review this package

(In reply to Vincent Batts from comment #3)
> (In reply to Daniel Walsh from comment #1)
> > Should this be just a subpackage of docker?
> 
> I don't think so. While it is from the dotcloud team, it is a separate
> project, subject to its own releases/fixes/etc.

+1 This is a separate project. It doesn't even require docker to run. This package should be left as-is.

> /me starts to review this package

Thanks!

Perhaps in the *.service file, it needs some logic to evaluate `firewall-cmd --add-port=${REGISTRY_PORT}/tcp` or similar for the configured port. Like a ExecStartPost and ExecStopPost to --remove-port=${REGISTRY_PORT}/tcp. Though in my trials of getting the docker-registry.service to handle this, the starting and stopping of the service just hang.

Maybe this needs to just be a documentation point for folks running a registry server, then they may want to have a permanent rule, like `firewall-cmd --zone=public --permanent --add-port=5000/tcp` just to be explicit.
What do you think?

The %files should include a line like '%doc LICENSE README.md'

Hi Vincent,

Thanks!

IMHO we should leave opening the firewall to the user, entirely. We don't do this for HTTPD, we shouldn't do this here too. I'll add a wiki page that describes how to use the registry later.

I've added the license file and readme files to the package as request. Forgot about that :)

Spec URL: http://goldmann.fedorapeople.org/package_review/docker-registry/2/docker-registry.spec
SRPM URL: http://goldmann.fedorapeople.org/package_review/docker-registry/2/docker-registry-0.6.0-2.fc20.src.rpm

Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6231055

should we add conditionals for rhel right away, or better after review?

i was chewing on that too.

something like:

%if 0%{?fedora} >= 18 || 0%{?rhel} >= 7
%post
%systemd_post %{name}.service

%preun
%systemd_preun %{name}.service

%postun
%systemd_postun_with_restart %{name}.service
%endif

And elsewhere, it works, but you still have to resolve all of the runtime dependencies (which are not in EPEL). 
Though, since all the runtime dependencies are available and used internally, then it would simplify things to have a single spec that works everywhere.

My POV is that the review request here is about Fedora, Rawhide especially. I have locally some code prepared to add EPEL support, but I'm waiting for approval, so I can work on a stable base.

Please let me know if you see blocking issues / you want to do additional review or please set the fedora-review flag to +, so I can proceed.

I am not blocking on that. I also am not sure on the process of adding el6 dist to the fedora build/publish process later. We'll almost certainly be hoping for a RHEL build eventually.

Otherwise, LGTM.

Thanks!

I'll work on adding EPEL support immediately after I build it for Fedora.

New Package SCM Request
=======================
Package Name: docker-registry
Short Description: Registry server for Docker
Owners: goldmann
Branches: f19 f20 el6

Git done (by process-git-requests).

docker-registry-0.6.0-2.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/docker-registry-0.6.0-2.fc19

docker-registry-0.6.0-2.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/docker-registry-0.6.0-2.fc20

docker-registry-0.6.0-2.fc20 has been pushed to the Fedora 20 testing repository.

docker-registry-0.6.0-3.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/docker-registry-0.6.0-3.fc19

docker-registry-0.6.0-3.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/docker-registry-0.6.0-3.el6

docker-registry-0.6.0-3.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/docker-registry-0.6.0-3.fc20

docker-registry-0.6.0-4.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/docker-registry-0.6.0-4.fc19

docker-registry-0.6.0-4.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/docker-registry-0.6.0-4.fc20

docker-registry-0.6.0-4.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/docker-registry-0.6.0-4.el6

docker-registry-0.6.0-4.fc19 has been pushed to the Fedora 19 stable repository.  If problems still persist, please make note of it in this bug report.

docker-registry-0.6.0-4.fc20 has been pushed to the Fedora 20 stable repository.  If problems still persist, please make note of it in this bug report.

docker-registry-0.6.0-4.el6 has been pushed to the Fedora EPEL 6 stable repository.  If problems still persist, please make note of it in this bug report.

Package Change Request
======================
Package Name: docker-registry
New Branches: epel7
Owners: goldmann

Git done (by process-git-requests).