mon: do not hardcode ipv4

Problem: fail to deploy a containerized Ceph cluster with ipv6
Solution: do not hardcode ipv4 when bootstrapping the container.

Now use ip_version: ipv6 to get a containerized cluster deployed with
ipv6.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1451786
Signed-off-by: Sébastien Han <seb@redhat.com>

Merge pull request #1532 from ceph/bpk-script

contrib: add backport script

contrib: add backport script

To ease the backport, I wrote a quick script.

Usage: ./backport.sh.sh stable-2.2
6892670d317698771be7e96ce9032bc27d3fd1e5 my-work

We can also pass multiple commits.
Follow up on @ktdreyer write up here:
https://github.com/ceph/ceph-ansible/pull/1529

Signed-off-by: Sébastien Han <seb@redhat.com>

Merge pull request #1546 from ceph/testinfra-version

tests: pin testinfra to 1.6.0

tests: pin testinfra to 1.6.0

1.6.1 fails with 'testinfra is in an unsupported or invalid wheel'

See: https://github.com/philpep/testinfra/issues/201

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #1531 from ceph/wip-1495

docker: Retry OSD disk prepare to workaround race condition

Merge pull request #1513 from flokli/monitor_interface

monitor_interface: document need to use monitor_address when using IPv6

Merge pull request #1528 from ceph/control-path

Control path

Merge pull request #1527 from ceph/piwi3910-master

added support for Sandisk FusionIO devices

Merge pull request #1500 from yanyixing/master

ignore error when key is not exist

Merge pull request #1526 from ceph/wip-rgw-default-quota-set-testing

test: check if default bucket object quota is applied

Docker: rm container before retry of ceph osd prepare

In addition to `196fa7e` this commit check if a container has been
already launched and delete it before retrying the ceph osd prepare
process.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>

test: check if default bucket object quota is applied

Resolves: rhbz#1391500

Signed-off-by: Ali Maredia <amaredia@redhat.com>

ansible: add a note in ansible.cfg

Comments inside this file must be set BEFORE the option. NOT after the
option, otherwise the comment will be interpreted as a value to that
option.

Signed-off-by: Sébastien Han <seb@redhat.com>

ansible: fix control path line

If the comment is put after the line then it is interpreted so we need
to move it before and have a dedicated line.

Signed-off-by: Sébastien Han <seb@redhat.com>

added support for Sandisk FusionIO devices

Signed-off-by: Pascal Watteel <pascal.watteel@emc.com>

Merge pull request #1522 from ceph/rhcs-jewel

tests: rhcs should set ceph_stable_release to jewel

Merge pull request #1516 from ceph/docker-registry

generate_group_vars_sample: merge role variables

tests: rhcs should set ceph_stable_release to jewel

rhcs is based off of jewel, so we need to set this var in the tests so
that the ceph-mgr role is skipped.

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #1512 from ceph/ansible-ssh-control-path

ansible: shorten default control_path

group_vars: resync all.yml.sample with ceph-docker-common

Signed-off-by: Sébastien Han <seb@redhat.com>

generate_group_vars_sample: ability to merge roles

Problem: the meta declaration just includes the role, it does nothing
with the group_vars. For ansible to use files defined in group_vars/ the
name of the file must match a host group. Like mons, osds, etc. There is
no group docker-common so the variables defined there are never used, as
proved by https://bugzilla.redhat.com/show_bug.cgi?id=1447179 and the
ansible documentation.

Solution: bring the ability to merge roles files. So now by default,
ceph-docker-common and ceph-common will go into all.yml.sample

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1447179
Signed-off-by: Sébastien Han <seb@redhat.com>

generate_group_vars_sample: do not use expr

As mentionned in https://github.com/koalaman/shellcheck/wiki/SC2003
expr is antiquated.

Signed-off-by: Sébastien Han <seb@redhat.com>

generate_group_vars_sample.sh: fix script indent

Using indent of 2 on the entire script.

Signed-off-by: Sébastien Han <seb@redhat.com>

rgw: resync group_vars

At some point, during a commit changing `roles/ceph-rgw/defaults/main.yml`
we forgot to run `generate_group_vars_sample.sh`.

Signed-off-by: Sébastien Han <seb@redhat.com>

Merge pull request #1518 from ceph/pgs-error-message

ceph-common: improve error message on restart osd daemon script

Merge pull request #1515 from ceph/mgr-key

mon: do not create mgr key on jewel

ceph-common: improve error message on restart osd daemon script

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

mon: do not create mgr key on jewel

The CI on Docker is reporting the following error:

STDERR:
Error EINVAL: bad entity name

This is due to the fact that this auth entity name does not exist on
Jewel so we should not create that key when running Jewel containers.

Fixes: https://github.com/ceph/ceph-ansible/issues/1514
Signed-off-by: Sébastien Han <seb@redhat.com>

Merge pull request #1507 from ceph/rpm-strip-coreos

rpm: do not package coreos-related files

Merge pull request #1511 from ceph/rm-ansible-install

ceph-aio-no-vagrant.sh: check if ansible is installed

docker: Retry osd disk prep to workaround race condition

Fixes: https://github.com/ceph/ceph-ansible/issues/1495
Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1510 from ktdreyer/ansible-2.2.3

tox: bump ansible version to 2.2.3

monitor_interface: document need to use monitor_address when using IPv6

Already documented in the Red Hat Ceph Storage 2 Installation Guide
for Red Hat Enterprise Linux, but not here

Signed-off-by: Florian Klink <flokli@flokli.de>

ansible: shorten default control_path

Default ansible control_path option is too long, so we shorten it by
changing the ansible.cfg file.

For more info see: https://github.com/ansible/ansible/issues/11536
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1447569
Signed-off-by: Sébastien Han <seb@redhat.com>

Merge pull request #1508 from ceph/wip-rgw-conf-var-move

rgw: move default bucket quota conf vars to global

ceph-aio-no-vagrant.sh: check if ansible is installed

Since we don't provide install-ansible.sh anymore we just check if it
exists. If not we fail and ask for installation.

Signed-off-by: Sébastien Han <seb@redhat.com>

tox: bump ansible version to 2.2.3

The Ansible team has released v2.2.3, which fixes two CVEs:

CVE-2017-7481
CVE-2017-7466

https://www.ansible.com/security

rpm: do not package coreos-related files

These are unnecessary on RPM-based platforms, and it makes it harder to
audit the RPM-packaged files for security when we ship things that are
unneeded.

rgw: move default bucket quota conf vars to global

"rgw override bucket index max shards" and
"rgw bucket default quota max objects" were in the
client section of the ceph.conf and not being
applied, this commit moves them to global

Resolves: bz#1391500

Signed-off-by: Ali Maredia <amaredia@redhat.com>

Merge pull request #1505 from ceph/remove-install-ansible-script

remove the install-ansible.sh script

remove the install-ansible.sh script

I'm unsure why this is needed, users should be able to install ansible
on their own in the method they prefer.

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #1460 from albertomurillo/clearlinux

Add clearlinux to supported platforms

Merge pull request #1504 from ceph/docker-osd-typo

ceph-osd: fix typo in containerized OSD systemd unit

ceph-osd: fix typo in containerized OSD systemd unit

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #1502 from ceph/update-test-stable-release

tests: update default of CEPH_STABLE_RELEASE to kraken

tests: update default of CEPH_STABLE_RELEASE to kraken

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #1484 from ceph/wip-rgw-testing

rgw: test functionality of conf vars and pool creation for tuning

Merge pull request #1501 from ceph/try-jewel-xenial-container

tests: use the xenial container for jewel instead of centos7

tests: use the xenial container for jewel instead of centos7

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

rgw: test functionality of conf vars and pool creation for tuning

Signed-off-by: Ali Maredia <amaredia@redhat.com>

ignore error when key is not exist

Merge pull request #1499 from ceph/fix_1494

Common: Do not install ceph-mgr packages on jewel

tests: all tests must use the node fixture

Otherwise the logic we have for skipping tests does not work.

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

tests: do not test mgr nodes on jewel

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Common: Do not install ceph-mgr packages on jewel

ceph-mgr tasks has to be skipped on jewel.

Fix: #1494

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>

Merge pull request #1492 from ceph/add-luminous

ceph-common: add luminous to ceph_release_num

Merge pull request #1468 from ceph/logging

ansible: add logging path

ceph-common: add luminous to ceph_release_num

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #1482 from ceph/remove-debian-ceph-packages

ceph-common: remove debian_ceph_packages

Merge pull request #1490 from ceph/insecure-registry

tests: allow for insecure docker registries when testing rhcs

tests: allow for insecure docker registries when testing rhcs

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #1487 from ceph/docker-jewel-tests

tests: actually test jewel when we say we are

docs: update tox docs to reflect the removed environment variables

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

tests: switch to using key/value extra-vars instead of json

I continue to have issues with extra-vars as json. The latest issue
being that the ceph_docker_image_tag config option included in the json
was being ignored. I can't find the root cause, by using the key/value
format seems to work.

I've also removed several options here to simply the interface. We can
add those back if they become necessary.

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

tests: actually test a jewel container for the jewel-* docker tests

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #1485 from ceph/docker-osd-tests

tests: test all docker osds are up and in

tests: test that all docker OSDs are up and in

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

tests: adds a way to mark docker only tests

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

ceph-common: remove debian_ceph_packages

We shouldn't need this anymore as the upgrade bug that
debian_ceph_packages was used to workaround should have
been fixed as of jewel.

See https://github.com/ceph/ceph-ansible/issues/1481 for more
detailed information.

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #1477 from ceph/quick_fix

Common: Fix condition on osd restart handler

Common: Fix condition on osd restart handler

Fix the condition on the osd restart handler.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>

Merge pull request #1476 from WingkaiHo/improve-shrink-osd.yml

improve shrink-osd.yml can shrink osd when disk damage

improve shrink-osd.yml can shrink osd when disk damage

Merge pull request #1458 from ceph/wip-rgw-tuning

rgw: set tuning parameters

rgw: set tuning parameters

Change civetweb_num_thread default to 100

Add capability to override number of pgs for
rgw pools.

Add ceph.conf vars to enable default bucket
object quota at users choosing into the ceph.conf.j2
template

Resolves: rhbz#1437173
Resolves: rhbz#1391500

Signed-off-by: Ali Maredia <amaredia@redhat.com>

Merge pull request #1464 from ceph/purge-ceph-target

purge-cluster: remove unneeded tasks

purge-cluster: check for any running ceph process after purge

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

ansible: add logging path

Ability to log ansible plays in a log file. Default location is
/var/log/ansible.log
Be sure the user running Ansible has permissions on the logfile.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1445065
Signed-off-by: Sébastien Han <seb@redhat.com>

Merge pull request #1466 from ceph/upgrade-calamari

mon: upgrade calamari when running rolling_update playbook

Add clearlinux to supported platforms

Signed-off-by: Alberto Murillo Silva <alberto.murillo.silva@intel.com>

Merge pull request #1455 from ceph/restart_daemons

Common: Restore check_socket

Common: Restore check_socket

Restore the check_socket that was removed by `5bec62b`.
This commit also improves the logging in `restart_*_daemon.sh` scripts

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>

mon: upgrade calamari when running rolling_update playbook

Prior to this change, ansible was only checking for the existence of the
package, now if upgrade_ceph_packages is true this means we are
performing an upgrade.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1442016
Signed-off-by: Sébastien Han <seb@redhat.com>

Merge pull request #1461 from ceph/wip-remove-osd-directory-scenario

remove osd directory scenario

remove osd directory scenario

Proof-of-concept clusters or actual production clusters will never want to use this. We also do not test it anywhere for this same reason.

Signed-off-by: Gregory Meno <gmeno@redhat.com>

purge-cluster: we don't support sysv or upstart anymore

Now that ceph-ansible only supports > jewel we don't need
to bother with sysv or upstart

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

purge-cluster: do not need to check for running ceph processes

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

purge-cluster: no need to remove ceph.target

The package uninstalls will stop ceph.target

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

Merge pull request #1463 from ceph/mgr-rhcs

tests: mgr nodes should have the ceph-mon repo install

Merge pull request #1459 from jcsp/wip-19713

mon: update mgr key capabilities

tests: mgr nodes should have the ceph-mon repo install

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

mon: update mgr key capabilities

This is to allow ceph-mgr daemons to remote control
osd and mds daemons with MCommand messages.

Fixes: http://tracker.ceph.com/issues/19713
Signed-off-by: John Spray <john.spray@redhat.com>

Merge pull request #1456 from netservers/master

Apparmor on Ubuntu Xenial will not permit containers to mount devices.

Apparmor on Ubuntu Xenial will not permit containers to mount devices, even with CAP SYS_ADMIN.

Merge pull request #1453 from ceph/docs-fixtures

[docs] fixtures and Python tests

Merge pull request #1454 from ceph/tox-docs

docs: write docs for tox.ini file

docs: review feedback on testing/tox.rst

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

docs: fix rst error in scenarios.rst

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

docs: write docs for tox.ini file

Signed-off-by: Andrew Schoen <aschoen@redhat.com>