Re: Review Request 25187: test_delete_account and test_releaseIP failing in advanced zone

2014-09-16 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25187/#review53510 --- Commit 06b96d1ccf85455587d5098402de14c2930328f0 in cloudstack's bran

Re: Review Request 25187: test_delete_account and test_releaseIP failing in advanced zone

2014-09-04 Thread Chandan Purushothama
> On Sept. 4, 2014, 8:24 p.m., Chandan Purushothama wrote: > > Ship It! I reviewed the patch. I read the entire test suite to understand the Use Cases. The Suite does refer to the wrong dictionary "nat_rule" instead of "lb_rule". Alex's patch should fix the issue. - Chandan ---

Re: Review Request 25187: test_delete_account and test_releaseIP failing in advanced zone

2014-09-04 Thread Chandan Purushothama
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25187/#review52348 --- Ship it! Ship It! - Chandan Purushothama On Sept. 3, 2014, 3:17

Re: Review Request 25187: test_delete_account and test_releaseIP failing in advanced zone

2014-09-03 Thread Alex Brett
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25187/ --- (Updated Sept. 3, 2014, 3:17 p.m.) Review request for cloudstack, Ashutosh Kelk

Review Request 25187: test_delete_account and test_releaseIP failing in advanced zone

2014-08-29 Thread Alex Brett
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25187/ --- Review request for cloudstack, Ashutosh Kelkar and Santhosh Edukulla. Bugs: CLO