Project

General

Profile

Bug #6428

`ceph-deploy osd` calls missing $PATH

Added by Alfredo Deza over 10 years ago. Updated over 10 years ago.

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

0%

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

Description

There are a few functions left in `ceph_deploy/osd.py` that use the old connecting library and cause issues because $PATH is not defined in the remote host and break horribly when a user calls ceph-deploy on a server that does not set that for SSH.

> [ceph_deploy.osd][DEBUG ] zapping /dev/sdc on ceph.host.name
> [ceph_deploy.sudo_pushy][DEBUG ] will use a remote connection with sudo
> Traceback (most recent call last):
>  File "/usr/bin/ceph-deploy", line 21, in <module>
>    sys.exit(main())
>  File "/usr/lib/python2.6/site-packages/ceph_deploy/util/decorators.py",
> line 83, in newfunc
>    return f(*a, **kw)
>  File "/usr/lib/python2.6/site-packages/ceph_deploy/cli.py", line 147, in
> main
>    return args.func(args)
>  File "/usr/lib/python2.6/site-packages/ceph_deploy/osd.py", line 381, in
> disk
>    disk_zap(args)
>  File "/usr/lib/python2.6/site-packages/ceph_deploy/osd.py", line 317, in
> disk_zap
>    zap_r(disk)
>  File "/usr/lib/python2.6/site-packages/pushy/protocol/proxy.py", line 255,
> in <lambda>
>    (conn.operator(type_, self, args, kwargs))
>  File "/usr/lib/python2.6/site-packages/pushy/protocol/connection.py", line
> 66, in operator
>    return self.send_request(type_, (object, args, kwargs))
>  File "/usr/lib/python2.6/site-packages/pushy/protocol/baseconnection.py",
> line 329, in send_request
>    return self.__handle(m)
>  File "/usr/lib/python2.6/site-packages/pushy/protocol/baseconnection.py",
> line 645, in __handle
>    raise e
> pushy.protocol.proxy.ExceptionProxy: [Errno 2] No such file or directory

History

#1 Updated by Alfredo Deza over 10 years ago

  • Status changed from New to Fix Under Review

#2 Updated by Alfredo Deza over 10 years ago

  • Status changed from Fix Under Review to Resolved

Merged into ceph-deploy master branch with hash: 0ffd809

Also available in: Atom PDF