Re: KVM unit tests issue

2015-05-29 Thread Wilder Rodrigues
It was indeed fixed on libvirt:

https://github.com/schubergphilis/cloudstack/commit/982a0235a0143864e5b0eda1836fb913d91252f6

and it was merged 9 days ago:

https://github.com/schubergphilis/cloudstack/commit/982a0235a0143864e5b0eda1836fb913d91252f6

It seems another PR just got rid of my changes.

I will try to trace the root cause.

Cheers,
Wilder


On 29 May 2015, at 08:58, Wilder Rodrigues 
mailto:wrodrig...@schubergphilis.com>> wrote:

Sorry guys, I missed this email.

A couple of weeks ago I replaced many IPs by 127.0.0.1 in the tests. Missed 
that one.

Haven’t ran into this issue - not local, remotely or seen any complain on ASF 
builds. Reason for not reacting before.

Will fix.

Cheers,
Wilder


On 25 May 2015, at 16:01, Rafael Fonseca 
mailto:rsafons...@gmail.com>>
 wrote:

I am getting the same in my environments, can't you put that 
UnsatistfiedLinkError in a try/catch to let it not fail if the exception isn't 
thrown? :)

On Mon, May 25, 2015 at 3:55 PM, Rohit Yadav 
mailto:rohit.ya...@shapeblue.com>>
 wrote:
Hi Wilder,

In my environment, when building the kvm plugin on master branch I’m getting 
AssertionErrors instead of UnsatisfiedLinkError for the testMigrateCommand. The 
IP in use “10.1.1.100” is not available in my environment so tests are also 
taking a lot of time to execute. Is there a way to fix this? Can we use 
127.0.0.1 as IP?

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +91 88 262 30892 | 
rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: 
@_bhaisaab



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: KVM unit tests issue

2015-05-29 Thread Wilder Rodrigues
Sorry, that’s the PR:

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

Cheers,
Wilder

On 29 May 2015, at 09:03, Wilder Rodrigues 
mailto:wrodrig...@schubergphilis.com>> wrote:

It was indeed fixed on libvirt:

https://github.com/schubergphilis/cloudstack/commit/982a0235a0143864e5b0eda1836fb913d91252f6

and it was merged 9 days ago:

https://github.com/schubergphilis/cloudstack/commit/982a0235a0143864e5b0eda1836fb913d91252f6

It seems another PR just got rid of my changes.

I will try to trace the root cause.

Cheers,
Wilder


On 29 May 2015, at 08:58, Wilder Rodrigues 
mailto:wrodrig...@schubergphilis.com>> wrote:

Sorry guys, I missed this email.

A couple of weeks ago I replaced many IPs by 127.0.0.1 in the tests. Missed 
that one.

Haven’t ran into this issue - not local, remotely or seen any complain on ASF 
builds. Reason for not reacting before.

Will fix.

Cheers,
Wilder


On 25 May 2015, at 16:01, Rafael Fonseca 
mailto:rsafons...@gmail.com>>
 wrote:

I am getting the same in my environments, can't you put that 
UnsatistfiedLinkError in a try/catch to let it not fail if the exception isn't 
thrown? :)

On Mon, May 25, 2015 at 3:55 PM, Rohit Yadav 
mailto:rohit.ya...@shapeblue.com>>
 wrote:
Hi Wilder,

In my environment, when building the kvm plugin on master branch I’m getting 
AssertionErrors instead of UnsatisfiedLinkError for the testMigrateCommand. The 
IP in use “10.1.1.100” is not available in my environment so tests are also 
taking a lot of time to execute. Is there a way to fix this? Can we use 
127.0.0.1 as IP?

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +91 88 262 30892 | 
rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: 
@_bhaisaab



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: CLOUDSTACK-8515: Skipping snapshot test c...

2015-05-29 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request:

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

CLOUDSTACK-8515: Skipping snapshot test cases for HyperV and LXC

Skipping because snapshot not supported on HyperV and LXC.

Concurrent Snapshots ... SKIP: Snapshot is not supported on hyperv
Concurrent Snapshots ... SKIP: Snapshot is not supported on hyperv

--
Ran 2 tests in 1.157s

OK (SKIP=2)

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

$ git pull https://github.com/gauravaradhye/cloudstack 8515

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

https://github.com/apache/cloudstack/pull/324.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 #324


commit 893903c87dfe63bb16f9a982c6548859fb71170a
Author: Gaurav Aradhye 
Date:   2015-05-29T07:05:28Z

CLOUDSTACK-8515: Skipping snapshot test cases for HyperV and LXC




---
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: KVM unit tests issue

2015-05-29 Thread Wilder Rodrigues
Hi there again,

The file on master doesn’t contains my changes, as we already know:

https://github.com/apache/cloudstack/blob/master/plugins/hypervisors/kvm/test/com/cloud/hypervisor/kvm/resource/LibvirtComputingResourceTest.java


If you check the history, the changes were gone 7 days ago, on the 25th May:

https://github.com/apache/cloudstack/commits/master/plugins/hypervisors/kvm/test/com/cloud/hypervisor/kvm/resource/LibvirtComputingResourceTest.java

I will put them back.

Cheers,
Wilder


On 29 May 2015, at 09:04, Wilder Rodrigues 
mailto:wrodrig...@schubergphilis.com>> wrote:

Sorry, that’s the PR:

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

Cheers,
Wilder

On 29 May 2015, at 09:03, Wilder Rodrigues 
mailto:wrodrig...@schubergphilis.com>> wrote:

It was indeed fixed on libvirt:

https://github.com/schubergphilis/cloudstack/commit/982a0235a0143864e5b0eda1836fb913d91252f6

and it was merged 9 days ago:

https://github.com/schubergphilis/cloudstack/commit/982a0235a0143864e5b0eda1836fb913d91252f6

It seems another PR just got rid of my changes.

I will try to trace the root cause.

Cheers,
Wilder


On 29 May 2015, at 08:58, Wilder Rodrigues 
mailto:wrodrig...@schubergphilis.com>> wrote:

Sorry guys, I missed this email.

A couple of weeks ago I replaced many IPs by 127.0.0.1 in the tests. Missed 
that one.

Haven’t ran into this issue - not local, remotely or seen any complain on ASF 
builds. Reason for not reacting before.

Will fix.

Cheers,
Wilder


On 25 May 2015, at 16:01, Rafael Fonseca 
mailto:rsafons...@gmail.com>>
 wrote:

I am getting the same in my environments, can't you put that 
UnsatistfiedLinkError in a try/catch to let it not fail if the exception isn't 
thrown? :)

On Mon, May 25, 2015 at 3:55 PM, Rohit Yadav 
mailto:rohit.ya...@shapeblue.com>>
 wrote:
Hi Wilder,

In my environment, when building the kvm plugin on master branch I’m getting 
AssertionErrors instead of UnsatisfiedLinkError for the testMigrateCommand. The 
IP in use “10.1.1.100” is not available in my environment so tests are also 
taking a lot of time to execute. Is there a way to fix this? Can we use 
127.0.0.1 as IP?

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +91 88 262 30892 | 
rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: 
@_bhaisaab



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: CLOUDSTACK-8515: Skipping snapshot test c...

2015-05-29 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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: KVM unit tests issue

2015-05-29 Thread Wilder Rodrigues
That’s a bunch of emails within 10minutes. :)

I checked the branch again and found out that not all the IPs have been 
replaced. It means that you commit on the 25th was fine.

Will fix it now and merge.

Cheers,
Wilder


> On 29 May 2015, at 09:09, Wilder Rodrigues  
> wrote:
> 
> Hi there again,
> 
> The file on master doesn’t contains my changes, as we already know:
> 
> https://github.com/apache/cloudstack/blob/master/plugins/hypervisors/kvm/test/com/cloud/hypervisor/kvm/resource/LibvirtComputingResourceTest.java
> 
> 
> If you check the history, the changes were gone 7 days ago, on the 25th May:
> 
> https://github.com/apache/cloudstack/commits/master/plugins/hypervisors/kvm/test/com/cloud/hypervisor/kvm/resource/LibvirtComputingResourceTest.java
> 
> I will put them back.
> 
> Cheers,
> Wilder
> 
> 
> On 29 May 2015, at 09:04, Wilder Rodrigues 
> mailto:wrodrig...@schubergphilis.com>> wrote:
> 
> Sorry, that’s the PR:
> 
> https://github.com/apache/cloudstack/pull/270
> 
> Cheers,
> Wilder
> 
> On 29 May 2015, at 09:03, Wilder Rodrigues 
> mailto:wrodrig...@schubergphilis.com>> wrote:
> 
> It was indeed fixed on libvirt:
> 
> https://github.com/schubergphilis/cloudstack/commit/982a0235a0143864e5b0eda1836fb913d91252f6
> 
> and it was merged 9 days ago:
> 
> https://github.com/schubergphilis/cloudstack/commit/982a0235a0143864e5b0eda1836fb913d91252f6
> 
> It seems another PR just got rid of my changes.
> 
> I will try to trace the root cause.
> 
> Cheers,
> Wilder
> 
> 
> On 29 May 2015, at 08:58, Wilder Rodrigues 
> mailto:wrodrig...@schubergphilis.com>> wrote:
> 
> Sorry guys, I missed this email.
> 
> A couple of weeks ago I replaced many IPs by 127.0.0.1 in the tests. Missed 
> that one.
> 
> Haven’t ran into this issue - not local, remotely or seen any complain on ASF 
> builds. Reason for not reacting before.
> 
> Will fix.
> 
> Cheers,
> Wilder
> 
> 
> On 25 May 2015, at 16:01, Rafael Fonseca 
> mailto:rsafons...@gmail.com>>
>  wrote:
> 
> I am getting the same in my environments, can't you put that 
> UnsatistfiedLinkError in a try/catch to let it not fail if the exception 
> isn't thrown? :)
> 
> On Mon, May 25, 2015 at 3:55 PM, Rohit Yadav 
> mailto:rohit.ya...@shapeblue.com>>
>  wrote:
> Hi Wilder,
> 
> In my environment, when building the kvm plugin on master branch I’m getting 
> AssertionErrors instead of UnsatisfiedLinkError for the testMigrateCommand. 
> The IP in use “10.1.1.100” is not available in my environment so tests are 
> also taking a lot of time to execute. Is there a way to fix this? Can we use 
> 127.0.0.1 as IP?
> 
> Regards,
> Rohit Yadav
> Software Architect, ShapeBlue
> M. +91 88 262 30892 | 
> rohit.ya...@shapeblue.com
> Blog: bhaisaab.org | Twitter: 
> @_bhaisaab
> 
> 
> 
> 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: Swift storage

2015-05-29 Thread Andrija Panic
auth v1, old libraries, lot's of limitations (cant extract/download
ISO/template from ACS, dont see if register/download of ISO/template from
URL has started at all - no progress percentage, etc). Problems deleting
any ISO/template from GUI (at the moment, zoneid is or is not sent, in
contrast what it should be - also try for S3 - any region wide SS, afaik)

MGMT server mounts Secondary Storage on itself, and downloads systemVM
template from database records (so systemVM from download.cloud.com) - so
there is no preseed of systemVM template manually.

S3 interface (Swift can provide S3 support) seems to be working much better
- but we are still to test it all later...

:)





On 29 May 2015 at 08:09, prapul cool 
wrote:

> Hi,
> Has  any one added swift as secondary storage in cloudstack, if yes
> please share the instructions for it.
>How do we seed systemvm template in this case?
> Thanks,Prapul sriram,PSI cloud solutions.




-- 

Andrija Panić


[GitHub] cloudstack pull request: Replacing real IPs by 127.0.0.1.

2015-05-29 Thread wilderrodrigues
GitHub user wilderrodrigues opened a pull request:

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

Replacing real IPs by 127.0.0.1.

  - It was causing problems in some environments
  - The Ips should have been removed in a previous commit, but some of them 
were missed

@bhaisaab @rsafonseca 

There goes the PR with the IP changes you mentioned on the 25th May.

ACS builds successfully with Java 7 and 8 on MacOS and CentOS 7 machines.
Only unit tests were changed.

Cheers,
Wilder

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

$ git pull https://github.com/schubergphilis/cloudstack 
fix/libvirttest_remaining_ips

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

https://github.com/apache/cloudstack/pull/325.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 #325


commit 9ab2f5d5949bb2043dc875d795f92f45efab992f
Author: wilderrodrigues 
Date:   2015-05-29T07:36:19Z

Replacing real IPs by 127.0.0.1.
  - It was causing problems in some environments
  - The Ips should have been removed in a previous commit, but some of them 
were missed




---
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: KVM unit tests issue

2015-05-29 Thread Rohit Yadav
Thanks Wilder :)

> On 29-May-2015, at 9:18 am, Wilder Rodrigues  
> wrote:
>
> That’s a bunch of emails within 10minutes. :)
>
> I checked the branch again and found out that not all the IPs have been 
> replaced. It means that you commit on the 25th was fine.
>
> Will fix it now and merge.
>
> Cheers,
> Wilder
>
>
>> On 29 May 2015, at 09:09, Wilder Rodrigues  
>> wrote:
>>
>> Hi there again,
>>
>> The file on master doesn’t contains my changes, as we already know:
>>
>> https://github.com/apache/cloudstack/blob/master/plugins/hypervisors/kvm/test/com/cloud/hypervisor/kvm/resource/LibvirtComputingResourceTest.java
>>
>>
>> If you check the history, the changes were gone 7 days ago, on the 25th May:
>>
>> https://github.com/apache/cloudstack/commits/master/plugins/hypervisors/kvm/test/com/cloud/hypervisor/kvm/resource/LibvirtComputingResourceTest.java
>>
>> I will put them back.
>>
>> Cheers,
>> Wilder
>>
>>
>> On 29 May 2015, at 09:04, Wilder Rodrigues 
>> mailto:wrodrig...@schubergphilis.com>> wrote:
>>
>> Sorry, that’s the PR:
>>
>> https://github.com/apache/cloudstack/pull/270
>>
>> Cheers,
>> Wilder
>>
>> On 29 May 2015, at 09:03, Wilder Rodrigues 
>> mailto:wrodrig...@schubergphilis.com>> wrote:
>>
>> It was indeed fixed on libvirt:
>>
>> https://github.com/schubergphilis/cloudstack/commit/982a0235a0143864e5b0eda1836fb913d91252f6
>>
>> and it was merged 9 days ago:
>>
>> https://github.com/schubergphilis/cloudstack/commit/982a0235a0143864e5b0eda1836fb913d91252f6
>>
>> It seems another PR just got rid of my changes.
>>
>> I will try to trace the root cause.
>>
>> Cheers,
>> Wilder
>>
>>
>> On 29 May 2015, at 08:58, Wilder Rodrigues 
>> mailto:wrodrig...@schubergphilis.com>> wrote:
>>
>> Sorry guys, I missed this email.
>>
>> A couple of weeks ago I replaced many IPs by 127.0.0.1 in the tests. Missed 
>> that one.
>>
>> Haven’t ran into this issue - not local, remotely or seen any complain on 
>> ASF builds. Reason for not reacting before.
>>
>> Will fix.
>>
>> Cheers,
>> Wilder
>>
>>
>> On 25 May 2015, at 16:01, Rafael Fonseca 
>> mailto:rsafons...@gmail.com>>
>>  wrote:
>>
>> I am getting the same in my environments, can't you put that 
>> UnsatistfiedLinkError in a try/catch to let it not fail if the exception 
>> isn't thrown? :)
>>
>> On Mon, May 25, 2015 at 3:55 PM, Rohit Yadav 
>> mailto:rohit.ya...@shapeblue.com>>
>>  wrote:
>> Hi Wilder,
>>
>> In my environment, when building the kvm plugin on master branch I’m getting 
>> AssertionErrors instead of UnsatisfiedLinkError for the testMigrateCommand. 
>> The IP in use “10.1.1.100” is not available in my environment so tests are 
>> also taking a lot of time to execute. Is there a way to fix this? Can we use 
>> 127.0.0.1 as IP?
>>
>> Regards,
>> Rohit Yadav
>> Software Architect, ShapeBlue
>> M. +91 88 262 30892 | 
>> rohit.ya...@shapeblue.com
>> Blog: bhaisaab.org | Twitter: 
>> @_bhaisaab
>>
>>
>>
>> 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.
>>
>>
>>
>>
>>
>

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +91 88 262 30892 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab



Find out more about ShapeBlue and our range of CloudStack related services

IaaS Cloud Design & Build

[GitHub] cloudstack pull request: Replacing real IPs by 127.0.0.1.

2015-05-29 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/325#issuecomment-106740346
  
LGTM, I'll wait for Travis to go green just in case.


---
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: Replacing real IPs by 127.0.0.1.

2015-05-29 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/325#issuecomment-106740792
  
Yes, please wait for Trave.

Thanks, @bhaisaab 

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: Build and create tomcat, fix several othe...

2015-05-29 Thread rsafonseca
Github user rsafonseca commented on the pull request:

https://github.com/apache/cloudstack/pull/238#issuecomment-106741425
  
Hi @remibergsma 
First of all. thank you for your comment :)
I've been very busy these past couple of days, so i haven't gotten around 
to do what you suggested, but i will take care of that over the weekend (squash 
commits, clean description of fixes), I will also try to get a couple more 
stuff in as separate pull requests and clean up a couple more things ;)

As for testing this, there is no other way to test this other than manually 
since marvin doesn't even use tomcat for testing, altough the travis setup 
scripts download an old version of tomcat into the hard disk the tests do 
nothing with it :) That should also be cleaned up.
Another handy thing that's really missing in the tests is a test to 
packaging! In the past weeks a few things got into master that completely broke 
packaging.. although Rohit fixed most of that already, centos6 packaging is 
still broken.
As a side note, master also still doesn't work in Centos 7.1 and Fedora 
21.. that's all in here hehe ;)

If you guys would like, i can also alter the test scripts to use the same 
tomcat that will be used in the actual product instead of jetty, which will 
make for a test scenario closer to reality. This can be done in a way which 
will easily allow switching between tomcat and jetty in the future :)

What do you think of that?

BR,
Rafael




---
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: Build and create tomcat, fix several othe...

2015-05-29 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/238#issuecomment-106743307
  
Hi @rsafonseca 

I got your PR to build the RPMs and do some manual tests on it.

RPMs were created successfully! I then copied them into a new machine, 
brand new, and installed common/management RPMs there. When starting the 
Management server I got the following in the logs:

29-May-2015 03:38:41.413 WARNING [main] 
org.apache.catalina.startup.ClassLoaderFactory.validateFile Problem with 
directory [/usr/share/cloudstack-mysql-ha/lib/*jar], exists: [false], 
isDirectory: [false], canRead: [false]

29-May-2015 03:38:41.567 WARNING [main] 
org.apache.catalina.startup.Catalina.load Unable to load server configuration 
from [/usr/share/cloudstack-management/conf/server.xml]

29-May-2015 03:38:41.572 WARNING [main] 
org.apache.catalina.startup.Catalina.load Unable to load server configuration 
from [/usr/share/cloudstack-management/conf/server.xml]

29-May-2015 03:38:41.572 SEVERE [main] 
org.apache.catalina.startup.Catalina.start Cannot start server. Server instance 
is not configured.

Errors are clear for me: /usr/share/cloudstack-management/conf/server.xml  
and /usr/share/cloudstack-mysql-ha/lib/*jar don't exist. Are those some new 
stuff?

I already asked @remibergsma if he would know about it, but unfortunately 
he doesn't.

Do you have any hints on those ones? :)

Thanks a lot, dude!

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: Replacing real IPs by 127.0.0.1.

2015-05-29 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/325#issuecomment-106748023
  
@bhaisaab 

Everything seems happy and green. ;)

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


Re: Swift storage

2015-05-29 Thread Sanjeev N
No issues with S3 provider.

On Fri, May 29, 2015 at 12:48 PM, Andrija Panic 
wrote:

> auth v1, old libraries, lot's of limitations (cant extract/download
> ISO/template from ACS, dont see if register/download of ISO/template from
> URL has started at all - no progress percentage, etc). Problems deleting
> any ISO/template from GUI (at the moment, zoneid is or is not sent, in
> contrast what it should be - also try for S3 - any region wide SS, afaik)
>
> MGMT server mounts Secondary Storage on itself, and downloads systemVM
> template from database records (so systemVM from download.cloud.com) - so
> there is no preseed of systemVM template manually.
>
> S3 interface (Swift can provide S3 support) seems to be working much better
> - but we are still to test it all later...
>
> :)
>
>
>
>
>
> On 29 May 2015 at 08:09, prapul cool 
> wrote:
>
> > Hi,
> > Has  any one added swift as secondary storage in cloudstack, if yes
> > please share the instructions for it.
> >How do we seed systemvm template in this case?
> > Thanks,Prapul sriram,PSI cloud solutions.
>
>
>
>
> --
>
> Andrija Panić
>


[GitHub] cloudstack pull request: Build and create tomcat, fix several othe...

2015-05-29 Thread rsafonseca
Github user rsafonseca commented on the pull request:

https://github.com/apache/cloudstack/pull/238#issuecomment-106752449
  
Hi @wilderrodrigues 

Both the issues you hit are legacy behaviour :)

I'm guessing that you've deployed it in debian, because the server.xml 
issue is handled properly by the RPM installers, but not by the debian one.

To have it working after install on debian, just run the 
cloudstack-setup-management script, which i think is the standard way of 
setting things up, that's why I didn't bother to get the server.xml properly 
linked on package install in debian yet.

I've been thinking about doing something better (at least that's my 
opinion, please drop a comment on this) which is setup the management server to 
ALWAYS run with both HTTP and HTTPS right after setup (using generated cert 
until user provides one)

About that mysql-ha lib i guess it's one of the optionally packed 
libraries? i still haven done any testing on those. I just left this bit as it 
was.

Happy testing :)

RF


---
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: Swift storage

2015-05-29 Thread Andrija Panic
Nice to hear that :)

Sanjeev, is migration api supported if we use 2 nfs boxes now for secondary
storage or if it must be only 1 nfs box(that will ve converted to staging
store)?
Thx
On May 29, 2015 11:02 AM, "Sanjeev N"  wrote:

> No issues with S3 provider.
>
> On Fri, May 29, 2015 at 12:48 PM, Andrija Panic 
> wrote:
>
> > auth v1, old libraries, lot's of limitations (cant extract/download
> > ISO/template from ACS, dont see if register/download of ISO/template from
> > URL has started at all - no progress percentage, etc). Problems deleting
> > any ISO/template from GUI (at the moment, zoneid is or is not sent, in
> > contrast what it should be - also try for S3 - any region wide SS, afaik)
> >
> > MGMT server mounts Secondary Storage on itself, and downloads systemVM
> > template from database records (so systemVM from download.cloud.com) -
> so
> > there is no preseed of systemVM template manually.
> >
> > S3 interface (Swift can provide S3 support) seems to be working much
> better
> > - but we are still to test it all later...
> >
> > :)
> >
> >
> >
> >
> >
> > On 29 May 2015 at 08:09, prapul cool 
> > wrote:
> >
> > > Hi,
> > > Has  any one added swift as secondary storage in cloudstack, if yes
> > > please share the instructions for it.
> > >How do we seed systemvm template in this case?
> > > Thanks,Prapul sriram,PSI cloud solutions.
> >
> >
> >
> >
> > --
> >
> > Andrija Panić
> >
>


[GitHub] cloudstack pull request: Build and create tomcat, fix several othe...

2015-05-29 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/238#issuecomment-106755009
  
Hi @rsafonseca 

I built the RPM on CentOS7:

[root@cs5 ~]# uname -a
Linux cs5 3.10.0-229.1.2.el7.x86_64 #1 SMP Fri Mar 27 03:04:26 UTC 2015 
x86_64 x86_64 x86_64 GNU/Linux

[root@cs5 ~]# cat /etc/centos-release
CentOS Linux release 7.1.1503 (Core) 

I will try executing the cloudstack-setup-management script on my centOS as 
well.

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: Replacing real IPs by 127.0.0.1.

2015-05-29 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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: Build and create tomcat, fix several othe...

2015-05-29 Thread rsafonseca
Github user rsafonseca commented on the pull request:

https://github.com/apache/cloudstack/pull/238#issuecomment-106758023
  
Indeed.. it's only setup on centos 6 and not 7, sorry for misleading you 
there :)
cloudstack-setup-management will link the server.xml in any environment 
anyway.

The current version on master drops a server.xml in the folder (for Centos7 
only), but it doesn't support SSL. I just restored the behaviour to be the same 
as in other OSs (you get a non-ssl and ssl version and link is handled by the 
setup scripts).

Configuring SSL by default will remove the need to have a separate 
server-ssl.xml and server-nonssl.xml and will enable to just drop the 
server.xml config file in /etc/cloudstack/management .

The behaviour should be consistent across distros

Thanks for taking the time to test it out :)


---
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-8527: Skipping VPC tests on Hy...

2015-05-29 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request:

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

CLOUDSTACK-8527: Skipping VPC tests on HyperV

Skipping because VP is not supported on HyperV.

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

$ git pull https://github.com/gauravaradhye/cloudstack 8527

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

https://github.com/apache/cloudstack/pull/326.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 #326


commit 0d1820405a3ec1f7b41f0ba09ac2db503a4a9b40
Author: Gaurav Aradhye 
Date:   2015-05-29T09:43:28Z

CLOUDSTACK-8527: Skipping VPC tests on HyperV




---
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: Build and create tomcat, fix several othe...

2015-05-29 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/238#issuecomment-106760113
  
Hi @rsafonseca 

I executed the script then...

lrwxrwxrwx. 1 root root 44 May 29 05:39 server.xml -> 
/etc/cloudstack/management/server-nonssl.xml

[root@cs5 bin]# systemctl status cloudstack-management
cloudstack-management.service - CloudStack Management Server
   Loaded: loaded (/usr/lib/systemd/system/cloudstack-management.service; 
enabled)
   Active: active (running) since Fri 2015-05-29 05:39:16 EDT; 34s ago
  Process: 16149 ExecStart=/usr/sbin/cloudstack-management-sysd start 
(code=exited, status=0/SUCCESS)
 Main PID: 16164 (java)
   CGroup: /system.slice/cloudstack-management.service
   └─16164 /usr/lib/jvm/java/bin/java -Djava.awt.headless=true 
-Dcom.sun.management.jmxremote=false -Xmx2g -XX:+HeapDumpOnOutOfMemoryError 
-XX:HeapDumpPath=/var/log/cloudstack/management/ -XX:PermSize=512M 
-XX:MaxPermSize=800m -Djava.net.preferIPv4Stack=true -D...

May 29 05:39:16 cs5 systemd[1]: Starting CloudStack Management Server...
May 29 05:39:16 cs5 systemd[1]: Started CloudStack Management Server.

:)

I just have to configure the DB to see how it will behave.

Perhaps if you create a new PR, with the packaging stuff only, it would be 
easier to test and get it merged onto master. The steps to test the packaging 
are quite straight forward and we are glad to help on that.

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: CLOUDSTACK-8527: Skipping VPC tests on Hy...

2015-05-29 Thread asfgit
Github user asfgit closed the pull request at:

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


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


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

2015-05-29 Thread Andrija Panic
Hi guys,

anyone maybe looking into this issue ?
https://issues.apache.org/jira/browse/CLOUDSTACK-8289

Adding region is not working from UI - if there is any workarround - maybe
adding records to cloud.region table is enough ?


Thanks,

-- 

Andrija Panić


Re: [PROPOSAL] iSCSI and HA support in Hyper-V

2015-05-29 Thread Anshul Gangwar
Have updated the FS with more details.

Code is available for review at 
https://github.com/anshul1886/cloudstack-1/tree/hyperv-fccsv in hyperv-fccsv 
branch.

Regards,
Anshul

On 07-May-2015, at 4:07 pm, Anshul Gangwar 
mailto:anshul.gang...@citrix.com>> wrote:

Daan,
Its using existing framework. Its similar to VMware in implementation for HA of 
VMs and for CSV its using the existing PreSetup storage type. It has mainly the 
agent side changes which are specific to Hyper-V.

Regards,
Anshul

On 07-May-2015, at 2:28 pm, Daan Hoogland 
mailto:daan.hoogl...@gmail.com>>
 wrote:

Anshul,

Good thing to work on. Can you include parallels and references to other
hypervisors. I would not like to see this implemented in every hypervisor
in a different way. We are creating or rather extending technical debt if
we don't unify the way this is implemented across hypervisors.

cc'ing some known stakeholders

Op do 7 mei 2015 om 10:06 schreef Anshul Gangwar 
mailto:anshul.gang...@citrix.com>
:

Hi,

I would like to propose the iSCSI and HA support in Hyper-V feature. With
this feature we will be adding support for Failover clustering in Hyper-V
which will enable us to utilize the native HA and CSV support provided by
the Hyper-V. With addition of CSV we can support any storage option
including iSCSI which is supported by Hyper-V.

Please review FS for feature located at
https://cwiki.apache.org/confluence/display/CLOUDSTACK/iSCSI+and+HA+support+in+Hyper-V
and let me know your comments.

Regards,
Anshul






[GitHub] cloudstack pull request: CLOUDSTACK-8526: Use more memory for VM d...

2015-05-29 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request:

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

CLOUDSTACK-8526: Use more memory for VM deployment on HyperV when SSH is 
tried to VM

HyperV VMs are GUI based. So if SSH to VM is to be successful, they need at 
least 1 GB memory.

Log:
Positive test for stopped VM test path - T1 ... === TestName: 
test_01_pt_deploy_vm_without_startvm | Status : SUCCESS ===
ok

--
Ran 1 test in 2644.895s

OK

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

$ git pull https://github.com/gauravaradhye/cloudstack 8526

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

https://github.com/apache/cloudstack/pull/327.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 #327


commit 3c241362356617ba5e1d2207803d16aa7cc176a3
Author: Gaurav Aradhye 
Date:   2015-05-29T11:05:08Z

CLOUDSTACK-8526: Use more memory for VM deployment on HyperV when SSH is 
tried to VM




---
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-8526: Use more memory for VM d...

2015-05-29 Thread gauravaradhye
Github user gauravaradhye closed the pull request at:

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


---
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-8525: NPE while updating the s...

2015-05-29 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/321#issuecomment-106782174
  
@karuturi 

Will start testing now. :)


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


Shall we soon do a 4.5.2 release?

2015-05-29 Thread Remi Bergsma
Hi guys,

I've been playing with KVM on CentOS 7.1 and found it does not work well
with 4.5.1 due to a problem that Rohit already solved last week (thanks!).
See PR: https://github.com/apache/cloudstack/pull/290

Shall we do a new 4.5 release in the coming week or so? Then we can benefit
from some of the fixes that have been done.

Regards,
Remi


Re: Shall we soon do a 4.5.2 release?

2015-05-29 Thread Wido den Hollander
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1



On 05/29/2015 02:20 PM, Remi Bergsma wrote:
> Hi guys,
> 
> I've been playing with KVM on CentOS 7.1 and found it does not work
> well with 4.5.1 due to a problem that Rohit already solved last
> week (thanks!). See PR:
> https://github.com/apache/cloudstack/pull/290
> 
> Shall we do a new 4.5 release in the coming week or so? Then we can
> benefit from some of the fixes that have been done.
> 

Seems sane to me. Looking at the commits I see a lot of small fixes
from which users could really benefit.

Wido

> Regards, Remi
> 
-BEGIN PGP SIGNATURE-
Version: GnuPG v1

iQIcBAEBAgAGBQJVaFq7AAoJEAGbWC3bPspCVvAP/15xtyhXnhW8WeoLwUp3yurP
DwCchTw9WSQLO/NPK+ggdniuiP2vAeQQCSV8GUvzziO309elRDvaYR5tgrksRLzk
1Q4F4tKNxr1PE/pAYJ4eogLbBa1RXJLN+h7ILYxghRNaZWFozG+VlXDF4Twob2e6
VVyq6azBd7i9NQD9k1Z9WbTvxW32dKlGEU/L0R2I7sVdkvtc7bkMDVDLKJsLdTqs
bk+GswnUTjUQWLHVEH2/Hnla34x+0Vw/OOooMzbc6NkX0nLYnKvE85xrqJaZpIim
g639ahe/jntJsvayKwpJXduYRe4xmXd+W/tSJ09PP7ImhGSxgInwmp2cPdjWxA9l
0pmYlaAQ/+dRmiTER8aRsSmrYI334CCfkgzEe3WN+eMPH0WSE6iLVIWPasZQ/syH
RzekbUhFKRD8jb4e5n5ni6b7DRw+iIMlj1VqcPRo9cACYtrR4rdisgGQB/VYCLJx
HJJW/sodbcf+7lwYYVyZvG1p8XzLuMhJNaoA7y5Ll19xro9k5rSdPLnSP3meeUR1
6KhzRoWaoCNlCpzsYHKVQxCLeDjyvsa/ge2rSKR+Aj+aPkXCiylLxfAeKiJIQuPz
nc3WDKuwwbbpGSf7KkBat8QgNv0od1SOEkfPVtCXuGHkZ9F5JP/i/LqgX5Ms1viu
I6HqlDzIGvrGAmQy/byA
=tmHL
-END PGP SIGNATURE-


Re: Shall we soon do a 4.5.2 release?

2015-05-29 Thread Daan Hoogland
Remi, let's doe 4.4.4 this weekend first;) if no further issues come up
Or do we want a 4.5.2 irrespective and simultaneous?

Op vr 29 mei 2015 om 14:25 schreef Wido den Hollander :

> -BEGIN PGP SIGNED MESSAGE-
> Hash: SHA1
>
>
>
> On 05/29/2015 02:20 PM, Remi Bergsma wrote:
> > Hi guys,
> >
> > I've been playing with KVM on CentOS 7.1 and found it does not work
> > well with 4.5.1 due to a problem that Rohit already solved last
> > week (thanks!). See PR:
> > https://github.com/apache/cloudstack/pull/290
> >
> > Shall we do a new 4.5 release in the coming week or so? Then we can
> > benefit from some of the fixes that have been done.
> >
>
> Seems sane to me. Looking at the commits I see a lot of small fixes
> from which users could really benefit.
>
> Wido
>
> > Regards, Remi
> >
> -BEGIN PGP SIGNATURE-
> Version: GnuPG v1
>
> iQIcBAEBAgAGBQJVaFq7AAoJEAGbWC3bPspCVvAP/15xtyhXnhW8WeoLwUp3yurP
> DwCchTw9WSQLO/NPK+ggdniuiP2vAeQQCSV8GUvzziO309elRDvaYR5tgrksRLzk
> 1Q4F4tKNxr1PE/pAYJ4eogLbBa1RXJLN+h7ILYxghRNaZWFozG+VlXDF4Twob2e6
> VVyq6azBd7i9NQD9k1Z9WbTvxW32dKlGEU/L0R2I7sVdkvtc7bkMDVDLKJsLdTqs
> bk+GswnUTjUQWLHVEH2/Hnla34x+0Vw/OOooMzbc6NkX0nLYnKvE85xrqJaZpIim
> g639ahe/jntJsvayKwpJXduYRe4xmXd+W/tSJ09PP7ImhGSxgInwmp2cPdjWxA9l
> 0pmYlaAQ/+dRmiTER8aRsSmrYI334CCfkgzEe3WN+eMPH0WSE6iLVIWPasZQ/syH
> RzekbUhFKRD8jb4e5n5ni6b7DRw+iIMlj1VqcPRo9cACYtrR4rdisgGQB/VYCLJx
> HJJW/sodbcf+7lwYYVyZvG1p8XzLuMhJNaoA7y5Ll19xro9k5rSdPLnSP3meeUR1
> 6KhzRoWaoCNlCpzsYHKVQxCLeDjyvsa/ge2rSKR+Aj+aPkXCiylLxfAeKiJIQuPz
> nc3WDKuwwbbpGSf7KkBat8QgNv0od1SOEkfPVtCXuGHkZ9F5JP/i/LqgX5Ms1viu
> I6HqlDzIGvrGAmQy/byA
> =tmHL
> -END PGP SIGNATURE-
>


[GitHub] cloudstack pull request: Reinstate working sessions in browser

2015-05-29 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/308#issuecomment-106800322
  
Hi @rsafonseca thanks for the update and patch. It's a good idea to use 
HTTPOnly flag on cookie to ensure we have the authentication going on and at 
the same time not expose the cookie value to javascript. But the current patch 
will break SAML and any redirected authentication which relies on cookies being 
read by JS and then destroyed. As an alternative, let's discuss how we can 
improve this:

1. When a user is authenticated, we set the sessionkey cookie (httponly). 
This btw, only protects it from being read by JavaScript and not MITM attacks. 
For this, we can use a global setting I had added (isSecureSessionCookieEnabled 
method in ApiServer can be used for this) to enable secure flag on it as well. 
Please use Cookie class for this instead of adding raw strings like: 
resp.addCookie(new cookie(with flags etc enabled.)
2. In HttpUtils, add helper method:
+public static String findCookie(final Cookie[] cookies, String key) {
+for (Cookie cookie: cookies) {
+if (cookie.getName().equals(key)) {
+return cookie.getValue();
+}
+}
+return null;
+}
3. In ApiServlet, use the above method to find the cookie value (if 
available) when there is no sessionkey or empty sessionkey passed in the API 
params, check against session's sessionkey value to invalidate session. 
4. When users use default login method (login command), the login response 
still has the sessionkey. So, we can discuss to remove this as a security 
measure as well. If users are using an external logging system such as SAML 
based, usually the plugin would set the cookie. If the cookie becomes httponly 
(so JavaScript won't be able to read it), we will need to fix the JavaScript 
code such that it does not rely on sessionkey parameter at all.

Please review the above ideas and comment/share on them.

TLDR? To fix the login/UI issue, I think we need to not depend on passing a 
sessionkey as API argument but instead use HTTPOnly (and enable secure flag, so 
cookies are passed on HTTPs if enabled) "sessionkey" cookie along with the 
JSESSIONID cookie. Doing this would make sure we can run ACS UI across tabs and 
let the UI work when a external auth system such as those based on SAML, 
2-factor or OAuth can work.


---
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: [Feature] Cloudstack KVM with RBD

2015-05-29 Thread Logan Barfield
Wido,

I completely understand your position.  I hope I didn't give the
impression that I thought this was your responsibility.  I just
mentioned your name because you're more or less the "authority" on the
Ceph/Cloudstack integration components if anyone is.

As I mentioned before I think just changing the current method to
support thin snapshot images would help quite a bit.  I've already
determined that it's possible to do that by just not explicitly
stating the source image type in the snapshot code, and allowing
qemu-img to auto-detect it.  I just haven't had time to test it with
all of the supported versions of qemu yet.  If I can ever get through
my current work backlog I'm hoping to spend some more time working on
this stuff myself.  I don't think I can write "production quality"
code, but I can definitely help come up with proof of concept
implementations.

Thank You,

Logan Barfield
Tranquil Hosting


On Thu, May 28, 2015 at 9:15 PM, Star Guo  wrote:
> +1 , wait for it.
>
> Best Regards,
> Star Guo
>
> ===
>
> +2 :)
>
> On 28 May 2015 at 17:21, Andrei Mikhailovsky  wrote:
>
>> +1 for this
>> - Original Message -
>>
>> From: "Logan Barfield" 
>> To: dev@cloudstack.apache.org
>> Sent: Thursday, 28 May, 2015 3:48:09 PM
>> Subject: Re: [Feature] Cloudstack KVM with RBD
>>
>> Hi Star,
>>
>> I'll +1 this. I would like to see support for RBD snapshots as well,
>> and maybe have a method to "backup" the snapshots to secondary
>> storage. Right now for large volumes it can take an hour or more to
>> finish the snapshot.
>>
>> I have already discussed this with Wido, and was able to determine
>> that even without using native RBD snapshots we could improve the copy
>> time by saving the snaps as thin volumes instead of full raw files.
>> Right now the snapshot code when using RBD specifically converts the
>> volumes to a full raw file, when saving as a qcow2 image would use
>> less space. When restoring a snapshot the code current specifically
>> indicates the source image as being a raw file, but if we change the
>> code to not indicate the source image type qemu-img should
>> automatically detect it. We just need to see if that's the case with
>> all of the supported versions of libvirt/qemu before submitting a pull
>> request.
>>
>> Thank You,
>>
>> Logan Barfield
>> Tranquil Hosting
>>
>>
>> On Wed, May 27, 2015 at 9:18 PM, Star Guo  wrote:
>> > Hi everyone,
>> >
>> > Since I have test cloudstack 4.4.2 + kvm + rbd, deploy an instance
>> > is so fast apart from the first deployment because copy template
>> > from secondary storage (NFS) to primary storage (RBD). That is no problem.
>> > However, when I do some volume operation, such as create snapshot,
>> > create template, template deploy ect, it also take some time to
>> > finish because
>> copy
>> > data between primary storage and secondary storage.
>> > So I think that if we support the same rbd as secondary storage, and
>> > use ceph COW feature, it may reduce the time and just some seconds.
>> (OpenStack
>> > can make glance and cinder as the same rbd)
>> >
>> > Best Regards,
>> > Star Guo
>> >
>>
>>
>
>
> --
>
> Andrija Panić
>


Re: Shall we soon do a 4.5.2 release?

2015-05-29 Thread Remi Bergsma
That's OK Daan. Let's get some final patches in (if any) before we'll
create a RC for 4.5.2 somewhere next week. In the mean time we'll work to
get 4.4.4 out.

Regards,
Remi

2015-05-29 15:00 GMT+02:00 Daan Hoogland :

> Remi, let's doe 4.4.4 this weekend first;) if no further issues come up
> Or do we want a 4.5.2 irrespective and simultaneous?
>
> Op vr 29 mei 2015 om 14:25 schreef Wido den Hollander :
>
> > -BEGIN PGP SIGNED MESSAGE-
> > Hash: SHA1
> >
> >
> >
> > On 05/29/2015 02:20 PM, Remi Bergsma wrote:
> > > Hi guys,
> > >
> > > I've been playing with KVM on CentOS 7.1 and found it does not work
> > > well with 4.5.1 due to a problem that Rohit already solved last
> > > week (thanks!). See PR:
> > > https://github.com/apache/cloudstack/pull/290
> > >
> > > Shall we do a new 4.5 release in the coming week or so? Then we can
> > > benefit from some of the fixes that have been done.
> > >
> >
> > Seems sane to me. Looking at the commits I see a lot of small fixes
> > from which users could really benefit.
> >
> > Wido
> >
> > > Regards, Remi
> > >
> > -BEGIN PGP SIGNATURE-
> > Version: GnuPG v1
> >
> > iQIcBAEBAgAGBQJVaFq7AAoJEAGbWC3bPspCVvAP/15xtyhXnhW8WeoLwUp3yurP
> > DwCchTw9WSQLO/NPK+ggdniuiP2vAeQQCSV8GUvzziO309elRDvaYR5tgrksRLzk
> > 1Q4F4tKNxr1PE/pAYJ4eogLbBa1RXJLN+h7ILYxghRNaZWFozG+VlXDF4Twob2e6
> > VVyq6azBd7i9NQD9k1Z9WbTvxW32dKlGEU/L0R2I7sVdkvtc7bkMDVDLKJsLdTqs
> > bk+GswnUTjUQWLHVEH2/Hnla34x+0Vw/OOooMzbc6NkX0nLYnKvE85xrqJaZpIim
> > g639ahe/jntJsvayKwpJXduYRe4xmXd+W/tSJ09PP7ImhGSxgInwmp2cPdjWxA9l
> > 0pmYlaAQ/+dRmiTER8aRsSmrYI334CCfkgzEe3WN+eMPH0WSE6iLVIWPasZQ/syH
> > RzekbUhFKRD8jb4e5n5ni6b7DRw+iIMlj1VqcPRo9cACYtrR4rdisgGQB/VYCLJx
> > HJJW/sodbcf+7lwYYVyZvG1p8XzLuMhJNaoA7y5Ll19xro9k5rSdPLnSP3meeUR1
> > 6KhzRoWaoCNlCpzsYHKVQxCLeDjyvsa/ge2rSKR+Aj+aPkXCiylLxfAeKiJIQuPz
> > nc3WDKuwwbbpGSf7KkBat8QgNv0od1SOEkfPVtCXuGHkZ9F5JP/i/LqgX5Ms1viu
> > I6HqlDzIGvrGAmQy/byA
> > =tmHL
> > -END PGP SIGNATURE-
> >
>


[GitHub] cloudstack pull request: Replacing real IPs by 127.0.0.1.

2015-05-29 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/325#issuecomment-106811796
  
@wilderrodrigues you bet :)


---
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: Shall we soon do a 4.5.2 release?

2015-05-29 Thread Rohit Yadav
Hi,

I think let’s do 4.4.4 first to ensure upgrade paths in 4.5.2, master etc.

I’ve identified few more issues in 4.5.1 that need to be fixed so I'm planning 
to drive efforts into releasing 4.5.2 in about 1-2 weeks from now when I think 
we would have made those fixes related to KVM, VMWare, UI and Login/security. 
There are some interesting open PRs that I would like to see in 4.5.2.

> On 29-May-2015, at 3:00 pm, Daan Hoogland  wrote:
>
> Remi, let's doe 4.4.4 this weekend first;) if no further issues come up
> Or do we want a 4.5.2 irrespective and simultaneous?
>
> Op vr 29 mei 2015 om 14:25 schreef Wido den Hollander :
>
>> -BEGIN PGP SIGNED MESSAGE-
>> Hash: SHA1
>>
>>
>>
>> On 05/29/2015 02:20 PM, Remi Bergsma wrote:
>>> Hi guys,
>>>
>>> I've been playing with KVM on CentOS 7.1 and found it does not work
>>> well with 4.5.1 due to a problem that Rohit already solved last
>>> week (thanks!). See PR:
>>> https://github.com/apache/cloudstack/pull/290
>>>
>>> Shall we do a new 4.5 release in the coming week or so? Then we can
>>> benefit from some of the fixes that have been done.
>>>
>>
>> Seems sane to me. Looking at the commits I see a lot of small fixes
>> from which users could really benefit.
>>
>> Wido
>>
>>> Regards, Remi
>>>
>> -BEGIN PGP SIGNATURE-
>> Version: GnuPG v1
>>
>> iQIcBAEBAgAGBQJVaFq7AAoJEAGbWC3bPspCVvAP/15xtyhXnhW8WeoLwUp3yurP
>> DwCchTw9WSQLO/NPK+ggdniuiP2vAeQQCSV8GUvzziO309elRDvaYR5tgrksRLzk
>> 1Q4F4tKNxr1PE/pAYJ4eogLbBa1RXJLN+h7ILYxghRNaZWFozG+VlXDF4Twob2e6
>> VVyq6azBd7i9NQD9k1Z9WbTvxW32dKlGEU/L0R2I7sVdkvtc7bkMDVDLKJsLdTqs
>> bk+GswnUTjUQWLHVEH2/Hnla34x+0Vw/OOooMzbc6NkX0nLYnKvE85xrqJaZpIim
>> g639ahe/jntJsvayKwpJXduYRe4xmXd+W/tSJ09PP7ImhGSxgInwmp2cPdjWxA9l
>> 0pmYlaAQ/+dRmiTER8aRsSmrYI334CCfkgzEe3WN+eMPH0WSE6iLVIWPasZQ/syH
>> RzekbUhFKRD8jb4e5n5ni6b7DRw+iIMlj1VqcPRo9cACYtrR4rdisgGQB/VYCLJx
>> HJJW/sodbcf+7lwYYVyZvG1p8XzLuMhJNaoA7y5Ll19xro9k5rSdPLnSP3meeUR1
>> 6KhzRoWaoCNlCpzsYHKVQxCLeDjyvsa/ge2rSKR+Aj+aPkXCiylLxfAeKiJIQuPz
>> nc3WDKuwwbbpGSf7KkBat8QgNv0od1SOEkfPVtCXuGHkZ9F5JP/i/LqgX5Ms1viu
>> I6HqlDzIGvrGAmQy/byA
>> =tmHL
>> -END PGP SIGNATURE-
>>

Regards,
Rohit Yadav
Software Architect, ShapeBlue
M. +91 88 262 30892 | rohit.ya...@shapeblue.com
Blog: bhaisaab.org | Twitter: @_bhaisaab



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: Shall we soon do a 4.5.2 release?

2015-05-29 Thread Andrija Panic
Wido, do we have support for RBD resize currently in 4.5 branch ? I tested
4.5.1 tag, and it didnt work - Im talking here about DATA volume resize,
while VM is off...
2015-05-28 16:14:56,257 DEBUG [cloud.agent.Agent]
(agentRequest-Handler-1:null) Seq 1-4646870390516222369:  { Ans: , MgmtId:
90520740254323, via: 1, Ver: v1, Flags: 10, [{"com.cloud.agent.api.
storage.ResizeVolumeAnswer":{"newSize":0,"result":false,"details":"Failed
to resize volume: Cannot determine resize type from pool type RBD","wait":0}}]
}

Sorry for offtopic...


On 29 May 2015 at 15:45, Remi Bergsma  wrote:

> That's OK Daan. Let's get some final patches in (if any) before we'll
> create a RC for 4.5.2 somewhere next week. In the mean time we'll work to
> get 4.4.4 out.
>
> Regards,
> Remi
>
> 2015-05-29 15:00 GMT+02:00 Daan Hoogland :
>
> > Remi, let's doe 4.4.4 this weekend first;) if no further issues come
> up
> > Or do we want a 4.5.2 irrespective and simultaneous?
> >
> > Op vr 29 mei 2015 om 14:25 schreef Wido den Hollander :
> >
> > > -BEGIN PGP SIGNED MESSAGE-
> > > Hash: SHA1
> > >
> > >
> > >
> > > On 05/29/2015 02:20 PM, Remi Bergsma wrote:
> > > > Hi guys,
> > > >
> > > > I've been playing with KVM on CentOS 7.1 and found it does not work
> > > > well with 4.5.1 due to a problem that Rohit already solved last
> > > > week (thanks!). See PR:
> > > > https://github.com/apache/cloudstack/pull/290
> > > >
> > > > Shall we do a new 4.5 release in the coming week or so? Then we can
> > > > benefit from some of the fixes that have been done.
> > > >
> > >
> > > Seems sane to me. Looking at the commits I see a lot of small fixes
> > > from which users could really benefit.
> > >
> > > Wido
> > >
> > > > Regards, Remi
> > > >
> > > -BEGIN PGP SIGNATURE-
> > > Version: GnuPG v1
> > >
> > > iQIcBAEBAgAGBQJVaFq7AAoJEAGbWC3bPspCVvAP/15xtyhXnhW8WeoLwUp3yurP
> > > DwCchTw9WSQLO/NPK+ggdniuiP2vAeQQCSV8GUvzziO309elRDvaYR5tgrksRLzk
> > > 1Q4F4tKNxr1PE/pAYJ4eogLbBa1RXJLN+h7ILYxghRNaZWFozG+VlXDF4Twob2e6
> > > VVyq6azBd7i9NQD9k1Z9WbTvxW32dKlGEU/L0R2I7sVdkvtc7bkMDVDLKJsLdTqs
> > > bk+GswnUTjUQWLHVEH2/Hnla34x+0Vw/OOooMzbc6NkX0nLYnKvE85xrqJaZpIim
> > > g639ahe/jntJsvayKwpJXduYRe4xmXd+W/tSJ09PP7ImhGSxgInwmp2cPdjWxA9l
> > > 0pmYlaAQ/+dRmiTER8aRsSmrYI334CCfkgzEe3WN+eMPH0WSE6iLVIWPasZQ/syH
> > > RzekbUhFKRD8jb4e5n5ni6b7DRw+iIMlj1VqcPRo9cACYtrR4rdisgGQB/VYCLJx
> > > HJJW/sodbcf+7lwYYVyZvG1p8XzLuMhJNaoA7y5Ll19xro9k5rSdPLnSP3meeUR1
> > > 6KhzRoWaoCNlCpzsYHKVQxCLeDjyvsa/ge2rSKR+Aj+aPkXCiylLxfAeKiJIQuPz
> > > nc3WDKuwwbbpGSf7KkBat8QgNv0od1SOEkfPVtCXuGHkZ9F5JP/i/LqgX5Ms1viu
> > > I6HqlDzIGvrGAmQy/byA
> > > =tmHL
> > > -END PGP SIGNATURE-
> > >
> >
>



-- 

Andrija Panić


[GitHub] cloudstack pull request: Reinstate working sessions in browser

2015-05-29 Thread rsafonseca
Github user rsafonseca commented on the pull request:

https://github.com/apache/cloudstack/pull/308#issuecomment-106818248
  
Hi Rohit, thank you for your view on this :)
Here's my comment on it:

1. No use on protecting the sessionkey from MITM attacks, as the 
username/password are sent in plain text, and so are all the other commands, so 
anyone can pickup your credentials if they're listening on the wire ;). For 
MITM protection SSL should be used. I did notice that 
isSecureSessionCookieEnabled i actually placed the cookie in raw to be exactly 
the same as the sessioncookie. I did use the cookie class for reading though 
:). So.. both cookies should be inserted in the same fashion.

2. Will take care of that :)

3. I miss the purpose.. session cleanup? Can you explain a bit more? :)

4. If you don't return it in the loginResponse, regular (non-browser) calls 
to the API will fail, because i guess all integrations with the API depend on 
it, that's why i didn't remove it. It can be in the response, as long as 
javascript doesn't store it for later access.

Regarding the non-bulleted stuff:

Secure cookie:
I don't agree with this one, that will make it just not work at all in 
HTTP. We aim to fix a browser vulnerability here, not a communication channel 
one. If someone wants on the wire security, they should just switch to SSL.

Not depend on sessionkey as API argument:
Will that not break some API implementations? I think supporting both 
methods is the best for now.

About SAML and redirect based auth not working:
Are you sure? Did you test? I left it in a way it would likely work... 
cookie is still read an put into the javascript variable if it was created on 
redirection. Since the SAML plugin just puts it back in a cookie so i didn't 
bother destroying it in that scenario, the plugin should be modded for that. 
Either way.. it *should* work :)

Thanks for reviewing.



---
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: Custom validation of a custom config value

2015-05-29 Thread Mike Tutkowski
Thanks

On Friday, May 29, 2015, Rajani Karuturi  wrote:

> we do not have validation framework for the configs. We only do simple
> validation of datatype check ie) if its boolean, parseBoolean etc.
>
> ~Rajani
>
> On Fri, May 29, 2015 at 10:54 AM, Mike Tutkowski <
> mike.tutkow...@solidfire.com > wrote:
>
> > Hi,
> >
> > If you have a plug-in that adds a custom config value and an admin
> updates
> > that config value, is there a way to have custom validation logic
> invoked?
> >
> > For example, let's say you want to add a custom config value that's
> > associated with each and every account (ex. total.miniops). If the admin
> > changes this value on an account, I'd like to be able to validate that
> the
> > change is OK (ex. that you didn't reduce total.miniops to an amount that
> > puts it under what has already been allocated by users of the account).
> >
> > Thanks,
> > Mike
> >
> > --
> > *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
*™*


Re: Swift storage

2015-05-29 Thread Sanjeev N
yes all the existing nfs storages will be converted to staging storage
after migrating from nfs to s3.

On Fri, May 29, 2015 at 2:50 PM, Andrija Panic 
wrote:

> Nice to hear that :)
>
> Sanjeev, is migration api supported if we use 2 nfs boxes now for secondary
> storage or if it must be only 1 nfs box(that will ve converted to staging
> store)?
> Thx
> On May 29, 2015 11:02 AM, "Sanjeev N"  wrote:
>
> > No issues with S3 provider.
> >
> > On Fri, May 29, 2015 at 12:48 PM, Andrija Panic  >
> > wrote:
> >
> > > auth v1, old libraries, lot's of limitations (cant extract/download
> > > ISO/template from ACS, dont see if register/download of ISO/template
> from
> > > URL has started at all - no progress percentage, etc). Problems
> deleting
> > > any ISO/template from GUI (at the moment, zoneid is or is not sent, in
> > > contrast what it should be - also try for S3 - any region wide SS,
> afaik)
> > >
> > > MGMT server mounts Secondary Storage on itself, and downloads systemVM
> > > template from database records (so systemVM from download.cloud.com) -
> > so
> > > there is no preseed of systemVM template manually.
> > >
> > > S3 interface (Swift can provide S3 support) seems to be working much
> > better
> > > - but we are still to test it all later...
> > >
> > > :)
> > >
> > >
> > >
> > >
> > >
> > > On 29 May 2015 at 08:09, prapul cool  >
> > > wrote:
> > >
> > > > Hi,
> > > > Has  any one added swift as secondary storage in cloudstack, if
> yes
> > > > please share the instructions for it.
> > > >How do we seed systemvm template in this case?
> > > > Thanks,Prapul sriram,PSI cloud solutions.
> > >
> > >
> > >
> > >
> > > --
> > >
> > > Andrija Panić
> > >
> >
>


Re: Swift storage

2015-05-29 Thread Andrija Panic
Interesting so its suported to have 2 staging nfs. Great.

Thx Sanjeev
On May 29, 2015 5:32 PM, "Sanjeev N"  wrote:

> yes all the existing nfs storages will be converted to staging storage
> after migrating from nfs to s3.
>
> On Fri, May 29, 2015 at 2:50 PM, Andrija Panic 
> wrote:
>
> > Nice to hear that :)
> >
> > Sanjeev, is migration api supported if we use 2 nfs boxes now for
> secondary
> > storage or if it must be only 1 nfs box(that will ve converted to staging
> > store)?
> > Thx
> > On May 29, 2015 11:02 AM, "Sanjeev N"  wrote:
> >
> > > No issues with S3 provider.
> > >
> > > On Fri, May 29, 2015 at 12:48 PM, Andrija Panic <
> andrija.pa...@gmail.com
> > >
> > > wrote:
> > >
> > > > auth v1, old libraries, lot's of limitations (cant extract/download
> > > > ISO/template from ACS, dont see if register/download of ISO/template
> > from
> > > > URL has started at all - no progress percentage, etc). Problems
> > deleting
> > > > any ISO/template from GUI (at the moment, zoneid is or is not sent,
> in
> > > > contrast what it should be - also try for S3 - any region wide SS,
> > afaik)
> > > >
> > > > MGMT server mounts Secondary Storage on itself, and downloads
> systemVM
> > > > template from database records (so systemVM from download.cloud.com)
> -
> > > so
> > > > there is no preseed of systemVM template manually.
> > > >
> > > > S3 interface (Swift can provide S3 support) seems to be working much
> > > better
> > > > - but we are still to test it all later...
> > > >
> > > > :)
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > On 29 May 2015 at 08:09, prapul cool
>  > >
> > > > wrote:
> > > >
> > > > > Hi,
> > > > > Has  any one added swift as secondary storage in cloudstack, if
> > yes
> > > > > please share the instructions for it.
> > > > >How do we seed systemvm template in this case?
> > > > > Thanks,Prapul sriram,PSI cloud solutions.
> > > >
> > > >
> > > >
> > > >
> > > > --
> > > >
> > > > Andrija Panić
> > > >
> > >
> >
>


Re: [RFC] SAML2 plugin improvements

2015-05-29 Thread Rohit Yadav
Hi,

Just want to share that SAML plugin now supports HTTP-POST and HTTP-Redirect 
bindings and in my local setup it seems to be working with Shibboleth and also 
with SSOCircle, OpenFiede and TestShib: 
https://github.com/apache/cloudstack/commits/saml-production-grade

Erik - the current SAML implementation aims to support SAML v2.0 spec and 
qualify samlint.org compatibility guide. I’m not sure about ADFS and how 
compatible it is with SAML 2.0. In theory, you can also use Microsoft AD with 
Shibboleth. If you plan on using it with ADFS, can you test this branch and 
suggest errors you encounter?

I’m testing with available opensource IdPs such as Shibboleth IdP and public 
IdP servers such as SSOCircle, OpenFiede, OneLogin etc. In case you're planning 
to use the SAML auth plugin in your environment, please share your feedback and 
use-cases so the plugin implementation can be more general purpose and support 
a wide variety of IdP servers. Thanks.

> On 12-May-2015, at 10:02 pm, Erik Weber  wrote:
>
> I don't actually remember the specifics, and I've scratched the lab.
>
> But I think there was an issue with fetching the metadata (from the IdP)
> atleast.
>
> Plus, ADFS is claims based, I don't know if the current SAML 2.0
> implementation in CloudStack is claims aware or not?
>
>
> --
> Erik
>
> On Tue, May 12, 2015 at 9:46 PM, Rohit Yadav 
> wrote:
>
>> Hi Erik,
>>
>> Thanks for your feedback, can you share more details about your use-case.
>> I remember we had a discussion where we tried to make it work, but don’t
>> remember why it failed for your environment. What SAML bindings do we need
>> to support to make it work with MS ADFS any other subtle details?
>>
>>> On 12-May-2015, at 6:33 pm, Erik Weber  wrote:
>>>
>>> Great news Rohit,
>>>
>>> Would love to see it support Microsoft ADFS as IdP.
>>>
>>>
>>> Erik
>>>
>>> Den tirsdag 12. mai 2015 skrev Rohit Yadav 
>>> følgende:
>>>
 Hi all,

 Based on the feedback several friends in the community on different
 use-cases of using a federated login system based on SAML2 with
>> CloudStack,
 I’m soon planning to address them in the SAML plugin implement focusing
>> on
 pain points around interoperability, IdP support, security and ease of
>> use.

 I’ve updated the specification to reflect some of those aspects I’ve
 gathered in last few months:


>> https://cwiki.apache.org/confluence/display/CLOUDSTACK/SAML+2.0+Plugin#SAML2.0Plugin-Version2:InProgress

 Please advise improvements you would like to see, or share pain points
 with the current implementation. Thanks.

 Regards,
 Rohit Yadav
 Software Architect, ShapeBlue
 M. +91 88 262 30892 | rohit.ya...@shapeblue.com 
 Blog: bhaisaab.org | Twitter: @_bhaisaab



 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.

>>
>> Regards,
>> Rohit Yadav
>> Software Architect, ShapeBlue
>> M. +91 88 262 30892 | rohit.ya...@shapeblue.com
>> Blog: bhaisaab.org | Twitter: @_bhaisaab
>>
>>
>>
>> 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 Eng

[GitHub] cloudstack pull request: Cleanup awsapi leftovers

2015-05-29 Thread rsafonseca
GitHub user rsafonseca opened a pull request:

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

Cleanup awsapi leftovers

Just a couple of things that were still in the code :)

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

$ git pull https://github.com/rsafonseca/cloudstack clean-awsapi

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

https://github.com/apache/cloudstack/pull/328.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 #328


commit 78828188d126213abc8fe26c4b1fa04ed75227b2
Author: Rafael da Fonseca 
Date:   2015-05-29T17:02:55Z

Cleanup awsapi leftovers




---
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: Cleanup awsapi leftovers

2015-05-29 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/328#issuecomment-106874647
  
I swear I did remove them, probably some merge issue. LGTM, merging now.


---
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: Cleanup awsapi leftovers

2015-05-29 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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: Cleanup awsapi leftovers

2015-05-29 Thread rsafonseca
Github user rsafonseca commented on the pull request:

https://github.com/apache/cloudstack/pull/328#issuecomment-106875402
  
If if comes back we will just kill it again :)


---
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: [Feature] Cloudstack KVM with RBD

2015-05-29 Thread Gerolamo Valcamonica

Il 28/05/2015 03:18, Star Guo ha scritto:

However, when I do some volume operation, such as create snapshot, create
template, template deploy ect, it also take some time to finish because copy
data between primary storage and secondary storage.
So I think that if we support the same rbd as secondary storage, and use
ceph COW feature, it may reduce the time and just some seconds. (OpenStack
can make glance and cinder as the same rbd)


I think that the issue is related to this, too:
https://issues.apache.org/jira/browse/CLOUDSTACK-4858
(marked as Unresolved)

--
Gerolamo Valcamonica

--
Gerolamo Valcamonica



[GitHub] cloudstack pull request: Fix agent/management conflict

2015-05-29 Thread rsafonseca
GitHub user rsafonseca opened a pull request:

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

Fix agent/management conflict

There is an entry for the agent log directory in the management packaging, 
creating a conflict between the two RPMs.
This issue occurs in both 4.4.x and 4.5.x
This fixes not being able to install cloudstack-agent and 
cloudstack-management in the same machine in CentOS.

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

$ git pull https://github.com/rsafonseca/cloudstack 45_rpmconflict

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

https://github.com/apache/cloudstack/pull/329.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 #329


commit 1138f54c323a5340b426a15f8efeb293bc738c2d
Author: Rafael da Fonseca 
Date:   2015-05-29T17:55:32Z

Fix agent/management conflict




---
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: Shall we soon do a 4.5.2 release?

2015-05-29 Thread Daan Hoogland
making a new rc as I don't have time the rest of the weekend. no problem if
I need to do another. vote thread coming up

Op vr 29 mei 2015 om 16:00 schreef Andrija Panic :

> Wido, do we have support for RBD resize currently in 4.5 branch ? I tested
> 4.5.1 tag, and it didnt work - Im talking here about DATA volume resize,
> while VM is off...
> 2015-05-28 16:14:56,257 DEBUG [cloud.agent.Agent]
> (agentRequest-Handler-1:null) Seq 1-4646870390516222369:  { Ans: , MgmtId:
> 90520740254323, via: 1, Ver: v1, Flags: 10, [{"com.cloud.agent.api.
> storage.ResizeVolumeAnswer":{"newSize":0,"result":false,"details":"Failed
> to resize volume: Cannot determine resize type from pool type
> RBD","wait":0}}]
> }
>
> Sorry for offtopic...
>
>
> On 29 May 2015 at 15:45, Remi Bergsma  wrote:
>
> > That's OK Daan. Let's get some final patches in (if any) before we'll
> > create a RC for 4.5.2 somewhere next week. In the mean time we'll work to
> > get 4.4.4 out.
> >
> > Regards,
> > Remi
> >
> > 2015-05-29 15:00 GMT+02:00 Daan Hoogland :
> >
> > > Remi, let's doe 4.4.4 this weekend first;) if no further issues come
> > up
> > > Or do we want a 4.5.2 irrespective and simultaneous?
> > >
> > > Op vr 29 mei 2015 om 14:25 schreef Wido den Hollander  >:
> > >
> > > > -BEGIN PGP SIGNED MESSAGE-
> > > > Hash: SHA1
> > > >
> > > >
> > > >
> > > > On 05/29/2015 02:20 PM, Remi Bergsma wrote:
> > > > > Hi guys,
> > > > >
> > > > > I've been playing with KVM on CentOS 7.1 and found it does not work
> > > > > well with 4.5.1 due to a problem that Rohit already solved last
> > > > > week (thanks!). See PR:
> > > > > https://github.com/apache/cloudstack/pull/290
> > > > >
> > > > > Shall we do a new 4.5 release in the coming week or so? Then we can
> > > > > benefit from some of the fixes that have been done.
> > > > >
> > > >
> > > > Seems sane to me. Looking at the commits I see a lot of small fixes
> > > > from which users could really benefit.
> > > >
> > > > Wido
> > > >
> > > > > Regards, Remi
> > > > >
> > > > -BEGIN PGP SIGNATURE-
> > > > Version: GnuPG v1
> > > >
> > > > iQIcBAEBAgAGBQJVaFq7AAoJEAGbWC3bPspCVvAP/15xtyhXnhW8WeoLwUp3yurP
> > > > DwCchTw9WSQLO/NPK+ggdniuiP2vAeQQCSV8GUvzziO309elRDvaYR5tgrksRLzk
> > > > 1Q4F4tKNxr1PE/pAYJ4eogLbBa1RXJLN+h7ILYxghRNaZWFozG+VlXDF4Twob2e6
> > > > VVyq6azBd7i9NQD9k1Z9WbTvxW32dKlGEU/L0R2I7sVdkvtc7bkMDVDLKJsLdTqs
> > > > bk+GswnUTjUQWLHVEH2/Hnla34x+0Vw/OOooMzbc6NkX0nLYnKvE85xrqJaZpIim
> > > > g639ahe/jntJsvayKwpJXduYRe4xmXd+W/tSJ09PP7ImhGSxgInwmp2cPdjWxA9l
> > > > 0pmYlaAQ/+dRmiTER8aRsSmrYI334CCfkgzEe3WN+eMPH0WSE6iLVIWPasZQ/syH
> > > > RzekbUhFKRD8jb4e5n5ni6b7DRw+iIMlj1VqcPRo9cACYtrR4rdisgGQB/VYCLJx
> > > > HJJW/sodbcf+7lwYYVyZvG1p8XzLuMhJNaoA7y5Ll19xro9k5rSdPLnSP3meeUR1
> > > > 6KhzRoWaoCNlCpzsYHKVQxCLeDjyvsa/ge2rSKR+Aj+aPkXCiylLxfAeKiJIQuPz
> > > > nc3WDKuwwbbpGSf7KkBat8QgNv0od1SOEkfPVtCXuGHkZ9F5JP/i/LqgX5Ms1viu
> > > > I6HqlDzIGvrGAmQy/byA
> > > > =tmHL
> > > > -END PGP SIGNATURE-
> > > >
> > >
> >
>
>
>
> --
>
> Andrija Panić
>


Re: Shall we soon do a 4.5.2 release?

2015-05-29 Thread Rafael Fonseca
Here's one for current versions :)

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

On Fri, May 29, 2015 at 8:06 PM, Daan Hoogland 
wrote:

> making a new rc as I don't have time the rest of the weekend. no problem if
> I need to do another. vote thread coming up
>
> Op vr 29 mei 2015 om 16:00 schreef Andrija Panic  >:
>
> > Wido, do we have support for RBD resize currently in 4.5 branch ? I
> tested
> > 4.5.1 tag, and it didnt work - Im talking here about DATA volume resize,
> > while VM is off...
> > 2015-05-28 16:14:56,257 DEBUG [cloud.agent.Agent]
> > (agentRequest-Handler-1:null) Seq 1-4646870390516222369:  { Ans: ,
> MgmtId:
> > 90520740254323, via: 1, Ver: v1, Flags: 10, [{"com.cloud.agent.api.
> > storage.ResizeVolumeAnswer":{"newSize":0,"result":false,"details":"Failed
> > to resize volume: Cannot determine resize type from pool type
> > RBD","wait":0}}]
> > }
> >
> > Sorry for offtopic...
> >
> >
> > On 29 May 2015 at 15:45, Remi Bergsma  wrote:
> >
> > > That's OK Daan. Let's get some final patches in (if any) before we'll
> > > create a RC for 4.5.2 somewhere next week. In the mean time we'll work
> to
> > > get 4.4.4 out.
> > >
> > > Regards,
> > > Remi
> > >
> > > 2015-05-29 15:00 GMT+02:00 Daan Hoogland :
> > >
> > > > Remi, let's doe 4.4.4 this weekend first;) if no further issues come
> > > up
> > > > Or do we want a 4.5.2 irrespective and simultaneous?
> > > >
> > > > Op vr 29 mei 2015 om 14:25 schreef Wido den Hollander <
> w...@widodh.nl
> > >:
> > > >
> > > > > -BEGIN PGP SIGNED MESSAGE-
> > > > > Hash: SHA1
> > > > >
> > > > >
> > > > >
> > > > > On 05/29/2015 02:20 PM, Remi Bergsma wrote:
> > > > > > Hi guys,
> > > > > >
> > > > > > I've been playing with KVM on CentOS 7.1 and found it does not
> work
> > > > > > well with 4.5.1 due to a problem that Rohit already solved last
> > > > > > week (thanks!). See PR:
> > > > > > https://github.com/apache/cloudstack/pull/290
> > > > > >
> > > > > > Shall we do a new 4.5 release in the coming week or so? Then we
> can
> > > > > > benefit from some of the fixes that have been done.
> > > > > >
> > > > >
> > > > > Seems sane to me. Looking at the commits I see a lot of small fixes
> > > > > from which users could really benefit.
> > > > >
> > > > > Wido
> > > > >
> > > > > > Regards, Remi
> > > > > >
> > > > > -BEGIN PGP SIGNATURE-
> > > > > Version: GnuPG v1
> > > > >
> > > > > iQIcBAEBAgAGBQJVaFq7AAoJEAGbWC3bPspCVvAP/15xtyhXnhW8WeoLwUp3yurP
> > > > > DwCchTw9WSQLO/NPK+ggdniuiP2vAeQQCSV8GUvzziO309elRDvaYR5tgrksRLzk
> > > > > 1Q4F4tKNxr1PE/pAYJ4eogLbBa1RXJLN+h7ILYxghRNaZWFozG+VlXDF4Twob2e6
> > > > > VVyq6azBd7i9NQD9k1Z9WbTvxW32dKlGEU/L0R2I7sVdkvtc7bkMDVDLKJsLdTqs
> > > > > bk+GswnUTjUQWLHVEH2/Hnla34x+0Vw/OOooMzbc6NkX0nLYnKvE85xrqJaZpIim
> > > > > g639ahe/jntJsvayKwpJXduYRe4xmXd+W/tSJ09PP7ImhGSxgInwmp2cPdjWxA9l
> > > > > 0pmYlaAQ/+dRmiTER8aRsSmrYI334CCfkgzEe3WN+eMPH0WSE6iLVIWPasZQ/syH
> > > > > RzekbUhFKRD8jb4e5n5ni6b7DRw+iIMlj1VqcPRo9cACYtrR4rdisgGQB/VYCLJx
> > > > > HJJW/sodbcf+7lwYYVyZvG1p8XzLuMhJNaoA7y5Ll19xro9k5rSdPLnSP3meeUR1
> > > > > 6KhzRoWaoCNlCpzsYHKVQxCLeDjyvsa/ge2rSKR+Aj+aPkXCiylLxfAeKiJIQuPz
> > > > > nc3WDKuwwbbpGSf7KkBat8QgNv0od1SOEkfPVtCXuGHkZ9F5JP/i/LqgX5Ms1viu
> > > > > I6HqlDzIGvrGAmQy/byA
> > > > > =tmHL
> > > > > -END PGP SIGNATURE-
> > > > >
> > > >
> > >
> >
> >
> >
> > --
> >
> > Andrija Panić
> >
>


[VOTE] release candidate 4.4.4

2015-05-29 Thread Daan Hoogland
Hi All,

I've created a 4.4.4 release, with the following artifacts up for a vote:

Git Branch and Commit
SH:https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/
4.4-RC20150529T2004
Commit: 7d2320cd1d27842fae73c7f43ef422da354f

List of 
changes:https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/tags/4.4.4

Source release (checksums and signatures are available at the same
location):https://dist.apache.org/repos/dist/dev/cloudstack/4.4.4/

PGP release keys (signed using
AA4736F3):https://dist.apache.org/repos/dist/release/cloudstack/KEYS
Vote will be open for 72 hours.

For sanity in tallying the vote, can PMC members please be sure to
indicate "(binding)" with their vote?

[ ] +1  approve
[ ] +0  no opinion
[ ] -1  disapprove (and reason why)


[GitHub] cloudstack pull request: Fix agent/management conflict

2015-05-29 Thread rsafonseca
Github user rsafonseca commented on the pull request:

https://github.com/apache/cloudstack/pull/329#issuecomment-106893942
  
@DaanHoogland too late for 4.4.4? :)


---
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: Fix agent/management conflict

2015-05-29 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/329#issuecomment-106894818
  
@rsafonseca , just a second or so, sorry. If this is really blocking you I 
can create a new RC on monday. It is an old known bug, though. Can't really 
worry to much about it. Maybe you have a good argument?


---
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: Fix agent/management conflict

2015-05-29 Thread rsafonseca
Github user rsafonseca commented on the pull request:

https://github.com/apache/cloudstack/pull/329#issuecomment-106897613
  
nah not blocking a thing on my side.. just thought i'd contribute something 
more to your release, but it wasn't on time :)
one old bug less for 4.4.5 i guess ;)


---
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: Change tests random number generator to /...

2015-05-29 Thread rsafonseca
GitHub user rsafonseca opened a pull request:

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

Change tests random number generator to /dev/urandom

This fixes a big performance issue with random number generation with more 
recent kernels and java versions in linux, the impact is specially higher on VMs
Some tests (that use random number generation) in some environments take 
randomly 1 - 20 mins to perform tests that should take only seconds to run.
This PR is just intended for fixing it on the tests.

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

$ git pull https://github.com/rsafonseca/cloudstack fast_tests

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

https://github.com/apache/cloudstack/pull/330.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 #330


commit 82847d4405f20920a1ca066765071d485931ff19
Author: Rafael da Fonseca 
Date:   2015-05-29T18:24:41Z

Change tests random number generator to /dev/urandom
This fixes a big performance issue with random number generation with more 
recent kernels and java versions in linux




---
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: template, api: fix format desc, add OVA f...

2015-05-29 Thread resmo
GitHub user resmo opened a pull request:

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

template, api: fix format desc, add OVA format



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

$ git pull https://github.com/resmo/cloudstack patch-1

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

https://github.com/apache/cloudstack/pull/331.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 #331


commit 8548757aedcaf6e15a4acb6e227d5f849041c6a6
Author: René Moser 
Date:   2015-05-29T18:44:12Z

template, api: fix format desc, add OVA format




---
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: Fix agent/management conflict

2015-05-29 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/329#issuecomment-106903590
  
@rsafonseca this is not an issue on 4.5/master, I had already fixed it. 
possibly for 4.4; will apply it on 4.4 nonetheless. @DaanHoogland no need of 
another RC, it's a minor issue if you happen to install both mgmt server and 
agent on same kvm node.


---
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: Fix agent/management conflict

2015-05-29 Thread rsafonseca
Github user rsafonseca commented on the pull request:

https://github.com/apache/cloudstack/pull/329#issuecomment-106904018
  
perhaps there was a regression? I got the issue in 4.4 and 4.5
master is fine though :)


---
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: Fix agent/management conflict

2015-05-29 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/329#issuecomment-106904206
  
@rsafonseca not applicable latest 4.5; applied your PR on 4.4. Thanks, 
please close it manually (asfgit failed to close it).


---
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: template, api: fix format desc, add OVA f...

2015-05-29 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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: Change tests random number generator to /...

2015-05-29 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/330#discussion_r31354958
  
--- Diff: pom.xml ---
@@ -958,6 +958,9 @@
   org.apache.maven.plugins
   maven-surefire-plugin
   2.18.1
+  
+  -Djava.security.egd=file:/dev/./urandom
--- End diff --

why not simply, "/dev/urandom" ?


---
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-8525: NPE while updating the s...

2015-05-29 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/321#issuecomment-106904868
  
LGTM, @wilderrodrigues since you're testing please merge once it's green on 
your side. Thanks @karuturi 


---
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: template, api: fix format desc, add OVA f...

2015-05-29 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/331#issuecomment-106904312
  
LGTM, thanks @resmo 


---
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: Reinstate working sessions in browser

2015-05-29 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/308#issuecomment-106906370
  
Hi @rsafonseca, thanks for replying.

- I agree, it's best to force and use SSL than use secure flag.
- Regarding point 3, what I mean to explain is that since rely on 
sessionkey passed in the HTTP API parameter when we set sessionkey httponly 
cookie we should put a fix to check sessionkey in the cookie as well on a 
request. The issue here is that for normal login it would work, because the 
login response will send the sessionkey which javascript code will save in a g_ 
(global) variable and reuse it for future API requests. In case of alternative 
auth/login, for example the SAML plugin, we rely only on cookies but if you 
make the sessionkey cookie httponly - the JS code won't be able to read that 
and future APIs will send empty sessionkey API arg/parameter. Now, to make this 
work - the APIServlet needs to check for sessionkey cookie in case sessionkey 
HTTP parameter is missing or empty. The SAML plugin would set a sessionkey 
cookie that is readable but since in the fix after we authenticate() you're 
setting sessionkey cookie to be httponly (so after login/redirection, JS code 
won
 't be able to read that cookie). If this is confusing, leave it for me - I can 
help fix this.
- Regarding point 4, I guess what I meant to say is not to remove the 
argument but cover the above case. I agree we don't need to remove it since API 
clients may break.

I guess we have some agreement here, please go ahead and make the changes. 
I'm working on making the SAML plugin production grade, so I can help test all 
aspects of logging/authentication around it as well.

Cheers.


---
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-8231: fixed merge conflicts wi...

2015-05-29 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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-8308-Adding-automation-test-ca...

2015-05-29 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/229#issuecomment-106907661
  
I don't have infra to test/run this, @pritisarap12 @gauravaradhye  - can 
you review/test/merge this. 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: Automation for issue, Private template no...

2015-05-29 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/241#issuecomment-106908016
  
@shwetaag @gauravaradhye @pavanb018 any progress on this one thanks? (sorry 
don't have infra to run/test this)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project 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-6276: affinitygroup: implement...

2015-05-29 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/264#issuecomment-106908111
  
@resmo thanks, okay just saw WIP. Let us know when it's ready for 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: Fix agent/management conflict

2015-05-29 Thread rsafonseca
Github user rsafonseca commented on the pull request:

https://github.com/apache/cloudstack/pull/329#issuecomment-106908778
  
@bhaisaab you are right! .. though it happened on the 4.5 version i built 
locally... perhaps my copy of the branch was not up to date when i tested :)
Closing!



---
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: Fix agent/management conflict

2015-05-29 Thread rsafonseca
GitHub user rsafonseca reopened a pull request:

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

Fix agent/management conflict

There is an entry for the agent log directory in the management packaging, 
creating a conflict between the two RPMs.
This issue occurs in both 4.4.x and 4.5.x
This fixes not being able to install cloudstack-agent and 
cloudstack-management in the same machine in CentOS.

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

$ git pull https://github.com/rsafonseca/cloudstack 45_rpmconflict

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

https://github.com/apache/cloudstack/pull/329.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 #329


commit 1138f54c323a5340b426a15f8efeb293bc738c2d
Author: Rafael da Fonseca 
Date:   2015-05-29T17:55:32Z

Fix agent/management conflict




---
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: Fix agent/management conflict

2015-05-29 Thread rsafonseca
Github user rsafonseca closed the pull request at:

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


---
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: Fix agent/management conflict

2015-05-29 Thread rsafonseca
Github user rsafonseca closed the pull request at:

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


---
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: Fix agent/management conflict

2015-05-29 Thread rsafonseca
Github user rsafonseca commented on the pull request:

https://github.com/apache/cloudstack/pull/329#issuecomment-106908889
  
.


---
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-8231: fixed merge conflicts wi...

2015-05-29 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/302#issuecomment-106907445
  
@vadimkim thanks for the fix but it did not apply cleanly. Nevertheless, I 
applied it manually and merged on both master and 4.5 branch. Can you see if 
that's alright and test 4.5/master. I know about this issue but don't have a 
setup to test it, but the code looks good 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.
---


[GitHub] cloudstack pull request: Change tests random number generator to /...

2015-05-29 Thread rsafonseca
Github user rsafonseca commented on the pull request:

https://github.com/apache/cloudstack/pull/330#issuecomment-106909482
  
there's a builtin symlink that redirects /dev/urandom to /dev/random. this 
is the only way to make it work :)
This is a known issue with a lot of java apps running in linux


---
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: [GitHub] cloudstack pull request: Fix agent/management conflict

2015-05-29 Thread Remi Bergsma
Looks like asfgit only closes PRs when we commit to master branch and not to 
any another one. Can we maybe ask for some more branches to be monitored, like 
4.4 and 4.5 etc. Would save manual work. 

Regards, Remi 

Sent from my iPhone

> On 29 May 2015, at 21:01, bhaisaab  wrote:
> 
> Github user bhaisaab commented on the pull request:
> 
>https://github.com/apache/cloudstack/pull/329#issuecomment-106904206
> 
>@rsafonseca not applicable latest 4.5; applied your PR on 4.4. Thanks, 
> please close it manually (asfgit failed to close it).
> 
> 
> ---
> 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: IPv6 ideas for Basic Networking

2015-05-29 Thread Suresh Ramamurthy
Hi Wido,

After reading your IPv6 ideas for Basic Networking, I realized that couple
of them can be reused for Advanced Networking too.

We have come up with a proposal for IPv6 support in VPC and it is posted in
wiki

https://cwiki.apache.org/confluence/display/CLOUDSTACK/IPv6+in+VPC+Router

Did you get a chance to look at it? Let me know your feedback on the DD.

I work from bay area, so I will not be able to attend the meetup at
Amsterdam. But, I would like to have a call/chat with you to discuss on
further details about IPv6 support.

I would like to schedule a conference call with you. Would you be available
for the call?

Thanks,
Suresh



On Sat, May 23, 2015 at 11:18 PM, Remi Bergsma 
wrote:

> At Schuberg Philis we’ve been working on a design voor IPv6 in VPC
> networks (so this is Advanced Networking) and I indeed had a look at your
> functional spec. I’ll finalise what we’ve come up with and publish it early
> next week so we can align and discuss and work from there. Nice to see
> there is a design for Basic Networking as well!
>
> Regards,
> Remi
>
> > On 24 May 2015, at 02:47, Marcus  wrote:
> >
> > Did you guys review the functional spec that has been floating around on
> > cwiki?
> > On May 23, 2015 8:27 AM, "Wido den Hollander"  wrote:
> >
> >> -BEGIN PGP SIGNED MESSAGE-
> >> Hash: SHA1
> >>
> >>
> >>
> >> On 05/22/2015 11:05 PM, server24 Cloudstack wrote:
> >>> Hi Wido,
> >>>
> >>> was nice talking to you about this.
> >>>
> >>> On 5/21/2015 8:59 PM, Wido den Hollander wrote:
> >>>
>  (IPv6) routers should send out RAs (Router Advertisements) with
>  the managed-other-flag [0][1], telling Instances to ONLY use that
>  routers as their default gateways and NOT to use SLAAC to
>  autoconfigure their IP-Address.
> >>>
> >>> OK, so no autonomous flag
> >>>
> >>
> >> No, the "managed other flag" as described in RFC 4862. Meaning that
> >> the Routers should only be used as a default gateway and DHCPv6 should
> >> be used for obtaining a address.
> >>
>  The (ip6tables) Security Groups should allow ICMPv6 by default.
>  IPv6 traffic breaks really hard without ICMPv6 traffic, for
>  example PMTU doesn't work properly and breaks IPv6 connections.
> >>> yes, and default ip(6)tables should be in place to block
> >>> VNC-related traffic except to the Virtual Console (as currently VNC
> >>> ports on IPv6 are world-wide-open in BASIC network)!
> >>>
> >>
> >> Yes, but in that case you are talking about the Console Proxy which
> >> should be firewalled properly.
> >>
>  In CloudStack we might configure a /48, but tell it to hand out
>  addresses for each instance from a /64 out of that /48. That
>  means we can have 65k Instances in that pod. Some firewall
>  policies block a complete /64 when they see malicious traffic
>  coming from that subnet, so if the subnet is big enough we should
>  try to keep all the IPv6 addresses from one Instance in the same
>  /64 subnet. This could also simplify the iptable rules.
> >>> so one /48 per pod? RIRs provide either /48 or /32 (the latter to
> >>> the providers) IPv6 blocks. So this should then be configurable,
> >>> both per instance and per pod. One /48 per pod still looks large to
> >>> me..
> >>>
> >>
> >> A /48 should be a possibility. If you only have a /64 available that
> >> should be no problem either.
> >>
> >>> On the other hand any prefix more specific than /64 could break
> >>> IPv6 features, so that there are at least some practical values to
> >>> rely on.
>  Security grouping has to be extended to also support IPv6, but
>  should allow ICMPv6 by default.
> >>> yes, ICMPv6 should be on by default (maybe it should be forced to
> >>> be always on for IPv6?).
> >>>
>  At the end of June 2015 we want to keep a one-day meetup in
>  Amsterdam with various developers to discuss some more details.
> >>>
> >>> great work and very good meeting, was a pleasure to be there.
> >>>
> >>> Thomas Moroder
> >>>
> >>> -- Incubatec GmbH - Srl Via Scurcia'str. 36, 39046 Ortisei(BZ),
> >>> ITALY Registered with the chamber of commerce of Bolzano the 8th of
> >>> November 2001 with REA-No. 168204 (s.c. of EUR 10.000 f.p.u.)
> >>> President: Thomas Moroder, VAT-No. IT 02283140214 Tel:
> >>> +39.0471796829 - Fax: +39.0471797949
> >>>
> >>> IMPRINT: http://www.incubatec.com/imprint.html PRIVACY:
> >>> http://www.server24.it/informativa_completa.html
> >>>
> >> -BEGIN PGP SIGNATURE-
> >> Version: GnuPG v1
> >>
> >> iQIcBAEBAgAGBQJVYJwhAAoJEAGbWC3bPspCWXEQAISPV1PdGWa6KOck9IsTVXBt
> >> jUTpyFyg+qnmlG+QQ3LWOFjXRVUvQroryBbxkBnEbNm5d5qOsKptwwOaXMOut4A2
> >> Nv4WCcFlAjnj78c9C/mpJvqu+Bh/WLKy4mBaMEqLiSAzqoz+CPlaiubJ/TDXR+jp
> >> XSY3XNk/jhdI02QTPNHvYc1ZbWNjuZrb+YVqEzFLra25I0bfXuq2tcBVDEMr1zmA
> >> qQVfCabkAx/a8wW2wGnz2GSg1UFDJUHOb7c6bae9nE5wo1MYpXyjpO53IBpRQuPt
> >> +VMzkjyf75yS1LFel9zS/BzV97mgEBxux9Nb3M9f/ZJW0QvS9onZdIhYgCeDyxJe
> >> M/XTD6M0O+ha4mFaYTeSudWoQnv/ZZ1P5RuPTQyQRD4P7nSkorz6QOR/SVb

[GitHub] cloudstack pull request: Use same sudoers config for all distros

2015-05-29 Thread rsafonseca
GitHub user rsafonseca opened a pull request:

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

Use same sudoers config for all distros

Each distro was using its own sudo config method, now all use the same 
config file and are setup by the packaging and have the user variable evaluated 
from config
Cleanup python code for sudoers file editing with hardcoded user

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

$ git pull https://github.com/rsafonseca/cloudstack sudoconfig

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

https://github.com/apache/cloudstack/pull/332.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 #332


commit 7deec642ef9c4c137e0cb1d01efedcada9afb8ab
Author: Rafael da Fonseca 
Date:   2015-05-29T20:10:39Z

Use same sudoers config for all distros
Cleanup buggy code for sudoers file editing




---
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: Reinstate working sessions in browser

2015-05-29 Thread rsafonseca
Github user rsafonseca commented on the pull request:

https://github.com/apache/cloudstack/pull/308#issuecomment-106937060
  
Hi @bhaisaab 
I don't think that is an issue :)
Whatever your login method, the cookie will always be set on the first 
successful command and subsequent ones.
On normal login, the js doesn't really store it in a variable, it tries to 
read the httponly cookie and fails , and all subsequent API commands have the 
param sessionkey="null".. that's why i'm checking if command.equals("null") to 
get the cookie :)
If you have tested and have issues, might also be the case that the saml 
codes tries to set the cookie, overwriting the existing httponly cookie with a 
plain one, thus breaking auth.


---
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: Reflections upgrade to 0.9.9

2015-05-29 Thread rsafonseca
GitHub user rsafonseca opened a pull request:

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

Reflections upgrade to 0.9.9

Reflection's 0.9.8 dependency javassist doen't properly support java8 and 
was throwing:

 Error: java.io.IOException: invalid constant type: 18

Running some unit tests with PowerMockRunner caused this issue, so this 
fixes it for future tests that use it.

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

$ git pull https://github.com/rsafonseca/cloudstack reflections

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

https://github.com/apache/cloudstack/pull/333.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 #333


commit ef4a2f6c592a911dc12c63ef30ed3028ce56e2da
Author: Rafael da Fonseca 
Date:   2015-05-26T21:29:11Z

Reinstate working sessions in browser while fixing same domain different 
instance issue

commit 775365ae30dd4d93ea2063d6fd06f358bcae293b
Author: Rafael da Fonseca 
Date:   2015-05-26T22:03:04Z

Fix all cookie references. Cleanup a couple of extra cookies on logout.

commit 7c2502a9c2cd538336c5d87a7613f3b541bd70ec
Author: Rafael da Fonseca 
Date:   2015-05-27T10:37:06Z

Set sessionkey on HttpOnly cookie handled by ApiServlet
Remove sessionkey data stored in js variable

commit 376497aeb9ede7e79eaa65cf632e18b7db013fb4
Author: Rafael da Fonseca 
Date:   2015-05-29T21:32:35Z

Upgrade Reflections to 0.9.9




---
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: Reflections upgrade to 0.9.9

2015-05-29 Thread rsafonseca
Github user rsafonseca commented on the pull request:

https://github.com/apache/cloudstack/pull/333#issuecomment-106941899
  
ops :)


---
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-8231: fixed merge conflicts wi...

2015-05-29 Thread vadimkim
Github user vadimkim commented on the pull request:

https://github.com/apache/cloudstack/pull/302#issuecomment-106997127
  
I have checked 4.5/master and it looks good


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have 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.
---