Project

General

Profile

Actions

Bug #4642

closed

ceph-deploy: disk zap can throw a better error message

Added by Tamilarasi muthamizhan about 11 years ago. Updated over 10 years ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
ceph-deploy
Target version:
-
% Done:

0%

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

Description

ceph-deploy zap disk can throw a better error message, when no disks are specified

tamil@ubuntu:~/ceph-deploy-latest/ceph-deploy$ ./ceph-deploy disk zap burnupi06
Traceback (most recent call last):
File "./ceph-deploy", line 9, in <module>
load_entry_point('ceph-deploy==0.0.1', 'console_scripts', 'ceph-deploy')()
File "/home/tamil/ceph-deploy-latest/ceph-deploy/ceph_deploy/cli.py", line 112, in main
return args.func(args)
File "/home/tamil/ceph-deploy-latest/ceph-deploy/ceph_deploy/osd.py", line 312, in disk
disk_zap(args)
File "/home/tamil/ceph-deploy-latest/ceph-deploy/ceph_deploy/osd.py", line 258, in disk_zap
zap_r(disk)
File "/home/tamil/ceph-deploy-latest/ceph-deploy/virtualenv/local/lib/python2.7/site-packages/pushy-0.5.1-py2.7.egg/pushy/protocol/proxy.py", line 255, in <lambda>
(conn.operator(type_, self, args, kwargs))
File "/home/tamil/ceph-deploy-latest/ceph-deploy/virtualenv/local/lib/python2.7/site-packages/pushy-0.5.1-py2.7.egg/pushy/protocol/connection.py", line 66, in operator
return self.send_request(type_, (object, args, kwargs))
File "/home/tamil/ceph-deploy-latest/ceph-deploy/virtualenv/local/lib/python2.7/site-packages/pushy-0.5.1-py2.7.egg/pushy/protocol/baseconnection.py", line 323, in send_request
return self.__handle(m)
File "/home/tamil/ceph-deploy-latest/ceph-deploy/virtualenv/local/lib/python2.7/site-packages/pushy-0.5.1-py2.7.egg/pushy/protocol/baseconnection.py", line 639, in __handle
raise e
pushy.protocol.proxy.ExceptionProxy: coercing to Unicode: need string or buffer, NoneType found

Actions #1

Updated by Tamilarasi muthamizhan about 11 years ago

  • Assignee changed from Sage Weil to Dan Mick
Actions #2

Updated by Tamilarasi muthamizhan about 11 years ago

  • Assignee deleted (Dan Mick)
Actions #3

Updated by Anonymous almost 11 years ago

  • Priority changed from Normal to High
Actions #4

Updated by Alfredo Deza almost 11 years ago

  • Assignee set to Alfredo Deza
Actions #5

Updated by Alfredo Deza over 10 years ago

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

Updated by Alfredo Deza over 10 years ago

  • Status changed from Fix Under Review to Resolved

Merged into ceph-deploy master, basically catching None arguments before they go off.

Hash: aeada3ec6659474ceb1b1419c3d86fcfe79f3215

Actions

Also available in: Atom PDF