Bug 16822

Summary: koha-common init.d script should run koha-plack without quiet
Product: Koha Reporter: Liz Rea <liz>
Component: Command-line UtilitiesAssignee: Tomás Cohen Arazi <tomascohen>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: trivial    
Priority: P5 - low CC: f.demians, julian.maurice, kyle, m.de.rooy, robin, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 16822: Make koha-common call koha-plack without --quiet
Bug 16822: Make koha-common call koha-plack without --quiet
Bug 16822: Make koha-common call koha-plack without --quiet

Description Liz Rea 2016-06-28 00:45:07 UTC
We should alert during installation, restart, or upgrade for all daemon services stopping and starting.

For example, stopping and starting the SIP server alerts during the apt install process. 

We expect (and sometimes depend on) things like Apache and MySQL(MariaDB) telling us when they are restarting during upgrades, I think we should treat our daemons the same way.

All this really means is making the init.d run koha-plack without --quiet.
Comment 1 Tomás Cohen Arazi 2016-07-26 13:30:12 UTC Comment hidden (obsolete)
Comment 2 Liz Rea 2016-09-09 03:49:57 UTC Comment hidden (obsolete)
Comment 3 Marcel de Rooy 2016-09-09 06:33:09 UTC
Created attachment 55395 [details] [review]
Bug 16822: Make koha-common call koha-plack without --quiet

This patch removes the --quiet option switch in koha-common.init so
problem information is not hidden for sysadmin users.

To test:

- Have plack enabled for a koha instance:
  $ sudo koha-plack --enable kohadev
  $ sudo koha-plack --start  kohadev
- Run:
  $ cd kohaclone
  $ debian/koha-common.init {start|stop|restart}
=> SUCCESS: No behaviour change
- Disable the 'cgi' apache module:
  $ sudo a2dismod cgi
- Run:
  $ debian/koha-common.init {start|stop|restart}
=> FAIL: No warning is shown
- Apply the patch
- Run:
  $ debian/koha-common.init {start|stop|restart}
=> SUCCESS: Warnings show up
- Sign off :-D

Signed-off-by: Liz Rea <liz@catalyst.net.nz>
Perfect thanks :)

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 4 Kyle M Hall 2016-09-09 10:26:53 UTC
Pushed to master for 16.11, thanks Tomas!
Comment 5 Frédéric Demians 2016-09-13 06:45:39 UTC
Pushed in 16.05. Will be in 16.05.04.
Comment 6 Julian Maurice 2016-09-16 08:39:50 UTC
Pushed to 3.22.x, will be in 3.22.11