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

Signed-off-by: Alfredo Deza <adeza@redhat.com>
(cherry picked from commit b04d18988e7f4c3da332af8941550ba0af18365a)

Merge pull request #1506 from ceph/backport-1505

backport: "remove the install-ansible.sh script #1505"

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>
(cherry picked from commit 1dc0fab21ca3126bc06aa54af4daae87feedeeee)

Merge pull request #1491 from ceph/backport-1490

backport: "tests: allow for insecure docker registries when testing rhcs #1490"

tests: allow for insecure docker registries when testing rhcs

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
(cherry picked from commit 2fada9bd6b0b0099f191be875edeee23a2fe3daa)

Conflicts:
tests/functional/rhcs_setup.yml

Merge pull request #1489 from ceph/backport-1487

Backport: "tests: actually test jewel when we say we are #1487"

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>
(cherry picked from commit e384b2498755e5537595158ca498e03013e33673)

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

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
(cherry picked from commit 1fcc3283d8891055ae394d03f3ded1f6a4c8ec64)

Merge pull request #1488 from ceph/backport-1427

tests: enable the rhcs-7-extras-nightly repo on test nodes

tests: enable the rhcs-7-extras-nightly repo on test nodes

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
(cherry picked from commit 35aab1274a5ec6ad696040965c87a87310bb019e)

Merge pull request #1486 from ceph/backport-1485

Backport: "tests: test all docker osds are up and in #1485"

tests: test that all docker OSDs are up and in

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
(cherry picked from commit 10d39e98dde5042d98d234887ead17ff7d278e55)

tests: adds a way to mark docker only tests

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
(cherry picked from commit 06b36fddf9689659b97c480b8b6644775fac907b)

Merge pull request #1478 from ceph/backport_1477

Backport: 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 #1474 from ceph/wip-2.2-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 #1473 from ceph/backport-1464

backport: "purge-cluster: remove unneeded tasks #1464"

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

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
(cherry picked from commit 5a3f95dfc1bc9bba7ad62999969997ce0a27da13)

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>
(cherry picked from commit 26bdd59f5d445db7991eecd0d13f7c8615bc4812)

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

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
(cherry picked from commit 7ca2bddcce76dd505b169457fa8727b62b861756)

purge-cluster: no need to remove ceph.target

The package uninstalls will stop ceph.target

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
(cherry picked from commit aac79df3b376afcb70b4e8888a72ad016079576b)

Merge pull request #1470 from ceph/backport-1466

backport-1466: mon: upgrade calamari when running rolling_update playbook

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>
(cherry picked from commit 84d96be197617ff8cdf8141e4a52657f172b2809)

Merge pull request #1467 from ceph/backport_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>

Merge pull request #1450 from ceph/backport-1439

Backport: 'tests: fix ceph_rhcs setup #1439'

Merge pull request #1451 from ceph/doc-static-dirs

Doc static dirs

docs: force an empty _templates dir so sphinx is happy

Signed-off-by: Alfredo Deza <adeza@redhat.com>
(cherry picked from commit 66a9817e56d69b046425dc04a03869fb9a9d8369)
(cherry picked from commit df315fef897705c77bf12780d31ddf17376388fb)

docs: force an empty _static dir so sphinx is happy

Signed-off-by: Alfredo Deza <adeza@redhat.com>
(cherry picked from commit aacbbf0efe802e9282793dfc8bf7b175596c5aad)
(cherry picked from commit d26ed99dcc1b930a3001ca1bc71f034b9da4f091)

tests: print contents of group_vars/all after modification

This is just nice to see in the test output so we know exactly what
configuration is going to be used.

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
(cherry picked from commit 0e6d89b9db1b4677db42bf04c88547ebc00e1194)

ceph-common: is ceph_rchs is True do not include install_on_redhat.yml

That task includes logic for upstream installs that we do not want to
run when deploying RHCS.

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
(cherry picked from commit 37d38b122bde02efc1bde0d61ceeee3999ebb7ff)

tests: set needed config in group_vars/all for rhcs testing

Instead of relying on environment variables and --extra-vars simply
modify the group_vars/all that ships with the specific testing scenario
to enable ceph_rchs testing.

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
(cherry picked from commit 386afbec83b72dcc3a552682d4376d1bf6a99417)

Conflicts:
tests/functional/rhcs_setup.yml

tests: remove CEPH_STABLE environment variable

I'm removing this because when we use an 'rhcs' scenario then we attempt
to set CEPH_STABLE=false as an environment variable. The issue with that
is because the value is coming from an environment variable it is always
treated as a string and ansible treats that as a boolean True. I plan to
set the ceph_stable value with our rhcs_setup.yml playbook instead of
relying on ---extra-vars and environment variables.

Related ansible issue: https://github.com/ansible/ansible/issues/17193

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
(cherry picked from commit 63983fa0b728571518ecdeca249ef37506a00a64)

Merge pull request #1425 from ceph/bump-kraken

common: bump ceph version to kraken

common: bump ceph version to kraken

Kraken has been out for a couple of weeks now and the CI can test both
Kraken and Jewel.

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

Merge pull request #1377 from ceph/ceph-mgr

mgr: add new role for ceph-mgr

test: add mgr bootstrap support

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

docker: fix monitors name

Prior to this change we were deploying a monitor using tis fqdn name but
we were checking its state and performing actions on it using its
shortname.

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

mgr: add new role for ceph-mgr

The Ceph Manager daemon (ceph-mgr) runs alongside monitor daemons, to
provide additional monitoring and interfaces to external monitoring and
management systems.

Only works as of the Kraken release.

Co-Authored-By: Guillaume Abrioux <gabrioux@redhat.com>
Signed-off-by: Sébastien Han <seb@redhat.com>

Merge pull request #1421 from ceph/tox-docs

tox-driven docs

Merge pull request #1422 from k0ste/newfeature

ceph-common: persist "disable_transparent_hugepage"

ceph-common: disable_transparent_hugepage now persist

docs: add missing sections/refs in scenarios.rst

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

docs: include testing conventions and testinfra in layout.rst

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

docs: fix typo in testing/index.rst

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

docs: include the tests and tox files in the glossary

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

docs: create a tests doc file

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

docs: create a separate tox doc file

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

docs: use glossary for the :doc: directive

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

docs: add the testing glossary in the toctree

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

docs: normalize the glossary vs. contents files

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

docs: create a separate docs tox.ini

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

Merge pull request #1412 from guits/fix_handlers

Common: Fix handlers that are not properly triggered.

Merge pull request #1416 from ceph/WingkaiHo-master

enable ceph-mon.target service after monitor start

Merge pull request #1418 from ceph/docs

Docs: an initial take on only test documentation

docs: initial structure for tests docs

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

docs create a separate glossary file

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

docs: the master doc is the glossary file

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

docs: ignore build contents from git

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

docs: initial take on doc structure

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

Merge pull request #1417 from ceph/docker-env-vars

tests: add docker environment variables to tox.ini

tests: add docker environment variables to tox.ini

This allows us to control the docker registry, image name and tag name
for the containers.

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

enable ceph-mon.target service after monitor start

ceph-create-keys unit file was removed here:

* https://github.com/ceph/ceph/commit/8bcb4646b6b9846bb965cdec3ca2a21eb3b26bab
* https://github.com/ceph/ceph/commit/dc5fe8d415858358bd0baf5d8dce0a753f5e0cea

As a consequence the systemctl preset command now fails to run since the
unit does not exist anymore. Due to the redirection in /dev/null we
don't know what's happening.

Ultimately the mon unit doesn't get enabled and the mon service won't
start after reboot.
Removing the old/non-existent unit makes the command succeed now.

ceph fix: https://github.com/ceph/ceph/pull/14226

Signed-off-by: WingkaiHo <sanguosfiang@163.com>
Co-Authored-By: Sébastien Han <seb@redhat.com>

Common: Fix handlers that are not properly triggered.

Until now, only the first task were executed.
The idea here is to use `listen` statement to be able to notify multiple
handler and regroup all of them in `./handlers/main.yml` as notifying an
included handler task is not possible.

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

Merge pull request #1415 from dahorak/osd-autodiscovery-fix-check-holders

osd autodiscovery mode: fix holders detection

osd autodiscovery mode: fix holders detection

Small fix for (probably copy&paste) issue from 42ffe6301.

Signed-off-by: Daniel Horak <dahorak@redhat.com>

Merge pull request #1414 from ceph/scenario-alias

tests: add an alias of cluster for the centos7_cluster scenario

tests: add an alias of cluster for the centos7_cluster scenario

When testing this downstream it makes more sense for this scenario to be
named just 'cluster' because have 'centos7' in the name is misleading.

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

Merge pull request #1411 from ceph/bump-ansible

tests: bump the version of ansible to 2.2.2

Merge pull request #1409 from ceph/auto-dm

osd: autodiscovery mode, use holders to detect device

Merge pull request #1410 from ceph/kraken-tests

tests: run all tests for jewel and kraken

tests: bump the version of ansible to 2.2.2

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

osd: autodiscovery mode, use holders to detect device

As reported in
https://github.com/ceph/ceph-ansible/issues/1403 when devices are held
by lvm and `osd_auto_discovery` is set to true, it's not enough to check
for a partition count = 0 since Ansible does not report.
This patch also looks for 'holders' which in a case of lvm corresponds
to the name of the pv. Now we also look for holders = 0.

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

tests: run all tests for jewel and kraken

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

Merge pull request #1404 from ceph/check-osd-id

playbook: homogenize the way list osd ids

playbook: homogenize the way list osd ids

Problem: too many different commands to do the same thing. The 'cut'
command on infrastructure-playbooks/purge-cluster.yml was also wrong.
This sed command from osixia in ceph-docker
https://github.com/ceph/ceph-docker/pull/580/ addresses all the
scenarios.

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

Merge pull request #1399 from guits/fix_ntp_install

Common: Do not install ntp when ntp_service_enabled is false

Common: Do not install ntp when ntp_service_enabled is false

ntp is still installed even if ntp_service_enabled is set to false.
That could be a problem if the time synchronization is managed by
something else than ceph-ansible or if you want to use different NTP
implementation as suggested in #1354.

Fixes: #1354
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
Signed-off-by: Guits <gabrioux@redhat.com>

Merge pull request #1397 from ceph/debug

vagrant: add debug option

vagrant: add debug option

If debug is set to true in vagrant_variables.yml then during the vagrant
provision phase Ansible will run with -vvvv option

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

Merge pull request #1386 from WingkaiHo/master

Create recover-osds-after-ssd-journal-failure.yml

Merge pull request #1391 from k0ste/newfeature

Resolve issues when groups names not in default value.

Merge pull request #1396 from guits/fix_1374

BugFix: Fix #1374

Resolve issues when groups names not in default value.

BugFix: Fix #1374

If a group of hosts is empty, (for instance 'mdss', in case of a
deployment without any mds node), the playbook will fails when trying
to restart service with `"'dict object' has no attribute u'XXX'"` error.

The idea here is to force the `with_items` statements in all included handler tasks
to get at least an empty array.

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

Merge pull request #1394 from concubidated/master

ceph-common: update sysctl file location

remove the change

enable ceph-mon.target service after monitor start, fix the ceph monitor service do not start when the ceph reboot

Merge https://github.com/ceph/ceph-ansible

ceph-common: update sysctl file location

systctl tuning should be in the sysctl.d directory. This creates
a seperation from what values were set specific to ceph, and what
values were set by the operator.

Signed-off-by: Tyler Brekke <tbrekke@redhat.com>

Merge pull request #1389 from ceph/downstream-testing

tests: enable testing of rhcs

tests: change ceph/ubuntu-xenial boxes to rhel7

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

tests: when testing rhcs default CEPH_STABLE=false

This will keep ceph-ansible from installing the upstream repo file on
the testing nodes.

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

command module instead shell

remove service enable

break it into lines since you already use the string block synta and fix disable it here and enable again in later task

remove this detection and simply rely on {{ cluster }}

tests: change hosts in first play in rhcs_setup to localhost

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

tests: set MTU to 1400 on test node interfaces

In the environment we were testing on, MTU was set to 1500 which causes
download failures of our yum repos. There might be a better way to set
this instead of doing it here in ansible.

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