Project

General

Profile

Actions

Bug #53572

closed

cephadm should not require running as root for --help

Added by John Mulligan over 2 years ago. Updated almost 2 years ago.

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

0%

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

Description

[ceph@ceph0 ~]$ cephadm --help
ERROR: cephadm should be run as root
[ceph@ceph0 ~]$ cephadm bootstrap --help
ERROR: cephadm should be run as root

Running `sudo cephadm --help` & `sudo cephadm bootstrap --help` prints help text.

The options should probably be parsed before the check to verify if the process is root. This way a user executing cephadm commands can get use --help as a quick reference for options prior to running the real command as root (with sudo, etc).

Actions #1

Updated by John Mulligan over 2 years ago

If the team is interested in taking this report, I'd be happy to look into the required changes myself.

Actions #2

Updated by Sebastian Wagner over 2 years ago

  • Assignee set to John Mulligan
Actions #3

Updated by John Mulligan over 2 years ago

  • Pull request ID set to 44498
Actions #4

Updated by John Mulligan over 2 years ago

  • Status changed from New to Pending Backport
Actions #5

Updated by Redouane Kachach Elhichou almost 2 years ago

  • Backport set to quincy
Actions #6

Updated by Redouane Kachach Elhichou almost 2 years ago

Seems to be already merged to quincy by 6f01977a68c3a2bba7bf2dfb7322728e7da96d0d

Actions #7

Updated by Redouane Kachach Elhichou almost 2 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF