Fix not creating pypy_binary_direcotry

Merge pull request #861 from font/test_issues

Fail when we cannot retrieve docker image

Fail when we cannot retrieve docker image

If the docker image cannot be retrieved we will fail this task silently
and the playbook ultimately succeeds without a successful deployment.
This change makes it so we fail the playbook immediately.

Signed-off-by: Ivan Font <ivan.font@redhat.com>

Merge pull request #857 from dang/url

Add URL option to vagrant

Add URL option to vagrant

Vagrant has the ability to download images from an arbitraty URL.  Add
this support to the Vagrantfile and vagrant_varables.yml

Signed-off-by: Daniel Gryniewicz <dang@redhat.com>

Merge pull request #855 from jimcurtis/master

docker: fix missing quotes

docker: fix missing quotes

Merge pull request #853 from ceph/rhbz-1348130

ceph-agent: install the correct package for the agent in Ubuntu

ceph-agent: install the correct package for the agent in Ubuntu

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Resolves: rhbz#1348130

Merge pull request #852 from rootfs/doc-fix

be explicit on Atomic VirtualBox instruction

Merge pull request #851 from ceph/rhbz-1346812

Enable rhcs repos according to the released version

fix rgw containerized deployment

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

fix mds containerized deployment

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

be explicit on Atomic VirtualBox instruction

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

ceph-common: use rh_storage_version variable to enable the correct repos

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Resolves: rhbz#1346812

ceph-common: do not enable optional repos for rhel7

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Resolves: rhbz#1346812

ceph-common: default to 1.3 for rh_storage_version

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Resolves: rhbz#1346812

group_vars: define the rh_storage_version

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Resolves: rhbz#1346812

ceph-common: declare mon_containerized_deployment

If not declared, the play will fail.

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

Merge pull request #846 from dang/docker-configs

Put path back to playbook_dir for common template

Put path back to playbook_dir for common template

The config template is in ceph-common, not in the individual roles, so
roles referencing it need to use playbook_dir, not role_path.

Signed-off-by: Daniel Gryniewicz <dang@redhat.com>

ceph-mon: fix wrong indent

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

Merge pull request #812 from rootfs/fix

some fixes for container deployment

Merge pull request #844 from matthewrees/master

Various fixes for check_firewall task

Fix typo for ceph monitor port

Clean up checks and with_item loops that are no longer needed due to
other fixes in this PR

Merge pull request #840 from jimcurtis/master

docker: enable mds container on Atomic host environment

Fix for nmap arguments and grep matching, as well as only running
local_actions once

docker: review feedback on PR #840

ceph.conf.j2: check if interface is defined before using it
ceph-mon.service.j2: remove redundant --name option from docker command

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

Fix to reference the monitor_address variable correctly in hostvars

Add fix for those who are using monitor_address

Update sample vars for change in default value of check_firewall

Change default to False for checking firewalls

Only include the firewall checks when a user wants them checked

Use the command module in place of the shell module

Various fixes for check_firewall task:
- Check for nmap being available was not running as a local_action, when the checks using nmap were
- Various fixes on Ansible 2.x now that the above is working

Merge pull request #839 from dcwangmit01/centos_remove_plugin_priorities

Remove Centos7 dependency yum-plugin-priorities

Merge pull request #804 from dang/docker-dev

Allow running on local dev docker images

docker: review feedback on PR #840

Ran generate_group_vars_sample.sh

The changes here are not anything to do with removing
  yum-plugin-priorities.  They must be leftover from
  prior commits that did not run generate_group_vars_sample.sh

Added comments from ./group_vars/all.sample back into roles/ceph-common/defaults/main.yml

Remove Centos7 dependency yum-plugin-priorities

This causes ceph-ansible scripts to fail when targeting Centos7 machines.
Installation fails because newer ceph package dependencies provided
  by ceph-release-{version}.noarch.rpm were overridden by older
  package dependency versions in default distribution repositories,
  due to the fact that default distribution repositories have higher
  priority.

Allow running on local dev docker images

Docker makes it difficult to use images that are not on signed
registries.  This is a problem for developers, who likely won't have
access to a registry with proper signed certificates.

This allows the ability to use any docker image on the machine running
vagrant/ansible.  The way it works is that the image in question is
exported locally, then sent to each target box and imported there.

Signed-off-by: Daniel Gryniewicz <dang@redhat.com>

update osd default vars

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

fix containerized deployment on centos

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

Merge pull request #841 from TeutoNet/master

- prepare for python3

- prepare for python3

Merge pull request #837 from ceph/rhbz-1342117

Do not ignore ceph-disk issues

docker: enable mds container on Atomic host environment

Merge pull request #836 from ceph/RHBZ-1337601

include the red hat storage apt-key as a file

Merge pull request #838 from ksingh7/fix-819

Fix for #819

ceph-osd: do not ignore ceph-disk errors in raw multi journal

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Resolves: rhbz#1342117

ceph-osd: do not ignore ceph-disk errors in journal_collocation

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Resolves: rhbz#1342117

ceph-osd: do not ignore ceph-disk errors in bluestore

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Resolves: rhbz#1342117

include the red hat storage apt-key as a file

This will allow nodes to install rhcs that do
not have access to the internet.

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
Resolves: rhbz#1337601

Merge pull request #833 from ceph/template-path

ceph-common: uniformize template path calls

ceph-common: uniformize template path calls

In order to align all Ansible versions, we now use the full path for the
template. We rely on `role_path` variable. Now all the tasks using
the template module have a uniform syntax.

Might fix issue raised in #483

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

Merge pull request #831 from SamYaple/fix_when

The when conditional was improperly being checked

Add undefined vars to ceph-common role

This is a temporary action to get the playbooks functional before a
refactor to better correct this issue.

Revert "Fix commit conditionals"

This reverts commit 9ed45f7367bc03ccaa5c40e574e5a5036493c856.

Fix for #819

This commit removes /etc/yum.repos.d/rh_storage.repo file
for RedHat OS with RHCS ISO install during cluster purge

Merge pull request #832 from ceph/RHBZ-1339439

remove the gpgkey line from the red hat storage repo template

Merge pull request #828 from jimcurtis/master

docker: enable rgw container on Atomic host environment

remove the gpgkey line from the red hat storage repo template

See: https://bugzilla.redhat.com/show_bug.cgi?id=1339439

Signed-off-by: Andrew Schoen <aschoen@redhat.com>
Resolves: rhbz#1339439

Add defaults to False for undefined variables

Fix commit conditionals

The goal is to _fail_ this when conditional when one (and only one) is
set. The 'not's were incorrectly added.

The when conditional was improperly being checked

The scenarios were not being accurately compared to ensure that:
* A single scenario was choosen
* ONLY a single scenario was choosen

This solution does not scale for long, but that can be addressed in a
different patchset.

docker: review feedback on PR #828

Merge pull request #827 from gcharot/gcharot-master

LSB Release is not installed by default on minimal OS installs (for e…

docker: enable rgw container on Atomic host environment

Merge pull request #823 from ceph/rhbz-1339096

ceph-common: ceph-base is not part of rhstorage, remove it

LSB Release is not installed by default on minimal OS installs (for example cloud-images). Script fails with a "command not found" erroor.
Checking if lsb_release is present BEFORE doing the test
Change os_VERSION to grab the major OS version instead of the distro codename
When installing EPEL, use $os_VERSION to grab the correct rpm instead of using hardcoded '7' package that would fail on other versions.
os_Version can also be use to deprecate old version on which ceph-ansible is not supported.

ceph-common: install ceph-common packages for clients on debian for rhcs

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Resolves: rhbz#1339096

ceph-common: install ceph-common packages for clients on redhat for rhcs

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Resolves: rhbz#1339096

Merge pull request #822 from gcharot/patch-2

Specify OSD journal size unit

Merge pull request #821 from gcharot/patch-1

Ambiguous "version is only supported on RHEL 7.1"

Specify OSD journal size unit

Specify unit to use for the journal_size parameter.

Ambiguous "version is only supported on RHEL 7.1"

Change from "only supported on RHEL 7.1" to "only supported on RHEL >= 7.1" for RHS

Merge pull request #816 from pamelia/master

allow dash in cluster name

allow dash in cluster name

in containerized deployment, use the right interface name variable in mon; remove one line inline files

cleanup on docker osd tasks

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

Merge pull request #810 from ceph/rhbz-1339096

Include rh_storage repo/installs when specified

ceph-common: include the new install_rh_storage_on_redhat playbook when ceph_stable_rh_storage is on:

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Resolves: rhbz#1339096

ceph-common: create a new install task for rh storage on redhat

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Resolves: rhbz#1339096

Merge pull request #809 from ceph/rhbz-1339068

ceph-common: specify file path for repo key

ceph-common: specify file path for repo key

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Resolves: rhbz#1339068

Merge pull request #808 from jimcurtis/master

docker: Prefix OpenStack VM names to prevent collision

Merge pull request #789 from ceph/default-ceph-path

ceph-common: fix ceph options default path again

docker: Prefix OpenStack VM names to prevent collision

Merge pull request #806 from ceph/BZ-1338548

ceph-common: there is no Installer repo, it is now the Tools repo

ceph-common: there is no Installer repo, it is now the Tools repo

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

Merge pull request #787 from ceph/civet-threads

ceph-rgw: implement num_threads for civetweb

ceph-rgw: implement num_threads for civetweb

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

Merge pull request #800 from ceph/BZ-1337320

ceph-common: on rh storage on debian add the repositories and keys (ISO)

Merge pull request #797 from ceph/rgw-debian-rhcs

install ceph-mds and radosgw on debian systems when using red hat ceph storage

Merge pull request #802 from ceph/BZ-1337305

ceph-common: redhat repo template: remove the Calamari reference

Merge pull request #801 from ceph/fix-shell-typo

ceph-osd: fix typo when calling sgdisk second time around

ceph-common: redhat repo template: remove the Calamari reference, no longer a distinct repo

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

ceph-common: on rh storage on debian do not add the rh key by default

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