build/ops: RPM: move runtime dependencies to ceph-base and fix other packaging issues #7826
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
smithfarm
force-pushed
the
wip-move-requires
branch
2 times, most recently
from
0ddff54
to
f1e2487
Compare
The ceph package is now a metapackage, so move all its runtime dependencies to ceph-base as we already have done in the Debian packaging. Signed-off-by: Nathan Cutler <ncutler@suse.com>
Flask is used by MON (in the Ceph REST API) and by RGW (powerdns). Therefore, it only needs to be in the ceph-mon and ceph-radosgw packages. Also, this commit encloses the specification of the python-flask runtime dependency in distro-conditional blocks to account for a minor difference in the package name between RHEL and SUSE. Signed-off-by: Nathan Cutler <ncutler@suse.com>
In Jewel, there is no reason to support anything less than openSUSE 13. Signed-off-by: Nathan Cutler <ncutler@suse.com>
ceph-disk uses sgdisk, which resides in the gdisk and gptfdisk packages on RHEL and SUSE derivatives, respectively. ceph-disk is included in the ceph-osd package, so the sgdisk dependency belongs there. Signed-off-by: Nathan Cutler <ncutler@suse.com>
ceph-detect-init uses pkg_resources.py http://tracker.ceph.com/issues/14864 Fixes: ceph#14864 Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm
force-pushed
the
wip-move-requires
branch
from
01e30fb
to
cba211d
Compare
|
Pushed to gitbuilders. |
|
This PR passes the deb+rpm amd64 gitbuilders. |
smithfarm
changed the title
|
However, OBS says: |
|
Now we are getting: |
|
Drop duplicate rados-classes libraries from ceph-osd and align the RPM packaging with Debian by using a wildcard to package everything that autotools puts in the rados-classes directory. Signed-off-by: Nathan Cutler <ncutler@suse.com>
The udev rules should be packaged in ceph-osd only. Signed-off-by: Nathan Cutler <ncutler@suse.com>
As of a recent commit, we had /var/lib/ceph/{mds,mon,osd} directories
packaged twice, once with %attr and once without. Drop the latter.
Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm
force-pushed
the
wip-move-requires
branch
from
6107ea1
to
4de86bf
Compare
|
@ktdreyer How does this look? (Testing now in OBS) |
|
Looks right to me. Thanks. If it passes gitbuilder, we can merge |
|
Pushed latest version to gitbuilders. |
|
@ktdreyer: It passes the RPM gitbuilders. I can't make out whether it passed the deb ones or not - can you? |
|
http://gitbuilder.sepia.ceph.com/gitbuilder-ceph-deb-trusty-amd64-basic/log.cgi?log=4de86bf373e8c3041bbb4fbaf560e4427aed42ab looks good! (it has |
ktdreyer
added a commit
that referenced
this pull request
Feb 29, 2016
RPM: move runtime dependencies to ceph-base and fix other packaging issues Reviewed-by: Ken Dreyer <kdreyer@redhat.com>
ghost
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR aggregates a number of packaging fixes. Hopefully this will ease the load on the gitbuilders.