You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Gaurav Aradhye <ga...@clogeny.com> on 2013/09/26 12:00:19 UTC
Review Request 14346: CLOUDSTACK:4647 - Fixed test_snapshot_gc.py and common
utility function to check snapshot on NFS
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14346/
-----------------------------------------------------------
Review request for cloudstack and Prasanna Santhanam.
Repository: cloudstack-git
Description
-------
1. test_snapshot_gc.py :
a) Removed the account and vm from cleanup, as we are already deleting account in the test case execution itself. Adding it to cleanup was throwing exception during cleanup operation.
b) Once account deleted, listAccount API should fail (as against it should give None)
2. utils.py
Changes in "is_snapshot_on_nfs" function:
a) Corrected the mount URL, earlier mgt server IP was passed instead of nfs url
b) Removed the extra colon between host and dir, colon is already present in the host variable on KVM and VMWare which is correct behaviour. This fails on Xen Sever as the colon is not present. Bug filed for Xen Server >>> https://issues.apache.org/jira/browse/CLOUDSTACK-4741
b) If snapshot not present in db , then return False as the snapshot is not present
Diffs
-----
test/integration/component/test_snapshot_gc.py aec9761
tools/marvin/marvin/integration/lib/utils.py a6abe06
Diff: https://reviews.apache.org/r/14346/diff/
Testing
-------
Thanks,
Gaurav Aradhye
Re: Review Request 14346: CLOUDSTACK:4647 - Fixed test_snapshot_gc.py and
common utility function to check snapshot on NFS
Posted by Rayees Namathponnan <ra...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14346/#review26422
-----------------------------------------------------------
Is it tested on KVM, Xen and vmware ?
Tested or not missing in review
- Rayees Namathponnan
On Sept. 26, 2013, 10 a.m., Gaurav Aradhye wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14346/
> -----------------------------------------------------------
>
> (Updated Sept. 26, 2013, 10 a.m.)
>
>
> Review request for cloudstack and Prasanna Santhanam.
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> 1. test_snapshot_gc.py :
> a) Removed the account and vm from cleanup, as we are already deleting account in the test case execution itself. Adding it to cleanup was throwing exception during cleanup operation.
> b) Once account deleted, listAccount API should fail (as against it should give None)
>
> 2. utils.py
> Changes in "is_snapshot_on_nfs" function:
>
> a) Corrected the mount URL, earlier mgt server IP was passed instead of nfs url
> b) Removed the extra colon between host and dir, colon is already present in the host variable on KVM and VMWare which is correct behaviour. This fails on Xen Sever as the colon is not present. Bug filed for Xen Server >>> https://issues.apache.org/jira/browse/CLOUDSTACK-4741
> b) If snapshot not present in db , then return False as the snapshot is not present
>
>
> Diffs
> -----
>
> test/integration/component/test_snapshot_gc.py aec9761
> tools/marvin/marvin/integration/lib/utils.py a6abe06
>
> Diff: https://reviews.apache.org/r/14346/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Gaurav Aradhye
>
>
Re: Review Request 14346: CLOUDSTACK:4647 - Fixed test_snapshot_gc.py and
common utility function to check snapshot on NFS
Posted by Prasanna Santhanam <ts...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14346/#review27009
-----------------------------------------------------------
Ship it!
- Prasanna Santhanam
On Sept. 27, 2013, 4:18 a.m., Gaurav Aradhye wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14346/
> -----------------------------------------------------------
>
> (Updated Sept. 27, 2013, 4:18 a.m.)
>
>
> Review request for cloudstack and Prasanna Santhanam.
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> 1. test_snapshot_gc.py :
> a) Removed the account and vm from cleanup, as we are already deleting account in the test case execution itself. Adding it to cleanup was throwing exception during cleanup operation.
> b) Once account deleted, listAccount API should fail (as against it should give None)
>
> 2. utils.py
> Changes in "is_snapshot_on_nfs" function:
>
> a) Corrected the mount URL, earlier mgt server IP was passed instead of nfs url
> b) Removed the extra colon between host and dir, colon is already present in the host variable on KVM and VMWare which is correct behaviour. This fails on Xen Sever as the colon is not present. Bug filed for Xen Server >>> https://issues.apache.org/jira/browse/CLOUDSTACK-4741
> b) If snapshot not present in db , then return False as the snapshot is not present
>
>
> Diffs
> -----
>
> test/integration/component/test_snapshot_gc.py aec9761
> tools/marvin/marvin/integration/lib/utils.py a6abe06
>
> Diff: https://reviews.apache.org/r/14346/diff/
>
>
> Testing
> -------
>
> Tested on KVM, VMware, and XenServer.
> Fails on XenServer due to above mentioned issue.
>
>
> Thanks,
>
> Gaurav Aradhye
>
>
Re: Review Request 14346: CLOUDSTACK:4647 - Fixed test_snapshot_gc.py and
common utility function to check snapshot on NFS
Posted by Gaurav Aradhye <ga...@clogeny.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14346/
-----------------------------------------------------------
(Updated Sept. 27, 2013, 9:48 a.m.)
Review request for cloudstack and Prasanna Santhanam.
Repository: cloudstack-git
Description
-------
1. test_snapshot_gc.py :
a) Removed the account and vm from cleanup, as we are already deleting account in the test case execution itself. Adding it to cleanup was throwing exception during cleanup operation.
b) Once account deleted, listAccount API should fail (as against it should give None)
2. utils.py
Changes in "is_snapshot_on_nfs" function:
a) Corrected the mount URL, earlier mgt server IP was passed instead of nfs url
b) Removed the extra colon between host and dir, colon is already present in the host variable on KVM and VMWare which is correct behaviour. This fails on Xen Sever as the colon is not present. Bug filed for Xen Server >>> https://issues.apache.org/jira/browse/CLOUDSTACK-4741
b) If snapshot not present in db , then return False as the snapshot is not present
Diffs
-----
test/integration/component/test_snapshot_gc.py aec9761
tools/marvin/marvin/integration/lib/utils.py a6abe06
Diff: https://reviews.apache.org/r/14346/diff/
Testing (updated)
-------
Tested on KVM, VMware, and XenServer.
Fails on XenServer due to above mentioned issue.
Thanks,
Gaurav Aradhye