[ANNOUNCE] New committer: Simon Weller

2016-04-28 Thread Erik Weber
The Project Management Committee (PMC) for Apache CloudStack has asked Simon Weller to become a committer and we are pleased to announce that they have accepted. Being a committer allows many contributors to contribute more autonomously. For developers, it makes it easier to submit changes and el

[GitHub] cloudstack pull request: CLOUDSTACK-9365 : updateVirtualMachine wi...

2016-04-28 Thread nlivens
Github user nlivens commented on the pull request: https://github.com/apache/cloudstack/pull/1523#issuecomment-215336954 @DaanHoogland, @remibergsma, @swill, @jburwell, these are rather small changes, if any of you could review, that would be great! --- If your project is set up for

RE: [ANNOUNCE] New committer: Simon Weller

2016-04-28 Thread Paul Angus
Congratulations Simon! Kind regards, Paul Angus Regards, Paul Angus paul.an...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, London WC2N 4HSUK @shapeblue -Original Message- From: Erik Weber [mailto:terbol...@gmail.com] Sent: 28 April 2016 08:24 To: dev Subject

[GitHub] cloudstack pull request: kvm: Aqcuire lock when running security g...

2016-04-28 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1408#issuecomment-215338991 @rhtyd Hmm, that's odd. My logs showed that the lock prevented the script from running twice and our problems were also solved with this. Before we had race co

Re: [LOGISTICS] Move 'apache/cloudstack' to 'apache-cloudstack/cloudstack'

2016-04-28 Thread Bertrand Delacretaz
Hi Will, On Wed, Apr 27, 2016 at 8:01 PM, Will Stevens wrote: > ...I have CC'ed in both the ACS dev@ list and the ASF board@ list to try to > keep everyone on the same page... Thanks - with my board member hat on what's important is that relevant changes are mentioned in your next report to the

Re: [ANNOUNCE] New committer: Simon Weller

2016-04-28 Thread Nux!
Congrats! :-) -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Paul Angus" > To: dev@cloudstack.apache.org > Sent: Thursday, 28 April, 2016 08:44:36 > Subject: RE: [ANNOUNCE] New committer: Simon Weller > Congratulations Simon! > >

[GitHub] cloudstack pull request: kvm: Aqcuire lock when running security g...

2016-04-28 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1408#issuecomment-215346544 @rhtyd: I tested and you are partially right. My test script: https://gist.github.com/wido/a236e9f065cef85b0497990e3d4798e6 When using the 'with open() as X' s

Re: IPv6 progress in Basic Networking

2016-04-28 Thread Abhinandan Prateek
On 25/04/16, 4:39 PM, "Wido den Hollander" wrote: > >> Op 25 april 2016 om 12:45 schreef Abhinandan Prateek >> : >> >> >> >> At shapeblue we are also working towards enabling ipv6 specially for >> advanced networking/ospf. >> Our approach is more of bottom up where we plan to streamline ba

[GitHub] cloudstack pull request: Kvm plugin restructure

2016-04-28 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1521#issuecomment-215347141 I am not against this restructure perse. But shall we not try to change to many things at once? We have enough on our plates, so talking about leaving Maven se

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers don't ha...

2016-04-28 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-215347296 @dsclose I think it is better to split this PR into some isolated PRs, as the issues are isolated. to be honest, some commits looks good to me ( as we have s

[GitHub] cloudstack pull request: CLOUDSTACK-9194: Making the console popup...

2016-04-28 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1270#issuecomment-215348912 Changes LGTM. UI changes so CI results doesn't matter. @anshul1886 From the fix looks like applicable to all browsers, is that correct? --- If your project

[GitHub] cloudstack pull request: CLOUDSTACK-9200: Fixed failed to delete s...

2016-04-28 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1282#issuecomment-215350146 @anshul1886 The fix is only for XS snapshots. What about other HVs? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: CLOUDSTACK-9348: Use non-blocking SSL han...

2016-04-28 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1493#issuecomment-215350128 @swill there are in total 25 malicious clients that can block for 60s for all 5 (max.) server worker threads; so worst case we should have waited for at least 25*60/5

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-04-28 Thread NuxRo
Github user NuxRo commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-215350363 Applied successfully to 4.8 and tested, it works fine as far as I could tell, but the SG change operation goes through even if the VM is online, so "CLOUDSTACK-9203 P

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

2016-04-28 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-215352448 @DaanHoogland @wido @mlsorensen @jburwell @agneya2001 @koushik-das @terbolous @resmo @K0zka review and LGTM please? @swill I think we've resolved all outsta

[GitHub] cloudstack pull request: kvm: Aqcuire lock when running security g...

2016-04-28 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1408#issuecomment-215353110 @wido +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 en

[GitHub] cloudstack pull request: kvm: Aqcuire lock when running security g...

2016-04-28 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1408#issuecomment-215353392 reviewed updated PR, LGTM cc @swill --- 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 proje

Re: IPv6 progress in Basic Networking

2016-04-28 Thread Nux!
Hi, see in-line -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro >>Since we know the MAC address and the /64 prefix we can *calculate* the >>address >>a Instance will take. With that we do not have to store the address in a >>database. > > Doing bit of research on this ma

[GitHub] cloudstack pull request: CLOUDSTACK-8970 Centos 6.{1,2,3,4,5} gues...

2016-04-28 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/956#issuecomment-215364487 Changes LGTM. @swill This can be merged. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If y

Re: [ANNOUNCE] New committer: Simon Weller

2016-04-28 Thread Nick LIVENS
Congrats Simon! Kind regards, Nick Livens On Thu, Apr 28, 2016 at 10:15 AM, Nux! wrote: > Congrats! :-) > > -- > Sent from the Delta quadrant using Borg technology! > > Nux! > www.nux.ro > > - Original Message - > > From: "Paul Angus" > > To: dev@cloudstack.apache.org > > Sent: Thursda

[GitHub] cloudstack pull request: CLOUDSTACK-8969 VPN customer gateway can'...

2016-04-28 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/955#issuecomment-215365287 @SudharmaJain Any updates? --- 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 n

[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-8950 Hypervisor Paramete...

2016-04-28 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/928#issuecomment-215366187 @karuturi Any updates on this? --- 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 do

[GitHub] cloudstack pull request: [4.7] secure and hidden config values are...

2016-04-28 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/927#issuecomment-215366546 @karuturi Any updates? --- 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 h

[GitHub] cloudstack pull request: CLOUDSTACK-9194: Making the console popup...

2016-04-28 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/1270#issuecomment-215368331 @koushik-das Other browsers except IE stopped supporting that attribute. So it will not have impact. This can also be verified by seeing in other browsers consol

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

2016-04-28 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1500#issuecomment-215372191 LGTM. Had already given one earlier based on simulator testing. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: CLOUDSTACK-9200: Fixed failed to delete s...

2016-04-28 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/1282#issuecomment-215375588 @koushik-das All hypervisor uses XenServerSnapshotStrategy so it should be applicable on all hypervisors. --- If your project is set up for it, you can reply to

Incorrect system vm templates for master build.

2016-04-28 Thread Bharat Kumar
Hi All, looks like the systemvm templates are not getting built correctly in http://jenkins.buildacloud.org/job/build-systemvm64-master. The build version is in correctly set to 4.6, Im not sure but i think the build version in /etc/cloudstck-release is also getting wrongly set inside the temp

Re: Incorrect system vm templates for master build.

2016-04-28 Thread Rajani Karuturi
In the job configuration, I see the version as 4.6.0. I am not sure if that is where it reads from. I changed it to 4.9.0 - export version=4.6.0 + export version=4.9.0 I will keep an eye on the job. Let me know if there are any issues with this change. ~Rajani On Thu, Apr 28, 2016 at 3:34 PM,

Re: Incorrect system vm templates for master build.

2016-04-28 Thread Erik Weber
AFAIK it hasn't been necessary to update the systemvm template yet, that is probably why it still shows as 4.6.0. -- Erik On Thu, Apr 28, 2016 at 12:04 PM, Bharat Kumar wrote: > Hi All, > > looks like the systemvm templates are not getting built correctly in > http://jenkins.buildacloud.org/jo

Re: IPv6 progress in Basic Networking

2016-04-28 Thread Wido den Hollander
> Op 28 april 2016 om 10:50 schreef Nux! : > > > Hi, see in-line > > -- > Sent from the Delta quadrant using Borg technology! > > Nux! > www.nux.ro > > >>Since we know the MAC address and the /64 prefix we can *calculate* the > >>address > >>a Instance will take. With that we do not have to

Rafael Weingartner joining the PMC

2016-04-28 Thread Daan Hoogland
People, The PMC has ask Rafael to join them to oversee the project and he has gracefully accepted. Please join me in congratulating Rafael and wish him wisdom in his new task. ​on behalve of the PMC,​ -- Daan

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

2016-04-28 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-215389142 +1 on the feature. Based on testing done on the simulator with root admin and normal user for a few APIs +0 on the immediate removal of command.properties/st

[ANNOUNCE] Rafael Weingartner joining the PMC

2016-04-28 Thread Daan Hoogland
Guess I should have facilitated everybody's mail filters On Thu, Apr 28, 2016 at 12:59 PM, Daan Hoogland wrote: > People, > > The PMC has ask Rafael to join them to oversee the project and he has > gracefully accepted. Please join me in congratulating Rafael and wish him > wisdom in his new task

Re: Rafael Weingartner joining the PMC

2016-04-28 Thread Sebastien Goasguen
Congrats Rafael, > On Apr 28, 2016, at 12:59 PM, Daan Hoogland wrote: > > People, > > The PMC has ask Rafael to join them to oversee the project and he has > gracefully accepted. Please join me in congratulating Rafael and wish him > wisdom in his new task. > > ​on behalve of the PMC,​ > --

[GitHub] cloudstack pull request: CLOUDSTACK-8818: Use MySQL native connect...

2016-04-28 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1054#issuecomment-215391701 @swill @rhtyd: I looked through all the code and fixed it. Is was just two more scripts. wido@wido-desktop:~/repos/cloudstack$ grep -R "MySQLdb" * wido@wid

[GitHub] cloudstack pull request: CLOUDSTACK-8818: Use MySQL native connect...

2016-04-28 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1054#issuecomment-215392929 @wido that is true, so will simply fixing the imports and usage work? I found using `git grep MySQLdb`: ``` client/bindir/cloud-update-xenserver-licens

Re: [ANNOUNCE] New committer: Simon Weller

2016-04-28 Thread Daan Hoogland
welcome, kiwi. Good work and keep going. On Thu, Apr 28, 2016 at 11:25 AM, Nick LIVENS wrote: > Congrats Simon! > > Kind regards, > Nick Livens > > On Thu, Apr 28, 2016 at 10:15 AM, Nux! wrote: > > > Congrats! :-) > > > > -- > > Sent from the Delta quadrant using Borg technology! > > > > Nux! >

Re: Incorrect system vm templates for master build.

2016-04-28 Thread Rajani Karuturi
ah..makes sense. Thanks Erik. I will revert the config change. ~Rajani On Thu, Apr 28, 2016 at 4:11 PM, Erik Weber wrote: > AFAIK it hasn't been necessary to update the systemvm template yet, that is > probably why it still shows as 4.6.0. > > -- > Erik > > On Thu, Apr 28, 2016 at 12:04 PM, Bha

RE: CaaS in CloudStack with Kubernetes

2016-04-28 Thread Sanjeev Neelarapu
Congratulations Sebastien !! Best Regards, Sanjeev N Chief Product Engineer, Accelerite Off: +91 40 6722 9368 | EMail: sanjeev.neelar...@accelerite.com -Original Message- From: Sebastien Goasguen [mailto:run...@gmail.com] Sent: Wednesday, April 27, 2016 11:41 PM To: dev@cloudstack.apa

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

2016-04-28 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1500#issuecomment-215401063 @swill CI Test good, 2+ reviews. Ready to Merge. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

Re: Rafael Weingartner joining the PMC

2016-04-28 Thread Nux!
Congrats Rafael :) -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Daan Hoogland" > To: "dev" > Sent: Thursday, 28 April, 2016 11:59:28 > Subject: Rafael Weingartner joining the PMC > People, > > The PMC has ask Rafael to join them

Re: CaaS in CloudStack with Kubernetes

2016-04-28 Thread Simon Weller
Very exciting Sebastien. Congratulations! We're a heavy user of containers, so we'll definitely be taking a look at this when it is released. - Si From: Sanjeev Neelarapu Sent: Thursday, April 28, 2016 6:41 AM To: dev@cloudstack.apache.org Cc: market...

Re: Rafael Weingartner joining the PMC

2016-04-28 Thread Simon Weller
Congratulations Rafael! - Si From: Nux! Sent: Thursday, April 28, 2016 7:06 AM To: dev@cloudstack.apache.org Subject: Re: Rafael Weingartner joining the PMC Congrats Rafael :) -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro

Re: IPv6 progress in Basic Networking

2016-04-28 Thread Nux!
Ok Wido, thanks for the clarification, it sounds good. >From my point of view, if there is a clear relation between IPs and VMs, then >as an operator I am happy. Lucian -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Wido den Hollan

[GitHub] cloudstack pull request: Log asynchronous responses in the api log

2016-04-28 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1522#issuecomment-215407693 LGTM. Verified in api.log that job id is getting appended. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

2016-04-28 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1500#issuecomment-215408560 is there any UI change possible? --- 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: CLOUDSTACK-9283: add pid to java argument...

2016-04-28 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1409#issuecomment-215409715 Tested this in a 4.8 lab. I agree that there is little point in running CI against this, as it's a config change. LGTM. --- If your project is set up f

Re: Incorrect system vm templates for master build.

2016-04-28 Thread Will Stevens
4.9 will require a new system vm template when we add the strongswan support and ospf, but I don't think I have added anything yet which will require a new system vm (yet). Rohit started a dev@ thread about a week or so ago on this topic for us. On Apr 28, 2016 7:38 AM, "Rajani Karuturi" wrote:

[GitHub] cloudstack pull request: CLOUDSTACK-9365 : updateVirtualMachine wi...

2016-04-28 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1523#issuecomment-215410690 thought the code looks sane, could you be bothered to show the error in a marvin test, Nick. It seems a good candidate to enter the regression suite. --- If y

[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

2016-04-28 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1500#issuecomment-215410772 We'll take a look at this in a separate PR. We don't use the UI. --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] cloudstack pull request: CLOUDSTACK-8818: Use MySQL native connect...

2016-04-28 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1054#issuecomment-215419241 @rhtyd I grepped *after* my modifications :) Yes, the API between MySQLdb and mysql.connector is the same. The only difference is the import and the connection

[GitHub] cloudstack pull request: CLOUDSTACK-9365 : updateVirtualMachine wi...

2016-04-28 Thread nlivens
Github user nlivens commented on the pull request: https://github.com/apache/cloudstack/pull/1523#issuecomment-215421778 @DaanHoogland, you already have a UserData test script, do you want us to add this test to the same script, or create a new one for it? --- If your project is set

[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

2016-04-28 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1455#issuecomment-215427658 @swill , it is all green now. Can you please merge this PR ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

Re: Rafael Weingartner joining the PMC

2016-04-28 Thread Patrick Dube
Congratulations! On Thu, Apr 28, 2016 at 8:19 AM, Simon Weller wrote: > Congratulations Rafael! > > - Si > > > From: Nux! > Sent: Thursday, April 28, 2016 7:06 AM > To: dev@cloudstack.apache.org > Subject: Re: Rafael Weingartner joining the PMC > > Congr

Re: IPv6 progress in Basic Networking

2016-04-28 Thread Wido den Hollander
> Op 28 april 2016 om 14:22 schreef Nux! : > > > Ok Wido, thanks for the clarification, it sounds good. > > From my point of view, if there is a clear relation between IPs and VMs, then > as an operator I am happy. > Yes, there will. So for a POD VLAN you will assing *ONE* IPv6 /64. That giv

[GitHub] cloudstack pull request: Kvm plugin restructure

2016-04-28 Thread jburwell
Github user jburwell commented on the pull request: https://github.com/apache/cloudstack/pull/1521#issuecomment-215438021 I would love to dump Maven for Gradle. However, I agree with @wido -- we currently have much higher priorities to address. For me, it is fun to banter a bit abou

[GitHub] cloudstack pull request: Kvm plugin restructure

2016-04-28 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1521#issuecomment-215446492 Enjoying the discussion I would like to -1 for the entire refactor all at once. I changed this project because it was feasible and convenient. I can see myself

[GitHub] cloudstack pull request: CLOUDSTACK-9365 : updateVirtualMachine wi...

2016-04-28 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1523#issuecomment-215447324 @nlivens I don't care for more scripts, just for a test showing the problem so we no it doesn't regress. Whatever meets the goal easiest. --- If your project

Re: [LOGISTICS] Move 'apache/cloudstack' to 'apache-cloudstack/cloudstack'

2016-04-28 Thread Will Stevens
Thanks Bertrand, I can put a report together for the next board meeting if that will be helpful. Would you prefer that I remove the board@ list from this thread and only keep the ASF Infra team CC'ed? Cheers, Will On Thu, Apr 28, 2016 at 3:58 AM, Bertrand Delacretaz wrote: > Hi Will, > > On W

[GitHub] cloudstack pull request: CLOUDSTACK-8970 Centos 6.{1,2,3,4,5} gues...

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/956#issuecomment-215451916 Do we have any verification that this does what it is supposed to? It would be good if we could get some form of verification posted that this behaves as expected.

[GitHub] cloudstack pull request: CLOUDSTACK-9200: Fixed failed to delete s...

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1282#issuecomment-215460320 @anshul1886 I see in some other places in that file when `snapshotDao.remove(snapshotId);` is called there is a `return true` right after it. Can you help me underst

[GitHub] cloudstack pull request: Log asynchronous responses in the api log

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1522#issuecomment-215463060 This change is pretty straight forward and it has been manually verified by @koushik-das in addition to the output originally posted by @nathanejohnson. I think this

[GitHub] cloudstack pull request: CLOUDSTACK-8818: Use MySQL native connect...

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1054#issuecomment-215472985 So are we happy with the state of this? How should I adjust my CI to validate this fix? I currently have: ``` yum -y install maven tomcat mkisofs

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-215479369 In reading this, is it the case that if someone upgrades from a previous version to 4.9 (with this code included) they will not be able to take advantage of this func

[GitHub] cloudstack pull request: Cloudstack 9339: Virtual Routers don't ha...

2016-04-28 Thread dsclose
Github user dsclose commented on the pull request: https://github.com/apache/cloudstack/pull/1519#issuecomment-215480948 @ustcweizhou How would you recommend I separate this? I can imagine separating the issues broadly into two parts: 1. **Routing tables and iptables rules sho

[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1455#issuecomment-215481337 Yes, I think this one is ready... Can I get one more LGTM code review on this one? @pdube @kiwiflyer @DaanHoogland @rafaelweingartner --- If your projec

[GitHub] cloudstack pull request: Log asynchronous responses in the api log

2016-04-28 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1522#issuecomment-215486284 Code is straight forward and makes sense. I validated it works in the lab. LGTM. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: Log asynchronous responses in the api log

2016-04-28 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1522#issuecomment-215486826 @swill This is Ready to Merge --- 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

[GitHub] cloudstack pull request: Handle private gateways more reliably

2016-04-28 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1474#issuecomment-215492977 We're pulling this in for testing. --- 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: Log asynchronous responses in the api log

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1522#issuecomment-215493805 its added to my merge queue. I need to get through this queue today, it is getting long now. :) --- If your project is set up for it, you can reply to this email a

[GitHub] cloudstack pull request: Handle private gateways more reliably

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1474#issuecomment-215494066 cool thanks. :) --- 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 fea

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

2016-04-28 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-215501683 @swill existing users won't be automatically migrated to use this feature, see the admin-doc PR that documents the upgrade/migrate procedure; though new installations

[GitHub] cloudstack pull request: CLOUDSTACK-8818: Use MySQL native connect...

2016-04-28 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1054#issuecomment-215502330 @swill for the CI you can install using pip with: (this is also in tools/travis/install.sh) pip install --upgrade --egg http://cdn.mysql.com/Downloads/Connector-P

[GitHub] cloudstack pull request: CLOUDSTACK-8818: Use MySQL native connect...

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1054#issuecomment-215504839 I see this in the output when Marvin is installed. ``` Requirement already up-to-date: mysql-connector-python>=1.1.6 ``` If I check the version I h

[GitHub] cloudstack pull request: [4.7] vmware: improve support for disks

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1365#issuecomment-215505081 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 ``` **Associated Uploads** **

[GitHub] cloudstack pull request: [4.7] vmware: improve support for disks

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1365#issuecomment-215506015 These CI results basically show that the standard VM workflow in KVM is not affected. I don't have the ability to test this on VMware, but I know you guys are alread

[ASF PERKS] JetBrains products - free to committers

2016-04-28 Thread ilya
Just found a cool perk many would benefit from on this list. JetBrains is a company behind many development tools. AppCode CLion DataGrip dotCover dotMemory dotPeek dotTrace Hub IntelliJ IDEA Kotlin MPS PhpStorm PyCharm ReSharper RubyMine TeamCity Upsource WebStorm YouTrack If you are ASF comm

[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

2016-04-28 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1455#issuecomment-215512619 @sanju1010 though the code looks alright I don't like the integration test should be snooping under the hood. I would rather see them behave as black-box tests

[GitHub] cloudstack pull request: CLOUDSTACK-9340: General DB Optimization

2016-04-28 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1466#issuecomment-215529568 @rhtyd @rafaelweingartner I could discover the problem by deleting `cloud` and `cloud_usage` database, then `cloudstack-databases-setup` to create them blank and r

[GitHub] cloudstack pull request: Bump ssh retries to prevent false positiv...

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1473#issuecomment-215531806 @remibergsma can you force push again so we can try to get it all green? Thanks... --- If your project is set up for it, you can reply to this email and have your r

Re: Rafael Weingartner joining the PMC

2016-04-28 Thread Rafael Weingärtner
Thank you all for the warm welcome. It is an honor to be part of the Apache CloudStack (ACS) committee. I will try to work as much as I can to help the PMC, committers, and contributors on improving the ACS. Sometimes I might not be as present as others (as it is happening now), but that is due to

Re: [ANNOUNCE] New committer: Simon Weller

2016-04-28 Thread Rafael Weingärtner
Congratulations Simon, Let’s keep up with the good work ;) On Thu, Apr 28, 2016 at 8:34 AM, Daan Hoogland wrote: > welcome, kiwi. Good work and keep going. > > On Thu, Apr 28, 2016 at 11:25 AM, Nick LIVENS < > nick.liv...@nuagenetworks.net > > wrote: > > > Congrats Simon! > > > > Kind regards,

[GitHub] cloudstack pull request: CLOUDSTACK-9142 Migrate VM changes xmlDes...

2016-04-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1348 --- 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 pull request: 4.9 mvn version safeupgradeonly

2016-04-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1510 --- 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 pull request: Log asynchronous responses in the api log

2016-04-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1522 --- 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 pull request: CLOUDSTACK-9194: Making the console popup...

2016-04-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1270 --- 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 pull request: CLOUDSTACK-9349: Enable root disk detach ...

2016-04-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1500 --- 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 pull request: CLOUDSTACK-9340: General DB Optimization

2016-04-28 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1466#issuecomment-215546030 @nvazquez, What does that mean? First we are executing all of the `schema-XXXtoYYY.sql`, then all of the `schemaXXXtoYYY-cleanup.sql`? Instea

[GitHub] cloudstack pull request: CLOUDSTACK-9323: Fix cancel host maintena...

2016-04-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1454 --- 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 pull request: engine/schema: fix upgrade path to work w...

2016-04-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1517 --- 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 pull request: CLOUDSTACK-9283: add pid to java argument...

2016-04-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1409 --- 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 pull request: CLOUDSTACK-9340: General DB Optimization

2016-04-28 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1466#issuecomment-215548103 @rafaelweingartner exactly, I meant to explain that I realized that first all `schema-XXXtoYYY.sql` were executed and after they finished, all `schema-XXXtoYYY-cle

[GitHub] cloudstack pull request: CLOUDSTACK-9340: General DB Optimization

2016-04-28 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1466#issuecomment-215548664 @nvazquez that's by design -- all upgrade paths (migration code and the schema files) are run first and after that only the cleanup scripts are ran --- If your proje

[GitHub] cloudstack pull request: [Bug-Id: CLOUDSTACK-9306]Replace testdata...

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1436#issuecomment-215548871 @sanju1010 please review the CI run results when you have a second. Thanks... --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: CLOUDSTACK-9362: Skip VXLANs when rewriti...

2016-04-28 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1513#issuecomment-215549283 We'll pull this in for testing. --- 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 do

[GitHub] cloudstack pull request: CLOUDSTACK-9340: General DB Optimization

2016-04-28 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1466#issuecomment-215549970 I confess that without looking the code, I thought the same as @nvazquez. First we would execute schema-XXXtoYYY.sql, schemaXXXtoYYY-cleanup.sql and th

[GitHub] cloudstack pull request: CLOUDSTACK-9305: Cloudstack Usage Breaks ...

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1433#issuecomment-215551223 Running CI now. Will have results later tonight or tomorrow... --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack pull request: CLOUDSTACK-8818: Use MySQL native connect...

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1054#issuecomment-215551291 Running CI now. Will have results later tonight or tomorrow... --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack pull request: Kvm plugin restructure

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1521#issuecomment-215552234 @DaanHoogland please rebase as master has caused merge conflicts... --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] cloudstack pull request: CLOUDSTACK-9350: KVM-HA- Fix CheckOnHost ...

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1496#issuecomment-215552813 @abhinandanprateek please rebase as this currently has merge conflicts with master. Thanks... --- If your project is set up for it, you can reply to this email and

  1   2   >