Cleanup #16808
closedMerge "ceph-fs-common" into "ceph-common"
0%
Description
After the merge of https://github.com/ceph/ceph/pull/10243 the ceph-fs-common package (which exists only in Debian) contains only two files:
usr/sbin/mount.ceph sbin usr/share/man/man8/mount.ceph.8
On the RPM side, mount.ceph is in the "ceph-common" package.
So, we need a PR that moves mount.ceph to ceph-common and drops the ceph-fs-common package.
Updated by Nathan Cutler almost 8 years ago
- Related to Cleanup #16035: Remove "cephfs" CLI added
Updated by Nathan Cutler almost 8 years ago
- Status changed from New to Fix Under Review
master PRs:
https://github.com/ceph/ceph/pull/10433 (ceph)
https://github.com/ceph/ceph-qa-suite/pull/1098 (ceph-qa-suite)
(checked the packages.yaml in teuthology - it is OK)
Updated by Nathan Cutler almost 8 years ago
@John: Would it make sense to backport #16035 and this one to jewel?
Updated by John Spray almost 8 years ago
I don't feel strongly. My intuition would be that we should avoid changes in jewel that might confuse anyone, unless this is actively causing pain.
Updated by John Spray almost 8 years ago
Oh, and for 16035 we definitely shouldn't backport, we should only remove things in new versions.
Updated by Kefu Chai almost 8 years ago
- Status changed from Fix Under Review to Resolved
Updated by Nathan Cutler almost 8 years ago
- Status changed from Resolved to 7
https://github.com/ceph/ceph-qa-suite/pull/1098 is still open