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

2017-03-17 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/2003 thank you sir. :) --- 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 #2003: CLOUDSTACK-9811: fixed an issue if the dev is not in...

2017-03-17 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/2003 @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

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

2017-03-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2003 @borisstoyanov 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

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

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

2017-03-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2003 @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 issue #2003: CLOUDSTACK-9811: fixed an issue if the dev is not in...

2017-03-17 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/2003 not sure about the CI tests @swill, I think the easiest way to kick Travis tests is the close/reopen the PR. I'll pick it up as soon as possible. I'll rebuild and run the smoketests.

[GitHub] cloudstack pull request #2010: VPC Inline LB - CLOUDSTACK-9282

2017-03-17 Thread urvis
GitHub user urvis opened a pull request: https://github.com/apache/cloudstack/pull/2010 VPC Inline LB - CLOUDSTACK-9282 BUG-ID: CLOUDSTACK-9282 Design-Doc: https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=61340894 You can merge this pull request into a Git

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

2017-03-17 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/2003 @borisstoyanov alright, this is ready for you to start testing. Can you kick off CI on this as well? I will be doing testing of this locally as well. This implementation is

[GitHub] cloudstack issue #1994: CLOUDSTACK-9827: Storage tags stored in multiple pla...

2017-03-17 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1994 @karuturi I added marvin tests to simulate tests performed by @mike-tutkowski. This are results in our env: [root@ussarlabcsmgt41 cloudstack]# cat

Re: [ANNOUNCE] Wei Zhou and Rene Moser are now PMC members

2017-03-17 Thread Marco Sinhoreli
Congratulation and welcome aboard Wei and Rene! Marco Sinhoreli Managing Consultant marco.sinhor...@shapeblue.com mobile: +55 21 98276 3636 Praia de Botafogo 501, bloco 1 - sala 101 – Botafogo Rio de Janeiro, RJ - Brazil - CEP 22250-040 office: + 55 21 2586 6390 | fax: +55 21 2586 6002

[GitHub] cloudstack pull request #1994: CLOUDSTACK-9827: Storage tags stored in multi...

2017-03-17 Thread nvazquez
Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1994#discussion_r106724343 --- Diff: engine/schema/src/com/cloud/storage/dao/StoragePoolTagsDaoImpl.java --- @@ -77,4 +92,71 @@ public void deleteTags(long poolId) {

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

2017-03-17 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack/pull/2003 Thanks @remibergsma. I am going to use `source_nat` as it better represents what it is I am trying to establish and it seems to always be in the object where `first_i_p` does not always seem to

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

2017-03-17 Thread remibergsma
Github user remibergsma commented on the issue: https://github.com/apache/cloudstack/pull/2003 @swill FYI if you look for `first_i_p` as seen in the json on the Python side, then on the Java side it's called `firstIP`, as the gson lib replaces every capital with an underscore and

Re: [ANNOUNCE] Wei Zhou and Rene Moser are now PMC members

2017-03-17 Thread Tutkowski, Mike
Congratulations! > On Mar 17, 2017, at 8:57 AM, Rafael Weingärtner > wrote: > > Congratulation guys!! > You deserved it for your great contributions. > > Let’s keep the pace making ACS better every single PR ;) > >> On Fri, Mar 17, 2017 at 9:35 AM, Simon Weller

Re: More Jira tickets clean up

2017-03-17 Thread Rafael Weingärtner
Thanks, Mike for the feedback. No problem Daan ;) Tickets closed; depending on tickets, some I marked as closed, others as resolved. I also configured the status of some to invalid, others to incomplete, and others as fixed and so on. On Wed, Mar 15, 2017 at 6:38 PM, Tutkowski, Mike

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

2017-03-17 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/2003 @borisstoyanov could you please test with my suggestion? I think it should fix the duplicated nics in VR. --- If your project is set up for it, you can reply to this email and have your

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

2017-03-17 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1867 ### ACS CI BVT Run **Sumarry:** Build Number 471 Hypervisor xenserver NetworkType Advanced Passed=148 Failed=17 Skipped=10 _Link to logs Folder

[GitHub] cloudstack pull request #1582: CLOUDSTACK-9408 for the move away from downlo...

2017-03-17 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1582#discussion_r106675353 --- Diff: api/src/org/apache/cloudstack/api/response/AutoScaleVmProfileResponse.java --- @@ -75,6 +75,7 @@ @Parameter(name =

Re: [ANNOUNCE] Wei Zhou and Rene Moser are now PMC members

2017-03-17 Thread Will Stevens
Congrats. Both well deserved. :) On Mar 17, 2017 9:36 AM, "Simon Weller" wrote: > Congrats Rene and Wei! > > > From: Daan Hoogland > Sent: Friday, March 17, 2017 2:06 AM > To: dev@cloudstack.apache.org > Subject:

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

2017-03-17 Thread DaanHoogland
Github user DaanHoogland commented on the issue: https://github.com/apache/cloudstack/pull/1582 @karuturi I rebased on latest master, is anything else expected? --- 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

Re: [ANNOUNCE] Wei Zhou and Rene Moser are now PMC members

2017-03-17 Thread Nicolas Vazquez
Congratulations Wei and Rene! 2017-03-17 11:56 GMT-03:00 Rafael Weingärtner : > Congratulation guys!! > You deserved it for your great contributions. > > Let’s keep the pace making ACS better every single PR ;) > > On Fri, Mar 17, 2017 at 9:35 AM, Simon Weller

[GitHub] cloudstack pull request #1582: CLOUDSTACK-9408 for the move away from downlo...

2017-03-17 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1582#discussion_r106670299 --- Diff: api/src/org/apache/cloudstack/api/response/AutoScaleVmProfileResponse.java --- @@ -75,6 +75,7 @@ @Parameter(name =

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

2017-03-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2008 @borisstoyanov 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

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

2017-03-17 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/2008 @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

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

2017-03-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2008 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-593 --- 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

Re: [ANNOUNCE] Wei Zhou and Rene Moser are now PMC members

2017-03-17 Thread Rafael Weingärtner
Congratulation guys!! You deserved it for your great contributions. Let’s keep the pace making ACS better every single PR ;) On Fri, Mar 17, 2017 at 9:35 AM, Simon Weller wrote: > Congrats Rene and Wei! > > > From: Daan Hoogland

[GitHub] cloudstack pull request #1582: CLOUDSTACK-9408 for the move away from downlo...

2017-03-17 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1582#discussion_r106667548 --- Diff: api/src/org/apache/cloudstack/api/response/AutoScaleVmProfileResponse.java --- @@ -75,6 +75,7 @@ @Parameter(name =

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

2017-03-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/2008 @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 issue #2008: Fix datetime format of snapshots events

2017-03-17 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/2008 Thanks for this PR @olivierlemasle, could you please add a jira id for this change and rename the PR to [JIRA-ID]: Fix datetime ... same as the others @blueorangutan package --- If

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

2017-03-17 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/2003 Hi @swill, could you please let us know when do you expect to address the changes you have mentioned, so I could schedule the testing accordingly. Thanks. --- If your project is set up

Re: [ANNOUNCE] Wei Zhou and Rene Moser are now PMC members

2017-03-17 Thread Simon Weller
Congrats Rene and Wei! From: Daan Hoogland Sent: Friday, March 17, 2017 2:06 AM To: dev@cloudstack.apache.org Subject: [ANNOUNCE] Wei Zhou and Rene Moser are now PMC members dear devs, The PMC has invited Rene Moser and Wei Zhou

[GitHub] cloudstack issue #1994: CLOUDSTACK-9827: Storage tags stored in multiple pla...

2017-03-17 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1994 Hi @karuturi, I've been working on marvin tests, I hope posting them today --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

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

2017-03-17 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/1867 @koushik-das Failing tests doesn't seems to be related to this patch. Rebased against latest master to trigger the build. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack issue #1853: CLOUDSTACK-9696: Fixed Virtual Router deployment fai...

2017-03-17 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/1853 @koushik-das Made the changes as per review comments. --- 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

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

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

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

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

[GitHub] cloudstack issue #1797: CLOUDSTACK-9630: Cannot use listNics API as advertis...

2017-03-17 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/1797 @pdion891 removed broadcasturi from api response. Also added marvin 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

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

2017-03-17 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1859 @rajesh-battala I won't accept this PR, sorry to share this but the number of commits are simply outrageous. From my experience RM-ing for 4.3, 4.5, 4.9 -- the git history is pretty

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

2017-03-17 Thread rajesh-battala
Github user rajesh-battala commented on the issue: https://github.com/apache/cloudstack/pull/1859 Current commits are in the order of some milestones. As @nitin-maharan told if we squash them into one commit we loose contributions of induviduals. On Thu, Mar 16,

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

2017-03-17 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

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

2017-03-17 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

Re: Welcoming Wido as the new ACS VP

2017-03-17 Thread Rene Moser
Thanks Will for all the efforts! Congrats Wido and I have no doubts you keep up the good work ;) -Rene

Re: [ANNOUNCE] Wei Zhou and Rene Moser are now PMC members

2017-03-17 Thread Raja Pullela
congratulations Rene and Wei!! cheers! Raja Pullela Engineering Team, Accelerite, www.accelerite.com, 2055 Laurelwood Road, Santa Clara, CA, 95054 On 3/17/17, 12:36 PM, "Daan Hoogland" wrote: dear devs, The PMC has invited Rene Moser and Wei Zhou over the last

Re: Welcoming Wido as the new ACS VP

2017-03-17 Thread Daan Hoogland
Thanks to both of you great Ws. Have a good retirement Will! Good luck in your new capacity Wido! On 17/03/17 08:32, "Paul Angus" wrote: Thanks Will for all the great work. And congratulations Wido - good luck. Kind regards, Paul Angus

[GitHub] cloudstack issue #2009: CLOUDSTACK-9369 Fixed Ldap regression

2017-03-17 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/2009 @rhtyd Can you review the fix? --- 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 pull request #2009: CLOUDSTACK-9369 Fixed Ldap regression

2017-03-17 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/2009 CLOUDSTACK-9369 Fixed Ldap regression Ldap auto creation of accounts is broken due to the security fix for CLOUDSTACK-9369. There was an explicit check to not allow login incase the

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

2017-03-17 Thread DaanHoogland
Github user DaanHoogland commented on the issue: https://github.com/apache/cloudstack/pull/1582 sorry @karuturi should I update? This is really not on my radar that much anymore. I will look at it this afternoon. --- If your project is set up for it, you can reply to this email and

RE: Welcoming Wido as the new ACS VP

2017-03-17 Thread Paul Angus
Thanks Will for all the great work. And congratulations Wido - good luck. Kind regards, Paul Angus paul.an...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, London WC2N 4HSUK @shapeblue -Original Message- From: Raja Pullela

RE: [ANNOUNCE] Wei Zhou and Rene Moser are now PMC members

2017-03-17 Thread Paul Angus
Congratulations Rene and Wei. You're great supporters of the CloudStack cause, it's well deserved. Kind regards, Paul Angus paul.an...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, London WC2N 4HSUK @shapeblue -Original Message- From: Daan Hoogland

Re: [ANNOUNCE] Wei Zhou and Rene Moser are now PMC members

2017-03-17 Thread Boris Stoyanov
Congrats Rene and Wei!!! Well deserved. Thanks, Boris Stoyanov boris.stoya...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, London WC2N 4HSUK @shapeblue > On Mar 17, 2017, at 9:06 AM, Daan Hoogland > wrote: > > dear devs, > > The PMC

[GitHub] cloudstack issue #1911: CLOUDSTACK-9742:Simultaneous snapshots for detached ...

2017-03-17 Thread mrunalinikankariya
Github user mrunalinikankariya commented on the issue: https://github.com/apache/cloudstack/pull/1911 [ConcurrentSnapshots.docx](https://github.com/apache/cloudstack/files/849895/ConcurrentSnapshots.docx) Attached file contains screenshot of the database quesry and the

Re: Welcoming Wido as the new ACS VP

2017-03-17 Thread Rohit Yadav
Congrats Wido! Thanks Will for your effort and work. Regards. From: Will Stevens Sent: 16 March 2017 22:30:02 To: dev@cloudstack.apache.org; us...@cloudstack.apache.org Subject: Welcoming Wido as the new ACS VP Hello Everyone, It has been a

Re: [ANNOUNCE] Wei Zhou and Rene Moser are now PMC members

2017-03-17 Thread Rohit Yadav
Congratulations Rene and Wei! Well deserved. Regards. From: Daan Hoogland Sent: 17 March 2017 12:36:14 To: dev@cloudstack.apache.org Subject: [ANNOUNCE] Wei Zhou and Rene Moser are now PMC members dear devs, The PMC has invited

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

2017-03-17 Thread anshul1886
GitHub user anshul1886 reopened a pull request: https://github.com/apache/cloudstack/pull/1867 CLOUDSTACK-9706: Added snapshots cleanup in start and storage GC thre… …ad if they are failed to cleanup during DeleteSnapshot command You can merge this pull request into a Git

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

2017-03-17 Thread anshul1886
Github user anshul1886 closed the pull request at: https://github.com/apache/cloudstack/pull/1867 --- 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 #1867: CLOUDSTACK-9706: Added snapshots cleanup in start an...

2017-03-17 Thread koushik-das
Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1867 Code changes LGTM @anshul1886 Please check on travis failure --- 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

[ANNOUNCE] Wei Zhou and Rene Moser are now PMC members

2017-03-17 Thread Daan Hoogland
dear devs, The PMC has invited Rene Moser and Wei Zhou over the last couple of weeks to join their ranks. Both have accepted. Please join me in congratulating both, thanking them for their contributions so far and encouraging them to show their best for Apache CloudStack ;) \o/ ... regards,

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

2017-03-17 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

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

2017-03-17 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-17 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

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

2017-03-17 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