----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47427/#review133528 -----------------------------------------------------------
Ship it! Ship It! - Robert Levas On May 16, 2016, 5:53 p.m., Tom Beerbower wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47427/ > ----------------------------------------------------------- > > (Updated May 16, 2016, 5:53 p.m.) > > > Review request for Ambari, John Speidel and Robert Levas. > > > Bugs: AMBARI-16693 > https://issues.apache.org/jira/browse/AMBARI-16693 > > > Repository: ambari > > > Description > ------- > > The error 'kafka_broker_hosts' was not found in configurations dictionary! is > seen during upgrade ... > > "stderr" : "Traceback (most recent call last):\n File > \"/var/lib/ambari-agent/cache/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata_server.py\", > line 165, in <module>\n MetadataServer().execute()\n File > \"/usr/lib/python2.6/site-packages/resource_management/libraries/script/script.py\", > line 254, in execute\n method(env)\n File > \"/var/lib/ambari-agent/cache/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata_server.py\", > line 82, in stop\n import params\n File > \"/var/lib/ambari-agent/cache/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py\", > line 139, in <module>\n if not len(kafka_broker_hosts) == 0:\n File > \"/usr/lib/python2.6/site-packages/resource_management/libraries/script/config_dictionary.py\", > line 73, in __getattr__\n raise Fail(\"Configuration parameter '\" + > self.name + \"' was not found in configurations > dictionary!\")\nresource_management.core.exceptions.Fail: Configuration > parameter 'kafka_broker_hosts' was not found in c onfigurations dictionary!", > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py > fb4a55f > > Diff: https://reviews.apache.org/r/47427/diff/ > > > Testing > ------- > > manual test install > > mvn clean test > > > Thanks, > > Tom Beerbower > >