Commit Graph

2352 Commits (cb504f6d88b0259ec8a4b6cad8bc9ed9a25c2d14)
 

Author SHA1 Message Date
Alfredo Deza cb504f6d88 docs: create a separate tox doc file
Signed-off-by: Alfredo Deza <adeza@redhat.com>
2017-04-07 12:07:26 -04:00
Alfredo Deza 41815f6b53 docs: use glossary for the :doc: directive
Signed-off-by: Alfredo Deza <adeza@redhat.com>
2017-04-07 12:06:36 -04:00
Alfredo Deza ae6349fd21 docs: add the testing glossary in the toctree
Signed-off-by: Alfredo Deza <adeza@redhat.com>
2017-04-07 12:06:16 -04:00
Alfredo Deza a5b2ede0ff docs: normalize the glossary vs. contents files
Signed-off-by: Alfredo Deza <adeza@redhat.com>
2017-04-07 12:05:55 -04:00
Alfredo Deza 14d46cb2be docs: create a separate docs tox.ini
Signed-off-by: Alfredo Deza <adeza@redhat.com>
2017-04-07 11:41:59 -04:00
Sébastien Han e2c8f1a404 Merge pull request #1412 from guits/fix_handlers
Common: Fix handlers that are not properly triggered.
2017-04-07 02:36:27 +02:00
Sébastien Han a5caccaefb Merge pull request #1416 from ceph/WingkaiHo-master
enable ceph-mon.target service after monitor start
2017-04-07 02:36:17 +02:00
Andrew Schoen 8e45c17a40 Merge pull request #1418 from ceph/docs
Docs: an initial take on only test documentation
2017-04-06 15:05:40 -05:00
Alfredo Deza 1f3d78e6c2 docs: initial structure for tests docs
Signed-off-by: Alfredo Deza <adeza@redhat.com>
2017-04-06 13:17:54 -04:00
Alfredo Deza 0d0e427a6c docs create a separate glossary file
Signed-off-by: Alfredo Deza <adeza@redhat.com>
2017-04-06 13:17:54 -04:00
Alfredo Deza 20210c35de docs: the master doc is the glossary file
Signed-off-by: Alfredo Deza <adeza@redhat.com>
2017-04-06 13:17:54 -04:00
Alfredo Deza 1ad331b026 docs: ignore build contents from git
Signed-off-by: Alfredo Deza <adeza@redhat.com>
2017-04-06 13:11:52 -04:00
Alfredo Deza 1a181fff4c docs: initial take on doc structure
Signed-off-by: Alfredo Deza <adeza@redhat.com>
2017-04-06 13:11:52 -04:00
Andrew Schoen f02679e440 Merge pull request #1417 from ceph/docker-env-vars
tests: add docker environment variables to tox.ini
2017-04-06 12:11:16 -05:00
Andrew Schoen 16e889c30a 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>
2017-04-06 10:29:16 -05:00
WingkaiHo 6ad8288c91 enable ceph-mon.target service after monitor start
ceph-create-keys unit file was removed here:

* 8bcb4646b6
* dc5fe8d415

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>
2017-04-06 16:58:32 +02:00
Guillaume Abrioux 5bec62ba7c 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>
2017-04-06 16:19:58 +02:00
Sébastien Han e04bfdc4f0 Merge pull request #1415 from dahorak/osd-autodiscovery-fix-check-holders
osd autodiscovery mode: fix holders detection
2017-04-06 10:56:42 +02:00
Daniel Horak ce06dc1460 osd autodiscovery mode: fix holders detection
Small fix for (probably copy&paste) issue from 42ffe6301.

Signed-off-by: Daniel Horak <dahorak@redhat.com>
2017-04-06 09:11:32 +02:00
Alfredo Deza ff34513c6a Merge pull request #1414 from ceph/scenario-alias
tests: add an alias of cluster for the centos7_cluster scenario
2017-04-05 17:42:06 -04:00
Andrew Schoen 4ca0f7fa71 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>
2017-04-05 13:36:51 -05:00
Sébastien Han 3c59ed58a5 Merge pull request #1411 from ceph/bump-ansible
tests: bump the version of ansible to 2.2.2
2017-04-05 16:03:20 +02:00
Sébastien Han 20a8ce90cc Merge pull request #1409 from ceph/auto-dm
osd: autodiscovery mode, use holders to detect device
2017-04-05 15:58:41 +02:00
Sébastien Han 716457c6f7 Merge pull request #1410 from ceph/kraken-tests
tests: run all tests for jewel and kraken
2017-04-05 13:53:17 +02:00
Alfredo Deza 3f6cdbb646 tests: bump the version of ansible to 2.2.2
Signed-off-by: Alfredo Deza <adeza@redhat.com>
2017-04-05 07:38:58 -04:00
Sébastien Han 42ffe63017 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>
2017-04-04 10:37:14 +02:00
Andrew Schoen 1b6b6d6255 tests: run all tests for jewel and kraken
Signed-off-by: Andrew Schoen <aschoen@redhat.com>
2017-04-03 16:11:55 -05:00
Sébastien Han 5ac072699a Merge pull request #1404 from ceph/check-osd-id
playbook: homogenize the way list osd ids
2017-03-30 14:52:58 +02:00
Sébastien Han c37aaa41f4 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>
2017-03-30 11:51:38 +02:00
Sébastien Han a3ca0b3eb3 Merge pull request #1399 from guits/fix_ntp_install
Common: Do not install ntp when ntp_service_enabled is false
2017-03-29 11:50:27 +02:00
Guillaume Abrioux 4e62627202 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>
2017-03-28 19:51:34 +02:00
Sébastien Han b57c51fda2 Merge pull request #1397 from ceph/debug
vagrant: add debug option
2017-03-28 12:15:49 +02:00
Sébastien Han 2d0565b072 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>
2017-03-28 12:00:32 +02:00
Sébastien Han 35a90ae283 Merge pull request #1386 from WingkaiHo/master
Create recover-osds-after-ssd-journal-failure.yml
2017-03-28 09:50:39 +02:00
Sébastien Han 6a9a32de82 Merge pull request #1391 from k0ste/newfeature
Resolve issues when groups names not in default value.
2017-03-27 18:48:33 +02:00
Sébastien Han 6e3c0971a9 Merge pull request #1396 from guits/fix_1374
BugFix: Fix #1374
2017-03-27 18:13:51 +02:00
Konstantin Shalygin 1662976fc0
Resolve issues when groups names not in default value. 2017-03-27 21:44:30 +07:00
Guillaume Abrioux a99e04a9b7 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>
2017-03-27 16:24:07 +02:00
Sébastien Han 5b617fbae5 Merge pull request #1394 from concubidated/master
ceph-common: update sysctl file location
2017-03-27 11:12:15 +02:00
WingkaiHo a0ed14fe88 remove the change 2017-03-27 11:06:32 +08:00
WingkaiHo 3366e86966 enable ceph-mon.target service after monitor start, fix the ceph monitor service do not start when the ceph reboot 2017-03-27 11:01:35 +08:00
WingkaiHo ac1498b0d7 Merge https://github.com/ceph/ceph-ansible 2017-03-27 10:50:38 +08:00
Concubidated 0454d540b0 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>
2017-03-24 12:59:34 -07:00
Andrew Schoen bbe6f752e8 Merge pull request #1389 from ceph/downstream-testing
tests: enable testing of rhcs
2017-03-24 10:09:12 -05:00
Andrew Schoen f2aaaa4970 tests: change ceph/ubuntu-xenial boxes to rhel7
Signed-off-by: Andrew Schoen <aschoen@redhat.com>
2017-03-23 08:43:11 -05:00
Andrew Schoen 5c34c9ccf1 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>
2017-03-23 06:42:43 -05:00
WingkaiHo ebb56ccebf command module instead shell 2017-03-23 17:38:41 +08:00
WingkaiHo 2d44c1cee6 remove service enable 2017-03-23 15:28:14 +08:00
WingkaiHo 14c189fee5 break it into lines since you already use the string block synta and fix disable it here and enable again in later task 2017-03-23 14:49:10 +08:00
WingkaiHo 62c37042fe remove this detection and simply rely on {{ cluster }} 2017-03-23 09:22:06 +08:00