cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gaurav Aradhye" <>
Subject Review Request 26165: CLOUDSTACK-7646: - Fixed basic indentation issues, list index issues, marked it as invalid because the test cases needs attention by the original author as it does not pass even after fixing the issues. Also moving it to smoke folder as clearly it is a Basic Validation test
Date Tue, 30 Sep 2014 05:53:13 GMT

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

Review request for cloudstack, Suresh Ramamurthy and SrikanteswaraRao Talluri.


Repository: cloudstack-git


It fails even after fixing the indetation and list index issues.
Also it has hard coded information about the nuage VSP plugin.

I request Suresh to take a look, move the hard coded information to config and read it from
Also it should be ok to move test to Smoke as it is a basic validation test?
Inputs from Suresh would be helpful.

Changes in this patch:
1. Fix indentation
2. Fix list index issues
3. Fix pep8 issues
4. Move it to Smoke folder


  test/integration/component/ 096b9d1 
  test/integration/smoke/ PRE-CREATION 



Test case does not pass.


Gaurav Aradhye

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