Review feedback

Adding playbook directory for creating private docker registry

Merge pull request #583 from ceph/wip-sudo-env

Pass CEPH_EXTRA_CMAKE_ARGS through sudo to pbuilder build

Pass CEPH_EXTRA_CMAKE_ARGS through sudo to pbuilder build

This should finally allow notcmalloc builds on debian.

Setting the variable on the sudo command line does affect the
resulting pbuilder debian/rules execution (tested with a dummy
debian package)

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

Merge pull request #581 from ceph/ceph-dev-new-build

Ceph dev new build

ceph-dev-new-setup: fetch tags from ceph.git

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

ceph-dev-new-setup: update the config name to normalized convention

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

ceph-dev-new: update/normalize names for '-new' jobs

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

ceph-dev-new-build: create a job to fetch from ceph-dev-new-setup

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

ceph-dev-new-setup: normalize on '-new' names

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

Merge pull request #580 from ceph/revert-579-add-ca-scenario

Revert "ceph-ansible-prs: add 2 scenarios"

Revert "ceph-ansible-prs: add 2 scenarios"

Merge pull request #579 from ceph/add-ca-scenario

ceph-ansible-prs: add 2 scenarios

ceph-ansible-prs: add 2 scenarios

Both:

* xenial_mon_osd: tests a 1 mon 1 osd xenial cluster using
raw_multi_journal OSD scenario

* xenial_conf_tests: test a 3 mon cluster and ensures
ceph.conf is rendered correctly

were missing, so adding them to the CI.

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

Merge pull request #578 from ceph/new-tox-envs

ceph-ansible-prs: adds the docker_cluster scenario

ceph-ansible-prs: adds the docker_cluster scenario

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

Merge pull request #577 from ceph/ceph-dev-setup-new

Ceph dev setup new

ceph-dev-new-trigger: rename to reflect actual job name

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

ceph-dev-new: rename to reflect actual job name

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

ceph-dev-new: use ceph-dev-setup-new to use the correct git url

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

ceph-dev-setup-new: clone ceph-dev-setup temporarily for ceph-ci.git

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

Merge pull request #576 from ceph/fix-alfredo-doodoo-part-deux

create a ceph-dev-new trigger just for ceph-ci

ceph-dev-new-trigger: create a new job only for ceph-ci

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

ceph-dev-trigger: remove triggers from ceph.git to ceph-dev-new

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

Merge pull request #575 from ceph/fix-alfredo-doodoo

ceph-dev-new: fix the collision on the job name

ceph-dev-new: fix the collision on the job name

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

Merge pull request #574 from ceph/trigger-ceph-new

ceph-dev-trigger: trigger ceph-dev-new

ceph-dev-trigger: trigger ceph-dev-new

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

Merge pull request #565 from ceph/ceph-dev-new

ceph-dev-new: create a sister job for ceph-dev to build from ceph-ci

ceph-dev-new: create a sister job for ceph-dev to build from ceph-ci

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

Merge pull request #573 from ceph/new-tox-envs

ceph-ansible-prs: use underscores instead of dashes for scenarios

ceph-ansible-prs: use underscores instead of dashes for scenarios

This avoids an issue in tox where it splits on dashes and picks an
incorrect scenario because we had dashes in our environment names.

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

Merge pull request #572 from ceph/arch-ansible-syntax

ceph-ansible-pr-syntax-check: adds a x86_64 label

ceph-ansible-pr-syntax-check: adds a x86_64 label

We don't want this job picking up our arm64 slaves.

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

Merge pull request #571 from ceph/tox-workdir

ceph-ansible-prs: set a working directory for tox

ceph-ansible-prs: set a working directory for tox

We need to set a working directory for tox because the default for some
scenarios are too long and the job fails to call pip because it exceeds
the char limit.

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

Merge pull request #570 from ceph/ansible-scenarios-osds

enable other ceph-ansible scenarios

ceph-ansible-prs: add journal, collocation, and dmcrypt scenarios

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

Merge pull request #568 from ceph/prado-support

Prado support

ansible: start the libvirt services for slave_libvirt

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

ansible: forcefully add a slave_libvirt.yml file for prado to work with

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

ansible: update slave_libvirt example to prado changes

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

ansible: forcefully add a slave.yml file for prado to work with

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

ansible: update slave example to prado changes

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

Merge pull request #569 from ceph/contrib

ceph-ansible-pr-syntax-check: use yml file

ceph-ansible-pr-syntax-check: use yml file

We now use yaml files so we need to change the mv accordingly.

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

Merge pull request #567 from ceph/ansible-galaxy

ceph-ansible-galaxy: fix workspace path

ceph-ansible-galaxy: fix workspace path

The current path to execute the script is wrong, see:
bash:
/home/jenkins-build/build/workspace/ceph-ansible-galaxy/contrib/push-roles-to-ansible-galaxy.sh:
No such file or directory

It tries to execute
ceph-ansible-galaxy/contrib/push-roles-to-ansible-galaxy.sh where the
actual script is in
ceph-ansible-galaxy/ceph-ansible/contrib/push-roles-to-ansible-galaxy.sh

This commit fixes this.

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

Merge pull request #566 from ceph/fix-group-vars

ceph-ansible-syntax: update syntax check

ceph-ansible-syntax: update syntax check

We now use group_vars/alll.yml files. So only yaml files, this
ceph-ansible PR introduced that:
https://github.com/ceph/ceph-ansible/pull/1113

So we apply the change here as well.

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

ansible: include libssl and libffi as requirements as well

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

Merge pull request #564 from ceph/ceph-ansible-contrib

ceph-ansible-galaxy: update the splitup script

ceph-ansible-galaxy: update the splitup script

We now use the latest version of the splitup.sh script that now pushes
branches and tags from ceph-ansible to ansible-ceph-* repositories.

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

Merge pull request #563 from ceph/ceph-ansible-labels

ceph-ansible-prs: update labels required

ceph-ansible-prs: update labels required

Using a '-' breaks the service that creates machines based on labels
since it evaluates it as Python and '-' is seen as an operator

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

ceph-ansible-prs-* remove dirs per job. Templates take care of it now

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

Merge pull request #561 from ceph/ceph-ansible-prs

ceph-ansible-prs: create a template for all jobs

ceph-ansible-prs: create a template for all jobs

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

Merge pull request #562 from ceph/ansible-syntax-check

ceph-ansible-pr-syntax-check: add a trigger phrase

ceph-ansible-pr-syntax-check: add a trigger phrase

Adds the trigger phrase 'jenkins test playbook syntax' to start a build
of this job.

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

Merge pull request #560 from ceph/ceph-ansible-rm

ceph-ansible-pull-requests: remove job - moving to granular ones

ceph-ansible-pull-requests: remove job - moving to granular ones

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

Merge pull request #559 from ceph/ansible-centos7-cluster

Adds the ceph-ansible-prs-centos7-cluster job

ceph-ansible-prs-xenial-cluster: fix the tox environment name

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

Adds the ceph-ansible-prs-centos7-cluster job

This runs the centos7-cluster functional testing scenario.

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

Merge pull request #558 from ceph/ansible-scenarios

Ansible scenario testing for prs (xenial-cluster)

ceph-ansible-prs-xenial-cluster: create the teardown script

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

ceph-ansible-prs-xenial-cluster: create the build script

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

ceph-ansible-prs-xenial-cluster: create a config

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

Merge pull request #557 from ceph/ansible-libvirt

ansible: create a libvirt slave playbook

ansible: create a libvirt slave playbook

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

Merge pull request #556 from ceph/ceph-ansible-concurrent

ceph-ansible-pull-requests: make the job concurrent so other slaves can do work too

ceph-ansible-pull-requests: make the job concurrent so other slaves can do work too

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

Merge pull request #555 from ceph/ceph-pr-submodules

Ceph PRs submodules check

ceph-pr-submodules: define either trusty or centos to avoid using master

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

ceph-pr-submodules: add docs snippet for shell variables in foreach

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

ceph-pr-submodules: create a configuration for the new job

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

ceph-pr-submodules: create a build script to check for modified submodules

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

Merge pull request #554 from ceph/wip-kernel-trigger

kernel: a new kernel-trigger job

kernel: filter BRANCH in setup

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>

kernel: a new kernel-trigger job

This job triggers kernel builds for testing, master and for-linus.
No wip-* for now - need to clean up old WIP branches first.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>

kernel: use https instead of git

No need to use git and bother with credentials.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>

Merge pull request #553 from ceph/kernel-chacra-update

kernel: upload binaries to a dev chacra instance

kernel: upload binaries to a dev chacra instance

The chacra instances are now configured to always keep
a build for the 'testing' ref, so it should be safe
to store these in the dev chacra instances now.

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

Merge pull request #526 from ceph/ceph-installer-rpm

add ceph-installer-rpm job

add ceph-installer-rpm job

This job builds RPMs for each ceph-installer GitHub branch.

Merge pull request #552 from ceph/testing-updates

ceph-ansible-pull-requests: only run teardown on failure

ceph-ansible-pull-requests: only run teardown on failure

If the tests succeed tox will clean up everything for us so
we only need to run this on failure.

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

Merge pull request #549 from ceph/ceph-ansible-tox

Ceph ansible tox

ceph-ansible-pull-requests: teardown from the tests directory

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

ceph-ansible-pull-requests: tie the job to the correct labels

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

ceph-ansible-pull-requests: update the job configuration, enabling it

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

ceph-ansible-pull-requests: forcefully destroy every scenario when tests are completed

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

ceph-ansible-pull-requests: use the new scenario-based tox test invocation

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

Merge pull request #551 from ceph/ceph-setup-arch

Ceph setup arch

ceph-setup: ask for x86_64 archs

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

ceph-dev-setup: ask for x86_64 archs

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

Merge pull request #548 from ceph/disable-ceph-ansible

ceph-ansible-pull-requests: disable until libvirt/vagrant tests are i…

ceph-ansible-pull-requests: disable until libvirt/vagrant tests are in place

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

Merge pull request #546 from ceph/wip-kernel-extra

kernel: Send package_manager_version information