Github user bvbharatk commented on the pull request:
https://github.com/apache/cloudstack/pull/1307#issuecomment-202054442
### ACS CI BVT Run
**Sumarry:**
Build Number 135
Hypervisor xenserver
NetworkType Advanced
Passed=105
Failed=0
Skipped=4
Github user rafaelweingartner commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1438#discussion_r57527860
--- Diff:
services/secondary-storage/server/src/org/apache/cloudstack/storage/resource/NfsSecondaryStorageResource.java
---
@@ -395,16 +391,9
Github user bvbharatk commented on the pull request:
https://github.com/apache/cloudstack/pull/1304#issuecomment-202117240
### ACS CI BVT Run
**Sumarry:**
Build Number 136
Hypervisor xenserver
NetworkType Advanced
Passed=105
Failed=0
Skipped=4
Github user cristofolini commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1451#discussion_r57535012
--- Diff:
core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java ---
@@ -180,7 +179,7 @@ private Answer applyConfig(NetworkEle
Github user cristofolini commented on the pull request:
https://github.com/apache/cloudstack/pull/1450#issuecomment-202152926
@ProjectMoon According to that comment on line 781 in `CommandSetupHelper`
the conditional that follows is there to enable sourceNAT, yet you removed the
comma
Github user alexandrelimassantana commented on the pull request:
https://github.com/apache/cloudstack/pull/1451#issuecomment-202161988
@cristofolini there is no _timeout_ variable in the scope you commented.
His change is valid because _VRScripts.DEFAULT_EXECUTEINVR_TIMEOUT_ is mapped
Github user pedro-martins commented on the pull request:
https://github.com/apache/cloudstack/pull/1452#issuecomment-202163598
Hi @prashanthvarma.
How about to use String.format() to create the strings in the loggers? The
use of String format will turn the strings in the logs
Github user cristofolini commented on the pull request:
https://github.com/apache/cloudstack/pull/1451#issuecomment-202164340
@alexandrelimassantana Ah, I see that now. Thanks for the clarification! :)
---
If your project is set up for it, you can reply to this email and have your
rep
Github user pedro-martins commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1451#discussion_r57537243
--- Diff:
core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java ---
@@ -180,7 +179,7 @@ private Answer applyConfig(NetworkEl
Github user bvbharatk commented on the pull request:
https://github.com/apache/cloudstack/pull/1297#issuecomment-202176758
### ACS CI BVT Run
**Sumarry:**
Build Number 137
Hypervisor xenserver
NetworkType Advanced
Passed=101
Failed=1
Skipped=4
GitHub user abhinandanprateek opened a pull request:
https://github.com/apache/cloudstack/pull/1454
CLOUDSTACK-9323: Fix cancel host maintenance canâ¦
so that if maintenance is cancelled the host come back to normal state
gracefully.
Added marvin tests for host maintennac
Github user jburwell commented on the pull request:
https://github.com/apache/cloudstack/pull/1454#issuecomment-202217738
@abhinandanprateek I don't see the Marvin test case in the PR. Have you
pushed the latest commit?
Also, most of the changes seem to be formatting changes
Github user abhinandanprateek commented on the pull request:
https://github.com/apache/cloudstack/pull/1454#issuecomment-202221938
@jsb added the marvin file and reverted to pre-commit formatted code.
---
If your project is set up for it, you can reply to this email and have your
repl
Github user GabrielBrascher commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1326#discussion_r57545397
--- Diff: test/integration/component/test_add_remove_network.py ---
@@ -1021,6 +1021,103 @@ def test_29_remove_nic_CS22503(self):
Github user bvbharatk commented on the pull request:
https://github.com/apache/cloudstack/pull/1287#issuecomment-202238647
### ACS CI BVT Run
**Sumarry:**
Build Number 138
Hypervisor xenserver
NetworkType Advanced
Passed=106
Failed=0
Skipped=4
Github user abhinandanprateek commented on the pull request:
https://github.com/apache/cloudstack/pull/1454#issuecomment-202256571
Marvin test output:
root@ccp:~/cloudstack(host-maint)# ./host_maint.sh
++ date
+ echo Mon Mar 28 11:47:45 IST 2016
Mon Mar 28 11:47:45
16 matches
Mail list logo
