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

build/ops: ceph.spec.in: ceph-common subpackage def needs tweaking for SUSE/openSUSE #4883

Merged
1 commit merged into from Jul 14, 2015

Conversation

smithfarm
Copy link
Contributor

ceph-common needs python-argparse in SUSE/openSUSE and
needs redhat-lsb-core only in RHEL/CentOS/Fedora.

http://tracker.ceph.com/issues/11638 Fixes: ceph#11638

Signed-off-by: Nathan Cutler <ncutler@suse.cz>
(cherry picked from commit 363d957)
@smithfarm smithfarm added this to the hammer milestone Jun 7, 2015
@theanalyst
Copy link
Member

@ktdreyer / @smithfarm this has passed the first run of integration tests for hammer backports tracked at http://tracker.ceph.com/issues/11990 Do you think it is ready to merge

@smithfarm
Copy link
Contributor Author

Let Ken make the final decision on this one. IMO it can be merged provided we also backport #4970.

@ghost
Copy link

ghost commented Jul 10, 2015

@ktdreyer ping ?

@ktdreyer
Copy link
Member

Apologies for the delay. I see now argparse will be covered separately in http://tracker.ceph.com/issues/12269 . So this PR is good to go.

ghost pushed a commit that referenced this pull request Jul 14, 2015
ceph.spec.in: ceph-common subpackage def needs tweaking for SUSE/openSUSE

Reviewed-by: Ken Dreyer <kdreyer@redhat.com>
@ghost ghost merged commit 3d72652 into ceph:hammer Jul 14, 2015
@smithfarm smithfarm deleted the wip-11638-hammer branch July 14, 2015 19:34
@ghost ghost changed the title ceph.spec.in: ceph-common subpackage def needs tweaking for SUSE/openSUSE build/ops: ceph.spec.in: ceph-common subpackage def needs tweaking for SUSE/openSUSE Aug 4, 2015
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants