[GitHub] cloudstack pull request #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

2016-08-15 Thread marcaurele
Github user marcaurele commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1638#discussion_r74739489
  
--- Diff: pom.xml ---
@@ -92,22 +85,22 @@
 1.5.1
 1.2.8
 2.0.4
-2.5
+3.1.0
 1.2
 1.2.1
 
1.0-20081010.060147
 6.0
-
3.2.16.RELEASE
+
4.2.7.RELEASE
--- End diff --

Pushed a PR on your branch with the fixes


---
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 issue #1619: Add the Transifex config for next version of CS (4.1...

2016-08-15 Thread pdion891
Github user pdion891 commented on the issue:

https://github.com/apache/cloudstack/pull/1619
  
LGTM, 

Does this command most be run every build or one time per branch only ? 
Shouldn't we document this process in the building doc or the wiki ?

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


[RESULT][VOTE] Apache CloudStack 4.8.1

2016-08-15 Thread Will Stevens
Hi all,

After 72+ hours, the vote for CloudStack 4.8.1 *passes* with
2 PMC + 1 non-PMC votes.

+1 (PMC / binding)
* Mike Tutkowski
* Milamber

+1 (non binding)
* Simon Weller

0
none

-1
none

Thanks to everyone participating.

I will now prepare the release announcement to go out after 24 hours to
give the mirrors time to catch up.


[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-08-15 Thread nvazquez
Github user nvazquez commented on the issue:

https://github.com/apache/cloudstack/pull/1542
  
Thanks @rafaelweingartner 
This is the link to JIRA ticket: 
https://issues.apache.org/jira/browse/CLOUDSTACK-9457


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


Working/Supported versions for VMware vCenter and ESXi Hosts/Cluster for CloudStack 4.9

2016-08-15 Thread Jonas Schlichtenbrede
Hi Dev Team,


we have a question regarding the current working/supported version for
VMware.


Which version can/shall we use for a new Zone/Cluster for vCenter and for
the ESXi Hosts/Cluster?



Will it be possible to do the following:

- Using an existing vCenter 6 and ESXi 6 hosts

- Setup a new VMware Cluster with 2-4 ESXi 6 hosts controlled by CloudStack

In the Installation Guide only version 5.5 for vCenter and ESXi is listed.
However, in the mailing lists it seems that also version 6 of vCenter and
ESXi shall work - but we would like to reduce our testing efforts ... ;-)

Thanks for sharing your experience and advise!

Jonas


[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-08-15 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue:

https://github.com/apache/cloudstack/pull/1542
  
I am the one thas has to thank you @nvazquez, great 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: [RESULT][VOTE] Apache CloudStack 4.8.1

2016-08-15 Thread Milamber

Hello,

This vote isn't compliant with the ASF release vote. We need as least 3 
PMC +1 vote [1].


Will, you can just vote +1 and resend the result. (Or another PMC member)

Milamber

[1] http://www.apache.org/foundation/voting.html#ReleaseVotes


On 15/08/2016 15:16, Will Stevens wrote:

Hi all,

After 72+ hours, the vote for CloudStack 4.8.1 *passes* with
2 PMC + 1 non-PMC votes.

+1 (PMC / binding)
* Mike Tutkowski
* Milamber

+1 (non binding)
* Simon Weller

0
none

-1
none

Thanks to everyone participating.

I will now prepare the release announcement to go out after 24 hours to
give the mirrors time to catch up.





Re: [RESULT][VOTE] Apache CloudStack 4.8.1

2016-08-15 Thread Will Stevens
Thanks Milamber.  I will add my own vote.

*Will STEVENS*
Lead Developer

*CloudOps* *| *Cloud Solutions Experts
420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
w cloudops.com *|* tw @CloudOps_

On Mon, Aug 15, 2016 at 11:45 AM, Milamber  wrote:

> Hello,
>
> This vote isn't compliant with the ASF release vote. We need as least 3
> PMC +1 vote [1].
>
> Will, you can just vote +1 and resend the result. (Or another PMC member)
>
> Milamber
>
> [1] http://www.apache.org/foundation/voting.html#ReleaseVotes
>
>
>
> On 15/08/2016 15:16, Will Stevens wrote:
>
>> Hi all,
>>
>> After 72+ hours, the vote for CloudStack 4.8.1 *passes* with
>> 2 PMC + 1 non-PMC votes.
>>
>> +1 (PMC / binding)
>> * Mike Tutkowski
>> * Milamber
>>
>> +1 (non binding)
>> * Simon Weller
>>
>> 0
>> none
>>
>> -1
>> none
>>
>> Thanks to everyone participating.
>>
>> I will now prepare the release announcement to go out after 24 hours to
>> give the mirrors time to catch up.
>>
>>
>


[RESULT][VOTE] Apache CloudStack 4.8.1 (take two)

2016-08-15 Thread Will Stevens
Hi all,

After 72+ hours, the vote for CloudStack 4.8.1 *passes* with
3 PMC + 1 non-PMC votes.

+1 (PMC / binding)
* Mike Tutkowski
* Milamber
* Will Stevens

+1 (non binding)
* Simon Weller

0
none

-1
none

Thanks to everyone participating.

I will now prepare the release announcement to go out after 24 hours to
give the mirrors time to catch up.


Re: [VOTE] Apache Cloudstack 4.8.1 RC2

2016-08-15 Thread Will Stevens
+1 (binding)

I have installed and tested this release in my CI environment without
issues.

*Will STEVENS*
Lead Developer

*CloudOps* *| *Cloud Solutions Experts
420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
w cloudops.com *|* tw @CloudOps_

On Fri, Aug 12, 2016 at 2:19 AM, Milamber  wrote:

> Hello,
>
> My vote +1 (binding)
>
> Tests are passed on a virtual topology of servers  (CS over CS)
> (1mgr+2nodes+1nfs) :
>
> 1/ Fresh install of 4.8.1RC2 (adv net) on Ubuntu 14.04.5 + KVM + NFS : OK
> Some standard tests with success (create vm, migration, HA, create
> networks, create user, create ssh key, destroy vm, register template,
> create snapshot, restore snapshot, create template, ip association, ip
> release, static nat, firewall rule)
> Some tests with cloudstack ansible module with success too (create
> network, register templates, create vm, ip, firewall rule)
>
> 2/ Test upgrade from 4.8.0 to 4.8.1RC2 : OK
>
> 3/ Tests of all localization Web UI for 4.8.1RC2:
> Localization works well except Spanish: the Web UI won't display due of 2
> localization strings with a escaped quote (from Transifex). This is the
> same issue that the 4.9.0RC1.
> Fixed in the PR1636 (not a blocker to release)
> https://github.com/apache/cloudstack/pull/1636
>
>
> Perhaps add in the Release notes this Spanish l10n issue.
>
> Thanks to the RM.
>
> Milamber
>
>
>
> On 08/08/2016 15:29, Will Stevens wrote:
>
>> Hi All,
>>
>> Sorry for the version mix up with RC1.  This will be the 4.8.1 release,
>> not
>> the 4.8.2 release.
>>
>> I've created a 4.8.1 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.8.1-RC20160808T1006
>> Commit: a63db21d16072821a1e27473813fddf36accfdd4
>>
>> List of changes will be produced with the release notes once the RC
>> passes.
>>
>> Source release (checksums and signatures are available at the same
>> location):
>> https://dist.apache.org/repos/dist/dev/cloudstack/4.8.1/
>> 
>>
>> PGP release keys (signed using CB818F64):
>> 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 #1593: CLOUDSTACK-9417: Usage module refactoring

2016-08-15 Thread nvazquez
Github user nvazquez commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1593#discussion_r74807418
  
--- Diff: usage/src/com/cloud/usage/UsageSanityChecker.java ---
@@ -72,7 +72,9 @@ protected boolean checkItemCountByPstmt(CheckCase 
checkCase) throws SQLException
 try (PreparedStatement pstmt = 
conn.prepareStatement(checkCase.sqlTemplate)) {
 if(checkCase.checkId) {
 pstmt.setInt(1, lastId);
-pstmt.setInt(2, maxId);
+if (maxId > 0) {
+pstmt.setInt(2, maxId);
--- End diff --

Hi @rafaelweingartner sorry I haven't seen you comment earlier, thanks for 
reviewing!
Actually the first parameter on line 74 belongs to the one set on line 216, 
and the one inside the `if` is only set if it was set on the query, on line 194.


---
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 #1639: CLOUDSTACK-9453: WIP : Marvin optimizations a...

2016-08-15 Thread abhinandanprateek
GitHub user abhinandanprateek opened a pull request:

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

CLOUDSTACK-9453: WIP : Marvin optimizations and fixes.

  1. Adding small sized macchinina template as built in templates
  2. Adding methods so the test writers haev choice to use any of these
  3. Updateing test_host_ha to use macchinina template
  4. Misc fixes encoutered while running component tests

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

$ git pull https://github.com/shapeblue/cloudstack marvin_opt

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

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


commit f691810926b7df55c297d8a19473a703348cbc75
Author: Abhinandan Prateek 
Date:   2016-08-16T04:57:28Z

CLOUDSTACK-9453: WIP
  1. Adding small sized macchinina template as built in templates
  2. Adding methods so the test writers haev choice to use any of these
  3. Updateing test_host_ha to use macchinina template
  4. Misc fixes encoutered while running component tests




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


[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

2016-08-15 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1638
  
@blueorangutan package


---
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 issue #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

2016-08-15 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1638
  
Thanks @marcaurele for your fix, it has been included in the PR.


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


[GitHub] cloudstack issue #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

2016-08-15 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1638
  
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


---
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 #1640: CLOUDSTACK-9458: Fix HA bug when VMs are stop...

2016-08-15 Thread marcaurele
GitHub user marcaurele opened a pull request:

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

CLOUDSTACK-9458: Fix HA bug when VMs are stopped on agent disconnect

VM are being restarted even if they don't have HA enabled. If the agent
gets back online before the command expires some VMs are being stopped
and not even restarted.

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

$ git pull https://github.com/exoscale/cloudstack bug/CLOUDSTACK-9458

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

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


commit c8f57541006536c4719044405b7691fad244fbf7
Author: Marc-Aurèle Brothier 
Date:   2016-08-16T06:10:51Z

CLOUDSTACK-9458: Fix HA bug when VMs are stopped on agent disconnect

VM are being restarted even if they don't have HA enabled. If the agent
gets back online before the command expires some VMs are being stopped
and not even restarted.




---
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 issue #1638: CLOUDSTACK-9456: Use Spring 4 and Java 8

2016-08-15 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1638
  
Packaging result: ✔centos6 ✔centos7 ✔debian repo: 
http://packages.shapeblue.com/cloudstack/pr/1638
Job ID-86


---
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: Working/Supported versions for VMware vCenter and ESXi Hosts/Cluster for CloudStack 4.9

2016-08-15 Thread Rohit Yadav
Hi Jonas,


We've not yet updated the docs as we've not performed tests on/with 
vCenter/ESXi 6 exhaustively. In practice, it does seem to work. We've an 
automated Trillian [1] based environment where we're able to deploy an advanced 
(CloudStack 4.9) zone with vCenter/ESXi 6 and get basic vm lifecycle working 
but we've not yet performed all VMware specific tests/operations/features to 
conclude full support/compatibility.


I would advise to go ahead and test vCenter/ESXi 6 with CloudStack 4.9.0 for 
your use-cases and please do share the results with us (if it works, or in case 
you hit any bug). This will help us ascertain VMware 6.x compatibility/support 
when using with CloudStack.


[1] 
http://www.shapeblue.com/cloudstack-test-automation-with-trillian-and-jenkins/


Regards.


From: Jonas Schlichtenbrede 
Sent: 15 August 2016 19:53:40
To: dev@cloudstack.apache.org
Subject: Working/Supported versions for VMware vCenter and ESXi Hosts/Cluster 
for CloudStack 4.9

Hi Dev Team,


we have a question regarding the current working/supported version for
VMware.


Which version can/shall we use for a new Zone/Cluster for vCenter and for
the ESXi Hosts/Cluster?



Will it be possible to do the following:

- Using an existing vCenter 6 and ESXi 6 hosts

- Setup a new VMware Cluster with 2-4 ESXi 6 hosts controlled by CloudStack

In the Installation Guide only version 5.5 for vCenter and ESXi is listed.
However, in the mailing lists it seems that also version 6 of vCenter and
ESXi shall work - but we would like to reduce our testing efforts ... ;-)

Thanks for sharing your experience and advise!

Jonas

rohit.ya...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue