[GitHub] cloudstack pull request: CLOUDSTACK-8457: Add option to authorize ...

2015-07-10 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/575#issuecomment-120254575 Hi @bhaisaab Code looks good and screenshot as well. :+1: Hints to test it manually? After hitting the check-box, what should I do?

RE: Introducing Boris Schrijver

2015-07-10 Thread Boris Schrijver
Hi all, Thanks for the warm welcome! I've setup my development environment and I'm ready to rock! Best regards, Boris Schrijver PCextreme B.V. MAIL: bo...@pcextreme.nl On July 10, 2015 at 10:15 AM Giles Sirett giles.sir...@shapeblue.com wrote: Welcome Boris Kind

[GitHub] cloudstack pull request: Improved message for peeples to actually ...

2015-07-10 Thread snuf
GitHub user snuf opened a pull request: https://github.com/apache/cloudstack/pull/576 Improved message for peeples to actually see what the problem is. The original message obfuscates what the real problem is, and requires people to dig through the source to figure out that a

[GitHub] cloudstack pull request: Improved message for peeples to actually ...

2015-07-10 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/576#issuecomment-120314141 LGTM, thanks for wading, @snuf --- 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

openssl/cloudstack

2015-07-10 Thread John Kinsella
Folks - just put up a brief blog post about the latest OpenSSL issue and how that affects CloudStack. Long story short - we think it does, but are verifying that. Hopefully will have an update by the end of the day. https://blogs.apache.org/cloudstack/entry/cloudstack_and_openssl_cve_2015 Will

[GitHub] cloudstack pull request: CLOUDSTACK-8622: Reinstate working sessio...

2015-07-10 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/574#issuecomment-120276323 LGTM be sure to add 2 'this closes #' lines. one for this and one for 308 --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: CLOUDSTACK-8622: Reinstate working sessio...

2015-07-10 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/574#issuecomment-120256667 page refreshes, opening multiple tabs, Nice! I also liked the String.format() calls. I should use it more as well. For some stuff still

RE: Introducing Boris Schrijver

2015-07-10 Thread Giles Sirett
Welcome Boris Kind Regards Giles D: +44 20 3603 0541 | M: +44 796 111 2055 giles.sir...@shapeblue.com -Original Message- From: Wido den Hollander [mailto:w...@widodh.nl] Sent: 08 July 2015 16:08 To: dev@cloudstack.apache.org Cc: bo...@pcextreme.nl Subject: Introducing Boris

[GitHub] cloudstack pull request: CLOUDSTACK-8605: KVM: Config Drive and ge...

2015-07-10 Thread kishankavala
GitHub user kishankavala opened a pull request: https://github.com/apache/cloudstack/pull/577 CLOUDSTACK-8605: KVM: Config Drive and getVmIp support - CLOUDSTACK-8324 added support for External DHCP/DNS support along with ConfigDrive for XenServer - FS link:

[GitHub] cloudstack pull request: CLOUDSTACK-8457: Add option to authorize ...

2015-07-10 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/575#issuecomment-120389991 @wilderrodrigues thanks. You can select the authorise sso checkbox and submit if you've a SAML IDP server. For test setup, you can follow this:

[GitHub] cloudstack pull request: Cloudstack 8612

2015-07-10 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/562#discussion_r34351937 --- Diff: framework/jobs/src/org/apache/cloudstack/framework/jobs/dao/SyncQueueItemDaoImpl.java --- @@ -141,23 +141,46 @@ public int

[GitHub] cloudstack pull request: Cloudstack 8612

2015-07-10 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/562#discussion_r34351953 --- Diff: framework/jobs/src/org/apache/cloudstack/framework/jobs/dao/SyncQueueItemDaoImpl.java --- @@ -141,23 +141,46 @@ public int

[GitHub] cloudstack pull request: CLOUDSTACK-8620 :skip test cases if rbd s...

2015-07-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/570 --- 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

Build failed in Jenkins: simulator-singlerun #1356

2015-07-10 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/1356/ -- Started by upstream project build-master-simulator build number 2173 originally caused by: Started by an SCM change Started by upstream project build-master build number 2460 originally

Jenkins build is back to normal : build-4.5 #497

2015-07-10 Thread jenkins
See http://jenkins.buildacloud.org/job/build-4.5/497/changes

[GitHub] cloudstack pull request: CLOUDSTACK-8620 :skip test cases if rbd s...

2015-07-10 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/570#issuecomment-120408068 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

[GitHub] cloudstack pull request: CLOUDSTACK-8622: Reinstate working sessio...

2015-07-10 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/574#issuecomment-120396037 Thanks for the reviews, merged on 4.5/master now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

Build failed in Jenkins: simulator-4.5-singlerun #252

2015-07-10 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-4.5-singlerun/252/ -- Started by upstream project build-4.5-simulator build number 307 originally caused by: Started by upstream project build-4.5 build number 497 originally caused by: Started by an SCM

[GitHub] cloudstack pull request: Improved message for peeples to actually ...

2015-07-10 Thread kishankavala
Github user kishankavala commented on the pull request: https://github.com/apache/cloudstack/pull/576#issuecomment-120400649 Can we rephrase it as Expected 1 VDI template but found... --- 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-8622: Reinstate working sessio...

2015-07-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/574 --- 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: Reinstate working sessions in browser

2015-07-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/308 --- 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 8612

2015-07-10 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/562#discussion_r34352466 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java --- @@ -5035,9 +5039,16 @@ public

[GitHub] cloudstack pull request: Cloudstack 8612

2015-07-10 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/562#issuecomment-120406704 Can you add unit- and/or integration tests? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

Is ACS will Support Docker

2015-07-10 Thread Keerthiraja SJ
Hi All, Can anyone say will the ACS will support docker / RancherOS / CoreOS so that we can create a container via cloudstack and assign same as now public IP to the container. Thanks, Keerthi

[CLOUDSTACK SV MEETUP] Call for Speakers - August 4th

2015-07-10 Thread Karen Vuong
Hi all, Is anyone going to be traveling to the Silicon Valley on August 4th? The CloudStack Silicon Valley User Group is planning to hold a meetup that evening. Currently, we have David Nalley on the agenda and if anyone else is interested in speaking at the meetup - please let me, John Kinsella

[GitHub] cloudstack pull request: CLOUDSTACK-8379:add support to marvin to ...

2015-07-10 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/158#discussion_r34343974 --- Diff: tools/marvin/marvin/deployDataCenter.py --- @@ -790,8 +790,13 @@ def createZones(self, zones):

[GitHub] cloudstack pull request: CLOUDSTACK-8379:add support to marvin to ...

2015-07-10 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/158#issuecomment-120371884 I don't see that the travis failures are related to this code but the change is not so significant that push through makes sense. Would be nice to have quick

[GitHub] cloudstack pull request: CLOUDSTACK-8457: Add option to authorize ...

2015-07-10 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/575#issuecomment-120391833 on Chrome, dev tools if you set g_idpList to null, you should see the saml stuff in the form. --- If your project is set up for it, you can reply to this email

Build failed in Jenkins: package-rhel63-4.5 #292

2015-07-10 Thread jenkins
See http://jenkins.buildacloud.org/job/package-rhel63-4.5/292/ -- Started by upstream project build-4.5 build number 497 originally caused by: Started by an SCM change [EnvInject] - Loading node environment variables. Building remotely on rpmbuilder-2 in

Re: [GitHub] cloudstack pull request: dhcp/dns offload feature config dirve cha...

2015-07-10 Thread Daan Hoogland
my bad, I would like to see $LGTM (as in on the start of a line), though. This looks like a comment at first. On Fri, Jul 10, 2015 at 3:42 PM, sateesh-chodapuneedi g...@git.apache.org wrote: Github user sateesh-chodapuneedi commented on the pull request:

[GitHub] cloudstack pull request: dhcp/dns offload feature config dirve cha...

2015-07-10 Thread sateesh-chodapuneedi
Github user sateesh-chodapuneedi commented on the pull request: https://github.com/apache/cloudstack/pull/568#issuecomment-120414615 @DaanHoogland I think there are 2 LGTM on this one - from @sanju1010 @kishankavala sanju1010 added a note a day ago Need to

Build failed in Jenkins: simulator-singlerun #1357

2015-07-10 Thread jenkins
See http://jenkins.buildacloud.org/job/simulator-singlerun/1357/ -- Started by upstream project build-master-simulator build number 2174 originally caused by: Started by an SCM change Started by upstream project build-master build number 2461 originally

Re: [GitHub] cloudstack pull request: dhcp/dns offload feature config dirve cha...

2015-07-10 Thread Daan Hoogland
I don't see two LGTM on this one. What happened? On Fri, Jul 10, 2015 at 6:20 AM, asfgit g...@git.apache.org wrote: Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/568 --- If your project is set up for it, you can reply to this email and have

Re: Custom dictionary implementation

2015-07-10 Thread Rohit Yadav
On 01-Jul-2015, at 1:52 pm, Ramzi Youssefi ryouss...@gmail.commailto:ryouss...@gmail.com wrote: I'm implementing a UI Plugin and I would like to have a custom dictionary for the labels. Could you guys please provide an example on how to do this? I need three different languages and I would like

Jenkin automated job to create release package

2015-07-10 Thread Pierre-Luc Dion
Hi, I've create a collection of jenkins jobs [1] allowing to create packages of a release and publish them on cloudstack.apt-get.eu so current community yum and apt repos get updated. I've also add pgp signature[2] for RPM's and generate MD5sum. The idea is to make the packaging generation super

Re: Jenkin automated job to create release package

2015-07-10 Thread Daan Hoogland
great work Pierre-Luc, no we are working on 4.5.2, nothing yet On Fri, Jul 10, 2015 at 8:17 PM, Pierre-Luc Dion pdion...@apache.org wrote: Hi, I've create a collection of jenkins jobs [1] allowing to create packages of a release and publish them on cloudstack.apt-get.eu so current community

[GitHub] cloudstack pull request: CLOUDSTACK-8623: CPVM fails to start afte...

2015-07-10 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/571#issuecomment-120476327 I would like to see tests with such a change as well --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

Re: [DISCUSS] Release principles for Apache CloudStack

2015-07-10 Thread Rohit Yadav
While I like the ideas generally [1], some concerns and observations that I wish could be considered; - active contributor crunch: we don’t have large number of active people working towards testing, fixing bugs and release, and reviewing/merging PRs on *master*; this affects the agility of

Re: CS 4.3.2 RPM builds have dependencies on i686 glibc?! while 4.3.1 do not.

2015-07-10 Thread Rohit Yadav
France, Sorry I missed this email. Was your issue solved? The ACS repo you’re using is same as upstream 4.3.2 release, including the required. dependencies. On 28-Jun-2015, at 7:13 pm, France mailingli...@isg.simailto:mailingli...@isg.si wrote: Hi guys, we are upgrading CS 4.3.1 to 4.3.2

[GitHub] cloudstack pull request: Findbugs issues with instance acces to st...

2015-07-10 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/573#issuecomment-120456014 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

[GitHub] cloudstack pull request: Improved message for peeples to actually ...

2015-07-10 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/576#issuecomment-120455357 LGTM. @snuf see if you can improve the statement as Kishan suggests --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: Coverity resource leaks

2015-07-10 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/578#issuecomment-120455619 LGTM (squash maybe) --- 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-8623: CPVM fails to start afte...

2015-07-10 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/571#issuecomment-120456206 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

[GitHub] cloudstack pull request: findbugs: deal with all the encoding issu...

2015-07-10 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/467#issuecomment-120495209 @DaanHoogland was this committed to master? --- 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: [4.5] Issue logging in under User account

2015-07-10 Thread Rohit Yadav
Try with latest 4.5? The API you listed is called when you open the UI, it checks for certain capabilities to initialize some global javascript variable (those with g_ prefix in cloudStack.js etc) and checks if the user is logged in. On 09-Jul-2015, at 10:35 am, Mike Tutkowski

Build failed in Jenkins: package-rhel63-4.5 #293

2015-07-10 Thread jenkins
See http://jenkins.buildacloud.org/job/package-rhel63-4.5/293/ -- Started by user Pierre-Luc Dion [EnvInject] - Loading node environment variables. Building remotely on rpmbuilder-2 in workspace http://jenkins.buildacloud.org/job/package-rhel63-4.5/ws/

Re: noredist on build.apache.o

2015-07-10 Thread Rohit Yadav
On 16-Jun-2015, at 2:12 pm, Daan Hoogland daan.hoogl...@gmail.commailto:daan.hoogl...@gmail.com wrote: Is there any objections from the foundation to put noredist files on builds.apache.orghttp://builds.apache.org for a noredist build? I would like the pullrequest builder to report on all

Re: Can we use components based on MIT License?

2015-07-10 Thread Rohit Yadav
On 16-Jun-2015, at 5:23 am, ilya ilya.mailing.li...@gmail.commailto:ilya.mailing.li...@gmail.com wrote: General question, can we add/use components based on MIT License? cc Sebastien/David - just to be sure I think so, we’re already using jquery. See NOTICE file for details. Regards, Rohit

Re: Support OVA files with multiple disks for templates: CLOUDSTACK-4757

2015-07-10 Thread Rohit Yadav
Likitha, any update on merging this branch? On 16-Jun-2015, at 3:37 am, ilya ilya.mailing.li...@gmail.commailto:ilya.mailing.li...@gmail.com wrote: Hi Likitha, Hope your day is well, Just a friendly follow up to see if you might have a chance in merging this branch in. Thanks ilya On

Re: libvirt exception

2015-07-10 Thread Rohit Yadav
Wei - is this issue fixed on 4.5/master? On 11-May-2015, at 12:32 pm, Wei ZHOU ustcweiz...@gmail.commailto:ustcweiz...@gmail.com wrote: Hi Mike, Please apply the following patch to cloudstack, and copy the file agent/bindir/libvirtqemuhook.in to repace /etc/libvirt/hooks/qemu on your hosts

Re: 4.5.2 open issues remaining

2015-07-10 Thread Rohit Yadav
Thanks for the effort Daan. Though, by removing the fix version it will be harder to search issues that need to be fixed on 4.5.2 etc. On 09-Jul-2015, at 3:50 pm, Daan Hoogland daan.hoogl...@gmail.commailto:daan.hoogl...@gmail.com wrote: H, Can everybody look at [1] and look if further

Re: [4.5] Issue logging in under User account

2015-07-10 Thread Mike Tutkowski
I can try to update again. I noticed this about two days ago when I had completed a rebase off of the newest at that time. Thanks On Fri, Jul 10, 2015 at 1:27 PM, Rohit Yadav rohit.ya...@shapeblue.com wrote: Try with latest 4.5? The API you listed is called when you open the UI, it checks

Re: [DISCUSS] Moving to Java 8

2015-07-10 Thread Rohit Yadav
Ping Wilder - any progress/plan on moving forward (perhaps after 4.6?). On 01-May-2015, at 4:07 pm, Wilder Rodrigues wrodrig...@schubergphilis.commailto:wrodrig...@schubergphilis.com wrote: Hi Marcus, Sorry for the push, I think after doing the whole CitrixResourceBase refactor I also got a

[GitHub] cloudstack pull request: Coverity resource leaks

2015-07-10 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/578 Coverity resource leaks You can merge this pull request into a Git repository by running: $ git pull https://github.com/DaanHoogland/cloudstack coverity-resource-leaks

Re: Jenkin automated job to create release package

2015-07-10 Thread Wido den Hollander
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/10/2015 08:17 PM, Pierre-Luc Dion wrote: Hi, I've create a collection of jenkins jobs [1] allowing to create packages of a release and publish them on cloudstack.apt-get.eu so current community yum and apt repos get updated. I've also

Re: [DISCUSS] Moving to Java 8

2015-07-10 Thread Wido den Hollander
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/10/2015 09:22 PM, Rohit Yadav wrote: Ping Wilder - any progress/plan on moving forward (perhaps after 4.6?). I don't think there is? Since Ubuntu 14.04 doesn't support Java 8 in any package form we can't really continue. Ubuntu 16.04

[GitHub] cloudstack pull request: findbugs: deal with all the encoding issu...

2015-07-10 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/467#issuecomment-120524298 yes this is on master. is there doubt? --- 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: 4.5.2 open issues remaining

2015-07-10 Thread Daan Hoogland
I have only moved those that had been moved several times and had no activity in between. On Fri, Jul 10, 2015 at 9:27 PM, Rohit Yadav rohit.ya...@shapeblue.com wrote: Thanks for the effort Daan. Though, by removing the fix version it will be harder to search issues that need to be fixed on

Re: Can we use components based on MIT License?

2015-07-10 Thread Daan Hoogland
The link in Rafael's answer contains a list of allowed licenses. MIT is in the list. On Fri, Jul 10, 2015 at 9:07 PM, Rohit Yadav rohit.ya...@shapeblue.com wrote: On 16-Jun-2015, at 5:23 am, ilya ilya.mailing.li...@gmail.com wrote: General question, can we add/use components based on MIT

[GitHub] cloudstack pull request: findbugs: deal with all the encoding issu...

2015-07-10 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/467#issuecomment-120541217 @DaanHoogland some weird confusion, nevermind --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If