Project

General

Profile

Cleanup #16035

Remove "cephfs" CLI

Added by John Spray about 4 years ago. Updated almost 4 years ago.

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

0%

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

Related to fs - Cleanup #16808: Merge "ceph-fs-common" into "ceph-common" Resolved 07/25/2016
Duplicated by fs - Bug #15485: drop /usr/bin/cephfs Duplicate 04/13/2016

History

#1 Updated by John Spray about 4 years ago

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

#2 Updated by John Spray about 4 years ago

  • Duplicated by Bug #15485: drop /usr/bin/cephfs added

#3 Updated by Greg Farnum about 4 years ago

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

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

#5 Updated by Ken Dreyer almost 4 years ago

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

#6 Updated by Ken Dreyer almost 4 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

#7 Updated by John Spray almost 4 years ago

  • Status changed from In Progress to 4

#8 Updated by John Spray almost 4 years ago

  • Status changed from 4 to Fix Under Review

#9 Updated by Nathan Cutler almost 4 years ago

  • Backport set to jewel

#10 Updated by Nathan Cutler almost 4 years ago

  • Status changed from Fix Under Review to Pending Backport

#11 Updated by Nathan Cutler almost 4 years ago

  • Related to Cleanup #16808: Merge "ceph-fs-common" into "ceph-common" added

#12 Updated by Nathan Cutler almost 4 years ago

  • Status changed from Pending Backport to Fix Under Review
  • Backport deleted (jewel)

#14 Updated by Nathan Cutler almost 4 years ago

  • Status changed from Fix Under Review to Resolved

Also available in: Atom PDF