> On July 23, 2014, 2:58 p.m., Santhosh Edukulla wrote: > > Ship It!
Pushed the patch to master: 9f4f921..bb1c70b master -> master One small note though. Please add branch and bug information to review requests. - Santhosh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23847/#review48509 ----------------------------------------------------------- On July 23, 2014, 12:17 p.m., Miguel Ferreira wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23847/ > ----------------------------------------------------------- > > (Updated July 23, 2014, 12:17 p.m.) > > > Review request for cloudstack, daan Hoogland, Santhosh Edukulla, and Hugo > Trippaers. > > > Repository: cloudstack-git > > > Description > ------- > > Running a marvin test that does not specify a zone in the config file will > produce an exception while parsing the config: > > Exception Occurred Under init ['Traceback (most recent call last):\n', ' > File "/usr/local/lib/python2.7/site-packages/marvin/marvinInit.py", line 118, > in __setHypervisorAndZoneInfo\n for zone in self.__parsedConfig.zones:\n', > "TypeError: 'NoneType' object is not iterable\n"] > > This happens when the code starts iterating over a list of zones that does > not exist. > I've added a check for the existence of the list of zones before iterating. > > > Diffs > ----- > > tools/marvin/marvin/marvinInit.py ce9b43f > > Diff: https://reviews.apache.org/r/23847/diff/ > > > Testing > ------- > > Tests now run without the mentioned exception > > > Thanks, > > Miguel Ferreira > >
