Bug 18105

Summary:	Should we restart koha-indexer differently i.e. like koha-plack?
Product:	Koha	Reporter:	Marcel de Rooy <m.de.rooy>
Component:	Packaging	Assignee:	Tomás Cohen Arazi (tcohen) <tomascohen>
Status:	NEW ---	QA Contact:	Testopia <testopia>
Severity:	enhancement
Priority:	P5 - low	CC:	magnus, mirko, tomascohen, veron
Version:	Main
Hardware:	All
OS:	All
Change sponsored?:	---	Patch complexity:	---
Documentation contact:		Documentation submission:
Text to go in the release notes:		Version(s) released in:
Circulation function:

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 (tcohen) 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? :)