Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/665
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/665#issuecomment-134920085
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enab
Github user nitt10prashant commented on the pull request:
https://github.com/apache/cloudstack/pull/665#issuecomment-132583048
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this featur
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/665#issuecomment-132573747
@nitt10prashant Can you please review this PR again?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user pritisarap12 commented on the pull request:
https://github.com/apache/cloudstack/pull/665#issuecomment-132555104
Test result:
Test multiple volume snapshot in zwps ... === TestName:
test_01_multiple_snapshot_in_zwps | Status : SUCCESS ===
ok
--
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/665#issuecomment-131213094
@pritisarap12 Please squash the commits, they have the same titles which is
confusing. If you need multiple commits, make sure they have a descriptive
first line
Github user nitt10prashant commented on the pull request:
https://github.com/apache/cloudstack/pull/665#issuecomment-129846827
can you update test results ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user nitt10prashant commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/665#discussion_r36735392
--- Diff: test/integration/testpaths/testpath_multiple_snapshot.py ---
@@ -0,0 +1,279 @@
+# Licensed to the Apache Software Foundation (ASF) und
Github user nitt10prashant commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/665#discussion_r36734361
--- Diff: test/integration/testpaths/testpath_multiple_snapshot.py ---
@@ -0,0 +1,279 @@
+# Licensed to the Apache Software Foundation (ASF) und
Github user nitt10prashant commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/665#discussion_r36734207
--- Diff: test/integration/testpaths/testpath_multiple_snapshot.py ---
@@ -0,0 +1,279 @@
+# Licensed to the Apache Software Foundation (ASF) und
Github user nitt10prashant commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/665#discussion_r36733882
--- Diff: test/integration/testpaths/testpath_multiple_snapshot.py ---
@@ -0,0 +1,279 @@
+# Licensed to the Apache Software Foundation (ASF) und
Github user pritisarap12 commented on the pull request:
https://github.com/apache/cloudstack/pull/665#issuecomment-129314182
Updated testcase:
-Removed redundant code
-Added validate_list function for list snapshot operation
---
If your project is set up for it, you
Github user sanju1010 commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/665#discussion_r36601607
--- Diff: test/integration/testpaths/testpath_multiple_snapshot.py ---
@@ -0,0 +1,255 @@
+# Licensed to the Apache Software Foundation (ASF) under on
Github user sanju1010 commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/665#discussion_r36601462
--- Diff: test/integration/testpaths/testpath_multiple_snapshot.py ---
@@ -0,0 +1,255 @@
+# Licensed to the Apache Software Foundation (ASF) under on
Github user sanju1010 commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/665#discussion_r36601402
--- Diff: test/integration/testpaths/testpath_multiple_snapshot.py ---
@@ -0,0 +1,255 @@
+# Licensed to the Apache Software Foundation (ASF) under on
GitHub user pritisarap12 opened a pull request:
https://github.com/apache/cloudstack/pull/665
CLOUDSTACK-8716: Verify creation of snapshot from volume when the task is
performed repeatedly in zone wide primary Storage
On VMWare with a Zone wide primary storage and more than two clus
16 matches
Mail list logo
