Merge pull request #310 from tchaikov/throws-at-mons-in-mon_add

[BZ-1231203] throws if multiple mon are passed to "mon add"

doc: "mon destroy" accepts multiple nodes

Signed-off-by: Kefu Chai <kchai@redhat.com>

[BZ-1231203] throws if multiple mon are passed to "mon add"

Signed-off-by: Kefu Chai <kchai@redhat.com>

Merge pull request #307 from honkiko/for_mint

support for distro Mint which is based on ubuntu

support for distro Mint which is based on ubuntu

Signed-off-by: Hong Zhiguo <zhiguohong@tencent.com>

Merge pull request #306 from trhoden/RM-11572

[RM-11572] use http to get GPG key on Wheezy

[RM-11572] use http to get GPG key on Wheezy

Fixes: #11572
Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Merge pull request #304 from alfredodeza/RM-11788

at runtime, display all flags and values

Fixes: #11788
Reviewed-by: Travis Rhoden <trhoden@redhat.com>

Merge pull request #305 from trhoden/RM-11406

Use git.ceph.com for GPG keys

[RM-11406] use gpg.url() for cephdeploy.conf template

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

[RM-11788] create a bare class for testing strict attributes

Signed-off-by: Alfredo Deza <adeza@redhat.com>

[RM-11788] if the attribute is private do not log it

Signed-off-by: Alfredo Deza <adeza@redhat.com>

[RM-11788] add tests for the new arg logger utility

Signed-off-by: Alfredo Deza <adeza@redhat.com>

[RM-11406] use gpg.url() in SUSE too

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

[RM-11406] Support http or https for gpg url

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

[RM-11406] Get GPG key URL by function call

Everywhere but SUSE.  SUSE requires a bit more work to handle
http vs https.

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

[RM-11406] Add function to get ceph.com GPG Key URL

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

[RM-11406] Point to git.ceph.com for GPG keys

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

[RM-11788] nicely output the options and flags in use after overrides

Signed-off-by: Alfredo Deza <adeza@redhat.com>

Merge pull request #303 from alfredodeza/BZ-1219310

allow to skip rpm imports with `cephdeploy.conf`

Reviewed-by: Travis Rhoden <trhoden@redhat.com>

[BZ-1219310] add a doc section explaining sub-command overrides

Signed-off-by: Alfredo Deza <adeza@redhat.com>

[BZ-1219310] when overriding, interpret truthy values before setting them

Signed-off-by: Alfredo Deza <adeza@redhat.com>

[BZ-1219310] ensure that truthy values are interpreted correctly

Signed-off-by: Alfredo Deza <adeza@redhat.com>

1.5.25

[RM-11764] bump version

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

[RM-11764] Update changelog

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Merge pull request #302 from trhoden/BZ-1221830

[BZ-1221830] radosgw pkg is ceph-radosgw on RPM systems

[BZ-1221830] radosgw pkg is ceph-radosgw on RPM systems

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Merge pull request #300 from trhoden/RM-11694-admin-perms

Fix CVE-2015-4053 world-readable permissions on client.admin key

Reviewed-by: Dan Mick <dmick@redhat.com>
Reviewed-by: Sage Weil <sweil@redhat.com>

[RM-11694] Always write admin keyring with mode 0600

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

[RM-11694] Add file mode option to write_file()

You an now pass a file mode to write_file with "mode=" and
it will set the file mode that after writing the file.

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

[RM-11694] Add test for admin key file permissions

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

[RM-11694] Add optional directory to write_file()

This is mostly for testing purposes.  The method has no way to
specify a directory independently from the file name, which it
makes it difficult to use with automated tests.  It will get
refactored more later, but for now add a new kwarg that pass
in a dir to write to.  Since a lot of callers are passing in
absolute paths (.e.g. "/etc/ceph/keyring"), we look for the
beginning slash and chop it off so that we can use os.path.join()
and write into something like /tmp/etc/ceph.

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

[RM-11694] Remove extraneous line

No need to get to the shortname of the host -- it was never used
and was also overwriting the variable used in the loop.

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

[RM-11694] Add CLI test for "ceph-deploy admin"

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

[RM-11694] Require at least one host for 'admin'

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Merge pull request #295 from osynge/wip_zypper_refresh_should_happen

suse/install.py: SUSE zypper refresh

Reviewed-by: Travis Rhoden <trhoden@redhat.com>

suse/install.py: SUSE zypper refresh

zypper may not pull in latest packages if the repository cache is not updated
so we should do a 'zypper ref' before installing packages.

Signed-off-by: Owen Synge <osynge@suse.com>

Merge pull request #296 from osynge/wip_suse_rgw_package_is_called_ceph_radosgw

suse.uninstall: ceph-deploy purge fails on SUSE.

Reviewed-by: Travis Rhoden <trhoden@redhat.com>

suse.uninstall: ceph-deploy purge fails on SUSE.

This is because ceph-deploy tries to remove a package that does not exist.

To fix this we need to change package name to correct name ceph-radosgw
from the old name radosgw.

Signed-off-by: Owen Synge <osynge@suse.com>

1.5.24

Merge pull request #292 from trhoden/RM-11672-v1.5.24

Prepare for new release

[RM-11672] Update version

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

[RM-11672] Update changelog

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Merge pull request #291 from trhoden/BZ-1222505_new_sysvinit_nocustom

Disallow custom cluster names with 'new' command and sysvinit hosts

[BZ-1222505] Disallow 'new' cmd with sysvinit and custom name

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

[BZ-1222505] Add ClusterNameError exception

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Merge pull request #289 from alfredodeza/BZ-1218962

Warn that mds failures on RHEL may be expected

Reviewed-by: Travis Rhoden <trhoden@redhat.com>

[BZ-1218962] add an entry to the changelog

Signed-off-by: Alfredo Deza <adeza@redhat.com>

[BZ-1218962] log as error that we can't deploy MDS in RHEL yet

Signed-off-by: Alfredo Deza <adeza@redhat.com>

Merge pull request #288 from goldwynr/mds-systemd-enable

mds: enable on create with systemd

Merge pull request #290 from trhoden/admin_cli_install

Add --cli/--common option to ceph-deploy install

RM-11642: Remove unused flag from test_install.py

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

RM-11642: Add install-common to install tests

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

RM-11642: Add install of ceph-common only

Add options --cli and --common, both of which allow a user
to install only the ceph-common package on a node.

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

mds: enable on create with systemd

Enable MDS services for distros having systemd.

Signed-off-by: Goldwyn Rodrigues <rgoldwyn@suse.com>

Merge pull request #287 from alfredodeza/1219344

fix `calamari connect`

Reviewed-by: Travis Rhoden <trhoden@redhat.com>

remove --release flag as it is not longer needed for custom repos

Signed-off-by: Alfredo Deza <adeza@redhat.com>

be verbose about the need to have a repo that has calamari packages

Signed-off-by: Alfredo Deza <adeza@redhat.com>

remove custom repository installation from calamari connect

Signed-off-by: Alfredo Deza <adeza@redhat.com>

Merge pull request #285 from alfredodeza/pkg-split

support for package split

Reviewed-by: Travis Rhoden <trhoden@redhat.com>

default to an emtpy list to avoid getting KeyErrors or handling a None.

This should *never* happen, because the install module will always pass
something *unless* it is being explicitly asked not to install anything.

Signed-off-by: Alfredo Deza <adeza@redhat.com>

remove default component imports

Signed-off-by: Alfredo Deza <adeza@redhat.com>

tests for the added behavior

Signed-off-by: Alfredo Deza <adeza@redhat.com>

if --repo is used return [], fallback to defaults when nothing is selected

Signed-off-by: Alfredo Deza <adeza@redhat.com>

remove 'ceph' from install tests as it is no longer used

Signed-off-by: Alfredo Deza <adeza@redhat.com>

remove 'ceph' from being installed as it will no longer make sense

Signed-off-by: Alfredo Deza <adeza@redhat.com>

remove 'ceph' from being installed as it will no longer make sense

Signed-off-by: Alfredo Deza <adeza@redhat.com>

make default_components use an actual list

Signed-off-by: Alfredo Deza <adeza@redhat.com>

add an is_rpm/is_deb attribute to re-use in package checks

Signed-off-by: Alfredo Deza <adeza@redhat.com>

add tests for using distro in detect_components for deb/rpm

Signed-off-by: Alfredo Deza <adeza@redhat.com>

Merge pull request #284 from trhoden/rgw-service

RGW service fixes for RPM systems

Merge pull request #281 from trhoden/11561_rgw_naming

Better handling of custom RGW daemon names

Add tests for RGW daemon naming

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Update docs and changelog about rgw daemon naming

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Always prefix RGW daemons wth 'rgw.'

If a user specifics a custom RGW daemon name, automatically prefix it
with 'rgw.'

Fixes: http://tracker.ceph.com/issues/11561
Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Enable ceph-radosgw service after creating rgw daemon

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Fix passing of custom service names to enable_service()

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Remove rgw name from call to service start

The init script does not use the name, so don't pass it.

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

The RGW daemon name already has "rgw." prefixed

Remove redundant "rgw." in daemon name when starting service

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

rgw: call service ceph-radosgw instead of ceph

On RPM systems, the sysvinit script for rgw is "ceph-radosgw"

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Merge pull request #283 from trhoden/mkdirs_rgw

Create /var/lib/ceph/radosgw/{cluster}-{name} recursively

Recursively create /var/lib/ceph/radosgw/... path

Previously we would fail if /var/lib/ceph/radosgw didn't already
exist.  Go ahead and make that directory if needed.

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Add safe_makedirs function

safe_makedirs will recursively create paths, just like mkdirs -p

The "safe" part is that it wont error/except if any of the paths already
exist.

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Merge pull request #280 from alfredodeza/11347

use new remoto version `0.0.25`

use the new 0.0.25 version of remoto

Signed-off-by: Alfredo Deza <adeza@redhat.com>

update the changelog

Signed-off-by: Alfredo Deza <adeza@redhat.com>

Merge pull request #277 from ktdreyer/wip-changelog-cve

doc: add CVE-2015-3010 to changelog

Merge pull request #275 from joehandzik/master

Updating the url used to clone the remoto repository

doc: add CVE-2015-3010 to changelog

One of the changes in ceph-deploy 1.5.23 is tracked as CVE-2015-3010.
Document this in the changelog.

Signed-off-by: Ken Dreyer <kdreyer@redhat.com>

remove unnecessary parentheses from an if statement

Try the ceph.com mirror first. If that fails, retry with the github url and the https protocol.

Update the run function to cascade the called process's return code back to its caller

Updating github url to point to the ceph fork of remoto

1.5.23

Merge pull request #276 from trhoden/hammer

change default release to Hammer

Update version

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Update changelog for 1.5.23

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

change default release to Hammer

Signed-off-by: Travis Rhoden <trhoden@redhat.com>

Updating the url used to clone the remoto repository