Project

General

Profile

Actions

Bug #11101

closed

ceph tell: broken error message / misleading hinting

Added by Federico Lucifredi about 9 years ago. Updated over 7 years ago.

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

0%

Source:
other
Tags:
Backport:
hammer
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Description of problem:

root@master:~# ceph osd tell
"osd tell" is deprecated; try "tell osd.<id>" instead (id can be "*") 
root@master:~# ceph tell osd.*
Cannot use 'tell' with interactive mode
root@master:~#

Expected results:

I am not in interactive mode!

Besides that, I'd want the hint to point me to something I can try out without further understanding, like

"osd tell" is deprecated; try "tell osd.<id> bench" instead (id can be "*")

"ceph osd tell" is mentioned in the "top 10 commands" blogs in several places and on the ceph wiki - separately requested a fix there. That gets users started down this path, just like it did me.


Related issues 1 (0 open1 closed)

Copied to Ceph - Backport #12494: ceph tell: broken error message / misleading hintingResolvedLoïc Dachary03/11/2015Actions
Actions #1

Updated by Kefu Chai almost 9 years ago

  • Description updated (diff)
  • Regression set to No
Actions #2

Updated by Kefu Chai almost 9 years ago

  • Status changed from New to Fix Under Review
  • Assignee set to Kefu Chai
Actions #3

Updated by Ken Dreyer almost 9 years ago

  • Backport set to hammer

Thanks!

Actions #4

Updated by Kefu Chai almost 9 years ago

  • Status changed from Fix Under Review to Pending Backport
Actions #5

Updated by Loïc Dachary over 8 years ago

  • Status changed from Pending Backport to Resolved
Actions #6

Updated by Kefu Chai over 8 years ago

  • Status changed from Resolved to Fix Under Review
Actions #7

Updated by Kefu Chai over 8 years ago

per Harish

We need to change the message something like this:

"Ceph 'tell' requires additional arguments. <print usage for tell command>"

Actions #9

Updated by Kefu Chai about 8 years ago

  • Status changed from Fix Under Review to Pending Backport
Actions #10

Updated by Loïc Dachary about 8 years ago

  • Status changed from Pending Backport to Resolved
Actions #11

Updated by Kefu Chai over 7 years ago

  • Status changed from Resolved to Pending Backport

@Loïc, https://github.com/ceph/ceph/pull/7106 is not backported yet.

Actions #13

Updated by Nathan Cutler over 7 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF