Change modes to string values

Currently deploying a MON fails with "bad symbolic permission for mode"
errors due to the file/directory modes not being interpreted as octal
values.  This commit updates roles/ceph-common/tasks/main.yml to set
the file/directory modes to strings so they can be interpreted
correctly.

Closes issue #525

Merge pull request #524 from ceph/fix-typo

ceph-rgw: fix docker typo

ceph-rgw: fix docker typo

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

Merge pull request #521 from ceph/vagrant-container

ceph-ansible: allow to run containerized daemons

ceph-ansible: allow to run containerized daemons

run containerized daemons in virtual machines.
to enable it simply do:

`cp site-docker.yml.sample site-docker.yml`

and set `docker: true` in `vagrant_variables.yml`

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

Merge pull request #519 from ceph/restapi-meta

ceph-restapi: fix meta file

ceph-restapi: fix meta file

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

Merge pull request #512 from guits/quick_fix

Refact code using `set_fact` to handle 'legacy' versions

Refact code using `set_fact`

At the moment, all the tasks using the file module are duplicated to have differents ownerships depending on the fact `is_ceph_infernalis`.
The goal of this commit is to have a new logic for this:
- First set facts depending on the `is_ceph_infernalis` fact
- Create the files or directories using the setted facts as ownerships.

Merge pull request #516 from ceph/ansible-v2-warnings

ceph-common: replace 'sudo' by 'become'

ceph-common: replace 'sudo' by 'become'

to avoid warning from ansible v2. works for both `1.9.4` and `2.0.0.2`.

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

Merge pull request #514 from ceph/pkg-install-tags

Adds a package-install tag to all installation tasks

Adds a package-install tag to all installation tasks

We have a requirement to install the packages first without
configuration. These tags should allow us to target the tasks need to do
that.

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

Merge pull request #508 from mattt416/upgrade_flag

Control when ceph packages get updated

Merge pull request #511 from ceph/use-pythonuuid

ceph-common: use a generic way to create uuid

ceph-common: use a generic way to create uuid

as reported in #510 some systems don't have uuidgen installed so we
better use a more global way to generate it. It sounds like python
should be available in case uuidgen is not.
Otherwise we will have to find another way :)

closes #510

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

Control when ceph packages get updated

Currently, all the ceph package installation resources use
"state=latest", which means subsequent runs of the ceph playbooks
could result in ceph being upgraded if there are package updates
available in the selected repo.

This commit adds a new variable to ceph-common called
'upgrade_ceph_packages' which defaults to False.  This variable is used
in the package installation resources for ceph packages to determine if
the resource should use "state=present" or "state=latest".  If the
variable gets set to True, "state=latest" will be used.

Additionally, we update rolling_update.yml to override
upgrade_ceph_packages to true to permit package upgrades in this
context specifically.

Closes issue #506

Merge pull request #493 from ceph/sudo-test

CI does not use root, but sudo will do fine

fix type on ice template

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

Merge pull request #502 from ceph/use-dnf

use dnf when yum is not available

use dnf when yum is not available

fixes the `importError: No module named yum` see here:
https://jenkins.ceph.com/job/ceph-ansible-pull-requests/45/console

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

remove remote_user from test.yml so it uses the current user

We need this so that the CI tests with jenkins will use the correct
user with sudo access.

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

create a single-machine inventory for testing

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

CI does not use root, but sudo will do fine

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

Merge pull request #501 from ceph/common-defaults

Add defaults for ceph_stable_ice_tmp_path and ceph_stable_ice_kmod

Add defaults for ceph_stable_ice_tmp_path and ceph_stable_ice_kmod

It seems that in ansible 2.0 even if a task is skipped by it's `when`
clause not evaluating to true the variables in the play are still
rendered. Because these were not defined in defaults/main.yml ansible
was failing in installs/install_on_redhat where those variables are
being used in a `with_items` stanza.

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

Merge pull request #500 from ceph/python-pip

Fix pip package name on debian

Fix pip package name on debian

use python-pip and not pip

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

Merge pull request #498 from ceph/fsid-monitor-key

remove unecessary variables

remove unecessary variables

these variables will be created on the fly, there is no need to force
them.

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

Merge pull request #497 from ceph/vm-kybte

update group_vars/all.sample with min_free_kbytes

update group_vars/all.sample with min_free_kbytes

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

Merge pull request #490 from ceph/ansible-2

Necessary change to work on ansible v2

Necessary change to work on ansible v2

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

Merge pull request #489 from ceph/vagrant-virtualbox-updates

updates related to using vagrant and virtualbox for testing

update the README on using vagrant with virtualbox

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

Include os_tuning_params in the settings used for vagrant testing

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

adds os_tuning_params to the vagrant_variables.yml.sample file

When using virtualbox with vagrant the default set of os_tuning_params
was locking up my vms. Specifically the vm.vfs_cache_pressure setting.

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

Merge pull request #488 from PiotrProkop/fix

Adding python-setuptools dependency for CentOS 7.

Adding newline

Adding python-setuptools dependency for CentOS 7

Merge pull request #487 from mcsage/master

Changed vm.min_free_kbytes to 4 GB

Changed vm.min_free_kbytes to 4 GB

vm.min_free_kbytes was in bytes not in kbytes.

Merge pull request #485 from byronmccollum/conf-directory-permissions

Configurable Ceph Conf Directory Permissions

Configurable Ceph Conf Directory Permissions

This change allows for configurable Ceph Conf Directory permissions. This
is required for integrators of Ceph, like OpenStack Cinder, which needs to
read from /etc/ceph for operation.

Merge pull request #484 from ceph/clarify-ceph-origin-repo

Clarify ceph_origin usage

Clarify ceph_origin usage

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

Merge pull request #483 from bengland2/rhcs_repo_template

help ansible find repo template from installs subdirectory

help ansible find repo template from installs subdirectory

Merge pull request #478 from rootfs/systemd

systemd unit files fixes for infernalis

review feedback: convert to yml format

Signed-off-by: Huamin Chen <hchen@redhat.com>

make systemd enable work for osd instance

Signed-off-by: Huamin Chen <hchen@redhat.com>

make systemd enable work for mon instance

Signed-off-by: Huamin Chen <hchen@redhat.com>

Merge pull request #476 from Logan2211/master

allow customization of debian ceph stable repo url

adding repo url to group_vars/all

allow customization of debian ceph stable repo url

Merge pull request #474 from fcharlier/patch-1

Fix failing install-ansible test

Fix failing install-ansible test

Fix test for Red Hat Enterprise Server and add a default case.

Merge pull request #473 from ceph/fix-install-ansible

Fix install for rhel

Fix install for rhel

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

Merge pull request #472 from ceph/nmap-optionnal

Do not make nmap mandatory

Do not make nmap mandatory

We can not force people to have nmap installed so instead we raise a
warning is not present.

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

Merge pull request #471 from ceph/tiny-fix-nmap

Fix firewall check

Fix firewall check

Use command module instead of shell since we do not do anything fancy
here. Remove the duplicate register.

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

Merge pull request #469 from ceph/firewall-checks

Check for blocked ports

Check for blocked ports

As raised in #466 it is important in order to avoid unnecessary
troubleshooting to check that ceph ports are allowed on the platform.
The check runs a nmap command from the host running Ansible
to all the ceph nodes with their respective ports.

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

Merge pull request #468 from bengland2/fix_check_system

use proper syntax for expanding variables

Merge pull request #467 from ceph/restapi-infernalis

Fix condition logic for infernalis in restapi

use proper syntax for expanding variables

Fix condition logic for infernalis in restapi

fixes: #465

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

Merge pull request #460 from crcceph/master

fixed for infernalis on EL7

moved set_fact higher in file

moved set_fact for is_ceph_infernalis near top of file so that it can be used in other tests

Merge pull request #461 from ceph/templating

Using config_template

move set_fact above test

moved the set_fact above where it is being tested

Using config_template

Thanks to @cloudnull great patch at
https://github.com/ansible/ansible/pull/12555
we now have the ability to add more configuration options instead of
having to push a PR to add a new option to the template. So you can
dynamically add and remove flags.

To use it, edit `ceph_conf_overrides` in `group_vars/all` like so:

```
ceph_conf_overrides
 global:
     foo: 12345
     bar: 6789
```

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

added python-setuptools as dependency

CentOS7 needs python-setuptools for /usr/bin/ceph-init-detect to work correctly

changed permissions under ceph user

when running under the user ceph the rbd client directory should be owned by ceph

Merge pull request #459 from ceph/civetweb-port2

Change update group_vars ivetweb port

Change update group_vars ivetweb port

follow up on #458

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

Merge pull request #458 from ceph/civetweb-port

Change default civetweb port to 8080

Change default civetweb port to 8080

Because of some permission issue, likely due to the recent ceph user, if
80 is used for civetweb we get:

set_ports_option: cannot bind to 80: 13 (Permission denied)

Changing the port to 8080 until this gets solved.

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

Merge pull request #457 from ceph/rgw-log-dir

Use /var/log/ceph for rgw logs

Use /var/log/ceph for rgw logs

We should not bother having another directory for this. So we stick with
the default logging dir.

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

Merge pull request #442 from ceph/infernalis-for-rh-storage

Support infernalis for rh storage

Merge pull request #456 from Abhishekvrshny/fix-rgw-print-continue

removed rgw print continue = false, to set it to its default value of…

Support infernalis for rh storage

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

removed rgw print continue = false, to set it to its default value of true

Merge pull request #449 from nexecook/pre-infernalis_fixes

fixes ceph_stable_release logic mentioned in issue #446

fix comment headings

add ceph_stable_releases to group_vars/all.sample

Add ceph_stable_releases array to ceph-common/defaults/main.yml

rework the various `when' tests to use that array.

Merge pull request #452 from BjoernT/bug/451

Adding osd_scrub_begin_hour and osd_scrub_end_hour

Adding osd_scrub_begin_hour and osd_scrub_end_hour to limit scrub window of 24h if needed
Fixes-Bug: #451

Merge pull request #447 from lpabon/libvirt

Support for libvirt provider in vagrant

Merge pull request #450 from ceph/tun

Reserve some memory

Reserve some memory

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

Merge pull request #448 from mattt416/add_run_once

Add run_once to 'copy keys to the ansible server'

fixes ceph_stable_release logic mentioned in issue #446

I changed the argument used for starting the mds server. (pre
infernalis)
```
service ceph start mds
```
errors, while
```
service ceph start mds.$hostname
```
correctly starts the service.

I changed the mds directory ownership from ceph:cephh to root:root
again, for pre-infernalis.

And finally, add the ceph_stable_releases checks for the upstart
activation task `for or after infernalis release'.

Add run_once to 'copy keys to the ansible server'

I have seen a number of failures on this task due to mismatch of
checksum of source file and destination.  I suspect this is due to a
race condition caused by several hosts simultaneously copying the same
file to single location on the deployment server.

This change simply updates the 'copy keys to the ansible server' task
by adding 'run_once', which limits the task to being run on a single
MON host.

Closes issue #410

Update README.md