[GitHub] cloudstack issue #1854: 4.9 multiplex testing

2016-12-22 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1854 @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 #1854: 4.9 multiplex testing

2016-12-22 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1854 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-413 --- 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 #1854: 4.9 multiplex testing

2016-12-22 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1854 @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 #1827: CLOUDSTACK-9673 : Exception occured while creating t...

2016-12-22 Thread sateesh-chodapuneedi
Github user sateesh-chodapuneedi commented on the issue: https://github.com/apache/cloudstack/pull/1827 @rhtyd Thanks. Re-based against 4.9 branch, and changed base branch to 4.9 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack issue #1826: CLOUDSTACK-9675: Cloudstack Metrics Miscellaneous bu...

2016-12-22 Thread rashmidixit
Github user rashmidixit commented on the issue: https://github.com/apache/cloudstack/pull/1826 @rhtyd - There have been four things fixed here. I have explained each commit below. Your review/comments is most appreciated. 1. Goto Zone metrics or Hosts metrics. NUmerical values

[GitHub] cloudstack issue #1828: CLOUDSTACK-9676 Start instance fails after reverting...

2016-12-22 Thread sateesh-chodapuneedi
Github user sateesh-chodapuneedi commented on the issue: https://github.com/apache/cloudstack/pull/1828 @rhtyd Thanks. Done with rebase against `4.9` branch, also changed the PR's base branch to `4.9`. --- If your project is set up for it, you can reply to this email and have your re

[GitHub] cloudstack pull request #1853: CLOUDSTACK-9696: Fixed Virtual Router deploym...

2016-12-22 Thread anshul1886
Github user anshul1886 closed the pull request at: https://github.com/apache/cloudstack/pull/1853 --- 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 featur

[GitHub] cloudstack pull request #1853: CLOUDSTACK-9696: Fixed Virtual Router deploym...

2016-12-22 Thread anshul1886
GitHub user anshul1886 reopened a pull request: https://github.com/apache/cloudstack/pull/1853 CLOUDSTACK-9696: Fixed Virtual Router deployment failing if there are… … no shared resources available and has access to resources through parent domain heirarachy You can mer

[GitHub] cloudstack pull request #1855: CLOUDSTACK-9697: Better error message if user...

2016-12-22 Thread rashmidixit
GitHub user rashmidixit opened a pull request: https://github.com/apache/cloudstack/pull/1855 CLOUDSTACK-9697: Better error message if user tries to shrink VM ROOT Volume size Skip the API call altogether if the UI detects this case and throw a more user friendly message C

[GitHub] cloudstack pull request #1856: CLOUDSTACK-9569: propagate global configurati...

2016-12-22 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1856 CLOUDSTACK-9569: propagate global configuration router.aggregation.command.each.timeout to KVM agent The router.aggregation.command.each.timeout in global configuration is only applied on

[GitHub] cloudstack issue #1826: CLOUDSTACK-9675: Cloudstack Metrics Miscellaneous bu...

2016-12-22 Thread rashmidixit
Github user rashmidixit commented on the issue: https://github.com/apache/cloudstack/pull/1826 Added one more fix to this: 1. List of Virtual Machines will be incorrect if the number is greater than the pagesize. Added -1 to the call of the API. --- If your project is set up for

[GitHub] cloudstack issue #1854: 4.9 multiplex testing

2016-12-22 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1854 Fixes most of the failures from test_volumes, which turns out to be env related. Test Volume creation for all Disk Offerings (incl. custom) ... === TestName: test_01_create_volume | Statu

[GitHub] cloudstack issue #1854: 4.9 multiplex testing

2016-12-22 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1854 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-414 --- 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 #1854: 4.9 multiplex testing

2016-12-22 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1854 CentOS6 pkg re-kicked. --- 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 #1856: CLOUDSTACK-9569: propagate global configurati...

2016-12-22 Thread murali-reddy
Github user murali-reddy commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1856#discussion_r93599467 --- Diff: core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java --- @@ -259,8 +260,18 @@ private Answer execute(GetDomRVersio

[GitHub] cloudstack issue #1856: CLOUDSTACK-9569: propagate global configuration rout...

2016-12-22 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1856 @ustcweizhou this will be useful for 4.9 as well, would you mind rebasing the patch to 4.9? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1854: 4.9 multiplex testing

2016-12-22 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1854 @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 #1854: 4.9 multiplex testing

2016-12-22 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1854 @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 pull request #1856: CLOUDSTACK-9569: propagate global configurati...

2016-12-22 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1856#discussion_r93603576 --- Diff: core/src/com/cloud/agent/api/SetHostParamsCommand.java --- @@ -0,0 +1,43 @@ +// +// Licensed to the Apache Software Foundation (ASF)

[GitHub] cloudstack pull request #1856: CLOUDSTACK-9569: propagate global configurati...

2016-12-22 Thread ustcweizhou
Github user ustcweizhou commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1856#discussion_r93604300 --- Diff: core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java --- @@ -259,8 +260,18 @@ private Answer execute(GetDomRVersion

[GitHub] cloudstack issue #1856: CLOUDSTACK-9569: propagate global configuration rout...

2016-12-22 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1856 rebase with apache/4.9 --- 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 issue #1856: CLOUDSTACK-9569: propagate global configuration rout...

2016-12-22 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1856 @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 a

[GitHub] cloudstack issue #1856: CLOUDSTACK-9569: propagate global configuration rout...

2016-12-22 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1856 @borisstoyanov 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 re

[GitHub] cloudstack pull request #1857: BUG-ID: CLOUDSTACK-8284

2016-12-22 Thread adwaitpatankar
GitHub user adwaitpatankar opened a pull request: https://github.com/apache/cloudstack/pull/1857 BUG-ID: CLOUDSTACK-8284 On expunge VM, initiate resource count update at domain level. Setting accountId as null ensures resource count is done at domain level and any hierarchies be

[GitHub] cloudstack pull request #1856: CLOUDSTACK-9569: propagate global configurati...

2016-12-22 Thread ustcweizhou
Github user ustcweizhou commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1856#discussion_r93606574 --- Diff: core/src/com/cloud/agent/api/SetHostParamsCommand.java --- @@ -0,0 +1,43 @@ +// +// Licensed to the Apache Software Foundation (ASF)

[GitHub] cloudstack issue #1700: CLOUDSTACK-9359: IPv6 for Basic Networking

2016-12-22 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1700 @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 #1700: CLOUDSTACK-9359: IPv6 for Basic Networking

2016-12-22 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1700 @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 #1856: CLOUDSTACK-9569: propagate global configuration rout...

2016-12-22 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1856 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-415 --- 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 #1700: CLOUDSTACK-9359: IPv6 for Basic Networking

2016-12-22 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1700 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-416 --- 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

Re: Error on Master

2016-12-22 Thread Will Stevens
No problem. I think it is because I have been upgrading this env from master to master periodically and the schema changes were only recently added in the upgrade sql. We should be good. Thanks for checking. I got my env working with the details in this thread, so hopefully it will help others if t

[GitHub] cloudstack issue #1837: [4.9] Smoketest Health

2016-12-22 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1837 Trillian test result (tid-708) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 36003 seconds Marvin logs: https://github.com/blueoranguta

[GitHub] cloudstack pull request #1858: README: Happy Christmas, happy holidays!

2016-12-22 Thread rhtyd
GitHub user rhtyd opened a pull request: https://github.com/apache/cloudstack/pull/1858 README: Happy Christmas, happy holidays! Here's a screenshot to confirm the UI change: ![screenshot from 2016-12-22 23-42-31](https://cloud.githubusercontent.com/assets/95203/21435538/64d5fdd

[GitHub] cloudstack issue #1858: README: Happy Christmas, happy holidays!

2016-12-22 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1858 LGTM :+1: --- 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, o

[GitHub] cloudstack issue #1858: README: Happy Christmas, happy holidays!

2016-12-22 Thread K0zka
Github user K0zka commented on the issue: https://github.com/apache/cloudstack/pull/1858 🥇 👍 :) --- 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 #1858: README: Happy Christmas, happy holidays!

2016-12-22 Thread kiwiflyer
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1858 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 if th

[GitHub] cloudstack issue #1858: README: Happy Christmas, happy holidays!

2016-12-22 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/1858 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 if the fea

[GitHub] cloudstack issue #1858: README: Happy Christmas, happy holidays!

2016-12-22 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1858 Lgtm happy holidays On Thu, Dec 22, 2016 at 13:48 Simon Weller wrote: > LGTM > > — > You are receiving this because you were mentioned. > Reply to this email directly,

[GitHub] cloudstack issue #1858: README: Happy Christmas, happy holidays!

2016-12-22 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1858 👍 --- 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] cloudstack issue #1858: README: Happy Christmas, happy holidays!

2016-12-22 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1858 +1 happy holidays! --- 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 #1854: 4.9 multiplex testing

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

[GitHub] cloudstack issue #1849: CLOUDSTACK-9690: Scale CentOS7 VM fails with error

2016-12-22 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/1849 @rhtyd Thanks for the Review. I have amended the code as per your suggestion. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] cloudstack issue #1854: 4.9 multiplex testing

2016-12-22 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1854 Trillian test result (tid-718) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 31629 seconds Marvin logs: https://github.com/blueoranguta

[GitHub] cloudstack issue #1858: README: Happy Christmas, happy holidays!

2016-12-22 Thread pdube
Github user pdube commented on the issue: https://github.com/apache/cloudstack/pull/1858 Happy holidays! --- 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] cloudstack issue #1854: 4.9 multiplex testing

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

[GitHub] cloudstack issue #1858: README: Happy Christmas, happy holidays!

2016-12-22 Thread milamberspace
Github user milamberspace commented on the issue: https://github.com/apache/cloudstack/pull/1858 LGTM and Happy holidays! --- 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

[GitHub] cloudstack issue #1858: README: Happy Christmas, happy holidays!

2016-12-22 Thread DaanHoogland
Github user DaanHoogland commented on the issue: https://github.com/apache/cloudstack/pull/1858 If we have to On Thu, Dec 22, 2016 at 9:02 PM, Milamber wrote: > LGTM and Happy holidays! > > — > You are receiving this because you were mentioned. > Rep

[GitHub] cloudstack pull request #1838: CLOUDSTACK-9682: Block VM migration to a stor...

2016-12-22 Thread anshul1886
Github user anshul1886 closed the pull request at: https://github.com/apache/cloudstack/pull/1838 --- 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 featur

[GitHub] cloudstack issue #1854: 4.9 multiplex testing

2016-12-22 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1854 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-417 --- 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 #1858: README: Happy Christmas, happy holidays!

2016-12-22 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1858 Alright merging this based on reviews, LGTMs and that Travis is green :smile: Happy holidays! --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request #1858: README: Happy Christmas, happy holidays!

2016-12-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1858 --- 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