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

jewel: build/ops: enable build of ceph-resource-agents package on rpm-based os #13606

Merged
merged 3 commits into from Apr 20, 2017

Conversation

Projects
None yet
2 participants
@smithfarm
Contributor

smithfarm commented Feb 23, 2017

rpm: build ceph-resource-agents by default
To align with debian build

Fixes: http://tracker.ceph.com/issues/17613
Signed-off-by: Nathan Cutler <ncutler@suse.com>
(cherry picked from commit 3e157bf)

@smithfarm smithfarm self-assigned this Feb 23, 2017

@smithfarm smithfarm added this to the jewel milestone Feb 23, 2017

@smithfarm smithfarm added bug fix core build/ops and removed core labels Feb 23, 2017

@smithfarm smithfarm changed the title from jewel: Build ceph-resource-agents package for rpm based os to jewel: build/ops: enable build of ceph-resource-agents package on rpm-based os Apr 6, 2017

smithfarm added some commits Apr 7, 2017

build/ops: rpm: explicitly provide --with-ocf to configure
Fixes: http://tracker.ceph.com/issues/19546
Signed-off-by: Nathan Cutler <ncutler@suse.com>

(Note: This cannot be cherry-picked because master uses cmake, but
the fix does bring the jewel spec file into better alignment its master
counterpart, at least as far as this one little bit is concerned.)
build/ops: rpm: move $CEPH_EXTRA_CONFIGURE_ARGS to right place
Signed-off-by: Nathan Cutler <ncutler@suse.com>

@smithfarm smithfarm requested a review from ktdreyer Apr 7, 2017

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Apr 7, 2017

Contributor

@ktdreyer This PR was breaking the CentOS build - see http://tracker.ceph.com/issues/19546 - so I added a commit to fix it, and then I couldn't resist adding another cleanup-style commit as well.

Contributor

smithfarm commented Apr 7, 2017

@ktdreyer This PR was breaking the CentOS build - see http://tracker.ceph.com/issues/19546 - so I added a commit to fix it, and then I couldn't resist adding another cleanup-style commit as well.

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Apr 9, 2017

Contributor

Passed an upgrade/hammer-x suite at http://tracker.ceph.com/issues/19538#note-7

Contributor

smithfarm commented Apr 9, 2017

Passed an upgrade/hammer-x suite at http://tracker.ceph.com/issues/19538#note-7

@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Apr 20, 2017

Contributor

This pased an upgrade/hammer-x suite at http://tracker.ceph.com/issues/19538#note-35 and an upgrade/client-upgrade suite at http://tracker.ceph.com/issues/19538#note-36

Contributor

smithfarm commented Apr 20, 2017

This pased an upgrade/hammer-x suite at http://tracker.ceph.com/issues/19538#note-35 and an upgrade/client-upgrade suite at http://tracker.ceph.com/issues/19538#note-36

@smithfarm smithfarm merged commit 7008c64 into ceph:jewel Apr 20, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@smithfarm smithfarm deleted the smithfarm:wip-19062-jewel branch Apr 20, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment