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.

CommentFileSizeAuthor
#4 varnish_multi_server.patch21 KBwouters_f
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kmasood’s picture

Ping. Anyone ... ?

kmasood’s picture

This works:

  $conf['varnish_control_terminal'] = '11.22.33.44:6082 22.33.44.55:6082 55.44.33.11:6082';

notice, no array.

Hopefully, helps someone else.

Thanks NL!

nerdcore’s picture

I 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.

wouters_f’s picture

Issue summary: View changes
FileSize
21 KB

Hi nerdcore, I have a patch for thi. See attachement.

t0xicCode’s picture

@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.