[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

[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

[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 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

[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

[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

[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

[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

[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 #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

[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 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

[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

[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 |

[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

[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

[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

[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. >

[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

[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

[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

[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 #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,

[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

[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

[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,

[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

[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:

[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.

[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:

[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:

[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

[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

[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

[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

[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 #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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

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

[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: