[GitHub] cloudstack pull request: CLOUDSTACK-9264: Make /32 static routes f...

2016-02-01 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1383#issuecomment-178167084
  
LGTM based on the code. any test for this available?


---
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-9266: Make deleting static rou...

2016-02-01 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1386#issuecomment-178165074
  
makes sense looks good, was it put in production without test?


---
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-9252: Support configurable NFS...

2016-02-01 Thread nvazquez
Github user nvazquez commented on the pull request:

https://github.com/apache/cloudstack/pull/1361#issuecomment-178190394
  
Hi @rafaelweingartner 

Thanks for your comments! I've been working following your indications, 
however I didn't create any test case. 
I wanted to ask you, as this methods use properties from database, which 
way could be better to test it. I thought on unit tests, mocking daos to 
retrieve nfs version, would that be enough?


---
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 #3130

2016-02-01 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 
[2.029s]
[INFO] Apache CloudStack . SUCCESS [2.857s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [0.959s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [19.438s]
[INFO] Apache CloudStack Utils ... SUCCESS [1:32.635s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.104s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [52.990s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [28.082s]
[INFO] Apache CloudStack API . SUCCESS [1:47.766s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [16.232s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [29.937s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.092s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [27.993s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [25.310s]
[INFO] Apache CloudStack Core  SUCCESS [1:20.985s]
[INFO] Apache CloudStack Agents .. SUCCESS [36.824s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [36.255s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [14.270s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [2:05.927s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [40.704s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [25.853s]
[INFO] Apache CloudStack Server .. SUCCESS [4:12.186s]
[INFO] Apache CloudStack Framework - Quota ... SUCCESS [37.804s]
[INFO] Apache CloudStack Usage Server  SUCCESS [44.370s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS 
[1:23.659s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.068s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.486s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [54.844s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [48.121s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [29.622s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [26.736s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [22.162s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [20.802s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [35.681s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [12.018s]
[INFO] Apache CloudStack Cloud Engine Service  SUCCESS [7.537s]
[INFO] Apache CloudStack Plugin POM .. SUCCESS [1.002s]
[INFO] Apache CloudStack Plugin - API Rate Limit . SUCCESS [26.928s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS 
[23.996s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS 
[36.661s]
[INFO] Apache CloudStack Plugin - API SolidFire .. SUCCESS [17.359s]
[INFO] Apache CloudStack Plugin - API Discovery .. SUCCESS [23.395s]
[INFO] Apache CloudStack Plugin - ACL Static Role Based .. SUCCESS [15.801s]
[INFO] Apache CloudStack Plugin - Host Anti-Affinity Processor  SUCCESS 
[16.509s]
[INFO] Apache 

[GitHub] cloudstack pull request: Display hostname the VPC router runs on

2016-02-01 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1384#issuecomment-178165509
  
convenient cos(mat)ic change: LGTM


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


[GitHub] cloudstack pull request: Restore iptables at once using iptables-r...

2016-02-01 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1389#issuecomment-178209325
  
@borisroman 
  systemvm/patches/debian/config/opt/cloud/bin/cs_iptables_save.py needs a 
license.


---
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-7836: Make s2s VPN CIDR list l...

2016-02-01 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1388#issuecomment-178211407
  
@remibergsma a better solution in the make?


---
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 Create/CleanUp Virual Router After upgraded to ACS 4.6.2

2016-02-01 Thread mvs babu
Hi All,


I have tested ACS 4.7.1 with 4.6 template and getting same below “flask” error 
in Virtual Router, due to this error we are unable to create new networks/clean 
up existing Virtual Router.


File “/opt/cloud/bin/baremetal-vr.py”, line 28, in 




from flask import Flask

ImportError: No module name flask


Thank you,
Suneel Mallela
AxiomIO.





From: mvs babu
Sent: ‎Friday‎, ‎January‎ ‎29‎, ‎2016 ‎6‎:‎36‎ ‎PM
To: us...@cloudstack.apache.org
Cc: dev@cloudstack.apache.org





Sure, I will test with 4.7.1 and get back with status.

Simon, I din not get you on below, can you please elaborate it.



“The new method of pushing features into the virtual routers is to utilize the 
mounted ISO for deployment”


Thank you,
Suneel Mallela

AxiomIO



From: Simon Weller
Sent: ‎Friday‎, ‎January‎ ‎29‎, ‎2016 ‎6‎:‎29‎ ‎PM
To: us...@cloudstack.apache.org
Cc: dev@cloudstack.apache.org






Suneel,

The 4.6.x virtual routers work with 4.7+ now. The new method of pushing 
features into the virtual routers is to utilize the mounted ISO for deployment.

Not having to upgrade the routers with every release saves a huge amount of 
time and also simplifies deployment of new releases.

- Si



From: mvs babu 
Sent: Friday, January 29, 2016 6:24 AM
To: us...@cloudstack.apache.org
Cc: dev@cloudstack.apache.org
Subject: Re: Unable Create/CleanUp Virual Router After upgraded to ACS 4.6.2

Hi Remi,

Thanks for suggestion, sure we will test with 4.7. But, there is no System VM 
Template available for 4.7.


Thank you,
Suneel Mallela

AxiomIO.



From: Remi Bergsma
Sent: ‎Friday‎, ‎January‎ ‎29‎, ‎2016 ‎5‎:‎45‎ ‎PM
To: us...@cloudstack.apache.org
Cc: dev@cloudstack.apache.org





Hi Suneel,

If you want to upgrade, use 4.7.1 instead of 4.6.x as many bugs have been 
fixed. In another thread we discussed a timeout setting you can bump to make it 
work, too.

Regards,
Remi




On 29/01/16 12:17, "mvs babu"  wrote:

>Hi All,
>
>Getting same issue (http://picpaste.com/pics/VPC-rhH48Qao.1454066282.PNG) with 
>below stable/latest builds of System VM Template.
>
>
>
>http://jenkins.buildacloud.org/job/build-systemvm64-master/lastStableBuild/artifact/tools/appliance/dist/systemvm64template-master-4.6.0-xen.vhd.bz2
>
>http://jenkins.buildacloud.org/job/build-systemvm64-master/lastSuccessfulBuild/artifact/tools/appliance/dist/systemvm64template-master-4.6.0-xen.vhd.bz2
>
>Please help to resolve this issue ASAP, because we planning to upgrade our 
>production environment from 4.3.1.
>
>
>Thank you,
>Suneel Mallela
>
>
>
>
>
>From: mvs babu
>Sent: ‎Wednesday‎, ‎January‎ ‎27‎, ‎2016 ‎5‎:‎34‎ ‎PM
>To: us...@cloudstack.apache.org
>Cc: dev@cloudstack.apache.org
>
>
>
>
>
>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: Allow VM snapshots and volume snapshots t...

2016-02-01 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/672#discussion_r51486290
  
--- Diff: 
plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java
 ---
@@ -4251,6 +4257,34 @@ public void rebootVM(final Connection conn, final VM 
vm, final String vmName) th
 }
 }
 
+protected String skipOrRemoveSR(Connection conn, SR sr) {
+if (sr == null) {
+return null;
+}
+if (s_logger.isDebugEnabled()) {
+s_logger.debug(logX(sr, "Removing SR"));
+}
+try {
+Set vdis = sr.getVDIs(conn);
+for (VDI vdi : vdis) {
+Map 
currentOperation = vdi.getCurrentOperations(conn);
+if (currentOperation == null || currentOperation.size() == 
0) {
+continue;
+}
+return null;
+}
+removeSR(conn, sr);
+return null;
+} catch (XenAPIException e) {
+s_logger.warn(logX(sr, "Unable to get current opertions " + 
e.toString()), e);
+} catch (XmlRpcException e) {
+s_logger.warn(logX(sr, "Unable to get current opertions " + 
e.getMessage()), e);
+}
+String msg = "Remove SR failed";
+s_logger.warn(msg);
+return msg;
--- End diff --

Don't worry about being rude with me. worry about building the best system 
we can ;)


---
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 #3131

2016-02-01 Thread jenkins
See 

--
[...truncated 28685 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 
[2.177s]
[INFO] Apache CloudStack . SUCCESS [3.296s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [1.002s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [19.606s]
[INFO] Apache CloudStack Utils ... SUCCESS [1:32.306s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.107s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [54.040s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [27.280s]
[INFO] Apache CloudStack API . SUCCESS [1:49.571s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [16.258s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [30.781s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.093s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [28.029s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [24.860s]
[INFO] Apache CloudStack Core  SUCCESS [1:23.464s]
[INFO] Apache CloudStack Agents .. SUCCESS [36.272s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [36.637s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [13.984s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [2:07.943s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [40.675s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [25.296s]
[INFO] Apache CloudStack Server .. SUCCESS [4:15.533s]
[INFO] Apache CloudStack Framework - Quota ... SUCCESS [37.482s]
[INFO] Apache CloudStack Usage Server  SUCCESS [44.002s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS 
[1:23.896s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.070s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.540s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [53.354s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [48.958s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [30.545s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [26.447s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [25.669s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [20.116s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [35.201s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [12.241s]
[INFO] Apache CloudStack Cloud Engine Service  SUCCESS [7.932s]
[INFO] Apache CloudStack Plugin POM .. SUCCESS [1.030s]
[INFO] Apache CloudStack Plugin - API Rate Limit . SUCCESS [26.445s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS 
[24.005s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS 
[35.700s]
[INFO] Apache CloudStack Plugin - API SolidFire .. SUCCESS [17.551s]
[INFO] Apache CloudStack Plugin - API Discovery .. SUCCESS [23.849s]
[INFO] Apache CloudStack Plugin - ACL Static Role Based .. SUCCESS [14.751s]
[INFO] Apache CloudStack Plugin - Host Anti-Affinity Processor  SUCCESS 
[16.959s]
[INFO] Apache 

[GitHub] cloudstack pull request: CLOUDSTACK-9252: Support configurable NFS...

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

https://github.com/apache/cloudstack/pull/1361#issuecomment-178267025
  
Hi @nvazquez, 
I see that you have created a spring bean as I said. However, I noticed 
some things that may not be needed or that seemed out of place to me. Before I 
start pointing them out, do you know how spring works? It is not a problem not 
to know in depth this kind of framework, but if you do not I could prepare some 
explanation in detail, so it facilitates in the future for you.

At the “spring-engine-storage-image-core-context.xml” line 31, I 
noticed that you added the bean “imageStoreDetailsUtilImpl” as a dependency 
of the “templateServiceImpl” bean; that is not necessary, if the bean 
“templateServiceImpl” has a property of type 
“imageStoreDetailsUtilImpl” annotated with "@Inject/@Autowired", Spring 
will automatically sort and create the hierarchy of beans dependencies to 
instantiate and inject. That “depends-on” configuration can be used for 
some things that are slightly different; if you are curious about that we can 
chat in off, just call me on slack or shoot me an email.

Still on “spring-engine-storage-image-core-context.xml” at line 40, you 
declared the bean “imageStoreDetailsUtilImpl”, that is only necessary if 
you were not using the “@Component” annotation (there are others that you 
could use too, such as @Service, @Bean and others, each one to mark a different 
use type of bean); having said that, there is no need to declare the bean in 
the XML.

 If you tried to build and run the ACS with your changes, but ended up with 
the application not going up because some problem with Spring dependency 
resolution; that might have happened because of ACS application contexts 
hierarchy, If that happened I can help you find the best place to declare the 
bean; otherwise, you can just use the annotation that is fine, no need to 
re-declare the bean in an XML file. I do not think that problem will happen 
because the bean will be created at one of the top-level application contexts 
of ACS.

Now about the bean itself; I noticed that you created an interface to 
declare the bean's methods. Normally, when we are creating DAO or service 
classes that is the right way to do things, create an interface and then the 
implementation, allowing to use object orientation (O.O.) to change the 
implementation in the future with configurations in an XML file; (an opinion) 
this is not the same as creating code for the future, but it is preparing the 
architecture of a system for the future, I dislike the first one and like the 
second one. However, with the use of annotations, it is not that easy to change 
implementation as it is when using XML spring beans declaration; it is not 
possible to inject a different object that implemented the same interface, 
since annotations make everything pretty straightforward, so I think it is 
better to lose the interface and work just with a single class that is the 
component itself.

Additionally, I noticed that in your interface (that I suggest you not to 
use in this specific case) you extended the interface “Manager” that brings 
a lot of things that you do not use, I am guessing you did that because you 
have seen some other classes, and they all do that. Well, guess what, in your 
case that is not needed. Actually, in most cases that the "Manager" interface 
is being used that interface is not needed; I find the “Manager” interface 
hierarchy a real nightmare, but that is a topic for another chat. 

In all of the places you injected the” imageStoreDetailsUtil” object, I 
suggest you removing the “_” from the attribute name and making them 
private.

Now the problem with poor application architecture planning appears (it is 
not your fault, you are actually doing a great job). In some 
“service/manager/others” that should work as singletons, but are not, your 
“@Inject” will not work.
These classes “VmwareStorageManagerImpl”, “VmwareStorageProcessor”, 
“VmwareStorageSubsystemCommandHandler” and “DownloadManagerImpl” are 
manually instantiated (I might have missed some other), which means that they 
do not pass through the Spring framework lifecycle, which means that @Inject on 
them will not work. To transform them into Spring managed objects, it would 
require much more effort than you should be doing (you are already doing 
applying a great effort on this). What you can do is to get that bean during 
the object initialization at their constructor; you can use the 
“com.cloud.utils.component.ComponentContext.getApplicationContext()” to 
retrieve the Spring application context, and then the getBean method, to 
retrieve the desired bean; then you are good to set that bean in an attribute 
of the object and use it as you are now. Just do not forget to remove the 
@Inject 

Re: upgrading 4.5.2 -> 4.6.0 virtualrouter upgrade timeout

2016-02-01 Thread Andrei Mikhailovsky
Hi Remi,

Is this patched merged into 4.7.1 or 4.8.0, which was recently released? I am 
planning to do the upgrade and wanted to double check.

Thanks

Andrei
- Original Message -
> From: "Remi Bergsma" 
> To: "dev" 
> Sent: Tuesday, 5 January, 2016 11:20:31
> Subject: Re: upgrading 4.5.2 -> 4.6.0 virtualrouter upgrade timeout

> Hi Andrei,
> 
> You indeed need to build CloudStack for this to work.
> 
> You can create packages with ./packaging/package.sh script in the source tree.
> The PR is against 4.7 and when you create RPMs those will be 4.7.1-SHAPSHOT. I
> do run this in production and it resolved the issue. Let me know if it works
> for you too.
> 
> Regards,
> Remi
> 
> 
> 
> 
> On 05/01/16 10:07, "Andrei Mikhailovsky"  wrote:
> 
>>Hi Remi,
>>
>>I've not tried the patch. I've missed it. Do I need to rebuild the ACS to 
>>apply
>>the patch or would making changes to the two files suffice?
>>
>>Thanks
>>
>>Andrei
>>- Original Message -
>>> From: "Remi Bergsma" 
>>> To: "dev" 
>>> Sent: Tuesday, 5 January, 2016 05:49:05
>>> Subject: Re: upgrading 4.5.2 -> 4.6.0 virtualrouter upgrade timeout
>>
>>> Hi Andrei,
>>> 
>>> Did you try it in combination with the patch I created (PR1291)? You need 
>>> both
>>> changes.
>>> 
>>> Regards, Remi
>>> 
>>> Sent from my iPhone
>>> 
 On 04 Jan 2016, at 22:17, Andrei Mikhailovsky  wrote:
 
 Hi Remi,
 
 Thanks for your reply. However, your suggestion of increasing the
 router.aggregation.command.each.timeout didn't help. I've tried setting the
 value to 120 at no avail. Still fails with the same error.
 
 Andrei
 
 - Original Message -
> From: "Remi Bergsma" 
> To: "dev" 
> Sent: Monday, 4 January, 2016 10:44:43
> Subject: Re: upgrading 4.5.2 -> 4.6.0 virtualrouter upgrade timeout
 
> Hi Andrei,
> 
> Missed that mail, sorry. I created a PR that allows for longer timeouts 
> [1].
> 
> Also, you can bump the router.aggregation.command.each.timeout global 
> setting to
> say 15-30 so it will allow to boot.
> 
> Next, we need to find why it takes so long in the first place. In our
> environment it at least starts now.
> 
> Regards,
> Remi
> 
> [1] https://github.com/apache/cloudstack/pull/1291
> 
> 
> 
> 
> 
>> On 04/01/16 11:41, "Andrei Mikhailovsky"  wrote:
>> 
>> Hello guys,
>> 
>> Tried the user's mailing list without any luck. Perhaps the dev guys 
>> know if
>> this issue is being looked at for the next release?
>> 
>> I've just upgraded to 4.6.2 and have similar issues with three virtual 
>> routers
>> out of 22 in total. They are all failing exactly the same way as 
>> described
>> here.
>> 
>> Has anyone found a permanent workaround for this issue?
>> 
>> Thanks
>> 
>> Andrei
>> 
>> - Original Message -
>>> From: "Stephan Seitz" 
>>> To: "users" 
>>> Sent: Monday, 30 November, 2015 19:53:57
>>> Subject: Re: upgrading 4.5.2 -> 4.6.0 virtualrouter upgrade timeout
>> 
>>> Does anybody else experiemce problems due to (very) slow deployment of
>>> VRs?
>>> 
>>> 
>>> Am Dienstag, den 24.11.2015, 16:31 +0100 schrieb Stephan Seitz:
 Update / FYI:
 After faking the particular VRu in sql, I tried to restart that
 network,
 and it always fails. To me it looks like the update_config.py - which
 takes almost all cpu ressources - runs way longer any watchdog will
 accept.
 
 I'm able to mitigate that by very nasty workarounds:
 a) start the router
 b) wait until its provisioned
 c) restart cloudstack-management
 d)  update vm_instance
 set state='Running',
 power_state='PowerOn' where name = 'r-XXX-VM';
 e) once: update domain_router
 set template_version="Cloudstack Release 4.6.0 Wed Nov 4 08:22:47 UTC
 2015",
 scripts_version="546c9e7ac38e0aa16ecc498899dac8e2"
 where id=XXX;
 f) wait until update_config.py finishes (for me thats about 15
 minutes)
 
 Since I expect the need for VR restarts in the future, this behaviour
 is
 somehow unsatisfying. It needs a lot of errorprone intervention.
 
 I'm quite unsure if it's introduced with the update or the particular
 VR
 just has simply not been restarted after getting configured with lots
 of
 ips and rules.
 
 
 Am Dienstag, den 24.11.2015, 12:29 +0100 schrieb Stephan Seitz:

Re: [RESULT][VOTE] Apache CloudStack 4.8.0

2016-02-01 Thread Andrei Mikhailovsky
Hi 

Could you please point to the release notes document for both 4.8.0 and 4.7.1? 
I couldn't find anything on the cloudstack.org.

Many thanks

Andrei

- Original Message -
> From: "Remi Bergsma" 
> To: "dev" 
> Sent: Tuesday, 26 January, 2016 07:29:33
> Subject: [RESULT][VOTE] Apache CloudStack 4.8.0

> Hi all,
> 
> After 72+ hours, the vote for CloudStack 4.8.0 [1] *passes* with 5 PMC + 1
> non-PMC votes.
> 
> +1 (PMC / binding)
> * Daan
> * Milamber
> * Remi
> * Boris
> * Nux
> 
> +1 (non binding)
> * Glenn
> 
> 0
> Suresh Sadhu
> 
> -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.
> 
> [1] http://cloudstack.markmail.org/message/crhpnaz7kjexa3pa


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

2016-02-01 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.796s]
[INFO] Apache CloudStack . SUCCESS [2.115s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [0.784s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [21.257s]
[INFO] Apache CloudStack Utils ... SUCCESS [1:31.492s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.114s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [52.999s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [27.168s]
[INFO] Apache CloudStack API . SUCCESS [1:48.714s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [17.218s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [29.707s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.085s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [28.297s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [25.300s]
[INFO] Apache CloudStack Core  SUCCESS [1:23.253s]
[INFO] Apache CloudStack Agents .. SUCCESS [36.242s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [36.735s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [14.478s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [2:07.709s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [40.893s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [25.495s]
[INFO] Apache CloudStack Server .. SUCCESS [4:16.517s]
[INFO] Apache CloudStack Framework - Quota ... SUCCESS [37.275s]
[INFO] Apache CloudStack Usage Server  SUCCESS [44.177s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS 
[1:22.032s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.069s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.421s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [55.359s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [48.519s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [29.528s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [26.429s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [31.939s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [22.904s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [35.408s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [11.966s]
[INFO] Apache CloudStack Cloud Engine Service  SUCCESS [7.756s]
[INFO] Apache CloudStack Plugin POM .. SUCCESS [1.015s]
[INFO] Apache CloudStack Plugin - API Rate Limit . SUCCESS [26.650s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS 
[23.732s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS 
[37.830s]
[INFO] Apache CloudStack Plugin - API SolidFire .. SUCCESS [17.333s]
[INFO] Apache CloudStack Plugin - API Discovery .. SUCCESS [23.279s]
[INFO] Apache CloudStack Plugin - ACL Static Role Based .. SUCCESS [14.939s]
[INFO] Apache CloudStack Plugin - Host Anti-Affinity Processor  SUCCESS 
[16.856s]
[INFO] Apache 

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

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

https://github.com/apache/cloudstack/pull/1360#issuecomment-177962933
  
I would prefer just leaving one pull request here, but with two commits. 
Each commit would have the file changes + unit tests for the SSVM and CPVM, 
respectively. Not sure a JIRA ticket is really needed for this in my opinion.


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


[GitHub] cloudstack pull request: CLOUDSTACK-7836: Make s2s VPN CIDR list l...

2016-02-01 Thread remibergsma
Github user remibergsma closed the pull request at:

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


---
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: Display hostname the VPC router runs on

2016-02-01 Thread borisroman
Github user borisroman commented on the pull request:

https://github.com/apache/cloudstack/pull/1384#issuecomment-178064538
  
LGTM :+1: Based on screenshots and code-review

Already running in PROD at SBP.


---
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-9266: Make deleting static rou...

2016-02-01 Thread borisroman
Github user borisroman commented on the pull request:

https://github.com/apache/cloudstack/pull/1386#issuecomment-178064345
  
LGTM :+1: 

Already running in PROD at SBP. 


---
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-9264: Make /32 static routes f...

2016-02-01 Thread borisroman
Github user borisroman commented on the pull request:

https://github.com/apache/cloudstack/pull/1383#issuecomment-178064659
  
LGTM :+1: Based on codereview

Already running in PROD at SBP.


---
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 #3127

2016-02-01 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 
[2.273s]
[INFO] Apache CloudStack . SUCCESS [3.392s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [1.009s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [19.642s]
[INFO] Apache CloudStack Utils ... SUCCESS [1:31.068s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.123s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [53.432s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [26.802s]
[INFO] Apache CloudStack API . SUCCESS [1:49.870s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [16.120s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [29.338s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.104s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [28.180s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [25.077s]
[INFO] Apache CloudStack Core  SUCCESS [1:21.496s]
[INFO] Apache CloudStack Agents .. SUCCESS [37.325s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [37.598s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [14.381s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [2:07.721s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [40.282s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [27.056s]
[INFO] Apache CloudStack Server .. SUCCESS [4:14.716s]
[INFO] Apache CloudStack Framework - Quota ... SUCCESS [38.440s]
[INFO] Apache CloudStack Usage Server  SUCCESS [44.035s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS 
[1:22.512s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.067s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.569s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [54.674s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [47.874s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [29.340s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [26.597s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [25.932s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [22.628s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [44.970s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [12.661s]
[INFO] Apache CloudStack Cloud Engine Service  SUCCESS [7.791s]
[INFO] Apache CloudStack Plugin POM .. SUCCESS [0.950s]
[INFO] Apache CloudStack Plugin - API Rate Limit . SUCCESS [26.413s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS 
[23.936s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS 
[36.335s]
[INFO] Apache CloudStack Plugin - API SolidFire .. SUCCESS [17.676s]
[INFO] Apache CloudStack Plugin - API Discovery .. SUCCESS [24.144s]
[INFO] Apache CloudStack Plugin - ACL Static Role Based .. SUCCESS [15.790s]
[INFO] Apache CloudStack Plugin - Host Anti-Affinity Processor  SUCCESS 
[16.858s]
[INFO] Apache 

Re: [PROPOSAL] Freeze everything until we get CI

2016-02-01 Thread Bharat Kumar
Sure Remi,

Once I am done consolidating the scripts i will put them in a separate repo on 
github.

Thanks,
Bharat.


> On 30-Jan-2016, at 5:14 PM, Remi Bergsma  wrote:
> 
> Please put it in a separate repo. There’s way too much stuff in the 
> cloudstack repo already, IMHO and we should be splitting out, not adding more 
> :-)
> 
> Regards,
> Remi
> 
> 
> 
> 
> 
> On 29/01/16 08:22, "Erik Weber"  wrote:
> 
>> I'd love to see this in the cloudstack repository.
>> Others might have an easier time getting access to hardware, and could use
>> it to help test releases/PR
>> 
>> 
>> Erik
>> 
>> Den fredag 29. januar 2016 skrev Bharat Kumar 
>> følgende:
>> 
>>> yes, we would be sharing it with the community and get this running in the
>>> ACS infra.
>>> Currently it can create a cloudstack test bed, runs tests and email the
>>> results.
>>> 
>>> Here are some details on how this works and what is needed to set this up.
>>> 
>>>  *   we use jenkins, cobbler, puppet and marvin to create cloudstack
>>> setup.
>>>  *   jenkins triggers the test runs, collects the test results and mails
>>> them.
>>>  *   cobbler is use to image the hosts and create Management server.
>>>  *   The management server is a VM and each time a test run is triggered
>>> we pull the latest code, build (dev setup) the MS and run it.
>>>  *   Need IPMI enabled servers to uses and Hosts in cloudstack setup.
>>> Cobbler installs the required OS on these hosts.
>>>  *   We use a XenServer to create management server VMs.
>>> 
>>> The resources required to set this up.
>>> 
>>>  *   We need two servers to host the VMs used in CI, one XenServer to
>>> host the Cloustack management servers and at least 3 IPMI enabled servers
>>> per cloudstack setup to run the BVTs.
>>>  *   some set of IPs (public and private IPs) and vlans.
>>> 
>>> Once we have the resources in ACS infra we can start setting this up. But
>>> some work needs to
>>> be done to integrate this with the github to test and post the results in
>>> the PRs instead of mailing them.
>>> 
>>> I think the best way to share it will be by implementing this in the ACS
>>> infra. Once we do this every one can pitch in, replicate and further
>>> contribute to this.
>>> 
>>> Meanwhile i will commit the scripts to set this up and keep this going.
>>> 
>>> Thanks,
>>> Bharat.
>>> 
>>> 
>>> On 28-Jan-2016, at 7:37 PM, Erik Weber 
>>> >> wrote:
>>> 
>>> Why not share it as is, then the community could help improving this,
>>> rather than this being a single company effort?
>>> 
>>> --
>>> Erik
>>> 
>>> On Thu, Jan 28, 2016 at 1:49 PM, Bharat Kumar >> >>
>>> wrote:
>>> 
>>> Hi All,
>>> 
>>> I agree that we need to have a CI to deal with the large volume of PRs.
>>> The current travis CI is not good enough as it runs only simulator tests.
>>> We identified this issue and came up with a effective CI for automating
>>> test runs for a each PR. This is already functional, with few github
>>> integration aspects pending. We are internally stabilizing it before
>>> sharing it.
>>> 
>>> We have been in touch with David Nalley ( CC’ed )  in making this
>>> operational for entire community using ACS infra.
>>> 
>>> 
>>> For your reference, here is the FS I have shared with the community
>>> earlier and also in this thread before, your feedback is welcome.
>>> (
>>> 
>>> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Test+bed+orchestrator+and+test+runner+to+enable+continuos+integration
>>> ).
>>> 
>>> Thanks,
>>> Bharat.
>>> 
>>> 
>>> 
>>> 
>>> On 28-Jan-2016, at 4:26 PM, Rohit Yadav >> >> rohit.ya...@shapeblue.com >> wrote:
>>> 
>>> All,
>>> 
>>> I’m sorry to get to have the PRs merged without adhering to the strict
>>> testing requirements. While I think PRs were alright and it did not break
>>> anything, the way it was merged made people uncomfortable that there is
>>> some sort of haste in doing this fast which there is none.
>>> 
>>> I’ll not repeat this and hope you understand that I never had any hidden
>>> agenda but to simply help people with some PRs.
>>> 
>>> Regards.
>>> 
>>> On 28-Jan-2016, at 11:36 AM, Sebastien Goasguen >> >> run...@gmail.com >> wrote:
>>> 
>>> Hi Folks,
>>> 
>>> My proposal to freeze until we get CI was indeed due to seeing Rohit’s
>>> commit but was by no means a personal attack or judgment.
>>> 
>>> We have lots of PR pending (as mentioned before by Remi) and we need
>>> people to help review and test.
>>> So thanks to Rohit.
>>> 
>>> My only concerns were two fold:
>>> 
>>> 1- We need  to keep to adhere to our release principles:
>>> 
>>> 
>>> 

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

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

https://github.com/apache/cloudstack/pull/1360#issuecomment-177942109
  
@rodrigo93 I think that a JIRA ticket wouldn't help much, considering that 
JIRA is for report issues (not PRs). I think that the point here is to analyze 
the code and give suggestions that could improve this code; if the work done by 
@ProjectMoon is ok, then a LGTM would do.


---
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: Display hostname the VPC router runs on

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

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


---
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-8324: config drive data set/ge...

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

https://github.com/apache/cloudstack/pull/1379#issuecomment-178400682
  
I will add the suggested 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: CLOUDSTACK-9252: Support configurable NFS...

2016-02-01 Thread nvazquez
Github user nvazquez commented on the pull request:

https://github.com/apache/cloudstack/pull/1361#issuecomment-178346694
  
@rafaelweingartner thanks for your reply!

I'm familiar to Spring, I've worked with it, I know how it works but I 
wouldn't say I know it in depth. If you don't mind I would find your detailed 
explanation really helpful for this feature and future tasks.

About bean injection in TemplateServiceImpl, I first tryied 
@Inject annotation, but ACS couldn't start because it couldn't 
find a bean of type ImageStoreDetailsUtil. I had the same issue 
with @Autowired annotation. The only solution I could find was 
adding bean in xml context file.

I agree with you about not extending Manager interface, I saw 
it was always done like that and just followed the pattern. I will remove that 
from ImageStoreDetailsUtil

I could start ACS but didn't fully test Vmware* classes. I 
would try to get the context and then the bean from it as you suggested.

Thanks a lot for your help!


---
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: Restore iptables at once using iptables-r...

2016-02-01 Thread borisroman
Github user borisroman commented on the pull request:

https://github.com/apache/cloudstack/pull/1389#issuecomment-178000474
  
Integration test results:

```
Test router internal advanced zone ... === TestName: 
test_02_router_internal_adv | Status : SUCCESS ===
ok
Test restart network ... === TestName: test_03_restart_network_cleanup | 
Status : SUCCESS ===
ok
Test router basic setup ... === TestName: test_05_router_basic | Status : 
SUCCESS ===
ok
Test router advanced setup ... === TestName: test_06_router_advanced | 
Status : SUCCESS ===
ok
Test stop router ... === TestName: test_07_stop_router | Status : SUCCESS 
===
ok
Test start router ... === TestName: test_08_start_router | Status : SUCCESS 
===
ok
Test reboot router ... === TestName: test_09_reboot_router | Status : 
SUCCESS ===
ok
Test reset virtual machine on reboot ... === TestName: 
test_01_reset_vm_on_reboot | Status : SUCCESS ===
ok
Test advanced zone virtual router ... === TestName: 
test_advZoneVirtualRouter | Status : SUCCESS ===
ok
Test Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : 
SUCCESS ===
ok
Test Multiple Deploy Virtual Machine ... === TestName: 
test_deploy_vm_multiple | Status : SUCCESS ===
ok
Test Stop Virtual Machine ... === TestName: test_01_stop_vm | Status : 
SUCCESS ===
ok
Test Start Virtual Machine ... === TestName: test_02_start_vm | Status : 
SUCCESS ===
ok
Test Reboot Virtual Machine ... === TestName: test_03_reboot_vm | Status : 
SUCCESS ===
ok
Test destroy Virtual Machine ... === TestName: test_06_destroy_vm | Status 
: SUCCESS ===
ok
Test recover Virtual Machine ... === TestName: test_07_restore_vm | Status 
: SUCCESS ===
ok
Test migrate VM ... === TestName: test_08_migrate_vm | Status : SUCCESS ===
ok
Test destroy(expunge) Virtual Machine ... === TestName: test_09_expunge_vm 
| Status : SUCCESS ===
ok
Test to create service offering ... === TestName: 
test_01_create_service_offering | Status : SUCCESS ===
ok
Test to update existing service offering ... === TestName: 
test_02_edit_service_offering | Status : SUCCESS ===
ok
Test to delete service offering ... === TestName: 
test_03_delete_service_offering | Status : SUCCESS ===
ok
Test for delete account ... === TestName: test_delete_account | Status : 
SUCCESS ===
ok
Test for Associate/Disassociate public IP address for admin account ... === 
TestName: test_public_ip_admin_account | Status : SUCCESS ===
ok
Test for Associate/Disassociate public IP address for user account ... === 
TestName: test_public_ip_user_account | Status : SUCCESS ===
ok
Test for release public IP address ... === TestName: test_releaseIP | 
Status : SUCCESS ===
ok
Test create VPC offering ... === TestName: test_01_create_vpc_offering | 
Status : SUCCESS ===
ok
Test VPC offering without load balancing service ... === TestName: 
test_03_vpc_off_without_lb | Status : SUCCESS ===
ok
Test VPC offering without static NAT service ... === TestName: 
test_04_vpc_off_without_static_nat | Status : SUCCESS ===
ok
Test VPC offering without port forwarding service ... === TestName: 
test_05_vpc_off_without_pf | Status : SUCCESS ===
ok
Test VPC offering with invalid services ... === TestName: 
test_06_vpc_off_invalid_services | Status : SUCCESS ===
ok
Test update VPC offering ... === TestName: test_07_update_vpc_off | Status 
: SUCCESS ===
ok
Test list VPC offering ... === TestName: test_08_list_vpc_off | Status : 
SUCCESS ===
ok
test_09_create_redundant_vpc_offering 
(integration.component.test_vpc_offerings.TestVPCOffering) ... === TestName: 
test_09_create_redundant_vpc_offering | Status : SUCCESS ===
ok
Test start/stop of router after addition of one guest network ... === 
TestName: test_01_start_stop_router_after_addition_of_one_guest_network | 
Status : SUCCESS ===
ok
Test reboot of router after addition of one guest network ... === TestName: 
test_02_reboot_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to change service offering of router after addition of one guest 
network ... === TestName: 
test_04_chg_srv_off_router_after_addition_of_one_guest_network | Status : 
SUCCESS ===
ok
Test destroy of router after addition of one guest network ... === 
TestName: test_05_destroy_router_after_addition_of_one_guest_network | Status : 
SUCCESS ===
ok
Test to stop and start router after creation of VPC ... === TestName: 
test_01_stop_start_router_after_creating_vpc | Status : SUCCESS ===
ok
Test to reboot the router after creating a VPC ... === TestName: 
test_02_reboot_router_after_creating_vpc | Status : SUCCESS ===
ok
Tests to change service offering of the Router after ... === TestName: 
test_04_change_service_offerring_vpc | 

[GitHub] cloudstack pull request: Restore iptables at once using iptables-r...

2016-02-01 Thread borisroman
GitHub user borisroman opened a pull request:

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

Restore iptables at once using iptables-restore instead of calling ip 
tables numerous times



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

$ git pull https://github.com/borisroman/cloudstack 
vr-iptables-save-rebased-wip

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

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


commit 5ea9d7e2c57dc319be7b2bc0eba0d91319a6b858
Author: Boris Schrijver 
Date:   2016-01-19T10:57:05Z

Restore iptables at once using iptables-restore instead of calling iptables 
numerous times




---
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: Allow VM snapshots and volume snapshots t...

2016-02-01 Thread rodrigo93
Github user rodrigo93 commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/672#discussion_r51428218
  
--- Diff: 
plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java
 ---
@@ -4251,6 +4257,34 @@ public void rebootVM(final Connection conn, final VM 
vm, final String vmName) th
 }
 }
 
+protected String skipOrRemoveSR(Connection conn, SR sr) {
+if (sr == null) {
+return null;
+}
+if (s_logger.isDebugEnabled()) {
+s_logger.debug(logX(sr, "Removing SR"));
+}
+try {
+Set vdis = sr.getVDIs(conn);
+for (VDI vdi : vdis) {
+Map 
currentOperation = vdi.getCurrentOperations(conn);
+if (currentOperation == null || currentOperation.size() == 
0) {
+continue;
+}
+return null;
+}
+removeSR(conn, sr);
+return null;
+} catch (XenAPIException e) {
+s_logger.warn(logX(sr, "Unable to get current opertions " + 
e.toString()), e);
+} catch (XmlRpcException e) {
+s_logger.warn(logX(sr, "Unable to get current opertions " + 
e.getMessage()), e);
+}
+String msg = "Remove SR failed";
+s_logger.warn(msg);
+return msg;
--- End diff --

@DaanHoogland I might have choose the wrongs words, but summarizing my 
idea, what I wanted to say is that I thought it would be good to have a well 
documented system of this size. It might take a few minutes to do so when 
someone writes the code, but it would turn the system "more standardized".
Sorry if I sounded rude before, but I was just trying to help. 


---
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-02-01 Thread rodrigo93
Github user rodrigo93 commented on the pull request:

https://github.com/apache/cloudstack/pull/1360#issuecomment-178002003
  
@GabrielBrascher @ProjectMoon, You are right. I thought I saw a some 
section in Jira for minor improvements, but I was wrong. Besides that, the code 
and work done here LGTM. 


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


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

2016-02-01 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.767s]
[INFO] Apache CloudStack . SUCCESS [2.148s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [0.784s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [19.027s]
[INFO] Apache CloudStack Utils ... SUCCESS [1:29.981s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.111s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [54.353s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [28.056s]
[INFO] Apache CloudStack API . SUCCESS [1:48.292s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [16.689s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [29.795s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.091s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [28.219s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [25.368s]
[INFO] Apache CloudStack Core  SUCCESS [1:22.817s]
[INFO] Apache CloudStack Agents .. SUCCESS [36.002s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [36.446s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [14.475s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [2:08.875s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [40.617s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [26.196s]
[INFO] Apache CloudStack Server .. SUCCESS [4:12.537s]
[INFO] Apache CloudStack Framework - Quota ... SUCCESS [37.709s]
[INFO] Apache CloudStack Usage Server  SUCCESS [44.641s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS 
[1:24.336s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.087s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.480s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [54.432s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [47.492s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [30.929s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [26.546s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [26.168s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [22.832s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [35.114s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [13.354s]
[INFO] Apache CloudStack Cloud Engine Service  SUCCESS [7.585s]
[INFO] Apache CloudStack Plugin POM .. SUCCESS [0.990s]
[INFO] Apache CloudStack Plugin - API Rate Limit . SUCCESS [26.896s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS 
[24.105s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS 
[36.029s]
[INFO] Apache CloudStack Plugin - API SolidFire .. SUCCESS [17.135s]
[INFO] Apache CloudStack Plugin - API Discovery .. SUCCESS [23.669s]
[INFO] Apache CloudStack Plugin - ACL Static Role Based .. SUCCESS [16.000s]
[INFO] Apache CloudStack Plugin - Host Anti-Affinity Processor  SUCCESS 
[16.853s]
[INFO] Apache 

[GitHub] cloudstack pull request: CLOUDSTACK-8830 CLONE - VM snapshot fails...

2016-02-01 Thread alexandrelimassantana
Github user alexandrelimassantana commented on the pull request:

https://github.com/apache/cloudstack/pull/798#issuecomment-178042092
  
Hi @maneesha-p I see you did the extraction, but could you also write the 
method javadoc along with it? As simple as it is, it can help people understand 
it when using it.

Thanks for your time.


---
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 #3129

2016-02-01 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 
[2.779s]
[INFO] Apache CloudStack . SUCCESS [3.170s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [1.310s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [19.203s]
[INFO] Apache CloudStack Utils ... SUCCESS [1:30.442s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.120s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [53.613s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [27.352s]
[INFO] Apache CloudStack API . SUCCESS [1:48.648s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [16.231s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [30.217s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.086s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [27.889s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [25.250s]
[INFO] Apache CloudStack Core  SUCCESS [1:22.284s]
[INFO] Apache CloudStack Agents .. SUCCESS [36.390s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [36.562s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [13.981s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [2:05.825s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [41.135s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [25.337s]
[INFO] Apache CloudStack Server .. SUCCESS [4:13.955s]
[INFO] Apache CloudStack Framework - Quota ... SUCCESS [36.411s]
[INFO] Apache CloudStack Usage Server  SUCCESS [44.102s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS 
[1:20.868s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.066s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.482s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [53.787s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [48.373s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [30.268s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [25.950s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [31.263s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [20.756s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [35.222s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [12.605s]
[INFO] Apache CloudStack Cloud Engine Service  SUCCESS [7.722s]
[INFO] Apache CloudStack Plugin POM .. SUCCESS [1.021s]
[INFO] Apache CloudStack Plugin - API Rate Limit . SUCCESS [26.621s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS 
[23.281s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS 
[35.778s]
[INFO] Apache CloudStack Plugin - API SolidFire .. SUCCESS [17.635s]
[INFO] Apache CloudStack Plugin - API Discovery .. SUCCESS [22.993s]
[INFO] Apache CloudStack Plugin - ACL Static Role Based .. SUCCESS [15.041s]
[INFO] Apache CloudStack Plugin - Host Anti-Affinity Processor  SUCCESS 
[16.854s]
[INFO] Apache