Spec URL: https://pagure.io/moby-engine/raw/master/f/moby-engine.spec SRPM URL: https://pagure.io/moby-engine/blob/master/f/SRPMS/moby-engine-17.03.2-1.gitf5ec1e2.fc27.src.rpm Description: Docker is an open source project to build, ship and run any application as a lightweight container. Docker containers are both hardware-agnostic and platform-agnostic. This means they can run anywhere, from your laptop to the largest EC2 compute instance and everything in between - and they don't require you to use a particular language, framework or packaging system. That makes them great building blocks for deploying and scaling web apps, databases, and backend services without depending on a particular stack or provider. koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=24472934 Fedora Account System Username: lsm5
Source3 in this spec file is containerd. containerd is already packaged for Fedora, and provides the commands containerd, containerd-shim, and ctr. Based on my current understanding of this brave new Moby world, Source3 and relevant sections can be dropped and instead just require containerd.
I came across this today. https://github.com/moby/moby/pull/34226 I think it's worth considering starting this package with those binary names to avoid having to change them later.
Spec URL: https://pagure.io/moby-engine/raw/master/f/moby-engine.spec SRPM URL: https://pagure.io/moby-engine/blob/master/f/SRPMS/moby-engine-17.03.2-2.ce.gitf5ec1e2.fc27.src.rpm koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=25221033 binaries renamed: docker -> moby dockerd -> moby-engine docker-containerd -> moby-containerd docker-containerd-ctr -> moby-containerd-ctr docker-containerd-shim -> moby-containerd-shim docker-init -> moby-init docker-proxy -> moby-proxy docker-runc -> moby-runc RE: container storage, it uses /var/lib/docker which would be the same as that for the docker package. I'd just like to make sure common dir wouldn't cause problems ahead.
I am fine with using the same storage until moby changes the default.
Seems like the rename process has sore o hung
sort of hung
We also want to get some of our patches merged into this package. We should be packaging up projectatomic/moby not moby/moby
RE: binary renames, I'll include docker-* as symlinks to the moby-* binaries.
(In reply to Daniel Walsh from comment #7) > We also want to get some of our patches merged into this package. We should > be packaging up projectatomic/moby not moby/moby Sure, that can be done once we have projectatomic/moby ready with the patches.
SRPM URL: https://pagure.io/moby-engine/blob/master/f/SRPMS/moby-engine-17.03.2-3.ce.gitf5ec1e2.fc27.src.rpm docker-* are symlinks to moby-* in this build. Spec location is the same.
rpmlint *.rpm moby-engine.x86_64: W: incoherent-version-in-changelog 17.03.2-3.ce ['17.03.2-3.ce.gitf5ec1e2.fc26', '17.03.2-3.ce.gitf5ec1e2'] moby-engine.x86_64: E: statically-linked-binary /usr/libexec/moby-engine/moby-init moby-engine.x86_64: W: udev-rule-in-etc /etc/udev/rules.d/80-moby.rules moby-engine.x86_64: W: non-conffile-in-etc /etc/udev/rules.d/80-moby.rules moby-engine.x86_64: E: wrong-script-interpreter /usr/share/bash-completion/completions/moby /usr/bin/env bash moby-engine.x86_64: E: non-executable-script /usr/share/bash-completion/completions/moby 644 /usr/bin/env bash moby-engine.x86_64: W: manual-page-warning /usr/share/man/man1/moby-run.1.gz 1125: warning: macro `You' not defined moby-engine.x86_64: W: no-manual-page-for-binary docker moby-engine.x86_64: W: no-manual-page-for-binary moby-engine moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/armon/go-metrics/metrics.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/armon/go-metrics/sink.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/armon/go-metrics/start.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/armon/go-metrics/statsite.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/google/certificate-transparency/go/asn1/asn1.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/google/certificate-transparency/go/asn1/common.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/google/certificate-transparency/go/asn1/marshal.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/google/certificate-transparency/go/x509/cert_pool.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/google/certificate-transparency/go/x509/pem_decrypt.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/google/certificate-transparency/go/x509/pkcs1.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/google/certificate-transparency/go/x509/pkix/pkix.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/google/certificate-transparency/go/x509/root.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/google/certificate-transparency/go/x509/root_unix.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/google/certificate-transparency/go/x509/verify.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/google/certificate-transparency/go/x509/x509.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/pborman/uuid/dce.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/pborman/uuid/node.go moby-engine-debuginfo.x86_64: E: script-without-shebang /usr/src/debug/moby-f5ec1e2936dcbe7b5001c2b817188b095c700c27/_build/src/github.com/docker/docker/vendor/github.com/pborman/uuid/time.go moby-engine-fish-completion.x86_64: W: summary-not-capitalized C fish completion files for Docker moby-engine-fish-completion.x86_64: W: no-documentation moby-engine-nano.x86_64: W: summary-not-capitalized C nano syntax highlighting files for Moby moby-engine-nano.x86_64: W: no-documentation moby-engine-vim.x86_64: W: summary-not-capitalized C vim syntax highlighting files for Moby moby-engine-vim.x86_64: W: no-documentation moby-engine-zsh-completion.x86_64: W: summary-not-capitalized C zsh completion files for Moby moby-engine-zsh-completion.x86_64: W: no-documentation 7 packages and 0 specfiles checked; 21 errors, 14 warnings.
MUST: A package must own all directories that it creates. If it does not create a directory that it uses, then it should require a package which does create that directory. file /usr/libexec/moby-engine is not owned by any package
incoherent-version-in-changelog is a false positive (Am I missing a difference? The scanner seems confused by .ce suffix.) udev-rule-in-etc - pull request created non-conffile-in-etc - goes away with the udev rule non-executable-script - bash completions are non-executable on purpose manual-page-warning - should probably be fixed upstream no-manual-page-for-binary docker - true, but judging by the content of the man pages, they need bigger changes than just a symlink.
fixed some rpmlint errors. Currently I see: $ rpmlint moby-engine.spec RPMS/x86_64/* SRPMS/moby-engine-17.03.2-4.ce.gitf5ec1e2.fc28.src.rpm moby-engine.x86_64: E: statically-linked-binary /usr/libexec/moby-engine/moby-init moby-engine.x86_64: W: manual-page-warning /usr/share/man/man1/moby-run.1.gz 1125: warning: macro `You' not defined moby-engine.x86_64: W: no-manual-page-for-binary docker moby-engine.x86_64: W: no-manual-page-for-binary moby-engine moby-engine-debuginfo.x86_64: E: useless-provides debuginfo(build-id) moby-engine-debugsource.x86_64: W: no-documentation moby-engine-fish-completion.x86_64: W: no-documentation moby-engine-nano.x86_64: W: summary-not-capitalized C nano syntax highlighting files for Moby moby-engine-nano.x86_64: W: no-documentation moby-engine-vim.x86_64: W: no-documentation moby-engine-zsh-completion.x86_64: W: no-documentation 8 packages and 1 specfiles checked; 2 errors, 9 warnings. Spec URL: https://pagure.io/moby-engine/blob/master/f/moby-engine.spec SRPM URL: https://pagure.io/moby-engine/blob/master/f/SRPMS/moby-engine-17.03.2-4.ce.gitf5ec1e2.fc28.src.rpm
The statically linked binary is needed to run inside a container with unknown content. Other warnings seem quite minor.
Any progress here in the last couple of months? The docker package in F28 is getting long-in-the-tooth.
What features are you looking for? Have you checked out podman?
The --mount option for docker run since "-v" cannot handle paths with : in them.
Sadly this is an option that podman does not have either. Will add it to the todo list.
Can we please have an updated docker package?
Frantisek, can you set the fedora-review+ again? Since, the review was 60 days ago, fedpkg request-repo refuses to work. Thanks,
Since the review has already been done, I'll do it...
(In reply to Neal Gompa from comment #22) > Since the review has already been done, I'll do it... Thanks Neal
(fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/moby-engine
moby-engine-18.06.0-1.ce.git0ffa825.fc28 has been submitted as an update to Fedora 28. https://bodhi.fedoraproject.org/updates/FEDORA-2018-a37148fc00
I noticed the reviewed version didn't work well on Fedora :\ . So, I kept the binary locations and unitfile similar to what docker-ce upstream rpm provides for now. I'll check on using fedora's packaged containerd instead of bundled containerd in future builds. I undid the symlinks to "moby" for now, best to wait until we hear from upstream. Let me know how this works..
moby-engine-18.06.0-1.ce.git0ffa825.fc28 has been pushed to the Fedora 28 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2018-a37148fc00
Will people be able to type "yum install docker" "systemctl enable docker" and "docker run -it fedora bash" with this package?
Will the versioning track the Docker CE version closely so that people understand what this is?
moby-engine-18.06.0-1.ce.git0ffa825.fc28 has been pushed to the Fedora 28 stable repository. If problems still persist, please make note of it in this bug report.