Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 629066 - net-proxy/squid init script and multiple instances
Summary: net-proxy/squid init script and multiple instances
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal major (vote)
Assignee: Mikle Kolyada (RETIRED)
URL:
Whiteboard:
Keywords: PullRequest
Depends on:
Blocks:
 
Reported: 2017-08-27 13:58 UTC by Vieri
Modified: 2019-03-04 20:02 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
patch for the init script (squid.initd-r5.patch,1.81 KB, patch)
2017-08-27 13:58 UTC, Vieri
Details | Diff
squid init script (squid.initd-r5,3.57 KB, text/plain)
2017-11-09 13:20 UTC, Vieri
Details
squid confd file (squid.confd-r2,717 bytes, text/plain)
2017-11-09 13:20 UTC, Vieri
Details
squid init script (squid.initd-r5,4.12 KB, text/plain)
2017-11-16 11:06 UTC, Vieri
Details
squid init script (squid.initd-r5,4.19 KB, text/plain)
2017-11-16 17:39 UTC, Vieri
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Vieri 2017-08-27 13:58:42 UTC
Created attachment 490872 [details, diff]
patch for the init script

Currently, the Gentoo Squid package ships an init script that allows the configuration of multiple instances with the use of SVCNAME, but does not take into account that service names have to be unique according to https://wiki.squid-cache.org/MultipleInstances.

The attached patch for the init script deals with the -n option which is OK for both single and multi setups.

Apart from this, maybe a note and link to https://wiki.squid-cache.org/MultipleInstances should also be added as an einfo in the squid ebuild post-installation step so the user is aware about other settings such as "unique_hostname", "pid_filename", "cache_dir", "cache_log", etc.
Comment 1 Vieri 2017-09-29 08:36:08 UTC
Could please let me know if you are considering merging this to the official tree or not?
Thanks
Comment 2 Vieri 2017-11-09 13:20:04 UTC
Created attachment 503268 [details]
squid init script

I'm attaching a new init script for Squid. It includes support for "multiple instances", but also adds a double-shutdown option to allow faster shutdown of the squid process.
See attached conf file too.
Comment 3 Vieri 2017-11-09 13:20:58 UTC
Created attachment 503270 [details]
squid confd file

Adds support for shutting down squid faster.
Comment 4 Vieri 2017-11-16 11:06:24 UTC
Created attachment 504496 [details]
squid init script

I'm adding yet another Squid init script which includes my previous comments + adds support for checking include conf files for directives such as pid_filename.
Comment 5 Vieri 2017-11-16 17:39:54 UTC
Created attachment 504520 [details]
squid init script

replaces previous init script with minor changes
Comment 6 Mikle Kolyada (RETIRED) archtester Gentoo Infrastructure gentoo-dev Security 2019-03-04 20:02:04 UTC
Should be addressed in 4.x bump