Add recipe for adding official varnish project apt_repo (v2.1 & v3.0, plus more OS/distro support)

Patrick Connolly added a comment - 05/Apr/12 3:07 PM Nothing even changes until you add the apt_repo recipe, and even then, it still adds 2.1 by default (likely more platforms, although probably needs more logic for that) https://github.com/opscode-cookbooks/varnish/pull/2

Patrick Connolly added a comment - 05/Apr/12 3:16 PM Seems that at the very minimum, varnish v3.0 needs a restart in order for our /etc/default/varnish template DAEMON_OPTS to take affect... ~$ tail -13 Vagrantfile config.vm.provision :chef_solo do |chef| chef.cookbooks_path = "cookbooks" chef.add_recipe "varnish::varnish_repo" chef.add_recipe "apt" chef.add_recipe "varnish" chef.json = { :varnish => { :version => "3.0" } } end end ~$ vagrant up [ default ] Importing base box 'lucid64'... ... [Thu, 05 Apr 2012 08:03:28 -0700] INFO: Report handlers complete ~$ vagrant ssh ~vagrant$ ps ax | grep -i varnish | grep -v grep 1289 ? Ss 0:00 /usr/sbin/varnishd -P / var /run/varnishd.pid -a :6081 -T localhost:6082 -f /etc/varnish/ default .vcl -S /etc/varnish/secret -s malloc,256m 1291 ? Sl 0:00 /usr/sbin/varnishd -P / var /run/varnishd.pid -a :6081 -T localhost:6082 -f /etc/varnish/ default .vcl -S /etc/varnish/secret -s malloc,256m ~vagrant$ sudo service varnish restart * Stopping HTTP accelerator varnishd ...done. * Starting HTTP accelerator varnishd ...done. ~vagrant$ ps ax | grep -i varnish | grep -v grep 1568 ? Ss 0:00 /usr/sbin/varnishd -P / var /run/varnishd.pid -a :6081 -f /etc/varnish/ default .vcl -T 127.0.0.1:6082 -u varnish -g varnish -t 120 -w 5,500,300 -s file,/ var /lib/varnish/lucid64/varnish_storage.bin,1G -S /etc/varnish/secret -n lucid64 1569 ? Sl 0:00 /usr/sbin/varnishd -P / var /run/varnishd.pid -a :6081 -f /etc/varnish/ default .vcl -T 127.0.0.1:6082 -u varnish -g varnish -t 120 -w 5,500,300 -s file,/ var /lib/varnish/lucid64/varnish_storage.bin,1G -S /etc/varnish/secret -n lucid64 This might also affect how the varnish ['start'] attr works, as it seems varnish starts automatically for v3.0

Patrick Connolly added a comment - 05/Apr/12 3:20 PM Just tossing that last comment out there for the convo, but since 2.1 works just the same by default, I see no reason not to bring this into the cookbook, so that people can start contributing the fixes that get the cookbook working on more platforms and varnish versions. Thanks!

Patrick Connolly added a comment - 05/Apr/12 3:28 PM Actually, just adding a restart notification solved the issue in v3.0. Misunderstood varnish ['start'] attr, and adding that had no effect. Should be good for a pull if you don't have unless you have any concerns

Patrick Connolly added a comment - 05/Apr/12 3:32 PM Just waiting on review. Oy, I didn't realize we needed to mark things as "Resolved" for consideration... This would explain some things

Patrick Connolly added a comment - 05/Apr/12 3:38 PM For posterity, here are the major changes that happen between 2.1 and 3.0, which we maaaay want to be wary of in small ways: https://www.varnish-cache.org/docs/trunk/installation/upgrade.html

jtimberman added a comment - 09/Apr/12 7:30 PM This is cool, and merged. Thanks!

jtimberman added a comment - 09/Apr/12 7:44 PM I renamed the recipe to apt_repo to make it more clear about the intention. This is released in varnish v0.9.0^H2. Thanks!

Patrick Connolly added a comment - 10/Apr/12 1:43 AM Wooo! First full recipe accepted. Thanks JT! I'll give myself an imaginary achievement badge. There's a picture of a meerkat on it. I'm not exactly sure why.