]> git.apps.os.sepia.ceph.com Git - ceph-ansible.git/log
ceph-ansible.git
8 years agonfs: add automated testing for nfs-ganesha roles
Ali Maredia [Mon, 21 Aug 2017 17:36:38 +0000 (13:36 -0400)]
nfs: add automated testing for nfs-ganesha roles

Signed-off-by: Ali Maredia <amaredia@redhat.com>
8 years agonfs-ganesha: add dev, stable, and rhcs nfs-ganesha's for ceph-nfs role
Ali Maredia [Mon, 21 Aug 2017 12:38:21 +0000 (08:38 -0400)]
nfs-ganesha: add dev, stable, and rhcs nfs-ganesha's for ceph-nfs role

Signed-off-by: Ali Maredia <amaredia@redhat.com>
8 years agoMerge pull request #1803 from ceph/disk-auto
Sébastien Han [Thu, 7 Sep 2017 20:51:40 +0000 (22:51 +0200)]
Merge pull request #1803 from ceph/disk-auto

ceph-osd: fix autodetection activation

8 years agoMerge pull request #1867 from ceph/rhcs-version
Andrew Schoen [Thu, 7 Sep 2017 15:51:48 +0000 (10:51 -0500)]
Merge pull request #1867 from ceph/rhcs-version

tests: set ceph_stable_release to luminous for rhcs

8 years agoceph-osd: fix autodetection activation 1803/head
Sébastien Han [Thu, 24 Aug 2017 09:17:56 +0000 (11:17 +0200)]
ceph-osd: fix autodetection activation

Prior to this patch this activation sequence for autodetection was
always skipped because we were asking to activate on device without
partitions, which doesn't make sense.

We also fix the way we lookup for a device, since the data partition is
always numbered 1, we take the min element of the dict.

Closes: https://github.com/ceph/ceph-ansible/issues/1782
Signed-off-by: Sébastien Han <seb@redhat.com>
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
8 years agoMerge pull request #1842 from ceph/purge
Sébastien Han [Thu, 7 Sep 2017 15:36:34 +0000 (17:36 +0200)]
Merge pull request #1842 from ceph/purge

purge: get lockbox mountpoint and unmount it

8 years agotests: set ceph_stable_release to luminous for rhcs 1867/head
Andrew Schoen [Thu, 7 Sep 2017 14:45:56 +0000 (09:45 -0500)]
tests: set ceph_stable_release to luminous for rhcs

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
8 years agopurge: get lockbox mountpoint and unmount it 1842/head
Sébastien Han [Thu, 31 Aug 2017 16:22:34 +0000 (18:22 +0200)]
purge: get lockbox mountpoint and unmount it

Prior command was avoiding the lockbox mountpoint and the playbook was
failing with:

rmtree failed: [Errno 30] Read-only file system:
'/var/lib/ceph/osd-lockbox/4e9d8052-87c2-4fde-a56c-b8c108a3eefc/key-management-mode'

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1850 from fultonj/issue/1848
Sébastien Han [Thu, 7 Sep 2017 11:51:11 +0000 (13:51 +0200)]
Merge pull request #1850 from fultonj/issue/1848

Add option to create client keyring file but not import it

8 years agoMerge pull request #1859 from ceph/container-limit
Sébastien Han [Thu, 7 Sep 2017 10:51:34 +0000 (12:51 +0200)]
Merge pull request #1859 from ceph/container-limit

container: introduce resource limitation for containers

8 years agoMerge pull request #1862 from ceph/fail-ansible
Sébastien Han [Thu, 7 Sep 2017 06:44:01 +0000 (08:44 +0200)]
Merge pull request #1862 from ceph/fail-ansible

fail if ansible version < 2.3

8 years agofail if ansible version < 2.3 1862/head
Sébastien Han [Wed, 6 Sep 2017 15:52:49 +0000 (17:52 +0200)]
fail if ansible version < 2.3

We only test and support 2.3.x at the moment.

Closes: https://github.com/ceph/ceph-ansible/issues/1858
Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1812 from ceph/switch-migration-conta
Sébastien Han [Thu, 7 Sep 2017 05:30:34 +0000 (07:30 +0200)]
Merge pull request #1812 from ceph/switch-migration-conta

switch-from-non-containerized-to-containerized: mask unit files

8 years agotests: force docker variable for switch-to-containers scenario 1812/head
Guillaume Abrioux [Tue, 5 Sep 2017 13:18:01 +0000 (15:18 +0200)]
tests: force docker variable for switch-to-containers scenario

we need to force the value of `docker` variable which is initially set
to `false` since it's a migration from non-containerized to
containerized cluster.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
8 years agoAdd option to create client keyring file but not import it 1850/head
John Fulton [Sat, 2 Sep 2017 01:30:04 +0000 (01:30 +0000)]
Add option to create client keyring file but not import it

Add new boolean parameter for client config create_key_file_only
with a default of false. When create_key_file_only is true, the
client tasks to connect to the external ceph cluster to verify
the key `ceph auth import` the key are skipped.

Fixes: #1848
8 years agocontainer: introduce resource limitation for containers 1859/head
Sébastien Han [Wed, 6 Sep 2017 08:53:30 +0000 (10:53 +0200)]
container: introduce resource limitation for containers

This can be controlled via 2 options:

* ceph_$DAEMON_docker_memory_limit
* ceph_$DAEMON_docker_cpu_limit

All daemons default to 1GB for memory and 1 CPU by default.
Recommendations from:
https://access.redhat.com/documentation/en-us/red_hat_ceph_storage/2/html/red_hat_ceph_storage_hardware_guide/minimum_recommendations

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1860 from ceph/ssh-timeout
Guillaume Abrioux [Wed, 6 Sep 2017 09:36:17 +0000 (11:36 +0200)]
Merge pull request #1860 from ceph/ssh-timeout

ansible.cfg: increase ssh timeout

8 years agoansible.cfg: increase ssh timeout 1860/head
Sébastien Han [Wed, 6 Sep 2017 09:25:47 +0000 (11:25 +0200)]
ansible.cfg: increase ssh timeout

CI slaves are slow so by setting a higher value we can avoid the following error:
Timeout (12s) waiting for privilege escalation prompt:

Now we wait for 32 sec...

More info here: https://github.com/ansible/ansible/issues/14426

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1854 from ceph/add_mgr_node
Sébastien Han [Tue, 5 Sep 2017 14:17:40 +0000 (16:17 +0200)]
Merge pull request #1854 from ceph/add_mgr_node

tests: Add mgr node for all scenarios

8 years agoswitch-from-non-containerized-to-containerized: mask unit files
Sébastien Han [Fri, 25 Aug 2017 12:36:07 +0000 (14:36 +0200)]
switch-from-non-containerized-to-containerized: mask unit files

We must mask the image so we are sure that even if the system reboots
then the OSDs won't start.

Also remove Ceph udev rules if found on the system prior to deploy
containers. If we don't do this we are exposed to conflicts between udev
rules and sytemd unit files.

Also add the CI will now test the migration from a non-containerized cluster to a
containerized cluster.

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1853 from ceph/fix-prepare
Sébastien Han [Tue, 5 Sep 2017 11:59:40 +0000 (13:59 +0200)]
Merge pull request #1853 from ceph/fix-prepare

ceph-osd: do not re-prepare if already prepared

8 years agoMerge pull request #1819 from ceph/no-container-log
Sébastien Han [Tue, 5 Sep 2017 09:47:11 +0000 (11:47 +0200)]
Merge pull request #1819 from ceph/no-container-log

ceph-docker-common: do not log inside the container

8 years agotests: Add mgr node for all scenarios 1854/head
Guillaume Abrioux [Mon, 4 Sep 2017 21:58:56 +0000 (23:58 +0200)]
tests: Add mgr node for all scenarios

With Luminous we need to have mgr daemon.
This commit adds an mgr daemon for all scenarios.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
8 years agoceph-osd: do not re-prepare if alreadyy prepared 1853/head
Sébastien Han [Mon, 4 Sep 2017 21:22:12 +0000 (23:22 +0200)]
ceph-osd: do not re-prepare if alreadyy prepared

I forgot to re-add the partition check while refactoring the osd

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1851 from ceph/shrkin-time
Sébastien Han [Tue, 5 Sep 2017 07:17:22 +0000 (09:17 +0200)]
Merge pull request #1851 from ceph/shrkin-time

shrink-mon: wait a little bit for the mon to be out

8 years agoshrink-mon: wait a little bit for the mon to be out 1851/head
Sébastien Han [Mon, 4 Sep 2017 20:13:17 +0000 (22:13 +0200)]
shrink-mon: wait a little bit for the mon to be out

Monitor removal from the monmap is not immediate, so let's wait a little
bit and then fail if the monitor is still in the monmap.
We try twice in total with 10 sec intervals.

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1852 from ceph/fix-client
Sébastien Han [Mon, 4 Sep 2017 21:08:41 +0000 (23:08 +0200)]
Merge pull request #1852 from ceph/fix-client

client: copy admin key so we can create pools and keys

8 years agoclient: copy admin key so we can create pools and keys 1852/head
Sébastien Han [Mon, 4 Sep 2017 21:03:58 +0000 (23:03 +0200)]
client: copy admin key so we can create pools and keys

Needed when user_config is set to true

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1849 from ceph/client-external v3.0.0rc6
Sébastien Han [Fri, 1 Sep 2017 22:57:24 +0000 (00:57 +0200)]
Merge pull request #1849 from ceph/client-external

client: do not copy admin key by default

8 years agoclient: do not copy admin key by default 1849/head
Sébastien Han [Fri, 1 Sep 2017 22:52:55 +0000 (00:52 +0200)]
client: do not copy admin key by default

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1847 from ceph/luminous
Sébastien Han [Fri, 1 Sep 2017 22:48:12 +0000 (00:48 +0200)]
Merge pull request #1847 from ceph/luminous

ci: test luminous

8 years agoMerge pull request #1845 from ceph/rgw
Sébastien Han [Fri, 1 Sep 2017 18:27:21 +0000 (20:27 +0200)]
Merge pull request #1845 from ceph/rgw

ceph-rgw: fix systemd unit layout

8 years agoMerge pull request #1835 from ceph/remove-untest
Sébastien Han [Fri, 1 Sep 2017 17:59:30 +0000 (19:59 +0200)]
Merge pull request #1835 from ceph/remove-untest

infra playbook: move untested scenario to a new dir

8 years agoinfra playbook: move untested scenario to a new dir 1835/head
Sébastien Han [Wed, 30 Aug 2017 21:30:49 +0000 (23:30 +0200)]
infra playbook: move untested scenario to a new dir

Move untested/with few confidence playbooks in a untested-by-ci
directory.
Also removing this directory from the package build.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1461551
Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1836 from ceph/shrink-osd-mon
Sébastien Han [Fri, 1 Sep 2017 17:57:44 +0000 (19:57 +0200)]
Merge pull request #1836 from ceph/shrink-osd-mon

shrink mon and osd

8 years agoci: test luminous 1847/head
Sébastien Han [Fri, 1 Sep 2017 17:52:54 +0000 (19:52 +0200)]
ci: test luminous

Luminous is out so let's test it instead of Kraken.

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoshrink mon and osd 1836/head
Sébastien Han [Wed, 30 Aug 2017 22:07:28 +0000 (00:07 +0200)]
shrink mon and osd

Rework shrinking a monitor and an OSD playbook. Also adding test
scenario.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1366807
Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoceph-rgw: fix systemd unit layout 1845/head
Sébastien Han [Fri, 1 Sep 2017 17:02:48 +0000 (19:02 +0200)]
ceph-rgw: fix systemd unit layout

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1827 from andymcc/rgw_systemd_fix
Sébastien Han [Fri, 1 Sep 2017 16:12:23 +0000 (18:12 +0200)]
Merge pull request #1827 from andymcc/rgw_systemd_fix

Fix RGW systemd directory

8 years agoMerge pull request #1844 from ceph/rhcs-dev-ceph-release-fix v3.0.0rc5
Andrew Schoen [Thu, 31 Aug 2017 20:19:28 +0000 (15:19 -0500)]
Merge pull request #1844 from ceph/rhcs-dev-ceph-release-fix

common do not filter by distro when dev is set for figuring out ceph_release

8 years agocommon do not filter by distro when dev is set for figuring out ceph_release 1844/head
Alfredo Deza [Thu, 31 Aug 2017 20:18:08 +0000 (16:18 -0400)]
common do not filter by distro when dev is set for figuring out ceph_release

Signed-off-by: Alfredo Deza <adeza@redhat.com>
8 years agoMerge pull request #1839 from ceph/colonwq-update-docker-rgw-exec
Sébastien Han [Thu, 31 Aug 2017 17:47:16 +0000 (19:47 +0200)]
Merge pull request #1839 from ceph/colonwq-update-docker-rgw-exec

Update ceph_rgw_docker_extra_env to add bind ip

8 years agoMerge pull request #1838 from ceph/rgw-units
Sébastien Han [Thu, 31 Aug 2017 17:38:23 +0000 (19:38 +0200)]
Merge pull request #1838 from ceph/rgw-units

Rgw units

8 years agoMerge pull request #1841 from ceph/lvm-partitions
Andrew Schoen [Thu, 31 Aug 2017 17:09:19 +0000 (12:09 -0500)]
Merge pull request #1841 from ceph/lvm-partitions

lvm-osds: test with a partition and an lv as journals

8 years agoceph-defaults: fix handlers for mds and rgw 1838/head
Sébastien Han [Thu, 31 Aug 2017 09:22:33 +0000 (11:22 +0200)]
ceph-defaults: fix handlers for mds and rgw

The way we handle the restart for both mds and rgw is not ideal, it will
try to restart the daemon on the host that don't run the daemon,
resulting in a service file being created (see bug description).

Now we restart each daemon precisely and in a serialized fashion.

Note: the current implementation does NOT support multiple mds or rgw on
the same node.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1469781
Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1843 from ceph/useless-changed-task
Sébastien Han [Thu, 31 Aug 2017 16:29:13 +0000 (18:29 +0200)]
Merge pull request #1843 from ceph/useless-changed-task

ceph-common: remove useless changed task

8 years agoceph-common: remove useless changed task 1843/head
Sébastien Han [Thu, 31 Aug 2017 16:27:43 +0000 (18:27 +0200)]
ceph-common: remove useless changed task

There is no need to show a "changed" at the end of the play for a
"command" module task.

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agotests parted should create gpt labels on new disk 1841/head
Alfredo Deza [Thu, 31 Aug 2017 13:33:23 +0000 (09:33 -0400)]
tests parted should create gpt labels on new disk

But only for the first partition, so that a new label doesn't
blow away the previous partition created

Signed-off-by: Alfredo Deza <adeza@redhat.com>
8 years agoUpdate ceph_rgw_docker_extra_env to add bind ip 1839/head
Keith Schincke [Thu, 17 Aug 2017 17:25:20 +0000 (13:25 -0400)]
Update ceph_rgw_docker_extra_env to add bind ip

This patch adds passing the RGW_CIVETWEB_IP to the docker
container. This IP defaults to the value of radosgw_civetweb_bind_ip.
radosgw_civetweb_bind_ip default to ipv4.default

Without this value, the RGW containter will bind to 0.0.0.0

8 years agoceph-rgw: do not run a privileged rgw container
Sébastien Han [Thu, 31 Aug 2017 09:50:18 +0000 (11:50 +0200)]
ceph-rgw: do not run a privileged rgw container

There is no need for a privileged rgw container

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agotests conftest conditions the no_lvm_scenario marker
Alfredo Deza [Thu, 31 Aug 2017 12:48:24 +0000 (08:48 -0400)]
tests conftest conditions the no_lvm_scenario marker

Signed-off-by: Alfredo Deza <adeza@redhat.com>
8 years agotests osds units are not enabled in lvm scenarios
Alfredo Deza [Thu, 31 Aug 2017 12:47:42 +0000 (08:47 -0400)]
tests osds units are not enabled in lvm scenarios

Signed-off-by: Alfredo Deza <adeza@redhat.com>
8 years agorgw: cleanup old code and remove systemd condition
Sébastien Han [Thu, 31 Aug 2017 06:29:30 +0000 (08:29 +0200)]
rgw: cleanup old code and remove systemd condition

Remove the old check prior systemd.
We only support systemd so there is no need to run a condition on
systemd. The playbook will fail if systemd is not present.

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agotests: add a filesystem on /dev/sdc1 for lvm osd testing
Andrew Schoen [Wed, 30 Aug 2017 21:16:04 +0000 (16:16 -0500)]
tests: add a filesystem on /dev/sdc1 for lvm osd testing

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
8 years agoceph-osd: add support for --journal vg/lv for lvm osds
Andrew Schoen [Wed, 30 Aug 2017 20:55:16 +0000 (15:55 -0500)]
ceph-osd: add support for --journal vg/lv for lvm osds

This also updates the tests

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
8 years agotests: create 2 partitions on /dev/sdc for lvm scenario testing
Andrew Schoen [Wed, 30 Aug 2017 20:53:28 +0000 (15:53 -0500)]
tests: create 2 partitions on /dev/sdc for lvm scenario testing

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
8 years agoMerge pull request #1833 from ceph/dev-repository
Andrew Schoen [Wed, 30 Aug 2017 19:27:59 +0000 (14:27 -0500)]
Merge pull request #1833 from ceph/dev-repository

Dev repository fixes

8 years agocommon dev repos should not need to specify a 'release' 1833/head
Alfredo Deza [Wed, 30 Aug 2017 17:37:24 +0000 (13:37 -0400)]
common dev repos should not need to specify a 'release'

Signed-off-by: Alfredo Deza <adeza@redhat.com>
8 years agocommon: ceph_repository should not be rhcs or dev
Alfredo Deza [Wed, 30 Aug 2017 17:33:00 +0000 (13:33 -0400)]
common: ceph_repository should not be rhcs or dev

Signed-off-by: Alfredo Deza <adeza@redhat.com>
8 years agocommon: use the value of ceph_repository in the error message
Alfredo Deza [Wed, 30 Aug 2017 16:53:09 +0000 (12:53 -0400)]
common: use the value of ceph_repository in the error message

Signed-off-by: Alfredo Deza <adeza@redhat.com>
8 years agoMerge pull request #1741 from ceph/refactor-installation
Sébastien Han [Wed, 30 Aug 2017 15:42:29 +0000 (17:42 +0200)]
Merge pull request #1741 from ceph/refactor-installation

common: refactor installation method

8 years agoMerge pull request #1724 from ceph/container-multi-journal
Sébastien Han [Wed, 30 Aug 2017 15:41:42 +0000 (17:41 +0200)]
Merge pull request #1724 from ceph/container-multi-journal

osd: allow multi dedicated journals for containers

8 years agoMerge pull request #1831 from ceph/rbd-key
Sébastien Han [Wed, 30 Aug 2017 14:04:10 +0000 (16:04 +0200)]
Merge pull request #1831 from ceph/rbd-key

ceph-docker-common: re-organize stat ceph file

8 years agoceph-docker-common: re-organize stat ceph file 1831/head
Sébastien Han [Wed, 30 Aug 2017 12:44:34 +0000 (14:44 +0200)]
ceph-docker-common: re-organize stat ceph file

Use a single file to run the checks instead of duplicating code.

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1830 from ceph/ceph-client-ceph-conf
Sébastien Han [Wed, 30 Aug 2017 12:08:47 +0000 (14:08 +0200)]
Merge pull request #1830 from ceph/ceph-client-ceph-conf

site-docker.yml.sample: delegate facts

8 years agoosd: allow multi dedicated journals for containers 1724/head
Sébastien Han [Tue, 25 Jul 2017 15:54:26 +0000 (17:54 +0200)]
osd: allow multi dedicated journals for containers

Fix: https://bugzilla.redhat.com/show_bug.cgi?id=1475820
Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agosite-docker.yml.sample: delegate facts 1830/head
Sébastien Han [Wed, 30 Aug 2017 08:44:18 +0000 (10:44 +0200)]
site-docker.yml.sample: delegate facts

Now we can use --limit on the container deployment too. This is useful
while deploying client nodes.
e.g: ansible-playbook -i inventory -l clients site-docker.yml.sample

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agocommon: refactor installation method 1741/head
Sébastien Han [Thu, 3 Aug 2017 13:30:25 +0000 (15:30 +0200)]
common: refactor installation method

The installation process is now described as follow:

* you still have to choose a 'ceph_origin' installation method. The
origin can be a 'repository' (add a new repository), distro (it will use
the packages provided by the native repo source of your distribution),
local (only available on redhat system, it installs locally built
packages). This option is not well tested, so use it carefully

* if ceph_origin == 'repository' you will have to decide what kind of
repository you want to enable:
  - community: corresponds to the stable upstream/community version
  - enterprise: corresponds to the stable enterprise/downstream version
    (basically you are a red hat customer)
  - dev: it will install ceph from packages built out of the github
    development branches

Signed-off-by: Sébastien Han <seb@redhat.com>
Co-Authored-by: Guillaume Abrioux <gabrioux@redhat.com>
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
8 years agoMerge pull request #1829 from ceph/wip-osd-test-fix
Sébastien Han [Wed, 30 Aug 2017 08:10:37 +0000 (10:10 +0200)]
Merge pull request #1829 from ceph/wip-osd-test-fix

tests: fix duplicate osd service test

8 years agotests: fix duplicate osd service test 1829/head
Ali Maredia [Wed, 30 Aug 2017 01:24:13 +0000 (21:24 -0400)]
tests: fix duplicate osd service test

Signed-off-by: Ali Maredia <amaredia@redhat.com>
8 years agoMerge pull request #1824 from bengland2/fix-issue-1818-try2
Sébastien Han [Tue, 29 Aug 2017 21:15:58 +0000 (23:15 +0200)]
Merge pull request #1824 from bengland2/fix-issue-1818-try2

dont use devices var anymore, works for osd_auto_discover

8 years agoFix RGW systemd directory 1827/head
Andy McCrae [Tue, 29 Aug 2017 16:24:52 +0000 (17:24 +0100)]
Fix RGW systemd directory

The ceph RGW systemd services are actually named "ceph-radosgw" and not
"ceph-rgw", this patch fixes that for the systemd overrides file.

8 years agoMerge pull request #1826 from ceph/add-mimic v3.0.0rc4
Andrew Schoen [Tue, 29 Aug 2017 15:24:25 +0000 (10:24 -0500)]
Merge pull request #1826 from ceph/add-mimic

common: add mimic release facts

8 years agocommon: add mimic release facts 1826/head
Sébastien Han [Tue, 29 Aug 2017 15:21:37 +0000 (17:21 +0200)]
common: add mimic release facts

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1822 from ceph/rhcs-container-release
Sébastien Han [Tue, 29 Aug 2017 10:16:20 +0000 (12:16 +0200)]
Merge pull request #1822 from ceph/rhcs-container-release

ceph-docker-common: detect ceph version

8 years agoMerge pull request #1825 from ceph/fix-item
Sébastien Han [Tue, 29 Aug 2017 10:15:46 +0000 (12:15 +0200)]
Merge pull request #1825 from ceph/fix-item

ceph-docker-common: fix empty array

8 years agoMerge pull request #1814 from ceph/handler-defaults
Sébastien Han [Tue, 29 Aug 2017 09:09:35 +0000 (11:09 +0200)]
Merge pull request #1814 from ceph/handler-defaults

handler: default to empty array if task skipped

8 years agoceph-docker-common: fix empty array 1825/head
Sébastien Han [Mon, 28 Aug 2017 22:16:31 +0000 (00:16 +0200)]
ceph-docker-common: fix empty array

The list can not be evaluated properly if it containers '[]', which is
the case when using the filter "default([])". To fix this, we have to
properly merge the lists.

This is fixing the issue: "list object has no element 1"

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoceph-docker-common: detect ceph version 1822/head
Sébastien Han [Mon, 28 Aug 2017 21:23:36 +0000 (23:23 +0200)]
ceph-docker-common: detect ceph version

By detecting the ceph version running in the container we can easily
apply conditions like:
ceph_release_num.{{ ceph_release }} >= ceph_release_num.luminous

We do that already, in ceph-docker-common/tasks/fetch_configs.yml.

This fixes the error:

TASK [ceph-docker-common : register rbd bootstrap key]
******************************************************

fatal: [magna005]: FAILED! => {"failed": true, "msg": "The conditional
check 'ceph_release_num.{{ ceph_release }} >= ceph_release_num.luminous'
failed. The error was: error while evaluating conditional
(ceph_release_num.{{ ceph_release }} >= ceph_release_num.luminous):
'dict object' has no attribute 'dummy'\n\nThe error appears to have been
in
'/home/ubuntu/ceph-ansible/roles/ceph-docker-common/tasks/fetch_configs.yml':
line 2, column 3, but may\nbe elsewhere in the file depending on the
exact syntax problem.\n\nThe offending line appears to be:\n\n---\n-
name: register rbd bootstrap key\n  ^ here\n"}

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1486062
Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agodont use devices var anymore, works for osd_auto_discover 1824/head
Ben England [Mon, 28 Aug 2017 21:27:01 +0000 (17:27 -0400)]
dont use devices var anymore, works for osd_auto_discover

8 years agoMerge pull request #1821 from ceph/test-sitepackages
David Galloway [Mon, 28 Aug 2017 16:40:00 +0000 (12:40 -0400)]
Merge pull request #1821 from ceph/test-sitepackages

tests: always use sitepackages=True

8 years agotests: always use sitepackages=True 1821/head
Andrew Schoen [Mon, 28 Aug 2017 14:59:28 +0000 (09:59 -0500)]
tests: always use sitepackages=True

This is mostly important in rhcs testing so that our tests can use
packages installed on the distro.

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
8 years agoceph-docker-common: do not log inside the container 1819/head
Sébastien Han [Mon, 28 Aug 2017 10:04:49 +0000 (12:04 +0200)]
ceph-docker-common: do not log inside the container

Logging inside the container is not useful since it writes to the
overlayfs partition, resulting in potential performance degradation on
the container.

If you need to check the logs, just look at journald.

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1815 from ceph/container-key-perms
Sébastien Han [Mon, 28 Aug 2017 09:15:41 +0000 (11:15 +0200)]
Merge pull request #1815 from ceph/container-key-perms

ceph-docker-common: apply 0600 to key permissions

8 years agoMerge pull request #1517 from ceph/rolling
Sébastien Han [Mon, 28 Aug 2017 09:14:55 +0000 (11:14 +0200)]
Merge pull request #1517 from ceph/rolling

rolling_update: nicer way to set osd flags

8 years agohandler: default to empty array if task skipped 1814/head
Sébastien Han [Fri, 25 Aug 2017 16:00:03 +0000 (18:00 +0200)]
handler: default to empty array if task skipped

with_items is evaluated before the when condition so if the task that
registers the 'results' is skipped the task will fail with:

{"failed": true, "msg": "'dict object' has no attribute 'results'"}

Defaulting to an empty array fixes the issue.

Reverts: abdd66619e1d5c94f74e994bff667b5ccc6b529f
Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1482061
Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agorolling_update: nicer way to set osd flags 1517/head
Sébastien Han [Fri, 12 May 2017 13:59:52 +0000 (15:59 +0200)]
rolling_update: nicer way to set osd flags

Prior to this patch, we were applying the osd flags like this:

"
General pre tasks
Set flags
Upgrade OSDs on a host
Unset flags <-- this triggers pending scrub to start
Set flags
Upgrade OSDs on a hosts
Unset flags <-- this triggers pending scrub to start
.
.
.
General post tasks
"

Now instead, we apply the flag once before starting the OSD update and
unset them once the last OSD is finished.

"
General pre tasks
Set flags and wait for any scrubs to finish
Upgrade OSDs on a host
Upgrade OSDs on a host
.
.
.
Unset flags
General post tasks
"

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1450754
Signed-off-by: Sébastien Han <seb@redhat.com>
Co-Authored-by: Guillaume Abrioux <gabrioux@redhat.com>
8 years agoceph-docker-common: apply 0600 to key permissions 1815/head
Sébastien Han [Fri, 25 Aug 2017 16:13:09 +0000 (18:13 +0200)]
ceph-docker-common: apply 0600 to key permissions

Keys should only be readable and writable by their respective owners and that's all.

Closes: https://github.com/ceph/ceph-ansible/issues/1760
Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1811 from ceph/push-galaxy
Sébastien Han [Fri, 25 Aug 2017 12:54:09 +0000 (14:54 +0200)]
Merge pull request #1811 from ceph/push-galaxy

contrib: do not rework if tag exist

8 years agocontrib: do not rework if tag exist 1811/head
Sébastien Han [Fri, 25 Aug 2017 10:15:34 +0000 (12:15 +0200)]
contrib: do not rework if tag exist

We now compare local tags versus remote tags and do nothing if they both
exist.

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1810 from ceph/config-meta
Sébastien Han [Thu, 24 Aug 2017 22:17:08 +0000 (00:17 +0200)]
Merge pull request #1810 from ceph/config-meta

update meta for ansible galaxy

8 years agoupdate meta for ansible galaxy 1810/head
Sébastien Han [Thu, 24 Aug 2017 22:04:29 +0000 (00:04 +0200)]
update meta for ansible galaxy

Closes: https://github.com/ceph/ceph-ansible/issues/1637
Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1808 from ceph/role-path
Sébastien Han [Thu, 24 Aug 2017 21:49:41 +0000 (23:49 +0200)]
Merge pull request #1808 from ceph/role-path

ceph-mon: detect ANSIBLE_ROLES_PATH if present

8 years agoMerge pull request #1809 from ceph/no-sudo-fetch-dir
Sébastien Han [Thu, 24 Aug 2017 16:43:51 +0000 (18:43 +0200)]
Merge pull request #1809 from ceph/no-sudo-fetch-dir

ceph-config: when using local_action set become: false

8 years agoceph-config: when using local_action set become: false 1809/head
Andrew Schoen [Thu, 24 Aug 2017 15:05:46 +0000 (10:05 -0500)]
ceph-config: when using local_action set become: false

There should be no need to use sudo when writing or using these files.
It creates an issue when the user running ansible-playbook does not
have sudo privs.

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
8 years agoceph-mon: detect ANSIBLE_ROLES_PATH if present 1808/head
Sébastien Han [Thu, 24 Aug 2017 14:19:39 +0000 (16:19 +0200)]
ceph-mon: detect ANSIBLE_ROLES_PATH if present

Some deployments can't copy infrastructure playbooks outside of the
infrastructure-playbooks directory. Thus they use ANSIBLE_ROLES_PATH to
overcome this. However some roles have 'playbook_dir' hardcoded, which
results in wrong path since the execution comes from
infrastructure-playbooks. Basically the role triggered by a playbook
from infrastructure-playbooks believes that the roles are in
infrastructure-playbooks/roles. This commit fixes that.

Signed-off-by: Sébastien Han <seb@redhat.com>
8 years agoMerge pull request #1807 from ceph/rpm-build-fix
Alfredo Deza [Thu, 24 Aug 2017 13:24:00 +0000 (09:24 -0400)]
Merge pull request #1807 from ceph/rpm-build-fix

rpm update the DOC section to point to rst

8 years agorpm update the DOC section to point to rst 1807/head
Alfredo Deza [Thu, 24 Aug 2017 13:18:17 +0000 (09:18 -0400)]
rpm update the DOC section to point to rst

Signed-off-by: Alfredo Deza <adeza@redhat.com>
8 years agoMerge pull request #1806 from ceph/resync-group
Sébastien Han [Thu, 24 Aug 2017 11:42:06 +0000 (13:42 +0200)]
Merge pull request #1806 from ceph/resync-group

resync groups_vars

8 years agoresync groups_vars 1806/head
Sébastien Han [Thu, 24 Aug 2017 11:40:36 +0000 (13:40 +0200)]
resync groups_vars

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