Project

General

Profile

Actions

Bug #12171

closed

ceph-deploy pkg command argparse config is not sane

Added by Travis Rhoden almost 9 years ago. Updated almost 9 years ago.

Status:
Resolved
Priority:
Low
Assignee:
-
Target version:
% Done:

0%

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

Description

The options to "ceph-deploy pkg" are --install and --remove, followed by one more hosts.

however, these two options are not in argparse as mutually exclusive, which implies that it is fine to provide both (install some packages on a machine and also remove them). But there is later code that makes it do only one or the other (with install taking precedence).

We should either make them both work, or make the mutually exclusive. The latter preserves current behavior.

Actions #1

Updated by Travis Rhoden almost 9 years ago

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

Updated by Travis Rhoden almost 9 years ago

  • Status changed from Fix Under Review to Resolved
  • Target version set to 1.5.26
Actions

Also available in: Atom PDF