Project

General

Profile

Actions

Bug #16709

closed

No output for "ceph mds rmfailed 0 --yes-i-really-mean-it" command

Added by Rohith Radhakrishnan almost 8 years ago. Updated over 6 years ago.

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

0%

Source:
other
Tags:
Backport:
jewel, kraken
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
Labels (FS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

there is no output for the command ceph mds rmfailed 0 --yes-i-really-mean-it. The command is successful how many ever times we run it. There is nothing displayed to actually show if the command is successful or not or whether the mds has actually been failed.

ceph mds rmfailed 0 --yes-i-really-mean-it
ems@rack2-client-3:~$ ceph mds rmfailed 1 --yes-i-really-mean-it
Error EINVAL: Rank '1' not foundinvalid role '1'
ems@rack2-client-3:~$ ceph mds rmfailed 0 --yes-i-really-mean-it
ems@rack2-client-3:~$ ceph mds rmfailed 0 --yes-i-really-mean-it
ems@rack2-client-3:~$ ceph mds rmfailed 0 --yes-i-really-mean-it


Related issues 2 (0 open2 closed)

Copied to CephFS - Backport #19482: jewel: No output for "ceph mds rmfailed 0 --yes-i-really-mean-it" commandResolvedNathan CutlerActions
Copied to CephFS - Backport #19483: kraken: No output for "ceph mds rmfailed 0 --yes-i-really-mean-it" commandResolvedNathan CutlerActions
Actions #1

Updated by John Spray almost 8 years ago

Without looking at the code, I would imagine that you're seeing EINVAL for rank 1 because there is no such rank (so it doesn't make sense to try and remove it from the failed set).

You're seeing 0 for rank 0 because Ceph CLI commands are idempotent, so whenever we have a removal -type operation we will return success if it is not present.

This isn't a bug as such but probably could be a bit friendlier by printing a message saying that the rank you're asking to remove from the failed set was already removed.

Actions #2

Updated by Rohith Radhakrishnan almost 8 years ago

Yes that would be ideal. as of now we cannot be sure if it has been actually removed or not.

Actions #3

Updated by Greg Farnum almost 8 years ago

  • Category set to Administration/Usability
Actions #4

Updated by John Spray about 7 years ago

  • Status changed from New to Fix Under Review
  • Backport set to jewel, kraken
Actions #5

Updated by John Spray about 7 years ago

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

Updated by Nathan Cutler about 7 years ago

  • Copied to Backport #19482: jewel: No output for "ceph mds rmfailed 0 --yes-i-really-mean-it" command added
Actions #7

Updated by Nathan Cutler about 7 years ago

  • Copied to Backport #19483: kraken: No output for "ceph mds rmfailed 0 --yes-i-really-mean-it" command added
Actions #8

Updated by Nathan Cutler over 6 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF