Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

packaging: move parted requirement to -osd subpkg #9415

Merged
merged 1 commit into from Jun 1, 2016
Merged

Conversation

ktdreyer
Copy link
Member

Prior to this change, ceph-base required the "parted" package, which meant that any installation of ceph-osd, ceph-mon, or ceph-mds would pull in the parted package.

Move the parted dependency to ceph-osd, since ceph-disk is the only thing that uses parted.

The advantage of this change is that Mon and MDS systems will not need to install the parted package.

Fixes: http://tracker.ceph.com/issues/16095

Signed-off-by: Ken Dreyer kdreyer@redhat.com

Prior to this change, ceph-base required the "parted" package, which
meant that any installation of ceph-osd, ceph-mon, or ceph-mds would
pull in the parted package.

Move the parted dependency to ceph-osd, since ceph-disk is the only
thing that uses parted.

The advantage of this change is that Mon and MDS systems will not need
to install the parted package.

Fixes: http://tracker.ceph.com/issues/16095

Signed-off-by: Ken Dreyer <kdreyer@redhat.com>
@ktdreyer ktdreyer modified the milestone: jewel May 31, 2016
@ktdreyer
Copy link
Member Author

CC @BRANTO1 @smithfarm in case they want to review.

@smithfarm
Copy link
Contributor

Reviewed-by: Nathan Cutler <ncutler@suse.com>

@b-ranto
Copy link
Contributor

b-ranto commented Jun 1, 2016

A quick grep on sources reveals that parted is only used in ceph-disk and qa workunit that tests ceph-disk + the gitbuilders look fine -> merging.

@b-ranto b-ranto merged commit 7b93841 into master Jun 1, 2016
@b-ranto b-ranto deleted the wip-16095-parted branch June 1, 2016 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants