[GitHub] cloudstack pull request: findbugs: impossible cast(s) fixed by cha...

2015-06-29 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/538#issuecomment-117022539 Changes looks good. --- 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 th

[GitHub] cloudstack pull request: CLOUDSTACK-8415. SSVM shutdown during sna...

2015-06-29 Thread likitha
GitHub user likitha opened a pull request: https://github.com/apache/cloudstack/pull/540 CLOUDSTACK-8415. SSVM shutdown during snapshot operation leaves behin… …d partial unusable disks and VM snapshots in secondary storage. Enhance storage garbage collector to delete co

Re: Windows Template Startup time

2015-06-29 Thread Francois Gaudreault
Windows instances are usually having a much bigger size than the typical Linux VM. It's normal to take longer. Even with 10GB network, it can take up to 5 minutes depending of the load on our SS. FG On 2015-06-29 10:28 AM, Timothy Lothering wrote: Hi Prashant, There are a couple of questions

Build failed in Jenkins: build-4.5 #492

2015-06-29 Thread jenkins
See Changes: [Rohit Yadav] engine/storage/snapshot: Fix NPE when parent could be null [Rohit Yadav] XenserverSnapshotStrategy: check if primary is null before proceeding in backupSnapshot -- [...

Build failed in Jenkins: simulator-singlerun #1326

2015-06-29 Thread jenkins
See -- Started by upstream project "build-master-simulator" build number 2143 originally caused by: Started by an SCM change Started by upstream project "build-master" build number 2438 origin

[GitHub] cloudstack pull request: CLOUDSTACK-8566: Filter sensitive details...

2015-06-29 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/536#issuecomment-116744035 @serverchief IMO ideally we should not be returning username/passwords. One can use config management systems to update their host passwords and use updateHost API

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

2015-06-29 Thread jenkins
See Changes: [Rohit Yadav] saml2: Use UTF-8 based charset for encoding bytes to strings -- [...truncated 493 lines...] at org.junit.runners.ParentRunner$3.run(ParentRunner.java:238

[GitHub] cloudstack pull request: CLOUDSTACK-8566: Filter sensitive details...

2015-06-29 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/536#discussion_r33479990 --- Diff: api/src/org/apache/cloudstack/api/command/admin/host/ListHostsCmd.java --- @@ -203,9 +201,22 @@ public void execute() { ho

Re: Build failed in Jenkins: build-master-slowbuild #1689

2015-06-29 Thread Rohit Yadav
Hi Daan, > On 29-Jun-2015, at 4:45 pm, Daan Hoogland wrote: > > Rohit, I guess this was your saml merge, Can you fix (parameterizing > getBytes() with either StringUtils.getPreferredEncoding() or > Charset.defaultCharset(), depending on what you considder applicable)? Yes, look like findbugs det

Re: [RFC] SAML2 plugin improvements

2015-06-29 Thread Rohit Yadav
Hi, I’ve updated the docs: http://docs.cloudstack.apache.org/projects/cloudstack-administration/en/4.5/accounts.html#using-a-saml-2-0-identity-provider-for-user-authentication You can add a metadata XML with multiple IdPs, you can now add a metadata xml file in /etc/cloudstack/management (in the

Re: Build failed in Jenkins: build-master-slowbuild #1689

2015-06-29 Thread Daan Hoogland
Rohit, I guess this was your saml merge, Can you fix (parameterizing getBytes() with either StringUtils.getPreferredEncoding() or Charset.defaultCharset(), depending on what you considder applicable)? About the upgrade; I would wait untill 4.5.2 is out. We don't want stale upgrades in the codebase

RE: Windows Template Startup time

2015-06-29 Thread Timothy Lothering
Hi Prashant, There are a couple of questions here: 1. Has your hardware been configured for optimal I/O throughput? 2. Have you installed XenTools on the VM? Slow VM Template Deployment: Initial Template download from SS is dependent on the SSVM, CPU, RAM and Network speed. Check that all of t

[GitHub] cloudstack pull request: findbugs: write to static from instance

2015-06-29 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/539 findbugs: write to static from instance these are the trivial cases of ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD as reported by findbugs You can merge this pull request into a Git repo

Build failed in Jenkins: build-master-slowbuild #1689

2015-06-29 Thread jenkins
See Changes: [Rohit Yadav] CLOUDSTACK-8457: SAML auth plugin improvements for production usage [Rohit Yadav] database: fix upgrade paths from 4.5.2 to 4.6.0 -- [...truncated 27392 l

Re: Windows Template Startup time

2015-06-29 Thread Andrija Panic
start/reboot from inside Windows - or are you refering to reboot/start as seen from ACS UI ? On 29 June 2015 at 16:06, prashant priyam wrote: > Hi, > I have create a windows server 2012 R2 64 bit template with 8 vCPU and > 16Gbs RAM and 350Gbs as Root Disk and created template on cloudstack 4.3

Windows Template Startup time

2015-06-29 Thread prashant priyam
Hi, I have create a windows server 2012 R2 64 bit template with 8 vCPU and 16Gbs RAM and 350Gbs as Root Disk and created template on cloudstack 4.3 and xen server 6.2 . Now when I porvision vm with this template it takes around 15 minutes to start the VM and after creation it again takes around

[GitHub] cloudstack pull request: findbugs: impossible cast(s) fixed by cha...

2015-06-29 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/538 findbugs: impossible cast(s) fixed by changing field types By model of Laszlo's earlier changes, this one was bothering me as the model was a litle more complex at first sight. Turns out it

New Defects reported by Coverity Scan for cloudstack

2015-06-29 Thread scan-admin
Hi, Please find the latest report on new defect(s) introduced to cloudstack found with Coverity Scan. 2 new defect(s) introduced to cloudstack found with Coverity Scan. 9 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan. New defect(

[GitHub] cloudstack-cloudmonkey pull request: Dockerfile

2015-06-29 Thread runseb
Github user runseb commented on a diff in the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/7#discussion_r33458814 --- Diff: Dockerfile --- @@ -0,0 +1,41 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agr

[GitHub] cloudstack-cloudmonkey pull request: Dockerfile

2015-06-29 Thread runseb
Github user runseb commented on a diff in the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/7#discussion_r33458732 --- Diff: README.md --- @@ -7,6 +7,33 @@ with Apache CloudStack 4.0-incubating and above. ![version badge](https://badge.fury.io/py/clo

[GitHub] cloudstack-cloudmonkey pull request: Dockerfile

2015-06-29 Thread runseb
Github user runseb commented on a diff in the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/7#discussion_r33458727 --- Diff: README.md --- @@ -7,6 +7,33 @@ with Apache CloudStack 4.0-incubating and above. ![version badge](https://badge.fury.io/py/clo

Re: [RFC] SAML2 plugin improvements

2015-06-29 Thread Erik Weber
Been testing the lastest SAML work, and it looks good. - Fetching metadata now works - Setting a different default sig alg works Two things; - Is it possible to give IdPs a friendly name? - How do you add more than one? -- Erik On Wed, Jun 3, 2015 at 8:55 PM, Erik Weber wrote: > On Wed, Jun

Build failed in Jenkins: simulator-singlerun #1325

2015-06-29 Thread jenkins
See -- Started by upstream project "build-master-simulator" build number 2142 originally caused by: Started by upstream project "build-master" build number 2437 originally caused by: Started

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

2015-06-29 Thread jenkins
See -- Started by upstream project "build-4.5" build number 491 originally caused by: Started by an SCM change [EnvInject] - Loading node environment variables. Building remotely on rpmbuilder-2 i

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

2015-06-29 Thread jenkins
See -- Started by upstream project "build-4.5-simulator" build number 304 originally caused by: Started by upstream project "build-4.5" build number 491 originally caused by: Started by an

Build failed in Jenkins: simulator-singlerun #1324

2015-06-29 Thread jenkins
See -- Started by upstream project "build-master-simulator" build number 2141 originally caused by: Started by an SCM change Started by an SCM change [EnvInject] - Loading node environment vari

[GitHub] cloudstack-cloudmonkey pull request: Dockerfile

2015-06-29 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/7#issuecomment-116615323 Please note when we change cloudmonkey version number we have to update Dockerfile and docker hub builds... --- If your project is set up for it, you c

[GitHub] cloudstack-cloudmonkey pull request: Dockerfile

2015-06-29 Thread pdion891
GitHub user pdion891 opened a pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/7 Dockerfile build docker container for cloudmonkey, support automated build. * add Dockerfile * add default config file (config.docker) * update readme.md You can me

RE: [GitHub] cloudstack pull request: As an operator I want to be able to chang...

2015-06-29 Thread Stephen Turner
I agree. One enormous change isn't useful when you try and understand the changes later. Github lets you view either the diff for the pull request as a whole or for individual commits, so there should be no reason to squash them. -- Stephen Turner -Original Message- From: Daan Hooglan

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

2015-06-29 Thread France
Hi Rohit Yadav, I am sending this mail personally to you, while CCin mailing lists with hope someone will address this. I see that this package was build on your internal cloud network (cs4angusnet.local) VM named SBPatch-RPM-Builder and it seems like you actually can get fixes for my bug repor

[GitHub] cloudstack pull request: CLOUDSTACK-8457: SAML auth plugin improve...

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

Re: http://cloudstack.apt-get.eu/rhel/ seems down for a few hours now

2015-06-29 Thread France
Tnx Wildo, I saw it came back, after we already upgraded to Shapeblues version. Found that mirror also. :-) On 29 Jun 2015, at 08:25, Wido den Hollander wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > > > On 06/28/2015 08:38 PM, Nux! wrote: >> France, >> >> Works now, at least f

Re: [GitHub] cloudstack pull request: As an operator I want to be able to chang...

2015-06-29 Thread Daan Hoogland
Do not squash commits and hide history unless it makes reading it much and obviously clearer. It is only good in my not so humble opinion in case of to and fro commits where a thing has been reverted and reapplied and reverted again and so on. Also do not rebase commits but make merge commits, thi

Re: [PROPOSAL] Commit to master through PR only

2015-06-29 Thread sebgoa
Ok we are on, Starting today, commit to master through PR only. 2 LGTM needed for merge. If Travis fails, we can still merge given a good explanation of why (since travis has issues once in a while). I will keep an eye on commit, at least once a day, and ping the list if I see a commit that wen

Roadmap

2015-06-29 Thread sebgoa
Hi, A note to everyone about the roadmap. We need to get going on this, add description to items already there, link to JIRA and offer to contribute: https://cwiki.apache.org/confluence/display/CLOUDSTACK/Roadmap Please chime in, thanks, -sebastien

[GitHub] cloudstack pull request: As an operator I want to be able to chang...

2015-06-29 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/527#issuecomment-116545842 I created the issue and added to the last commit. Did it last week: https://issues.apache.org/jira/browse/CLOUDSTACK-8589 Perhaps you missed

Build failed in Jenkins: simulator-singlerun #1323

2015-06-29 Thread jenkins
See -- Started by upstream project "build-master-simulator" build number 2140 originally caused by: Started by an SCM change Started by upstream project "build-master" build number 2436 origin

[GitHub] cloudstack pull request: As an operator I want to be able to chang...

2015-06-29 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/527#issuecomment-116535759 I was with @bhaisaab on this, I would have like to see a JIRA issue. We did not call it so yet, but we should consider being in master freeze right now, and only thin

[GitHub] cloudstack pull request: CLOUDSTACK-8582: Skipping unsuitable test...

2015-06-29 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/532#issuecomment-116534604 these tests used to be required_hardware=true, and now it's false. So when are we running these tests ? Are they included in Travis runs, in which case we are loosing

[GitHub] cloudstack pull request: As an operator I want to be able to chang...

2015-06-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/527 --- 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-8582: Skipping unsuitable test...

2015-06-29 Thread gauravaradhye
Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/532#issuecomment-116523709 Need a second LGTM, anyone? --- 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 doe