It is possible we want to enable a service on boot, but don't want to
ensure it is currently running. This splits our systemd tasks into 2
different checks now.
Change-Id: Ib37371031a871d0d57b390e2430ea1efb3094171
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
While testing out molecule, the following warnings were raised with its
linter. Lets fix them to keep it happy.
Change-Id: I698f65daa58822f7befc72b9dcbf1f2d668cab02
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Now that we only support systemd, we can switch to the systemd task in
ansible. This add support for the daemon-reload option.
Change-Id: I3ea9b5b54536cfca18739304fcc737e70769e0e8
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>
Increase the ability for a user to toggle control over the specific
service / config files.
Change-Id: I5011bdd0b299a4f09a3fe49b3888d48b658908b9
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
If we are not managing the service, don't both managing the service
init script and service config.
Change-Id: Ibe19aa867f89401b11993bddaf542b150b2d9b1c
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Even though we are not templating files, it doesn't mean users won't.
So, replace copy with template.
Change-Id: I82f28c1951221a81083de66e4495de5fdcc5bdee
Depends-On: I4ac2a4b9de91b0cd949f835b6f06d1aca3720150
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>
There might be a time were we only want run the pre task, so rather
then copying the code into another modules, just allow the user to
override the task list that runs.
Change-Id: I15f58454607c5c56f9bb092036a656618b6b3c20
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>
It is possible that a user will not want the roles to control the
service settings. For example, if a service dies and you don't want to
auto restart it.
Change-Id: Iaed9769c943412d14674b7c5a9f2d7b899480746
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>
At the moment, we don't have a good control on when to actually
restart nodepool. So, it is possible weird things can happen. Also,
most of the configuration of nodepool is read in realtime, so reloads
are really needed.
Change-Id: Ice81ffb8ecd08b88e42ef1eb775720ed1744e08c
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>
Move code block up a level so we don't duplicated logic in redhat.yaml
and debian.yaml
Change-Id: Ie0b8ac7af7d67ec7e0ea8c0468eff13ebeaba600
Signed-off-by: Paul Belanger <pabelanger@redhat.com>