Project

General

Profile

Actions

Feature #6124

closed

`ceph-deploy new` should accept node:IP pairs

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

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

0%

Source:
other
Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

Issue 5763 prevented from users creating a new config by passing an IP alone, but the validator that was added
prevented any IP.

The fix here is to allow name:IP making the validator less restrictive.

Actions #1

Updated by Sage Weil over 10 years ago

one clarification: mon create ... takes a host or fqdn or ip; it doesn't matter, it's just a target to connect to. it's ceph-deploy new that cares about hostnames vs fqdn/ip. see #6120

should probably close this issue?

Actions #2

Updated by Alfredo Deza over 10 years ago

  • Subject changed from `ceph-deploy mon create` should accept node:IP pairs to `ceph-deploy new` should accept node:IP pairs
  • Description updated (diff)

Updated the wording. It has nothing to do with `mon create` but `new`.

Got confused with the help menu :/

Actions #3

Updated by Sage Weil over 10 years ago

  • Tracker changed from Bug to Feature
Actions #4

Updated by Alfredo Deza over 10 years ago

  • Status changed from 12 to Resolved

Merged on ceph-deploy's master branch, hash: cdc8f8a8c38543587e7120fbc17954c909c1b0fa

The argument validator was being overly strict so the fix was to relax the constraint to accept host:IP pairs.

Actions

Also available in: Atom PDF