cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Miguel Ferreira" <>
Subject Re: Review Request 23847: Check if config specifies zones before iterating
Date Wed, 23 Jul 2014 12:10:27 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated July 23, 2014, 12:10 p.m.)

Review request for cloudstack, daan Hoogland, Santhosh Edukulla, and Hugo Trippaers.


Address remark

Repository: cloudstack-git


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/",
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 (updated)

  tools/marvin/marvin/ ce9b43f 



Tests now run without the mentioned exception


Miguel Ferreira

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message