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

luminous: build/ops: deb: require fuse for ceph-fuse #23693

Merged
merged 1 commit into from Sep 17, 2018

Conversation

Projects
None yet
5 participants
@tserlin
Contributor

tserlin commented Aug 22, 2018

Installing "ceph-fuse" should pull in the "fuse" package automatically.

This is a backport of #23675

Signed-off-by: Thomas Serlin tserlin@redhat.com

@tserlin tserlin requested a review from ktdreyer Aug 22, 2018

@tchaikov tchaikov added this to the luminous milestone Aug 22, 2018

@tserlin

This comment has been minimized.

Contributor

tserlin commented Aug 30, 2018

@tchaikov Do you mind merging this? Thanks.

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Aug 31, 2018

@tserlin i am pushing this change to ceph-ci to make sure it builds.

@tchaikov tchaikov requested a review from smithfarm Aug 31, 2018

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Aug 31, 2018

Note: there is still a full-stop on merging PRs into luminous. This is properly staged and will be tested/merged in the 12.2.9 cycle.

@smithfarm

@tserlin Please re-do the commit using the following command:

git cherry-pick -x b012016cd5679a4f7f757cc850c1cfb51efb0b9e

Thanks!

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Aug 31, 2018

Additional backporting note: git cherry-pick -x will create the commit message for you. Please do not change it, or if you feel the need to add something, then only by appending to the end of the commit message prepared by git. Thanks!

@smithfarm smithfarm removed the backport label Aug 31, 2018

debian/control: require fuse for ceph-fuse
This is the Ubuntu equivalent for:

#17120
http://tracker.ceph.com/issues/21057

Installing "ceph-fuse" should pull in the "fuse" package automatically.

Signed-off-by: Thomas Serlin <tserlin@redhat.com>
(cherry picked from commit b012016)

@tserlin tserlin force-pushed the tserlin:tserlin-require-fuse-luminous branch from f65a2b6 to 6b497c7 Aug 31, 2018

@tserlin

This comment has been minimized.

Contributor

tserlin commented Aug 31, 2018

@smithfarm I re-did the commit.

commit was re-done as requested

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Sep 11, 2018

@tserlin Could you please git cherry-pick -x 6fba9cc3f910442d3d09b41f01f454a39bfdbc48 into this PR as well? It appears to be missing in luminous.

Nevermind, sorry. Now I see that commit was cherry-picked to luminous by 18442e1.

@yuriw

This comment has been minimized.

Contributor

yuriw commented Sep 13, 2018

@yuriw yuriw merged commit 0f827a9 into ceph:luminous Sep 17, 2018

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details

@smithfarm smithfarm changed the title from luminous: debian/control: require fuse for ceph-fuse to luminous: build/ops: deb: require fuse for ceph-fuse Oct 26, 2018

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