Cleanup #16035: Remove "cephfs" CLI - CephFS - Ceph

Actions

Copy link

Cleanup #16035

closed

Remove "cephfs" CLI

Added by John Spray almost 8 years ago. Updated almost 8 years ago.

Status:

Resolved

Priority:

Normal

Assignee:

John Spray

Category:

Administration/Usability

Target version:

% Done:

Tags:

Backport:

Reviewed:

Affected Versions:

Component(FS):

tools

Labels (FS):

Pull request ID:

Description

IIRC we talked about this in standup the other day, and this pull request (https://github.com/ceph/ceph/pull/9338) reminded me: let's remove the old cephfs CLI tool (src/cephfs.cc) that uses ioctls for setting layouts.

The message telling users the tool was deprecated was added in July 2014:

commit af740ecd498e73ae9c5101ae7ba377be6fd8f2eb
Author: John Spray <john.spray@redhat.com>
Date:   Tue Jul 15 12:45:34 2014 +0100

    cephfs: pool safety & print deprecation warning

    Print a message indicating to users that this tool
    is now deprecated in favour of vxattrs.

    Signed-off-by: John Spray <john.spray@redhat.com>

Related issues 2 (0 open — 2 closed)

Actions

Copy link

Updated by John Spray almost 8 years ago

Status changed from New to In Progress
Assignee set to John Spray

Actions

Copy link

Updated by John Spray almost 8 years ago

Has duplicate Bug #15485: drop /usr/bin/cephfs added

Actions

Copy link

Updated by Greg Farnum almost 8 years ago

Category changed from 66 to Administration/Usability
Component(FS) tools added

Actions

Copy link

Updated by Nathan Cutler almost 8 years ago

After the cephfs tool is dropped, mount.ceph will be the only thing remaining in the (deb-only) "ceph-fs-common" package.

The cephfs-data-scan, cephfs-journal-tool, and cephfs-table-tool utilies were placed in ceph-common, maybe we should move mount.ceph there, too, to align the deb packaging with rpm?

Then the ceph-fs-common package could be dropped.

Actions

Copy link

Updated by Ken Dreyer almost 8 years ago

(Agreed with merging "ceph-fs-common" into "ceph-common". I've never found an explanation for why that was its own package.)

Actions

Copy link

Updated by Ken Dreyer almost 8 years ago

For additional info, quoting Sage from an internal RH bug (sorry this is restricted, not sure why. https://bugzilla.redhat.com/1205497)

Sage Weil 2015-03-25 13:55:52 EDT
In debian-land we have ceph-fs-common, which contains /usr/bin/cephfs
(deprecated, need to remove) and mount.ceph.  In rpm-land they are part of the
main ceph package.  I think we should just move mount.ceph into ceph-common and
kill ceph-fs-common for debian and be done with it

Actions

Copy link