[GitHub] cloudstack pull request: CLOUDSTACK-9008 - Pass hypervisor snapsho...

2015-10-30 Thread mike-tutkowski
GitHub user mike-tutkowski opened a pull request:

https://github.com/apache/cloudstack/pull/1016

CLOUDSTACK-9008 - Pass hypervisor snapshot reserve field in when crea…

…ting compute and disk offerings

https://issues.apache.org/jira/browse/CLOUDSTACK-9008

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mike-tutkowski/cloudstack hsr_marvin

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1016.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1016


commit c2d4d2972dc7d87e0adc94cb809e27d5e51f95c1
Author: Mike Tutkowski 
Date:   2015-10-31T04:13:56Z

CLOUDSTACK-9008 - Pass hypervisor snapshot reserve field in when creating 
compute and disk offerings




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9004: Add features to HyperVEn...

2015-10-30 Thread jharshman
Github user jharshman commented on the pull request:

https://github.com/apache/cloudstack/pull/1013#issuecomment-152675128
  
My apologies Daan,
I am happy to clarify.

The PR for this Jira ticket (9004), was created as a subtask of 
CLOUDSTACK-8978.  It's intent is to add the ability to set certain hv 
parameters with an end goal in 8978 of implementing hyper-V enlightenment 
features for Windows Server 2008 guests.

The current code for HyperVEnlightenmentFeatureDef only allowed the setting 
of the hv_relaxed bit.  This bit disables a windows sanity check that commonly 
results in a BSOD when the VM is running on a host under heavy load.

The change here adds the ability to set the hv_vapic and hv_spinlocks bits 
as well as retry time.  The hv_vapic bit tries to reduce interrupt overhead in 
guests.  The hv_spinlock bit is used by the guest to notify the hypervisor that 
the calling virtual processor is attempting to access a resource that may be 
held by another virtual processor.  For the host, the retry value for 
hv_spinlock indicates the number of times the virtual processor should attempt 
to access before the spinlock is considered excessive.

Basically these changes are setting the stage for changes I am going to 
submit in CLOUDSTACK-8974.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: Add Debian 8.0 in OS List for Template

2015-10-30 Thread Milamber


Oh! I need to test this asap. Thanks Daan.

On 30/10/2015 11:18, Daan Hoogland wrote:

Milamber, there are api's for this:
http://cloudstack.apache.org/api/apidocs-4.5/root_admin/addGuestOs.html and
http://cloudstack.apache.org/api/apidocs-4.5/root_admin/addGuestOsMapping.html

Or am I missing your point?

On Fri, Oct 30, 2015 at 8:48 AM, Milamber  wrote:


Hello,

There are any special reason to not add Debian 8.0 32/64 bits on the list
of OS in the template form ? like missing support from kvm or xenserver?

Or we just need to add the INSERT into the sql schema like this commit [1]
?

That would be great if the next release 4.6 could have the Debian 8.0 in
their list (and perhaps too: Debian testing)

Thanks

Mialber


[1]
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blobdiff;f=setup/db/db/schema-442to450.sql;h=cd2bf6e5a3756bf3ea499f20f5bc81ab219dbd89;hp=e2615d6cf6541cc7a6b6763421ba8398d593;hb=66afce66b469e9772989ffcef3015bdab9584ffc;hpb=9928d66fdaf71b43412e20b129975c42e0552d6f








Re: New PMC member: Erik Weber

2015-10-30 Thread Milamber


Congrats Erik! Good works!

On 30/10/2015 14:01, Giles Sirett wrote:

All
The PMC is delighted to announce that we have asked Erik weber to join the PMC 
and he has accepted

Please join me in congratulating Erik

Kind Regards
Giles

D: +44 20 3603 0541 | M: +44 796 111 
2055
giles.sir...@shapeblue.com
Find out more about ShapeBlue and our range of CloudStack related services

IaaS Cloud Design & Build
CSForge – rapid IaaS deployment framework
CloudStack Consulting
CloudStack Software 
Engineering
CloudStack Infrastructure 
Support
CloudStack Bootcamp Training Courses

This email and any attachments to it may be confidential and are intended solely 
for the use of the individual to whom it is addressed. Any views or opinions 
expressed are solely those of the author and do not necessarily represent those of 
Shape Blue Ltd or related companies. If you are not the intended recipient of this 
email, you must neither take any action based upon its contents, nor copy or show 
it to anyone. Please contact the sender if you believe you have received this email 
in error. Shape Blue Ltd is a company incorporated in England & Wales. 
ShapeBlue Services India LLP is a company incorporated in India and is operated 
under license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda is a company 
incorporated in Brasil and is operated under license from Shape Blue Ltd. ShapeBlue 
SA Pty Ltd is a company registered by The Republic of South Africa and is traded 
under license from Shape Blue Ltd. ShapeBlue is a registered trademark.





Re: New PMC member: Erik Weber

2015-10-30 Thread Pierre-Luc Dion
Congrat. Erik!
On Oct 30, 2015 17:02, "Wei ZHOU"  wrote:

> Congratulations, Erik !
>
> 2015-10-30 15:01 GMT+01:00 Giles Sirett :
>
> > All
> > The PMC is delighted to announce that we have asked Erik weber to join
> the
> > PMC and he has accepted
> >
> > Please join me in congratulating Erik
> >
> > Kind Regards
> > Giles
> >
> > D: +44 20 3603 0541 | M: +44 796 111 2055
> > 
> > giles.sir...@shapeblue.com
> > Find out more about ShapeBlue and our range of CloudStack related
> services
> >
> > IaaS Cloud Design & Build<
> > http://shapeblue.com/iaas-cloud-design-and-build//>
> > CSForge – rapid IaaS deployment framework
> > CloudStack Consulting
> > CloudStack Software Engineering<
> > http://shapeblue.com/cloudstack-software-engineering/>
> > CloudStack Infrastructure Support<
> > http://shapeblue.com/cloudstack-infrastructure-support/>
> > CloudStack Bootcamp Training Courses<
> > http://shapeblue.com/cloudstack-training/>
> >
> > This email and any attachments to it may be confidential and are intended
> > solely for the use of the individual to whom it is addressed. Any views
> or
> > opinions expressed are solely those of the author and do not necessarily
> > represent those of Shape Blue Ltd or related companies. If you are not
> the
> > intended recipient of this email, you must neither take any action based
> > upon its contents, nor copy or show it to anyone. Please contact the
> sender
> > if you believe you have received this email in error. Shape Blue Ltd is a
> > company incorporated in England & Wales. ShapeBlue Services India LLP is
> a
> > company incorporated in India and is operated under license from Shape
> Blue
> > Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in
> Brasil
> > and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd
> is
> > a company registered by The Republic of South Africa and is traded under
> > license from Shape Blue Ltd. ShapeBlue is a registered trademark.
> >
>


[GitHub] cloudstack pull request: CLOUDSTACK-8964: Ovm3HypervisorGuru answe...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1015#issuecomment-152668125
  
ping @miguelaferreira @wilderrodrigues @karuturi @remibergsma 
@therestoftheworld as @snuf asked I applied what I suggested myself. It is to 
easy an improvement to not do it in my opinion.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9004: Add features to HyperVEn...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1013#issuecomment-152667236
  
I can see that your code does what you say in the description @jharsman but 
I totally lack the background in windows to judge whether this make sense. Can 
you expand on the reason behind this change here or in the jira ticket? Code 
looks good, change I cannot judge.

Also I can see you created this as a subtask but not to what. Can you give 
that context?

If you are already discussing this with someone else in the community 
please ping them here so they can comment and compensate for my ignorance.

Thanks for working on Apache CloudStack!


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9006 - ListTemplates API retur...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1009#issuecomment-152662658
  
code lgtm, but I'm wondering on testing this (in integration sense) I think 
a unit test would by nice as @bhaisaab. In addition to that if we want to 
verify the fix from a user perspective what do we do (without having to add 
1000+ templetes) @rags22489664 ? Can you give a short description?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8964: Ovm3HypervisorGuru wrong...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/975#issuecomment-152660873
  
#1015 made to apply my remarks


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: clean up code from #975

2015-10-30 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request:

https://github.com/apache/cloudstack/pull/1015

clean up code from #975



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/DaanHoogland/cloudstack pr/975

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1015.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1015


commit 66fed462b64c8e3fe9e7ee39c7a1c07ff33108f9
Author: Funs Kessen 
Date:   2015-10-23T13:58:40Z

simple change to prevent failure and keep OVM3 snapshots working

commit 6db4b7b0be28617feb3367e8984f95aabff97556
Author: Funs Kessen 
Date:   2015-10-23T14:50:26Z

Getting volume from snapshot working again... odd

commit c4afb76d8cce15a67b72e5849ff84e5ff5304b5e
Author: Funs Kessen 
Date:   2015-10-23T18:09:06Z

host delegation logging for XenServerGuru.java

commit fc18d1e8b11c82a18854234b0c8c827896a5b78d
Author: Daan Hoogland 
Date:   2015-10-30T08:00:47Z

CLOUDSTACK-8964 side effect isolation
 extract side effect away to emphasize the main commandDelegation
objective




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: New PMC member: Erik Weber

2015-10-30 Thread Wei ZHOU
Congratulations, Erik !

2015-10-30 15:01 GMT+01:00 Giles Sirett :

> All
> The PMC is delighted to announce that we have asked Erik weber to join the
> PMC and he has accepted
>
> Please join me in congratulating Erik
>
> Kind Regards
> Giles
>
> D: +44 20 3603 0541 | M: +44 796 111 2055
> 
> giles.sir...@shapeblue.com
> Find out more about ShapeBlue and our range of CloudStack related services
>
> IaaS Cloud Design & Build<
> http://shapeblue.com/iaas-cloud-design-and-build//>
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Software Engineering<
> http://shapeblue.com/cloudstack-software-engineering/>
> CloudStack Infrastructure Support<
> http://shapeblue.com/cloudstack-infrastructure-support/>
> CloudStack Bootcamp Training Courses<
> http://shapeblue.com/cloudstack-training/>
>
> This email and any attachments to it may be confidential and are intended
> solely for the use of the individual to whom it is addressed. Any views or
> opinions expressed are solely those of the author and do not necessarily
> represent those of Shape Blue Ltd or related companies. If you are not the
> intended recipient of this email, you must neither take any action based
> upon its contents, nor copy or show it to anyone. Please contact the sender
> if you believe you have received this email in error. Shape Blue Ltd is a
> company incorporated in England & Wales. ShapeBlue Services India LLP is a
> company incorporated in India and is operated under license from Shape Blue
> Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil
> and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is
> a company registered by The Republic of South Africa and is traded under
> license from Shape Blue Ltd. ShapeBlue is a registered trademark.
>


Re: New PMC member: Erik Weber

2015-10-30 Thread Erik Weber
Thanks everyone! :-)


-- 
Erik

On Fri, Oct 30, 2015 at 3:32 PM, Wido den Hollander  wrote:

> Congrats! Welcome :)
>
> On 30-10-15 15:01, Giles Sirett wrote:
> > All
> > The PMC is delighted to announce that we have asked Erik weber to join
> the PMC and he has accepted
> >
> > Please join me in congratulating Erik
> >
> > Kind Regards
> > Giles
> >
> > D: +44 20 3603 0541 | M: +44 796 111 2055
> 
> > giles.sir...@shapeblue.com
> > Find out more about ShapeBlue and our range of CloudStack related
> services
> >
> > IaaS Cloud Design & Build<
> http://shapeblue.com/iaas-cloud-design-and-build//>
> > CSForge – rapid IaaS deployment framework
> > CloudStack Consulting
> > CloudStack Software Engineering<
> http://shapeblue.com/cloudstack-software-engineering/>
> > CloudStack Infrastructure Support<
> http://shapeblue.com/cloudstack-infrastructure-support/>
> > CloudStack Bootcamp Training Courses<
> http://shapeblue.com/cloudstack-training/>
> >
> > This email and any attachments to it may be confidential and are
> intended solely for the use of the individual to whom it is addressed. Any
> views or opinions expressed are solely those of the author and do not
> necessarily represent those of Shape Blue Ltd or related companies. If you
> are not the intended recipient of this email, you must neither take any
> action based upon its contents, nor copy or show it to anyone. Please
> contact the sender if you believe you have received this email in error.
> Shape Blue Ltd is a company incorporated in England & Wales. ShapeBlue
> Services India LLP is a company incorporated in India and is operated under
> license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda is a
> company incorporated in Brasil and is operated under license from Shape
> Blue Ltd. ShapeBlue SA Pty Ltd is a company registered by The Republic of
> South Africa and is traded under license from Shape Blue Ltd. ShapeBlue is
> a registered trademark.
> >
>


Re: [ANNOUNCE] CloudStack Certification !!!!

2015-10-30 Thread Erik Weber
My local test center can't find the exam, has anyone else had any issues?
They can see LPI examns, but not LPI-japan ones..


-- 
Erik

On Tue, Oct 27, 2015 at 6:16 PM, Giles Sirett 
wrote:

> All
>
> [cross posting- sorry]
>
> *GET CERTIFICED ON APACHE CLOUDSTACK *
>
> As some of you may of heard, LPI Japan have recently  launched the Apache
> CloudStack certification exam.
>
> Personally, I think it's a massive milestone for ACS to have its own
> professional certification scheme. From experience, many people entering
> our industry tend to get drawn to technologies that they can achieve
> certification in - and cloudstack is now one of those.
>
> The development of this exam was a collaborative effort between LPI Japan
> and the Japanese Cloudstack users group.
>
> So, I would like to do a shameless plug and encourage people here to have
> a go at the certification.  To be frank, my guess is that the exam will
> only evolve and keep updated if enough people are interested in actually
> taking it
>
> So, please have a go at the exam, encourage your friends & colleagues and
> spread the word.
>
> *Raise funds for CloudStack community*
> ShapeBlue have agreed to become a reseller of the exam. We have decided
> that we will do this on a non-profit basis and donate ALL money made back
> into the community.
> So,if you are going to take the exam, please use the registration code on
> our website: for every exam registered, we will donate $50 (1/3 of the $150
> cost of the exam) to be used for the next collaboration conference or to
> support other community activity.
>
> Further details and how to register are here:
> http://www.shapeblue.com/apache-cloudstack-certification-accel/
>
>
>
>
>
> Kind Regards
> Giles
>
> Find out more about ShapeBlue and our range of CloudStack related services
>
> IaaS Cloud Design & Build<
> http://shapeblue.com/iaas-cloud-design-and-build//>
> CSForge - rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Software Engineering<
> http://shapeblue.com/cloudstack-software-engineering/>
> CloudStack Infrastructure Support<
> http://shapeblue.com/cloudstack-infrastructure-support/>
> CloudStack Bootcamp Training Courses<
> http://shapeblue.com/cloudstack-training/>
>
> This email and any attachments to it may be confidential and are intended
> solely for the use of the individual to whom it is addressed. Any views or
> opinions expressed are solely those of the author and do not necessarily
> represent those of Shape Blue Ltd or related companies. If you are not the
> intended recipient of this email, you must neither take any action based
> upon its contents, nor copy or show it to anyone. Please contact the sender
> if you believe you have received this email in error. Shape Blue Ltd is a
> company incorporated in England & Wales. ShapeBlue Services India LLP is a
> company incorporated in India and is operated under license from Shape Blue
> Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil
> and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is
> a company registered by The Republic of South Africa and is traded under
> license from Shape Blue Ltd. ShapeBlue is a registered trademark.
>


[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/961#issuecomment-152631348
  
@K0zka FYI: it also didn't work in tomcat 7


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9013: Virtual router failed to...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1014#issuecomment-152624797
  
Ping @wilderrodrigues to have a look


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9013: Virtual router failed to...

2015-10-30 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request:

https://github.com/apache/cloudstack/pull/1014

CLOUDSTACK-9013: Virtual router failed to start on KVM

This fix a typo of commit 4a177031b055f3649e3b4a00c80eddb5cafa1dd7

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ustcweizhou/cloudstack CLOUDSTACK-9013

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1014.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1014


commit 9f7f42330aeb33ad819075586529e88db4d5c90a
Author: Wei Zhou 
Date:   2015-10-30T16:04:14Z

CLOUDSTACK-9013: Virtual router failed to start on KVM

This fix a typo of commit 4a177031b055f3649e3b4a00c80eddb5cafa1dd7




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9004: Add features to HyperVEn...

2015-10-30 Thread jharshman
GitHub user jharshman opened a pull request:

https://github.com/apache/cloudstack/pull/1013

CLOUDSTACK-9004: Add features to HyperVEnlightenmentFeatureDef

Add function to set vapic, spinlock and retries
Add function to get retry value
Modify toString to output appropriate XML for spinlock value if set

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jharshman/cloudstack CLOUDSTACK-9004

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1013.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1013


commit a34f269237a2506800854a1e76a37c04caac9071
Author: Josh Harshman 
Date:   2015-10-30T18:14:52Z

CLOUDSTACK-9004: Add features to HyperVEnlightenmentFeatureDef

Add function to set vapic, spinlock and retries
Add function to get retry value
Modify toString to output appropriate XML for spinlock value if set




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: [4.6] VM Snapshots no longer work with managed storage

2015-10-30 Thread Mike Tutkowski
Hi Remi,

Yeah, sorry...I should have provided more details in the ticket. It was
late when I encountered the issue and just wanted to record that we have a
situation where data is being destroyed on accident (which seemed like a
blocker to me, but I'm OK if you'd like to change that status). I went
ahead and updated the ticket and am working it now.

Thanks!
Mike

On Fri, Oct 30, 2015 at 9:43 AM, Remi Bergsma 
wrote:

> Hi Mike,
>
> Thanks for reporting this issue.
>
> Can you provide some more details please? The Jira issue has none. When
> you put a blocker, we need to have some information so we can understand
> what is going on and help with debugging / fixing. Even better would be to
> discuss this. Let’s focus on resolving this.
>
> On the comments Wei notices this may be due to the Guru change. You may
> want to contact him.
>
> Keep me posted over the weekend, please.
>
> Regards,
> Remi
>
>
>
>
>
> On 30/10/15 06:30, "Mike Tutkowski"  wrote:
>
> >Hi Remi,
> >
> >I have opened the following ticket against 4.6:
> >
> >https://issues.apache.org/jira/browse/CLOUDSTACK-9008
> >
> >I found this issue tonight while running regression tests against 4.6.
> >
> >As the ticket indicates, when using managed storage for your root disk,
> >attempting to revert a VM to a VM snapshot corrupts the state of the
> >virtual disk.
> >
> >I am not sure when this regression happened. At present, I kick these
> >regression tests off manually and noticed this problem just now.
> >
> >I marked it as a blocker because it leads to data corruption.
> >
> >I plan to look into it tomorrow.
> >
> >Thanks,
> >
> >--
> >*Mike Tutkowski*
> >*Senior CloudStack Developer, SolidFire Inc.*
> >e: mike.tutkow...@solidfire.com
> >o: 303.746.7302
> >Advancing the way the world uses the cloud
> >*™*
>



-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the cloud
*™*


[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-30 Thread K0zka
Github user K0zka commented on the pull request:

https://github.com/apache/cloudstack/pull/961#issuecomment-152597661
  
I do not know if a new web UI is needed, but tomcat 6 is a walking dead.

@miguelaferreira yes correct, the session can not be created once the 
headers are out on the pipe. I have only tested with jetty, that's the 
problem...

I try to find some time to experiment with tomcat 6.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


RE: Build fails on LocalTemplateDownloadTest

2015-10-30 Thread Josh Harshman
Oh wow, I feel dumb for not noticing that...
Builds fine now.  Thanks :)
Will submit a PR shortly.

From: Wido den Hollander 
Sent: Friday, October 30, 2015 1:01 AM
To: dev@cloudstack.apache.org
Subject: Re: Build fails on LocalTemplateDownloadTest



Josh Harshman
Cloud Engineer
Security+
Intrinium

On 29-10-15 15:36, Josh Harshman wrote:
> Build environment isn't Windows, it's Ubuntu 14.04 that I am getting this 
> build failure on.
>

Is there something special about the /tmp on your system? Eg not enough
space for example.

Wido

> 
> From: Wido den Hollander 
> Sent: Thursday, October 29, 2015 1:50 AM
> To: dev@cloudstack.apache.org
> Subject: Re: Build fails on LocalTemplateDownloadTest
>
>
>
> Josh Harshman
> Cloud Engineer
> Security+
> Intrinium
>
> On 29-10-15 00:34, Josh Harshman wrote:
>> Is anyone else experiencing the same behavior?
>>
>> Building off master, get the following build failure:
>>
>
> I think you are building on Windows?
>
> I send a PR for this: https://github.com/apache/cloudstack/pull/884
>
> This was reverted, but due to the code, but I merged it without the
> proper permissions.
>
> We would have to merge this one in to fix it.
>
> Wido
>
>>
>>
>> ---snip---
>>
>> [INFO] Apache CloudStack Framework - IPC . SUCCESS [5.250s]
>> [INFO] Apache CloudStack Cloud Engine  SUCCESS [0.042s]
>> [INFO] Apache CloudStack Cloud Engine API  SUCCESS [5.073s]
>> [INFO] Apache CloudStack Framework - Security  SUCCESS [2.782s]
>> [INFO] Apache CloudStack Core  FAILURE [15.655s]
>> [INFO] Apache CloudStack Agents .. SKIPPED
>> [INFO] Apache CloudStack Framework - Clustering .. SKIPPED
>> ---snip---
>>
>>
>> com.cloud.storage.template.LocalTemplateDownloaderTest.txt:
>>
>> ---
>> Test set: com.cloud.storage.template.LocalTemplateDownloaderTest
>> ---
>> Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.028 sec 
>> <<< FAILURE! - in com.cloud.storage.template.LocalTemplateDownloaderTest
>> localTemplateDownloaderTest(com.cloud.storage.template.LocalTemplateDownloaderTest)
>>   Time elapsed: 0.026 sec  <<< FAILURE!
>> java.lang.AssertionError: Failed download
>> at org.junit.Assert.fail(Assert.java:88)
>> at 
>> com.cloud.storage.template.LocalTemplateDownloaderTest.localTemplateDownloaderTest(LocalTemplateDownloaderTest.java:37)
>>
>>
>>
>>
>> Josh Harshman
>>
>> Cloud Engineer
>> Security+
>>
>>
>> Intrinium
>> Tel: (509) 465-1234 x5259
>> Fax: (866) 565-4578
>> Lync / Skype: josh.harsh...@intrinium.com
>> Web: http://intrinium.com
>>
>>
>> [Intrinium Long Sig Logo]
>>
>>
>> [Facebook] 
>> [Twitter]   [Linkedin] 
>> 
>>   [Youtube]   [Blog] 
>> 
>>
>> Information Security and Compliance Consulting | Managed IT and Security 
>> Services | Cloud Services
>>
>>
>>
>>
>>
>> 
>>
>> This email and any files transmitted with it are confidential and intended 
>> solely for the use of the individual or entity to whom they are addressed. 
>> If you have received this email in error please notify the system manager. 
>> This message contains confidential information and is intended only for the 
>> individual named. If you are not the named addressee you should not 
>> disseminate, distribute or copy this e-mail. Please notify the sender 
>> immediately by e-mail if you have received this e-mail by mistake and delete 
>> this e-mail from your system. If you are not the intended recipient you are 
>> notified that disclosing, copying, distributing or taking any action in 
>> reliance on the contents of this information is strictly prohibited.
>>


Re: test_vpc_vpn.py failing 2 BVTs on all Advnetworking config

2015-10-30 Thread Raja Pullela
Super, thanks Remi!

> On Oct 30, 2015, at 10:35 PM, Remi Bergsma  
> wrote:
> 
> Hi Raja,
> 
> Talked to Michael and he will improve the test soon.
> 
> Regards,
> Remi
> 
> 
> 
> 
> 
>> On 30/10/15 17:44, "Raja Pullela"  wrote:
>> 
>> Hi Michael
>> 
>> Can you please add a wait for the template  to download before the VM 
>> creation?
>> 
>> Raja
>> Begin forwarded message:
>> From: Raja Pullela mailto:raja.pull...@citrix.com>>
>> Date: October 29, 2015 at 11:49:59 AM GMT+5:30
>> To: "CloudStack Dev 
>> (dev@cloudstack.apache.org)" 
>> mailto:dev@cloudstack.apache.org>>
>> Subject: Few BVTs failing
>> Reply-To: mailto:dev@cloudstack.apache.org>>
>> 
>> Hi,
>> 
>> In the latest BVTs, following tests are failing consistently -
>> 
>> * 2015-10-29 00:27:43,987 - CRITICAL - FAILED: 
>> test_vpc_remote_access_vpn: ['Traceback (most recent call last):\n', '  File 
>> "/usr/lib/python2.7/unittest/case.py", line 329, in run\n
>> testMethod()\n', '  File 
>> "/root/cloudstack/test/integration/smoke/test_vpc_vpn.py", line 333, in 
>> test_vpc_remote_access_vpn\nself.fail(e)\n', '  File 
>> "/usr/lib/python2.7/unittest/case.py", line 410, in fail\nraise 
>> self.failureException(msg)\n', 'AssertionError: Execute cmd: 
>> deployvirtualmachine failed, due to: errorCode: 530, errorText:Template 202 
>> has not been completely downloaded to zone 1\n']
>> 
>> * 2015-10-29 00:28:02,194 - CRITICAL - EXCEPTION: 
>> test_vpc_site2site_vpn: ['Traceback (most recent call last):\n', '  File 
>> "/usr/lib/python2.7/unittest/case.py", line 329, in run\n
>> testMethod()\n', '  File 
>> "/root/cloudstack/test/integration/smoke/test_vpc_vpn.py", line 589, in 
>> test_vpc_site2site_vpn\nself.assert_(vm1 is not None, "VM failed to 
>> deploy")\n', "UnboundLocalError: local variable 'vm1' referenced before 
>> assignment\n"]
>> 
>> * 2015-10-29 02:02:15,167 - CRITICAL - EXCEPTION: 
>> test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80: ['Traceback (most recent 
>> call last):\n', '  File "/usr/lib/python2.7/unittest/case.py", line 329, in 
>> run\ntestMethod()\n', '  File 
>> "/root/cloudstack/test/integration/smoke/test_internal_lb.py", line 576, in 
>> test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80\nclient_vm = 
>> self.deployvm_in_network(vpc, network_guestnet.id)\n', '  File 
>> "/root/cloudstack/test/integration/smoke/test_internal_lb.py", line 382, in 
>> deployvm_in_network\nraise Exception("Deployment failed of VM: %s" % 
>> e)\n', "Exception: Deployment failed of VM: Job failed: {jobprocstatus : 0, 
>> created : u'2015-10-29T02:02:09+', jobresult : {errorcode : 533, 
>> errortext : u'Unable to create a deployment for VM[User|i-36-69-VM]'}, cmd : 
>> u'org.apache.cloudstack.api.command.admin.vm.DeployVMCmdByAdmin', userid : 
>> u'afe432e8-7dd7-11e5-9ca5-b230fb1ebb83', jobstatus : 2, jobid : 
>> u'de5a239a-c70c-4039-9e8e-2ea9b80ed28b', jobresultcode : 530, jobinstanceid 
>> : u'47770620-0c37-4e7c-91e0-666bf7fe5eaa', jobresulttype : u'object', 
>> jobinstancetype : u'VirtualMachine', accountid : 
>> u'afe422b2-7dd7-11e5-9ca5-b230fb1ebb83'}\n"]
>> 
>> * 2015-10-29 02:00:33,135 - CRITICAL - EXCEPTION: 
>> test02_internallb_haproxy_stats_on_all_interfaces: ['Traceback (most recent 
>> call last):\n', '  File "/usr/lib/python2.7/unittest/case.py", line 329, in 
>> run\ntestMethod()\n', '  File 
>> "/root/cloudstack/test/integration/smoke/test_internal_lb.py", line 686, in 
>> test02_internallb_haproxy_stats_on_all_interfaces\nsettings = 
>> self.get_lb_stats_settings()\n', '  File 
>> "/root/cloudstack/test/integration/smoke/test_internal_lb.py", line 646, in 
>> get_lb_stats_settings\nself.fail("Failed to retrieve stats settings " % 
>> e)\n', 'TypeError: not all arguments converted during string formatting\n']
>> 
>> Looks like these were added by Michael.
>> 
>> Michael,
>> Can you optimize the tests with the following
>> 
>> 1)  Before the VM Deployment or right after Template Register, can you 
>> put a loop to check if the Template is in a ready state before continuing 
>> with the test ? The reason is Template download is not getting completed and 
>> as a result the test cases are failing.
>> 
>> 2)  Any reason for not using test_data.py to have all the 
>> configuration/settings you have defined the test file ?
>> 
>> Please let me know if you have any questions,
>> Raja
>> 


Re: test_vpc_vpn.py failing 2 BVTs on all Advnetworking config

2015-10-30 Thread Remi Bergsma
Hi Raja,

Talked to Michael and he will improve the test soon.

Regards,
Remi





On 30/10/15 17:44, "Raja Pullela"  wrote:

>Hi Michael
>
>Can you please add a wait for the template  to download before the VM creation?
>
>Raja
>Begin forwarded message:
>From: Raja Pullela mailto:raja.pull...@citrix.com>>
>Date: October 29, 2015 at 11:49:59 AM GMT+5:30
>To: "CloudStack Dev 
>(dev@cloudstack.apache.org)" 
>mailto:dev@cloudstack.apache.org>>
>Subject: Few BVTs failing
>Reply-To: mailto:dev@cloudstack.apache.org>>
>
>Hi,
>
>In the latest BVTs, following tests are failing consistently -
>
>* 2015-10-29 00:27:43,987 - CRITICAL - FAILED: 
>test_vpc_remote_access_vpn: ['Traceback (most recent call last):\n', '  File 
>"/usr/lib/python2.7/unittest/case.py", line 329, in run\ntestMethod()\n', 
>'  File "/root/cloudstack/test/integration/smoke/test_vpc_vpn.py", line 333, 
>in test_vpc_remote_access_vpn\nself.fail(e)\n', '  File 
>"/usr/lib/python2.7/unittest/case.py", line 410, in fail\nraise 
>self.failureException(msg)\n', 'AssertionError: Execute cmd: 
>deployvirtualmachine failed, due to: errorCode: 530, errorText:Template 202 
>has not been completely downloaded to zone 1\n']
>
>* 2015-10-29 00:28:02,194 - CRITICAL - EXCEPTION: 
>test_vpc_site2site_vpn: ['Traceback (most recent call last):\n', '  File 
>"/usr/lib/python2.7/unittest/case.py", line 329, in run\ntestMethod()\n', 
>'  File "/root/cloudstack/test/integration/smoke/test_vpc_vpn.py", line 589, 
>in test_vpc_site2site_vpn\nself.assert_(vm1 is not None, "VM failed to 
>deploy")\n', "UnboundLocalError: local variable 'vm1' referenced before 
>assignment\n"]
>
>* 2015-10-29 02:02:15,167 - CRITICAL - EXCEPTION: 
>test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80: ['Traceback (most recent 
>call last):\n', '  File "/usr/lib/python2.7/unittest/case.py", line 329, in 
>run\ntestMethod()\n', '  File 
>"/root/cloudstack/test/integration/smoke/test_internal_lb.py", line 576, in 
>test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80\nclient_vm = 
>self.deployvm_in_network(vpc, network_guestnet.id)\n', '  File 
>"/root/cloudstack/test/integration/smoke/test_internal_lb.py", line 382, in 
>deployvm_in_network\nraise Exception("Deployment failed of VM: %s" % 
>e)\n', "Exception: Deployment failed of VM: Job failed: {jobprocstatus : 0, 
>created : u'2015-10-29T02:02:09+', jobresult : {errorcode : 533, errortext 
>: u'Unable to create a deployment for VM[User|i-36-69-VM]'}, cmd : 
>u'org.apache.cloudstack.api.command.admin.vm.DeployVMCmdByAdmin', userid : 
>u'afe432e8-7dd7-11e5-9ca5-b230fb1ebb83', jobstatus : 2, jobid : 
>u'de5a239a-c70c-4039-9e8e-2ea9b80ed28b', jobresultcode : 530, jobinstanceid : 
>u'47770620-0c37-4e7c-91e0-666bf7fe5eaa', jobresulttype : u'object', 
>jobinstancetype : u'VirtualMachine', accountid : 
>u'afe422b2-7dd7-11e5-9ca5-b230fb1ebb83'}\n"]
>
>* 2015-10-29 02:00:33,135 - CRITICAL - EXCEPTION: 
>test02_internallb_haproxy_stats_on_all_interfaces: ['Traceback (most recent 
>call last):\n', '  File "/usr/lib/python2.7/unittest/case.py", line 329, in 
>run\ntestMethod()\n', '  File 
>"/root/cloudstack/test/integration/smoke/test_internal_lb.py", line 686, in 
>test02_internallb_haproxy_stats_on_all_interfaces\nsettings = 
>self.get_lb_stats_settings()\n', '  File 
>"/root/cloudstack/test/integration/smoke/test_internal_lb.py", line 646, in 
>get_lb_stats_settings\nself.fail("Failed to retrieve stats settings " % 
>e)\n', 'TypeError: not all arguments converted during string formatting\n']
>
>Looks like these were added by Michael.
>
>Michael,
>Can you optimize the tests with the following
>
>1)  Before the VM Deployment or right after Template Register, can you put 
>a loop to check if the Template is in a ready state before continuing with the 
>test ? The reason is Template download is not getting completed and as a 
>result the test cases are failing.
>
>2)  Any reason for not using test_data.py to have all the 
>configuration/settings you have defined the test file ?
>
>Please let me know if you have any questions,
>Raja
>


Re: 4.6 status update

2015-10-30 Thread Remi Bergsma
Hi all,

The blocker mentioned below has been fixed. Its PR (1008) is almost ready to be 
merged. In the mean while, Mike reported an issue, that is now our single 
blocker. Waiting for more info on it.

Lots of PRs were merged, thanks to all that tested, reviewed or otherwise 
helped out.

Could you all please pick a PR from the list below and review it?

We’re aiming for a RC early next week.

Thanks and have a great weekend!

Regards,
Remi


PRs for 4.6
===
If you have PRs that should go into 4.6, please ping me or Rajani about it and 
make sure the PR gets reviewed and tested. Looks like we're getting close to RC 
so act now.

These are some PRs that fix bugs we want to consider. Please help testing and 
reviewing:

CLOUDSTACK-8964: Ovm3HypervisorGuru wrong answer and snapshot to volume 
implementation. #975
https://github.com/apache/cloudstack/pull/975

CLOUDSTACk-9002: VM deployment is successful even when dhcp entry com… #995
https://github.com/apache/cloudstack/pull/995

CLOUDSTACK-8940: Wrong value is inserted into nics table netmask fiel… #916
https://github.com/apache/cloudstack/pull/916

Update L10N Web UI resource files before 4.6 RC1 (v2) #1001
https://github.com/apache/cloudstack/pull/1001


UI
===
There is a bunch of rather small UI improvements we should consider. Please 
help testing and reviewing:

CLOUDSTACK-8961: Changes related to the UI of VPN Users management. #941
https://github.com/apache/cloudstack/pull/941

CLOUDSTACK-8976 - Sorting of security groups #937
https://github.com/apache/cloudstack/pull/937

CLOUDSTACK-8928: While adding VMs to LB rule, default NIC IP is always 
displayed rather than the IP corresponding to the NIC where LB is being created 
#903
https://github.com/apache/cloudstack/pull/903

CLOUDSTACK-8919: Slow UI response while loading the list of networks in network 
tab. #897
https://github.com/apache/cloudstack/pull/897





From: Remi Bergsma
Date: Thursday 29 October 2015 09:28
To: "dev@cloudstack.apache.org"
Subject: 4.6 status update

Hi,

We’re down to a single blocker:

1) CentOS 7 - systemd-tmpfiles - Operation not permitted
This is about the CentOS 7 packaging of the Management Server.
https://issues.apache.org/jira/browse/CLOUDSTACK-8812


PRs for 4.6
===
If you have PRs that should go into 4.6, please ping me or Rajani about it and 
make sure the PR gets reviewed and tested. Looks like we're getting close to RC 
so act now.

These are some PRs that fix bugs we want to consider. Please help testing and 
reviewing:

CLOUDSTACK-8793 Enable s2s VPN connection for projects #879
https://github.com/apache/cloudstack/pull/879

CLOUDSTACK-8964: Ovm3HypervisorGuru wrong answer and snapshot to volume 
implementation. #975
https://github.com/apache/cloudstack/pull/975

Rename xapi plugins for s3 and swift to make them work after renaming the calls 
#982
https://github.com/apache/cloudstack/pull/982

CLOUDSTACk-9002: VM deployment is successful even when dhcp entry com… #995
https://github.com/apache/cloudstack/pull/995

CLOUDSTACK-8962: Dedicated cluster is used for virtual routers that belong to 
non-dedicated account #945
https://github.com/apache/cloudstack/pull/945

CLOUDSTACK-8940: Wrong value is inserted into nics table netmask fiel… #916
https://github.com/apache/cloudstack/pull/916

Pass LbProtocol down to the HAProxyConfigurator #848
https://github.com/apache/cloudstack/pull/848

Update L10N Web UI resource files before 4.6 RC1 (v2) #1001
https://github.com/apache/cloudstack/pull/1001


UI
===
There is a bunch of rather small UI improvements we should consider. Please 
help testing and reviewing:


CLOUDSTACK-8929: The list of VMs that can be assigned to a load balancer rule 
is not updated after a VM is assigned to it. #949
https://github.com/apache/cloudstack/pull/949

CLOUDSTACK-8961: Changes related to the UI of VPN Users management. #941
https://github.com/apache/cloudstack/pull/941

CLOUDSTACK-8976 - Sorting of security groups #937
https://github.com/apache/cloudstack/pull/937

CLOUDSTACK-8928: While adding VMs to LB rule, default NIC IP is always 
displayed rather than the IP corresponding to the NIC where LB is being created 
#903
https://github.com/apache/cloudstack/pull/903

CLOUDSTACK-8919: Slow UI response while loading the list of networks in network 
tab. #897
https://github.com/apache/cloudstack/pull/897

CLOUDSTACK-8984: VPC Network offerings tab missing from UI #996
https://github.com/apache/cloudstack/pull/996


Let us all test / review 2 of the above?! More is better ;-)


To the authors of PRs: I know it's a pain, but please force-push again if your 
PR shows Travis / Jenkins failures. The rebasing we can do before testing, so 
let's not waste time waiting for that.

Thanks!

Regards,
The RMs

Rajani / Remi


PS: PR 981 and 998 are ready to be merged once Travis and Jenkins are green.



test_vpc_vpn.py failing 2 BVTs on all Advnetworking config

2015-10-30 Thread Raja Pullela
Hi Michael

Can you please add a wait for the template  to download before the VM creation?

Raja
Begin forwarded message:
From: Raja Pullela mailto:raja.pull...@citrix.com>>
Date: October 29, 2015 at 11:49:59 AM GMT+5:30
To: "CloudStack Dev 
(dev@cloudstack.apache.org)" 
mailto:dev@cloudstack.apache.org>>
Subject: Few BVTs failing
Reply-To: mailto:dev@cloudstack.apache.org>>

Hi,

In the latest BVTs, following tests are failing consistently -

* 2015-10-29 00:27:43,987 - CRITICAL - FAILED: 
test_vpc_remote_access_vpn: ['Traceback (most recent call last):\n', '  File 
"/usr/lib/python2.7/unittest/case.py", line 329, in run\ntestMethod()\n', ' 
 File "/root/cloudstack/test/integration/smoke/test_vpc_vpn.py", line 333, in 
test_vpc_remote_access_vpn\nself.fail(e)\n', '  File 
"/usr/lib/python2.7/unittest/case.py", line 410, in fail\nraise 
self.failureException(msg)\n', 'AssertionError: Execute cmd: 
deployvirtualmachine failed, due to: errorCode: 530, errorText:Template 202 has 
not been completely downloaded to zone 1\n']

* 2015-10-29 00:28:02,194 - CRITICAL - EXCEPTION: 
test_vpc_site2site_vpn: ['Traceback (most recent call last):\n', '  File 
"/usr/lib/python2.7/unittest/case.py", line 329, in run\ntestMethod()\n', ' 
 File "/root/cloudstack/test/integration/smoke/test_vpc_vpn.py", line 589, in 
test_vpc_site2site_vpn\nself.assert_(vm1 is not None, "VM failed to 
deploy")\n', "UnboundLocalError: local variable 'vm1' referenced before 
assignment\n"]

* 2015-10-29 02:02:15,167 - CRITICAL - EXCEPTION: 
test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80: ['Traceback (most recent 
call last):\n', '  File "/usr/lib/python2.7/unittest/case.py", line 329, in 
run\ntestMethod()\n', '  File 
"/root/cloudstack/test/integration/smoke/test_internal_lb.py", line 576, in 
test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80\nclient_vm = 
self.deployvm_in_network(vpc, network_guestnet.id)\n', '  File 
"/root/cloudstack/test/integration/smoke/test_internal_lb.py", line 382, in 
deployvm_in_network\nraise Exception("Deployment failed of VM: %s" % e)\n', 
"Exception: Deployment failed of VM: Job failed: {jobprocstatus : 0, created : 
u'2015-10-29T02:02:09+', jobresult : {errorcode : 533, errortext : u'Unable 
to create a deployment for VM[User|i-36-69-VM]'}, cmd : 
u'org.apache.cloudstack.api.command.admin.vm.DeployVMCmdByAdmin', userid : 
u'afe432e8-7dd7-11e5-9ca5-b230fb1ebb83', jobstatus : 2, jobid : 
u'de5a239a-c70c-4039-9e8e-2ea9b80ed28b', jobresultcode : 530, jobinstanceid : 
u'47770620-0c37-4e7c-91e0-666bf7fe5eaa', jobresulttype : u'object', 
jobinstancetype : u'VirtualMachine', accountid : 
u'afe422b2-7dd7-11e5-9ca5-b230fb1ebb83'}\n"]

* 2015-10-29 02:00:33,135 - CRITICAL - EXCEPTION: 
test02_internallb_haproxy_stats_on_all_interfaces: ['Traceback (most recent 
call last):\n', '  File "/usr/lib/python2.7/unittest/case.py", line 329, in 
run\ntestMethod()\n', '  File 
"/root/cloudstack/test/integration/smoke/test_internal_lb.py", line 686, in 
test02_internallb_haproxy_stats_on_all_interfaces\nsettings = 
self.get_lb_stats_settings()\n', '  File 
"/root/cloudstack/test/integration/smoke/test_internal_lb.py", line 646, in 
get_lb_stats_settings\nself.fail("Failed to retrieve stats settings " % 
e)\n', 'TypeError: not all arguments converted during string formatting\n']

Looks like these were added by Michael.

Michael,
Can you optimize the tests with the following

1)  Before the VM Deployment or right after Template Register, can you put 
a loop to check if the Template is in a ready state before continuing with the 
test ? The reason is Template download is not getting completed and as a result 
the test cases are failing.

2)  Any reason for not using test_data.py to have all the 
configuration/settings you have defined the test file ?

Please let me know if you have any questions,
Raja



[GitHub] cloudstack pull request: CLOUDSTACK-9006 - ListTemplates API retur...

2015-10-30 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1009#discussion_r43523867
  
--- Diff: framework/db/src/com/cloud/utils/db/Filter.java ---
@@ -89,7 +89,7 @@ public void addOrderBy(Class clazz, String field, 
boolean ascending) {
 if (_orderBy == null) {
 _orderBy = order.insert(0, " ORDER BY ").toString();
 } else {
-_orderBy = order.insert(0, _orderBy).toString();
+_orderBy = order.insert(0, _orderBy + ", ").toString();
--- End diff --

@rags22489664 also recommend if this should also go into 4.5 branch?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8940: Wrong value is inserted ...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/916#issuecomment-152581514
  
@wilderrodrigues any update on your review?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9006 - ListTemplates API retur...

2015-10-30 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1009#discussion_r43523817
  
--- Diff: framework/db/src/com/cloud/utils/db/Filter.java ---
@@ -89,7 +89,7 @@ public void addOrderBy(Class clazz, String field, 
boolean ascending) {
 if (_orderBy == null) {
 _orderBy = order.insert(0, " ORDER BY ").toString();
 } else {
-_orderBy = order.insert(0, _orderBy).toString();
+_orderBy = order.insert(0, _orderBy + ", ").toString();
--- End diff --

LGTM, but since this is a core change can you write a small unit test for 
this method? For the list template api, the temp_zone_pair should give a unique 
id (template_id + "_" + zone_id) so sorting on this should give us 
deterministic results.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACk-9002: VM deployment is success...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/995#issuecomment-152581144
  
@wilderrodrigues did you run more 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 your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1008#issuecomment-152578560
  
@davidamorimfaria Final request, please squash your commits. We need them 
to be atomic. Please ping me when done, 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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1008#issuecomment-152573997
  
This PR fixes CentOS 7 packaging and obsoletes #888 and CLOUDSTACK-8812. I 
will merge this and close the other. 


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: New PMC member: Erik Weber

2015-10-30 Thread Nux!
It's a boy, congrats! :)

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Remi Bergsma" 
> To: dev@cloudstack.apache.org
> Cc: "Erik Weber" 
> Sent: Friday, 30 October, 2015 15:48:40
> Subject: Re: New PMC member: Erik Weber

> Congrats Erik!
> 
> 
> 
> 
> On 30/10/15 15:01, "Giles Sirett"  wrote:
> 
>>All
>>The PMC is delighted to announce that we have asked Erik weber to join the PMC
>>and he has accepted
>>
>>Please join me in congratulating Erik
>>
>>Kind Regards
>>Giles
>>
>>D: +44 20 3603 0541 | M: +44 796 111
>>2055
>>giles.sir...@shapeblue.com
>>Find out more about ShapeBlue and our range of CloudStack related services
>>
>>IaaS Cloud Design & Build
>>CSForge – rapid IaaS deployment framework
>>CloudStack Consulting
>>CloudStack Software
>>Engineering
>>CloudStack Infrastructure
>>Support
>>CloudStack Bootcamp Training 
>>Courses
>>
>>This email and any attachments to it may be confidential and are intended 
>>solely
>>for the use of the individual to whom it is addressed. Any views or opinions
>>expressed are solely those of the author and do not necessarily represent 
>>those
>>of Shape Blue Ltd or related companies. If you are not the intended recipient
>>of this email, you must neither take any action based upon its contents, nor
>>copy or show it to anyone. Please contact the sender if you believe you have
>>received this email in error. Shape Blue Ltd is a company incorporated in
>>England & Wales. ShapeBlue Services India LLP is a company incorporated in
>>India and is operated under license from Shape Blue Ltd. Shape Blue Brasil
>>Consultoria Ltda is a company incorporated in Brasil and is operated under
>>license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company registered by
>>The Republic of South Africa and is traded under license from Shape Blue Ltd.
> >ShapeBlue is a registered trademark.


Re: extending macchinina image

2015-10-30 Thread Nux!
Hello,

Macchinina is based on buildroot.org, the configs I used can be found here:
https://github.com/NuxRo/macchinina/tree/master/configs

You can essentially build your own, the problem with this is that once you 
start adding new stuff it grows and grows. At some point it's not worth the 
trouble; just go for a minimal install of CentOS or Debian instead where you 
can easily add new software.
Let me know if you need further help with this.

Yes, I'm also behind the openvm.eu image.

Lucian

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Michael Andersen" 
> To: dev@cloudstack.apache.org
> Sent: Thursday, 29 October, 2015 19:29:56
> Subject: extending macchinina image

> Hi Nux, et all,
> 
> How would I go about adding a strongswan ipsec client to the macchinina
> template?
> 
> Are you also responsible for the images here?
> http://dl.openvm.eu/cloudstack/macchinina/x86_64/
> 
> 
> Michael
> 
> 
> On 08/05/15 16:12, "David Nalley"  wrote:
> 
>>https://github.com/citrix/foreman-cloudstack
>>
>>--David
>>
>>On Fri, May 8, 2015 at 8:00 AM, Nux!  wrote:
>>> Hello,
>>>
>>> I'm playing with The Foreman these days and promises a lot. Ideally I'd 
>>> like to
>>> use ACS as a Foreman provider (ie Foreman should be able to provision ACS
>>> instances), but I have found nothing about this in my searches.
>>> Is anyone doing anything with Foreman and Cloudstack?
>>>
>>> Lucian
>>>
>>> --
>>> Sent from the Delta quadrant using Borg technology!
>>>
>>> Nux!
> >> www.nux.ro


[GitHub] cloudstack pull request: CLOUDSTACK-9006 - ListTemplates API retur...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1009#issuecomment-152570782
  
LGTM, based on a set of tests that I run on this branch (which I rebased 
myself first):

```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a 
tags=advanced,required_hardware=true \
component/test_vpc_redundant.py \
component/test_routers_iptables_default_policy.py \
component/test_routers_network_ops.py \
component/test_vpc_router_nics.py \
smoke/test_loadbalance.py \
smoke/test_internal_lb.py \
smoke/test_ssvm.py \
smoke/test_network.py

```

Result:

```
Create a redundant VPC with two networks with two VMs in each network ... 
=== TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : 
SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network and 
check default routes ... === TestName: test_02_redundant_VPC_default_routes | 
Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policy on RouterVM ... === TestName: 
test_02_routervm_iptables_policies | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policies on VPC router ... === 
TestName: test_01_single_VPC_iptables_policies | Status : SUCCESS ===
ok
Stop existing router, add a PF rule and check we can access the VM ... === 
TestName: test_isolate_network_FW_PF_default_routes | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: 
test_RVR_Network_FW_PF_SSH_default_routes | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test nics 
after destroy ... === TestName: test_01_VPC_nics_after_destroy | Status : 
SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test default 
routes ... === TestName: test_02_VPC_default_routes | Status : SUCCESS ===
ok
Check the password file in the Router VM ... === TestName: 
test_isolate_network_password_server | Status : SUCCESS ===
ok
Check that the /etc/dhcphosts.txt doesn't contain duplicate IPs ... === 
TestName: test_router_dhcphosts | Status : SUCCESS ===
ok
Test to create Load balancing rule with source NAT ... === TestName: 
test_01_create_lb_rule_src_nat | Status : SUCCESS ===
ok
Test to create Load balancing rule with non source NAT ... === TestName: 
test_02_create_lb_rule_non_nat | Status : SUCCESS ===
ok
Test for assign & removing load balancing rule ... === TestName: 
test_assign_and_removal_lb | Status : SUCCESS ===
ok
Test to verify access to loadbalancer haproxy admin stats page ... === 
TestName: test02_internallb_haproxy_stats_on_all_interfaces | Status : SUCCESS 
===
ok
Test create, assign, remove of an Internal LB with roundrobin http traffic 
to 3 vm's ... === TestName: test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 
| Status : SUCCESS ===
ok
Test SSVM Internals ... === TestName: test_03_ssvm_internals | Status : 
SUCCESS ===
ok
Test CPVM Internals ... === TestName: test_04_cpvm_internals | Status : 
SUCCESS ===
ok
Test stop SSVM ... === TestName: test_05_stop_ssvm | Status : SUCCESS ===
ok
Test stop CPVM ... === TestName: test_06_stop_cpvm | Status : SUCCESS ===
ok
Test reboot SSVM ... === TestName: test_07_reboot_ssvm | Status : SUCCESS 
===
ok
Test reboot CPVM ... === TestName: test_08_reboot_cpvm | Status : SUCCESS 
===
ok
Test destroy SSVM ... === TestName: test_09_destroy_ssvm | Status : SUCCESS 
===
ok
Test destroy CPVM ... === TestName: test_10_destroy_cpvm | Status : SUCCESS 
===
ok
Test for port forwarding on source NAT ... === TestName: 
test_01_port_fwd_on_src_nat | Status : SUCCESS ===
ok
Test for port forwarding on non source NAT ... === TestName: 
test_02_port_fwd_on_non_src_nat | Status : SUCCESS ===
ok
Test for reboot router ... === TestName: test_reboot_router | Status : 
SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === 
TestName: test_network_rules_acquired_public_ip_1_static_nat_rule | Status : 
SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === 
TestName: test_network_rules_acquired_public_ip_2_nat_rule | Status : SUCCESS 
===
ok
Test for Router rules for network rules on acquired public IP ... === 
TestName: test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Status : 
SUCCESS ===
ok

--
Ran 29 tests in 12581.963s

OK

```


And:

```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a 
tags=advanced,required_hardware=false \
smoke/test_routers.py \
smoke/test_network_acl.py \
smoke/test_privategw_acl.py \
smoke/test_reset_vm_on_reboot.py \
smoke/test_vm_life_cycl

[GitHub] cloudstack pull request: CLOUDSTACK-8746: VM Snapshotting implemen...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/977#issuecomment-152569385
  
FYI: Results of tests that I run on this branch:

```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a 
tags=advanced,required_hardware=true \
component/test_vpc_redundant.py \
component/test_routers_iptables_default_policy.py \
component/test_routers_network_ops.py \
component/test_vpc_router_nics.py \
smoke/test_loadbalance.py \
smoke/test_internal_lb.py \
smoke/test_ssvm.py \
smoke/test_network.py

```

Result:

```
Create a redundant VPC with two networks with two VMs in each network ... 
=== TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : 
SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network and 
check default routes ... === TestName: test_02_redundant_VPC_default_routes | 
Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policy on RouterVM ... === TestName: 
test_02_routervm_iptables_policies | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policies on VPC router ... === 
TestName: test_01_single_VPC_iptables_policies | Status : SUCCESS ===
ok
Stop existing router, add a PF rule and check we can access the VM ... === 
TestName: test_isolate_network_FW_PF_default_routes | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: 
test_RVR_Network_FW_PF_SSH_default_routes | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test nics 
after destroy ... === TestName: test_01_VPC_nics_after_destroy | Status : 
SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test default 
routes ... === TestName: test_02_VPC_default_routes | Status : SUCCESS ===
ok
Check the password file in the Router VM ... === TestName: 
test_isolate_network_password_server | Status : SUCCESS ===
ok
Check that the /etc/dhcphosts.txt doesn't contain duplicate IPs ... === 
TestName: test_router_dhcphosts | Status : FAILED ===
FAIL
Test to create Load balancing rule with source NAT ... === TestName: 
test_01_create_lb_rule_src_nat | Status : SUCCESS ===
ok
Test to create Load balancing rule with non source NAT ... === TestName: 
test_02_create_lb_rule_non_nat | Status : SUCCESS ===
ok
Test for assign & removing load balancing rule ... === TestName: 
test_assign_and_removal_lb | Status : SUCCESS ===
ok
Test to verify access to loadbalancer haproxy admin stats page ... === 
TestName: test02_internallb_haproxy_stats_on_all_interfaces | Status : SUCCESS 
===
ok
Test create, assign, remove of an Internal LB with roundrobin http traffic 
to 3 vm's ... === TestName: test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 
| Status : SUCCESS ===
ok
Test SSVM Internals ... === TestName: test_03_ssvm_internals | Status : 
SUCCESS ===
ok
Test CPVM Internals ... === TestName: test_04_cpvm_internals | Status : 
SUCCESS ===
ok
Test stop SSVM ... === TestName: test_05_stop_ssvm | Status : SUCCESS ===
ok
Test stop CPVM ... === TestName: test_06_stop_cpvm | Status : SUCCESS ===
ok
Test reboot SSVM ... === TestName: test_07_reboot_ssvm | Status : SUCCESS 
===
ok
Test reboot CPVM ... === TestName: test_08_reboot_cpvm | Status : SUCCESS 
===
ok
Test destroy SSVM ... === TestName: test_09_destroy_ssvm | Status : SUCCESS 
===
ok
Test destroy CPVM ... === TestName: test_10_destroy_cpvm | Status : SUCCESS 
===
ok
Test for port forwarding on source NAT ... === TestName: 
test_01_port_fwd_on_src_nat | Status : SUCCESS ===
ok
Test for port forwarding on non source NAT ... === TestName: 
test_02_port_fwd_on_non_src_nat | Status : SUCCESS ===
ok
Test for reboot router ... === TestName: test_reboot_router | Status : 
SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === 
TestName: test_network_rules_acquired_public_ip_1_static_nat_rule | Status : 
SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === 
TestName: test_network_rules_acquired_public_ip_2_nat_rule | Status : SUCCESS 
===
ok
Test for Router rules for network rules on acquired public IP ... === 
TestName: test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Status : 
SUCCESS ===
ok
--
Ran 29 tests in 12630.025s

FAILED (failures=1)

```

The test that failed is `test_router_dhcphosts`, as it was recently 
added/fixed this is probably an issue not related to this PR.


And:

```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a 
tags=advanced,required_hardware=false \
smoke/test_routers.py \
smoke/test_netwo

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1008#issuecomment-152568600
  
based on the review of the changes and looking at the deployment of the 
management servers i think this LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8964: Ovm3HypervisorGuru wrong...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/975#issuecomment-152566972
  
@dahn are you OK to merge this as-is?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Fixed: Network Update from RVR offering t...

2015-10-30 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/818


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


CloudStack-8933 changes regressed HyperV and VMware BVTs - 8 tests in test_ssvm.py

2015-10-30 Thread Raja Pullela
Hi Wilder

The changes from this commit are failing the HyperV and VMware BVTs.  

Can you please let me know if these tests were passing in your environment- 
VMware, Hyperv? 

Raja

Error - "the cached link local should be same current local ip, but they are 
different!"



[GitHub] cloudstack pull request: another typo that appears when monitoring...

2015-10-30 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/913


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1005#issuecomment-152564991
  
@DaanHoogland Tests are running.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: New PMC member: Erik Weber

2015-10-30 Thread Remi Bergsma
Congrats Erik!




On 30/10/15 15:01, "Giles Sirett"  wrote:

>All
>The PMC is delighted to announce that we have asked Erik weber to join the PMC 
>and he has accepted
>
>Please join me in congratulating Erik
>
>Kind Regards
>Giles
>
>D: +44 20 3603 0541 | M: +44 796 111 
>2055
>giles.sir...@shapeblue.com
>Find out more about ShapeBlue and our range of CloudStack related services
>
>IaaS Cloud Design & Build
>CSForge – rapid IaaS deployment framework
>CloudStack Consulting
>CloudStack Software 
>Engineering
>CloudStack Infrastructure 
>Support
>CloudStack Bootcamp Training Courses
>
>This email and any attachments to it may be confidential and are intended 
>solely for the use of the individual to whom it is addressed. Any views or 
>opinions expressed are solely those of the author and do not necessarily 
>represent those of Shape Blue Ltd or related companies. If you are not the 
>intended recipient of this email, you must neither take any action based upon 
>its contents, nor copy or show it to anyone. Please contact the sender if you 
>believe you have received this email in error. Shape Blue Ltd is a company 
>incorporated in England & Wales. ShapeBlue Services India LLP is a company 
>incorporated in India and is operated under license from Shape Blue Ltd. Shape 
>Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
>operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
>registered by The Republic of South Africa and is traded under license from 
>Shape Blue Ltd. ShapeBlue is a registered trademark.


Re: [4.6] VM Snapshots no longer work with managed storage

2015-10-30 Thread Remi Bergsma
Hi Mike,

Thanks for reporting this issue.

Can you provide some more details please? The Jira issue has none. When you put 
a blocker, we need to have some information so we can understand what is going 
on and help with debugging / fixing. Even better would be to discuss this. 
Let’s focus on resolving this.

On the comments Wei notices this may be due to the Guru change. You may want to 
contact him.

Keep me posted over the weekend, please.

Regards,
Remi





On 30/10/15 06:30, "Mike Tutkowski"  wrote:

>Hi Remi,
>
>I have opened the following ticket against 4.6:
>
>https://issues.apache.org/jira/browse/CLOUDSTACK-9008
>
>I found this issue tonight while running regression tests against 4.6.
>
>As the ticket indicates, when using managed storage for your root disk,
>attempting to revert a VM to a VM snapshot corrupts the state of the
>virtual disk.
>
>I am not sure when this regression happened. At present, I kick these
>regression tests off manually and noticed this problem just now.
>
>I marked it as a blocker because it leads to data corruption.
>
>I plan to look into it tomorrow.
>
>Thanks,
>
>-- 
>*Mike Tutkowski*
>*Senior CloudStack Developer, SolidFire Inc.*
>e: mike.tutkow...@solidfire.com
>o: 303.746.7302
>Advancing the way the world uses the cloud
>*™*


[GitHub] cloudstack pull request: CLOUDSTACK-8677: use wrapper classes for ...

2015-10-30 Thread borisroman
Github user borisroman commented on the pull request:

https://github.com/apache/cloudstack/pull/987#issuecomment-152559399
  
@widi already building


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Use java.io.tmpdir instead of hardcoded /...

2015-10-30 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1012#issuecomment-152555222
  
Tested and that is not working on Windows.
I remember having fixed that into a PR that @DaanHoogland submitted
@wido, to fix the problem for windows you have to:
1.  In file “LocalTemplateDownloaderTest.java” change the line 34 to 
String url = new File("pom.xml").toURI().toURL().toString();
2.  In file “LocalTemplateDownloader.java” change the line 41 to  
String filename = new File(downloadUrl).getName();
That is all.



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9014 Rename xapi plugins for s...

2015-10-30 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/982


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9014 Rename xapi plugins for s...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/982#issuecomment-152553309
  
@karuturi Added Jira issue CLOUDSTACK-9014


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: New PMC member: Erik Weber

2015-10-30 Thread Patrick Dube
Congratulations Erik!

On Fri, Oct 30, 2015 at 10:33 AM Wido den Hollander  wrote:

> Congrats! Welcome :)
>
> On 30-10-15 15:01, Giles Sirett wrote:
> > All
> > The PMC is delighted to announce that we have asked Erik weber to join
> the PMC and he has accepted
> >
> > Please join me in congratulating Erik
> >
> > Kind Regards
> > Giles
> >
> > D: +44 20 3603 0541 | M: +44 796 111
> 2055
> > giles.sir...@shapeblue.com
> > Find out more about ShapeBlue and our range of CloudStack related
> services
> >
> > IaaS Cloud Design & Build<
> http://shapeblue.com/iaas-cloud-design-and-build//>
> > CSForge – rapid IaaS deployment framework
> > CloudStack Consulting
> > CloudStack Software Engineering<
> http://shapeblue.com/cloudstack-software-engineering/>
> > CloudStack Infrastructure Support<
> http://shapeblue.com/cloudstack-infrastructure-support/>
> > CloudStack Bootcamp Training Courses<
> http://shapeblue.com/cloudstack-training/>
> >
> > This email and any attachments to it may be confidential and are
> intended solely for the use of the individual to whom it is addressed. Any
> views or opinions expressed are solely those of the author and do not
> necessarily represent those of Shape Blue Ltd or related companies. If you
> are not the intended recipient of this email, you must neither take any
> action based upon its contents, nor copy or show it to anyone. Please
> contact the sender if you believe you have received this email in error.
> Shape Blue Ltd is a company incorporated in England & Wales. ShapeBlue
> Services India LLP is a company incorporated in India and is operated under
> license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda is a
> company incorporated in Brasil and is operated under license from Shape
> Blue Ltd. ShapeBlue SA Pty Ltd is a company registered by The Republic of
> South Africa and is traded under license from Shape Blue Ltd. ShapeBlue is
> a registered trademark.
> >
>


Re: New PMC member: Erik Weber

2015-10-30 Thread Wido den Hollander
Congrats! Welcome :)

On 30-10-15 15:01, Giles Sirett wrote:
> All
> The PMC is delighted to announce that we have asked Erik weber to join the 
> PMC and he has accepted
> 
> Please join me in congratulating Erik
> 
> Kind Regards
> Giles
> 
> D: +44 20 3603 0541 | M: +44 796 111 
> 2055
> giles.sir...@shapeblue.com
> Find out more about ShapeBlue and our range of CloudStack related services
> 
> IaaS Cloud Design & Build
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Software 
> Engineering
> CloudStack Infrastructure 
> Support
> CloudStack Bootcamp Training 
> Courses
> 
> This email and any attachments to it may be confidential and are intended 
> solely for the use of the individual to whom it is addressed. Any views or 
> opinions expressed are solely those of the author and do not necessarily 
> represent those of Shape Blue Ltd or related companies. If you are not the 
> intended recipient of this email, you must neither take any action based upon 
> its contents, nor copy or show it to anyone. Please contact the sender if you 
> believe you have received this email in error. Shape Blue Ltd is a company 
> incorporated in England & Wales. ShapeBlue Services India LLP is a company 
> incorporated in India and is operated under license from Shape Blue Ltd. 
> Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
> operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
> registered by The Republic of South Africa and is traded under license from 
> Shape Blue Ltd. ShapeBlue is a registered trademark.
> 


[GitHub] cloudstack pull request: Use java.io.tmpdir instead of hardcoded /...

2015-10-30 Thread wido
Github user wido commented on the pull request:

https://github.com/apache/cloudstack/pull/1012#issuecomment-152541675
  
@DaanHoogland I think it was @SudharmaJain

As long as the build still works on Linux/Mac we didn't break anything. I'm 
positive it's now also fixed on Windows.

There are still many places in CS where we refer to /tmp directly and we 
probably want to use java.io.tmpdir to give admins flexibility if they want to.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: New PMC member: Erik Weber

2015-10-30 Thread Daan Hoogland
enloy the ride, Erik. Welcome.

On Fri, Oct 30, 2015 at 3:17 PM, Mike Tutkowski <
mike.tutkow...@solidfire.com> wrote:

> Congratulations, Erik!
>
> On Friday, October 30, 2015, Giles Sirett 
> wrote:
>
> > All
> > The PMC is delighted to announce that we have asked Erik weber to join
> the
> > PMC and he has accepted
> >
> > Please join me in congratulating Erik
> >
> > Kind Regards
> > Giles
> >
> > D: +44 20 3603 0541 | M: +44 796 111
> > 2055
> > giles.sir...@shapeblue.com  > giles.sir...@shapeblue.com >
> > Find out more about ShapeBlue and our range of CloudStack related
> services
> >
> > IaaS Cloud Design & Build<
> > http://shapeblue.com/iaas-cloud-design-and-build//>
> > CSForge – rapid IaaS deployment framework
> > CloudStack Consulting
> > CloudStack Software Engineering<
> > http://shapeblue.com/cloudstack-software-engineering/>
> > CloudStack Infrastructure Support<
> > http://shapeblue.com/cloudstack-infrastructure-support/>
> > CloudStack Bootcamp Training Courses<
> > http://shapeblue.com/cloudstack-training/>
> >
> > This email and any attachments to it may be confidential and are intended
> > solely for the use of the individual to whom it is addressed. Any views
> or
> > opinions expressed are solely those of the author and do not necessarily
> > represent those of Shape Blue Ltd or related companies. If you are not
> the
> > intended recipient of this email, you must neither take any action based
> > upon its contents, nor copy or show it to anyone. Please contact the
> sender
> > if you believe you have received this email in error. Shape Blue Ltd is a
> > company incorporated in England & Wales. ShapeBlue Services India LLP is
> a
> > company incorporated in India and is operated under license from Shape
> Blue
> > Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in
> Brasil
> > and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd
> is
> > a company registered by The Republic of South Africa and is traded under
> > license from Shape Blue Ltd. ShapeBlue is a registered trademark.
> >
>
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the cloud
> *™*
>



-- 
Daan


Re: New PMC member: Erik Weber

2015-10-30 Thread Mike Tutkowski
Congratulations, Erik!

On Friday, October 30, 2015, Giles Sirett 
wrote:

> All
> The PMC is delighted to announce that we have asked Erik weber to join the
> PMC and he has accepted
>
> Please join me in congratulating Erik
>
> Kind Regards
> Giles
>
> D: +44 20 3603 0541 | M: +44 796 111
> 2055
> giles.sir...@shapeblue.com  giles.sir...@shapeblue.com >
> Find out more about ShapeBlue and our range of CloudStack related services
>
> IaaS Cloud Design & Build<
> http://shapeblue.com/iaas-cloud-design-and-build//>
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Software Engineering<
> http://shapeblue.com/cloudstack-software-engineering/>
> CloudStack Infrastructure Support<
> http://shapeblue.com/cloudstack-infrastructure-support/>
> CloudStack Bootcamp Training Courses<
> http://shapeblue.com/cloudstack-training/>
>
> This email and any attachments to it may be confidential and are intended
> solely for the use of the individual to whom it is addressed. Any views or
> opinions expressed are solely those of the author and do not necessarily
> represent those of Shape Blue Ltd or related companies. If you are not the
> intended recipient of this email, you must neither take any action based
> upon its contents, nor copy or show it to anyone. Please contact the sender
> if you believe you have received this email in error. Shape Blue Ltd is a
> company incorporated in England & Wales. ShapeBlue Services India LLP is a
> company incorporated in India and is operated under license from Shape Blue
> Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil
> and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is
> a company registered by The Republic of South Africa and is traded under
> license from Shape Blue Ltd. ShapeBlue is a registered trademark.
>


-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the cloud
*™*


[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-10-30 Thread davidamorimfaria
Github user davidamorimfaria commented on the pull request:

https://github.com/apache/cloudstack/pull/1008#issuecomment-152536728
  
I haven't installed a cloud with the packages made with this change, but so 
far the management servers start and talk to each other:

Oct 30 13:54:04 mgmt01 server: WARN  [c.c.c.ClusterManagerImpl] 
(Cluster-Notification-1:ctx-cb4bc2b4) Notifying management server join event 
took 0 ms

 These were the steps I did, using a mariadb 10 as db backend:
- yum localinstall cloudstack-common-4.6.0-SNAPSHOT.el7.centos.x86_64.rpm 
cloudstack-management-4.6.0-SNAPSHOT.el7.centos.x86_64.rpm 
cloudstack-usage-4.6.0-SNAPSHOT.el7.centos.x86_64.rpm 
cloudstack-cli-4.6.0-SNAPSHOT.el7.centos.x86_64.rpm
- [in mgmt01] cloudstack-setup-databases cloud:@localhost
- [in mgmt02] cloudstack-setup-databases 
cloud:@
- cloudstack-setup-management
- systemctl enable cloudstack-management
- systemctl start cloudstack-management



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Use java.io.tmpdir instead of hardcoded /...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1012#issuecomment-152536606
  
this lgtm but I am not running on any windows machines. @wido do you 
remember who the people that were bothered were and can you ping thm to verify?
basically we can ignore each and any verification that we have (semi) 
automate so far for this PR


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


New PMC member: Erik Weber

2015-10-30 Thread Giles Sirett
All
The PMC is delighted to announce that we have asked Erik weber to join the PMC 
and he has accepted

Please join me in congratulating Erik

Kind Regards
Giles

D: +44 20 3603 0541 | M: +44 796 111 
2055
giles.sir...@shapeblue.com
Find out more about ShapeBlue and our range of CloudStack related services

IaaS Cloud Design & Build
CSForge – rapid IaaS deployment framework
CloudStack Consulting
CloudStack Software 
Engineering
CloudStack Infrastructure 
Support
CloudStack Bootcamp Training Courses

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England & Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
registered by The Republic of South Africa and is traded under license from 
Shape Blue Ltd. ShapeBlue is a registered trademark.


Re: Building SNAPSHOT debian packages

2015-10-30 Thread Wido den Hollander


On 29-10-15 14:20, Jeff Hair wrote:
> Hi,
> 
> We are building Debian packages on unstable branches (4.5 branch to be
> specific). The Debian source format 30. rejects the version 4.5.3-SNAPSHOT
> with an error "Can't build with source format 3.0 native: native package
> version may not have a revision."
> 
> Source format 1.0 allows this value, and also dpkg 1.17.5 only emits a
> warning for this. We are pretty sure dpkg 1.17.6 is what upgrades this
> warning to an error.
> 
> Have the official CloudStack builds been running into this problem when
> building unstable debs?
> 

I did, but usually I worked around it a bit. I usually never build debs
for testing purposes, they are only build after a release.

Wido

> Jeff
> 


[GitHub] cloudstack pull request: Use java.io.tmpdir instead of hardcoded /...

2015-10-30 Thread wido
GitHub user wido opened a pull request:

https://github.com/apache/cloudstack/pull/1012

Use java.io.tmpdir instead of hardcoded /tmp

This was submitted earlier in PR #884 but that did not merge
properly.

This is a new PR with the same change.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/wido/cloudstack tmpdir-in-test

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1012.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1012


commit 1a962bf78e84fce546bc9447a1bd376a70582e7d
Author: Wido den Hollander 
Date:   2015-10-30T13:19:38Z

Use java.io.tmpdir instead of hardcoded /tmp

This was submitted earlier in PR #884 but that did not merge
properly.

This is a new PR with the same change.




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9003 Make VirtualMachineName i...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/988#issuecomment-152516868
  
@ProjectMoon : sounds good let me know of any more PoC code or further 
design.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9003 Make VirtualMachineName i...

2015-10-30 Thread ProjectMoon
Github user ProjectMoon commented on the pull request:

https://github.com/apache/cloudstack/pull/988#issuecomment-152513166
  
While I haven't pushed new work to this PR yet, I do have a prototype of 
what I intend to do, and am looking for thoughts on it here.

My plan:
* Move MachineNameService and UUID manager into one interface. Remove the 
methods from VirtualMachineNameService which are not used. Call this interface 
ResourceNamingPolicy.
* Create a ResourceNamingPolicy extension registry, and a new module for 
the default naming policy (which is the UUIDs ACS generates at the moment).
* When using methods from ResourceNamingPolicy, iterate through all 
registered policies until one returns a name that is not null. This is in line 
with other things like API authenticators, host planners, etc.
* Expand the use of the ResourceNamingPolicy beyond virtual machines and 
volumes (which is what VirtualMachineName/UUIDManager touch currently). The 
plan is to make use of this for these entities to begin with: VMs, volumes, 
templates, security groups.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9012 :automation of cores feat...

2015-10-30 Thread shwetaag
Github user shwetaag commented on the pull request:

https://github.com/apache/cloudstack/pull/1011#issuecomment-152510674
  
pasting result.txt

test1_coreos_VM_creation 
(integration.component.test_coreos.TestDeployVmWithCoreosTemplate) ... === 
TestName: test1_coreos_VM_creation | Status : SUCCESS ===
ok

--
Ran 1 test in 838.303s

OK



---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-9012 :automation of cores feat...

2015-10-30 Thread shwetaag
GitHub user shwetaag opened a pull request:

https://github.com/apache/cloudstack/pull/1011

CLOUDSTACK-9012 :automation of cores feature test path

https://issues.apache.org/jira/browse/CLOUDSTACK-9012
Automated a full scenario of coreos guest OS support:
it includes registering coreos templates present at 
http://dl.openvm.eu/cloudstack/coreos/x86_64/
1. based on hypervisor types of zone
2. creating ssh key pair
3. creating a sample user data
4. creating a coreos virtual machine using this ssh keypair and userdata
5. verifying ssh access to coreo os machine using keypair and core username
6. verifying userdata is applied on virtual machine and the service asked 
in sample data is actually running
7. Verifying userdata in router vm as well

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/shwetaag/cloudstack coreos

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1011.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1011


commit 95355960746fbe1bdc649a483bff250b115ee7e7
Author: shweta agarwal 
Date:   2015-10-30T11:24:16Z

automation of cores feature test path

corrected some entires in test data




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8677: use wrapper classes for ...

2015-10-30 Thread wido
Github user wido commented on the pull request:

https://github.com/apache/cloudstack/pull/987#issuecomment-152505938
  
@borisroman Can we run the tests on this one?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1005#issuecomment-152503018
  
@remibergsma can you spare a bubble for this one?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Fixed user_vm_view to only display keypai...

2015-10-30 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/1006


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: applying fixes, after 4.6 is out

2015-10-30 Thread Daan Hoogland
valid question but is this more work in comparison to having the
contribution be refused on the grounds of not being on the right version?
We encounter a conflict and then bring it back to the contributor...
further flow is untouched. agree?

On Fri, Oct 30, 2015 at 11:55 AM, Boris Schrijver 
wrote:

> Good idea! If there aren't any merge conflicts we could even automate it
> by means of a script.
>
> Though, who's responsible if there are conflicts? Extra work, testing
> against the other branch?
>
> On October 30, 2015 at 11:45 AM Daan Hoogland 
> wrote:
>
>
> H,
>
> Being kind of an optimist sometimes I want to start talking on how to
> handle PRs from 4.6.1 onward. If you don't understand what I'm talking
> about, please feel free to bother me with questions.
>
> I had a discussion with Remi last night about refusing PRs or not based on
> the premiss they are fixing bugs in the prior version as well. If a PR can
> be applied on a prior version I think we should just do this and not bother
> the contributor with it. We can apply it on a prior branch and merge it
> forward. It might give and extra burden on us for closing the PR but the
> other side is that it will not discourage contributors.
>
> thoughts?
> --
> Daan
>
>


-- 
Daan


Re: Git with Eclipse

2015-10-30 Thread Daan Hoogland
Ron, in eclipse i avoid doing any work involving multiple remotes. I do
those things from the commandline. Other things work fine and the part with
remotes might work well as well these days. If you use only your fork from
eclipse and fetch and rebase from apache or github  from out side eclipse
you won't have much issues either but integrating is really for the Devine.
I won't go as far as to say it doesn't work but I recognise your pain and
there is to much useful stuff to be done to dive into this IMHO.

What Sebastien describes is good with one addition: keep your fork up to
date by pushing a fetch from apache back to yours from time to time. What I
do is I fetch master from github/apache, branch, checkout my branch, edit
and commit on my branch and then push my branch to my fork (not using
master there  at all. The rest I do on the github ui as Sebastien
describes.


On Fri, Oct 30, 2015 at 10:58 AM, Sebastien Goasguen 
wrote:

> Ron,
>
> I don’t know how to use Github with eclipse, I use the command line for
> everything.
>
> But the documentation should be good:
>
> https://github.com/apache/cloudstack-docs-admin#contributing-to-the-documentation
>
> Basically you need an account on GitHub.
> You fork https://github.com/apache/cloudstack-docs-admin (or the other
> docs repo).
> You will then have a fork at https://github.com/
> /cloudstack-docs-admin
>
> Clone your own fork:
>
> $ git clone https://github.com//cloudstack-docs-admin
>
> Make your commits via eclipse and push to your own fork.
>
> Then on the GitHub UI, create a pull request to the upstream repo:
> https://github.com/apache/cloudstack-docs-admin
>
> -sebastien
>
> > On Oct 29, 2015, at 9:58 PM, Ron Wheeler 
> wrote:
> >
> > Trying to help out with the docs without doing too much damage.
> > I am new to git.
> >
> > I am using Eclipse with the egit/mylyn plug-in and I have managed to get
> the source for the docs-admin downloaded.
> >
> > The "Configure Fetch" has a Source Ref of "refs/heads/*" and a
> Desctination Ref  of "refs/remotes/origin/*"
> > I think that I have the right sources. They seem to match the published
> docs.
> >
> > I have made my changes and think that I need to create a pull request to
> get one of the Cloudstack gods to decide if my attempts are worthy.
> >
> > I have created the pull request in Eclipse but when I try to submit it I
> get an error box titled "Submit failed" popping up with the rather cryptic
> "Unexpected error: Invalid request. "base", "head" weren't supplied".
> > I think that I have to configure the "Remotes/origin/
> https://gethub.com/apache/cloudstack-doc-admin.git"; entry  using
> "Configure Push" menu.
> >
> > It has a section called "Ref mappings" that I gather is how git figures
> out how I want my changes mapped back into the remote repos.
> >
> > Am I looking in the right places?
> > If so, what are the right "Source Ref" and "Destination Ref" entries to
> get a pull request that will please the gods (both local and remote).
> >
> > It looks like editing and creating pull requests gets easy once this is
> working.
> >
> > Still have to work on getting the document production working so I can
> actually see the fruits of my labour.
> >
> > Thanks
> >
> > Ron
> >
> >
> > *
> > *
> >
> > --
> > Ron Wheeler
> > President
> > Artifact Software Inc
> > email: rwhee...@artifact-software.com
> > skype: ronaldmwheeler
> > phone: 866-970-2435, ext 102
> >
>
>


-- 
Daan


[GitHub] cloudstack pull request: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread wido
Github user wido commented on the pull request:

https://github.com/apache/cloudstack/pull/1005#issuecomment-152501126
  
@DaanHoogland I've just pushed a version with a comment in there to explain 
what I'm doing

Code-wise nothing changed


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8793 Enable s2s VPN connection...

2015-10-30 Thread pdion891
Github user pdion891 commented on the pull request:

https://github.com/apache/cloudstack/pull/879#issuecomment-152500179
  
Thanks @remibergsma I haven't got time to retest that branch again,  now 
that it's in master will retry...




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1005#issuecomment-152499836
  
@wido good answer based on code review LGTM so far


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---



[GitHub] cloudstack pull request: another typo that appears when monitoring...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/913#issuecomment-152499596
  
valid English replacement of strings I see @bhaisaab 's point in his 
comment but had eclipse check java references to the call and is never made. It 
must be implemented as on override of an interface method.

LGTM ( @remibergsma )


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: Add Debian 8.0 in OS List for Template

2015-10-30 Thread Daan Hoogland
Milamber, there are api's for this:
http://cloudstack.apache.org/api/apidocs-4.5/root_admin/addGuestOs.html and
http://cloudstack.apache.org/api/apidocs-4.5/root_admin/addGuestOsMapping.html

Or am I missing your point?

On Fri, Oct 30, 2015 at 8:48 AM, Milamber  wrote:

> Hello,
>
> There are any special reason to not add Debian 8.0 32/64 bits on the list
> of OS in the template form ? like missing support from kvm or xenserver?
>
> Or we just need to add the INSERT into the sql schema like this commit [1]
> ?
>
> That would be great if the next release 4.6 could have the Debian 8.0 in
> their list (and perhaps too: Debian testing)
>
> Thanks
>
> Mialber
>
>
> [1]
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=blobdiff;f=setup/db/db/schema-442to450.sql;h=cd2bf6e5a3756bf3ea499f20f5bc81ab219dbd89;hp=e2615d6cf6541cc7a6b6763421ba8398d593;hb=66afce66b469e9772989ffcef3015bdab9584ffc;hpb=9928d66fdaf71b43412e20b129975c42e0552d6f
>



-- 
Daan


Re: Automatically trigger a new Jenkins build

2015-10-30 Thread Daan Hoogland
So Rajani, you suggest to remove the automatic trigger and leave that to
reviewer. Sound fine to me.

On Fri, Oct 30, 2015 at 7:35 AM, Rajani Karuturi  wrote:

> Thats a good idea. I have seen other open source projects do it this way.
> (example: netty project)
> On demand build is a better way than doing for every commit change.
>
> especially when the code reviews are going on or there is WIP, its not
> intended to do a build on every commit or update. (The commit could just be
> a change in commit message)
>
> when the contributor is ready, he can call jenkins to verify it by just
> commenting @jenkins build
> once the code comments(if any) are addressed, we can ask it to build again.
>
> ~Rajani
>
> On Thu, Oct 29, 2015 at 7:41 PM, Remi Bergsma  >
> wrote:
>
> > Hi all,
> >
> > Just had a chat with Miguel.
> > He showed me that if we setup Github to notify Jenkins on “issue
> comments”
> > (next to “pull requests” we have now) and then set a “trigger phrase” in
> > Jenkins, we could automatically trigger a new build by typing “go build”.
> > No more need to force push commits by the author.
> >
> > Shall we set this up? Can we do this David?
> >
> > Regards,
> > Remi
> >
> >
> > From: Remi Bergsma
> > Date: Thursday 29 October 2015 08:47
> > To: "dev@cloudstack.apache.org"
> > Subject: Jenkins failures
> >
> > Hi,
> >
> > Can someone please look why Jenkins fails so often recently? It feels
> like
> > a casino and slows down merging of PRs since we want to merge when they
> > “are green”. This requires forse pushing many times now, including the
> wait.
> >
> > I’d prefer not to start ignoring them..
> >
> > Thanks for the help!
> >
> > Regards,
> > Remi
> >
> >
>



-- 
Daan


Re: [PROPOSAL] remove travis pull request checks from github

2015-10-30 Thread Daan Hoogland
On Fri, Oct 30, 2015 at 7:05 AM, Rajani Karuturi  wrote:
​+1'ed

and further

> On Fri, Oct 30, 2015 at 4:55 AM, David Nalley  wrote:
>
> > 2 hours before this message was sent, there was a message about
> > jenkins being unreliable.
>
​both are not perfect, travis is costing RMs and their minions, and
contributors time!​



> >
> > So I am starting to think that something other than Travis or Jenkins
> > is the problem.
>
​No, Jenkins and Travis are problems! maintenance wise they are costing
effort we are not putting into it.​


>
> > --David
> >
> > On Thu, Oct 29, 2015 at 5:33 PM, Daan Hoogland 
> > wrote:
> > > Guys and dolls,
> > >
> > > We have been having travis builds for a while now and they seem to
> > generate
> > > more work then help. There are a lot of time-outs, (false positive) and
> > > when a suite passes it is often because all tests where skipped (false
> > > negative). In addition, and this may be just me, I've never been able
> to
> > > pinpoint a problem based on travis output.
> > >
> > > therefore:
> > >
> > > I move that we completely disable travis runs on PRs to the cloudstack
> > > project. This will reduce confusion about whether a PR is good and will
> > > reduce the need for willing contributers to force push their branch
> over
> > > and over.
> > >
> > > --
> > > Daan
> >
>



-- 
Daan


[GitHub] cloudstack pull request: Rename xapi plugins for s3 and swift to m...

2015-10-30 Thread karuturi
Github user karuturi commented on the pull request:

https://github.com/apache/cloudstack/pull/982#issuecomment-152497680
  
manually tested by upgrading an existing xenserver setup and force 
reconnecting the host

Before
```
[root@rjn-xen65-host plugins]# ls -la /etc/xapi.d/plugins/ | grep xen
-rwxr-xr-x 1 root root 12708 Sep 29 10:44 s3xen
-rwxr-xr-x 1 root root  3235 Sep 29 10:44 swiftxen
```

After
```
[root@rjn-xen65-host plugins]# ls -la /etc/xapi.d/plugins/ | grep xen
-rwxr-xr-x 1 root root 12708 Sep 29 10:44 s3xen
-rwxr-xr-x 1 root root 12714 Oct 30 10:42 s3xenserver
-rwxr-xr-x 1 root root  3235 Sep 29 10:44 swiftxen
-rwxr-xr-x 1 root root  3241 Oct 30 10:42 swiftxenserver
```

I am guessing having the both the scripts after upgrade wont be an issue. I 
didnt see any issues after the change(manually tested launch new vm etc.) 
:+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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Fixed: Network Update from RVR offering t...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/818#issuecomment-152496955
  
LGTM I ran the compile on this one and did a code review.
@remi, no changes since your last check, you can merge at your will


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8715: Add VirtIO channel to al...

2015-10-30 Thread wido
Github user wido commented on the pull request:

https://github.com/apache/cloudstack/pull/985#issuecomment-152494985
  
@ustcweizhou @remibergsma I just pushed a new version of the commit.

On Ubuntu AppArmor needs to be disabled since the default profile for 
libvirt doesn't allow writing into /var/lib/libvirt/qemu. This is however 
already the case with the SSVM.

This could be fixed by adding this to 
'/etc/apparmor.d/abstractions/libvirt-qemu':
/var/lib/libvirt/qemu/channel/target/* rw,


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Fixed user_vm_view to only display keypai...

2015-10-30 Thread karuturi
Github user karuturi commented on the pull request:

https://github.com/apache/cloudstack/pull/1006#issuecomment-152493863
  
Manual Testing:
Before:
ssh_keypairs
```
mysql> select * from ssh_keypairs;

+++---+--+-+--+
| id | account_id | domain_id | keypair_name | fingerprint  
   | public_key 

  |

+++---+--+-+--+
|  1 | 60 | 3 | rajani   | 
60:f3:cb:9b:c4:b8:c1:cd:36:13:3a:28:17:26:01:b2 | ssh-rsa 
B3NzaC1yc2EDAQABgQDo6DcY9r8Br56JUD2Vv33KNiieElPQV0+Qepz+8lGHlIlYn8I2ubXcVOrGBtET6JGlHqVlSmjKEjeHXcvILPZsxxMjGX4skPQtBVOaH1kQcYU2YpYxxm5piQPpOo263AkWmQrfBb/W0n8ChWfOuBSYvbpzdqATsQUGnNCQuthBKQ==
 |
|  3 | 62 | 3 | bharat   | 
69:72:47:63:10:aa:76:37:d9:fc:c2:51:66:1e:b9:93 | ssh-rsa 
B3NzaC1yc2EDAQABgQDo6DcY9r8Br56JUD2Vv33KNiieElPQV0+Qepz+8lGHlIlYn8I2ubXcVOrGBtET6JGlHqVlSmjKEjeHXcvILPZsxxMjGX4skPQtBVOaH1kQcYU2YpYxxm5piQPpOo263AkWmQrfBb/W0n8ChWfOuBSYvbpzdqATsQUGnNCQuthBKQ==
 |

+++---+--+-+--+
```
user_vm_view;
```
mysql> select id,name,account_name,keypair_name from user_vm_view;
++--+--+--+
| id | name | account_name | keypair_name |
++--+--+--+
| 67 | rvm1 | rajanik  | rajani   |
| 68 | bvm1 | bharatk  | rajani   |
| 67 | rvm1 | rajanik  | bharat   |
| 68 | bvm1 | bharatk  | bharat   |
++--+--+--+
```
Ui vm screenshot:
![screen shot 2015-10-30 at 3 30 01 
pm](https://cloud.githubusercontent.com/assets/186833/10843914/f732beee-7f21-11e5-9682-c5207f3c4dfd.png)


1. checked out the PR
2. did mvn clean install -Pdeveloper,systemvm on an existing setup
3. manually ran the drop view and create view query
4. start jetty mvn  -pl client jetty:run -o

After the change.

```
user_vm_view
mysql> select id,name,account_name,keypair_name from user_vm_view;
++--+--+--+
| id | name | account_name | keypair_name |
++--+--+--+
| 67 | rvm1 | rajanik  | rajani   |
| 68 | bvm1 | bharatk  | bharat   |
++--+--+--+
2 rows in set (0.00 sec)
```
UI Vm screenshot
![screen shot 2015-10-30 at 3 37 13 
pm](https://cloud.githubusercontent.com/assets/186833/10843965/374adcc8-7f22-11e5-8e06-65a22040e5d4.png)

:+1: 

Can you create an issue for this in jira and add it to the commit message 
please?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-30 Thread terbolous
Github user terbolous commented on the pull request:

https://github.com/apache/cloudstack/pull/961#issuecomment-152491165
  
just proves that we really need a new web ui :-)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-30 Thread miguelaferreira
Github user miguelaferreira commented on the pull request:

https://github.com/apache/cloudstack/pull/961#issuecomment-152490396
  
After digging in a bit more, I've debugged the browser session when hitting 
the home page with `session="false"` what I see is that the page is not even 
completely downloaded. My best guess is that the server stops responding when 
the exceptions I've posted in the previous comment happen. And because the page 
was not completely downloaded it is also not well rendered. In addition the 
script tags are at the end of the page, so none of the dynamics will work 
either.

Researching for similar problems took me to a stackoverflow question

http://stackoverflow.com/questions/2055502/java-lang-illegalstateexception-pwc3999-cannot-create-a-session-after-the-resp?answertab=active#tab-top

In the accepted answer, the poster explains that the exception (which is 
the same as I see, "Cannot create a session after the response has been 
committed") has to do with trying to write to a response that has be already 
committed (i.e. already sent to the client). The poster also mentions that the 
response is automatically flushed (i.e. sent to the client) when it reached a 
size of 2K (I assume 2K bytes). Then looking at the size of the html generated 
for the ACS home page one can easily see that it is way larger than the 2K 
(actually it is 226K large).

While not being an expert on these matters, it seems to me that the culprit 
is the sheer size and complexity of this webpage. How it actually works with 
our the `session="false"` declaration is a mystery to me.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


applying fixes, after 4.6 is out

2015-10-30 Thread Daan Hoogland
H,

Being kind of an optimist sometimes I want to start talking on how to
handle PRs from 4.6.1 onward. If you don't understand what I'm talking
about, please feel free to bother me with questions.

I had a discussion with Remi last night about refusing PRs or not based on
the premiss they are fixing bugs in the prior version as well. If a PR can
be applied on a prior version I think we should just do this and not bother
the contributor with it. We can apply it on a prior branch and merge it
forward. It might give and extra burden on us for closing the PR but the
other side is that it will not discourage contributors.

thoughts?
-- 
Daan


[GitHub] cloudstack pull request: [4.5] CLOUDSTACK-9000: logrotate cloudsta...

2015-10-30 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/993


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: [master/4.6] CLOUDSTACK-9000: logrotate c...

2015-10-30 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/992


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: [master/4.6] CLOUDSTACK-9000: logrotate c...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/992#issuecomment-152486552
  
Double checked a compile:

```
[INFO] 

[INFO] BUILD SUCCESS
[INFO] 

[INFO] Total time: 6:57.299s
[INFO] Finished at: Fri Oct 30 10:21:07 GMT 2015
[INFO] Final Memory: 84M/417M
[INFO] 

+ exit 0
```

Will soon 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 not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8993: DHCP fails with "no addr...

2015-10-30 Thread serbaut
Github user serbaut commented on the pull request:

https://github.com/apache/cloudstack/pull/981#issuecomment-152481954
  
Afaict, the DHCP code is rewritten in 4.6 so this exact issue shouldn't 
affect 4.5.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: adjust packaging for centos7

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1008#issuecomment-152479205
  
Seems to start fine now, thanks @davidamorimfaria 

Haven't setup a cloud with this or tested it further. Just built RPM 
packages and started management server.

```
[root@cs2 ~]# systemctl start cloudstack-management
[root@cs2 ~]# systemctl status cloudstack-management
cloudstack-management.service - CloudStack Management Server
   Loaded: loaded (/usr/lib/systemd/system/cloudstack-management.service; 
disabled)
   Active: active (running) since Fri 2015-10-30 09:56:41 GMT; 7s ago
 Main PID: 11075 (java)
   CGroup: /system.slice/cloudstack-management.service
   └─11075 java -Djava.awt.headless=true 
-Dcom.sun.management.jmxremote=false -Xmx2g -XX:+HeapDumpOnOutOfMemo...

Oct 30 09:56:43 cs2 server[11075]: Oct 30, 2015 9:56:43 AM 
org.apache.catalina.core.StandardEngine startInternal
Oct 30 09:56:43 cs2 server[11075]: INFO: Starting Servlet Engine: Apache 
Tomcat/7.0.54
Oct 30 09:56:43 cs2 server[11075]: Oct 30, 2015 9:56:43 AM 
org.apache.catalina.startup.HostConfig deployDirectory
Oct 30 09:56:43 cs2 server[11075]: INFO: Deploying web application 
directory /usr/share/cloudstack-management/w...client
Oct 30 09:56:43 cs2 server[11075]: Oct 30, 2015 9:56:43 AM 
org.apache.catalina.loader.WebappClassLoader validateJarFile
Oct 30 09:56:43 cs2 server[11075]: INFO: 
validateJarFile(/usr/share/cloudstack-management/webapps/client/WEB-INclass
Oct 30 09:56:43 cs2 server[11075]: Oct 30, 2015 9:56:43 AM 
org.apache.catalina.loader.WebappClassLoader validateJarFile
Oct 30 09:56:43 cs2 server[11075]: INFO: 
validateJarFile(/usr/share/cloudstack-management/webapps/client/WEB-INclass
Oct 30 09:56:43 cs2 server[11075]: Oct 30, 2015 9:56:43 AM 
org.apache.catalina.loader.WebappClassLoader validateJarFile
Oct 30 09:56:43 cs2 server[11075]: INFO: 
validateJarFile(/usr/share/cloudstack-management/webapps/client/WEB-INclass
Hint: Some lines were ellipsized, use -l to show in full.
```


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: Git with Eclipse

2015-10-30 Thread Sebastien Goasguen
Ron,

I don’t know how to use Github with eclipse, I use the command line for 
everything.

But the documentation should be good:
https://github.com/apache/cloudstack-docs-admin#contributing-to-the-documentation

Basically you need an account on GitHub.
You fork https://github.com/apache/cloudstack-docs-admin (or the other docs 
repo).
You will then have a fork at 
https://github.com//cloudstack-docs-admin

Clone your own fork:

$ git clone https://github.com//cloudstack-docs-admin

Make your commits via eclipse and push to your own fork.

Then on the GitHub UI, create a pull request to the upstream repo:
https://github.com/apache/cloudstack-docs-admin

-sebastien

> On Oct 29, 2015, at 9:58 PM, Ron Wheeler  
> wrote:
> 
> Trying to help out with the docs without doing too much damage.
> I am new to git.
> 
> I am using Eclipse with the egit/mylyn plug-in and I have managed to get the 
> source for the docs-admin downloaded.
> 
> The "Configure Fetch" has a Source Ref of "refs/heads/*" and a Desctination 
> Ref  of "refs/remotes/origin/*"
> I think that I have the right sources. They seem to match the published docs.
> 
> I have made my changes and think that I need to create a pull request to get 
> one of the Cloudstack gods to decide if my attempts are worthy.
> 
> I have created the pull request in Eclipse but when I try to submit it I get 
> an error box titled "Submit failed" popping up with the rather cryptic 
> "Unexpected error: Invalid request. "base", "head" weren't supplied".
> I think that I have to configure the 
> "Remotes/origin/https://gethub.com/apache/cloudstack-doc-admin.git"; entry  
> using "Configure Push" menu.
> 
> It has a section called "Ref mappings" that I gather is how git figures out 
> how I want my changes mapped back into the remote repos.
> 
> Am I looking in the right places?
> If so, what are the right "Source Ref" and "Destination Ref" entries to get a 
> pull request that will please the gods (both local and remote).
> 
> It looks like editing and creating pull requests gets easy once this is 
> working.
> 
> Still have to work on getting the document production working so I can 
> actually see the fruits of my labour.
> 
> Thanks
> 
> Ron
> 
> 
> *
> *
> 
> -- 
> Ron Wheeler
> President
> Artifact Software Inc
> email: rwhee...@artifact-software.com
> skype: ronaldmwheeler
> phone: 866-970-2435, ext 102
> 



[GitHub] cloudstack pull request: [master/4.6] CLOUDSTACK-9000: logrotate c...

2015-10-30 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/992#issuecomment-152476140
  
@remibergsma should we merge this on master now? (jenkins failed due to 
some jvm issue)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-30 Thread miguelaferreira
Github user miguelaferreira commented on the pull request:

https://github.com/apache/cloudstack/pull/961#issuecomment-152474694
  
In tomcat I see the following errors:
```
Oct 30, 2015 9:39:13 AM org.apache.catalina.core.ApplicationDispatcher 
invoke
SEVERE: Servlet.service() for servlet jsp threw exception
java.lang.IllegalStateException: Cannot create a session after the response 
has been committed
at 
org.apache.catalina.connector.Request.doGetSession(Request.java:2934)
at 
org.apache.catalina.connector.Request.getSession(Request.java:2310)
at 
org.apache.catalina.connector.RequestFacade.getSession(RequestFacade.java:897)
at 
javax.servlet.http.HttpServletRequestWrapper.getSession(HttpServletRequestWrapper.java:229)
at 
org.apache.catalina.core.ApplicationHttpRequest.getSession(ApplicationHttpRequest.java:569)
at 
org.apache.catalina.core.ApplicationHttpRequest.getSession(ApplicationHttpRequest.java:514)
at 
org.apache.jasper.runtime.PageContextImpl._initialize(PageContextImpl.java:147)
at 
org.apache.jasper.runtime.PageContextImpl.initialize(PageContextImpl.java:126)
at 
org.apache.jasper.runtime.JspFactoryImpl.internalGetPageContext(JspFactoryImpl.java:112)
at 
org.apache.jasper.runtime.JspFactoryImpl.getPageContext(JspFactoryImpl.java:65)
at org.apache.jsp.dictionary_jsp._jspService(dictionary_jsp.java:66)
at 
org.apache.jasper.runtime.HttpJspBase.service(HttpJspBase.java:70)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:727)
at 
org.apache.jasper.servlet.JspServletWrapper.service(JspServletWrapper.java:432)
at 
org.apache.jasper.servlet.JspServlet.serviceJspFile(JspServlet.java:390)
at org.apache.jasper.servlet.JspServlet.service(JspServlet.java:334)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:727)
at 
org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:303)
at 
org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:208)
at 
org.apache.catalina.core.ApplicationDispatcher.invoke(ApplicationDispatcher.java:748)
at 
org.apache.catalina.core.ApplicationDispatcher.doInclude(ApplicationDispatcher.java:604)
at 
org.apache.catalina.core.ApplicationDispatcher.include(ApplicationDispatcher.java:543)
at 
org.apache.jasper.runtime.JspRuntimeLibrary.include(JspRuntimeLibrary.java:954)
at org.apache.jsp.index_jsp._jspService(index_jsp.java:2735)
at 
org.apache.jasper.runtime.HttpJspBase.service(HttpJspBase.java:70)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:727)
at 
org.apache.jasper.servlet.JspServletWrapper.service(JspServletWrapper.java:432)
at 
org.apache.jasper.servlet.JspServlet.serviceJspFile(JspServlet.java:390)
at org.apache.jasper.servlet.JspServlet.service(JspServlet.java:334)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:727)
at 
org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:303)
at 
org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:208)
at 
org.apache.tomcat.websocket.server.WsFilter.doFilter(WsFilter.java:52)
at 
org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:241)
at 
org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:208)
at 
org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:220)
at 
org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:122)
at 
org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:501)
at 
org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:171)
at 
org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:102)
at 
org.apache.catalina.valves.AccessLogValve.invoke(AccessLogValve.java:950)
at 
org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:116)
at 
org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:408)
at 
org.apache.coyote.http11.AbstractHttp11Processor.process(AbstractHttp11Processor.java:1040)
at 
org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:607)
at 
org.apache.tomcat.util.net.JIoEndpoint$SocketProcessor.run(JIoEndpoint.java:314)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615

[GitHub] cloudstack pull request: adjust packaging for centos7

2015-10-30 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1008#issuecomment-152471048
  
Thanks for the fix and the PR, @davidamorimfaria 

I will try to test/review this one today.

Cheers,
Wilder


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread wido
Github user wido commented on the pull request:

https://github.com/apache/cloudstack/pull/1005#issuecomment-152462532
  
@DaanHoogland I don't like them either, but these paths are currently 
hardcoded in LibvirtComputingResource and I want to verify that they don't 
change and break some functionality.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8715: Add VirtIO channel to al...

2015-10-30 Thread ustcweizhou
Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/985#issuecomment-152462216
  
@wido by the way, I just remember I have implemented some codes for 
qemu-guest-agent support , based on cloudstack 4.2.0 maybe.
It is not fully tested. I will share that with you if you need (maybe 
create a pull request to your github branch)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8715: Add VirtIO channel to al...

2015-10-30 Thread ustcweizhou
Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/985#issuecomment-152461778
  
@wido ja, you got it. The issue happened on a host running with Ubuntu 
12.04 (QEMU 1.2.1 and libvirt 0.9.13)
There is no issue on Ubuntu 14.04 (QEMU 2.0.0 and libvirt 1.2.2)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8715: Add VirtIO channel to al...

2015-10-30 Thread wido
Github user wido commented on the pull request:

https://github.com/apache/cloudstack/pull/985#issuecomment-152460936
  
@ustcweizhou Which version of libvirt are you using?

If you use libvirt 1.0.6 or newer, you can omit the path='...' 
attribute of the  element, and libvirt will manage things automatically 
on your behalf.




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1005#issuecomment-152459236
  
very onky indeed.
code looks good, though I don't like the hard coded paths


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1005#discussion_r43480510
  
--- Diff: 
plugins/hypervisors/kvm/test/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtUtilitiesHelperTest.java
 ---
@@ -0,0 +1,52 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements.  See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership.  The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License.  You may obtain a copy of the License at
+//
+//   http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied.  See the License for the
+// specific language governing permissions and limitations
+// under the License.
+package com.cloud.hypervisor.kvm.resource.wrapper;
+
+import java.util.UUID;
+
+import com.cloud.utils.script.Script;
+
+import junit.framework.TestCase;
+
+public class LibvirtUtilitiesHelperTest extends TestCase {
+
+public void testGenerateUUID() {
+LibvirtUtilitiesHelper helper = new LibvirtUtilitiesHelper();
+UUID uuid = UUID.fromString(helper.generateUUIDName());
+assertEquals(4, uuid.version());
+}
+
+public void testSSHKeyPaths() {
+LibvirtUtilitiesHelper helper = new LibvirtUtilitiesHelper();
+assertEquals("/root/.ssh", helper.retrieveSshKeysPath());
--- End diff --

this is a unit test and will not 'onky' run on kvm. will it introduce the 
/tmp on windows issue anew?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: adjust packaging for centos7

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1008#issuecomment-152457639
  
@davidamorimfaria I saw you didn't go for the tomcat instance creation, 
right?
changes look good and as we discussed off-line.
Can you add a jira ticket name to the PR?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: adjust packaging for centos7

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1008#issuecomment-152457228
  
@davidamorimfaria due to a known bug in jenkins/maven integration the build 
in jenkins failed. Could you repush with git push --force to your branch? I'm 
reviewing in the meanwhile.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8928: While adding VMs to LB r...

2015-10-30 Thread nitin-maharana
Github user nitin-maharana commented on the pull request:

https://github.com/apache/cloudstack/pull/903#issuecomment-152456961
  
Hi @runseb @remibergsma I don't understand how to write a test for this. If 
you have any idea of how to write one, please help me out. 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 feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Re: Build fails on LocalTemplateDownloadTest

2015-10-30 Thread Wido den Hollander


On 29-10-15 15:36, Josh Harshman wrote:
> Build environment isn't Windows, it's Ubuntu 14.04 that I am getting this 
> build failure on.
> 

Is there something special about the /tmp on your system? Eg not enough
space for example.

Wido

> 
> From: Wido den Hollander 
> Sent: Thursday, October 29, 2015 1:50 AM
> To: dev@cloudstack.apache.org
> Subject: Re: Build fails on LocalTemplateDownloadTest
> 
> 
> 
> Josh Harshman
> Cloud Engineer
> Security+
> Intrinium
> 
> On 29-10-15 00:34, Josh Harshman wrote:
>> Is anyone else experiencing the same behavior?
>>
>> Building off master, get the following build failure:
>>
> 
> I think you are building on Windows?
> 
> I send a PR for this: https://github.com/apache/cloudstack/pull/884
> 
> This was reverted, but due to the code, but I merged it without the
> proper permissions.
> 
> We would have to merge this one in to fix it.
> 
> Wido
> 
>>
>>
>> ---snip---
>>
>> [INFO] Apache CloudStack Framework - IPC . SUCCESS [5.250s]
>> [INFO] Apache CloudStack Cloud Engine  SUCCESS [0.042s]
>> [INFO] Apache CloudStack Cloud Engine API  SUCCESS [5.073s]
>> [INFO] Apache CloudStack Framework - Security  SUCCESS [2.782s]
>> [INFO] Apache CloudStack Core  FAILURE [15.655s]
>> [INFO] Apache CloudStack Agents .. SKIPPED
>> [INFO] Apache CloudStack Framework - Clustering .. SKIPPED
>> ---snip---
>>
>>
>> com.cloud.storage.template.LocalTemplateDownloaderTest.txt:
>>
>> ---
>> Test set: com.cloud.storage.template.LocalTemplateDownloaderTest
>> ---
>> Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.028 sec 
>> <<< FAILURE! - in com.cloud.storage.template.LocalTemplateDownloaderTest
>> localTemplateDownloaderTest(com.cloud.storage.template.LocalTemplateDownloaderTest)
>>   Time elapsed: 0.026 sec  <<< FAILURE!
>> java.lang.AssertionError: Failed download
>> at org.junit.Assert.fail(Assert.java:88)
>> at 
>> com.cloud.storage.template.LocalTemplateDownloaderTest.localTemplateDownloaderTest(LocalTemplateDownloaderTest.java:37)
>>
>>
>>
>>
>> Josh Harshman
>>
>> Cloud Engineer
>> Security+
>>
>>
>> Intrinium
>> Tel: (509) 465-1234 x5259
>> Fax: (866) 565-4578
>> Lync / Skype: josh.harsh...@intrinium.com
>> Web: http://intrinium.com
>>
>>
>> [Intrinium Long Sig Logo]
>>
>>
>> [Facebook] 
>> [Twitter]   [Linkedin] 
>> 
>>   [Youtube]   [Blog] 
>> 
>>
>> Information Security and Compliance Consulting | Managed IT and Security 
>> Services | Cloud Services
>>
>>
>>
>>
>>
>> 
>>
>> This email and any files transmitted with it are confidential and intended 
>> solely for the use of the individual or entity to whom they are addressed. 
>> If you have received this email in error please notify the system manager. 
>> This message contains confidential information and is intended only for the 
>> individual named. If you are not the named addressee you should not 
>> disseminate, distribute or copy this e-mail. Please notify the sender 
>> immediately by e-mail if you have received this e-mail by mistake and delete 
>> this e-mail from your system. If you are not the intended recipient you are 
>> notified that disclosing, copying, distributing or taking any action in 
>> reliance on the contents of this information is strictly prohibited.
>>


Re: Build error

2015-10-30 Thread Remi Bergsma
Please propose it as a new PR and we will include it soon. Easy enough to 
review, like last time.



On 30/10/15 08:34, "Daan Hoogland"  wrote:

>@wido, is this one due to the temp dir path on windows thing as well?
>
>On Fri, Oct 30, 2015 at 7:55 AM, B Prakash  wrote:
>
>> I am new to this and hence might be missing something, please could
>> someone help me with this build error?  I am having the latest build and
>> getting the following error;
>>
>>
>> 2015-10-29 23:37:07,435 INFO  [utils.net.NetUtilsTest] (main:) IP is
>> 1234:5678::c3d9:fa56:870:77a3
>> 2015-10-29 23:37:07,560 ERROR [utils.net.NetUtils] (main:) Failed to
>> convert a string to an IPv6 address
>> java.lang.IllegalArgumentException: can not parse []
>> at
>> com.googlecode.ipv6.IPv6Address.tryParseStringArrayIntoLongArray(IPv6Address.java:94)
>> at com.googlecode.ipv6.IPv6Address.fromString(IPv6Address.java:80)
>> at com.cloud.utils.net.NetUtils.countIp6InRange(NetUtils.java:1316)
>> at
>> com.cloud.utils.net.NetUtilsTest.testCountIp6InRangeWithNullStart(NetUtilsTest.java:152)
>> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
>> at
>> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
>>
>>
>> I am using a windows 10 machine running on java 8, I tried build on both
>> cygwin and eclipse, getting similar error (I know it no different but just
>> to make sure till I get familiar).  Complete build(mvn install) log, just
>> in case this helps;
>>
>>
>> [INFO] Scanning for projects...
>> [INFO]
>> 
>> [INFO] Reactor Build Order:
>> [INFO]
>> [INFO] Apache CloudStack Developer Tools - Checkstyle Configuration
>> [INFO] Apache CloudStack
>> [INFO] Apache CloudStack Maven Conventions Parent
>> [INFO] Apache CloudStack Framework - Managed Context
>> [INFO] Apache CloudStack Utils
>> [INFO] Apache CloudStack Framework
>> [INFO] Apache CloudStack Framework - Event Notification
>> [INFO] Apache CloudStack Framework - Configuration
>> [INFO] Apache CloudStack API
>> [INFO] Apache CloudStack Framework - REST
>> [INFO] Apache CloudStack Framework - IPC
>> [INFO] Apache CloudStack Cloud Engine
>> [INFO] Apache CloudStack Cloud Engine API
>> [INFO] Apache CloudStack Framework - Security
>> [INFO] Apache CloudStack Core
>> [INFO] Apache CloudStack Agents
>> [INFO] Apache CloudStack Framework - Clustering
>> [INFO] Apache CloudStack Framework - Event Notification
>> [INFO] Apache CloudStack Cloud Engine Schema Component
>> [INFO] Apache CloudStack Framework - Jobs
>> [INFO] Apache CloudStack Cloud Engine Internal Components API
>> [INFO] Apache CloudStack Server
>> [INFO] Apache CloudStack Usage Server
>> [INFO] Apache CloudStack Cloud Engine Orchestration Component
>> [INFO] Apache CloudStack Cloud Services
>> [INFO] Apache CloudStack Secondary Storage
>> [INFO] Apache CloudStack Secondary Storage Service
>> [INFO] Apache CloudStack Engine Storage Component
>> [INFO] Apache CloudStack Engine Storage Volume Component
>> [INFO] Apache CloudStack Engine Storage Image Component
>> [INFO] Apache CloudStack Engine Storage Data Motion Component
>> [INFO] Apache CloudStack Engine Storage Cache Component
>> [INFO] Apache CloudStack Engine Storage Snapshot Component
>> [INFO] Apache CloudStack Cloud Engine API
>> [INFO] Apache CloudStack Cloud Engine Service
>> [INFO] Apache CloudStack Plugin POM
>> [INFO] Apache CloudStack Plugin - API Rate Limit
>> [INFO] Apache CloudStack Plugin - Storage Volume default provider
>> [INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider
>> [INFO] Apache CloudStack Plugin - API SolidFire
>> [INFO] Apache CloudStack Plugin - API Discovery
>> [INFO] Apache CloudStack Plugin - ACL Static Role Based
>> [INFO] Apache CloudStack Plugin - Host Anti-Affinity Processor
>> [INFO] Apache CloudStack Plugin - Explicit Dedication Processor
>> [INFO] Apache CloudStack Plugin - User Concentrated Pod Deployment Planner
>> [INFO] Apache CloudStack Plugin - User Dispersing Deployment Planner
>> [INFO] Apache CloudStack Plugin - Implicit Dedication Planner
>> [INFO] Apache CloudStack Plugin - Skip Heurestics Planner
>> [INFO] Apache CloudStack Plugin - Host Allocator Random
>> [INFO] Apache CloudStack Plugin - Dedicated Resources
>> [INFO] Apache CloudStack Plugin - Hypervisor OracleVM
>> [INFO] Apache CloudStack Plugin - Open vSwitch
>> [INFO] Apache CloudStack Plugin - Hypervisor XenServer
>> [INFO] Apache CloudStack Plugin - Hypervisor KVM
>> [INFO] Apache CloudStack Plugin - RabbitMQ Event Bus
>> [INFO] Apache CloudStack Plugin - In Memory Event Bus
>> [INFO] Apache CloudStack Plugin - Kafka Event Bus
>> [INFO] Apache CloudStack Plugin - Hypervisor Baremetal
>> [INFO] Apache CloudStack Plugin - Hypervisor UCS
>> [INFO] Apache CloudStack Plugin - Hypervisor Hyper-V
>> [INFO] Apache CloudStack Plugin - Hypervisor OracleVM3
>> [INFO] Apache CloudStack Plugin - Network Elastic Load Balancer
>> [INFO] Apache CloudStack Plug

[GitHub] cloudstack pull request: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread wido
Github user wido commented on the pull request:

https://github.com/apache/cloudstack/pull/1005#issuecomment-152453969
  
@borisroman Fixed, that was indeed the problem


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


  1   2   >