koha-rebuild-zebra should check stuff like if the instance exists, parameters like -a/-b that should make the DB being reindexed twice and handle errors gracefuly.
Created attachment 17846 [details] [review] Bug 10159 - koha-rebuild-zebra error handling This patch makes koha-rebuild-zebra: - Check for the existence of the instance - Handle arbitrary amount of instance parameters - Capture -a and -b (which made the script rebuild twice the chosen DB). It adds the extended version (--authorities and --biblios respectively) of the option switch. This makes the -a and -b switches work as "only do authorities" (or viceversa). They can be used concurrently, which is the default behaviour when no switch is provided. - Handle -v (and --verbose) as koha-rebuild-zebra options, to ease the use for non-techie users. - The rebuild_zebra.pl is wrapped inside an IF to return error values if we ever make rebuild_zebra.pl return error codes (this is pointed in the code). - Added -h/--help switches and an 'usage' output :-D Please provide patches for any wording issues, or contact me to fix it. To test: - Apply the patch and create your packages (or just use the script from your branch) - Run it in this scenarios / options - -h/--help to see the available option switches - Mix them with one or more instances, of which some could be fake - try -a/--authorities and -b/--biblios in all possible combinations - try -v/--verbose works Regards To+ Sponsored-by: Universidad Nacional de Córdoba
Created attachment 18105 [details] [review] Bug 10159 - koha-rebuild-zebra error handling This patch makes koha-rebuild-zebra: - Check for the existence of the instance - Handle arbitrary amount of instance parameters - Capture -a and -b (which made the script rebuild twice the chosen DB). It adds the extended version (--authorities and --biblios respectively) of the option switch. This makes the -a and -b switches work as "only do authorities" (or viceversa). They can be used concurrently, which is the default behaviour when no switch is provided. - Handle -v (and --verbose) as koha-rebuild-zebra options, to ease the use for non-techie users. - The rebuild_zebra.pl is wrapped inside an IF to return error values if we ever make rebuild_zebra.pl return error codes (this is pointed in the code). - Added -h/--help switches and an 'usage' output :-D Please provide patches for any wording issues, or contact me to fix it. To test: - Apply the patch and create your packages (or just use the script from your branch) - Run it in this scenarios / options - -h/--help to see the available option switches - Mix them with one or more instances, of which some could be fake - try -a/--authorities and -b/--biblios in all possible combinations - try -v/--verbose works Regards To+ Sponsored-by: Universidad Nacional de Córdoba Signed-off-by: Robin Sheat <robin@catalyst.net.nz>
There is one (maybe) problem with this patch, but as it already exists as far as I can tell, I am going to push the patch as-is for 3.12.0: koha-rebuild-zebra does not check if an instance is disabled before reindexing. Arguably it should.
This patch has been pushed to master and 3.12.x.
(In reply to comment #3) > There is one (maybe) problem with this patch, but as it already exists as > far as I can tell, I am going to push the patch as-is for 3.12.0: > koha-rebuild-zebra does not check if an instance is disabled before > reindexing. Arguably it should. Specifically it shouldn't. If I'm recovering an instance where the indices have broken, I'll put it into maintenance mode by disabling it while I reindex rather than have people have issues with a half-working system.
(In reply to comment #5) > (In reply to comment #3) > > There is one (maybe) problem with this patch, but as it already exists as > > far as I can tell, I am going to push the patch as-is for 3.12.0: > > koha-rebuild-zebra does not check if an instance is disabled before > > reindexing. Arguably it should. > > Specifically it shouldn't. If I'm recovering an instance where the indices > have broken, I'll put it into maintenance mode by disabling it while I > reindex rather than have people have issues with a half-working system. Fair enough. However, it might be worth noting that in the help, as all the other Zebra scripts do not do anything when the instance is disabled.
Pushed to 3.10.x, will be in 3.10.7
Pushed to 3.8.x, will be in 3.8.14