Bug 11078: Add locking to rebuild_zebra
authorDoug Kingston <dpk@randomnotes.org>
Wed, 19 Feb 2014 06:56:58 +0000 (22:56 -0800)
committerGalen Charlton <gmc@esilibrary.com>
Fri, 28 Feb 2014 22:21:41 +0000 (22:21 +0000)
commit88e7faf8600b64884649f8c02104bf848df37a9b
tree3fbc940f950a26196b65c3858631a07aa6a12c5d
parentdaf2ebc4f5aeaf3a73e11915a17a2a81eafc5cb9
Bug 11078: Add locking to rebuild_zebra

This patch adds locking to rebuild_zebra.pl to ensure that simultaneous
changes are prevented (as one is likely to overwrite the other).
Incremental updates in daemon mode will skipped if the lock is busy
and they will be picked up on the next pass.  Non-daemon mode
invocations will also exit immediately if they cannot get the lock
unless the new flag -wait-for-lock is specified, in which case they
will wait until the get the lock and then proceed.

Supporting changes made to Makefile.PL and templates for the new
locking directory (paralleling the other zebra lock directories).
We stash the zebra_lockdir in koha-conf.xml so rebuild_zebra.pl
can find it.

To address earlier QA concerns we:
1. added code to check if flock is available and ignore locking if
it's missing (from M. de Rooy)

2. changed default for adhoc invocations to abort if they cannot
obtain the lock.  Added option -wait-for-lock if the user prefers
to wait until the lock is free, and then continue processing.

3. added missing entry to t/db_dependent/zebra_config.pl

4. added a fallback locking directory of /tmp

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Doug merged the original patch with the QA changes.
Just for the record, noting here that the original patch was tested
extensively too by Martin Renvoize.
I have added a followup for some exceptional cases.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Makefile.PL
debian/templates/koha-conf-site.xml.in
etc/koha-conf.xml
misc/bin/koha-zebra-ctl.sh
misc/migration_tools/rebuild_zebra.pl
skel/var/lock/koha/zebradb/rebuild/README [new file with mode: 0644]
t/db_dependent/zebra_config.pl