Bug 18105 - Should we restart koha-indexer differently i.e. like koha-plack?
Summary: Should we restart koha-indexer differently i.e. like koha-plack?
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Packaging (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Tomás Cohen Arazi
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-02-13 18:22 UTC by Marcel de Rooy
Modified: 2023-09-10 12:38 UTC (History)
4 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Marcel de Rooy 2017-02-13 18:22:06 UTC
Koha-indexer uses a daemon --restart:
A daemon restart does not really stop and start; the client processes are terminated, but the named daemon itself is not. Changed options are not taken into account.

Same for koha-restart-zebra btw.

Koha-plack implements its restart by doing a daemon --stop and daemon --start.
This makes that changed options are taken into account.

If you know how this work, no problem. But it is not consistent.
Comment 1 Magnus Enger 2017-02-16 07:31:14 UTC
(In reply to Marcel de Rooy from comment #0)
> If you know how this work, no problem. But it is not consistent.

+1 for consistency!
Comment 2 Marc Véron 2017-02-18 15:22:51 UTC
+1
Comment 3 Tomás Cohen Arazi 2019-11-29 18:56:11 UTC
I'm taking this one, as I'll be revisiting all the daemon handling scripts as part of the containerization effort.
Comment 4 Marcel de Rooy 2019-12-06 07:18:28 UTC
(In reply to Tomás Cohen Arazi from comment #3)
> I'm taking this one, as I'll be revisiting all the daemon handling scripts
> as part of the containerization effort.

Great! I am all for containerization. Did I miss something on this "containerization effort" ?
Comment 5 Katrin Fischer 2023-09-10 12:38:04 UTC
(In reply to Marcel de Rooy from comment #4)
> (In reply to Tomás Cohen Arazi from comment #3)
> > I'm taking this one, as I'll be revisiting all the daemon handling scripts
> > as part of the containerization effort.
> 
> Great! I am all for containerization. Did I miss something on this
> "containerization effort" ?

ping? :)