[GitHub] cloudstack pull request: Quota: findbug fixes

2016-01-27 Thread agneya2001
Github user agneya2001 commented on the pull request:

https://github.com/apache/cloudstack/pull/1289#issuecomment-175472278
  
This patch cleans up any find bug issues in cloud-framework-quota and 
cloud-plugin-database-quota, verifiy output below. I think we should not delay 
this @remibergsma @DaanHoogland @bhaisaab @rafaelweingartner 



mvn -P enablefindbugs -pl :cloud-framework-quota verify
...

[INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ 
cloud-framework-quota ---
[INFO] Nothing to compile - all classes are up to date
[INFO] 
[INFO] >>> findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) > 
:findbugs @ cloud-framework-quota >>>
[INFO] 
[INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ 
cloud-framework-quota ---
[INFO] Fork Value is true
[INFO] Done FindBugs Analysis
[INFO] 
[INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) < 
:findbugs @ cloud-framework-quota <<<
[INFO] 
[INFO] --- findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) @ 
cloud-framework-quota ---
[INFO] BugInstance size is 0
[INFO] Error size is 0
[INFO] No errors/warnings found
[INFO] 


mvn -P enablefindbugs -pl :cloud-plugin-database-quota verify

..
[INFO] >>> findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) > 
:findbugs @ cloud-plugin-database-quota >>>
[INFO] 
[INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ 
cloud-plugin-database-quota ---
[INFO] Fork Value is true
[INFO] Done FindBugs Analysis
[INFO] 
[INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) < 
:findbugs @ cloud-plugin-database-quota <<<
[INFO] 
[INFO] --- findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) @ 
cloud-plugin-database-quota ---
[INFO] BugInstance size is 0
[INFO] Error size is 0
[INFO] No errors/warnings found



---
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: [4.9+] WIP: Support Java 8, Maven 3.3, Up...

2016-01-27 Thread wido
Github user wido commented on the pull request:

https://github.com/apache/cloudstack/pull/1370#issuecomment-175473551
  
I agree with switching to Java 8, but shouldn't this be a small PR first 
where we ONLY switch to Java 8?

Other PRs can follow up quickly.


---
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: [4.9+] WIP: Support Java 8, Maven 3.3, Up...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1370#issuecomment-175477214
  
@wido I was getting some compilation issue specifically to F5, to which 
someone previously on ML had advised an issue related to dependency version so 
I've included upgrading dependency version (to stable/latest minor revision but 
no major version changes causing compatibility issues). Unfortunately, I did 
that but F5 was still not happy while getting compiled.


---
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: Quota: findbug fixes

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1289#issuecomment-175479331
  
LGTM @agneya2001 master seems to be unfrozen you'll need one more review to 
merge it yourself


---
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: Strongswan vpn feature

2016-01-27 Thread jayapalu
Github user jayapalu commented on the pull request:

https://github.com/apache/cloudstack/pull/872#issuecomment-175488909
  
@remibergsma 
Can you please try  running the  below tests once and let me know. For me 
the tests are passed. 

$nosetests-2.7 --with-marvin 
--marvin-config=/Users/jayapalreddy/advanced-acs.cfg 
~/cloudstack/test/integration/smoke/test_vpc_vpn.py:TestVpcSite2SiteVpn -a 
tags=advanced --zone=zone1 --hypervisor=xenserver


---
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: [4.9+] WIP: Support Java 8, Maven 3.3, Up...

2016-01-27 Thread jburwell
Github user jburwell commented on the pull request:

https://github.com/apache/cloudstack/pull/1370#issuecomment-175490258
  
To ease transition, it seems like the first step should be to support 
running ACS built to 1.7 on a 1.8 runtime.  This approach would allow customers 
unprepared to transition to continue using Java7.  We would then notify users 
that Java7 support has been deprecated and will be removed in two releases (two 
months).


---
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: FYI: master branch is on 4.9.0 now

2016-01-27 Thread Sebastien Goasguen

> On Jan 27, 2016, at 4:53 AM, Raja Pullela  wrote:
> 
> Hi Remi, Ilya, I had the same question... do we have a release cadence ?  
> 

The cadence is that as long as we stick to our current process we can release 
*anytime* both minor and majors.
So we are always only one VOTE away from a release, that’s 72 hours.

FWIW, it does mean that in addition to CI we need to automate release notes, 
docs version upgrade and websites publishing.

> Best,
> Raja
> 
> -Original Message-
> From: ilya [mailto:ilya.mailing.li...@gmail.com] 
> Sent: Wednesday, January 27, 2016 2:40 AM
> To: dev@cloudstack.apache.org
> Subject: Re: FYI: master branch is on 4.9.0 now
> 
> Remi,
> 
> Thanks for heads up.
> 
> Just to clear the confusion (i guess i have), are we on monthly releases 
> schedule now?
> 
> Regards
> ilya
> 
> On 1/26/16 12:43 PM, Remi Bergsma wrote:
>> Hi all,
>> 
>> Following the release of ACS 4.8.0, master is now on version 4.9.0. I tested 
>> several upgrades and that works fine.
>> 
>> If you have a bug fix, please point your pull request to the 4.7 branch and 
>> it will be merged to 4.8 and master as well after initial merge.
>> 
>> New features will land in master, so please open a PR against that branch.
>> 
>> Regards,
>> Remi
>> 



[GitHub] cloudstack pull request: [4.9+] WIP: Support Java 8, Maven 3.3, Up...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1370#issuecomment-175492873
  
Alright, let's keep this PR hanging then. We should then open another PR 
which fixes the packaging first to switch from JRE7 to JRE8 along with any 
JAVA_OPT or tomcat config changes 


---
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: Quota: findbug fixes

2016-01-27 Thread agneya2001
Github user agneya2001 commented on the pull request:

https://github.com/apache/cloudstack/pull/1289#issuecomment-175500735
  
merging LGTM from @DaanHoogland and @bhaisaab 


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


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

2016-01-27 Thread jenkins
See 

--
[...truncated 28713 lines...]
[INFO] 
[INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud 
---
[INFO] 
[INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) @ 
cloud-quickcloud <<<
[INFO] 
[INFO] --- findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) @ 
cloud-quickcloud ---
[INFO] 
[INFO] --- cobertura-maven-plugin:2.6:instrument (default-cli) @ 
cloud-quickcloud ---
[WARNING] No files to instrument.
[INFO] NOT adding cobertura ser file to attached artifacts list.
[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-quickcloud ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.2:testCompile (default-testCompile) @ 
cloud-quickcloud ---
[INFO] No sources to compile
[INFO] 
[INFO] --- maven-surefire-plugin:2.18.1:test (default-test) @ cloud-quickcloud 
---
[INFO] 
[INFO] <<< cobertura-maven-plugin:2.6:cobertura (default-cli) @ 
cloud-quickcloud <<<
[INFO] 
[INFO] --- cobertura-maven-plugin:2.6:cobertura (default-cli) @ 
cloud-quickcloud ---
[INFO] 
[INFO] Reactor Summary:
[INFO] 
[INFO] Apache CloudStack Developer Tools - Checkstyle Configuration  SUCCESS 
[1.708s]
[INFO] Apache CloudStack . SUCCESS [2.097s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [0.796s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [19.179s]
[INFO] Apache CloudStack Utils ... SUCCESS [1:30.980s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.103s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [53.854s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [27.227s]
[INFO] Apache CloudStack API . SUCCESS [1:47.702s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [16.047s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [31.026s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.095s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [28.736s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [25.467s]
[INFO] Apache CloudStack Core  SUCCESS [1:23.413s]
[INFO] Apache CloudStack Agents .. SUCCESS [35.855s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [35.753s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [14.470s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [2:06.869s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [41.461s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [24.885s]
[INFO] Apache CloudStack Server .. SUCCESS [4:12.530s]
[INFO] Apache CloudStack Framework - Quota ... SUCCESS [37.284s]
[INFO] Apache CloudStack Usage Server  SUCCESS [45.170s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS 
[1:21.112s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.070s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.447s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [55.337s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [47.907s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [29.712s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [27.978s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [26.002s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [22.841s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [35.511s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [13.015s]
[INFO] Apache CloudStack Cloud Engine Service  SUCCESS [7.907s]
[INFO] Apache CloudStack Plugin POM .. SUCCESS [0.982s]
[INFO] Apache CloudStack Plugin - API Rate Limit . SUCCESS [26.507s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS 
[23.600s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS 
[40.136s]
[INFO] Apache CloudStack Plugin - API SolidFire .. SUCCESS [17.941s]
[INFO] Apache CloudStack Plugin - API Discovery .. SUCCESS [23.157s]
[INFO] Apache CloudStack Plugin - ACL Static Role Based .. SUCCESS [14.745s]
[INFO] Apache CloudStack Plugin - Host Anti-Affinity Processor  SUCCESS 
[17.657s]
[INFO] Apache Cloud

[GitHub] cloudstack pull request: engine/schema: Use semantically correct u...

2016-01-27 Thread bhaisaab
Github user bhaisaab closed the pull request at:

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


---
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: engine/schema: Use semantically correct u...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1279#issuecomment-175504959
  
Such a waste


---
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: [4.9+] WIP: Support Java 8, Maven 3.3, Up...

2016-01-27 Thread jburwell
Github user jburwell commented on the pull request:

https://github.com/apache/cloudstack/pull/1370#issuecomment-175523813
  
@bhaisaab if we make the packaging require Java 1.8, then users will be 
automatically upgraded -- defeating the purpose of the deprecation.  It seems 
like an appropriate first step is to audit all dependencies for Java8 support, 
and upgrade all affected dependencies.  For example, we need to upgrade to 
Spring 4 because (Spring 3 does not support Java 
8)[http://spring.io/blog/2013/05/21/spring-framework-4-0-m1-3-2-3-available/].  
In my opinion, we should perform the dependency upgrades as part of 4.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: Quota: findbug fixes

2016-01-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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: OSPF: WIP: adding dynamically routing cap...

2016-01-27 Thread agneya2001
Github user agneya2001 closed the pull request at:

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


---
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-9161: fix the quota marvin tes...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1240#issuecomment-175560143
  
2 PRs, 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: CLOUDSTACK-9161: fix the quota marvin tes...

2016-01-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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-9174: A deleted account result...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1254#issuecomment-175562144
  
@agneya2001 okay, we need one more LGTM on this. I'll picked your marvin 
fix from #1240 , can you also share test results if any?


---
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: Quota: consolidated lockable account chec...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1350#issuecomment-175563484
  
@agneya2001 LGTM again, need one more review to get this merged along with 
some testing (though in this case, we've some unit tests for the change)


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


Unable Creare/CleanUp Virual Router After upgraded to ACS 4.6.2

2016-01-27 Thread mvs babu
Hi All,

We upgraded ACS from 4.3.1 to 4.6.2. After this we tried to create new 
VPC/Network and getting below error from Virtual Router then after router get 
shutdown.

 


We are using XenServer 6.2 and CentOS 6.3.

Please help us to fix this issue.


Thank you,
Suneel Mallela,

AxiomIO.

Re: Unable Creare/CleanUp Virual Router After upgraded to ACS 4.6.2

2016-01-27 Thread mvs babu
Sorry, it is not allowing to attach a screenshot. Please find below link.


http://picpaste.com/pics/VPC-rhH48Qao.1453891619.PNG






Thank you,
Suneel Mallela





From: mvs babu
Sent: ‎Wednesday‎, ‎January‎ ‎27‎, ‎2016 ‎4‎:‎14‎ ‎PM
To: us...@cloudstack.apache.org
Cc: dev@cloudstack.apache.org





Please find attachment.






Thank you,
Suneel Mallela





From: mvs babu
Sent: ‎Wednesday‎, ‎January‎ ‎27‎, ‎2016 ‎4‎:‎08‎ ‎PM
To: us...@cloudstack.apache.org
Cc: dev@cloudstack.apache.org





Hi All,

We upgraded ACS from 4.3.1 to 4.6.2. After this we tried to create new 
VPC/Network and getting below error from Virtual Router then after router get 
shutdown.

 


We are using XenServer 6.2 and CentOS 6.3.

Please help us to fix this issue.


Thank you,
Suneel Mallela,

AxiomIO.

Re: Unable Creare/CleanUp Virual Router After upgraded to ACS 4.6.2

2016-01-27 Thread Harikrishna Patnala
Hi,

Latest system vm template has flask installed in it. Can you please let us 
which system vm template you have used.

Thanks,
Harikrishna


On 27-Jan-2016, at 4:17 pm, mvs babu  wrote:

> Sorry, it is not allowing to attach a screenshot. Please find below link.
> 
> 
> http://picpaste.com/pics/VPC-rhH48Qao.1453891619.PNG
> 
> 
> 
> 
> 
> 
> Thank you,
> Suneel Mallela
> 
> 
> 
> 
> 
> From: mvs babu
> Sent: ‎Wednesday‎, ‎January‎ ‎27‎, ‎2016 ‎4‎:‎14‎ ‎PM
> To: us...@cloudstack.apache.org
> Cc: dev@cloudstack.apache.org
> 
> 
> 
> 
> 
> Please find attachment.
> 
> 
> 
> 
> 
> 
> Thank you,
> Suneel Mallela
> 
> 
> 
> 
> 
> From: mvs babu
> Sent: ‎Wednesday‎, ‎January‎ ‎27‎, ‎2016 ‎4‎:‎08‎ ‎PM
> To: us...@cloudstack.apache.org
> Cc: dev@cloudstack.apache.org
> 
> 
> 
> 
> 
> Hi All,
> 
> We upgraded ACS from 4.3.1 to 4.6.2. After this we tried to create new 
> VPC/Network and getting below error from Virtual Router then after router get 
> shutdown.
> 
> 
> 
> 
> We are using XenServer 6.2 and CentOS 6.3.
> 
> Please help us to fix this issue.
> 
> 
> Thank you,
> Suneel Mallela,
> 
> AxiomIO.



[GitHub] cloudstack pull request: OSPF: WIP: adding dynamically routing cap...

2016-01-27 Thread agneya2001
GitHub user agneya2001 opened a pull request:

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

OSPF: WIP: adding dynamically routing capability to VPC 

adding dynamically routing capability to VPC and adding config item 
structure to VR quagga configuration

OSPF: zone quagga config API

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

$ git pull https://github.com/shapeblue/cloudstack master-quagga

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

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


commit 7390f87f96cdd7b45f2eb341a27dc1567b9b570a
Author: Abhinandan Prateek 
Date:   2015-12-24T11:53:27Z

OSPF: WIP: adding dynamically routing capability to VPC and adding config
item structure to VR quagga configuration

OSPF: zone quagga config API




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


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

2016-01-27 Thread jenkins
See 

Changes:

[abhinandan.prateek] CLOUDSTACK-9161: fix the quota marvin test

[abhinandan.prateek] Quota: findbug fixes

--
[...truncated 28675 lines...]
[INFO] 
[INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud 
---
[INFO] 
[INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) @ 
cloud-quickcloud <<<
[INFO] 
[INFO] --- findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) @ 
cloud-quickcloud ---
[INFO] 
[INFO] --- cobertura-maven-plugin:2.6:instrument (default-cli) @ 
cloud-quickcloud ---
[WARNING] No files to instrument.
[INFO] NOT adding cobertura ser file to attached artifacts list.
[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-quickcloud ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.2:testCompile (default-testCompile) @ 
cloud-quickcloud ---
[INFO] No sources to compile
[INFO] 
[INFO] --- maven-surefire-plugin:2.18.1:test (default-test) @ cloud-quickcloud 
---
[INFO] 
[INFO] <<< cobertura-maven-plugin:2.6:cobertura (default-cli) @ 
cloud-quickcloud <<<
[INFO] 
[INFO] --- cobertura-maven-plugin:2.6:cobertura (default-cli) @ 
cloud-quickcloud ---
[INFO] 
[INFO] Reactor Summary:
[INFO] 
[INFO] Apache CloudStack Developer Tools - Checkstyle Configuration  SUCCESS 
[1.717s]
[INFO] Apache CloudStack . SUCCESS [2.143s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [0.819s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [18.730s]
[INFO] Apache CloudStack Utils ... SUCCESS [1:30.334s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.101s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [53.960s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [28.390s]
[INFO] Apache CloudStack API . SUCCESS [1:49.957s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [17.040s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [29.636s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.108s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [28.338s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [24.756s]
[INFO] Apache CloudStack Core  SUCCESS [1:22.480s]
[INFO] Apache CloudStack Agents .. SUCCESS [35.771s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [37.085s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [14.399s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [2:10.292s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [41.147s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [25.212s]
[INFO] Apache CloudStack Server .. SUCCESS [4:13.143s]
[INFO] Apache CloudStack Framework - Quota ... SUCCESS [37.768s]
[INFO] Apache CloudStack Usage Server  SUCCESS [43.966s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS 
[1:21.356s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.115s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.640s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [53.503s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [49.822s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [29.678s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [26.408s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [25.488s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [20.589s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [35.287s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [12.440s]
[INFO] Apache CloudStack Cloud Engine Service  SUCCESS [7.976s]
[INFO] Apache CloudStack Plugin POM .. SUCCESS [0.943s]
[INFO] Apache CloudStack Plugin - API Rate Limit . SUCCESS [26.471s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS 
[23.695s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS 
[37.016s]
[INFO] Apache CloudStack Plugin - API SolidFire .. SUCCESS [17.441s]
[INFO] Apache CloudStack Plugin - API Discovery .. SUCCESS [23.298s]
[INFO] Apache CloudStack Plugin - ACL Static Role Base

[GitHub] cloudstack pull request: Quota: consolidated lockable account chec...

2016-01-27 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1350#issuecomment-175599430
  
Hi @agneya2001, there are a lot of duplicated lines at 
“QuotaManagerImplTest”, would you mind removing them?

Additionally, the lines you use to set the domainId and the id at the 
AccountVO object, you do not need to execute them, the method that you want to 
test does not need them.



---
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: Quota: consolidated lockable account chec...

2016-01-27 Thread agneya2001
Github user agneya2001 commented on the pull request:

https://github.com/apache/cloudstack/pull/1350#issuecomment-175601093
  
@rafaelweingartner Can you comment on the duplicated lines. There is a way 
you can get to the code and comment there.



---
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: Quota: consolidated lockable account chec...

2016-01-27 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1350#issuecomment-175601814
  
I know that. I just did not want to go one by one and tell you that they 
are duplicated, I believe you can easily see that. Those duplicated lines I 
mentioned are in the test methods you created.


---
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: Quota: consolidated lockable account chec...

2016-01-27 Thread agneya2001
Github user agneya2001 commented on the pull request:

https://github.com/apache/cloudstack/pull/1350#issuecomment-175603740
  
@rafaelweingartner if you mean initialisation of AccountVO to be moved into 
a method, and replacing it with a method call. I guess I would prefer it this 
way as this keeps things simple. 
The initialisation for domainId and id is done as the method will receive 
similar object, so no harm in passing a object that is close to the one passed 
at runtime, though as you said the function tested is not using 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: Quota: consolidated lockable account chec...

2016-01-27 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1350#issuecomment-175609511
  
@agneya2001  that is exactly what I meant. 
I disagree with you, letting this as they are, it is not simple. Why 
duplicate code, when we can do better? Especially if we use a good and 
descriptive method name.

About the initialization, I also disagree with you, at that point those 
values are not used, so they are needed and you do not need to code those lines.
If those attributes are needed somewhere else, they should be used only at 
the point in which they are needed.

Please, keep in mind the less the better. We need quality in our code, not 
quantity.



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


New release manager wanted

2016-01-27 Thread Remi Bergsma
Hi all,


After picking up the RM role last summer, I've had an interesting ride with 
awesome results. First the release principles [1], then stabilizing master and 
finally 3 releases in a row: 4.6 in November, 4.7 in December and 4.8 in 
January (and as a bonus 3 more patch releases in between). As Seb pointed out, 
we could start a new vote at any time and release it. Given that we keep master 
stable, of course.


Hundreds of Pull Requests got merged and each one has had a series of 
integration tests executed [2] before merge. That drove quality and velocity.


I’m proud of the results we achieved and I’m also proud of the people who 
helped make it happen!


The other side of the story, is that it took me a GREAT amount of time to do 
this. Schuberg Philis allowed me to work on it during the day, and next to that 
I also spent many many evenings and weekends working on it. Right now, I feel 
it’s time to scale that down again to normal proportions. I will continue to 
contribute, but I just won't take the lead for now.


Long story short, please step up if you want to take over the RM role. I'd be 
happy to provide some guide lines and help.


Regards,

Remi


[1] 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Release+principles+for+Apache+CloudStack+4.6+and+up

[2] https://github.com/schubergphilis/MCT-shared




[GitHub] cloudstack pull request: Quota: consolidated lockable account chec...

2016-01-27 Thread agneya2001
Github user agneya2001 commented on the pull request:

https://github.com/apache/cloudstack/pull/1350#issuecomment-175624327
  
@rafaelweingartner quality is what we are discussing and I simply do not 
think that putting 3 lines of object initialisation in a method will make it 
high quality. Are you referring to some guidelines that I can see too ? 


---
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: Quota: consolidated lockable account chec...

2016-01-27 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1350#issuecomment-175630573
  
It is cultural; we have to stop duplicating lines; no matter where and no 
matter how simple those blocks may seem. 

We also have to stop coding things that are not needed as you did in the 
test. If a code is not needed, why bother writing it down? I am referring to 
the “accountVO.setId” and accountVO.setDomainId” at the tests you wrote. 

I do not know if ACS has a guideline for that, for me, it looks like common 
sense. That was my opinion, but if others here are ok with the duplication of 
lines, feel free to merge the PR.


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


[GitHub] cloudstack pull request: Quota: consolidated lockable account chec...

2016-01-27 Thread agneya2001
Github user agneya2001 commented on the pull request:

https://github.com/apache/cloudstack/pull/1350#issuecomment-175634392
  
@rafaelweingartner If it is a black box test then the person testing will 
obviously try to pass a realistic data set, that is what I have done. It is 
possible that tomorrow there are some changes to the tested method and the data 
that looks superficial may come into play and expose some erratic behaviour. 
The dataset passed mocks the dataset that will go thru the method in real 
runtime condition. Now offcourse we can see that the data is not being used in 
this method but does that mean I should change the dataset that mocks the real 
value ?


---
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: Refactor system VM default network creati...

2016-01-27 Thread ProjectMoon
Github user ProjectMoon commented on the pull request:

https://github.com/apache/cloudstack/pull/1360#issuecomment-175634193
  
Seems the Jenkins build is complaining about license missing. Don't see 
what file it is, but I assume it's the new unit test file?


---
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: Quota: consolidated lockable account chec...

2016-01-27 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1350#issuecomment-175637289
  
If in a remotely distant future someone that we do not even know yet 
changes that method and make use of some extra data (forgetting to check the 
test cases), the test cases should catch that and then the person that is doing 
that change should fix the tests. That is why we write test cases, for the code 
of today, not for the code of tomorrow.

IMO we should prepare the architecture of software to the future, but the 
code itself should not be developed waiting for some futuristic hypothetical 
condition that may not ever happen.


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


[4.2] System VM volumes stuck in expunging but not removed

2016-01-27 Thread Jeff Hair
Hi,

In a 4.2 environment, after removing the last host from a zone and
destroying the system VMs, they leave behind their volume in the database
stuck in Expunging, but the removed column is non-NULL. This prevents the
deletion of the primary storage. Was this fixed in later versions or does
it still happen?

Thanks,

Jeff


Re: [4.2] System VM volumes stuck in expunging but not removed

2016-01-27 Thread Jeff Hair
Also this is even after waiting for the main expunge thread to run. The
volumes remain non-removed after the expunger runs.

On Wed, Jan 27, 2016 at 1:56 PM, Jeff Hair  wrote:

> Hi,
>
> In a 4.2 environment, after removing the last host from a zone and
> destroying the system VMs, they leave behind their volume in the database
> stuck in Expunging, but the removed column is non-NULL. This prevents the
> deletion of the primary storage. Was this fixed in later versions or does
> it still happen?
>
> Thanks,
>
> Jeff
>



-- 
*Jeff Hair*
Technical Lead and Software Developer

Tel: (+354) 415 0200
j...@greenqloud.com
www.greenqloud.com


[GitHub] cloudstack pull request: Refactor system VM default network creati...

2016-01-27 Thread GabrielBrascher
Github user GabrielBrascher commented on the pull request:

https://github.com/apache/cloudstack/pull/1360#issuecomment-175639158
  
@ProjectMoon Yes, just need to add the license in 
"SecondaryStorageManagerTest" class.


---
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: Quota: consolidated lockable account chec...

2016-01-27 Thread agneya2001
Github user agneya2001 commented on the pull request:

https://github.com/apache/cloudstack/pull/1350#issuecomment-175656788
  
@rafaelweingartner  It is only about realistic test data that is being 
passed to the method tested. You should differentiate between the tested code 
and the code that is testing. The logic/code that is being tested should have 
all the characteristics that you are talking about ie any data that is not 
being used should not be there.
But when it comes to writing tests and creating mocks you do it in a more 
realistic manner without deducing what could be happening inside the method as 
you only test what the method is supposed to be doing. That makes it error 
proof in future and that was what i was talking about.



---
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-9185: [VMware DRS] VM sync fai...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1256#issuecomment-175662863
  
Merging as it has 2LGTMs


---
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-9185: [VMware DRS] VM sync fai...

2016-01-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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: Made the adding new keyboard language sup...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/669#issuecomment-175664522
  
@sedukull @anshul1886 please rebase against latest branch and help re-review


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


[GitHub] cloudstack pull request: CLOUDSTACK-8745 : verify usage after root...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/713#issuecomment-175665150
  
@nitt10prashant please rebase and meld into a single commit


---
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-8805: Domains become inactive ...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/775#issuecomment-175665698
  
@rafaelweingartner @nitin-maharana @DaanHoogland  guys, this is really old; 
can we have a re-review please?


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


[GitHub] cloudstack pull request: CLOUDSTACK-8800 : Improved the listVirtua...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/780#issuecomment-175665865
  
@maneesha-p please rebase against latest branch and resubmit


---
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-8827: Move the VM snapshots st...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/793#issuecomment-175666104
  
@anshul1886 ping


---
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-8833: Fixed Generating url and...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/803#issuecomment-175666319
  
@anshul1886 ping


---
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-8854: Apple Mac OS/X VM get cr...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/828#issuecomment-175667189
  
@sureshanaparti LGTM, can you add a unit test for the new methods and 
rebase against latest master


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your 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: Unable Creare/CleanUp Virual Router After upgraded to ACS 4.6.2

2016-01-27 Thread mvs babu
Hi,


Thank you for response.


Followed below documentation,
http://docs.cloudstack.apache.org/projects/cloudstack-release-notes/en/4.6.0/upgrade/upgrade-4.3.html


and templates are,
http://cloudstack.apt-get.eu/systemvm/4.5/systemvm64template-4.5-xen.vhd.bz2
http://cloudstack.apt-get.eu/systemvm/4.6/systemvm64template-4.6.0-xen.vhd.bz2






Thank you,
Suneel Mallela





From: Harikrishna Patnala
Sent: ‎Wednesday‎, ‎January‎ ‎27‎, ‎2016 ‎5‎:‎26‎ ‎PM
To: us...@cloudstack.apache.org
Cc: dev@cloudstack.apache.org





Hi,

Latest system vm template has flask installed in it. Can you please let us 
which system vm template you have used.

Thanks,
Harikrishna


On 27-Jan-2016, at 4:17 pm, mvs babu  wrote:

> Sorry, it is not allowing to attach a screenshot. Please find below link.
> 
> 
> http://picpaste.com/pics/VPC-rhH48Qao.1453891619.PNG
> 
> 
> 
> 
> 
> 
> Thank you,
> Suneel Mallela
> 
> 
> 
> 
> 
> From: mvs babu
> Sent: ‎Wednesday‎, ‎January‎ ‎27‎, ‎2016 ‎4‎:‎14‎ ‎PM
> To: us...@cloudstack.apache.org
> Cc: dev@cloudstack.apache.org
> 
> 
> 
> 
> 
> Please find attachment.
> 
> 
> 
> 
> 
> 
> Thank you,
> Suneel Mallela
> 
> 
> 
> 
> 
> From: mvs babu
> Sent: ‎Wednesday‎, ‎January‎ ‎27‎, ‎2016 ‎4‎:‎08‎ ‎PM
> To: us...@cloudstack.apache.org
> Cc: dev@cloudstack.apache.org
> 
> 
> 
> 
> 
> Hi All,
> 
> We upgraded ACS from 4.3.1 to 4.6.2. After this we tried to create new 
> VPC/Network and getting below error from Virtual Router then after router get 
> shutdown.
> 
> 
> 
> 
> We are using XenServer 6.2 and CentOS 6.3.
> 
> Please help us to fix this issue.
> 
> 
> Thank you,
> Suneel Mallela,
> 
> AxiomIO.

[GitHub] cloudstack pull request: Quota: consolidated lockable account chec...

2016-01-27 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1350#issuecomment-175667113
  
I do not agree.
If people are comfortable with it, so let it be.


---
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-8858: listVolumes API fails fo...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/830#issuecomment-175667434
  
Merging as it has 2LGTMs


---
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-8858: listVolumes API fails fo...

2016-01-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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-8805: Domains become inactive ...

2016-01-27 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/775#issuecomment-175667904
  
The question here is: will we write a test case for the change? If so, it 
has to be done, otherwise the code is ok.


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


[GitHub] cloudstack pull request: CLOUDSTACK-8850: revertSnapshot command d...

2016-01-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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-8850: revertSnapshot command d...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/831#issuecomment-175668135
  
Only text change and 2LGTMs, 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: CLOUDSTACK-7985: assignVM in Advanced zon...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/844#issuecomment-175668842
  
@ustcweizhou Can you please meld the commits into a single commit, and 
incorporate any remaining changes asked in the review?

Ping, can we have a re-review


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


[GitHub] cloudstack pull request: Bug-ID: CLOUDSTACK-8880: calculate free m...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/847#issuecomment-175669153
  
@kishankavala ping?


---
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: Quick note about $dayjob

2016-01-27 Thread Giles Sirett
Sebastien

Sorry to hear that you have to move on. Wishing you all the best for your 
future plans with skippbox  - a very exciting looking project

You have been a dedicated, thorough and truly inspirational VP for this project 
– I would like to personally thank you for that.

I hope very much that you’re able to stay engaged in the project in some form 
or another

Kind Regards
Giles


[ShapeBlue]
Giles Sirett
CEO ,   ShapeBlue


d:  +44  20 3603 0541 | s: +44 203 603 
0540 | 
 m:  +44 7961112055

e:  giles.sir...@shapeblue.com | t: 
 |  w:  
www.shapeblue.com

a:  53 Chandos Place, Covent Garden London WC2N 4HS UK


[cid:image303bc4.png@5b75fcac.45a282a7]


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




From: Madan Ganesh Velayudham [mailto:madangan...@actonmagic.com]
Sent: 27 January 2016 00:11
To: market...@cloudstack.apache.org; dev@cloudstack.apache.org
Cc: us...@cloudstack.apache.org
Subject: Re: Quick note about $dayjob

All the best Sebastien,

Thank you and Great job as VP role.  Cloudstack is getting stronger and 
stronger.

World is very small and believe there will be scope to work again.

Cheers,
Madan

On Tue, Jan 26, 2016 at 8:00 PM Sebastien Goasguen 
mailto:run...@gmail.com>> wrote:
Hi folks,

As some of you might have guessed I am changing $dayjob.

This is an exciting time for me, but it also means that I will become less 
involved with CloudStack. ( I said *less* involved, did not say I was going to 
disappear). I have informed the PMC and offered to stop my VP role ahead of the 
March deadline. As you know we have a one-year rotation for the VP role. 
However if the PMC does not see a problem with it I will gladly continue until 
March.

I will keep this brief but I wanted to let you know so that you don’t wonder 
why my involvement in recent releases and votes has decreased.

It is an exciting time for CloudStack as well, with ever greater releases ( in 
large part to Remi, Wilder and a few others), a strong proposal to offer LTS 
and the commitment from Accelerite.

Change is good, challenges are opportunities says Sensei Wu from Ninjago :)

PS:
If you wonder, I am launching my own container startup called Skippbox. I will 
grant myself a shameless plug, and if you know anyone interested in containers 
and kubernetes I sure could use some help ( https://github.com/skippbox , 
http://www.skippbox.com/blog/), github stars and PRs :)

Cheers,

-Sebastien
--
Madan Ganesh Velayudham
Founder & CEO
ActOnMagic

Data-Driven Hybrid Cloud Management Delivered!
www.actoncloud.com
Save Cost! Optimise Performance! Single Pane Of Glass!
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


[GitHub] cloudstack pull request: Removal of class AgentBasedStandaloneCons...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/855#issuecomment-175669973
  
Merging based on comments and 2LGTMs


---
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: Removal of class AgentBasedStandaloneCons...

2016-01-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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-8906: /var/log/cloud/ doesn't ...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/883#issuecomment-175671185
  
@SudharmaJain Please rebase and send for a re-review


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


[GitHub] cloudstack pull request: CLOUDSTACK-8908 After copying the templat...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/896#issuecomment-175671692
  
Ping review?


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


[GitHub] cloudstack pull request: Usage event fixes for deleted accounts

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/924#issuecomment-175672214
  
@ProjectMoon can you rebase against latest master, and squash changes to a 
single commit


---
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: [4.7] CLOUDSTACK-7984: Collect network st...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/926#issuecomment-175672456
  
@ustcweizhou Can you please rebase against latest master


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


disable github notifications ?

2016-01-27 Thread sebgoa
Hi folks,

Shall we disable github notifications to dev@ ?

It has become quite noisy.

On the other end you see all the comments fly by...

-sebastien


[GitHub] cloudstack pull request: CLOUDSTACK-8970 Centos 6.{1,2,3,4,5} gues...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/956#issuecomment-175673522
  
@SudharmaJain please rebase against latest 


---
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-8751 minimise downtime of netw...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/866#issuecomment-175670480
  
@bvbharat please squash the changes into a single commit

ping, re-review please


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


[GitHub] cloudstack pull request: Fixed return type Void to void in DataMot...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/969#issuecomment-175674716
  
LGTM
Merging based on comments, test results shared and 2+LGTMs


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


[GitHub] cloudstack pull request: Fixed return type Void to void in DataMot...

2016-01-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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: disable github notifications ?

2016-01-27 Thread Mike Tutkowski
Yeah, it is quite noisy. I would be fine with us disabling them.

On Wednesday, January 27, 2016, sebgoa  wrote:

> Hi folks,
>
> Shall we disable github notifications to dev@ ?
>
> It has become quite noisy.
>
> On the other end you see all the comments fly by...
>
> -sebastien
>


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


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

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/977#issuecomment-175675613
  
@ustcweizhou please rebase against latest and send for a re-review


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


[GitHub] cloudstack pull request: Usage event fixes for deleted accounts

2016-01-27 Thread ProjectMoon
Github user ProjectMoon commented on the pull request:

https://github.com/apache/cloudstack/pull/924#issuecomment-175675521
  
Squashing and rebasing is definitely needed. But isn't this a bug fix 
though? And thus should go on to 4.6 and then be forward merged?


---
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-5822: keep user-added sshkeys ...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1044#issuecomment-175677195
  
LGTM
Merging based on 2+LGTMs, test results shared in comments


---
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: disable github notifications ?

2016-01-27 Thread Wido den Hollander


On 27-01-16 16:01, Mike Tutkowski wrote:
> Yeah, it is quite noisy. I would be fine with us disabling them.
> 

Agree. I wouldn't mind.

> On Wednesday, January 27, 2016, sebgoa  wrote:
> 
>> Hi folks,
>>
>> Shall we disable github notifications to dev@ ?
>>
>> It has become quite noisy.
>>
>> On the other end you see all the comments fly by...
>>
>> -sebastien
>>
> 
> 


[GitHub] cloudstack pull request: CLOUDSTACK-8731-checking usage event for ...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1048#issuecomment-175679311
  
LGTM
Merging based on review, 2+LGTM and test result shared in comments


---
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: Remove extraenous log dir and add catalin...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1087#issuecomment-175680172
  
@PaulAngus please rebase against latest master and force push


---
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: update release to 4.7.0 so it use proper ...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1103#issuecomment-175681455
  
@pdion891 please rebase against latest and squash commits


---
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: Add47xmessagesproperties transifex

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1098#issuecomment-175680903
  
@milamberspace is this still valid, now that we've a 4.8 and master's on 
4.9? cc @remibergsma 


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


[GitHub] cloudstack pull request: bugfix ui/scripts/accounts.js L1339

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1104#issuecomment-175681911
  
@wolf29  please rebase against latest and meld commits into a single commit


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


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

2016-01-27 Thread jenkins
See 

--
[...truncated 28675 lines...]
[INFO] 
[INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud 
---
[INFO] 
[INFO] <<< findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) @ 
cloud-quickcloud <<<
[INFO] 
[INFO] --- findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) @ 
cloud-quickcloud ---
[INFO] 
[INFO] --- cobertura-maven-plugin:2.6:instrument (default-cli) @ 
cloud-quickcloud ---
[WARNING] No files to instrument.
[INFO] NOT adding cobertura ser file to attached artifacts list.
[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-quickcloud ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.2:testCompile (default-testCompile) @ 
cloud-quickcloud ---
[INFO] No sources to compile
[INFO] 
[INFO] --- maven-surefire-plugin:2.18.1:test (default-test) @ cloud-quickcloud 
---
[INFO] 
[INFO] <<< cobertura-maven-plugin:2.6:cobertura (default-cli) @ 
cloud-quickcloud <<<
[INFO] 
[INFO] --- cobertura-maven-plugin:2.6:cobertura (default-cli) @ 
cloud-quickcloud ---
[INFO] 
[INFO] Reactor Summary:
[INFO] 
[INFO] Apache CloudStack Developer Tools - Checkstyle Configuration  SUCCESS 
[1.789s]
[INFO] Apache CloudStack . SUCCESS [2.316s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [0.793s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [18.951s]
[INFO] Apache CloudStack Utils ... SUCCESS [1:30.320s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.108s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [54.474s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [27.165s]
[INFO] Apache CloudStack API . SUCCESS [1:48.482s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [16.849s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [29.154s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.082s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [28.327s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [25.272s]
[INFO] Apache CloudStack Core  SUCCESS [1:21.584s]
[INFO] Apache CloudStack Agents .. SUCCESS [35.939s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [38.605s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [14.150s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [2:07.382s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [41.644s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [25.520s]
[INFO] Apache CloudStack Server .. SUCCESS [4:15.134s]
[INFO] Apache CloudStack Framework - Quota ... SUCCESS [37.150s]
[INFO] Apache CloudStack Usage Server  SUCCESS [44.138s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS 
[1:24.515s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.078s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.479s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [53.884s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [48.559s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [29.820s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [26.574s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [30.466s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [22.896s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [35.369s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [12.250s]
[INFO] Apache CloudStack Cloud Engine Service  SUCCESS [7.876s]
[INFO] Apache CloudStack Plugin POM .. SUCCESS [0.994s]
[INFO] Apache CloudStack Plugin - API Rate Limit . SUCCESS [26.935s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS 
[23.413s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS 
[36.084s]
[INFO] Apache CloudStack Plugin - API SolidFire .. SUCCESS [18.189s]
[INFO] Apache CloudStack Plugin - API Discovery .. SUCCESS [23.579s]
[INFO] Apache CloudStack Plugin - ACL Static Role Based .. SUCCESS [16.788s]
[INFO] Apache CloudStack Plugin - Host Anti-Affinity Processor  SUCCESS 
[16.775s]
[INFO] Apache Cloud

[GitHub] cloudstack pull request: CLOUDSTACK-9064: The users should be able...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1115#issuecomment-175682672
  
@agneya2001 @abhinandanprateek please review and close this PR


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


[GitHub] cloudstack pull request: CID-1338387: Deletion of method endPointS...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1124#issuecomment-175683387
  
LGTM, @rafaelweingartner can you please rebase against latest master so we 
can merge 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-9088: Update the description f...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1126#issuecomment-175683694
  
LGTM anyone, needs one more?


---
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-5822: keep user-added sshkeys ...

2016-01-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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-9091: Update testpath for para...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1130#issuecomment-175683955
  
@sanju1010 looking forward to the test results


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


[GitHub] cloudstack pull request: CLOUDSTACK-9100: ISO.CREATE/TEMPLATE.CREA...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1157#issuecomment-175685066
  
@SudharmaJain can you comment on @pedro-martins 's comment?


---
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-8731-checking usage event for ...

2016-01-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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-9136: remove ssh keypairs alon...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1212#issuecomment-175687426
  
LGTM, if we don't have any outstanding objections -- let's merge 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: Bug-ID: CLOUDSTACK-8654: Added guest_os m...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1244#issuecomment-175687981
  
@kishankavala please rebase against latest master and move the sqls to the 
recent sql paths


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


[GitHub] cloudstack pull request: CLOUDSTACK-9196: Fixing null pointer exce...

2016-01-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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: Removed unused variables from "NetworkSta...

2016-01-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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: disable github notifications ?

2016-01-27 Thread sebgoa
I don't know exactly what to do,

but I believe Rene left the list because it was too noisy, …that's not good.

On Jan 27, 2016, at 4:10 PM, Giles Sirett  wrote:

> Is there any other way of doing it ? (i.e can we get a summary of the 
> comments sent to the list)
> 
> I filter out the GH notifications (because I too find them noisy) but then 
> often miss discussions that happen as GH comments
> 
> In theory, we have the rule "if it happens, it happens on dev@", I feel may 
> of the conversations on GH really should be somehow shown on dev@
> 
> 
> Kind Regards
> Giles
> 
> 
> 
>   
> Giles Sirett
> CEO,   ShapeBlue
> d:+44  20 3603 0541 | s: +44 203 603 0540   |  m: +44 7961112055
> e:giles.sir...@shapeblue.com | t:   |  w: 
> www.shapeblue.com
> a: 53 Chandos Place, Covent Garden London WC2N 4HS UK
> 
> 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.
> 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.
> 
> 
> -Original Message-
> From: sebgoa [mailto:run...@gmail.com] 
> Sent: 28 January 2016 00:00
> To: dev@cloudstack.apache.org
> Subject: disable github notifications ?
> 
> Hi folks,
> 
> Shall we disable github notifications to dev@ ?
> 
> It has become quite noisy.
> 
> On the other end you see all the comments fly by...
> 
> -sebastien
> 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



[GitHub] cloudstack pull request: trailing commas in javascripts removed

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1288#issuecomment-175689026
  
LGTM, merging based on comments and tests


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


[GitHub] cloudstack pull request: trailing commas in javascripts removed

2016-01-27 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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-9208: Assertion Error in VM_PO...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1307#issuecomment-175689441
  
Ping?


---
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-8302: Removing snapshots on RB...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1230#issuecomment-175687764
  
@dmytro-shevchenko can you rebase against latest master (with 4.9xx as the 
version) etc.


---
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-9211: Support passing vRAM siz...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1310#issuecomment-175689785
  
LGTM, can we have regression test results against VMware then we would be 
able to merge 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-9215]Test to verify vm deploy...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1316#issuecomment-175690062
  
@sanju1010 please squash changes in to a single commit and comment if all 
outstanding issues are resolved 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: CLOUDSTACK-8847: ListServiceOfferings is ...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1321#issuecomment-175690290
  
LGTM, should we merge 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: Followup fix for #1162: Add support for n...

2016-01-27 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1330#issuecomment-175690469
  
LGTM, let's merge after some 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.
---


  1   2   3   >