Github user cloudsadhu commented on the issue:
https://github.com/apache/cloudstack/pull/1813
added logic to handle global level full clone setting.
full clone parameter are handled differently (as per code its referring in
3 places due to that even if we set the full clone
Github user cloudsadhu commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1813#discussion_r108862410
--- Diff: test/integration/smoke/test_deploy_vm_root_resize.py ---
@@ -114,36 +134,46 @@ def test_00_deploy_vm_root_resize(self):
# 2
Github user cloudsadhu commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1813#discussion_r108092328
--- Diff: test/integration/smoke/test_deploy_vm_root_resize.py ---
@@ -114,7 +114,13 @@ def test_00_deploy_vm_root_resize(self):
# 2
Github user cloudsadhu commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1813#discussion_r106085688
--- Diff: test/integration/smoke/test_deploy_vm_root_resize.py ---
@@ -114,7 +114,13 @@ def test_00_deploy_vm_root_resize(self):
# 2
Github user cloudsadhu commented on the issue:
https://github.com/apache/cloudstack/pull/1813
@serg38 - thanks for your comment- I have added nfs support
---
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
Github user cloudsadhu commented on the pull request:
https://github.com/apache/cloudstack/pull/1458#issuecomment-213686569
Thanks Swill and now All the checks are passed.Please merge this PR.
---
If your project is set up for it, you can reply to this email and have your
Github user cloudsadhu commented on the pull request:
https://github.com/apache/cloudstack/pull/1498#issuecomment-212828874
reviewed your code change and looks good. Please post the test result.
LGTM based on code review !!
---
If your project is set up for it, you can reply
Github user cloudsadhu commented on the pull request:
https://github.com/apache/cloudstack/pull/1501#issuecomment-212766999
LGTM based on the code review !!
---
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 cloudsadhu commented on the pull request:
https://github.com/apache/cloudstack/pull/1458#issuecomment-208331373
HI,
Thanks for your review comment and i ran the few tests on the created
setup even though it won't impact the any testcases but for your request i
Github user cloudsadhu commented on the pull request:
https://github.com/apache/cloudstack/pull/1458#issuecomment-206272202
I have incorporated the all the review comments and squashed all my
commits to single commit.
This script prepares setup required to run advanced
Github user cloudsadhu commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1458#discussion_r58188259
--- Diff: tools/marvin/marvin/config/Baremtal_AdvZone_Vmware.cfg ---
@@ -0,0 +1,191 @@
+{
+"
Github user cloudsadhu commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1458#discussion_r58188294
--- Diff: tools/marvin/marvin/config/Baremtal_AdvZone_Vmware.cfg ---
@@ -0,0 +1,191 @@
+{
+"
Github user cloudsadhu commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1458#discussion_r58188213
--- Diff: tools/marvin/marvin/config/Baremtal_AdvZone_Vmware.cfg ---
@@ -0,0 +1,191 @@
+{
+"
GitHub user cloudsadhu opened a pull request:
https://github.com/apache/cloudstack/pull/1458
BUG-ID:CLOUDSTACK-9331:added code in marvin frame config file for
advBaremetal support â¦
Added code in marvin framework config file to support baremetal
advanced testcases
You can
Github user cloudsadhu closed the pull request at:
https://github.com/apache/cloudstack/pull/691
---
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
Github user cloudsadhu commented on the pull request:
https://github.com/apache/cloudstack/pull/691#issuecomment-179025093
for same issue Another PR created and checked in master so closing this PR
request.
---
If your project is set up for it, you can reply to this email and have
Github user cloudsadhu commented on the pull request:
https://github.com/apache/cloudstack/pull/1048#issuecomment-164737419
Sure please find the test results below:
@summary: Test volume delete event generation in error state condition ...
=== TestName
GitHub user cloudsadhu opened a pull request:
https://github.com/apache/cloudstack/pull/1048
CLOUDSTACK-8731-checking usage event for delete volume
checking usage event for delete volume. I have incorporated all the
suggested changes.
You can merge this pull request into a Git
Github user cloudsadhu commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/691#discussion_r38084154
--- Diff: test/integration/component/test_persistent_networks.py ---
@@ -1459,6 +1459,135 @@ def test_delete_account(self):
return
GitHub user cloudsadhu opened a pull request:
https://github.com/apache/cloudstack/pull/691
CLOUDSTACK-8731-checking usage event for delete volume
@summary: Test volume delete event generation in error state condition ...
=== TestName: test_volume_delete_event_errorState | Status
20 matches
Mail list logo