-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20174/
-----------------------------------------------------------

Review request for Aurora.


Repository: aurora


Description
-------

It seems thermos_observer, thermos_executor, and gc_executor did not get copied 
or linked to somewhere in the default PATH after the refactor to only one 
instance under Vagrant. This resulted in /etc/init/aurora-thermos-observer.conf 
to fail, and thermos_observer where not started.

I am unsure where the proper place to link is, but I tried to mimic the 
structure from before the refactor, i.e. link them under /usr/local/bin (and 
updated the run script accordingly).

Also note that I am not terribly familiar with Aurora, so there may be things I 
have misunderstood. The only thing I noticed was the thermos_observer did not 
start automatically, and this seems to be fixed by the patch.


Diffs
-----

  examples/vagrant/provision-dev-cluster.sh 
ec9554356cd789d308a5adce4784b04abe64ab95 
  examples/vagrant/upstart/aurora-thermos-observer.conf 
6db85aad5b0c2d484656bfb5d5bfa4ad4c9b3342 

Diff: https://reviews.apache.org/r/20174/diff/


Testing
-------

I have tested the patch using Vagrant on OS X 10.9.2, and it seems to work as 
intended.


Thanks,

Johan Liseborn

Reply via email to