Bug 1598522

Summary: rook RPM pulls ceph-mon, ceph-mgr, ceph-osd into RHEL Tools repository
Product: [Red Hat Storage] Red Hat Ceph Storage Reporter: Pratik Surve <prsurve>
Component: BuildAssignee: seb
Status: CLOSED ERRATA QA Contact: Pratik Surve <prsurve>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 3.1CC: hnallurv, kdreyer, seb, tserlin
Target Milestone: rc   
Target Release: 3.1   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: ceph-3.1-rhel-7-containers-candidate-64277-20180806190637 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-09-26 18:22:08 UTC Type: Bug
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: 1579472    
Attachments:
Description Flags
Revert "DAEMON_PACKAGES: Adding rook package" none

Comment 3 Ken Dreyer (Red Hat) 2018-07-05 19:12:45 UTC 
The rook.spec package has "Requires: ceph-mon", "Requires: ceph-osd", "Requires:      ceph-mgr".

Because rook depends on all these packages, Pungi pulls them into the Tools repository. We need to keep ceph-mon/mgr and ceph-osd out of the Tools repository.

Erwan, would you please build a rook package with those Requires lines removed?
Comment 4 Ken Dreyer (Red Hat) 2018-07-12 22:50:25 UTC 
I plan to remove rook from RHCS 3 until we can resolve this.
Comment 5 Ken Dreyer (Red Hat) 2018-07-13 20:48:43 UTC 
Created attachment 1458786 [details]
Revert "DAEMON_PACKAGES: Adding rook package"

Dropping rook from the product involves dropping rook from the RHCS container as well, and I'm attaching a patch to do that.

Erwan, would you please resolve the Requires lines in rook.spec so we don't have to drop rook?
Comment 6 Ken Dreyer (Red Hat) 2018-07-16 17:51:18 UTC 
"Revert 'DAEMON_PACKAGES: Adding rook package'" patch at https://github.com/ceph/ceph-container/pull/1113
Comment 13 errata-xmlrpc 2018-09-26 18:22:08 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2018:2819
Comment 14 Ken Dreyer (Red Hat) 2019-01-31 00:09:57 UTC 
Note for posterity:

rook.git's "images/ceph/Dockerfile.base" was the file that ran "yum install ceph-mon ceph-osd ceph-mds" etc. That led the original packager to add these RPM Requires: lines to rook.spec.

The Rook upstream project now bases their rook container image directly on the main Ceph image: https://github.com/rook/rook/pull/1714 . This means that all the Ceph daemon binaries will be available in the rook image that way.