> On April 6, 2016, 10:37 p.m., Jayush Luniya wrote: > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1.2.2/package/scripts/params.py, > > line 143 > > <https://reviews.apache.org/r/45837/diff/1/?file=1328886#file1328886line143> > > > > @Sriharsha > > > > There is a typo here that we missed in the CR. Missing ')' > > Can you fix it? > > Sriharsha Chintalapani wrote: > Thanks for the catch.
Can you run the unit tests for stacks (you can change the root of the path for your machine): cd /Users/smohanty/enlistments/asf-ambari/ambari-server/src/test/python export PYTHONPATH=/Users/smohanty/enlistments/asf-ambari/ambari-common/src/main/python:/Users/smohanty/enlistments/asf-ambari/ambari-agent/src/main/python:/Users/smohanty/enlistments/asf-ambari/ambari-common/src/main/python/ambari_jinja2:/Users/smohanty/enlistments/asf-ambari/ambari-common/src/main/python/ambari_commons:/Users/smohanty/enlistments/asf-ambari/ambari-common/src/test/python:/Users/smohanty/enlistments/asf-ambari/ambari-server/src/main/python:/Users/smohanty/enlistments/asf-ambari/ambari-server/src/main/python/ambari-server-state:/Users/smohanty/enlistments/asf-ambari/ambari-server/src/main/resources/custom_actions:/Users/smohanty/enlistments/asf-ambari/ambari-server/src/main/resources/scripts:/Users/smohanty/enlistments/asf-ambari/ambari-server/src/test/python /Users/smohanty/enlistments/asf-ambari/ambari-common/src/main/unix/ambari-python-wrap unitTests.py - Sumit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45837/#review127481 ----------------------------------------------------------- On April 6, 2016, 9:12 p.m., Sriharsha Chintalapani wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45837/ > ----------------------------------------------------------- > > (Updated April 6, 2016, 9:12 p.m.) > > > Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. > > > Repository: ambari > > > Description > ------- > > AMBARI-15747. Kafka jaas files are not generated on a Kafa upgrade if > security.inter.broker.protocol is set to SASL_PLAINTEXT > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/KAFKA/0.8.1.2.2/package/scripts/params.py > b18e62d > > Diff: https://reviews.apache.org/r/45837/diff/ > > > Testing > ------- > > > Thanks, > > Sriharsha Chintalapani > >