Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/638
---
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
LGTM
On Fri, Jul 31, 2015 at 4:42 PM, nitt10prashant wrote:
> Github user nitt10prashant commented on the pull request:
>
> https://github.com/apache/cloudstack/pull/638#issuecomment-126656909
>
> LGTM
>
>
>
> ---
> If your project is set up for it, you can reply to this email and have y
Github user nitt10prashant commented on the pull request:
https://github.com/apache/cloudstack/pull/638#issuecomment-126656909
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 featu
Github user pritisarap12 commented on the pull request:
https://github.com/apache/cloudstack/pull/638#issuecomment-126637367
Testcase Result-
Test hypervisor max limits effect ... === TestName:
test_check_hypervisor_max_limit_effect | Status : SUCCESS ===
ok
Ran 1
Github user pritisarap12 commented on the pull request:
https://github.com/apache/cloudstack/pull/638#issuecomment-126636676
Updated the testcase for:
-Count SSVM's on selected host
-Count VR on the host
-Count user VM's on the host
-Then update the m
How your logic works if there are no system vms on a host (ssvm, cpvm and
VR)? I mean if the cluster has more than one host and system vms were
deployed on another host in the cluster?
On Fri, Jul 31, 2015 at 2:01 PM, pritisarap12 wrote:
> Github user pritisarap12 commented on a diff in the pull
Github user pritisarap12 commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/638#discussion_r35955311
--- Diff: test/integration/testpaths/testpath_hypervisor_limit.py ---
@@ -0,0 +1,198 @@
+# Licensed to the Apache Software Foundation (ASF) under
Github user sanju1010 commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/638#discussion_r35955013
--- Diff: test/integration/testpaths/testpath_hypervisor_limit.py ---
@@ -0,0 +1,198 @@
+# Licensed to the Apache Software Foundation (ASF) under one
Github user sanju1010 commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/638#discussion_r35954965
--- Diff: test/integration/testpaths/testpath_hypervisor_limit.py ---
@@ -0,0 +1,198 @@
+# Licensed to the Apache Software Foundation (ASF) under one
Github user pritisarap12 commented on the pull request:
https://github.com/apache/cloudstack/pull/638#issuecomment-126606147
Modified test to check for empty vmList and updating maxguestlimit
accordingly.
And the test result is-
Test hypervisor max limits effect ... === Te
Github user nitt10prashant commented on the pull request:
https://github.com/apache/cloudstack/pull/638#issuecomment-126589627
can you post 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 do
Github user pritisarap12 commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/638#discussion_r35862152
--- Diff: test/integration/testpaths/testpath_hypervisor_limit.py ---
@@ -0,0 +1,191 @@
+# Licensed to the Apache Software Foundation (ASF) under
Github user sanju1010 commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/638#discussion_r35848734
--- Diff: test/integration/testpaths/testpath_hypervisor_limit.py ---
@@ -0,0 +1,191 @@
+# Licensed to the Apache Software Foundation (ASF) under one
Github user sanju1010 commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/638#discussion_r35848618
--- Diff: test/integration/testpaths/testpath_hypervisor_limit.py ---
@@ -0,0 +1,191 @@
+# Licensed to the Apache Software Foundation (ASF) under one
GitHub user pritisarap12 opened a pull request:
https://github.com/apache/cloudstack/pull/638
CLOUDSTACK-8689: Verify effect of changing value of XenServer Max guest
limit
Verify effect of changing value of "Max guest limit" of XenServer on
previously added hosts
-Check if
15 matches
Mail list logo
