Github user cloudmonger commented on the issue:
https://github.com/apache/cloudstack/pull/1812
### ACS CI BVT Run
**Sumarry:**
Build Number 202
Hypervisor xenserver
NetworkType Advanced
Passed=98
Failed=7
Skipped=7
_Link to logs Folder
Github user koushik-das commented on the issue:
https://github.com/apache/cloudstack/pull/1882
Code changes 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
enabled and
Github user sateesh-chodapuneedi commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1861#discussion_r94216180
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
---
@@ -230,6 +230,7 @@
import
GitHub user sureshanaparti opened a pull request:
https://github.com/apache/cloudstack/pull/1882
CLOUDSTACK-8737: Removed the missed out-of-band VR reboot code, not
required based on persistent VR changes.
You can merge this pull request into a Git repository by running:
$
Github user sateesh-chodapuneedi commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1861#discussion_r94216151
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java
---
@@ -123,12 +125,14 @@
import
Github user koteswararavipati commented on the issue:
https://github.com/apache/cloudstack/pull/865
LGTM:
I tested this issue manually and looks good.
---
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 koteswararavipati commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/865#discussion_r94215572
--- Diff: server/src/com/cloud/server/ManagementServerImpl.java ---
@@ -2488,6 +2479,23 @@ public int compare(final SummedCapacity arg0, final
Github user koteswararavipati commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/865#discussion_r94215553
--- Diff: server/src/com/cloud/server/ManagementServerImpl.java ---
@@ -2488,6 +2479,23 @@ public int compare(final SummedCapacity arg0, final
Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1862
@sateesh-chodapuneedi Code changes 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
Github user rashmidixit closed the pull request at:
https://github.com/apache/cloudstack/pull/1819
---
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
GitHub user rashmidixit reopened a pull request:
https://github.com/apache/cloudstack/pull/1819
CLOUDSTACK-9653 The system capacity was not getting calculated correctly
When listCapacity is used with the sortBy=usage option, the capacity was
not getting calculated correctly.
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/1841
Thanks @sureshanaparti
---
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
Github user sateesh-chodapuneedi commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1862#discussion_r94214404
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
---
@@ -5620,4 +5623,39 @@ private
Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1841
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
enabled and wishes so, or
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/1862
Thanks @sureshanaparti for review.
I have addressed that now. Pushed the code changes, please check.
---
If your project is set up for it, you can reply to this email and have
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1879
Packaging result: âcentos6 âcentos7 âdebian. JID-437
---
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 blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1879
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you
posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1879
@sureshanaparti can you push -f on the PR, Travis failed.
@blueorangutan package
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user karuturi commented on the issue:
https://github.com/apache/cloudstack/pull/1880
@blueorangutan test centos7 vmware-55u3
---
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
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1871
@rhtyd Could you please run the test again ?
---
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
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/775
LGTM for code. Looks like tests are successful too based on results
published by @bvbharatk
@karuturi Should we consider this for merge OR re-run tests again as the
previous
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/1758
@nitin-maharana LGTM for code.
---
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
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/1881
@karuturi As this PR is about removal of a deprecated configuration setting
which is no more referenced any where in the repo, can we merge this without
functional tests?
Or we
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/1880
@sureshanaparti Travis tests ended with timeout. Can you please try force
push?
```The job exceeded the maximum time limit for jobs, and has been
terminated.```
---
If your
24 matches
Mail list logo
