cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Santhosh Edukulla" <>
Subject Review Request 17452: Added Fi for CLOUDSTACK-5963
Date Tue, 28 Jan 2014 10:47:22 GMT

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

Review request for cloudstack and Girish Shilamkar.


Repository: cloudstack-git


1. The way we are verifying the response for test_public_ip_admin_account seems to have an
issue, it should ideally check for empty list.
2. Post the nat and firewall rule, still there could be a possibility such that the vm may
not be in running state. We are not passing mode flag here as such for advanced zone. So explicit
fail when vm status is not running post the ssh failure. 
3. try except block for nat rule deletion and listing.
4. Adding a sleep post delete.


  test/integration/smoke/ 732fe15 



Testing on private branch post other changes pending, tested locally.


Santhosh Edukulla

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