[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

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

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,

[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

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 >

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

[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

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

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:

[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

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

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

[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

[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

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:

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

[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

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

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

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

[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

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

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:

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!

[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

[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

[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

[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

[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

[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

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

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

[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

[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

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*

[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

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"

[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

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

[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

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

2016-04-28 Thread nlivens
GitHub user nlivens opened a pull request: https://github.com/apache/cloudstack/pull/1523 CLOUDSTACK-9365 : updateVirtualMachine with userdata should not error… … when a VM is attached to multiple networks from which one or more doesn't support userdata You can merge this pull

[GitHub] cloudstack pull request: CLOUDSTACK-9175: [VMware DRS] Adding new ...

2016-04-28 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1257#issuecomment-215326963 @sureshanaparti There are some open items, please address them --- 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-9182: Some running VMs turned ...

2016-04-28 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1252#issuecomment-215327572 @sureshanaparti There are some comments, please address them. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: Bug-ID: CLOUDSTACK-8654: Added guest_os m...

2016-04-28 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1244#issuecomment-215325140 @kishankavala The changes needs to be moved to the latest sql upgrade files. --- If your project is set up for it, you can reply to this email and have your

Build failed in Jenkins: build-master-jdk18 #509

2016-04-28 Thread jenkins
See Changes: [anshul.gangwar] CLOUDSTACK-9194: Making the console popup window resizable in IE to [Daan Hoogland] CLOUDSTACK-9142 Migrate VM changes xmlDesc in a safe way [David Amorim Faria] CLOUDSTACK-9283: add pid to java

[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-215583632 If you run your CI tests, it should detect if it breaks something. But more importantly would be to check if the performance was improved. But, I am sure

Re: [ANNOUNCE] New committer: Simon Weller

2016-04-28 Thread Simon Weller
Thanks everyone! I look forward to continuing to help out where I can as we march towards 4.9. - Si From: Pierre-Luc Dion Sent: Thursday, April 28, 2016 4:20 PM To: dev@cloudstack.apache.org Subject: Re: [ANNOUNCE] New committer:

Re: Build failed in Jenkins: build-master-jdk18 #509

2016-04-28 Thread Will Stevens
Some of the tests that were added can take some time, so is it possible to bump the timeout value on Jenkins? If we were close before, we are going to be over now. *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com

Re: [ANNOUNCE] New committer: Simon Weller

2016-04-28 Thread Pierre-Luc Dion
Congrats Simon! On Thu, Apr 28, 2016 at 3:52 PM, Rafael Weingärtner < rafaelweingart...@gmail.com> wrote: > 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

[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-215572438 off course ;) I was only waiting for that LGTM --- 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-9340: General DB Optimization

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1466#issuecomment-215581686 should I be running some sort of CI or validation for this change? How do I validate this works and more importantly, not break anything? I also need another LGTM.

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

2016-04-28 Thread serg38
Github user serg38 commented on the pull request: https://github.com/apache/cloudstack/pull/1466#issuecomment-215572026 With this change PR passed Travis and Jenkins. @rafaelweingartner and @rhtyd will you be able to give your blessing? --- If your project is set up for it, you can

Re: Build failed in Jenkins: build-master-jdk18 #509

2016-04-28 Thread Daan Hoogland
the timout on this job was 60, (or 400% of the last job) I upped it to 120 minuted On Thu, Apr 28, 2016 at 11:58 PM, Will Stevens wrote: > Some of the tests that were added can take some time, so is it possible to > bump the timeout value on Jenkins? If we were close

[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

[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

[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

[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

[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

[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

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

[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

[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

[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

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

[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

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

2016-04-28 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1466#issuecomment-215586085 OK. I will just do a standard CI run then. Thx. --- 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 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

[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-215614954 None of the failures above are related to this PR, they are all issues I see periodically in my environment. All of the supported usage tests passed, so CI has

[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-215615124 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 1 Errors: 0 ``` **Summary of the problem(s):** ```

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

2016-04-28 Thread Slair1
Github user Slair1 commented on the pull request: https://github.com/apache/cloudstack/pull/1433#issuecomment-215615141 That's ok, 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

[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-215615196 CI has passed. The only issue we see is a connectivity issue I sometimes see in my environment. --- If your project is set up for it, you can reply to this email

[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-215614748 ### CI RESULTS ``` Tests Run: 103 Skipped: 0 Failed: 3 Errors: 2 ``` **Summary of the problem(s):** ```

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

[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

[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

[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

[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

[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

[GitHub] cloudstack pull request: Fix Sync of template.properties in Swift

2016-04-28 Thread jburwell
Github user jburwell commented on the pull request: https://github.com/apache/cloudstack/pull/1331#issuecomment-215620806 @rafaelweingartner I apologize -- I did not understand that you were focused on ``final`` not ``final static``. You are correct, ``final`` has nothing to do with

[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

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

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 >

[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

[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

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

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

[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

[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

[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

[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

[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

  1   2   >