[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1753 Trillian test result (tid-502) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6 Total time taken: 37885 seconds Marvin logs: https://github.com/blueoran

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1754 Trillian test result (tid-497) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6 Total time taken: 36063 seconds Marvin logs: https://github.com/blueoran

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1754 Trillian test result (tid-498) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6 Total time taken: 34803 seconds Marvin logs: https://github.com/blueoran

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1754 Trillian test result (tid-499) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 28669 seconds Marvin logs: https://github.com/blueoranguta

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1753 Trillian test result (tid-501) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 26633 seconds Marvin logs: https://github.com/blueoranguta

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1659 Trillian test result (tid-483) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 27669 seconds Marvin logs: https://github.com/blueoranguta

[GitHub] cloudstack issue #1762: CLOUDSTACK-9595 Transactions are not getting retried...

2016-11-29 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1762 @rafaelweingartner Thanks a lot. I totally agree that resetting hosts doesn't really need to be a part of transaction and should be extracted to a new method. The same is for lines 527-546, and t

[GitHub] cloudstack issue #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1579 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-284 --- 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 d

[GitHub] cloudstack issue #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1579 @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 appe

[GitHub] cloudstack issue #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1579 @blueorangutan package --- 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

[GitHub] cloudstack issue #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-11-29 Thread prashanthvarma
Github user prashanthvarma commented on the issue: https://github.com/apache/cloudstack/pull/1579 LGTM From our internal code review and regression run results (including our added Nuage VSP SDN plugin specific Marvin tests, refer the previous comment for their corresponding

[GitHub] cloudstack issue #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-11-29 Thread prashanthvarma
Github user prashanthvarma commented on the issue: https://github.com/apache/cloudstack/pull/1579 Nuage VSP SDN plugin specific Marvin test code PEP8 & PyFlakes Compliance: CloudStack$ CloudStack$ pep8 test/integration/plugins/nuagevsp/*.py CloudStack$ CloudStack$ pyfl

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1754 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-283 --- 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 d

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1754 @rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt + vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests --- If your project is set up for it

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1753 @rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt + vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests --- If your project is set up for it

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1754 @blueorangutan test matrix --- 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 wi

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @blueorangutan test matrix --- 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 wi

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1754 Packaging result: ✔centos6 ✔centos7 ✖debian. JID-282 --- 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 d

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1753 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-281 --- 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 d

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1659 @murali-reddy a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your r

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-11-29 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 @blueorangutan test --- 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 wi

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1754 @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 appe

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1753 @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 appe

[GitHub] cloudstack issue #1753: [4.9] Latest health test run

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1753 @blueorangutan package --- 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

[GitHub] cloudstack issue #1754: [4.10] Latest health test run

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1754 @blueorangutan package --- 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

[GitHub] cloudstack pull request #1772: CLOUDSTACK-9627 Fix template sync for region ...

2016-11-29 Thread syed
GitHub user syed reopened a pull request: https://github.com/apache/cloudstack/pull/1772 CLOUDSTACK-9627 Fix template sync for region store. When using a region store like Swift or S3 as secondary storage, the `zoneId` can be null. This causes an exception when we try to conv

[GitHub] cloudstack pull request #1772: CLOUDSTACK-9627 Fix template sync for region ...

2016-11-29 Thread syed
Github user syed closed the pull request at: https://github.com/apache/cloudstack/pull/1772 --- 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 e

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1659 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-280 --- 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 d

[GitHub] cloudstack issue #1772: CLOUDSTACK-9627 Fix template sync for region store.

2016-11-29 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1772 @rhtyd Done! --- 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

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1659 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-279 --- 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 d

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1659 @murali-reddy a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your r

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-11-29 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1659 @blueorangutan test centos7 kvm-centos7 --- 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 fe

[GitHub] cloudstack pull request #1797: CLOUDSTACK-9630: Cannot use listNics API as a...

2016-11-29 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1797#discussion_r90009164 --- Diff: server/src/com/cloud/api/ApiResponseHelper.java --- @@ -3461,10 +3470,21 @@ public NicResponse createNicResponse(Nic result) { re

[GitHub] cloudstack issue #1579: CLOUDSTACK-9403 : Support for shared networks in Nua...

2016-11-29 Thread fmaximus
Github user fmaximus commented on the issue: https://github.com/apache/cloudstack/pull/1579 Rebased and squashed all commits. --- 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] cloudstack pull request #1798: API: affinitygroupids or affinitygroupnames m...

2016-11-29 Thread marcaurele
GitHub user marcaurele opened a pull request: https://github.com/apache/cloudstack/pull/1798 API: affinitygroupids or affinitygroupnames must be given Return an exception if both parameter are missing. Signed-off-by: Marc-Aurèle Brothier You can merge this pull request in

General requests while submitting PRs

2016-11-29 Thread Rohit Yadav
All, Please consider following guideline when submitting PRs for speedy review and acceptance: - Create a JIRA ID, or use any existing JIRA id related to your PR, in your PR summary and in your git commit summary. - If you're fixing a bug, consider sending this for 4.9 or 4.8 as they may be

[GitHub] cloudstack issue #1796: CLOUDSTACK-9626: Instance fails to start after unsuc...

2016-11-29 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1796 ### ACS CI BVT Run **Sumarry:** Build Number 149 Hypervisor xenserver NetworkType Advanced Passed=103 Failed=2 Skipped=7 _Link to logs Folder (searc

[GitHub] cloudstack issue #1772: CLOUDSTACK-9627 Fix template sync for region store.

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1772 @syed can you amend the git commit message with this format to include the JIRA id: `CLOUDSTACK-xxx: one line summary`? --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1793: CLOUDSTACK-9624 Incorrect hypervisor mapping of gues...

2016-11-29 Thread sateesh-chodapuneedi
Github user sateesh-chodapuneedi commented on the issue: https://github.com/apache/cloudstack/pull/1793 Thanks @rhtyd --- 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] cloudstack pull request #1793: CLOUDSTACK-9624 Incorrect hypervisor mapping ...

2016-11-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1793 --- 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] cloudstack issue #1793: CLOUDSTACK-9624 Incorrect hypervisor mapping of gues...

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1793 Thanks @sateesh-chodapuneedi I'll merge this now. --- 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] cloudstack issue #1519: Cloudstack 9339: Virtual Routers do not handle Multi...

2016-11-29 Thread dsclose
Github user dsclose commented on the issue: https://github.com/apache/cloudstack/pull/1519 PR #1659 appears to have superseded this PR. As such I'm concerned about recent activity on this PR. Is anyone available to clarify what should be done? I'd anticipated closing this PR

[GitHub] cloudstack issue #1793: CLOUDSTACK-9624 Incorrect hypervisor mapping of gues...

2016-11-29 Thread sateesh-chodapuneedi
Github user sateesh-chodapuneedi commented on the issue: https://github.com/apache/cloudstack/pull/1793 @rhtyd Yes, that's correct. The guest_os_name `windows7Server64Guest` is supported since vSphere 4.0 onwards. I've verified this in 5.0 as well as 6.0 setup. --- If your proj

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-11-29 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1659 Packaging result: ✖centos6 ✔centos7 ✖debian. JID-278 --- 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 d

[GitHub] cloudstack pull request #1797: CLOUDSTACK-9630: Cannot use listNics API as a...

2016-11-29 Thread ustcweizhou
Github user ustcweizhou commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1797#discussion_r89973281 --- Diff: server/src/com/cloud/api/ApiResponseHelper.java --- @@ -3461,10 +3470,21 @@ public NicResponse createNicResponse(Nic result) {

[GitHub] cloudstack pull request #1797: CLOUDSTACK-9630: Cannot use listNics API as a...

2016-11-29 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/1797 CLOUDSTACK-9630: Cannot use listNics API as advertised listNics API for a VM, "type" was not returned within API response. EXPECTED BEHAVIOR == The l

[GitHub] cloudstack issue #1793: CLOUDSTACK-9624 Incorrect hypervisor mapping of gues...

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1793 @sateesh-chodapuneedi is the guest_os_name `windows7Server64Guest` known by vmware versions 4.x, 5.x? The sql update will change guest_os_name for vmware versions 4.0, 4.1, 5.0, 5.1, 5.5 and 6.0.

[GitHub] cloudstack issue #1793: CLOUDSTACK-9624 Incorrect hypervisor mapping of gues...

2016-11-29 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1793 Test LGTM. One of the travis jobs took time while uploading the cache, but it passed all component tests: https://travis-ci.org/apache/cloudstack/jobs/179401939 Since, it's a simple fix a