[Bug 30260] New: Script which checks index should also check if the zebraqueue has uncommitted records
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30260 Bug ID: 30260 Summary: Script which checks index should also check if the zebraqueue has uncommitted records Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs@lists.koha-community.org Reporter: aleisha@catalyst.net.nz QA Contact: testopia@bugs.koha-community.org currently our monitoring checks if the indexer is running, and sends an alert if it it isn't. however we've found that sometimes the indexer looks like it is running but the process may not have started fully, and the indexer isn't working. new records and modified records don't show in search results. when we view the zebraqueue in the database, there are rows that have not been committed, i.e. `zebraqueue.done` == 0. checkindex.sh should check if there are rows in the zebraqueue that have not been committed. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30260 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|koha-bugs@lists.koha-commun |aleisha@catalyst.net.nz |ity.org | Change sponsored?|--- |Sponsored -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30260 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30260 --- Comment #1 from Aleisha Amohia <aleisha@catalyst.net.nz> --- Created attachment 131532 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=131532&action=edit Bug 30260: Add is_indexer_working check Function checks if the zebraqueue has rows that are not 'done' Will depend on a change in checkindex.sh which implements the check: https://git.koha-community.org/Koha-community/global/src/branch/master/koha-... To test: 1. disable the indexer sudo koha-indexer --stop INSTANCENAME 2. catalogue 5 new records 3. confirm they don't show in any search results 4. in your terminal, go to the shell sudo koha-shell INSTANCENAME 5. run the new script. run with -v for verbose or --rows X to specify how many uncommitted records should be checked for. 5 is the default perl misc/maintenance/check_zebraqueue.pl 6. confirm the script returns that the indexer is not working. The function in koha-functions.sh is required for the pending change to checkindex.sh Sponsored-by: Catalyst IT -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30260 --- Comment #2 from Aleisha Amohia <aleisha@catalyst.net.nz> --- Unfortunately I can't seem to push to the repo, can someone please help? I can attach the patch here but it won't apply to a koha clone. Am trying to push to this repo: https://git.koha-community.org/Koha-community/global/ -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30260 --- Comment #3 from Aleisha Amohia <aleisha@catalyst.net.nz> --- Created attachment 131582 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=131582&action=edit Bug 30260: [GLOBAL REPO] Implement is_indexer_working check -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30260 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@ptfs-europe | |.com, tomascohen@gmail.com --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Tomas or Martin, can you help with pushing to https://git.koha-community.org/Koha-community/global/? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30260 --- Comment #5 from Tomás Cohen Arazi <tomascohen@gmail.com> --- I had no idea that repo existed. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30260 --- Comment #6 from Aleisha Amohia <aleisha@catalyst.net.nz> --- How should I get that patch into the global repo? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30260 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30260 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #131532|0 |1 is obsolete| | --- Comment #7 from David Nind <david@davidnind.com> --- Created attachment 160395 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160395&action=edit Bug 30260: Add is_indexer_working check Function checks if the zebraqueue has rows that are not 'done' Will depend on a change in checkindex.sh which implements the check: https://git.koha-community.org/Koha-community/global/src/branch/master/koha-... To test: 1. disable the indexer sudo koha-indexer --stop INSTANCENAME 2. catalogue 5 new records 3. confirm they don't show in any search results 4. in your terminal, go to the shell sudo koha-shell INSTANCENAME 5. run the new script. run with -v for verbose or --rows X to specify how many uncommitted records should be checked for. 5 is the default perl misc/maintenance/check_zebraqueue.pl 6. confirm the script returns that the indexer is not working. The function in koha-functions.sh is required for the pending change to checkindex.sh Sponsored-by: Catalyst IT Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30260 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com --- Comment #8 from David Nind <david@davidnind.com> --- Patch still applies and it worked for me: I had to add 6 new records for the default of 5 to work. Maybe check with Jonathan about the repo - probably something to do with access when we changed to Gitea. Maybe the repository should be added to gitlab.com/koha-community -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30260 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #131582|0 |1 is obsolete| | --- Comment #9 from Aleisha Amohia <aleisha@catalyst.net.nz> --- Comment on attachment 131582 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=131582 Bug 30260: [GLOBAL REPO] Implement is_indexer_working check This patch has now been pushed to the global repo, so the other patch can be pushed to Koha. https://git.koha-community.org/Koha-community/global/commit/72308768457017c3... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30260 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |DUPLICATE Status|Signed Off |RESOLVED --- Comment #10 from Aleisha Amohia <aleisha@catalyst.net.nz> --- I don't think we need this fix anymore, with a combination of the following fixes instead: * Bug 36548 (the bug solved here was the believed cause for this fix) * https://git.koha-community.org/Koha-community/global/commit/704060f4ff5497b5... *** This bug has been marked as a duplicate of bug 36548 *** -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org