Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I am trying to set $conf['varnish_control_terminal'] using this:
$conf['varnish_control_terminal'] = array('11.22.33.44:6082 22.33.44.55:6082 55.44.33.11:6082');
or
$conf['varnish_control_terminal'] = array('11.22.33.44:6082','22.33.44.55:6082','55.44.33.11:6082');
but I get this error three times (once for each host ... ?) at http://host/admin/config/development/varnish:
Warning: Invalid argument supplied for foreach() in _varnish_terminal_run() (line 196 of /app/aegir/platforms/gwu-cms-1.0b/sites/all/modules/contrib/varnish/varnish.module).
This works just fine with one host:
$conf['varnish_control_terminal'] = array('11.22.33.44:6082');
I think I need the right syntax, but can't seem to figure it out.
Comment | File | Size | Author |
---|---|---|---|
#4 | varnish_multi_server.patch | 21 KB | wouters_f |
Comments
Comment #1
kmasood CreditAttribution: kmasood commentedPing. Anyone ... ?
Comment #2
kmasood CreditAttribution: kmasood commentedThis works:
notice, no array.
Hopefully, helps someone else.
Thanks NL!
Comment #3
nerdcore CreditAttribution: nerdcore commentedI have set this up using the space-separated syntax within the Varnish module's config pages.
Varnish Control Terminal:
varnish1:6082 varnish2:6082
But if these did not have their Varnish Control Key copied between them, is there a way to specify a Control Key for each instance?
Thanks.
Comment #4
wouters_f CreditAttribution: wouters_f commentedHi nerdcore, I have a patch for thi. See attachement.
Comment #5
t0xicCode CreditAttribution: t0xicCode at OpenConcept Consulting Inc. commented@wouters_f Would you mind re-rolling that patch on top of the current development branch? The patch you provided is on top of a release, not the latest development branch and is relative to a drupal root, not the module root.