Re: Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-09-03 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24696/#review52141 --- Pushed to master : 9b783d19ad7e784d3751f4eaa2a672401d122a01 - Srika

Re: Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-09-03 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24696/#review52138 --- Ship it! Ship It! - SrikanteswaraRao Talluri On Sept. 2, 2014, 1

Re: Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-09-02 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24696/#review52136 --- test/integration/smoke/test_volumes.py

Re: Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-09-02 Thread John Dilley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24696/ --- (Updated Sept. 2, 2014, 1:38 p.m.) Review request for cloudstack, Santhosh Eduk

Re: Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-08-14 Thread John Dilley
> On Aug. 14, 2014, 11:16 a.m., Santhosh Edukulla wrote: > > test/integration/smoke/test_volumes.py, line 640 > > > > > > Just an additional refactor, we can do the check at the start before > > attach volume and skip

Re: Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-08-14 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24696/#review50572 --- test/integration/smoke/test_volumes.py

Re: Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-08-14 Thread John Dilley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24696/#review50571 --- test/integration/smoke/test_volumes.py

Re: Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-08-14 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24696/#review50570 --- test/integration/smoke/test_volumes.py

Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-08-14 Thread John Dilley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24696/ --- Review request for cloudstack and Santhosh Edukulla. Bugs: CLOUDSTACK-7228