Removed erroneous check

Fixes
1) already subscribed machines w/ bad repos weren't cleaned
2) enable repos would always run if 'rhsm_repos|length > 0' conditional used

Signed-off-by: dgalloway <dgallowa@REDHAT.COM>

Merge pull request #104 from ceph/wip-var-loading-order

testnode: fix var file loading order

testnode: fix var file loading order

We want to load distro specific variables before we load major version
specific variables. Loading of vars should happen in this order: package
manager, distro, major version, version.

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

Merge pull request #102 from ceph/wip-epel-packages

testnode: put epel packages in their own list for rhel 6 and 7

Reviewed-by: Dan Mick <dmick@redhat.com>
Reviewed-by: Zack Cerza <zcerza@redhat.com>

Merge pull request #101 from ceph/wip-revoke-users

Add the ability to revoke user accounts

testnode: explicitly enable the epel repo when installing python-pip

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

testnode: put epel packages in their own list for rhel 6 and 7

This separates packages we install from epel into their own list for rhel 6 and 7.
We do this so that we can install them with the epel repo explicitly enabled.

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

Users: Fix broken example in README.rst

Signed-off-by: Zack Cerza <zack@redhat.com>

Users: Add the ability to revoke user accounts

Signed-off-by: Zack Cerza <zack@redhat.com>

Merge pull request #96 from ceph/wip-ansible-failures-module

A callback plugin that logs failure message

A callback plugin that logs failure messages

If the environment variable ANSIBLE_FAILURE_LOG is present then a log of
all failures in the playbook will be persisted to the file
path given in the ANSIBLE_FAILURE_LOG.

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

Merge pull request #99 from dmick/master

testnode/tasks/ntp: make sure ntpd is started

testnode/tasks/ntp: make sure ntpd is started

Tested with:

$ ansible-playbook -i ../ceph-octo-secrets/ansible/inventory/octo
--limit 'magna064*' --tag ntp-client testnodes.yml

Fixes: #12603
Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #90 from ceph/wip-wrap-readme

doc: wrap README.rst at 80 characters

Merge pull request #91 from ceph/wip-puddle-updates

Puddle updates

Merge pull request #97 from ceph/wip-teuthology-role

Add teuthology role and playbook

Add teuthology role and playbook

So far, it only depends on the sudo and users roles. The playbook also
runs the common role.

Signed-off-by: Zack Cerza <zack@redhat.com>

Merge pull request #95 from ceph/revert-94-wip-12424

Revert "Adds cgroups packages and net_cls kernel module to ubuntu sys…

Revert "Adds cgroups packages and net_cls kernel module to ubuntu systems"

Merge pull request #94 from ivotron/wip-12424

Adds cgroups packages and net_cls kernel module to ubuntu systems

Adds cgroups packages and net_cls kernel module to ubuntu systems

This commit is part of the work for http://tracker.ceph.com/issues/12424
which enables the ceph task in ceph/ceph-qa-suite to create and place
osd/mon/mds in control groups.

For ubuntu, kernels for precise and up have CONFIG_CGROUP_* enabled, so
the only thing that it's required is to install cgroup-lite which is a
service that mounts the cgroups virtual FS. The cgroup-bin package will
also be used from the ceph task to manipulate (create/modify/delete)
control groups.

The net_cls subsystem is included as a module (cls_cgroup) and is not
enabled by cgroup-lite, i.e. it has to be explicitly loaded. so we add
it to /etc/modules

puddle: add ceph-1.3.0 to async configs

Similar to the change in commit
84818a7a7bd00a3c18f0326c312dc3abef77c05e, add the the CEPH-1.3 release
to the CEPH-ASYNC config files.

The purpose of this change is to make it easier for QE to consume
the builds in Puddle repos that correspond to ASYNC advisories.

puddle: update distill directory name for ceph 1.3

The final Brew tag for 1.3 was "-compose", not "-candidate" or
"-pending".

puddle: add ceph-1.2 to async configs

Add the the CEPH-1.2 release to the CEPH-ASYNC config files.

The purpose of this change is to make it easier for QE to consume
the builds in Puddle repos that correspond to ASYNC advisories.

Prior to this change, QE had to make sure to enable the production Ceph
repositories with subscription-manager before using the packages in an
ASYNC advisory's Puddle. With this change, all the latest "1.2.3"
packages will be combined together with the "1.2-async" packages into a
single Puddle.

This has a side-effect advantage in that we can simplify the "externals"
sections of these config files.

Merge pull request #93 from ceph/wip-cobbler-update-cm

Cobbler: update CM repos before running ansible

Cobbler: update CM repos before running ansible

Signed-off-by: Zack Cerza <zack@redhat.com>

Merge pull request #92 from ceph/wip-debug-distros

Common: Log the OS/version of each node

Common: Log the OS/version of each node

I think this will help with debugging.

Signed-off-by: Zack Cerza <zack@redhat.com>

doc: wrap README.rst at 80 characters

My Eyes! The goggles do nothing!

Merge pull request #89 from ceph/wip-rhel67

Add RHEL 6.7

Add RHEL 6.7

Signed-off-by: Zack Cerza <zack@redhat.com>

Simplify extra_distros logic

Signed-off-by: Zack Cerza <zack@redhat.com>

Merge pull request #88 from ceph/wip-ipmi

Cobbler IPMI configuration and utilities

Merge pull request #75 from ceph/wip-testnode-docs

Documentation for the testnode role.

Reviewed-by: Dan Mick <dmick@redhat.com>

cobbler_systems: populate IPMI information

Signed-off-by: Zack Cerza <zack@redhat.com>

Cobbler: ship ipmi-related utility scripts

console.sh, reboot.sh, and reimage.sh

Signed-off-by: Zack Cerza <zack@redhat.com>

Merge pull request #87 from ceph/wip-cobbler-users

Cobbler: enable adding more user accounts

Reviewed-by: Dan Mick <dmick@redhat.com>

Cobbler: change default password

So that it's usably by third parties

Signed-off-by: Zack Cerza <zack@redhat.com>

Cobbler: discover port in restart.yml

... because that's the only place it's needed

Signed-off-by: Zack Cerza <zack@redhat.com>

Cobbler: write users.digest base on a var

... instead of a template, so we can add users on a per-lab basis.

Signed-off-by: Zack Cerza <zack@redhat.com>

Merge pull request #86 from ceph/wip-12380

Retry authorized_keys updates

Retry authorized_keys updates

http://tracker.ceph.com/issues/12380
We're seeing transient issues connecting to githubusercontent.com. This
commit will cause ansible to retry three times with a five-second delay
between tries. Hopefully that's good enough.

Signed-off-by: Zack Cerza <zack@redhat.com>

Merge pull request #85 from ceph/wip-12381

testnode: add dbench and bonnie++ for centos7

testnode: ensure yum-utils is present before yum-complete-transaction

This isn't there on centos 7 causing the yum-complete-transaction task
to fail.

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

testnode: add dbench and bonnie++ to package list for centos7

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

Merge pull request #84 from ceph/wip-12371

testnode: remove scsci-target-utils and iscsi-initiator-utils

testnode: remove scsci-target-utils and iscsi-initiator-utils

We don't want to install these anymore as they're causing issues with
the teuthology install task because of their dependency on librados.

See: http://tracker.ceph.com/issues/12371

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

Merge pull request #82 from ceph/wip-yum-core-base

Install the Core and Base groups on rpm systems

Install the Core and Base groups on rpm systems

We need to make sure basic things like bzip2 are installed.

Signed-off-by: Zack Cerza <zack@redhat.com>

f; more review feedback from dmick

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

f; attempt to make comments better in defaults/main.yml

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

f; try to explain how packages and common_packages work

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

testnode: adds common_apt_repos and apt_repos to the docs and defaults

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

f; review feedback from dan

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

testnode: add documentation for this role

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

testnode: add common_packages to defaults/main.yml

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

Merge pull request #81 from dmick/master

Fix warning in 'when' clause in 'Create the teuthology user.'

Fix warning in 'when' clause in 'Create the teuthology user.'

Warnings appeared each run:

[WARNING]: It is unnecessary to use '{{' in conditionals, leave variables in
loop expressions bare

Apparently 'when' clauses are already Jinja2 expressions, and so don't
need the normal variable quoting.

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #80 from ceph/wip-remove-mandatory-secrets-path

common: |mandatory is not needed for secrets_path, it is always defined.

Reviewed-by: Dan Mick <dmick@redhat.com>

common: |mandatory is not needed for secrets_path, it is always defined.

secrets_path comes from the secrets role and it is always defined.  It
will default to /etc/ansible/secrets if the env var ANSIBLE_SECRETS_PATH
is not defined. You can also define secrets_path by passing a variable
to the playbook being used.

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

Merge pull request #79 from ceph/wip-cobbler-git

cobbler: install git

Merge pull request #77 from ceph/wip-cobbler-system-defaults

cobbler_systems: properly find defaults

Merge pull request #78 from ceph/wip-cobbler-rh-firewall

Cobbler: ignore firewalld if it's not running

cobbler: install git

Signed-off-by: Zack Cerza <zack@redhat.com>

Cobbler: ignore firewalld if it's not running

Signed-off-by: Zack Cerza <zack@redhat.com>

cobbler_systems: properly find defaults

For interface and kernel_options

Signed-off-by: Zack Cerza <zack@redhat.com>

Make firewall changes immediate

Signed-off-by: Zack Cerza <zack@redhat.com>

Merge pull request #76 from ceph/wip-cobbler-rh-firewall

Manage the firewall on RHEL/CentOS 6/7

Manage the firewall on RHEL/CentOS 6/7

On EL7, poke holes for http and https. On EL6, just stop iptables.

Signed-off-by: Zack Cerza <zack@redhat.com>

Merge pull request #74 from ceph/wip-fix-entitlements

Fix entitlements

Merge pull request #68 from ceph/wip-var-docs

docs: explain where to define variables for use in roles

Fix replace_repos logic

The issues were with quoting of booleans, and |success vs. .rc == 0

Signed-off-by: Zack Cerza <zack@redhat.com>

Fix logic of finding entitlements.yml

We were mistakenly checking for the existence of entitlements.yml on the
*remote* end.

Signed-off-by: Zack Cerza <zack@redhat.com>

docs: explain where to define variables for use in roles

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

Merge pull request #73 from ceph/wip-12256

DNM: roles used by teuthology should not require a secrets repo

ansible-managed: skip adding pub_keys if vars is not defined

This lets this play gracefully skip if ansible_user and ansible_ssh_keys
is not defined.

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

common: default the epel_mirror_baseurl var

Defaults to the value used in our sepia lab.

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

testnode: default the gitbuilder_host var

Defaults to the value used in our sepia lab.

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

testnode: set a default for the var ntp_servers

Defaults to the value used in our sepia lab.

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

testnode: default git_mirror_host

Defaults to the value used in our sepia lab

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

testnode: default mirror_host var

This defaults to the value used in our sepia lab.

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

creates a new sudo role that ensures all plays are ran with sudo

This should be added as a dependancy to any other roles that require
sudo. This commits adds it to ansible-managed, common and testnode.

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

testnode: make lab_domain an optional var

lab_domain is used to strip the domain from the nodes hostname, this
allows that task to be skipped if lab_domain isn't provided.  I believe
we only do this because of some quirk in how cobbler images our rhel
nodes, we might not even need this functionality.

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

Merge pull request #72 from dachary/wip-fstab

implement modify_fstab to skip /etc/fstab modification

implement modify_fstab to persist mount options in /etc/fstab

http://tracker.ceph.com/issues/12250 Fixes: #12250

Signed-off-by: Loic Dachary <loic@dachary.org>

Merge pull request #69 from ceph/wip-cobbler-httpd

Help users locate cobbler's web UI

Help users locate cobbler's web UI

Signed-off-by: Zack Cerza <zack@redhat.com>

Merge pull request #67 from ceph/wip-fix-defaults

Skip ansible_user creation if it is undefined

Merge pull request #70 from ceph/wip-no-ubuntu-resolvconf

testnode: do not modify resolv.conf on ubuntu

testnode: do not modify resolv.conf on ubuntu

There is no need to modify resolv.conf for ubuntu anymore.  When using
cobbler this is set by using the resolvconf var in the cobbler role. For
vms, like ones created by openstack, this allows ubuntu nodes to use the
resolv.conf already provided.

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

Skip ansible_user creation if it is undefined

Signed-off-by: Zack Cerza <zack@redhat.com>

Merge pull request #66 from ceph/wip-puddle-updates

Puddle updates

puddle: add ceph 1.3 async

Add a configuration file to do Puddle generation for ceph 1.3 async.

puddle: update rh-common errata

Update Puddle's configuration for the latest RH-COMMON errata currently
in progress.

puddle: update URL variable conventions

1) Remove "_z" from the RHEL repository variables. There's no point in
distinguishing "RHEL 7 z" from "RHEL 7".

2) Add "_server" to distinguish the RHEL Server variant from the
Workstation/Client/ComputeNode variants.

Merge pull request #65 from ceph/wip-yum-tx

Remove saved yum transactions

Remove saved yum transactions

Signed-off-by: Zack Cerza <zack@redhat.com>

Don't try to remove a missing package

My previous commit had a bug. First, check if ceph-debuginfo is present.
If it's not, don't try to remove it.

Signed-off-by: Zack Cerza <zack@redhat.com>

Merge pull request #64 from ceph/wip-bz-1234967

Work around BZ#1234967

Work around BZ#1234967

We have some corrupt ceph-debuginfo packages that are causing yum to die
with a UnicodeDecodeError. Use rpm to remove them for now.

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

Signed-off-by: Zack Cerza <zack@redhat.com>