Now that tls support is required, add support for it.
Change-Id: I36f2c0a2b2209cfa974b4686c0c32f3fe32e9cae
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Do the things we need to do to get our testing working.
Change-Id: Ia9898a2406c601f45b07714b46a7d44632e229e5
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
It was a lot of overhead, with little gain, to run molecule lint.
Switch back to just using yamllint.
Change-Id: Ie4e4d5d77dcb864f7b76cd356cab7adf9516d1b3
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Zuul now supports multiple run playbooks, lets use that over post-run.
Change-Id: Id6d591023ef9b351e08e4260c281d25eec0836cc
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This is our first attempt at running nodepool from docker, consider it
experimental. The idea here, is you now have the option of using the
official docker images from https://hub.docker.com/u/zuul for nodepool.
Users themself will be required to manage / configure docker, this role
only installs / configures / starts nodepool.
Change-Id: I2f3aa301e626a7b09ff129a21a188cee06e31f18
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Molecule is the defacto testing tool for ansible roles. Switch to it to
make it easier for users to test.
Change-Id: I881e895877c9702e62c4415d48b0f33f34f7487b
Depends-On: https://review.openstack.org/593369
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
We can switch to using testinfra to help ensure ansible did everything
properly.
Change-Id: I9e21cd16f1149d76308043a97658c93e7f14ac42
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
While this isn't 100%, it is enough to actually get started using it.
Change-Id: I8e4bad97658f98ca5280eca694dc2d9cb4028ad1
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
We actually want to ensure users are able to use our roles outside of
openstack-infra. Since those users won't have wheel mirrors, disable
our jobs from using them too.
Change-Id: I0a3c44c81aff497bc1678b68fc7b1374f233901b
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Because we want users to support both pip / git install methods,
refactor testing to support the testing of that.
Change-Id: Ieef3763172fbfe07a5d0cc4c83c37a6dad30b3cd
Depends-On: https://review.openstack.org/560996
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Stop using /opt folder to store the location of git checkout. We don't
actually need to be root user to do that.
Change-Id: Icdb2905cf66fd9ab414d009ad3d782bc802e037d
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
It is helpful to have logs to debug failures.
Change-Id: I3d5170713a031dc8ccb4772a7778bec1e63ffa15
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
There is no more python2 support for nodepool, switch to using pip3 to
install it by default.
Change-Id: I9daf4c6e78c4069e6eb949cc89b2b76452b3a42f
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Now that zuulv3 is live, we can start having in repo jobs.
Change-Id: I0463d440abac26b4592fce0e26960be33c0a4efc
Depends-On: I7db3518f6dbd9c9315ebb5d4c40c6e18764b4e94
Depends-On: I3bff01ebac17c40988df60e7e4dd6e7fdc88a896
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Now that we are using systemd everywhere, we can simplify our service
configuration.
Change-Id: I9bacb3925f8e95587d459284758de337f9a270c3
Depends-On: I37ae527a93f5d5300c9ac4a1b4a6ca4363e7096d
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Move forward with only supporting ubuntu-xenial for our LTS.
Change-Id: I95194bd5555a0df18059f4aaf2ab1898dcfb59e6
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This is an optional setting, and not needed to start nodepool.
Change-Id: I603b4a79f9a15ad404df4b34377c2f9fc0f7ba17
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This is no longer a valid setting in nodepool.
Change-Id: Iee9399cf0989eb7cfab4c71c97baf5a69c3265d0
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This is a follow up to a previous commit, make sure the PID files
actually exist.
We only need this for initd scripts today.
Change-Id: Iedad736a9656aa04bc9b696051e421663061c0bb
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Here we are adding support to our existing systemd files for both
centos-7 and ubuntu-xenial.
Change-Id: I15e8db69a87bbfa75391f2faa97b312a034673bf
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
We now have the ability to scale out nodepool-launcher, on a single
instance.
Change-Id: Ib7792ea591c7e1ce953730791884566f77fcbd33
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
We now have the ability to scale out nodepool-deleter, on a single
instance.
Change-Id: Idce9a7a1f49ae7b433009967bc1df220b0c35e00
Depends-On: I84a727198f0f3b1d3fbd0564b9ccb120648069cd
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
The idea here is to expose information to users to allow them to make
decisions, for example restarting a service.
Change-Id: Ifbc45af72db820c842c83d7af1adb460969a7336
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
It was become too much work to deal with this. Additionally, if people
want to manually synchronize we expose the proper settings to allow
them to do so.
Change-Id: I68aedb16223146f1ff6d7c9115773e62ae4ba8c9
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
It was become too much work to deal with this. Additionally, if people
want to manually synchronize we expose the proper settings to allow
them to do so.
Change-Id: I68ae9e7cb6e4a69392ea95cd0be72e01f166190e
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Simplify things by only passing the secure.conf file, not a
template.
If users need more dynamic options, they are free to template the file
first then use us to copy the file in place.
Also tests file mode for nodepool.yaml
Change-Id: I4d2eba3292b2657ef9a607497056fa94cdd1bec8
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Simplify things by only passing the nodepool.yaml file, not a
template.
If users need more dynamic options, they are free to template the file
first then use us to copy the file in place.
We'll be doing this over a series of commits, rather then 1 large
patch.
Change-Id: Id751920e31edac9a469f4624ae2a30e841245d80
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Since it is possible to run both nodepool and nodepool-builder, we
should give the user 2 different configuration files.
Change-Id: I9e825a331d04124a2548203cfd8d77935fdf7bb6
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Move them back to their original location to be in sync with
openstack-infra.
Change-Id: I21a38b7be1c7a9ee2547a8bbd2f2f5018e51d15f
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
To keep things in sync with future packaging.
Change-Id: Iff3221cd4d9eb7c058aefcd8747ac7f8baa3988b
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
* add tests to validate files are installed
* change to match zuul style service configuration
* fix invalid tests added first time around
* add missing defaults
* drop nodepool.sysconfig and move to proper location
Change-Id: Iee0b11897d7f4916035ab2d92c5d2dd755341670
Upstart is dead moving forward, just skip using it until systemd is
default for Debian.
Change-Id: I3ca250523d2871097001df0c76b3b83f57eb5c99
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
There is an open review for project-config to enable this by default,
until then, we can have ansible set things up for us.
Change-Id: I64aaa7187568ff08ef6b0935d8829376dd6761e5
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Allow the user to override the service settings and add unit tests.
Change-Id: I2d9409d210015a202f33eca970d02be137617de7
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This will give the user more flexibility when defining source folders.
Change-Id: Ib57f81fc6892c55e67be647d2f1d2e56fd4f08ec
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Otherwise, we loose the execute bits for dib elements.
Change-Id: I83e193cd9aef7e7e9da71558b27af1463c80bc3b
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
nodepool_project_config_dir should contain the path of adirectory which
has both elements/ and scripts/ sub directories.
Change-Id: I3a3d2ba225f15cf785da327905915053b04340ea
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Like we did in ansible-role-zuul, start writing unit tests for our
role.
Change-Id: I8e7a0dd90323c67458caf2a06f5799c23df20ffb
Signed-off-by: Paul Belanger <pabelanger@redhat.com>