----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27030/#review57780 -----------------------------------------------------------
I see we don't use path attribute now. Let's remove it in that case / or use it path = ForcedListArgument(default=[]) - Andrew Onischuk On Oct. 22, 2014, 1:02 p.m., Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27030/ > ----------------------------------------------------------- > > (Updated Oct. 22, 2014, 1:02 p.m.) > > > Review request for Ambari, Andrew Onischuk and Vitalyi Brodetskyi. > > > Bugs: AMBARI-7898 > https://issues.apache.org/jira/browse/AMBARI-7898 > > > Repository: ambari > > > Description > ------- > > Using Ambari 1.7 to install Knox results in a installation error. The error > is due to the missing directory /usr/hdp/current/. Knox is installed in > /usr/hdp/2.2.0.0-913/knox/ but has no links in the directory /usr/hdp/current. > When other components like HDFS, Yarn etc are installed at the same time, the > installation of knox works fine since the directory /usr/hdp/current is > indeed created. > > > Diffs > ----- > > ambari-common/src/main/python/resource_management/core/providers/system.py > 61194bb > ambari-common/src/main/python/resource_management/core/shell.py ce3ae97 > > ambari-server/src/main/resources/stacks/HDP/2.2/services/KNOX/package/scripts/knox.py > abd38bd > > ambari-server/src/main/resources/stacks/HDP/2.2/services/KNOX/package/scripts/knox_gateway.py > ccb5450 > > ambari-server/src/main/resources/stacks/HDP/2.2/services/KNOX/package/scripts/params.py > 5c5abe4 > ambari-server/src/test/python/stacks/2.2/KNOX/test_knox_gateway.py e61e787 > > Diff: https://reviews.apache.org/r/27030/diff/ > > > Testing > ------- > > OK > ---------------------------------------------------------------------- > Total run:667 > Total errors:0 > Total failures:0 > OK > > > Thanks, > > Dmytro Sen > >
