Bug 1428638 - Cannot install on i686 or ppc64 since criu dep added
Summary: Cannot install on i686 or ppc64 since criu dep added
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: docker
Version: 26
Hardware: i686
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Antonio Murdaca
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard: AcceptedFreezeException
Depends On:
Blocks: F26AlphaFreezeException
TreeView+ depends on / blocked
 
Reported: 2017-03-03 01:34 UTC by Adam Williamson
Modified: 2017-03-30 19:24 UTC (History)
14 users (show)

Fixed In Version: docker-1.13.1-5.git5be1549.fc26
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-03-30 19:24:39 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Adam Williamson 2017-03-03 01:34:37 UTC
criu is currently built only for aarch64, armv7hl, ppc64le and x86_64. It is not built for i686 or ppc64. However, docker recently grew a dep on criu, meaning docker now cannot be installed on i686 or ppc64.

This is currently blocking most image builds for i686, as the installer environment pulls in the docker-anaconda-addon package, which depends on docker. We could skip that package on the affected arches if necessary, I think, but I figured it'd be a good first step to see if the criu dependency is really necessary on arches where criu is not available.

Comment 1 Antonio Murdaca 2017-03-03 14:54:31 UTC
is this enough?

```
diff --git a/docker.spec b/docker.spec
index b4d4b20..5191a0a 100644
--- a/docker.spec
+++ b/docker.spec
@@ -144,7 +144,9 @@ Requires: device-mapper-libs >= 1.02.90-1

 Requires: skopeo-containers
 Requires: gnupg
+%ifnarch i686 ppc64
 Requires: criu
+%endif

 # BZ#1399098
 Requires: python-rhsm-certificates
```

Comment 2 Adam Williamson 2017-03-03 15:04:06 UTC
Well, probably. But if you can do that, why is it 'Requires:' on other arches? What's the actual reason for the dependency?

Comment 3 Antonio Murdaca 2017-03-03 15:09:18 UTC
(In reply to Adam Williamson from comment #2)
> Well, probably. But if you can do that, why is it 'Requires:' on other
> arches? What's the actual reason for the dependency?

criu is needed for containers checkpoint & restore via "docker checkpoint" and "docker restore". If criu isn't available on those arches I believe it's save to skip the Requires as the commands will nicely tell you criu isn't installed.

Comment 4 Daniel Walsh 2017-03-03 15:18:20 UTC
Which means we should change this to an optional requires correct?

Comment 5 Antonio Murdaca 2017-03-03 15:22:37 UTC
isn't https://bugzilla.redhat.com/show_bug.cgi?id=1428638#c1 sufficient?

Comment 6 Adam Williamson 2017-03-03 15:40:18 UTC
It's sufficient to fix this problem, but it's worth considering whether the Requires: is correct in the first place, or if it would be better off as a Recommends: . If it was changed to a Recommends: , I believe that would also solve this problem.

Comment 7 Antonio Murdaca 2017-03-03 23:12:47 UTC
Given checkpoint&restore is an experimental docker feature, and we don't enable experimental in Fedora at all, I'm moving the Requires: to a Recommends: now. Dan, wdyt?

Comment 8 Daniel Walsh 2017-03-06 13:06:44 UTC
Yes makes sense.  Probably don't need it at all if the version of docker we ship does not cupport CRIU.

Comment 9 Antonio Murdaca 2017-03-20 09:18:55 UTC
changed to a Recommends in https://koji.fedoraproject.org/koji/taskinfo?taskID=18486056

let me know if that works out well

Comment 10 Adam Williamson 2017-03-20 15:30:52 UTC
Can you please also fix this for F26?

Comment 11 Dennis Gilmore 2017-03-20 16:06:14 UTC
proposing as a freeze exception for Alpha as it prevents us making all 32 bit x86 and ppc64 deliverables

Comment 12 Geoffrey Marr 2017-03-27 17:11:40 UTC
Discussed during the 2017-03-27 blocker review meeting: [1]

The decision was made to classify this bug as an AcceptedFreezeException was made as it would be nice to have this fixed in Alpha release.

[1] https://meetbot.fedoraproject.org/fedora-blocker-review/2017-03-27/f26-blocker-review.2017-03-27-16.01.txt

Comment 13 Fedora Update System 2017-03-28 17:53:39 UTC
docker-1.13.1-5.git5be1549.fc26 has been pushed to the Fedora 26 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-2017-6e04c798f1

Comment 14 Fedora Update System 2017-03-30 19:24:39 UTC
docker-1.13.1-5.git5be1549.fc26 has been pushed to the Fedora 26 stable repository. If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.