The opendev project has been moving away from puppet and this is one of
the puppet modules that is no longer used. To simplify things for us we
are taking the extra step of retiring this repo.
Change-Id: I1d3a8a3999e75be00844a314fa7f52c37e28f9d4
Addition of Zuul v3 initscripts raced the dynamic pidfile directory
creation fix Idfd5d19e5171278b9bb7005edb4ae927a01f3615. Port it to
them so that if the PIDFILE variable is overridden for a custom
path, we create the correct parent directory.
Change-Id: I3af4e0eb280eed5dfeab909ad4e3fb24ede75a5a
We are figuring out some github API interaction slowness and additional
api logging will help tremendously.
Change-Id: I80959b31d0fb8200a924d689e1bc27b9cde0ca31
This is no longer needed as if zuul is properly configured, will
create / delete the pidfile directly.
Change-Id: I51e100a293411695a31d438b5450e52498df7800
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
The proper way to shutdown the executor is to issue a 'stop' and then
wait for the executor to finish its current jobs before issuing a
'start'. Doing 'restart' doesn't ever seem to be the right thing to
do, so prevent folks from doing that.
Change-Id: I2630306a7290092c3e59f4be01a82950927bf60b
The zuul-fingergw in zuulv3 accepts the 'stop' command to stop
the daemon. This is the only valid way to stop it.
Change-Id: I86ac10c3a4d3af594fd32a5c0e316d626abf5a5e
When configuring the dir for the pidfiles in our init scripts don't
assume the path is /var/run/$NAME. The defaults are now /var/run/zuul
now and could be set to other values as well. Instead of assuming
/var/run/$NAME we instead take the dirname of $PIDFILE and configure
whatever that dir is instead.
Change-Id: Idfd5d19e5171278b9bb7005edb4ae927a01f3615
Zuulv3 now suport command socket to stop both merger and schedulers,
add logic to install new init scripts for them depending on zuulv2 or
zuulv3.
Change-Id: Iecc8c03896c145d29e42e6be6194d7db893c8515
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
ansible-playbook is getting hit with sigsegv; allow the process
to core dump so we might obtain clues.
Change-Id: Ice0bbd4abea2997ee56a92544128fd136dcef048
Otherwise this is leaked and we are not able to start zuul-executor
again.
Change-Id: I3639a74af67bfa07ad1814b7d0bc5a995e4ef14a
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
We don't yet have socket support for zuul-web, revert to default way
of doing this.
Change-Id: I2ec2c928c43def839d0ba8a212135d5414da9a52
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
We haven't added socket support yet to zuulv3, revert to the original
way of using SIGHUP for reloading zuul-scheduler.
Change-Id: Id466ccb28ea37a3b7171237e98ea08c8db7199cd
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Zuul v3 has a zuul-web service that provides web streaming and
eventually other web content. Add support for running it. Also add it to
the Apache mod_rewrite proxy info.
While we're adding things, add proxy info for plugin connections, which
should allow receiving github webhook events.
Change-Id: Ic79681b287dbd1a44469da70c680060940734f40
This reduces the log noise from gerrit -- we won't automatically
log all of the JSON values we get.
Change-Id: Ibfec14211f8c644a9c1e9104a96df5f47d5430ff
Without this patch, the SysV init scripts defined in this module will
not be recognized by systemd and puppet will fail to enable the
services. This is because between installing the init script and
starting the service, systemd needs to be reloaded. This patch adds a
new class to define the systemd reload. The conditional logic for
whether the reload exec should be in the catalog is defined in the
class. Then the file and service resources that depend on the reload
define their relationships to the new class rather than to the exec
resource so that the conditional logic does not have to be repeated for
every resource.
We also need to correct a bug in the init scripts themselves. The
Provides statement needs to give a unique service name, not just 'zuul'.
If it is not unique, insserv will error with "service zuul already
provided!" when systemd tries to enable the service.
Change-Id: Ica849094c6011806bdd0b205ba6b2b73856b7aa6
We didn't properly create our init script before, rebase from existing
zuul.init script.
We also need to add systemctl daemon-reload for scheduler / executor,
as we run them under xenial currently.
Change-Id: I0df2720750737fa7220c04a6d2d6506a45e2761e
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
zuul-executor will drop privileges to the zuul user.
Change-Id: I8f717a693e50a7e99c6d906860419161effc4718
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This is part of our zuulv3 effort. We want the ability to manage
zuul-executor on a server.
Change-Id: Iddac03a7187df47d2c72d957a50610f8b6a1dff5
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This is a halfway step to adding an explicit zuul::scheduler manifest to
run the zuulv3 scheduler daemon.
Change-Id: I99302505c4055559ee0ee09c42e69673aff57e97
This is a forceful yet clean stop. It will abort all running jobs
and return all nodes to nodepool before exiting. Zuul will
re-launch the aborted jobs because they completed without a
result.
Change-Id: Ia98a7c47259bce3277682b49821672295aa3919a
In reviewing the previous change, it became clear that these files were
very tab-happy, which makes reading changes in gertty sad. Clean that
up.
Change-Id: I922fb9b427a4147dc2368577a80f06d212e20c3e
This is the normally expected configuration for Zuul logging. This
reverts commit 5f780500c374221cb537389b6db1003042be30d5 from
the system-config repo. DEBUG level was temporarily set to help
debug a Zuul issue in production.
Of course, future changes to OpenStack's Zuul logging should be
parameterized, but even so, the value in this change should be
the default.
Change-Id: I429346047d5901ca75afa04baddc7802605010ba
The zuul.init and zuul-merger.init puppet module files did not check if
pidfiles existed when starting both services. The pidfile's might exist
in the event that the zuul-server and merger were killed, which doesn't
clean them up. This change checks for whether those files exist and notifies
the user to run the stop command which will clean up any residual pid files.
Change-Id: Ie65fdf372f1710ab0c32d03f6b359e1860f566db
Add a feature flag to allow this module to manage
its own log configuration files. This feature flag is
set to false currently.
Configuration files for logging come from
system-config/openstack-project. The related system-config
change is Ic5ceb7fb23a8403a31f9db2774ddd900fd838930.
Change-Id: I76851c8672968a7581622cac8ff0062c1f50e0ef
Set the description of the zuul-merger.init to 'Zuul Merger' to
differentiate from the main zuul-server daemon.
Change-Id: I9ee914df0f98157b2710cbc2f9c29f7b41726273
Remove obsolete config options from zuul module.
The server and merger classes are constructed so that they
may coexist on a single server. Also, the init section is
constructed so that it will install everything needed for
both services but without activating them (leaving maximum
flexibility for the operator).
Change-Id: I7b86fbbe4611c5edfb463a0a6944e0717f664188
The Zuul service opens a lot of TCP connections and files resulting in
many open file descriptors. The default limit of 2048 is bumped into
regularly. Bump the limit to 8192 when starting the Zuul service to give
it breathing room to grow into.
Change-Id: I9a3aee664c1c6c6139b012fc3dc0631725f59741
Remove zuul from jenkins server.
Remove transitional zuul config.
Add an extra argument to gerrit-git-prep for the git site name, so that it
doesn't have to be the same as the review site.
Serve zuul git repos via apache.
Change-Id: I342032bcedf317c0f48ff89b773546eb2ccd08f8
Reviewed-on: https://review.openstack.org/17949
Tested-by: Jenkins
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: James E. Blair <corvus@inaugust.com>
Change-Id: I4ddd69916db4d9df036d3d674fb58847981d2758
Reviewed-on: https://review.openstack.org/13810
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Antoine Musso <hashar@free.fr>
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
Clark Boylan