[RM-17107] hosts.centos: use ceph.target for systemd and is_systemd util for detection

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

Merge pull request #416 from ceph/RM-17086

[RM-17086] osd: use load_raw for reading/writing ceph conf

[RM-17086] osd: use load_raw for reading/writing ceph conf

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

Merge pull request #415 from minshenglin/RM-17053

[RM-17053] cleanup: using load_raw to replace old practice

[RM-17053] cleanup: using load_raw to replace old practice

Signed-off-by: MinSheng Lin <minsheng.l@inwinstack.com>

1.5.35

Merge pull request #414 from ceph/RM-17024

release 1.5.35

[RM-17024] ceph_deploy: bump to 1.5.35

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

[RM-17024] docs: update changelog for 1.5.35

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

Merge pull request #413 from SUSE/wip_zypper

RM-16979 zypper fix for remove

[RM-16979] Change test to use check function.

Check function testable in same way as run function.

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

[RM-16979] Change use of check function.

Check function should be testable in same way as run function.

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

[RM-16979] ceph_deploy.util.pkg_managers: Fix zypper handling

zypper can safely ignore non installed packages, with the --ignore-unknown
option.

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

[RM-16979] zypper remove each package separately

We ca not remove multiple packages with zypper in one line. as if a package is
not available in a repository, zypper will abort immediately rather than remove
the packages it was instructed to remove.

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

[RM-16979] zypper handle ZYPPER_EXIT_INF_CAP_NOT_FOUND return code

When removing package(s) with zypper, that are (all) installed zypper may
succeed and return 0, but when any package(s) are not present before requesting
them to be removed zypper will return 104 on success.

The function remoto.process.run raises a "RuntimeError" exception when the
command does not return 0 in the form:

    RuntimeError("Failed to execute command: %s" % " ".join(cmd))

Hence to maintain the expected behaviour, remoto.process.check is used, and the
error code is checked, and if it does not match 0 or 104 the exception is
raised.

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

Merge pull request #410 from ieth0/ieth0-patch-1

modify sysvinit if condition

Merge pull request #411 from arvimal/master

[RM-16651] Installs `diamond` while executing `ceph-deploy calamari c…

[RM-16651] Installs `diamond` while executing `ceph-deploy calamari connect`.

Fixes http://tracker.ceph.com/issues/16651

`ceph-deploy calamari connect` does not install `diamond`.

This patch tries to fix it by calling `distro.pkg.install()`
for `diamond`.

Signed-off-by: Vimal <arvimal@yahoo.in>

modify sysvinit if condition

If ceph packages aren't installed on the host with systemd preinstalled, script will return sysvinit instead of systemd which is invalid.

Merge pull request #407 from ceph/wip-ubuntu-package-split

Handle ubuntu package split for ceph-osd and ceph-mon

Merge pull request #409 from ceph/RM-16533

[RM-16533] install: define gpgcheck always

[RM-16533] install: define gpgcheck always

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

Merge pull request #404 from gaowanlong/nogpgcheck

Add the --nogpgcheck option

[RM-16524] Add the --nogpgcheck option

I met the deploy failure with local mirror and the official gpgkeys,
the error is like:

[node-1][WARNIN]  Failing package is: 1:librbd1-0.94.6-0.el7.x86_64
[node-1][WARNIN]  GPG Keys are configured as: file:///opt/ceph-deploy/repo/release.asc

Anyway, I do not care the gpg keys but want to install successfully
without gpgcheck.

Signed-off-by: Wanlong Gao <wanlong.gao@easystack.cn>

[BZ-1349999] Handle package split for ubuntu

ceph is now split into ceph-base, ceph-mon and ceph-osd

Signed-off-by: Vasu Kulkarni <vasu@redhat.com>

Merge pull request #406 from ceph/RM-16452

[RM-16452] docs: include gatherkeys.rst in the toctree as well

[RM-16452] docs: include gatherkeys.rst in the toctree as well

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

Merge pull request #405 from ceph/RM-16452

[RM-16452] docs: include admin.rst in the toctree

[RM-16452] docs: include admin.rst in the toctree

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

Merge pull request #388 from BlaXpirit/RM-15016

[RM-15016] Python 3 Compatibility

Reviewed-by: Josh Durgin <jdurgin@redhat.com>

[RM-15016] Bump remoto requirement to 0.0.29

Signed-off-by: Oleh Prypin <oleh@pryp.in>

[RM-15016] Rework the bootstrap script to properly install Python 3 on CentOS

Signed-off-by: Oleh Prypin <oleh@pryp.in>

Merge pull request #402 from gaowanlong/master

Fix rsync username when using local_mirror

Fix rsync username when using local_mirror

We should use the configured username for rsync to connect
to the remote host.

Signed-off-by: Wanlong Gao <wanlong.gao@easystack.cn>

[RM-15016] Opt out of a backwards-incompatible change in configparser

Signed-off-by: Oleh Prypin <oleh@pryp.in>

Merge branch 'master' into RM-15016

[RM-15016] Allow selecting Python version in bootstrap script

Signed-off-by: Oleh Prypin <oleh@pryp.in>

[RM-15016] Fix Python 3 compatibility after merge of RM-15451

Signed-off-by: Oleh Prypin <oleh@pryp.in>

Merge branch 'master' into RM-15016

[RM-15016] Add py34 to tox config

Signed-off-by: Oleh Prypin <oleh@pryp.in>

[RM-15016] Fixes related to changes in argparse's behavior

* No error when passing an incomplete command:
  http://bugs.python.org/issue9253#msg186387
* Different error message for too few arguments
* Different output stream for version number

Signed-off-by: Oleh Prypin <oleh@pryp.in>

[RM-15016] Make usage of bytes vs strings compatible with Python 3

Signed-off-by: Oleh Prypin <oleh@pryp.in>

1.5.34

Merge pull request #401 from ceph/RM-16181

release 1.5.34

[RM-16181] bump to 1.5.34

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

[RM-16181] update the changelog

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

Merge pull request #400 from ceph/RM-14099

partx/partprobe calls are no longer needed when zapping

[RM-14099] osd: partx/partprobe calls are no longer needed when zapping. ceph-disk does it

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

Merge pull request #399 from ceph/RM-16140

remove the ability to use ext4 in ceph-deploy

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

[RM-16140] tests: test btrfs instead of ext4

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

[RM-16140] tests: remove osd ext4 tests

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

[RM-16140] osd: remove ability to pick ext4

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

Merge pull request #387 from SUSE/wip-fix-mon-install-suse

Fix mon install suse

ceph_deploy.tests.unit.hosts.test_suse: Fix default init

The default init system for SUSE should be systemd, as all future releases of
SUSE are expected to be based upon systemd

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

[RM-14740] rcceph is no longer used with (open)SUSE

rcceph is no longer used with (open)SUSE and has been removd from ceph upstream.
Hence it is best to start the services directly and make the service start code
identical to Fedora.

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

[RM-14742] Set default init to systemd for (open)SUSE

(open)SUSE is now using systemd moving forward, this patch corrects behavior
on Tumbleweed and LEAP

openSUSE Tumbleweed distribution is a pure rolling release version of openSUSE
running the latest packages.

openSUSE LEAP is a rolling release version of openSUSE based on SUSE linux
Enterprise.

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

Merge pull request #393 from SUSE/fix-remove-ceph-create-keys-dep

[RM-15451] Gather keys directly from the mon

ceph_deploy.tests.test_gather_keys_with_mon: Mock shortname

Since gatherkeys now uses distro.conn.remote_module.shortname()
We need to mock this also.

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

ceph_deploy.gatherkeys: Normalize hostname

We should always use the short hostname for mon interactions.

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

ceph_deploy.gatherkeys: logging mon error improvment

Improve logging message so it is clear the error is not connecting the mon

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

[fix-remove-ceph-create-keys-dep] ceph_deploy.gatherkeys: logging mon keyring location

While testing we had issues finding the mon keyring.

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

[fix-remove-ceph-create-keys-dep_cp] docs.source.index: Update basic install

Updated the basic install instructions.

(1) Added a creating a new configuration section.
(2) Added reference to the 'new' subcommand
(3) Added reference to the 'mon' subcommand
(4) Improved Gather keys
(5) Added reference to the 'gatherkeys' subcommand
(6) Moved Admin hosts section as now it is more important.
(7) Added reference to the 'admin' subcommand

The gatherkeys subcommand does not depend upon ceph-create-keys so does not
have to have the side effect of making mons admin nodes. To get this in I
must add documentation to ceph-deploy.

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

[fix-remove-ceph-create-keys-dep_cp] docs.source.mon: Add note on changed behavior

The gatherkeys subcommand does not depend upon ceph-create-keys so does not
have to have the side effect of making mons admin nodes.

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

[fix-remove-ceph-create-keys-dep_cp] docs.source.gatherkeys: Add gatherkeys documentation

The gatherkeys subcommand did not have and documentation.

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

[fix-remove-ceph-create-keys-dep_cp] docs.source.admin: Add admin documentation

The admin command did not have and documentation.

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

[fix-remove-ceph-create-keys-dep_cp] ceph_deploy.tests.test_gather_keys_missing: Remove unneeded imports

Remove unneeded imports

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

[fix-remove-ceph-create-keys-dep_cp] ceph_deploy.gatherkeys: use comma-separated values in logs

It is prefured to use comma-separated values in logs

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

Merge pull request #398 from ceph/RM-15865

[RM-15865] bump the remoto requirement to 0.0.28

[RM-15865] bump the remoto requirement to 0.0.28

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

1.5.33

Merge pull request #397 from ceph/RM-15570

release 1.5.33

[RM-15570] update the changelog

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

[RM-15570] bump to 1.5.33

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

Merge pull request #396 from ceph/wip-jewel

default to jewel

install: default to jewel

Signed-off-by: Sage Weil <sage@redhat.com>

Merge pull request #395 from BlaXpirit/RM-15018

[RM-15018] Remove broken 'mon create' unittest

[RM-15018] Remove broken 'mon create' unittest

Signed-off-by: Oleh Prypin <oleh@pryp.in>

1.5.32

Merge pull request #394 from ceph/RM-15484

Get ready for releasing 1.5.32

[RM-15484] bumd ceph-deploy version in __init__

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

[RM-15484] update the changelog for 1.5.32

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

Merge pull request #391 from jordan41177/fix-function-name-of-osd-keyring-creation

Fix the function name of osd keyring creation

[RM-15409] Fix the function name of osd keyring creation

Change the function name of "create_osd" to "create_osd_keyring".
This function just do:
1. Check the remote node's osd keyring exist or not
2. If not, it create one.
The original name would confused the developers.

Signed-off-by: DesmondS <jordanjimmy41177@gmail.com>

Merge pull request #390 from vumrao/wip-vumrao-15112

[RM-15112] Rename ceph-deploy log file to ceph-deploy-${CLUSTER}.log

[RM-15112] Rename ceph-deploy log file to ceph-deploy-${CLUSTER}.log

This patch renames ceph-deploy log file to ceph-deploy-${CLUSTER}.log

Fixes: #15112
Signed-off-by: Vikhyat Umrao <vumrao@redhat.com>

Merge pull request #389 from ceph/wip-bluestore

osd,disk: add --bluestore flag

osd,disk: add --bluestore flag

This will pass --bluestore on to ceph-disk during the prepare/create
step.

Signed-off-by: Sage Weil <sage@redhat.com>

Merge pull request #383 from mazebuhu/RM-13847

[RM-13847] Using full paths to executables

[RM-13847] Using full paths for partx and partprobe executables.

Signed-off-by: Martin Palma <martin@palma.bz>

Merge pull request #386 from ceph/RM-14646

[RM-14646] check for upstart before checking for systemd

[RM-14646] check for upstart before checking for systemd

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

Merge pull request #384 from branto1/wip-disable-pkg-timeout

ceph-deploy: Disable timeout for pkg install/remove procedures

[BZ-1302675] Disable timeout for pkg install/remove procedures

Do not timeout too early, here. If we use the pkg command to install
e.g. ceph-selinux or some other package with long post script we can
easily timeout in 5 minutes, disabling the timeout completely for the
pkg install procedure makes this much more safe.

The same holds for pkg removal.

Signed-off-by: Boris Ranto <branto@redhat.com>

Merge pull request #385 from ceph/init-fallback

Init fallback for upstart

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

[BZ-1282484] use is_upstart detection on debian init check

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

[BZ-1282484] tests for the new is_upstart utility

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

[BZ-1282484] use the new utility when trying to stop a monitor

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

[BZ-1282484] create a utility to infer if a server uses upstart

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

Merge pull request #382 from vumrao/wip-vumrao-14355

[RM-14355] Remove "filestore xattr use omap = true"

[RM-14355] Remove "filestore xattr use omap = true"

This patch removes option "filestore_xattr_use_omap"
from default ceph.conf global section,
as this option was removed in #7408.

Cleanup #14355

Signed-off-by: Vikhyat Umrao <vumrao@redhat.com>