[GitHub] cloudstack issue #2003: CLOUDSTACK-9811: fixed an issue if the dev is not in...

2017-03-16 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/2003 @borisstoyanov I think this is caused by KVM plugin. can you please perform the following on KVM host ? (1) change INFO to DEBUG in /etc/cloudstack/agent/log4j-cloud.xml, and restart

[GitHub] cloudstack issue #2003: CLOUDSTACK-9811: fixed an issue if the dev is not in...

2017-03-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/2003 Thanks @ustcweizhou here's the StartCommand log ``` 2017-03-16 08:48:46,068 DEBUG [cloud.agent.Agent] (agentRequest-Handler-3:null) (logid:be1f1987) Request:Seq 1-208882580216

[GitHub] cloudstack issue #1269: CLOUDSTACK-8867: Added retry logic to reconnect to h...

2017-03-16 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/1269 @GabrielBrascher The other method is there for legacy purposes and is no longer get used in Cloudstack. This was getting used in 2.x versions for UI. Some third party tool might be using it s

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

2017-03-16 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1853#discussion_r106378310 --- Diff: server/src/com/cloud/dc/dao/DedicatedResourceDaoImpl.java --- @@ -312,6 +317,65 @@ public DedicatedResourceVO findByHostId(Long hostId) {

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

2017-03-16 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1853#discussion_r106380082 --- Diff: server/src/com/cloud/deploy/DeploymentPlanningManagerImpl.java --- @@ -584,105 +583,64 @@ private void checkForNonDedicatedResources(Virtua

[GitHub] cloudstack issue #2003: CLOUDSTACK-9811: fixed an issue if the dev is not in...

2017-03-16 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/2003 @borisstoyanov I suspect this is caused by the name of your public interface (p55p1). can you please try the following change ``` diff --git a/plugins/hypervisors/kvm/src/co

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

2017-03-16 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1853#discussion_r106381321 --- Diff: server/src/com/cloud/deploy/DeploymentPlanningManagerImpl.java --- @@ -584,105 +583,64 @@ private void checkForNonDedicatedResources(Virtua

[GitHub] cloudstack pull request #1867: CLOUDSTACK-9706: Added snapshots cleanup in s...

2017-03-16 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1867#discussion_r106384378 --- Diff: server/src/com/cloud/storage/StorageManagerImpl.java --- @@ -1078,6 +1082,16 @@ public void cleanupStorage(boolean recurring) {

[GitHub] cloudstack pull request #1867: CLOUDSTACK-9706: Added snapshots cleanup in s...

2017-03-16 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1867#discussion_r106385035 --- Diff: engine/storage/snapshot/src/org/apache/cloudstack/storage/snapshot/SnapshotObject.java --- @@ -191,7 +191,8 @@ public void processEvent(O

[GitHub] cloudstack pull request #1867: CLOUDSTACK-9706: Added snapshots cleanup in s...

2017-03-16 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1867#discussion_r106386503 --- Diff: engine/storage/snapshot/src/org/apache/cloudstack/storage/snapshot/XenserverSnapshotStrategy.java --- @@ -194,18 +194,22 @@ protected bool

[GitHub] cloudstack pull request #1867: CLOUDSTACK-9706: Added snapshots cleanup in s...

2017-03-16 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1867#discussion_r106387348 --- Diff: engine/storage/snapshot/src/org/apache/cloudstack/storage/snapshot/SnapshotObject.java --- @@ -191,7 +191,8 @@ public void processEvent(O

[GitHub] cloudstack pull request #1867: CLOUDSTACK-9706: Added snapshots cleanup in s...

2017-03-16 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1867#discussion_r106387626 --- Diff: server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java --- @@ -1181,6 +1181,17 @@ public boolean configure(String name, Map params)

[GitHub] cloudstack pull request #1867: CLOUDSTACK-9706: Added snapshots cleanup in s...

2017-03-16 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1867#discussion_r106388631 --- Diff: server/src/com/cloud/storage/StorageManagerImpl.java --- @@ -1078,6 +1082,16 @@ public void cleanupStorage(boolean recurring) {

[GitHub] cloudstack issue #1889: CLOUDSTACK-9718: Revamp the dropdown showing lists o...

2017-03-16 Thread rashmidixit
Github user rashmidixit commented on the issue: https://github.com/apache/cloudstack/pull/1889 @ustcweizhou I have made appropriate changes. Can you please check once the validations pass? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1889: CLOUDSTACK-9718: Revamp the dropdown showing lists o...

2017-03-16 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1889 @rashmidixit thanks for the update. Compared with old patch, I notice you changed the return if args.page == 1. However, it does not solve my issue. We have several hosts (more than 1

[GitHub] cloudstack issue #1889: CLOUDSTACK-9718: Revamp the dropdown showing lists o...

2017-03-16 Thread rashmidixit
Github user rashmidixit commented on the issue: https://github.com/apache/cloudstack/pull/1889 @ustcweizhou thanks. I guess there was a misunderstanding. I understand what you are saying. I think i will just remove that dialog. The problem is that we are lazy loading the list

[GitHub] cloudstack issue #1867: CLOUDSTACK-9706: Added snapshots cleanup in start an...

2017-03-16 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/1867 @koushik-das Made the changes as per review. --- 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 #1879: CLOUDSTACK-9719: [VMware] VR loses DHCP settings and...

2017-03-16 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1879 @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 #1879: CLOUDSTACK-9719: [VMware] VR loses DHCP settings and...

2017-03-16 Thread blueorangutan
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 appe

[GitHub] cloudstack issue #1859: CLOUDSTACK-8672 : NCC Integration with CloudStack

2017-03-16 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1859 @nitin-maharana it becomes easier to triage changes when changes are confined to a limited number of commits (ideally one per PR), please squash the commits based on the author (if not to a single

[GitHub] cloudstack issue #1879: CLOUDSTACK-9719: [VMware] VR loses DHCP settings and...

2017-03-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1879 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-592 --- 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 #2003: CLOUDSTACK-9811: fixed an issue if the dev is not in...

2017-03-16 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/2003 I am considering changing my implementation to be the same as the old implementation (which removed the IP from the dbag in the initial loop of the merge), but if `source_nat` is present and it is

[GitHub] cloudstack pull request #2008: Fix datetime format of snapshots events

2017-03-16 Thread olivierlemasle
Github user olivierlemasle closed the pull request at: https://github.com/apache/cloudstack/pull/2008 --- 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 fe

[GitHub] cloudstack pull request #2008: Fix datetime format of snapshots events

2017-03-16 Thread olivierlemasle
GitHub user olivierlemasle reopened a pull request: https://github.com/apache/cloudstack/pull/2008 Fix datetime format of snapshots events Include the timezone in datetime format of snapshot events, to be consistent with every other events. "eventDateTime" was added by @chipchil

Welcoming Wido as the new ACS VP

2017-03-16 Thread Will Stevens
Hello Everyone, It has been a pleasure working with you as the ACS VP over the past year. I would like to say Thank You to everyone who has supported me in this role and have supported the project as a whole. It is my pleasure to announce that Wido den Hollander has been voted in to replace me as

Re: Welcoming Wido as the new ACS VP

2017-03-16 Thread Erik Weber
Thanks for all you hard work and dedication, Will! Congratulations Wido! :-) -- Erik tor. 16. mar. 2017 kl. 18.00 skrev Will Stevens : > Hello Everyone, > It has been a pleasure working with you as the ACS VP over the past year. > I would like to say Thank You to everyone who has supported me i

Re: Welcoming Wido as the new ACS VP

2017-03-16 Thread Gabriel Beims Bräscher
Congratulations Wido! :) Thank you very much for all the hard work Will! 2017-03-16 14:20 GMT-03:00 Erik Weber : > Thanks for all you hard work and dedication, Will! > > Congratulations Wido! :-) > > -- > Erik > tor. 16. mar. 2017 kl. 18.00 skrev Will Stevens : > > > Hello Everyone, > > It has b

Re: Welcoming Wido as the new ACS VP

2017-03-16 Thread Tutkowski, Mike
Thanks for taking on this role over the past year, Will! You did a great job. :-) > On Mar 16, 2017, at 11:00 AM, Will Stevens wrote: > > Hello Everyone, > It has been a pleasure working with you as the ACS VP over the past year. > I would like to say Thank You to everyone who has supported me

Re: Welcoming Wido as the new ACS VP

2017-03-16 Thread Wei ZHOU
Congratulations Wido ! Thanks you for your work as ACS VP, Will! -Wei 2017-03-16 18:00 GMT+01:00 Will Stevens : > Hello Everyone, > It has been a pleasure working with you as the ACS VP over the past year. > I would like to say Thank You to everyone who has supported me in this role > and have

[GitHub] cloudstack issue #1889: CLOUDSTACK-9718: Revamp the dropdown showing lists o...

2017-03-16 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1889 @rashmidixit finally I decide to say LGTM to this PR. --- 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 no

[GitHub] cloudstack issue #1889: CLOUDSTACK-9718: Revamp the dropdown showing lists o...

2017-03-16 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1889 LGTM, please merge it. --- 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: Welcoming Wido as the new ACS VP

2017-03-16 Thread Özhan Rüzgar Karaman
Great News & Congratulations Wido :) Thanks Will for your all effort & energy :) On Thu, Mar 16, 2017 at 9:57 PM, Wei ZHOU wrote: > Congratulations Wido ! > > Thanks you for your work as ACS VP, Will! > > -Wei > > > 2017-03-16 18:00 GMT+01:00 Will Stevens : > > > Hello Everyone, > > It has been

Re: Welcoming Wido as the new ACS VP

2017-03-16 Thread Raja Pullela
congratulations Wido! All the best with your new responsibilities! cheers! Raja Pullela Engineering, Accelerite, www.accelerite.com 2055 Laurelwood Road, Santa Clara, CA, 95054 On 3/16/17, 10:30 PM, "Will Stevens" wrote: Hello Everyone, It has been a pleasure working with you as the ACS VP o

Re: Welcoming Wido as the new ACS VP

2017-03-16 Thread Raja Pullela
Thank you Will for all the great work! cheers! Raja Pullela Engineering Team, Accelerite, 2055 Laurelwood Road, Santa Clara, CA, 95054 On 3/16/17, 10:30 PM, "Will Stevens" wrote: Hello Everyone, It has been a pleasure working with you as the ACS VP over the past year. I would like to say Than

[GitHub] cloudstack pull request #1889: CLOUDSTACK-9718: Revamp the dropdown showing ...

2017-03-16 Thread rashmidixit
GitHub user rashmidixit reopened a pull request: https://github.com/apache/cloudstack/pull/1889 CLOUDSTACK-9718: Revamp the dropdown showing lists of hosts available for migration in a Zone Refer to [CLOUDSTACK-9718](https://issues.apache.org/jira/browse/CLOUDSTACK-9718) for more

[GitHub] cloudstack pull request #1889: CLOUDSTACK-9718: Revamp the dropdown showing ...

2017-03-16 Thread rashmidixit
Github user rashmidixit closed the pull request at: https://github.com/apache/cloudstack/pull/1889 --- 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 featu

[GitHub] cloudstack issue #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

2017-03-16 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1873 @ustcweizhou I will add disable and CLOUDSTACK-9413 in new feature. --- 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 proj

Re: Welcoming Wido as the new ACS VP

2017-03-16 Thread Nicolas Vazquez
Congratulations Wido! Thanks Will for your hard work! 2017-03-16 23:58 GMT-03:00 Raja Pullela : > Thank you Will for all the great work! > > cheers! > Raja Pullela > Engineering Team, > Accelerite, 2055 Laurelwood Road, > Santa Clara, CA, 95054 > > On 3/16/17, 10:30 PM, "Will Stevens" wrote: > >

Re: Welcoming Wido as the new ACS VP

2017-03-16 Thread Rajani Karuturi
Thank you Will for all the great work. Congratulations Wido. ~ Rajani http://cloudplatform.accelerite.com/ On March 16, 2017 at 10:30 PM, Will Stevens (sw...@apache.org) wrote: Hello Everyone, It has been a pleasure working with you as the ACS VP over the past year. I would like to say Thank Y

[GitHub] cloudstack issue #1582: CLOUDSTACK-9408 for the move away from download.clou...

2017-03-16 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1582 @DaanHoogland any update? --- 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 #1994: CLOUDSTACK-9827: Storage tags stored in multiple pla...

2017-03-16 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1994 @nvazquez any update? --- 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 wis

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

2017-03-16 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1853#discussion_r106591079 --- Diff: server/src/com/cloud/deploy/DeploymentPlanningManagerImpl.java --- @@ -584,105 +583,64 @@ private void checkForNonDedicatedResources(Virtu

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

2017-03-16 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1853#discussion_r106589966 --- Diff: server/src/com/cloud/dc/dao/DedicatedResourceDaoImpl.java --- @@ -312,6 +317,65 @@ public DedicatedResourceVO findByHostId(Long hostId) {

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

2017-03-16 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1853#discussion_r106590720 --- Diff: server/src/com/cloud/deploy/DeploymentPlanningManagerImpl.java --- @@ -584,105 +583,64 @@ private void checkForNonDedicatedResources(Virtu

[GitHub] cloudstack issue #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

2017-03-16 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1873 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 t