Bug 40304 - Zebrasrv config doesn't consider non-AMD64 CPUs
Summary: Zebrasrv config doesn't consider non-AMD64 CPUs
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: Searching - Zebra (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial
Assignee: Jake Deery
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-07-04 08:32 UTC by Jake Deery
Modified: 2025-07-04 10:47 UTC (History)
3 users (show)

See Also:
GIT URL:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 40304: Zebrasrv configs now work on ARM CPUs (3.31 KB, patch)
2025-07-04 08:43 UTC, Jake Deery
Details | Diff | Splinter Review
Bug 40304: Zebrasrv configs now work on ARM CPUs (3.35 KB, patch)
2025-07-04 08:54 UTC, Paul Derscheid
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jake Deery 2025-07-04 08:32:52 UTC
Hi folks,

As per this Mattermost conversation: https://chat.koha-community.org/koha-community/pl/ft6e47akutn6fbdg5tpwab5zbw 

I've identified an issue where running t/db_dependent/Search.t fails on AARCH64 machines. Upon investigation, I've found that in the default zebra-biblios-dom.cfg and zebra-authorities-dom.cfg files, the module search path does not include as many architectures as it should.

I have rectified this, by copying the lines found in the debian/templates folder to the relevant config files mentioned in the last paragraph.

Could I ask someone to check the patch I'm about to push, so we can get a quick conclusion to this bug?

Ta,
Jake
Comment 1 Jake Deery 2025-07-04 08:43:57 UTC
Created attachment 183783 [details] [review]
Bug 40304: Zebrasrv configs now work on ARM CPUs

This patch adds more module search paths to the zebra-biblios-dom.cfg and zebra-authorities-dom.cfg, so that a larger range of CPU architectures will work with Zebra search unit tests.

This will not change Debian package installs, but it will fix any git-based installs.

TO TEST (you'll need access to an ARM-based CPU, and, preferably, an x86-64 machine):
ON THE ARM-BASED MACHINE:
a)  start ktd, then ktd --shell
b)  in ktd, run: prove -v t/db_dependent/Search.t
c)  notice how the test now fails
APPLY PATCH
d)  rerun step b
e)  notice how the test now passes
ON THE X86-64-BASED MACHINE:
f)  run steps a-e
g)  notice how the test passes, both before and after applying the patch
SIGN OFF
Comment 2 Paul Derscheid 2025-07-04 08:54:05 UTC
Created attachment 183784 [details] [review]
Bug 40304: Zebrasrv configs now work on ARM CPUs

This patch adds more module search paths to the zebra-biblios-dom.cfg and zebra-authorities-dom.cfg, so that a larger range of CPU architectures will work with Zebra search unit tests.

This will not change Debian package installs, but it will fix any git-based installs.

TO TEST (you'll need access to an ARM-based CPU, and, preferably, an x86-64 machine):
ON THE ARM-BASED MACHINE:
a)  start ktd, then ktd --shell
b)  in ktd, run: prove -v t/db_dependent/Search.t
c)  notice how the test now fails
APPLY PATCH
d)  rerun step b
e)  notice how the test now passes
ON THE X86-64-BASED MACHINE:
f)  run steps a-e
g)  notice how the test passes, both before and after applying the patch
SIGN OFF

Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de>