Bug 40722 - Add logging to reset of elastic mappings files when rebuilding elastic
Summary: Add logging to reset of elastic mappings files when rebuilding elastic
Status: Pushed to main
Alias: None
Product: Koha
Classification: Unclassified
Component: Command-line Utilities (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Nick Clemens (kidclamp)
QA Contact: Martin Renvoize (ashimema)
URL:
Keywords: release-notes-needed
Depends on:
Blocks:
 
Reported: 2025-08-28 16:20 UTC by Nick Clemens (kidclamp)
Modified: 2025-10-15 22:15 UTC (History)
3 users (show)

See Also:
GIT URL:
Initiative type: ---
Sponsorship status: ---
Crowdfunding goal: 0
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
25.11.00
Circulation function:


Attachments
Bug 40722: Add a log when resetting mappings during elastic rebuild (1.36 KB, patch)
2025-08-28 16:22 UTC, Nick Clemens (kidclamp)
Details | Diff | Splinter Review
Bug 40722: Add a log when resetting mappings during elastic rebuild (1.41 KB, patch)
2025-08-28 16:23 UTC, Nick Clemens (kidclamp)
Details | Diff | Splinter Review
Bug 40722: Add a log when resetting mappings during elastic rebuild (1.48 KB, patch)
2025-10-01 13:23 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Clemens (kidclamp) 2025-08-28 16:20:04 UTC

    
Comment 1 Nick Clemens (kidclamp) 2025-08-28 16:22:39 UTC
Created attachment 185852 [details] [review]
Bug 40722: Add a log when resetting mappings during elastic rebuild

The command currently logs when deleting indexes and indexing records, it would be helpful
to have a log when resetting the mappings, and to log which mapping file has been used.

To test:
1 - perl misc/search_tools/rebuild_elasticsearch.pl -r
2 - No log of reset
3 - Apply patch
4 - perl misc/search_tools/rebuild_elasticsearch.pl -r
5 - Note reset is logged
Comment 2 Nick Clemens (kidclamp) 2025-08-28 16:23:21 UTC
Created attachment 185853 [details] [review]
Bug 40722: Add a log when resetting mappings during elastic rebuild

The command currently logs when deleting indexes and indexing records, it would be helpful
to have a log when resetting the mappings, and to log which mapping file has been used.

To test:
1 - perl misc/search_tools/rebuild_elasticsearch.pl -r
2 - No log of reset
3 - Apply patch
4 - perl misc/search_tools/rebuild_elasticsearch.pl -r
5 - Note reset is logged

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 3 Nick Clemens (kidclamp) 2025-08-28 16:23:39 UTC
Trivial logging change, moving to QA
Comment 4 Martin Renvoize (ashimema) 2025-10-01 13:23:58 UTC
Created attachment 187191 [details] [review]
Bug 40722: Add a log when resetting mappings during elastic rebuild

The command currently logs when deleting indexes and indexing records, it would be helpful
to have a log when resetting the mappings, and to log which mapping file has been used.

To test:
1 - perl misc/search_tools/rebuild_elasticsearch.pl -r
2 - No log of reset
3 - Apply patch
4 - perl misc/search_tools/rebuild_elasticsearch.pl -r
5 - Note reset is logged

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Comment 5 Lucas Gass (lukeg) 2025-10-15 22:15:37 UTC
Nice work everyone!

Pushed to main for 25.11