----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32791/#review78794 -----------------------------------------------------------
Ship it! Ship It! - Jonathan Hurley On April 2, 2015, 5:47 p.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32791/ > ----------------------------------------------------------- > > (Updated April 2, 2015, 5:47 p.m.) > > > Review request for Ambari, Jaimin Jetly, Jonathan Hurley, Sid Wagle, and > Yusaku Sako. > > > Bugs: AMBARI-10317 > https://issues.apache.org/jira/browse/AMBARI-10317 > > > Repository: ambari > > > Description > ------- > > When an Ubuntu 12 host is restarted, its /var/run/knox folder is deleted. > There's a bug in knox.py in which it doesn't check that all of the > directories it needs are created. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/KNOX/0.5.0.2.2/package/scripts/knox.py > 108ebb2 > ambari-server/src/test/python/stacks/2.2/KNOX/test_knox_gateway.py b1d9888 > > Diff: https://reviews.apache.org/r/32791/diff/ > > > Testing > ------- > > Spun up a 1 node cluster with Ubuntu, installed Knox and then stoped it, > restarted the machine, and Knox failed to start. > After deploying the patch, was able to start Knox. > > Local python unit tests passed, > ---------------------------------------------------------------------- > Total run:654 > Total errors:0 > Total failures:0 > OK > > > Thanks, > > Alejandro Fernandez > >
