Project

General

Profile

Actions

Bug #12287

closed

ceph command line tool chokes on ceph –w (the dash is unicode 'en dash' &ndash, copy-paste to reproduce)

Added by Samuel Just almost 9 years ago. Updated over 7 years ago.

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

0%

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

Description

ubuntu@mira040:~$ sudo ceph –w
Traceback (most recent call last):
File "/usr/bin/ceph", line 896, in <module>
retval = main()
File "/usr/bin/ceph", line 839, in main
sigdict, inbuf, verbose)
File "/usr/bin/ceph", line 409, in new_style_command
valid_dict = validate_command(sigdict, cmdargs, verbose)
File "/usr/lib/python2.7/dist-packages/ceph_argparse.py", line 920, in validate_command
matched = matchnum(args, sig, partial=True)
File "/usr/lib/python2.7/dist-packages/ceph_argparse.py", line 733, in matchnum
validate_one(word, desc, partial and (len(words) == 0))
File "/usr/lib/python2.7/dist-packages/ceph_argparse.py", line 705, in validate_one
desc.instance.valid(word, partial)
File "/usr/lib/python2.7/dist-packages/ceph_argparse.py", line 496, in valid
if self.prefix.startswith(s):
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 0: ordinal not in range(128)

Not sure what the right behavior here should be, probably should just catch UnicodeDecodeError and print a useful error message instead.


Related issues 1 (0 open1 closed)

Copied to Ceph - Backport #16798: jewel: ceph command line tool chokes on ceph –w (the dash is unicode 'en dash' &ndash, copy-paste to reproduce)ResolvedNathan CutlerActions
Actions #1

Updated by Kefu Chai almost 9 years ago

  • Assignee set to Kefu Chai
Actions #2

Updated by Kefu Chai almost 9 years ago

  • Status changed from New to Fix Under Review
Actions #3

Updated by Kefu Chai over 8 years ago

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

Updated by Kefu Chai almost 8 years ago

  • Status changed from Resolved to 12
Actions #5

Updated by Kefu Chai almost 8 years ago

  • Status changed from 12 to Fix Under Review
Actions #6

Updated by Ken Dreyer almost 8 years ago

  • Backport set to jewel
Actions #7

Updated by Kefu Chai over 7 years ago

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

Updated by Nathan Cutler over 7 years ago

  • Copied to Backport #16798: jewel: ceph command line tool chokes on ceph –w (the dash is unicode 'en dash' &ndash, copy-paste to reproduce) added
Actions #9

Updated by Nathan Cutler over 7 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF