split unit test into two, fix typo in comments

Signed-off-by: Dmitry Borodaenko <angdraug@gmail.com>

consistently remove domain from short hostname

Signed-off-by: Dmitry Borodaenko <angdraug@gmail.com>

Merge pull request #70 from ceph/wip-6266

new, mon: fix mon_hosts import

new, mon: fix mon_hosts import

Fixes: #6266
Signed-off-by: Sage Weil <sage@inktank.com>

Merge pull request #68 from angdraug/mon-create-host-ip

allow to specify host:fqdn/ip in mon create and destroy

allow to specify host:fqdn/ip in mon create and destroy

Signed-off-by: Dmitry Borodaenko <angdraug@gmail.com>

Merge pull request #69 from alfredodeza/6255

catch a possible non-json response from the status command

Reviewed-by: Sage Weil <sage@inktank.com>

catch a possible non-json response from the status command

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #67 from alfredodeza/6017-1

attempt to give some info about the actual mon status

attempt to give some info about the actual mon status

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #66 from alfredodeza/6017-1

Create a `mon_status` check

make sure the wrappers always return the check_call

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

create a mon_status that reports back with boolean flags

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #65 from alfredodeza/6104

detect sudo needs and switch connections for any use-case

detect sudo needs and switch connections for any use-case

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #64 from ceph/6245

fix wrong logic for mangling repos

fix wrong logic for mangling repos

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #62 from alfredodeza/6160-2

conditionally avoid installing from ceph repos

Merge pull request #63 from alfredodeza/6237

fix missing key argument on remote function

fix missing key argument on remote function

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

add boolean flags for mangling repos

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

accept adjust_repos as an argument flag

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #60 from alfredodeza/6132

create a helper to warn against incompatible hostnames

create a helper to warn against incompatible hostnames

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #59 from alfredodeza/5944

make use of the newer, better, remote helpers for prepare+create OSD

make use of the newer, better, remote helpers for prepare+activate OSD

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #58 from alfredodeza/5763-2

really complain about `ceph-deploy new [IP]`

better rendering of HTML target

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

make sure links to docs work

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

update the documentation to define what this tool is/is not

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

update the changelog

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

some tests for the new IP verification

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

make sure you do not allow single IPs for new

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #57 from chenull/master

a quick workaround for bug #721301 in debian causing install failure

a quick workaround for bug #721301 in debian causing install failure

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=721301

Signed-off-by: sayid munawar <sayid.munawar@gmail.com>

v1.2.3

Merge pull request #56 from alfredodeza/master

update version and changelog

update the changelog

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

bump the version

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #55 from alfredodeza/6138

Fix lack of correct remote paths

make disk_list use the new subprocess wrappers

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

set an appropriate path for the remote command

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

ceph-deploy.spec:  Remove disk util dependencies.

The disk utilities are required for ceph, but not for
ceph-deploy. The node being managed by ceph will require
the utilities, but they are installed along with ceph.

Signed-off-by: Gary Lowell <gary.lowell@inktank.com>

Merge pull request #54 from rmoe/master

Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
Reviewed-by: Sage Weil <sage.weil@inktank.com>

Exit from ceph-deploy script with return value of main()
Signed-off-by: Ryan Moe <rmoe@mirantis.com>

Merge pull request #53 from alfredodeza/master

test ipv6 addresses as well as ipv4

test ipv6 addresses as well as ipv4:

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #50 from alfredodeza/6102

use replacepkgs flag to quiet down installtion of epel

remove --force in favor of --replacepkgs

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #51 from alfredodeza/4642

fix zap command to require both hostname and disk, erroring properly

Merge pull request #52 from alfredodeza/6124

don't prevent HOST:IP from being used as it is valid

update tests to reflect new validator behavior

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

do not complain when IP is matched with a host

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

fix zap command to require both hostname and disk, erroring properly

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

ceph-deploy.spec:  'or' in requires not supported on all platforms

Signed-off-by: Gary Lowell <gary.lowell@inktank.com>

v1.2.2

Merge pull request #49 from alfredodeza/5499

`osd` should honor the `--cluster` flag and use it (fixes 5499)

update the changelog

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

make sure we pass the --cluster flag

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #48 from alfredodeza/6077

Handle pushy connection exceptions

udpate the changelog

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

catch ClientInitException at cli.py (for now)

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #47 from alfredodeza/6077

lsb module will fallback when lsb is not found

move the lsb_fallback to the lsb module

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

avoid circular import issues for lsb

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

update the CHANGELOG

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

update the version

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

use the lsb_fallback when lsb is not found

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #46 from alfredodeza/6019

fallback when lsb_release is not installed

use the return code to determine if lsb_release is present

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

removing unused tests

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

move install_epel to the centos/install module

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

create a Popen wrapper that can do remote stuff for us

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

I was not returning the remote function - sigh

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

some tests to excercise the code

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

forgot to add redhat

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

install lsb if it is not present

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

a single test for pkg_managers.rpm

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

add helpers for package managers

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

v1.2.1

Merge pull request #45 from alfredodeza/master

Bump for release 1.2.1

update the changelog, bump the version number to 1.2.1

Signed-off-by: Alfredo Deza <alfredo@deza.pe>

update the readme

Signed-off-by: Alfredo Deza <alfredo@deza.pe>

all tests refactored and passing

Signed-off-by: Alfredo Deza <alfredo@deza.pe>

Merge pull request #44 from alfredodeza/5895-2

Do not attempt to capture stdout/stderr on some `check_calls` D=

do not patch stdout/stderr on mon start

Signed-off-by: Alfredo Deza <alfredo@deza.pe>

allow an optional patch kwarg to check_call

Signed-off-by: Alfredo Deza <alfredo@deza.pe>

optiionally patch stderr and stdout in the context manager

Signed-off-by: Alfredo Deza <alfredo@deza.pe>

Merge pull request #43 from alfredodeza/5962

default to dumpling for installs

default to dumpling for installs

Signed-off-by: Alfredo Deza <alfredo@deza.pe>

Merge pull request #40 from ceph/wip-5911

better use of distro.init - thanks Mark Kirkwood!

Merge pull request #41 from alfredodeza/5895

re-raise so we halt execution of remote actions

re-raise so we halt execution of remote actions

Signed-off-by: Alfredo Deza <alfredo@deza.pe>

Merge pull request #39 from alfredodeza/5934

CLI updates for --version, --help and exceptions from exc.DeployError

better use of distro.init - thanks Mark Kirkwood!

Signed-off-by: Alfredo Deza <alfredo@deza.pe>

v1.2

build scripts:  Update for release builds.

Allow alternate signing keyid to be specified in environment
variable.  For debian add backport tags to release version
numbers if release flag is set.

Signed-off-by: Gary Lowell <gary.lowell@inktank.com>

rules:  use dh_python2

Signed-off-by: Gary Lowell <gary.lowell@inktank.com>

update the changelog with this branch changes

Signed-off-by: Alfredo Deza <alfredo@deza.pe>

make use of the __version__ in setup.py

Signed-off-by: Alfredo Deza <alfredo@deza.pe>