Project

General

Profile

Cleanup #16808

Merge "ceph-fs-common" into "ceph-common"

Added by Nathan Cutler almost 4 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
-
% Done:

0%

Tags:
Backport:
Reviewed:
Affected Versions:
Component(FS):
Labels (FS):
Pull request ID:

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.


Related issues

Related to fs - Cleanup #16035: Remove "cephfs" CLI Resolved 05/26/2016

History

#1 Updated by Nathan Cutler almost 4 years ago

#2 Updated by Nathan Cutler almost 4 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)

#3 Updated by Nathan Cutler almost 4 years ago

@John: Would it make sense to backport #16035 and this one to jewel?

#4 Updated by Nathan Cutler almost 4 years ago

  • Assignee set to Nathan Cutler

#5 Updated by John Spray almost 4 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.

#6 Updated by John Spray almost 4 years ago

Oh, and for 16035 we definitely shouldn't backport, we should only remove things in new versions.

#7 Updated by Kefu Chai almost 4 years ago

  • Status changed from Fix Under Review to Resolved

#8 Updated by Nathan Cutler almost 4 years ago

  • Status changed from Resolved to 7

#9 Updated by Nathan Cutler almost 4 years ago

  • Status changed from 7 to Resolved

Also available in: Atom PDF