Set owner of directory and keyrings according to owner of /var/lib/ceph.

If Ceph is running under a separate ceph user, it requires access
to various files under this user.
This patch tries to set owner of these files to the current owner
of base_dir (/var/lib/ceph).

If the ceph directory is owned by root, it expects old installation
and should work with old tools as well (avoids --setuser option).

Signed-off-by: Milan Broz <mbroz@redhat.com>

Merge pull request #311 from trhoden/RM-11742-test-cleanup

Add tests for argparse

[RM-11742] Remove more duplicate tests

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

[RM-11742] Add argparse tests for ceph-deploy calamari

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

[RM-11742] Add argparse tests for ceph-deploy config

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

[RM-11742] Add argparse tests for ceph-deploy admin

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

[RM-11742] Add argparse tests for ceph-deploy osd

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

[RM-11742] Add argparse tests for ceph-deploy disk

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

[RM-11742] Add argparse tests for gatherkeys

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

[RM-11742] Remove duplicate tests from test_cli_mon

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

[RM-11742] Add argparse tests for mon destroy

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

[RM-11742] Add argparse tests for mon add

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

Merge pull request #301 from trhoden/RM-11742-test-cleanup

Some test cleanup

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-11742] Correct test name

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

[RM-11742] Remove unneeded creation of .conf file

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

[RM-11742] add check for 'too few arguments'

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

[RM-11742] rename test to be accurate

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

[RM-11742] Add test for missing host(s) to 'new' cmd

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

[RM-11742] Cleanup imports in test_cli_new

Signed-off-by: Travis Rhoden <trhoden@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-11742] Cleanup imports in test_cli_mon

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

[RM-11742] Add test for CLI --version

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>