[GitHub] cloudstack pull request: CLOUDSTACK-9065: Packaging RPM, add optio...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1075#issuecomment-157638452
  
Quick question: were the generated packages installed successfully?

Cheers,
Wilder


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

[GitHub] cloudstack pull request: CLOUDSTACK-9065: Packaging RPM, add optio...

[davidamorimfaria]

Github user davidamorimfaria commented on the pull request:

https://github.com/apache/cloudstack/pull/1075#issuecomment-157643811
  
This is the package release version number, nothing to do with the thing 
that is packaged.

Its purpose is to allow for this script to be used with an automated 
package builder, and thus create a unique package version with each iteration, 
for example using a jenkins job number instead of always being "1". Also, 
having a distinct package version makes the package directly usable in a yum 
repository without having to nuke the repodata because of signature mismatches 
for files that were rebuild using the same name (and package version).

It is also not only the file name that is changed. If you instpect the repo 
you will see it in the "Release" details:
```
[root@CLOUD-REPO x86_64]# rpm -qip 
cloudstack-common-4.6.0-SNAPSHOT4.el7.centos.x86_64.rpm 
Name: cloudstack-common
Version : 4.6.0
Release : SNAPSHOT4.el7.centos
Architecture: x86_64
Install Date: (not installed)
Group   : System Environment/Libraries
Size: 112901174
License : ASL 2.0
Signature   : (none)
Source RPM  : cloudstack-4.6.0-SNAPSHOT4.el7.centos.src.rpm
Build Date  : ma 16 nov 2015 19:00:45 UTC
Build Host  : 597e5453e780
Relocations : (not relocatable)
Packager: Apache CloudStack 
Vendor  : Apache CloudStack 
Summary : Apache CloudStack common files and scripts
Description :
The Apache CloudStack files shared between agent and management server
```

```
[root@CLOUD-REPO x86_64]# rpm -qip 
cloudstack-common-4.6.0-1.el7.centos.x86_64.rpm 
Name: cloudstack-common
Version : 4.6.0
Release : 1.el7.centos
Architecture: x86_64
Install Date: (not installed)
Group   : System Environment/Libraries
Size: 112894853
License : ASL 2.0
Signature   : (none)
Source RPM  : cloudstack-4.6.0-1.el7.centos.src.rpm
Build Date  : di 17 nov 2015 13:22:29 UTC
Build Host  : dec63b100baa
Relocations : (not relocatable)
Packager: Apache CloudStack 
Vendor  : Apache CloudStack 
Summary : Apache CloudStack common files and scripts
Description :
The Apache CloudStack files shared between agent and management server
```


---
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-9051: update nic IP address of...

[ustcweizhou]

GitHub user ustcweizhou opened a pull request:

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

CLOUDSTACK-9051: update nic IP address of stopped vm

This provides the feature to change ip address of NIC on a stopped vm by 
API and UI.

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

$ git pull https://github.com/ustcweizhou/cloudstack update-nic-ipaddr

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

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


commit 92101236f07fe1edce40499c1411013b90119f8f
Author: Wei Zhou 
Date:   2015-11-04T12:07:39Z

CLOUDSTACK-9051: update nic IP address of stopped vm




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

[GitHub] cloudstack pull request: CLOUDSTACK-9065: Packaging RPM, add optio...

[davidamorimfaria]

Github user davidamorimfaria commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1075#discussion_r45172947
  
--- Diff: packaging/package.sh ---
@@ -18,26 +18,37 @@
 
 function usage() {
 echo ""
-echo "usage: ./package.sh [-p|--pack] [-h|--help] [ARGS]"
+echo "usage: ./package.sh [-h|--help] -d|--distribution  
[-r|--release ] [-p|--pack oss|OSS|noredist|NOREDIST] [-s|--simulator 
default|DEFAULT|simulator|SIMULATOR]"
 echo ""
-echo "The commonly used Arguments are:"
-echo "-p|--pack oss|OSS To package with only 
redistributable libraries (default)"
-echo "-p|--pack noredist|NOREDIST   To package with 
non-redistributable libraries"
-echo "-d centos7|centos63|fedora20|fedora21  To build a package for a 
distribution"
-echo "-s simulator|SIMULATORTo build for Simulator"
+echo "The supported arguments are:"
+echo "  To package with only redistributable libraries (default)"
+echo "-p|--pack oss|OSS"
+echo "  To package with non-redistributable libraries"
+echo "-p|--pack noredist|NOREDIST"
+echo "  To build a package for a distribution (mandatory)"
+echo "-d|--distribution centos7|centos63|fedora20|fedora21"
+echo "  To set the package release version (optional)"
+echo "  (default is 1 for normal and prereleases, empty for SNAPSHOT)"
+echo "-r|--release version(integer)"
+echo "  To build for Simulator (optional)"
+echo "-s|--simulator default|DEFAULT|simulator|SIMULATOR"
+echo "  To display this information"
+echo "-h|--help"
 echo ""
-echo "Examples: ./package.sh -p|--pack oss|OSS"
-echo "  ./package.sh -p|--pack noredist|NOREDIST"
-echo "  ./package.sh (Default OSS)"
-exit 1
+echo "Examples: ./package.sh --pack oss"
+echo "  ./package.sh --pack noredist"
+echo "  ./package.sh --pack oss --distribution centos7 
--release 42"
--- End diff --

quick answer, the release is the versioning for the package. Not 
necessarily related to the content of the package.


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

[GitHub] cloudstack pull request: CLOUDSTACK-9051: update nic IP address of...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1086#issuecomment-157647040
  
And how are we suppose to help reviewing/testing this PR, @ustcweizhou ?

Any time to add some unit/integration tests and write few steps so we can 
proceed?

Until that happens, :-1:  for this PR.

Cheers,
Wilder


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

[GitHub] cloudstack pull request: CLOUDSTACK-9065: Packaging RPM, add optio...

[davidamorimfaria]

Github user davidamorimfaria commented on the pull request:

https://github.com/apache/cloudstack/pull/1075#issuecomment-157647291
  
As for installing the package, having distinct package version numbers 
allows one to update packages instead of having to remove and install them in 
succession if they have the same version and release numbers. I haven't 
installed it yet, but the content should be the same, only filename and rpm 
metadata are modified.


---
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.6] Cannot list vlanipranges by keyword

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1085#issuecomment-157647900
  
This time you tested and showed us how to do it. Good!

The code LGTM, but I will also run the same test on this PR.

Cheers,
Wilder


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

[GitHub] cloudstack pull request: CLOUDSTACK-9055: fix NPE in updating Redu...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1073#issuecomment-157648979
  
let's get this merged. I need no intergration tests for this improvement. a 
disect of ```updateRoutersRedundantState``` to make the bit that is fixed and 
maybe some other bits unit testable would be nice but not related to 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.
---

Read/Write splitting on database

[Alireza Eskandari]

Hi all,
According to link below, does CloudStack split read/write request between 
master and slaves?
https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=34838207

regards,
Alireza 

Re: [KVM] NPE when createVolume by a snapshot whose source volume is deleted

[Wido den Hollander]

On 11/18/2015 08:23 AM, Qian Shaohua wrote:
> Hi all,
> 
>  
> 
> CS 4.5.2 KVM
> 
> KVMStorageProcessor checks source volume of a snapshot for the ImageFormat
> in createVolumeFromSnapshot(CopyCommand cmd).
> 
> If the source volume is deleted, we get NPE.
> 
>  
> 
> 2015-11-18 14:02:57,259 DEBUG [o.a.c.e.o.VolumeOrchestrator]
> (API-Job-Executor-35:ctx-0c1e2671 job-2795 ctx-5108a77c) Failed to create
> volume from snapshot:java.lang.NullPointerException
> 
> at
> com.cloud.hypervisor.kvm.storage.KVMStorageProcessor.createVolumeFromSnapsho
> t(KVMStorageProcessor.java:1237)
> 
> at
> com.cloud.storage.resource.StorageSubsystemCommandHandlerBase.execute(Storag
> eSubsystemCommandHandlerBase.java:97)
> 
> at
> com.cloud.storage.resource.StorageSubsystemCommandHandlerBase.handleStorageC
> ommands(StorageSubsystemCommandHandlerBase.java:53)
> 
> at
> com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.executeRequest(Li
> bvirtComputingResource.java:1406)
> 
> at com.cloud.agent.Agent.processRequest(Agent.java:503)
> 
> at com.cloud.agent.Agent$AgentRequestHandler.doTask(Agent.java:808)
> 
> at com.cloud.utils.nio.Task.run(Task.java:84)
> 
> at
> java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:11
> 45)
> 
> at
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:6
> 15)
> 
> at java.lang.Thread.run(Thread.java:744)
> 
>  
> 
> It isn't fix in master as we know.
> 

Can you submit a Pull Request on Github? That would be great!

> Shall we save format info in snapshots table and get rid of the dependency?
> 
> Or give the volume info in CopyCommand even if the volume is deleted?
> 

I don't have a clear answer, but we should always prevent NPEs :)

Wido

>  
> 
> --
> 
> Qian
> 
> 

[GitHub] cloudstack pull request: CLOUDSTACK-9065: Packaging RPM, add optio...

[NuxRo]

Github user NuxRo commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1075#discussion_r45174353
  
--- Diff: packaging/package.sh ---
@@ -18,26 +18,37 @@
 
 function usage() {
 echo ""
-echo "usage: ./package.sh [-p|--pack] [-h|--help] [ARGS]"
+echo "usage: ./package.sh [-h|--help] -d|--distribution  
[-r|--release ] [-p|--pack oss|OSS|noredist|NOREDIST] [-s|--simulator 
default|DEFAULT|simulator|SIMULATOR]"
 echo ""
-echo "The commonly used Arguments are:"
-echo "-p|--pack oss|OSS To package with only 
redistributable libraries (default)"
-echo "-p|--pack noredist|NOREDIST   To package with 
non-redistributable libraries"
-echo "-d centos7|centos63|fedora20|fedora21  To build a package for a 
distribution"
-echo "-s simulator|SIMULATORTo build for Simulator"
+echo "The supported arguments are:"
+echo "  To package with only redistributable libraries (default)"
+echo "-p|--pack oss|OSS"
+echo "  To package with non-redistributable libraries"
+echo "-p|--pack noredist|NOREDIST"
+echo "  To build a package for a distribution (mandatory)"
+echo "-d|--distribution centos7|centos63|fedora20|fedora21"
+echo "  To set the package release version (optional)"
+echo "  (default is 1 for normal and prereleases, empty for SNAPSHOT)"
+echo "-r|--release version(integer)"
+echo "  To build for Simulator (optional)"
+echo "-s|--simulator default|DEFAULT|simulator|SIMULATOR"
+echo "  To display this information"
+echo "-h|--help"
 echo ""
-echo "Examples: ./package.sh -p|--pack oss|OSS"
-echo "  ./package.sh -p|--pack noredist|NOREDIST"
-echo "  ./package.sh (Default OSS)"
-exit 1
+echo "Examples: ./package.sh --pack oss"
+echo "  ./package.sh --pack noredist"
+echo "  ./package.sh --pack oss --distribution centos7 
--release 42"
--- End diff --

Wilder,

This does not affect the contents in any way, just makes it easier to use 
with "yum/apt-get update" for e.g. when you want to issue updates.
It will come in handy as newer jenkins (as well as manual) built packages 
will now be able to gracefully upgrade old ones.
I used to fiddle around with cloud.spec to achieve the same and the idea 
was discussed last year with ke4qqq & spark404 but we never got anywhere. :)

Good job @davidamorimfaria !

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Wilder Rodrigues" 
> To: "apache/cloudstack" 
> Cc: "NuxRo" 
> Sent: Wednesday, 18 November, 2015 07:58:40
> Subject: Re: [cloudstack] CLOUDSTACK-9065: Packaging RPM, add option for 
package release version, cleanup and lint
> (#1075)

>>  echo ""
>> -echo "Examples: ./package.sh -p|--pack oss|OSS"
>> -echo "  ./package.sh -p|--pack noredist|NOREDIST"
>> -echo "  ./package.sh (Default OSS)"
>> -exit 1
>> +echo "Examples: ./package.sh --pack oss"
>> +echo "  ./package.sh --pack noredist"
>> +echo "  ./package.sh --pack oss --distribution centos7 
--release
>> 42"
> 
> @davidamorimfaria, what are the implications in giving a release number? 
For
> example, giving 4.6 or nothing will reflect in a different package or is 
it
> just for the final RPM file name?
> 
> For the looks on the code, it seems to be only the file name. :)
> 
> No big deal, I'm just curious. I will take some time to test your PR 
today.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/apache/cloudstack/pull/1075/files#r45169717



---
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.6.1] CLOUDSTACK-9015 - Redundant VPC V...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1070#issuecomment-157651712
  
@remibergsma and @miguelaferreira faced problems related to SSH timeout 
whilst testing this PR. I retested it and did not face the same issue:

```
Create a redundant VPC with two networks with two VMs in each network ... 
=== TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : 
SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network and 
check default routes ... === TestName: test_02_redundant_VPC_default_routes | 
Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network ... 
=== TestName: 
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Status : 
SUCCESS ===
ok

--
Ran 3 tests in 4882.851s

OK
/tmp//MarvinLogs/test_vpc_redundant_6MQWZS/results.txt (END)
```

We tested against the same virtualised environment, but running different 
steps. It makes it trick to find the root cause, so this PR will standstill for 
a while until I have time to retest and, hopefully, get an error as well so I 
can dig deeper.

Cheers,
Wilder


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

Re: [KVM] NPE when createVolume by a snapshot whose source volume is deleted

[Wei ZHOU]

Qian,

This is fixed in 4.6/master by
commit bef92052ee9a7303b427782838a8a97623de231c

diff --git
a/engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeDataFactoryImpl.java
b/engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeDataFactoryImp~
index bc51f5f..3e4cdff 100644
---
a/engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeDataFactoryImpl.java
+++
b/engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeDataFactoryImpl.java
@@ -77,7 +77,7 @@ public class VolumeDataFactoryImpl implements
VolumeDataFactory {

 @Override
 public VolumeInfo getVolume(long volumeId) {
-VolumeVO volumeVO = volumeDao.findById(volumeId);
+VolumeVO volumeVO = volumeDao.findByIdIncludingRemoved(volumeId);
 if (volumeVO == null) {
 return null;
 }


Kind regards,
Wei

2015-11-18 8:23 GMT+01:00 Qian Shaohua :

> Hi all,
>
>
>
> CS 4.5.2 KVM
>
> KVMStorageProcessor checks source volume of a snapshot for the ImageFormat
> in createVolumeFromSnapshot(CopyCommand cmd).
>
> If the source volume is deleted, we get NPE.
>
>
>
> 2015-11-18 14:02:57,259 DEBUG [o.a.c.e.o.VolumeOrchestrator]
> (API-Job-Executor-35:ctx-0c1e2671 job-2795 ctx-5108a77c) Failed to create
> volume from snapshot:java.lang.NullPointerException
>
> at
>
> com.cloud.hypervisor.kvm.storage.KVMStorageProcessor.createVolumeFromSnapsho
> t(KVMStorageProcessor.java:1237)
>
> at
>
> com.cloud.storage.resource.StorageSubsystemCommandHandlerBase.execute(Storag
> eSubsystemCommandHandlerBase.java:97)
>
> at
>
> com.cloud.storage.resource.StorageSubsystemCommandHandlerBase.handleStorageC
> ommands(StorageSubsystemCommandHandlerBase.java:53)
>
> at
>
> com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.executeRequest(Li
> bvirtComputingResource.java:1406)
>
> at com.cloud.agent.Agent.processRequest(Agent.java:503)
>
> at com.cloud.agent.Agent$AgentRequestHandler.doTask(Agent.java:808)
>
> at com.cloud.utils.nio.Task.run(Task.java:84)
>
> at
>
> java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:11
> 45)
>
> at
>
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:6
> 15)
>
> at java.lang.Thread.run(Thread.java:744)
>
>
>
> It isn't fix in master as we know.
>
> Shall we save format info in snapshots table and get rid of the dependency?
>
> Or give the volume info in CopyCommand even if the volume is deleted?
>
>
>
> --
>
> Qian
>
>

[GitHub] cloudstack pull request: [4.6] Cannot list vlanipranges by keyword

[ustcweizhou]

Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1085#issuecomment-157652943
  
OK, please 
double check is always necessary.




---
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-9067 - As I developer I want t...

[DaanHoogland]

Github user DaanHoogland commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1084#discussion_r45176128
  
--- Diff: utils/src/main/java/com/cloud/utils/StringUtils.java ---
@@ -310,6 +310,11 @@ public static String mapToString(final Map map) {
 return listWPagination;
 }
 
+public static String format(final String format, final Object... 
arguments) {
--- End diff --

this is usefull for a lot of logging ;)


---
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-9067 - As I developer I want t...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1084#issuecomment-157654321
  
less is more! reviewed java code only, didn't check the removed scripts. 
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: [4.6.1] CLOUDSTACK-9015 - Redundant VPC V...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1070#issuecomment-157655322
  
Although it’s based on an assumption, enforced by good pattern 
recognition, what happened to @miguelaferreira  and @remibergsma was:

1. the code does “delete_nat_rules()” and tries to SSH (expected 
failure). that works fine
2. The code does “add_nat_rules()” and tries to SSH again, which should 
pass - and did all the times I tried, and I’m not lying

so, what’s the problem then?

the “delete_nat_rule()” method is deleting the nat_rule, as it says, 
but not deleting the public IP that was acquired. So, when the code calls the 
“add_nat_rule()” method, it acquires a new IP and adds a nat rule to that 
IP.

what does it mean? that you guys got a situation where the test tried to 
SSH via the IP that doesn’t contain a NAT (PF) rule. that explain the ssh 
error.

I fixed it, but first need to test before pushing.

Cheers,
Wilder


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

[GitHub] cloudstack pull request: CLOUDSTACK-9062: Improve S3 implementatio...

[wido]

Github user wido commented on the pull request:

https://github.com/apache/cloudstack/pull/1083#issuecomment-157657472
  
Initially it looks good, but with S3 it's very hard to test this. At a 
first glance the code looks good. Much better then it was


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

答复: [KVM] NPE when createVolume by a snapshot whose source volume is deleted

[Qian Shaohua]

Oh, I see it, thanks!

-邮件原件-
发件人: Wei ZHOU [mailto:ustcweiz...@gmail.com] 
发送时间: 2015年11月18日 17:20
收件人: dev@cloudstack.apache.org
主题: Re: [KVM] NPE when createVolume by a snapshot whose source volume is deleted

Qian,

This is fixed in 4.6/master by
commit bef92052ee9a7303b427782838a8a97623de231c

diff --git
a/engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeDataFactoryImpl.java
b/engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeDataFactoryImp~
index bc51f5f..3e4cdff 100644
---
a/engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeDataFactoryImpl.java
+++
b/engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeDataFactoryImpl.java
@@ -77,7 +77,7 @@ public class VolumeDataFactoryImpl implements 
VolumeDataFactory {

 @Override
 public VolumeInfo getVolume(long volumeId) {
-VolumeVO volumeVO = volumeDao.findById(volumeId);
+VolumeVO volumeVO = 
+ volumeDao.findByIdIncludingRemoved(volumeId);
 if (volumeVO == null) {
 return null;
 }


Kind regards,
Wei

2015-11-18 8:23 GMT+01:00 Qian Shaohua :

> Hi all,
>
>
>
> CS 4.5.2 KVM
>
> KVMStorageProcessor checks source volume of a snapshot for the 
> ImageFormat in createVolumeFromSnapshot(CopyCommand cmd).
>
> If the source volume is deleted, we get NPE.
>
>
>
> 2015-11-18 14:02:57,259 DEBUG [o.a.c.e.o.VolumeOrchestrator]
> (API-Job-Executor-35:ctx-0c1e2671 job-2795 ctx-5108a77c) Failed to 
> create volume from snapshot:java.lang.NullPointerException
>
> at
>
> com.cloud.hypervisor.kvm.storage.KVMStorageProcessor.createVolumeFromS
> napsho
> t(KVMStorageProcessor.java:1237)
>
> at
>
> com.cloud.storage.resource.StorageSubsystemCommandHandlerBase.execute(
> Storag
> eSubsystemCommandHandlerBase.java:97)
>
> at
>
> com.cloud.storage.resource.StorageSubsystemCommandHandlerBase.handleSt
> orageC
> ommands(StorageSubsystemCommandHandlerBase.java:53)
>
> at
>
> com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.executeRequ
> est(Li
> bvirtComputingResource.java:1406)
>
> at com.cloud.agent.Agent.processRequest(Agent.java:503)
>
> at 
> com.cloud.agent.Agent$AgentRequestHandler.doTask(Agent.java:808)
>
> at com.cloud.utils.nio.Task.run(Task.java:84)
>
> at
>
> java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.j
> ava:11
> 45)
>
> at
>
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.
> java:6
> 15)
>
> at java.lang.Thread.run(Thread.java:744)
>
>
>
> It isn't fix in master as we know.
>
> Shall we save format info in snapshots table and get rid of the dependency?
>
> Or give the volume info in CopyCommand even if the volume is deleted?
>
>
>
> --
>
> Qian
>
>

[GitHub] cloudstack pull request: CLOUDSTACK-9058 - Respond with "saved_pas...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1079#issuecomment-157670606
  
Gave LGTM on #1062, extending it here.


---
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-9058 - Respond with "saved_pas...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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-9047 rename enums

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1049#issuecomment-157671208
  
As for ```test_09_destroy_ssvm``` the exception is a false positive due to 
timeout. s-1-VM is destroyed and s-58-VM comes back but the timeout is reached 
while it is still starting. I will run these tests once 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-9063 CitrixResourceBase refact...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1066#issuecomment-157671857
  
Ping @DaanHoogland @remibergsma @davidamorimfaria 

Just finished some tests here. Based on the output + the code review I did, 
this PR LGTM :+1: 

Cheers,
Wilder

* Environment
  * Management Server on CentOS 7.1
  * One XenServer 6.2
  * Advanced Zone

* Tests executed:

```
nosetests --with-marvin 
--marvin-config=/data/shared/marvin/mct-zone2-xen2-ISOLATED.cfg -s -a 
tags=advanced,required_hardware=false component/test_vpc_routers.py 
smoke/test_routers.py smoke/test_network_acl.py smoke/test_privategw_acl.py 
smoke/test_reset_vm_on_reboot.py smoke/test_vm_life_cycle.py 
smoke/test_vpc_vpn.py smoke/test_service_offerings.py 
component/test_vpc_offerings.py
```

* Test Results:

```
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 | Status : SUCCESS ===
ok
Test to destroy the router after creating a VPC ... === TestName: 
test_05_destroy_router_after_creating_vpc | Status : SUCCESS ===
ok
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_privategw_acl (integration.smoke.test_privategw_acl.TestPrivateGwACL) 
... === TestName: test_privategw_acl | 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
ok
Test migrate VM ... SKIP: At least two hosts should be present in the zone 
for migration
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 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: 

[GitHub] cloudstack pull request: CLOUDSTACK-9058 - Respond with "saved_pas...

[dsclose]

Github user dsclose commented on the pull request:

https://github.com/apache/cloudstack/pull/1079#issuecomment-157672288
  
@wilderrodrigues I'd be pleased to write a Marvin test though I've not 
worked with them before. I think the first step would be to get the tests 
running on my local system. Do you have any docs for that?


---
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-9047 rename enums

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1049#issuecomment-157673706
  
@wilderrodrigues @wido can you review. The test keeps giving an exception 
but the destroyed suystem vms come back without problems and the test suite can 
even be run again on the same environment. I think this PR is alright.


---
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-9067 - As I developer I want t...

[miguelaferreira]

Github user miguelaferreira commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1084#discussion_r45184075
  
--- Diff: utils/src/main/java/com/cloud/utils/StringUtils.java ---
@@ -310,6 +310,11 @@ public static String mapToString(final Map map) {
 return listWPagination;
 }
 
+public static String format(final String format, final Object... 
arguments) {
--- End diff --

Doesn't String.format(...) do the same thing?


---
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-9063 CitrixResourceBase refact...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1066#issuecomment-157676447
  
@franklouwers like doing a review on 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-9050 Virtual router should onl...

[dsclose]

Github user dsclose commented on the pull request:

https://github.com/apache/cloudstack/pull/1063#issuecomment-157678223
  
CLosing this pull request as the patched script is already deprecated.


---
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-9050 Virtual router should onl...

[dsclose]

Github user dsclose closed the pull request at:

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


---
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-8832 : Update Nuage VSP plugin...

[remibergsma]

Github user remibergsma commented on the pull request:

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

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

```

Result:

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

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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-8832 : Update Nuage VSP plugin...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/801#issuecomment-157681280
  
Congrats @nlivens @KrisSterckx this is now merged in master and will be 
included in 4.7.0. Thanks for all the work to get this in :-)


---
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-9062: Improve S3 implementatio...

[DaanHoogland]

Github user DaanHoogland commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1083#discussion_r45187932
  
--- Diff: core/src/com/cloud/storage/template/S3TemplateDownloader.java ---
@@ -19,303 +19,236 @@
 
 package com.cloud.storage.template;
 
-import static com.cloud.utils.StringUtils.join;
-import static java.util.Arrays.asList;
-
-import java.io.BufferedInputStream;
-import java.io.IOException;
-import java.io.InputStream;
-import java.util.Date;
-
+import com.amazonaws.event.ProgressEvent;
+import com.amazonaws.event.ProgressEventType;
+import com.amazonaws.event.ProgressListener;
+import com.amazonaws.services.s3.model.ObjectMetadata;
+import com.amazonaws.services.s3.model.PutObjectRequest;
+import com.amazonaws.services.s3.model.StorageClass;
+import com.amazonaws.services.s3.transfer.Upload;
+import com.cloud.agent.api.to.S3TO;
+import com.cloud.utils.net.HTTPUtils;
+import com.cloud.utils.net.Proxy;
+import com.cloud.utils.storage.S3.S3Utils;
 import org.apache.cloudstack.managed.context.ManagedContextRunnable;
 import org.apache.cloudstack.storage.command.DownloadCommand.ResourceType;
 import org.apache.commons.httpclient.ChunkedInputStream;
--- End diff --

no longer used?


---
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-9062: Improve S3 implementatio...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1083#issuecomment-157683314
  
@borisroman checkstyle reports unused imports


---
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-8832 : Update Nuage VSP plugin...

[nlivens]

Github user nlivens commented on the pull request:

https://github.com/apache/cloudstack/pull/801#issuecomment-157683273
  
Awesome! Thanks @jburwell for the thorough review, thanks @remibergsma for 
merging it in, thanks to all who helped getting this PR in! Much appreciated! :)


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

[PaulAngus]

GitHub user PaulAngus opened a pull request:

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

Remove extraenous log dir and add catalina.out log rotation.

remove unused /var/log/cloudstack-management directory
add cloudstack-catalina to logrotate.d directory so that 
/var/log/cloudstack/management/catalina.out is rotated.

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

$ git pull https://github.com/PaulAngus/cloudstack master

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

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


commit 19e5166b8ce35653d207a3823902d9c2178c35ca
Author: Paul Angus 
Date:   2015-11-16T13:17:03Z

remove unused /var/log/cloudstack-management directory
add cloudstack-catalina to logrotate.d directory so that 
/var/log/cloudstack/management/catalina.out is rotated




---
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-8956: NSX/Nicira Plugin does n...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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-9067 - As I developer I want t...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1084#issuecomment-157693480
  
@miguelaferreira discussed with me the over-engineering wrapper method just 
to add 1 test and we agreed on getting rid of it for the following reasons:

1. It could lead other committers to add simple tests - like the one I 
added - and forget about the real tests that we need;
2. An experienced Software Engineer reviewing the code will spot problems 
with the string formats used.

I will retest the PR and post the results here.

Cheers,
Wilder 


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

[GitHub] cloudstack pull request: Remove extraenous log dir and add catalin...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1087#issuecomment-157693735
  
@PaulAngus, every PR needs an issue on Apache Jira.

In addition, how did you test your changes? Some input will help us to test 
it as well.

Cheers,
Wilder


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

Re: cloudstack UI

[Abhinandan Prateek]

On 17/11/15, 1:49 PM, "Sebastien Goasguen"  wrote:

>
>> On Nov 17, 2015, at 7:24 AM, Abhinandan Prateek 
>>  wrote:
>>
>> Hi,
>>
>>   I am considering  some improvements to cloudstack UI. Like delinking the 
>> quick view with the details page. Such that the details page can contain 
>> listView.
>>
>>  Another change that can make UI more compact and flexible is to have 
>> multiple listViews in a listView i.e. Recursive listViews or just a 
>> complexListView.
>>
>> Any thoughts and comments on complexity, usefulness etc will help.
>>
>
>Abhi, do you know if there is a way to easily decouple the UI code from the 
>core of cloudstack ?
>
>That might be an interesting move.

Although I was currently looking at making the UI widgets more flexible. Coming 
to think about decoupling UI, it might be tricky as UI is tightly coupled with 
cloudstack APIs.
Like marvin tests I think we may have to just have it with cloudstack core, due 
to versioning complications.

Regards,
-abhi
Find out more about ShapeBlue and our range of CloudStack related services

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

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

[GitHub] cloudstack pull request: CLOUDSTACK-9062: Improve S3 implementatio...

[borisroman]

Github user borisroman commented on the pull request:

https://github.com/apache/cloudstack/pull/1083#issuecomment-157697285
  
@DaanHoogland You're right. Forgot to remove it in my last iteration. 
Thanks!


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

Build failed in Jenkins: build-master-noredist #4746

[jenkins]

See 

Changes:

[nicovazquez90] CLOUDSTACK-8956: Add VMware Api v5.5 and change pom.xml to use 
VMware

[nicovazquez90] CLOUDSTACK-8956: Log NSX Api Version

[nicovazquez90] CLOUDSTACK-8956: Deploy VM on NSX managed network changes if 
NSX Api

[nicovazquez90] CLOUDSTACK-8956: Remove assert(false) on opaque network and 
ping method

--
[...truncated 1635 lines...]
[WARNING] * If reporting an issue, report it against the plugin in*
[WARNING] * While this /may/ work fine, please look for plugin updates*
[WARNING] * and/or request plugins be made thread-safe.   *
[WARNING] * If reporting an issue, report it against the plugin in*
[WARNING] * question, not against maven-core  *
[WARNING] * question, not against maven-core  *
[WARNING] *
[WARNING] *
[WARNING] The following plugins are not marked @threadSafe in Apache CloudStack 
Framework - IPC:
[WARNING] The following plugins are not marked @threadSafe in Apache CloudStack 
VMware Base:
[WARNING] org.apache.maven.plugins:maven-site-plugin:3.1
[WARNING] Enable debug to see more precisely which goals are not marked 
@threadSafe.
[WARNING] *
[WARNING] org.apache.maven.plugins:maven-site-plugin:3.1
[WARNING] Enable debug to see more precisely which goals are not marked 
@threadSafe.
[WARNING] *
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ 
cloud-framework-cluster ---
[INFO] 
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ cloud-framework-ipc 
---
[INFO] Deleting 

 (includes = [**/*], excludes = [])
[INFO] Deleting 

 (includes = [**/*], excludes = [])
[INFO] Deleting 
 
(includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-framework-cluster ---
[INFO] Deleting 
 
(includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-framework-ipc ---
Downloading: 
http://repo.maven.apache.org/maven2/com/cloud/com/vmware/vmware-vim25/5.5/vmware-vim25-5.5.jar
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-framework-ipc ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-framework-cluster ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-framework-ipc ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 1 resource
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ 
cloud-framework-ipc ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 55 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-framework-cluster ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 1 resource
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ 
cloud-framework-cluster ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 28 source files to 


[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-framework-ipc ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 3 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.2:testCompile (default-testCompile) @ 
cloud-framework-ipc ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 10 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-framework-cluster ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] 

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/935#issuecomment-157700801
  
@nvazquez Can you look at this build failure: 
http://jenkins.buildacloud.org/job/build-master-noredist/4746/console

Does the slave need the 5.5 jar or is something else wrong?


---
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-8956: NSX/Nicira Plugin does n...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/935#issuecomment-157701917
  
According to Jenkins the build was fine:

```
Maven:  -B -f 
/home/jenkins/jenkins-slave/workspace/cloudstack-pull-analysis/pom.xml 
-Dmaven.repo.local=/home/jenkins/jenkins-slave/maven-repositories/0 
-Penablefindbugs clean install cobertura:cobertura
```
although it doesn't do the `-Pnoredist` profile.


---
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-9067 - As I developer I want t...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1084#issuecomment-157705996
  
good points, I do want to have a more generic way of creating complex log 
messages though. might be useful to construct womething like this for it anyway.


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

[PaulAngus]

Github user PaulAngus commented on the pull request:

https://github.com/apache/cloudstack/pull/1087#issuecomment-157708006
  
Apologies.
https://issues.apache.org/jira/browse/CLOUDSTACK-9073  created

removal of /var/log/cloudstack-management
and addition of catalina log rotation in /etc/logrotate.d/
has been running against 4.5.1 for 2 months with no issues.

updated packaging was built against current master and (centos6 and 7) 
deployed with files/directories as expected.
Left running overnight /var/log/cloudstack/management/catalina.log seen to 
rotate and compress previous 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: Remove extraenous log dir and add catalin...

[jburwell]

Github user jburwell commented on the pull request:

https://github.com/apache/cloudstack/pull/1087#issuecomment-157709261
  
@PaulAngus the change looks reasonable.  Have you checked upgrading the RPM 
to ensure that the any modified logrotate configuration files are left alone?


---
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-8956: NSX/Nicira Plugin does n...

[nvazquez]

Github user nvazquez commented on the pull request:

https://github.com/apache/cloudstack/pull/935#issuecomment-157718592
  
@remibergsma It looks like it is not finding 5.5 jar. These are the steps I 
followed:
* Download VMware-vSphere-SDK-5.5.0-1284541.zip file from 
https://my.vmware.com/group/vmware/get-download?downloadGroup=WEBSDK550
* Extract zip file, cd SDK/vsphere-ws/java/JAXWS/lib/ and rename 
vim25_51.jar to vim25_55.jar
* Copy SDK/vsphere-ws/java/JAXWS/lib/vim25_55.jar to CS deps folder
* In CS deps folder: ./install-non-oss.sh



---
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-8956: NSX/Nicira Plugin does n...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/935#issuecomment-157724852
  
@nvazquez Do you mean to say that we can just rename the old jar to contain 
the new version in the name and the sun shines again?


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

Build failed in Jenkins: build-master-noredist #4747

[jenkins]

See 

--
[...truncated 1710 lines...]
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ 
cloud-vmware-base ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 44 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-framework-ipc ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 3 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.2:testCompile (default-testCompile) @ 
cloud-framework-ipc ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 10 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-framework-cluster ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.2:testCompile (default-testCompile) @ 
cloud-framework-cluster ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 1 source file to 

[INFO] -
[ERROR] COMPILATION ERROR : 
[INFO] -
[ERROR] 
:[49,23]
 error: cannot find symbol
[ERROR]   symbol:   class HostOpaqueNetworkInfo
  location: package com.vmware.vim25
:[59,23]
 error: cannot find symbol
[ERROR]   symbol:   class VirtualEthernetCardOpaqueNetworkBackingInfo
  location: package com.vmware.vim25
:[362,13]
 error: cannot find symbol
[ERROR]   symbol:   class HostOpaqueNetworkInfo
  location: class HostMO
:[362,60]
 error: cannot find symbol
[ERROR]   symbol:   method getOpaqueNetwork()
  location: variable netInfo of type HostNetworkInfo
:[364,17]
 error: cannot find symbol
[ERROR]   symbol:   class HostOpaqueNetworkInfo
  location: class HostMO
:[134,62]
 error: method moveVirtualDiskTask in interface VimPortType cannot be applied 
to given types;
[ERROR]   required: 
ManagedObjectReference,String,ManagedObjectReference,String,ManagedObjectReference,Boolean
  found: 
ManagedObjectReference,String,ManagedObjectReference,String,ManagedObjectReference,boolean,
  reason: actual and formal argument lists differ in length
:[109,8]
 error: cannot find symbol
[ERROR]   symbol:   class VirtualEthernetCardOpaqueNetworkBackingInfo
  location: class VmwareHelper
:[109,69]
 error: cannot find symbol
[INFO] 8 errors 
[INFO] -
[INFO] 
[INFO] --- maven-surefire-plugin:2.18.1:test (default-test) @ 
cloud-framework-cluster ---
[INFO] Surefire report directory: 


---
 T E S T S
---
[INFO] 
[INFO] --- maven-surefire-plugin:2.18.1:test (default-test) @ 
cloud-framework-ipc ---
[INFO] Tests are skipped.
[INFO] 
[INFO] --- maven-jar-plugin:2.5:jar (default-jar) @ cloud-framework-ipc ---
[INFO] Building jar: 

Build failed in Jenkins: build-master-noredist #4748

[jenkins]

See 

--
[...truncated 1703 lines...]
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ 
cloud-vmware-base ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 44 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-framework-ipc ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 3 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.2:testCompile (default-testCompile) @ 
cloud-framework-ipc ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 10 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-framework-cluster ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.2:testCompile (default-testCompile) @ 
cloud-framework-cluster ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 1 source file to 

[INFO] -
[ERROR] COMPILATION ERROR : 
[INFO] -
[ERROR] 
:[49,23]
 error: cannot find symbol
[ERROR]   symbol:   class HostOpaqueNetworkInfo
  location: package com.vmware.vim25
:[59,23]
 error: cannot find symbol
[ERROR]   symbol:   class VirtualEthernetCardOpaqueNetworkBackingInfo
  location: package com.vmware.vim25
:[362,13]
 error: cannot find symbol
[ERROR]   symbol:   class HostOpaqueNetworkInfo
  location: class HostMO
:[362,60]
 error: cannot find symbol
[ERROR]   symbol:   method getOpaqueNetwork()
  location: variable netInfo of type HostNetworkInfo
:[364,17]
 error: cannot find symbol
[ERROR]   symbol:   class HostOpaqueNetworkInfo
  location: class HostMO
:[134,62]
 error: method moveVirtualDiskTask in interface VimPortType cannot be applied 
to given types;
[ERROR]   required: 
ManagedObjectReference,String,ManagedObjectReference,String,ManagedObjectReference,Boolean
  found: 
ManagedObjectReference,String,ManagedObjectReference,String,ManagedObjectReference,boolean,
  reason: actual and formal argument lists differ in length
:[109,8]
 error: cannot find symbol
[ERROR]   symbol:   class VirtualEthernetCardOpaqueNetworkBackingInfo
  location: class VmwareHelper
:[109,69]
 error: cannot find symbol
[INFO] 8 errors 
[INFO] -
[INFO] 
[INFO] --- maven-surefire-plugin:2.18.1:test (default-test) @ 
cloud-framework-cluster ---
[INFO] Surefire report directory: 


---
 T E S T S
---
[INFO] 
[INFO] --- maven-surefire-plugin:2.18.1:test (default-test) @ 
cloud-framework-ipc ---
[INFO] Tests are skipped.
[INFO] 
[INFO] --- maven-jar-plugin:2.5:jar (default-jar) @ cloud-framework-ipc ---
[INFO] Building jar: 

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

[nvazquez]

Github user nvazquez commented on the pull request:

https://github.com/apache/cloudstack/pull/935#issuecomment-157727519
  
@DaanHoogland Hi! No, just renaming the old jar (version 5.1) will fail 
because it doesn't contain opaque network classes. Those ones are introduced in 
the 5.5 version jar.
I added a line in deps/install-non-oss.sh which includes 5.5 jar to maven 
repo and also replaced cs.vmware.api.version property in pom.xml file from 5.1 
to 5.5


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

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/935#issuecomment-157727789
  
@nvazquez I tried to mv and copy the jar in the job but it doesn't work. 
now trying to see if your link yields a newer version then our job has


---
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-8956: NSX/Nicira Plugin does n...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/935#issuecomment-157729881
  
@nvazquez is the older one still needed?


---
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-8956: NSX/Nicira Plugin does n...

[nvazquez]

Github user nvazquez commented on the pull request:

https://github.com/apache/cloudstack/pull/935#issuecomment-157731551
  
@DaanHoogland No, it's not.


---
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-8956: NSX/Nicira Plugin does n...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/935#issuecomment-157736873
  
Ok, shouldn't it be removed from the install script and then?


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

Jenkins build is back to normal : build-master-noredist #4749

[jenkins]

See 

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

[nvazquez]

Github user nvazquez commented on the pull request:

https://github.com/apache/cloudstack/pull/935#issuecomment-157739765
  
@DaanHoogland Ok, sure


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

[jenkins]

See 

Changes:

[nick.livens] CLOUDSTACK-8832 : Update Nuage VSP plugin to work with Nuage VSP 
release

[nicovazquez90] CLOUDSTACK-8956: Add VMware Api v5.5 and change pom.xml to use 
VMware

[nicovazquez90] CLOUDSTACK-8956: Log NSX Api Version

[nicovazquez90] CLOUDSTACK-8956: Deploy VM on NSX managed network changes if 
NSX Api

[nicovazquez90] CLOUDSTACK-8956: Remove assert(false) on opaque network and 
ping method

--
[...truncated 27714 lines...]
[INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ 
cloud-quickcloud ---
[INFO] No sources to compile
[INFO] 
[INFO] >>> findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) @ 
cloud-quickcloud >>>
[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.804s]
[INFO] Apache CloudStack . SUCCESS [2.099s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [0.778s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [19.858s]
[INFO] Apache CloudStack Utils ... SUCCESS [1:28.939s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.112s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [52.791s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [27.391s]
[INFO] Apache CloudStack API . SUCCESS [1:54.842s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [16.896s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [29.135s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.086s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [26.956s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [24.385s]
[INFO] Apache CloudStack Core  SUCCESS [1:21.820s]
[INFO] Apache CloudStack Agents .. SUCCESS [34.578s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [36.238s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [15.850s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [2:15.150s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [43.514s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [26.075s]
[INFO] Apache CloudStack Server .. SUCCESS [4:31.654s]
[INFO] Apache CloudStack Usage Server  SUCCESS [49.288s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS 
[1:40.006s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.078s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.403s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [1:01.801s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [48.930s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [36.997s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [33.848s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [34.920s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [22.760s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [34.788s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [12.612s]
[INFO] Apache CloudStack Cloud Engine Service  SUCCESS [5.626s]
[INFO] Apache CloudStack Plugin POM .. SUCCESS [0.722s]
[INFO] Apache Cloud

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/935#issuecomment-157748781
  
@nvazquez can you please look at 
http://jenkins.buildacloud.org/job/build-master-slowbuild/2660/findbugsResult/new/
I think these are trivial, you just have to decide on which encoding to use 
and call the right util method (getPrefferedCharset() or getDefaultCharset()) 
from StringUtils.


---
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: [WIP] CLOUDSTACK-6276: project support in...

[pdube]

Github user pdube commented on the pull request:

https://github.com/apache/cloudstack/pull/508#issuecomment-157754858
  
Hey @wilderrodrigues @resmo,

I was looking to fixing this as well. Is there any way I can contribute? 
What is the status of this?

Thanks,
pdube


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

[WIKI] Create Design Document

[Nicolás Vázquez]

Hi all,

I would like to propose a new feature for NiciraNVP Plugin. I would like to
create a design document for 4.7 version in
https://cwiki.apache.org/confluence/display/CLOUDSTACK/4.7+Design+Documents
but I don't have the privileges. How can I proceed?

Thanks,
Nicolas

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9015 - Redundant VPC V...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1070#issuecomment-157764335
  
Ping @miguelaferreira @remibergsma 

Tests are fine, but of course we can only be sure if they work with you as 
well.

There was also a last sleep of 5 seconds that I removed. It now works as it 
should: no sleeps required!

```
Create a redundant VPC with two networks with two VMs in each network ... 
=== TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : 
SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network and 
check default routes ... === TestName: test_02_redundant_VPC_default_routes | 
Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network ... 
=== TestName: 
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Status : 
SUCCESS ===
ok

--
Ran 3 tests in 5040.173s

OK
/tmp//MarvinLogs/test_vpc_redundant_8WKYY7/results.txt (END)
```


---
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-8956: NSX/Nicira Plugin does n...

[nvazquez]

Github user nvazquez commented on the pull request:

https://github.com/apache/cloudstack/pull/935#issuecomment-157782057
  
@DaanHoogland those warnings are in files that don't belong to my pull 
request. 
I could see in 
http://jenkins.buildacloud.org/job/build-master-slowbuild/2660/changes#detail0 
that it compiled 5 commits, but the first one is commited by other contributor.


---
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-9067 - As I developer I want t...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1084#issuecomment-157785310
  
Ping @DaanHoogland @remibergsma @miguelaferreira @bhaisaab @karuturi 
@borisroman 

First round of tests done and got the same results! I will run some 
XenServer tests as well, but if you can help testing that will be much 
appreciated.

* Environment
  - Management Server on CentOS 7.1
  - One KVM Host on CentOS 7.1
  - Agent + Common RPMs built from source
  - Advanced Zone
  - Requires Hardware: TRUE

* Tests Executed

```
nosetests --with-marvin 
--marvin-config=/data/shared/marvin/mct-zone1-kvm1-ISOLATED.cfg -s -a 
tags=advanced,required_hardware=true component/test_vpc_redundant.py 
component/test_routers_iptables_default_policy.py 
component/test_routers_network_ops.py component/test_vpc_router_nics.py 
component/test_password_server.py component/test_router_dhcphosts.py 
smoke/test_loadbalance.py smoke/test_internal_lb.py smoke/test_ssvm.py 
smoke/test_vpc_vpn.py smoke/test_network.py
```

* Results

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

Re: cloudstack UI

[Miguel Ferreira]

I think that would be a good move too. The only impediment I see are the 
hardcoved url endpoints. Specifically "client/" and "client/api".

The UI code is already a separate module. But that gets bundled inside the 
management server WAR. We could separate that in two WARs if te UI would allow 
us to specify where to find the API endpoint.

I've actually deployed cloudstack with an endpoint different from "client/" and 
everything worked fine, except the UI.

Cheers,
--
Miguel

> On 18 Nov 2015, at 13:23, Abhinandan Prateek 
>  wrote:
> 
>> On 17/11/15, 1:49 PM, "Sebastien Goasguen"  wrote:
>> 
>> 
>>> On Nov 17, 2015, at 7:24 AM, Abhinandan Prateek 
>>>  wrote:
>>> 
>>> Hi,
>>> 
>>>  I am considering  some improvements to cloudstack UI. Like delinking the 
>>> quick view with the details page. Such that the details page can contain 
>>> listView.
>>> 
>>> Another change that can make UI more compact and flexible is to have 
>>> multiple listViews in a listView i.e. Recursive listViews or just a 
>>> complexListView.
>>> 
>>> Any thoughts and comments on complexity, usefulness etc will help.
>> 
>> Abhi, do you know if there is a way to easily decouple the UI code from the 
>> core of cloudstack ?
>> 
>> That might be an interesting move.
> 
> Although I was currently looking at making the UI widgets more flexible. 
> Coming to think about decoupling UI, it might be tricky as UI is tightly 
> coupled with cloudstack APIs.
> Like marvin tests I think we may have to just have it with cloudstack core, 
> due to versioning complications.
> 
> Regards,
> -abhi
> Find out more about ShapeBlue and our range of CloudStack related services
> 
> IaaS Cloud Design & Build
> CSForge – rapid IaaS deployment framework
> CloudStack Consulting
> CloudStack Software 
> Engineering
> CloudStack Infrastructure 
> Support
> CloudStack Bootcamp Training 
> Courses
> 
> This email and any attachments to it may be confidential and are intended 
> solely for the use of the individual to whom it is addressed. Any views or 
> opinions expressed are solely those of the author and do not necessarily 
> represent those of Shape Blue Ltd or related companies. If you are not the 
> intended recipient of this email, you must neither take any action based upon 
> its contents, nor copy or show it to anyone. Please contact the sender if you 
> believe you have received this email in error. Shape Blue Ltd is a company 
> incorporated in England & Wales. ShapeBlue Services India LLP is a company 
> incorporated in India and is operated under license from Shape Blue Ltd. 
> Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
> operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company 
> registered by The Republic of South Africa and is traded under license from 
> Shape Blue Ltd. ShapeBlue is a registered trademark.

[j.bac.o] Job configurations

[Frank Maximus]

Hi,

I want to set up a jenkins server,
and would to copy jobs from jenkins.buildacloud.org.
I'm interested to get either the job config files,
or a read-only account on jenkins,

Thanks In Avdance,
Frank



*Frank Maximus *

Senior Software Development Engineer



*nuage*networks.net
​Copernicuslaan 50

2018 Antwerp

Belgium



p: +32 3 240 73 81

Re: [WIKI] Create Design Document

[Nicolás Vázquez]

Done, thanks

2015-11-18 12:50 GMT-03:00 Nicolás Vázquez :

> Hi all,
>
> I would like to propose a new feature for NiciraNVP Plugin. I would like
> to create a design document for 4.7 version in
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/4.7+Design+Documents
> but I don't have the privileges. How can I proceed?
>
> Thanks,
> Nicolas
>

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/935#issuecomment-157805785
  
sorry, @nlivens can you have a look?


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

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

[jenkins]

See 

--
[...truncated 27715 lines...]
[INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ 
cloud-quickcloud ---
[INFO] No sources to compile
[INFO] 
[INFO] >>> findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) @ 
cloud-quickcloud >>>
[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.108s]
[INFO] Apache CloudStack . SUCCESS [3.202s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [1.176s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [22.251s]
[INFO] Apache CloudStack Utils ... SUCCESS [1:31.006s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.106s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [53.879s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [26.888s]
[INFO] Apache CloudStack API . SUCCESS [1:49.978s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [16.928s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [31.005s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.097s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [27.886s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [25.677s]
[INFO] Apache CloudStack Core  SUCCESS [1:20.801s]
[INFO] Apache CloudStack Agents .. SUCCESS [36.142s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [35.860s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [14.103s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [2:10.621s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [51.624s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [29.358s]
[INFO] Apache CloudStack Server .. SUCCESS [4:30.847s]
[INFO] Apache CloudStack Usage Server  SUCCESS [49.206s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS 
[1:32.639s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.085s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.451s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [1:05.757s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [54.668s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [33.124s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [27.980s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [31.294s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [20.635s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [35.835s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [12.616s]
[INFO] Apache CloudStack Cloud Engine Service  SUCCESS [17.947s]
[INFO] Apache CloudStack Plugin POM .. SUCCESS [6.723s]
[INFO] Apache CloudStack Plugin - API Rate Limit . SUCCESS [3:44.073s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS 
[1:37.662s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS 
[45.434s]
[INFO] Apache CloudStack Plugin - API SolidFire .. SUCCESS [27.753s]
[INFO] Apache CloudStack Plugin - API Discovery .. SUCCESS [44.190s]
[INFO] Apache CloudStack Plugin - 

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1084#issuecomment-157810368
  
@wilderrodrigues maybe we can ask @anshul1886 to run the test wuite on 
hyperv?


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

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1084#issuecomment-157818838
  
If @anshul1886 can test it, just to make sure all is fine, would be very 
nice!

@karuturi, do you have a HyperV test environment? :)

Cheers,
Wilder


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

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

[KrisSterckx]

Github user KrisSterckx commented on the pull request:

https://github.com/apache/cloudstack/pull/801#issuecomment-157825620
  
Thanks @remibergsma and all reviewers  -  This is great.  This is a great 
add to CloudStack and a great milestone for Nuage Networks.  CloudStack + SDN  
: a winning team !


---
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: [WIP] CLOUDSTACK-6276: project support in...

[ustcweizhou]

Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/508#issuecomment-157853386
  
@pdube you can create a pull request on github when you finish the coding 
and testing.
This PR have lots of issues, so it will not be merged of course.
I have similar fix for cloudstack 4.2.1.


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

[GitHub] cloudstack pull request: [WIP] CLOUDSTACK-6276: project support in...

[pdube]

Github user pdube commented on the pull request:

https://github.com/apache/cloudstack/pull/508#issuecomment-157854104
  
@ustcweizhou I have checked it out into another branch. I am working on it. 
And will create a PR when I am done. 


---
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-9067 - As I developer I want t...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1084#issuecomment-157859746
  
More test results:

* Environment
  - Management Server on CentOS 7.1
  - One KVM Host on CentOS 7.1
  - Agent + Common RPMs built from source
  - Advanced Zone
  - Requires Hardware: FALSE

* Tests Executed

```
nosetests --with-marvin 
--marvin-config=/data/shared/marvin/mct-zone1-kvm1-ISOLATED.cfg -s -a 
tags=advanced,required_hardware=false smoke/test_routers.py 
smoke/test_reset_vm_on_reboot.py smoke/test_vm_life_cycle.py 
component/test_vpc_routers.py smoke/test_service_offerings.py 
component/test_vpc_offerings.py smoke/test_network_acl.py 
smoke/test_privategw_acl.py smoke/test_network.py
```

* 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 ... SKIP: At least two hosts should be present in the zone 
for migration
Test destroy(expunge) Virtual Machine ... === TestName: test_09_expunge_vm 
| 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 | Status : SUCCESS ===
ok
Test to destroy the router after creating a VPC ... === TestName: 
test_05_destroy_router_after_creating_vpc | 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 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_

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

[jenkins]

See 

--
[...truncated 27715 lines...]
[INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ 
cloud-quickcloud ---
[INFO] No sources to compile
[INFO] 
[INFO] >>> findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) @ 
cloud-quickcloud >>>
[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.827s]
[INFO] Apache CloudStack . SUCCESS [2.078s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [0.777s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [19.748s]
[INFO] Apache CloudStack Utils ... SUCCESS [1:30.070s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.111s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [52.845s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [27.389s]
[INFO] Apache CloudStack API . SUCCESS [1:49.999s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [17.414s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [29.562s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.091s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [28.695s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [23.795s]
[INFO] Apache CloudStack Core  SUCCESS [1:21.086s]
[INFO] Apache CloudStack Agents .. SUCCESS [35.073s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [35.954s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [14.209s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [2:11.737s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [47.767s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [30.178s]
[INFO] Apache CloudStack Server .. SUCCESS [4:24.280s]
[INFO] Apache CloudStack Usage Server  SUCCESS [50.354s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS 
[1:32.218s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.085s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.455s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [59.481s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [1:00.723s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [33.237s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [26.143s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [23.112s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [20.214s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [36.500s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [12.434s]
[INFO] Apache CloudStack Cloud Engine Service  SUCCESS [7.019s]
[INFO] Apache CloudStack Plugin POM .. SUCCESS [0.785s]
[INFO] Apache CloudStack Plugin - API Rate Limit . SUCCESS [43.697s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS 
[3:42.990s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS 
[1:54.737s]
[INFO] Apache CloudStack Plugin - API SolidFire .. SUCCESS [21.907s]
[INFO] Apache CloudStack Plugin - API Discovery .. SUCCESS [47.046s]
[INFO] Apache CloudStack Plugin - A

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/801#issuecomment-157871920
  
@nlivens please have a look at 
http://jenkins.buildacloud.org/job/build-master-slowbuild/2662/
two occasions of default encoding that you should make explicit. Look at 
StringUtils (ours) for the methods ```getPreferredCharset()``` and 
```getDefaultCharset()``` and use them please instead of the  implicit default. 
thanks.


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

[GitHub] cloudstack pull request: [WIP] CLOUDSTACK-6276: project support in...

[pdube]

Github user pdube commented on the pull request:

https://github.com/apache/cloudstack/pull/508#issuecomment-157874321
  
@ustcweizhou Also if you have a fix for it, why not port it forward?


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

[STABILITY]} Large KVM Infrastructure with ACS

[ilya]

I'm curious if anyone runs ACS with atleast 250+ KVM hosts.

We've been noticing weird issues with KVM where occasionally lots of KVM
agents get Nio connection closed issue followed by barrage of alerts.

In some instances the agent reconnects right away and in other - it
attempts to reconnect but never receives an ACK from MS.

Please let me know if you notice anything like it and if you found a
solution.

Also, it would help to know what global settings have been tuned to make
things work better (aside from direct.agent.*) and how MS are running.

Thanks
ilya

[GitHub] cloudstack pull request: CLOUDSTACK-9053 security upgrade as per C...

[DaanHoogland]

GitHub user DaanHoogland opened a pull request:

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

CLOUDSTACK-9053 security upgrade as per COLLECTIONS-580

  cloustack is not vulnerable but as the classes are in they might
  be used in the future so we upgrade to prevent accidental
  vulnerabilities.

build, unit tested and starting regression tests on it now

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

$ git pull https://github.com/DaanHoogland/cloudstack CLOUDSTACK-9053

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

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


commit 67da6974250459f78dbe2a8e11bee6ea6278e6b4
Author: Daan Hoogland 
Date:   2015-11-18T21:54:25Z

CLOUDSTACK-9053 security upgrade as per COLLECTIONS-580

  cloustack is not vulnerable but as the classes are in they might
  be used in the future so we upgrade to prevent accidental
  vulnerabilities.




---
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: [STABILITY]} Large KVM Infrastructure with ACS

[Daan Hoogland]

sounds like a bad limit Ilya, i'll keep an eye out.

On Wed, Nov 18, 2015 at 10:10 PM, ilya  wrote:

> I'm curious if anyone runs ACS with atleast 250+ KVM hosts.
>
> We've been noticing weird issues with KVM where occasionally lots of KVM
> agents get Nio connection closed issue followed by barrage of alerts.
>
> In some instances the agent reconnects right away and in other - it
> attempts to reconnect but never receives an ACK from MS.
>
> Please let me know if you notice anything like it and if you found a
> solution.
>
> Also, it would help to know what global settings have been tuned to make
> things work better (aside from direct.agent.*) and how MS are running.
>
> Thanks
> ilya
>



-- 
Daan

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

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/977#issuecomment-157883615
  
reran the tests. I noticed time out on destroy ssvm en cpvm but all others 
succeed. These timeouts, I have seen before in unrelated PRs. the systemvms 
come up but not quick enough for the test to succeed.

LGTM


[977.network.results.txt](https://github.com/apache/cloudstack/files/38402/977.network.results.txt)

[977.vpc.results.txt](https://github.com/apache/cloudstack/files/38403/977.vpc.results.txt)




---
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-9053 security upgrade as per C...

[DaanHoogland]

Github user DaanHoogland closed the pull request at:

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


---
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-9053 security upgrade as per C...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1088#issuecomment-157887108
  
should have been against 4.6


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

[GitHub] cloudstack pull request: CLOUDSTACK-9053 security upgrade as per C...

[DaanHoogland]

GitHub user DaanHoogland opened a pull request:

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

CLOUDSTACK-9053 security upgrade as per COLLECTIONS-580

  cloustack is not vulnerable but as the classes are in they might
  be used in the future so we upgrade to prevent accidental
  vulnerabilities.

integration test against master going on.

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

$ git pull https://github.com/DaanHoogland/cloudstack CLOUDSTACK-9053

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

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


commit d40d3498a6faa62fb8dc0df4d4e14b07a8363cb3
Author: Daan Hoogland 
Date:   2015-11-18T21:54:25Z

CLOUDSTACK-9053 security upgrade as per COLLECTIONS-580

  cloustack is not vulnerable but as the classes are in they might
  be used in the future so we upgrade to prevent accidental
  vulnerabilities.




---
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-9053 security upgrade as per C...

[jburwell]

Github user jburwell commented on the pull request:

https://github.com/apache/cloudstack/pull/1089#issuecomment-157899791
  
Assuming Jenkin passes, 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.
---

Re: [STABILITY]} Large KVM Infrastructure with ACS

[Rafael Weingärtner]

When you say 250+, you mean 250+ host spread in lots of cluster, right?
If I am not mistaken, ACS limits the number of KVM hosts in a cluster,
something like 50? I do not remember now if that value can be configured,
may it can be.

I recall to have read something in a Red Hat doc about the KVM that it does
not have limit of hosts in a cluster. Actually, it does not seem to have
the figure of cluster at all. That is created solely in ACS, to facilitate
the management.

To debug the problem, I would start with the following questions:

Is every single cluster of your environment is presenting that problem?
What is the size of physical hosts that you have in your environment? Do
all of them have the same configuration?
Do you know the load (resource allocated and used) that is being imposed in
those hosts that had shown those problems?
What is your over commitment/provisioning factor that you are using?


On Wed, Nov 18, 2015 at 8:19 PM, Daan Hoogland 
wrote:

> sounds like a bad limit Ilya, i'll keep an eye out.
>
> On Wed, Nov 18, 2015 at 10:10 PM, ilya 
> wrote:
>
> > I'm curious if anyone runs ACS with atleast 250+ KVM hosts.
> >
> > We've been noticing weird issues with KVM where occasionally lots of KVM
> > agents get Nio connection closed issue followed by barrage of alerts.
> >
> > In some instances the agent reconnects right away and in other - it
> > attempts to reconnect but never receives an ACK from MS.
> >
> > Please let me know if you notice anything like it and if you found a
> > solution.
> >
> > Also, it would help to know what global settings have been tuned to make
> > things work better (aside from direct.agent.*) and how MS are running.
> >
> > Thanks
> > ilya
> >
>
>
>
> --
> Daan
>



-- 
Rafael Weingärtner

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

[jenkins]

See 

--
[...truncated 27715 lines...]
[INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ 
cloud-quickcloud ---
[INFO] No sources to compile
[INFO] 
[INFO] >>> findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) @ 
cloud-quickcloud >>>
[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.964s]
[INFO] Apache CloudStack . SUCCESS [2.336s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [0.798s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [21.442s]
[INFO] Apache CloudStack Utils ... SUCCESS [1:30.664s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.115s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [54.951s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [27.780s]
[INFO] Apache CloudStack API . SUCCESS [1:51.411s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [17.122s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [30.173s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.122s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [27.449s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [25.553s]
[INFO] Apache CloudStack Core  SUCCESS [1:23.240s]
[INFO] Apache CloudStack Agents .. SUCCESS [35.443s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [35.635s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [15.676s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [2:17.985s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [51.340s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [30.293s]
[INFO] Apache CloudStack Server .. SUCCESS [4:39.004s]
[INFO] Apache CloudStack Usage Server  SUCCESS [49.467s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS 
[1:30.954s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.081s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.535s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [1:03.913s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [56.392s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [31.813s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [27.483s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [22.673s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [21.926s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [35.108s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [12.245s]
[INFO] Apache CloudStack Cloud Engine Service  SUCCESS [6.945s]
[INFO] Apache CloudStack Plugin POM .. SUCCESS [0.917s]
[INFO] Apache CloudStack Plugin - API Rate Limit . SUCCESS [3:00.631s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS 
[2:32.998s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS 
[54.554s]
[INFO] Apache CloudStack Plugin - API SolidFire .. SUCCESS [25.603s]
[INFO] Apache CloudStack Plugin - API Discovery .. SUCCESS [42.998s]
[INFO] Apache CloudStack Plugin - A

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

[jenkins]

See 

--
[...truncated 27715 lines...]
[INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ 
cloud-quickcloud ---
[INFO] No sources to compile
[INFO] 
[INFO] >>> findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) @ 
cloud-quickcloud >>>
[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.799s]
[INFO] Apache CloudStack . SUCCESS [2.177s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [0.785s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [21.298s]
[INFO] Apache CloudStack Utils ... SUCCESS [1:30.177s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.116s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [52.422s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [27.309s]
[INFO] Apache CloudStack API . SUCCESS [1:57.721s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [16.987s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [29.159s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.108s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [27.482s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [24.259s]
[INFO] Apache CloudStack Core  SUCCESS [1:20.749s]
[INFO] Apache CloudStack Agents .. SUCCESS [35.912s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [35.849s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [13.910s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [2:14.169s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [49.174s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [28.947s]
[INFO] Apache CloudStack Server .. SUCCESS [4:29.626s]
[INFO] Apache CloudStack Usage Server  SUCCESS [50.392s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS 
[1:31.509s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.071s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.447s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [1:05.080s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [55.266s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [29.941s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [26.133s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [23.027s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [21.868s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [33.586s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [13.123s]
[INFO] Apache CloudStack Cloud Engine Service  SUCCESS [5.728s]
[INFO] Apache CloudStack Plugin POM .. SUCCESS [0.744s]
[INFO] Apache CloudStack Plugin - API Rate Limit . SUCCESS [2:52.801s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS 
[2:38.215s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS 
[51.125s]
[INFO] Apache CloudStack Plugin - API SolidFire .. SUCCESS [21.158s]
[INFO] Apache CloudStack Plugin - API Discovery .. SUCCESS [48.042s]
[INFO] Apache CloudStack Plugin - A

[GitHub] cloudstack pull request: CLOUDSTACK-9067 - As I developer I want t...

[karuturi]

Github user karuturi commented on the pull request:

https://github.com/apache/cloudstack/pull/1084#issuecomment-157945578
  
@wilderrodrigues nope. No HyperV environment. 


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

[jenkins]

See 

--
[...truncated 27715 lines...]
[INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ 
cloud-quickcloud ---
[INFO] No sources to compile
[INFO] 
[INFO] >>> findbugs-maven-plugin:3.0.1:check (cloudstack-findbugs) @ 
cloud-quickcloud >>>
[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.909s]
[INFO] Apache CloudStack . SUCCESS [2.158s]
[INFO] Apache CloudStack Maven Conventions Parent  SUCCESS [0.790s]
[INFO] Apache CloudStack Framework - Managed Context . SUCCESS [20.811s]
[INFO] Apache CloudStack Utils ... SUCCESS [1:29.159s]
[INFO] Apache CloudStack Framework ... SUCCESS [0.115s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [53.343s]
[INFO] Apache CloudStack Framework - Configuration ... SUCCESS [26.002s]
[INFO] Apache CloudStack API . SUCCESS [1:49.659s]
[INFO] Apache CloudStack Framework - REST  SUCCESS [17.905s]
[INFO] Apache CloudStack Framework - IPC . SUCCESS [30.154s]
[INFO] Apache CloudStack Cloud Engine  SUCCESS [0.104s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [27.772s]
[INFO] Apache CloudStack Framework - Security  SUCCESS [25.098s]
[INFO] Apache CloudStack Core  SUCCESS [1:21.104s]
[INFO] Apache CloudStack Agents .. SUCCESS [34.346s]
[INFO] Apache CloudStack Framework - Clustering .. SUCCESS [36.105s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [13.987s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [2:11.577s]
[INFO] Apache CloudStack Framework - Jobs  SUCCESS [48.365s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [29.275s]
[INFO] Apache CloudStack Server .. SUCCESS [4:36.729s]
[INFO] Apache CloudStack Usage Server  SUCCESS [47.990s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS 
[1:26.430s]
[INFO] Apache CloudStack Cloud Services .. SUCCESS [0.075s]
[INFO] Apache CloudStack Secondary Storage ... SUCCESS [0.434s]
[INFO] Apache CloudStack Secondary Storage Service ... SUCCESS [1:03.179s]
[INFO] Apache CloudStack Engine Storage Component  SUCCESS [56.562s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [34.257s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [27.710s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [23.233s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [23.160s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [34.918s]
[INFO] Apache CloudStack Cloud Engine API  SUCCESS [12.855s]
[INFO] Apache CloudStack Cloud Engine Service  SUCCESS [6.273s]
[INFO] Apache CloudStack Plugin POM .. SUCCESS [0.775s]
[INFO] Apache CloudStack Plugin - API Rate Limit . SUCCESS [2:38.480s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS 
[2:50.898s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS 
[54.156s]
[INFO] Apache CloudStack Plugin - API SolidFire .. SUCCESS [18.521s]
[INFO] Apache CloudStack Plugin - API Discovery .. SUCCESS [46.627s]
[INFO] Apache CloudStack Plugin - A

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

[bhaisaab]

Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1038#issuecomment-157971218
  
Guys can we get some +1s on this, so let's at least merge on master? cc 
@wilderrodrigues @DaanHoogland @NuxRo @wido @abhinandanprateek and others

@DaanHoogland I'm interested in getting some of the commits related to UI 
width, sorting of tables etc. on 4.5 (we can remove the metrics view related 
commits). The reasoning here is that this does not change existing mgmt server 
core code, nor the schema. It's UI changes that improve existing experience for 
users.


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