Re: Inconsistencies in tests

[Srikanteswararao Talluri]

+1

Gaurav,

You were talking about integrating static code analysis in
jenkins.b.a.c.o. How is it going?

Thanks,
~Talluri

On 14/04/15 3:50 pm, "Sebastien Goasguen"  wrote:

>
>> On Apr 14, 2015, at 9:06 AM, Gaurav Aradhye
>> wrote:
>> 
>> Hi all,
>> 
>> I feel that the test code in BVT and Regression tests (smoke and
>>component folders) should be more consistent. There are feature
>>developers adding tests into smoke and different test developers adding
>>more tests into component folder, it is evident that inconsistencies are
>>bound to arise. But with time, we should be striving to minimize them.
>> 
>> I have listed few of the inconsistencies below. Feel free to add to the
>>list if you think of more.
>> 
>> 1. List Methods:
>> 
>> If we want to list accounts, some tests use Account.list method, while
>>some use list_accounts method. Both methods finally do the same thing,
>>but there are many list methods in common.py file corresponding base
>>class methods in base.py. This is code duplication.
>> 
>> I feel that BaseClass.method() is more readable than list_xyz(), or
>>should be preferred.
>> 
>> 
>> 2. Checking empty/None lists:
>> 
>> We already have validateList utility method which checks both empty
>>lists and None objects. No need to use ³isinstance² method, or add
>>additional None checks.
>> 
>> 
>> 3. Pep8 issues:
>> Many files are not pep8 consistent. In the past I have tried to fix
>>pep8 issues in files and the mission is ON. Efforts need to be put into
>>this. We already have autopep8 tool which does most of the things.
>> 
>
>+1
>
>that¹s really a low hanging fruit and best practice. we should not be
>merging anything that breaks pep8
>we should also add pylint tests
>
>checking for pep8 and python of all python code can be done at the high
>level using something like tox, kind of like the RAT tests
>
>> 
>> I will be creating separate issues in JIRA for listed things. Everyone
>>is welcome to add pull requests for these tasks.
>> I personally will be putting more efforts into these in upcoming few
>>months. 2-3 months and we should see most of the tests consistent with
>>each other.
>> 
>> Regards,
>> Gaurav
>> 
>

Re: Inconsistencies in tests

[Gaurav Aradhye]

Talluri, I still have to do it. Didn’t get enough time on it. Not a jenkins 
expert.
Can anybody help me in setting jenkins job for tests which will do static code 
analysis (python) and pep8 analysis?

Regards,
Gaurav Aradhye

On Apr 15, 2015, at 12:28 PM, Srikanteswararao Talluri 
 wrote:

> +1
> 
> Gaurav,
> 
> You were talking about integrating static code analysis in
> jenkins.b.a.c.o. How is it going?
> 
> Thanks,
> ~Talluri
> 
> On 14/04/15 3:50 pm, "Sebastien Goasguen"  wrote:
> 
>> 
>>> On Apr 14, 2015, at 9:06 AM, Gaurav Aradhye
>>>  wrote:
>>> 
>>> Hi all,
>>> 
>>> I feel that the test code in BVT and Regression tests (smoke and
>>> component folders) should be more consistent. There are feature
>>> developers adding tests into smoke and different test developers adding
>>> more tests into component folder, it is evident that inconsistencies are
>>> bound to arise. But with time, we should be striving to minimize them.
>>> 
>>> I have listed few of the inconsistencies below. Feel free to add to the
>>> list if you think of more.
>>> 
>>> 1. List Methods:
>>> 
>>> If we want to list accounts, some tests use Account.list method, while
>>> some use list_accounts method. Both methods finally do the same thing,
>>> but there are many list methods in common.py file corresponding base
>>> class methods in base.py. This is code duplication.
>>> 
>>> I feel that BaseClass.method() is more readable than list_xyz(), or
>>> should be preferred.
>>> 
>>> 
>>> 2. Checking empty/None lists:
>>> 
>>> We already have validateList utility method which checks both empty
>>> lists and None objects. No need to use ³isinstance² method, or add
>>> additional None checks.
>>> 
>>> 
>>> 3. Pep8 issues:
>>> Many files are not pep8 consistent. In the past I have tried to fix
>>> pep8 issues in files and the mission is ON. Efforts need to be put into
>>> this. We already have autopep8 tool which does most of the things.
>>> 
>> 
>> +1
>> 
>> that¹s really a low hanging fruit and best practice. we should not be
>> merging anything that breaks pep8
>> we should also add pylint tests
>> 
>> checking for pep8 and python of all python code can be done at the high
>> level using something like tox, kind of like the RAT tests
>> 
>>> 
>>> I will be creating separate issues in JIRA for listed things. Everyone
>>> is welcome to add pull requests for these tasks.
>>> I personally will be putting more efforts into these in upcoming few
>>> months. 2-3 months and we should see most of the tests consistent with
>>> each other.
>>> 
>>> Regards,
>>> Gaurav

Re: Inconsistencies in tests

[Srikanteswararao Talluri]

I will help you set the job up.

On 15/04/15 12:34 pm, "Gaurav Aradhye"  wrote:

>Talluri, I still have to do it. Didn’t get enough time on it. Not a
>jenkins expert.
>Can anybody help me in setting jenkins job for tests which will do static
>code analysis (python) and pep8 analysis?
>
>Regards,
>Gaurav Aradhye
>
>On Apr 15, 2015, at 12:28 PM, Srikanteswararao Talluri
> wrote:
>
>> +1
>> 
>> Gaurav,
>> 
>> You were talking about integrating static code analysis in
>> jenkins.b.a.c.o. How is it going?
>> 
>> Thanks,
>> ~Talluri
>> 
>> On 14/04/15 3:50 pm, "Sebastien Goasguen"  wrote:
>> 
>>> 
 On Apr 14, 2015, at 9:06 AM, Gaurav Aradhye
  wrote:
 
 Hi all,
 
 I feel that the test code in BVT and Regression tests (smoke and
 component folders) should be more consistent. There are feature
 developers adding tests into smoke and different test developers
adding
 more tests into component folder, it is evident that inconsistencies
are
 bound to arise. But with time, we should be striving to minimize them.
 
 I have listed few of the inconsistencies below. Feel free to add to
the
 list if you think of more.
 
 1. List Methods:
 
 If we want to list accounts, some tests use Account.list method, while
 some use list_accounts method. Both methods finally do the same thing,
 but there are many list methods in common.py file corresponding base
 class methods in base.py. This is code duplication.
 
 I feel that BaseClass.method() is more readable than list_xyz(), or
 should be preferred.
 
 
 2. Checking empty/None lists:
 
 We already have validateList utility method which checks both empty
 lists and None objects. No need to use ³isinstance² method, or add
 additional None checks.
 
 
 3. Pep8 issues:
 Many files are not pep8 consistent. In the past I have tried to fix
 pep8 issues in files and the mission is ON. Efforts need to be put
into
 this. We already have autopep8 tool which does most of the things.
 
>>> 
>>> +1
>>> 
>>> that¹s really a low hanging fruit and best practice. we should not be
>>> merging anything that breaks pep8
>>> we should also add pylint tests
>>> 
>>> checking for pep8 and python of all python code can be done at the high
>>> level using something like tox, kind of like the RAT tests
>>> 
 
 I will be creating separate issues in JIRA for listed things. Everyone
 is welcome to add pull requests for these tasks.
 I personally will be putting more efforts into these in upcoming few
 months. 2-3 months and we should see most of the tests consistent with
 each other.
 
 Regards,
 Gaurav
>

Re: 4.5 clustered mgmt server

[Bharat Kumar]

Hi,

I guss you are correct. we need to add the newly created channel ch1 to the map 
after the handshake is complete.

_peers.put(peerName, ch) should be changed to _peers.put(peerName, ch1); 

and also return ch1 instead of ch.

Thanks,
Bharat.


On 15-Apr-2015, at 12:21 pm, Marcus  wrote:

> Is anyone running 4.5 mgmt server in cluster setup? I've run into an
> issue where the channel between the two servers is null, and I think
> I've tracked it down to "coverity fixes" from
> 78bfaa79cf3eaa170ef9422bf8fb1825c0cecfc1.
> 
> The method returns 'ch', which is null in the error. The coverity fix
> changes ch for ch1 and seems to initialize ch1 and then drop it on the
> floor. If someone is familiar with this code I'd appreciate a review
> of it.
> 
> diff --git 
> a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
> b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
> index 600dca2..e38489b 100755
> --- 
> a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
> +++ 
> b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
> @@ -498,18 +498,17 @@ public class ClusteredAgentManagerImpl extends
> AgentManagerImpl implements Clust
> } catch (UnknownHostException e) {
> throw new CloudRuntimeException("Unable to resolve " + 
> ip);
> }
> -try {
> -ch = SocketChannel.open(new
> InetSocketAddress(addr, Port.value()));
> -ch.configureBlocking(true); // make sure we are
> working at blocking mode
> -ch.socket().setKeepAlive(true);
> -ch.socket().setSoTimeout(60 * 1000);
> +try (SocketChannel ch1 = SocketChannel.open(new
> InetSocketAddress(addr, Port.value()));){
> +ch1.configureBlocking(true); // make sure we are
> working at blocking mode
> +ch1.socket().setKeepAlive(true);
> +ch1.socket().setSoTimeout(60 * 1000);
> try {
> SSLContext sslContext = Link.initSSLContext(true);
> sslEngine = sslContext.createSSLEngine(ip,
> Port.value());
> sslEngine.setUseClientMode(true);
> 
> sslEngine.setEnabledProtocols(SSLUtils.getSupportedProtocols(sslEngine.getEnabledProtocols()));
> 
> -Link.doHandshake(ch, sslEngine, true);
> +Link.doHandshake(ch1, sslEngine, true);
> s_logger.info("SSL: Handshake done");
> } catch (Exception e) {
> throw new IOException("SSL: Fail to init SSL! " + e);

Re: 4.5 clustered mgmt server

[Marcus]

Yeah, I backed out the change in my test environment and the issue went away.

On Wed, Apr 15, 2015 at 12:07 AM, Bharat Kumar  wrote:
> Hi,
>
> I guss you are correct. we need to add the newly created channel ch1 to the 
> map after the handshake is complete.
>
> _peers.put(peerName, ch) should be changed to _peers.put(peerName, ch1);
>
> and also return ch1 instead of ch.
>
> Thanks,
> Bharat.
>
>
> On 15-Apr-2015, at 12:21 pm, Marcus  wrote:
>
>> Is anyone running 4.5 mgmt server in cluster setup? I've run into an
>> issue where the channel between the two servers is null, and I think
>> I've tracked it down to "coverity fixes" from
>> 78bfaa79cf3eaa170ef9422bf8fb1825c0cecfc1.
>>
>> The method returns 'ch', which is null in the error. The coverity fix
>> changes ch for ch1 and seems to initialize ch1 and then drop it on the
>> floor. If someone is familiar with this code I'd appreciate a review
>> of it.
>>
>> diff --git 
>> a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
>> b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
>> index 600dca2..e38489b 100755
>> --- 
>> a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
>> +++ 
>> b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
>> @@ -498,18 +498,17 @@ public class ClusteredAgentManagerImpl extends
>> AgentManagerImpl implements Clust
>> } catch (UnknownHostException e) {
>> throw new CloudRuntimeException("Unable to resolve " + 
>> ip);
>> }
>> -try {
>> -ch = SocketChannel.open(new
>> InetSocketAddress(addr, Port.value()));
>> -ch.configureBlocking(true); // make sure we are
>> working at blocking mode
>> -ch.socket().setKeepAlive(true);
>> -ch.socket().setSoTimeout(60 * 1000);
>> +try (SocketChannel ch1 = SocketChannel.open(new
>> InetSocketAddress(addr, Port.value()));){
>> +ch1.configureBlocking(true); // make sure we are
>> working at blocking mode
>> +ch1.socket().setKeepAlive(true);
>> +ch1.socket().setSoTimeout(60 * 1000);
>> try {
>> SSLContext sslContext = Link.initSSLContext(true);
>> sslEngine = sslContext.createSSLEngine(ip,
>> Port.value());
>> sslEngine.setUseClientMode(true);
>>
>> sslEngine.setEnabledProtocols(SSLUtils.getSupportedProtocols(sslEngine.getEnabledProtocols()));
>>
>> -Link.doHandshake(ch, sslEngine, true);
>> +Link.doHandshake(ch1, sslEngine, true);
>> s_logger.info("SSL: Handshake done");
>> } catch (Exception e) {
>> throw new IOException("SSL: Fail to init SSL! " + e);
>

Build failed in Jenkins: simulator-singlerun #1109

[jenkins]

See 

Changes:

[Rajani Karuturi] findbugs and PMD fixes

--
[...truncated 3445 lines...]
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 4 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ 
cloud-plugin-hypervisor-xenserver ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 94 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-plugin-hypervisor-xenserver ---
[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-plugin-hypervisor-xenserver ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 7 source files to 

[INFO] 
[INFO] --- maven-surefire-plugin:2.18.1:test (default-test) @ 
cloud-plugin-hypervisor-xenserver ---
[INFO] Surefire report directory: 


---
 T E S T S
---
Running com.cloud.hypervisor.xenserver.resource.wrapper.XenServer56WrapperTest
log4j:WARN No appenders could be found for logger 
(com.cloud.hypervisor.xenserver.resource.XenServerConnectionPool).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more 
info.
Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.849 sec - in 
com.cloud.hypervisor.xenserver.resource.wrapper.XenServer56WrapperTest
Running com.cloud.hypervisor.xenserver.resource.wrapper.XcpServerWrapperTest
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.099 sec - in 
com.cloud.hypervisor.xenserver.resource.wrapper.XcpServerWrapperTest
Running com.cloud.hypervisor.xenserver.resource.wrapper.CitrixRequestWrapperTest
log4j:WARN No appenders could be found for logger 
(com.cloud.hypervisor.xenserver.resource.XenServerConnectionPool).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more 
info.
Tests run: 77, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 6.292 sec - in 
com.cloud.hypervisor.xenserver.resource.wrapper.CitrixRequestWrapperTest
Running com.cloud.hypervisor.xenserver.resource.wrapper.XenServer620WrapperTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.007 sec - in 
com.cloud.hypervisor.xenserver.resource.wrapper.XenServer620WrapperTest
Running 
com.cloud.hypervisor.xenserver.resource.wrapper.XenServer620SP1WrapperTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.067 sec - in 
com.cloud.hypervisor.xenserver.resource.wrapper.XenServer620SP1WrapperTest
Running 
com.cloud.hypervisor.xenserver.resource.wrapper.XenServer56FP1WrapperTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.057 sec - in 
com.cloud.hypervisor.xenserver.resource.wrapper.XenServer56FP1WrapperTest
Running com.cloud.hypervisor.xenserver.resource.wrapper.XenServer610WrapperTest
Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.239 sec - in 
com.cloud.hypervisor.xenserver.resource.wrapper.XenServer610WrapperTest

Results :

Tests run: 100, Failures: 0, Errors: 0, Skipped: 0

[INFO] 
[INFO] --- maven-jar-plugin:2.5:jar (default-jar) @ 
cloud-plugin-hypervisor-xenserver ---
[INFO] Building jar: 

[INFO] 
[INFO] --- maven-site-plugin:3.3:attach-descriptor (attach-descriptor) @ 
cloud-plugin-hypervisor-xenserver ---
[INFO] 
[INFO] --- maven-install-plugin:2.3.1:install (default-install) @ 
cloud-plugin-hypervisor-xenserver ---
[INFO] Installing 

 to 
/var/lib/jenkins/.m2/repository/org/apache/cloudstack/cloud-plugin-hypervisor-xenserver/4.6.0-SNAPSHOT/cloud-plugin-hypervisor-xenserver-4.6.0-SNAPSHOT.jar
[INFO] Installing 

 to 
/var/lib/jenkins/.m2/repository/org/apache/cloudstack/cloud-plugin-hypervisor-xenserver/4.6.0-SNAPSHOT/cloud-plugin-hypervisor-x

Re: 4.5 clustered mgmt server

[Marcus]

I'm opening a bug, should I assign it to you, or do you know of
someone who maintains this code? It is probably pretty urgent to go
out in the next 4.5 bugfix


On Wed, Apr 15, 2015 at 12:11 AM, Marcus  wrote:
> Yeah, I backed out the change in my test environment and the issue went away.
>
> On Wed, Apr 15, 2015 at 12:07 AM, Bharat Kumar  
> wrote:
>> Hi,
>>
>> I guss you are correct. we need to add the newly created channel ch1 to the 
>> map after the handshake is complete.
>>
>> _peers.put(peerName, ch) should be changed to _peers.put(peerName, ch1);
>>
>> and also return ch1 instead of ch.
>>
>> Thanks,
>> Bharat.
>>
>>
>> On 15-Apr-2015, at 12:21 pm, Marcus  wrote:
>>
>>> Is anyone running 4.5 mgmt server in cluster setup? I've run into an
>>> issue where the channel between the two servers is null, and I think
>>> I've tracked it down to "coverity fixes" from
>>> 78bfaa79cf3eaa170ef9422bf8fb1825c0cecfc1.
>>>
>>> The method returns 'ch', which is null in the error. The coverity fix
>>> changes ch for ch1 and seems to initialize ch1 and then drop it on the
>>> floor. If someone is familiar with this code I'd appreciate a review
>>> of it.
>>>
>>> diff --git 
>>> a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
>>> b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
>>> index 600dca2..e38489b 100755
>>> --- 
>>> a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
>>> +++ 
>>> b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
>>> @@ -498,18 +498,17 @@ public class ClusteredAgentManagerImpl extends
>>> AgentManagerImpl implements Clust
>>> } catch (UnknownHostException e) {
>>> throw new CloudRuntimeException("Unable to resolve " + 
>>> ip);
>>> }
>>> -try {
>>> -ch = SocketChannel.open(new
>>> InetSocketAddress(addr, Port.value()));
>>> -ch.configureBlocking(true); // make sure we are
>>> working at blocking mode
>>> -ch.socket().setKeepAlive(true);
>>> -ch.socket().setSoTimeout(60 * 1000);
>>> +try (SocketChannel ch1 = SocketChannel.open(new
>>> InetSocketAddress(addr, Port.value()));){
>>> +ch1.configureBlocking(true); // make sure we are
>>> working at blocking mode
>>> +ch1.socket().setKeepAlive(true);
>>> +ch1.socket().setSoTimeout(60 * 1000);
>>> try {
>>> SSLContext sslContext = Link.initSSLContext(true);
>>> sslEngine = sslContext.createSSLEngine(ip,
>>> Port.value());
>>> sslEngine.setUseClientMode(true);
>>>
>>> sslEngine.setEnabledProtocols(SSLUtils.getSupportedProtocols(sslEngine.getEnabledProtocols()));
>>>
>>> -Link.doHandshake(ch, sslEngine, true);
>>> +Link.doHandshake(ch1, sslEngine, true);
>>> s_logger.info("SSL: Handshake done");
>>> } catch (Exception e) {
>>> throw new IOException("SSL: Fail to init SSL! " + 
>>> e);
>>

Re: 4.5 clustered mgmt server

[Marcus]

https://issues.apache.org/jira/browse/CLOUDSTACK-8387 if anyone can take it.

On Wed, Apr 15, 2015 at 12:17 AM, Marcus  wrote:
> I'm opening a bug, should I assign it to you, or do you know of
> someone who maintains this code? It is probably pretty urgent to go
> out in the next 4.5 bugfix
>
>
> On Wed, Apr 15, 2015 at 12:11 AM, Marcus  wrote:
>> Yeah, I backed out the change in my test environment and the issue went away.
>>
>> On Wed, Apr 15, 2015 at 12:07 AM, Bharat Kumar  
>> wrote:
>>> Hi,
>>>
>>> I guss you are correct. we need to add the newly created channel ch1 to the 
>>> map after the handshake is complete.
>>>
>>> _peers.put(peerName, ch) should be changed to _peers.put(peerName, ch1);
>>>
>>> and also return ch1 instead of ch.
>>>
>>> Thanks,
>>> Bharat.
>>>
>>>
>>> On 15-Apr-2015, at 12:21 pm, Marcus  wrote:
>>>
 Is anyone running 4.5 mgmt server in cluster setup? I've run into an
 issue where the channel between the two servers is null, and I think
 I've tracked it down to "coverity fixes" from
 78bfaa79cf3eaa170ef9422bf8fb1825c0cecfc1.

 The method returns 'ch', which is null in the error. The coverity fix
 changes ch for ch1 and seems to initialize ch1 and then drop it on the
 floor. If someone is familiar with this code I'd appreciate a review
 of it.

 diff --git 
 a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
 b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
 index 600dca2..e38489b 100755
 --- 
 a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
 +++ 
 b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
 @@ -498,18 +498,17 @@ public class ClusteredAgentManagerImpl extends
 AgentManagerImpl implements Clust
 } catch (UnknownHostException e) {
 throw new CloudRuntimeException("Unable to resolve " + 
 ip);
 }
 -try {
 -ch = SocketChannel.open(new
 InetSocketAddress(addr, Port.value()));
 -ch.configureBlocking(true); // make sure we are
 working at blocking mode
 -ch.socket().setKeepAlive(true);
 -ch.socket().setSoTimeout(60 * 1000);
 +try (SocketChannel ch1 = SocketChannel.open(new
 InetSocketAddress(addr, Port.value()));){
 +ch1.configureBlocking(true); // make sure we are
 working at blocking mode
 +ch1.socket().setKeepAlive(true);
 +ch1.socket().setSoTimeout(60 * 1000);
 try {
 SSLContext sslContext = Link.initSSLContext(true);
 sslEngine = sslContext.createSSLEngine(ip,
 Port.value());
 sslEngine.setUseClientMode(true);

 sslEngine.setEnabledProtocols(SSLUtils.getSupportedProtocols(sslEngine.getEnabledProtocols()));

 -Link.doHandshake(ch, sslEngine, true);
 +Link.doHandshake(ch1, sslEngine, true);
 s_logger.info("SSL: Handshake done");
 } catch (Exception e) {
 throw new IOException("SSL: Fail to init SSL! " + 
 e);
>>>

Re: 4.5 clustered mgmt server

[Erik Weber]

Would this explain things like this in a cluster node log, or am I seeing
something else?

2015-04-08 22:22:47,471 INFO  [c.c.a.m.ClusteredAgentManagerImpl]
(StatsCollector-1:ctx-8f16a26f) SSL: Handshake done
2015-04-08 22:22:47,471 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
(StatsCollector-1:ctx-8f16a26f) Connection to peer opened: 345049427801,
ip: 10.126.0.2
2015-04-08 22:22:47,471 DEBUG [c.c.a.m.ClusteredAgentAttache]
(StatsCollector-1:ctx-8f16a26f) Seq 4-9218586962250694669: Unable to
forward null
2015-04-08 22:22:47,471 WARN  [c.c.a.m.AgentManagerImpl]
(StatsCollector-1:ctx-8f16a26f) Resource [Host:4] is unreachable: Host 4:
Unable to reach the peer that the agent is connected


-- 
Erik

On Wed, Apr 15, 2015 at 9:20 AM, Marcus  wrote:

> https://issues.apache.org/jira/browse/CLOUDSTACK-8387 if anyone can take
> it.
>
> On Wed, Apr 15, 2015 at 12:17 AM, Marcus  wrote:
> > I'm opening a bug, should I assign it to you, or do you know of
> > someone who maintains this code? It is probably pretty urgent to go
> > out in the next 4.5 bugfix
> >
> >
> > On Wed, Apr 15, 2015 at 12:11 AM, Marcus  wrote:
> >> Yeah, I backed out the change in my test environment and the issue went
> away.
> >>
> >> On Wed, Apr 15, 2015 at 12:07 AM, Bharat Kumar 
> wrote:
> >>> Hi,
> >>>
> >>> I guss you are correct. we need to add the newly created channel ch1
> to the map after the handshake is complete.
> >>>
> >>> _peers.put(peerName, ch) should be changed to _peers.put(peerName,
> ch1);
> >>>
> >>> and also return ch1 instead of ch.
> >>>
> >>> Thanks,
> >>> Bharat.
> >>>
> >>>
> >>> On 15-Apr-2015, at 12:21 pm, Marcus  wrote:
> >>>
>  Is anyone running 4.5 mgmt server in cluster setup? I've run into an
>  issue where the channel between the two servers is null, and I think
>  I've tracked it down to "coverity fixes" from
>  78bfaa79cf3eaa170ef9422bf8fb1825c0cecfc1.
> 
>  The method returns 'ch', which is null in the error. The coverity fix
>  changes ch for ch1 and seems to initialize ch1 and then drop it on the
>  floor. If someone is familiar with this code I'd appreciate a review
>  of it.
> 
>  diff --git
> a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
> 
> b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
>  index 600dca2..e38489b 100755
>  ---
> a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
>  +++
> b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
>  @@ -498,18 +498,17 @@ public class ClusteredAgentManagerImpl extends
>  AgentManagerImpl implements Clust
>  } catch (UnknownHostException e) {
>  throw new CloudRuntimeException("Unable to
> resolve " + ip);
>  }
>  -try {
>  -ch = SocketChannel.open(new
>  InetSocketAddress(addr, Port.value()));
>  -ch.configureBlocking(true); // make sure we are
>  working at blocking mode
>  -ch.socket().setKeepAlive(true);
>  -ch.socket().setSoTimeout(60 * 1000);
>  +try (SocketChannel ch1 = SocketChannel.open(new
>  InetSocketAddress(addr, Port.value()));){
>  +ch1.configureBlocking(true); // make sure we are
>  working at blocking mode
>  +ch1.socket().setKeepAlive(true);
>  +ch1.socket().setSoTimeout(60 * 1000);
>  try {
>  SSLContext sslContext =
> Link.initSSLContext(true);
>  sslEngine = sslContext.createSSLEngine(ip,
>  Port.value());
>  sslEngine.setUseClientMode(true);
> 
> 
> sslEngine.setEnabledProtocols(SSLUtils.getSupportedProtocols(sslEngine.getEnabledProtocols()));
> 
>  -Link.doHandshake(ch, sslEngine, true);
>  +Link.doHandshake(ch1, sslEngine, true);
>  s_logger.info("SSL: Handshake done");
>  } catch (Exception e) {
>  throw new IOException("SSL: Fail to init SSL!
> " + e);
> >>>
>

AW: Missing storage label in XenCenter after migrate

[S . Brüseke - proIO GmbH]

Hi,
 
thank you for testing. In 4.3 the name label is empty even when the vm is down.
 
Mit freundlichen Grüßen / With kind regards,
 
Swen Brüseke
 
Von: raja sekhar [mailto:rajsekhar@gmail.com] 
Gesendet: Dienstag, 14. April 2015 18:48
An: dev@cloudstack.apache.org; S. Brüseke - proIO GmbH
Betreff: Re: Missing storage label in XenCenter after migrate
 
Bruseke,
 
When the disks are live migrated from one primary to another this problem is 
arising, i'am also getting same problem, the disk labels are changed to 
"Cloud-".it seems to be a bug.
how ever it doesn't happens when the vm is down.
 
On Tue, Apr 14, 2015 at 6:11 PM, S. Brüseke - proIO GmbH  
wrote:
Hi,

can somebody please test this behavior in 4.5.1-rc2?

1. Create instance on XenServer hypervisor via CS.
2. Check "Name" of storage in XenCenter's storage tab for this instance. 
(should be ROOT-XXX)
3. Migrate primary storage to different storage via CS.
4. Take a look at "Name" of storage in XenCenter's storage tab again. (It is 
empty now)

If this is the case on your installation I consider this a bug. Can please 
someone verify?
Thank you very much!

Mit freundlichen Grüßen / With kind regards,

Swen Brüseke




- proIO GmbH -
Geschäftsführer: Swen Brüseke
Sitz der Gesellschaft: Frankfurt am Main

USt-IdNr. DE 267 075 918
Registergericht: Frankfurt am Main - HRB 86239

Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte Informationen.
Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich erhalten 
haben,
informieren Sie bitte sofort den Absender und vernichten Sie diese Mail.
Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieser Mail sind nicht 
gestattet.

This e-mail may contain confidential and/or privileged information.
If you are not the intended recipient (or have received this e-mail in error) 
please notify
the sender immediately and destroy this e-mail.
Any unauthorized copying, disclosure or distribution of the material in this 
e-mail is strictly forbidden.


- proIO GmbH -
Geschäftsführer: Swen Brüseke
Sitz der Gesellschaft: Frankfurt am Main

USt-IdNr. DE 267 075 918
Registergericht: Frankfurt am Main - HRB 86239

Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte Informationen. 
Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich erhalten 
haben, 
informieren Sie bitte sofort den Absender und vernichten Sie diese Mail. 
Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieser Mail sind nicht 
gestattet. 

This e-mail may contain confidential and/or privileged information. 
If you are not the intended recipient (or have received this e-mail in error) 
please notify 
the sender immediately and destroy this e-mail.  
Any unauthorized copying, disclosure or distribution of the material in this 
e-mail is strictly forbidden. 

Re: Missing storage label in XenCenter after migrate

[raja sekhar]

I have tested in ACS 4.3.1 with Xenserver 6.2 as hyperviser.
When vm is down the name label remains same,but when the vm is live name
label changes.

On Wed, Apr 15, 2015 at 1:01 PM, S. Brüseke - proIO GmbH <
s.brues...@proio.com> wrote:

> Hi,
>
>
>
> thank you for testing. In 4.3 the name label is empty even when the vm is
> down.
>
>
>
> Mit freundlichen Grüßen / With kind regards,
>
>
>
> Swen Brüseke
>
>
>
> *Von:* raja sekhar [mailto:rajsekhar@gmail.com]
> *Gesendet:* Dienstag, 14. April 2015 18:48
> *An:* dev@cloudstack.apache.org; S. Brüseke - proIO GmbH
> *Betreff:* Re: Missing storage label in XenCenter after migrate
>
>
>
> Bruseke,
>
>
>
> When the disks are live migrated from one primary to another this problem
> is arising, i'am also getting same problem, the disk labels are changed to
> "Cloud-".it seems to be a bug.
>
> how ever it doesn't happens when the vm is down.
>
>
>
> On Tue, Apr 14, 2015 at 6:11 PM, S. Brüseke - proIO GmbH <
> s.brues...@proio.com> wrote:
>
> Hi,
>
> can somebody please test this behavior in 4.5.1-rc2?
>
> 1. Create instance on XenServer hypervisor via CS.
> 2. Check "Name" of storage in XenCenter's storage tab for this instance.
> (should be ROOT-XXX)
> 3. Migrate primary storage to different storage via CS.
> 4. Take a look at "Name" of storage in XenCenter's storage tab again. (It
> is empty now)
>
> If this is the case on your installation I consider this a bug. Can please
> someone verify?
> Thank you very much!
>
> Mit freundlichen Grüßen / With kind regards,
>
> Swen Brüseke
>
>
>
>
> - proIO GmbH -
> Geschäftsführer: Swen Brüseke
> Sitz der Gesellschaft: Frankfurt am Main
>
> USt-IdNr. DE 267 075 918
> Registergericht: Frankfurt am Main - HRB 86239
>
> Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte
> Informationen.
> Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich
> erhalten haben,
> informieren Sie bitte sofort den Absender und vernichten Sie diese Mail.
> Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieser Mail sind
> nicht gestattet.
>
> This e-mail may contain confidential and/or privileged information.
> If you are not the intended recipient (or have received this e-mail in
> error) please notify
> the sender immediately and destroy this e-mail.
> Any unauthorized copying, disclosure or distribution of the material in
> this e-mail is strictly forbidden.
>
>
>
>
> --
> - proIO GmbH -
> Geschäftsführer: Swen Brüseke
> Sitz der Gesellschaft: Frankfurt am Main
>
> USt-IdNr. DE 267 075 918
> Registergericht: Frankfurt am Main - HRB 86239
>
>
> Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte Informationen.
>
> Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich 
> erhalten haben,
> informieren Sie bitte sofort den Absender und vernichten Sie diese Mail.
>
> Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieser Mail sind nicht 
> gestattet.
>
> This e-mail may contain confidential and/or privileged information.
>
> If you are not the intended recipient (or have received this e-mail in error) 
> please notify
> the sender immediately and destroy this e-mail.
>
> Any unauthorized copying, disclosure or distribution of the material in this 
> e-mail is strictly forbidden.
>
>

Re: 4.5 clustered mgmt server

[Marcus]

Yes
On Apr 15, 2015 12:30 AM, "Erik Weber"  wrote:

> Would this explain things like this in a cluster node log, or am I seeing
> something else?
>
> 2015-04-08 22:22:47,471 INFO  [c.c.a.m.ClusteredAgentManagerImpl]
> (StatsCollector-1:ctx-8f16a26f) SSL: Handshake done
> 2015-04-08 22:22:47,471 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> (StatsCollector-1:ctx-8f16a26f) Connection to peer opened: 345049427801,
> ip: 10.126.0.2
> 2015-04-08 22:22:47,471 DEBUG [c.c.a.m.ClusteredAgentAttache]
> (StatsCollector-1:ctx-8f16a26f) Seq 4-9218586962250694669: Unable to
> forward null
> 2015-04-08 22:22:47,471 WARN  [c.c.a.m.AgentManagerImpl]
> (StatsCollector-1:ctx-8f16a26f) Resource [Host:4] is unreachable: Host 4:
> Unable to reach the peer that the agent is connected
>
>
> --
> Erik
>
> On Wed, Apr 15, 2015 at 9:20 AM, Marcus  wrote:
>
> > https://issues.apache.org/jira/browse/CLOUDSTACK-8387 if anyone can take
> > it.
> >
> > On Wed, Apr 15, 2015 at 12:17 AM, Marcus  wrote:
> > > I'm opening a bug, should I assign it to you, or do you know of
> > > someone who maintains this code? It is probably pretty urgent to go
> > > out in the next 4.5 bugfix
> > >
> > >
> > > On Wed, Apr 15, 2015 at 12:11 AM, Marcus  wrote:
> > >> Yeah, I backed out the change in my test environment and the issue
> went
> > away.
> > >>
> > >> On Wed, Apr 15, 2015 at 12:07 AM, Bharat Kumar <
> bharat.ku...@citrix.com>
> > wrote:
> > >>> Hi,
> > >>>
> > >>> I guss you are correct. we need to add the newly created channel ch1
> > to the map after the handshake is complete.
> > >>>
> > >>> _peers.put(peerName, ch) should be changed to _peers.put(peerName,
> > ch1);
> > >>>
> > >>> and also return ch1 instead of ch.
> > >>>
> > >>> Thanks,
> > >>> Bharat.
> > >>>
> > >>>
> > >>> On 15-Apr-2015, at 12:21 pm, Marcus  wrote:
> > >>>
> >  Is anyone running 4.5 mgmt server in cluster setup? I've run into an
> >  issue where the channel between the two servers is null, and I think
> >  I've tracked it down to "coverity fixes" from
> >  78bfaa79cf3eaa170ef9422bf8fb1825c0cecfc1.
> > 
> >  The method returns 'ch', which is null in the error. The coverity
> fix
> >  changes ch for ch1 and seems to initialize ch1 and then drop it on
> the
> >  floor. If someone is familiar with this code I'd appreciate a review
> >  of it.
> > 
> >  diff --git
> >
> a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
> > 
> >
> b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
> >  index 600dca2..e38489b 100755
> >  ---
> >
> a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
> >  +++
> >
> b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
> >  @@ -498,18 +498,17 @@ public class ClusteredAgentManagerImpl extends
> >  AgentManagerImpl implements Clust
> >  } catch (UnknownHostException e) {
> >  throw new CloudRuntimeException("Unable to
> > resolve " + ip);
> >  }
> >  -try {
> >  -ch = SocketChannel.open(new
> >  InetSocketAddress(addr, Port.value()));
> >  -ch.configureBlocking(true); // make sure we are
> >  working at blocking mode
> >  -ch.socket().setKeepAlive(true);
> >  -ch.socket().setSoTimeout(60 * 1000);
> >  +try (SocketChannel ch1 = SocketChannel.open(new
> >  InetSocketAddress(addr, Port.value()));){
> >  +ch1.configureBlocking(true); // make sure we
> are
> >  working at blocking mode
> >  +ch1.socket().setKeepAlive(true);
> >  +ch1.socket().setSoTimeout(60 * 1000);
> >  try {
> >  SSLContext sslContext =
> > Link.initSSLContext(true);
> >  sslEngine = sslContext.createSSLEngine(ip,
> >  Port.value());
> >  sslEngine.setUseClientMode(true);
> > 
> > 
> >
> sslEngine.setEnabledProtocols(SSLUtils.getSupportedProtocols(sslEngine.getEnabledProtocols()));
> > 
> >  -Link.doHandshake(ch, sslEngine, true);
> >  +Link.doHandshake(ch1, sslEngine, true);
> >  s_logger.info("SSL: Handshake done");
> >  } catch (Exception e) {
> >  throw new IOException("SSL: Fail to init
> SSL!
> > " + e);
> > >>>
> >
>

Re: 4.5 clustered mgmt server

[Erik Weber]

Excellent, thanks :-)

-- 
Erik

On Wed, Apr 15, 2015 at 9:45 AM, Marcus  wrote:

> Yes
> On Apr 15, 2015 12:30 AM, "Erik Weber"  wrote:
>
> > Would this explain things like this in a cluster node log, or am I seeing
> > something else?
> >
> > 2015-04-08 22:22:47,471 INFO  [c.c.a.m.ClusteredAgentManagerImpl]
> > (StatsCollector-1:ctx-8f16a26f) SSL: Handshake done
> > 2015-04-08 22:22:47,471 DEBUG [c.c.a.m.ClusteredAgentManagerImpl]
> > (StatsCollector-1:ctx-8f16a26f) Connection to peer opened: 345049427801,
> > ip: 10.126.0.2
> > 2015-04-08 22:22:47,471 DEBUG [c.c.a.m.ClusteredAgentAttache]
> > (StatsCollector-1:ctx-8f16a26f) Seq 4-9218586962250694669: Unable to
> > forward null
> > 2015-04-08 22:22:47,471 WARN  [c.c.a.m.AgentManagerImpl]
> > (StatsCollector-1:ctx-8f16a26f) Resource [Host:4] is unreachable: Host 4:
> > Unable to reach the peer that the agent is connected
> >
> >
> > --
> > Erik
> >
> > On Wed, Apr 15, 2015 at 9:20 AM, Marcus  wrote:
> >
> > > https://issues.apache.org/jira/browse/CLOUDSTACK-8387 if anyone can
> take
> > > it.
> > >
> > > On Wed, Apr 15, 2015 at 12:17 AM, Marcus  wrote:
> > > > I'm opening a bug, should I assign it to you, or do you know of
> > > > someone who maintains this code? It is probably pretty urgent to go
> > > > out in the next 4.5 bugfix
> > > >
> > > >
> > > > On Wed, Apr 15, 2015 at 12:11 AM, Marcus 
> wrote:
> > > >> Yeah, I backed out the change in my test environment and the issue
> > went
> > > away.
> > > >>
> > > >> On Wed, Apr 15, 2015 at 12:07 AM, Bharat Kumar <
> > bharat.ku...@citrix.com>
> > > wrote:
> > > >>> Hi,
> > > >>>
> > > >>> I guss you are correct. we need to add the newly created channel
> ch1
> > > to the map after the handshake is complete.
> > > >>>
> > > >>> _peers.put(peerName, ch) should be changed to _peers.put(peerName,
> > > ch1);
> > > >>>
> > > >>> and also return ch1 instead of ch.
> > > >>>
> > > >>> Thanks,
> > > >>> Bharat.
> > > >>>
> > > >>>
> > > >>> On 15-Apr-2015, at 12:21 pm, Marcus  wrote:
> > > >>>
> > >  Is anyone running 4.5 mgmt server in cluster setup? I've run into
> an
> > >  issue where the channel between the two servers is null, and I
> think
> > >  I've tracked it down to "coverity fixes" from
> > >  78bfaa79cf3eaa170ef9422bf8fb1825c0cecfc1.
> > > 
> > >  The method returns 'ch', which is null in the error. The coverity
> > fix
> > >  changes ch for ch1 and seems to initialize ch1 and then drop it on
> > the
> > >  floor. If someone is familiar with this code I'd appreciate a
> review
> > >  of it.
> > > 
> > >  diff --git
> > >
> >
> a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
> > > 
> > >
> >
> b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
> > >  index 600dca2..e38489b 100755
> > >  ---
> > >
> >
> a/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
> > >  +++
> > >
> >
> b/engine/orchestration/src/com/cloud/agent/manager/ClusteredAgentManagerImpl.java
> > >  @@ -498,18 +498,17 @@ public class ClusteredAgentManagerImpl
> extends
> > >  AgentManagerImpl implements Clust
> > >  } catch (UnknownHostException e) {
> > >  throw new CloudRuntimeException("Unable to
> > > resolve " + ip);
> > >  }
> > >  -try {
> > >  -ch = SocketChannel.open(new
> > >  InetSocketAddress(addr, Port.value()));
> > >  -ch.configureBlocking(true); // make sure we
> are
> > >  working at blocking mode
> > >  -ch.socket().setKeepAlive(true);
> > >  -ch.socket().setSoTimeout(60 * 1000);
> > >  +try (SocketChannel ch1 = SocketChannel.open(new
> > >  InetSocketAddress(addr, Port.value()));){
> > >  +ch1.configureBlocking(true); // make sure we
> > are
> > >  working at blocking mode
> > >  +ch1.socket().setKeepAlive(true);
> > >  +ch1.socket().setSoTimeout(60 * 1000);
> > >  try {
> > >  SSLContext sslContext =
> > > Link.initSSLContext(true);
> > >  sslEngine = sslContext.createSSLEngine(ip,
> > >  Port.value());
> > >  sslEngine.setUseClientMode(true);
> > > 
> > > 
> > >
> >
> sslEngine.setEnabledProtocols(SSLUtils.getSupportedProtocols(sslEngine.getEnabledProtocols()));
> > > 
> > >  -Link.doHandshake(ch, sslEngine, true);
> > >  +Link.doHandshake(ch1, sslEngine, true);
> > >  s_logger.info("SSL: Handshake done");
> > >  } catch (Exception e) {
> > >  throw new IOException("SSL: Fail to init
> > SSL!
> > > " + e);
> > > >>>
>

Re: Inconsistencies in tests

[Gaurav Aradhye]

Thanks Talluri, I will catch up with you on this.

Regards,
Gaurav Aradhye

On Apr 15, 2015, at 12:37 PM, Srikanteswararao Talluri 
 wrote:

> I will help you set the job up.
> 
> On 15/04/15 12:34 pm, "Gaurav Aradhye"  wrote:
> 
>> Talluri, I still have to do it. Didn’t get enough time on it. Not a
>> jenkins expert.
>> Can anybody help me in setting jenkins job for tests which will do static
>> code analysis (python) and pep8 analysis?
>> 
>> Regards,
>> Gaurav Aradhye
>> 
>> On Apr 15, 2015, at 12:28 PM, Srikanteswararao Talluri
>>  wrote:
>> 
>>> +1
>>> 
>>> Gaurav,
>>> 
>>> You were talking about integrating static code analysis in
>>> jenkins.b.a.c.o. How is it going?
>>> 
>>> Thanks,
>>> ~Talluri
>>> 
>>> On 14/04/15 3:50 pm, "Sebastien Goasguen"  wrote:
>>> 
 
> On Apr 14, 2015, at 9:06 AM, Gaurav Aradhye
>  wrote:
> 
> Hi all,
> 
> I feel that the test code in BVT and Regression tests (smoke and
> component folders) should be more consistent. There are feature
> developers adding tests into smoke and different test developers
> adding
> more tests into component folder, it is evident that inconsistencies
> are
> bound to arise. But with time, we should be striving to minimize them.
> 
> I have listed few of the inconsistencies below. Feel free to add to
> the
> list if you think of more.
> 
> 1. List Methods:
> 
> If we want to list accounts, some tests use Account.list method, while
> some use list_accounts method. Both methods finally do the same thing,
> but there are many list methods in common.py file corresponding base
> class methods in base.py. This is code duplication.
> 
> I feel that BaseClass.method() is more readable than list_xyz(), or
> should be preferred.
> 
> 
> 2. Checking empty/None lists:
> 
> We already have validateList utility method which checks both empty
> lists and None objects. No need to use ³isinstance² method, or add
> additional None checks.
> 
> 
> 3. Pep8 issues:
> Many files are not pep8 consistent. In the past I have tried to fix
> pep8 issues in files and the mission is ON. Efforts need to be put
> into
> this. We already have autopep8 tool which does most of the things.
> 
 
 +1
 
 that¹s really a low hanging fruit and best practice. we should not be
 merging anything that breaks pep8
 we should also add pylint tests
 
 checking for pep8 and python of all python code can be done at the high
 level using something like tox, kind of like the RAT tests
 
> 
> I will be creating separate issues in JIRA for listed things. Everyone
> is welcome to add pull requests for these tasks.
> I personally will be putting more efforts into these in upcoming few
> months. 2-3 months and we should see most of the tests consistent with
> each other.
> 
> Regards,
> Gaurav
>> 
> 

[GitHub] cloudstack pull request: CLOUDSTACK-8335: use libvirt from central...

[K0zka]

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

https://github.com/apache/cloudstack/pull/163#discussion_r28400164
  
--- Diff: plugins/hypervisors/kvm/pom.xml ---
@@ -45,7 +38,7 @@
   ${project.version}
 
 
-  org.libvirt
+  com.github.K0zka
--- End diff --

@kishankavala yes that is possible. I have to see how that could work 
together with the apache infrastructure, but I will be working on it. I will 
close this PR for now.


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

[GitHub] cloudstack pull request: CLOUDSTACK-8335: use libvirt from central...

[K0zka]

Github user K0zka closed the pull request at:

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


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

listUsageRecord only for Admins? Why?

[Ramzi Youssefi]

Hello developers!

is there any reason why listUsageRecords api call is only limited to root
and admin users?
I think this use case is very common and it would be nice if clients can
see their usage records in their projects.
Is there any workaround for that?

Best,

-- 
Ramzi Youssefi
Görlitzer Str. 49
01099 Dresden, Germany.
Cell: +49 173 1965465
Email: ryouss...@gmail.com

AW: Missing storage label in XenCenter after migrate

[S . Brüseke - proIO GmbH]

Cannot confirm this! Are you talking about XenServer "name-label" or CS?
 
Mit freundlichen Grüßen / With kind regards,
 
Swen Brüseke
 
 
Von: raja sekhar [mailto:rajsekhar@gmail.com] 
Gesendet: Mittwoch, 15. April 2015 09:43
An: S. Brüseke - proIO GmbH
Cc: dev@cloudstack.apache.org
Betreff: Re: Missing storage label in XenCenter after migrate
 
I have tested in ACS 4.3.1 with Xenserver 6.2 as hyperviser.
When vm is down the name label remains same,but when the vm is live name label 
changes.
 
On Wed, Apr 15, 2015 at 1:01 PM, S. Brüseke - proIO GmbH  
wrote:
Hi,
 
thank you for testing. In 4.3 the name label is empty even when the vm is down.
 
Mit freundlichen Grüßen / With kind regards,
 
Swen Brüseke
 
Von: raja sekhar [mailto:rajsekhar@gmail.com] 
Gesendet: Dienstag, 14. April 2015 18:48
An: dev@cloudstack.apache.org; S. Brüseke - proIO GmbH
Betreff: Re: Missing storage label in XenCenter after migrate
 
Bruseke,
 
When the disks are live migrated from one primary to another this problem is 
arising, i'am also getting same problem, the disk labels are changed to 
"Cloud-".it seems to be a bug.
how ever it doesn't happens when the vm is down.
 
On Tue, Apr 14, 2015 at 6:11 PM, S. Brüseke - proIO GmbH  
wrote:
Hi,

can somebody please test this behavior in 4.5.1-rc2?

1. Create instance on XenServer hypervisor via CS.
2. Check "Name" of storage in XenCenter's storage tab for this instance. 
(should be ROOT-XXX)
3. Migrate primary storage to different storage via CS.
4. Take a look at "Name" of storage in XenCenter's storage tab again. (It is 
empty now)

If this is the case on your installation I consider this a bug. Can please 
someone verify?
Thank you very much!

Mit freundlichen Grüßen / With kind regards,

Swen Brüseke




- proIO GmbH -
Geschäftsführer: Swen Brüseke
Sitz der Gesellschaft: Frankfurt am Main

USt-IdNr. DE 267 075 918
Registergericht: Frankfurt am Main - HRB 86239

Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte Informationen.
Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich erhalten 
haben,
informieren Sie bitte sofort den Absender und vernichten Sie diese Mail.
Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieser Mail sind nicht 
gestattet.

This e-mail may contain confidential and/or privileged information.
If you are not the intended recipient (or have received this e-mail in error) 
please notify
the sender immediately and destroy this e-mail.
Any unauthorized copying, disclosure or distribution of the material in this 
e-mail is strictly forbidden.
 
 


- proIO GmbH -
Geschäftsführer: Swen Brüseke
Sitz der Gesellschaft: Frankfurt am Main

USt-IdNr. DE 267 075 918
Registergericht: Frankfurt am Main - HRB 86239

Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte Informationen. 
Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich erhalten 
haben, 
informieren Sie bitte sofort den Absender und vernichten Sie diese Mail. 
Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieser Mail sind nicht 
gestattet. 

This e-mail may contain confidential and/or privileged information. 
If you are not the intended recipient (or have received this e-mail in error) 
please notify 
the sender immediately and destroy this e-mail.  
Any unauthorized copying, disclosure or distribution of the material in this 
e-mail is strictly forbidden.


- proIO GmbH -
Geschäftsführer: Swen Brüseke
Sitz der Gesellschaft: Frankfurt am Main

USt-IdNr. DE 267 075 918
Registergericht: Frankfurt am Main - HRB 86239

Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte Informationen. 
Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich erhalten 
haben, 
informieren Sie bitte sofort den Absender und vernichten Sie diese Mail. 
Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieser Mail sind nicht 
gestattet. 

This e-mail may contain confidential and/or privileged information. 
If you are not the intended recipient (or have received this e-mail in error) 
please notify 
the sender immediately and destroy this e-mail.  
Any unauthorized copying, disclosure or distribution of the material in this 
e-mail is strictly forbidden. 

searching for a cloudstack developer

[S . Brüseke - proIO GmbH]

Hi,

we are searching for a cloudstack developer! We found and reported some bugs in 
CS, which are really annoying for our workflow, but  minor to the opensource 
project.
The problem is that we do not have the knowlegde and manpower to solve these 
bugs, but we are willing to pay for a solution and provide the code to the 
opensource project.

Please contact me, if you need more information.

Mit freundlichen Grüßen / With kind regards,

Swen Brüseke



- proIO GmbH -
Geschäftsführer: Swen Brüseke
Sitz der Gesellschaft: Frankfurt am Main

USt-IdNr. DE 267 075 918
Registergericht: Frankfurt am Main - HRB 86239

Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte Informationen. 
Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich erhalten 
haben, 
informieren Sie bitte sofort den Absender und vernichten Sie diese Mail. 
Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieser Mail sind nicht 
gestattet. 

This e-mail may contain confidential and/or privileged information. 
If you are not the intended recipient (or have received this e-mail in error) 
please notify 
the sender immediately and destroy this e-mail.  
Any unauthorized copying, disclosure or distribution of the material in this 
e-mail is strictly forbidden. 

Jenkins build is still unstable: simulator-4.5-singlerun #204

[jenkins]

See 

List of freelance developers (Was: Re: searching for a cloudstack developer)

[Erik Weber]

On Wed, Apr 15, 2015 at 10:05 AM, S. Brüseke - proIO GmbH <
s.brues...@proio.com> wrote:

> Hi,
>
> we are searching for a cloudstack developer! We found and reported some
> bugs in CS, which are really annoying for our workflow, but  minor to the
> opensource project.
> The problem is that we do not have the knowlegde and manpower to solve
> these bugs, but we are willing to pay for a solution and provide the code
> to the opensource project.
>
> Please contact me, if you need more information.
>
>
Renaming the thread for my input.

We also face some issues from time to time that is of low importance to the
project.

Would it make sense, and be possible, to compile a list of people /
companies / other resources that are willing to do per case development?

-- 
Erik

Re: List of freelance developers (Was: Re: searching for a cloudstack developer)

[Sebastien Goasguen]

> On Apr 15, 2015, at 10:20 AM, Erik Weber  wrote:
> 
> On Wed, Apr 15, 2015 at 10:05 AM, S. Brüseke - proIO GmbH <
> s.brues...@proio.com> wrote:
> 
>> Hi,
>> 
>> we are searching for a cloudstack developer! We found and reported some
>> bugs in CS, which are really annoying for our workflow, but  minor to the
>> opensource project.
>> The problem is that we do not have the knowlegde and manpower to solve
>> these bugs, but we are willing to pay for a solution and provide the code
>> to the opensource project.
>> 
>> Please contact me, if you need more information.
>> 
>> 
> Renaming the thread for my input.
> 
> We also face some issues from time to time that is of low importance to the
> project.
> 
> Would it make sense, and be possible, to compile a list of people /
> companies / other resources that are willing to do per case development?
> 

I think you could definitely start a wiki page and list folks there.


> -- 
> Erik

RE: List of freelance developers (Was: Re: searching for a cloudstack developer)

[Giles Sirett]

[shameless commercial plug] ShapeBlue are currently doing a lot of this type of 
work for customers, mainly new feature development - happy to help. [/shameless 
commercial  plug]


I would welcome a wiki page for this and people providing  other services 
around ACS (of which there is a growing number)


Kind Regards
Giles

D: +44 20 3603 0541 | M: +44 796 111 2055
giles.sir...@shapeblue.com




> -Original Message-
> From: Sebastien Goasguen [mailto:run...@gmail.com]
> Sent: 15 April 2015 09:45
> To: dev@cloudstack.apache.org
> Subject: Re: List of freelance developers (Was: Re: searching for a cloudstack
> developer)
>
>
> > On Apr 15, 2015, at 10:20 AM, Erik Weber  wrote:
> >
> > On Wed, Apr 15, 2015 at 10:05 AM, S. Brüseke - proIO GmbH <
> > s.brues...@proio.com> wrote:
> >
> >> Hi,
> >>
> >> we are searching for a cloudstack developer! We found and reported
> >> some bugs in CS, which are really annoying for our workflow, but
> >> minor to the opensource project.
> >> The problem is that we do not have the knowlegde and manpower to
> >> solve these bugs, but we are willing to pay for a solution and
> >> provide the code to the opensource project.
> >>
> >> Please contact me, if you need more information.
> >>
> >>
> > Renaming the thread for my input.
> >
> > We also face some issues from time to time that is of low importance
> > to the project.
> >
> > Would it make sense, and be possible, to compile a list of people /
> > companies / other resources that are willing to do per case development?
> >
>
> I think you could definitely start a wiki page and list folks there.
>
>
> > --
> > Erik

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.

AW: List of freelance developers (Was: Re: searching for a cloudstack developer)

[S . Brüseke - proIO GmbH]

That is a great idea! Who can create this page?

Mit freundlichen Grüßen / With kind regards,

Swen Brüseke

-Ursprüngliche Nachricht-
Von: Sebastien Goasguen [mailto:run...@gmail.com] 
Gesendet: Mittwoch, 15. April 2015 10:45
An: dev@cloudstack.apache.org
Betreff: Re: List of freelance developers (Was: Re: searching for a cloudstack 
developer)


> On Apr 15, 2015, at 10:20 AM, Erik Weber  wrote:
> 
> On Wed, Apr 15, 2015 at 10:05 AM, S. Brüseke - proIO GmbH < 
> s.brues...@proio.com> wrote:
> 
>> Hi,
>> 
>> we are searching for a cloudstack developer! We found and reported 
>> some bugs in CS, which are really annoying for our workflow, but  
>> minor to the opensource project.
>> The problem is that we do not have the knowlegde and manpower to 
>> solve these bugs, but we are willing to pay for a solution and 
>> provide the code to the opensource project.
>> 
>> Please contact me, if you need more information.
>> 
>> 
> Renaming the thread for my input.
> 
> We also face some issues from time to time that is of low importance 
> to the project.
> 
> Would it make sense, and be possible, to compile a list of people / 
> companies / other resources that are willing to do per case development?
> 

I think you could definitely start a wiki page and list folks there.


> --
> Erik




- proIO GmbH -
Geschäftsführer: Swen Brüseke
Sitz der Gesellschaft: Frankfurt am Main

USt-IdNr. DE 267 075 918
Registergericht: Frankfurt am Main - HRB 86239

Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte Informationen. 
Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich erhalten 
haben, 
informieren Sie bitte sofort den Absender und vernichten Sie diese Mail. 
Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieser Mail sind nicht 
gestattet. 

This e-mail may contain confidential and/or privileged information. 
If you are not the intended recipient (or have received this e-mail in error) 
please notify 
the sender immediately and destroy this e-mail.  
Any unauthorized copying, disclosure or distribution of the material in this 
e-mail is strictly forbidden. 

[GitHub] cloudstack pull request: CLOUDSTACK-8335: use libvirt from central...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/163#issuecomment-93274204
  
Hi @K0zka ,

I have to fix a bug on 4.4.2 and when trying to build from source got an 
issue with the old libvirt groupid: could not simple download it. Thus could 
not build.

[ERROR] Failed to execute goal on project cloud-plugin-hypervisor-kvm: 
Could not resolve dependencies for project 
org.apache.cloudstack:cloud-plugin-hypervisor-kvm:jar:4.4.2: Failed to collect 
dependencies at org.libvirt:libvirt:jar:0.5.1: Failed to read artifact 
descriptor for org.libvirt:libvirt:jar:0.5.1

After changing the groupid for the one you added, it worked just fine. is 
that okay if we get this change back ported to 4.3/4.4/4.5, at least?

I can also do that, no problem.

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

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-testpath

--Adding Snapshot Test cases that are relating secondary
  storage limits and snapshot limits

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-testpath-testpath-snapshot-limits

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

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


commit 38b4f5585279cb1b2ae0227ec5fdaeeefbaf19c7
Author: pritisarap12 
Date:   2015-04-15T09:19:46Z

CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-testpath
--Adding Snapshot Test cases that are relating secondary
  storage limits and snapshot limits




---
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: listUsageRecord only for Admins? Why?

[Koushik Das]

Agree. I don't think it's possible currently. The listUsageRecords needs to be 
modified. 

-Original Message-
From: Ramzi Youssefi [mailto:ryouss...@gmail.com] 
Sent: Wednesday, 15 April 2015 13:28
To: dev
Subject: listUsageRecord only for Admins? Why?

Hello developers!

is there any reason why listUsageRecords api call is only limited to root and 
admin users?
I think this use case is very common and it would be nice if clients can see 
their usage records in their projects.
Is there any workaround for that?

Best,

--
Ramzi Youssefi
Görlitzer Str. 49
01099 Dresden, Germany.
Cell: +49 173 1965465
Email: ryouss...@gmail.com

[GitHub] cloudstack pull request: CLOUDSTACK-8308-Adding-automation-test-ca...

[gauravaradhye]

Github user gauravaradhye commented on the pull request:

https://github.com/apache/cloudstack/pull/170#issuecomment-93288143
  
Please include test results.


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

[GitHub] cloudstack pull request: CLOUDSTACK-8308-Adding-automation-test-ca...

[gauravaradhye]

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

https://github.com/apache/cloudstack/pull/170#discussion_r28406940
  
--- Diff: test/integration/testpaths/testpath_snapshot_limits.py ---
@@ -0,0 +1,355 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+""" Test cases for VM/Volume snapshot Test Path
--- End diff --

Modify comment to specify what we are targeting to test. In this case 
snapshot limits


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

[GitHub] cloudstack pull request: CLOUDSTACK-8308-Adding-automation-test-ca...

[gauravaradhye]

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

https://github.com/apache/cloudstack/pull/170#discussion_r28407016
  
--- Diff: test/integration/testpaths/testpath_snapshot_limits.py ---
@@ -0,0 +1,355 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+""" Test cases for VM/Volume snapshot Test Path
+"""
+
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources)
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ DiskOffering,
+ Volume,
+ Resources,
+ VirtualMachine,
+ Snapshot
+ )
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template
+   )
+
+
+class TestStorageSnapshotsLimits(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestStorageSnapshotsLimits, 
cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata["ostype"])
+
+cls._cleanup = []
+
+try:
+
+# Create an account
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata["account"],
+domainid=cls.domain.id
+)
+cls._cleanup.append(cls.account)
+
+# Create user api client of the account
+cls.userapiclient = testClient.getUserApiClient(
+UserName=cls.account.name,
+DomainName=cls.account.domain
+)
+
+# Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata["service_offering"],
+)
+cls._cleanup.append(cls.service_offering)
+
+cls.disk_offering = DiskOffering.create(
+cls.apiclient,
+cls.testdata["disk_offering"],
+)
+
+cls._cleanup.append(cls.disk_offering)
+
+cls.vm = VirtualMachine.create(
+cls.userapiclient,
+cls.testdata["small"],
+templateid=cls.template.id,
+accountid=cls.account.name,
+domainid=cls.account.domainid,
+serviceofferingid=cls.service_offering.id,
+zoneid=cls.zone.id,
+mode=cls.zone.networktype
+)
+
+except Exception as e:
+cls.tearDownClass()
+raise e
+return
+
+@classmethod
+def tearDownClass(cls):
+try:
+cleanup_resources(cls.apiclient, cls._cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+
+def setUp(self):
+self.apiclient = self.testClient.getApiClient()
+self.dbclient = self.testClient.getDbConnection()
+self.cleanup = []
+
+def tearDown(self):
+try:
+cleanup_resources(self.apiclient, self.cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+return
+

[GitHub] cloudstack pull request: CLOUDSTACK-8308-Adding-automation-test-ca...

[gauravaradhye]

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

https://github.com/apache/cloudstack/pull/170#discussion_r28407125
  
--- Diff: test/integration/testpaths/testpath_snapshot_limits.py ---
@@ -0,0 +1,355 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+""" Test cases for VM/Volume snapshot Test Path
+"""
+
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources)
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ DiskOffering,
+ Volume,
+ Resources,
+ VirtualMachine,
+ Snapshot
+ )
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template
+   )
+
+
+class TestStorageSnapshotsLimits(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestStorageSnapshotsLimits, 
cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata["ostype"])
+
+cls._cleanup = []
+
+try:
+
+# Create an account
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata["account"],
+domainid=cls.domain.id
+)
+cls._cleanup.append(cls.account)
+
+# Create user api client of the account
+cls.userapiclient = testClient.getUserApiClient(
+UserName=cls.account.name,
+DomainName=cls.account.domain
+)
+
+# Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata["service_offering"],
+)
+cls._cleanup.append(cls.service_offering)
+
+cls.disk_offering = DiskOffering.create(
+cls.apiclient,
+cls.testdata["disk_offering"],
+)
+
+cls._cleanup.append(cls.disk_offering)
+
+cls.vm = VirtualMachine.create(
+cls.userapiclient,
+cls.testdata["small"],
+templateid=cls.template.id,
+accountid=cls.account.name,
+domainid=cls.account.domainid,
+serviceofferingid=cls.service_offering.id,
+zoneid=cls.zone.id,
+mode=cls.zone.networktype
+)
+
+except Exception as e:
+cls.tearDownClass()
+raise e
+return
+
+@classmethod
+def tearDownClass(cls):
+try:
+cleanup_resources(cls.apiclient, cls._cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+
+def setUp(self):
+self.apiclient = self.testClient.getApiClient()
+self.dbclient = self.testClient.getDbConnection()
+self.cleanup = []
+
+def tearDown(self):
+try:
+cleanup_resources(self.apiclient, self.cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+return
+

[GitHub] cloudstack pull request: CLOUDSTACK-8308-Adding-automation-test-ca...

[gauravaradhye]

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

https://github.com/apache/cloudstack/pull/170#discussion_r28407152
  
--- Diff: test/integration/testpaths/testpath_snapshot_limits.py ---
@@ -0,0 +1,355 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+""" Test cases for VM/Volume snapshot Test Path
+"""
+
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources)
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ DiskOffering,
+ Volume,
+ Resources,
+ VirtualMachine,
+ Snapshot
+ )
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template
+   )
+
+
+class TestStorageSnapshotsLimits(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestStorageSnapshotsLimits, 
cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata["ostype"])
+
+cls._cleanup = []
+
+try:
+
+# Create an account
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata["account"],
+domainid=cls.domain.id
+)
+cls._cleanup.append(cls.account)
+
+# Create user api client of the account
+cls.userapiclient = testClient.getUserApiClient(
+UserName=cls.account.name,
+DomainName=cls.account.domain
+)
+
+# Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata["service_offering"],
+)
+cls._cleanup.append(cls.service_offering)
+
+cls.disk_offering = DiskOffering.create(
+cls.apiclient,
+cls.testdata["disk_offering"],
+)
+
+cls._cleanup.append(cls.disk_offering)
+
+cls.vm = VirtualMachine.create(
+cls.userapiclient,
+cls.testdata["small"],
+templateid=cls.template.id,
+accountid=cls.account.name,
+domainid=cls.account.domainid,
+serviceofferingid=cls.service_offering.id,
+zoneid=cls.zone.id,
+mode=cls.zone.networktype
+)
+
+except Exception as e:
+cls.tearDownClass()
+raise e
+return
+
+@classmethod
+def tearDownClass(cls):
+try:
+cleanup_resources(cls.apiclient, cls._cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+
+def setUp(self):
+self.apiclient = self.testClient.getApiClient()
+self.dbclient = self.testClient.getDbConnection()
+self.cleanup = []
+
+def tearDown(self):
+try:
+cleanup_resources(self.apiclient, self.cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+return
+

[GitHub] cloudstack pull request: CLOUDSTACK-8308-Adding-automation-test-ca...

[gauravaradhye]

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

https://github.com/apache/cloudstack/pull/170#discussion_r28407187
  
--- Diff: test/integration/testpaths/testpath_snapshot_limits.py ---
@@ -0,0 +1,355 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+""" Test cases for VM/Volume snapshot Test Path
+"""
+
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources)
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ DiskOffering,
+ Volume,
+ Resources,
+ VirtualMachine,
+ Snapshot
+ )
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template
+   )
+
+
+class TestStorageSnapshotsLimits(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestStorageSnapshotsLimits, 
cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata["ostype"])
+
+cls._cleanup = []
+
+try:
+
+# Create an account
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata["account"],
+domainid=cls.domain.id
+)
+cls._cleanup.append(cls.account)
+
+# Create user api client of the account
+cls.userapiclient = testClient.getUserApiClient(
+UserName=cls.account.name,
+DomainName=cls.account.domain
+)
+
+# Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata["service_offering"],
+)
+cls._cleanup.append(cls.service_offering)
+
+cls.disk_offering = DiskOffering.create(
+cls.apiclient,
+cls.testdata["disk_offering"],
+)
+
+cls._cleanup.append(cls.disk_offering)
+
+cls.vm = VirtualMachine.create(
+cls.userapiclient,
+cls.testdata["small"],
+templateid=cls.template.id,
+accountid=cls.account.name,
+domainid=cls.account.domainid,
+serviceofferingid=cls.service_offering.id,
+zoneid=cls.zone.id,
+mode=cls.zone.networktype
+)
+
+except Exception as e:
+cls.tearDownClass()
+raise e
+return
+
+@classmethod
+def tearDownClass(cls):
+try:
+cleanup_resources(cls.apiclient, cls._cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+
+def setUp(self):
+self.apiclient = self.testClient.getApiClient()
+self.dbclient = self.testClient.getDbConnection()
+self.cleanup = []
+
+def tearDown(self):
+try:
+cleanup_resources(self.apiclient, self.cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+return
+

[GitHub] cloudstack pull request: CLOUDSTACK-8308-Adding-automation-test-ca...

[gauravaradhye]

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

https://github.com/apache/cloudstack/pull/170#discussion_r28407255
  
--- Diff: test/integration/testpaths/testpath_snapshot_limits.py ---
@@ -0,0 +1,355 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+""" Test cases for VM/Volume snapshot Test Path
+"""
+
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources)
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ DiskOffering,
+ Volume,
+ Resources,
+ VirtualMachine,
+ Snapshot
+ )
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template
+   )
+
+
+class TestStorageSnapshotsLimits(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestStorageSnapshotsLimits, 
cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata["ostype"])
+
+cls._cleanup = []
+
+try:
+
+# Create an account
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata["account"],
+domainid=cls.domain.id
+)
+cls._cleanup.append(cls.account)
+
+# Create user api client of the account
+cls.userapiclient = testClient.getUserApiClient(
+UserName=cls.account.name,
+DomainName=cls.account.domain
+)
+
+# Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata["service_offering"],
+)
+cls._cleanup.append(cls.service_offering)
+
+cls.disk_offering = DiskOffering.create(
+cls.apiclient,
+cls.testdata["disk_offering"],
+)
+
+cls._cleanup.append(cls.disk_offering)
+
+cls.vm = VirtualMachine.create(
+cls.userapiclient,
+cls.testdata["small"],
+templateid=cls.template.id,
+accountid=cls.account.name,
+domainid=cls.account.domainid,
+serviceofferingid=cls.service_offering.id,
+zoneid=cls.zone.id,
+mode=cls.zone.networktype
+)
+
+except Exception as e:
+cls.tearDownClass()
+raise e
+return
+
+@classmethod
+def tearDownClass(cls):
+try:
+cleanup_resources(cls.apiclient, cls._cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+
+def setUp(self):
+self.apiclient = self.testClient.getApiClient()
+self.dbclient = self.testClient.getDbConnection()
+self.cleanup = []
+
+def tearDown(self):
+try:
+cleanup_resources(self.apiclient, self.cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+return
+

[GitHub] cloudstack pull request: CLOUDSTACK-8308-Adding-automation-test-ca...

[gauravaradhye]

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

https://github.com/apache/cloudstack/pull/170#discussion_r28407300
  
--- Diff: test/integration/testpaths/testpath_snapshot_limits.py ---
@@ -0,0 +1,355 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+""" Test cases for VM/Volume snapshot Test Path
+"""
+
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources)
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ DiskOffering,
+ Volume,
+ Resources,
+ VirtualMachine,
+ Snapshot
+ )
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template
+   )
+
+
+class TestStorageSnapshotsLimits(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestStorageSnapshotsLimits, 
cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata["ostype"])
+
+cls._cleanup = []
+
+try:
+
+# Create an account
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata["account"],
+domainid=cls.domain.id
+)
+cls._cleanup.append(cls.account)
+
+# Create user api client of the account
+cls.userapiclient = testClient.getUserApiClient(
+UserName=cls.account.name,
+DomainName=cls.account.domain
+)
+
+# Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata["service_offering"],
+)
+cls._cleanup.append(cls.service_offering)
+
+cls.disk_offering = DiskOffering.create(
+cls.apiclient,
+cls.testdata["disk_offering"],
+)
+
+cls._cleanup.append(cls.disk_offering)
+
+cls.vm = VirtualMachine.create(
+cls.userapiclient,
+cls.testdata["small"],
+templateid=cls.template.id,
+accountid=cls.account.name,
+domainid=cls.account.domainid,
+serviceofferingid=cls.service_offering.id,
+zoneid=cls.zone.id,
+mode=cls.zone.networktype
+)
+
+except Exception as e:
+cls.tearDownClass()
+raise e
+return
+
+@classmethod
+def tearDownClass(cls):
+try:
+cleanup_resources(cls.apiclient, cls._cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+
+def setUp(self):
+self.apiclient = self.testClient.getApiClient()
+self.dbclient = self.testClient.getDbConnection()
+self.cleanup = []
+
+def tearDown(self):
+try:
+cleanup_resources(self.apiclient, self.cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+return
+

[GitHub] cloudstack pull request: CLOUDSTACK-8308-Adding-automation-test-ca...

[gauravaradhye]

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

https://github.com/apache/cloudstack/pull/170#discussion_r28407289
  
--- Diff: test/integration/testpaths/testpath_snapshot_limits.py ---
@@ -0,0 +1,355 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+""" Test cases for VM/Volume snapshot Test Path
+"""
+
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources)
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ DiskOffering,
+ Volume,
+ Resources,
+ VirtualMachine,
+ Snapshot
+ )
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template
+   )
+
+
+class TestStorageSnapshotsLimits(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestStorageSnapshotsLimits, 
cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata["ostype"])
+
+cls._cleanup = []
+
+try:
+
+# Create an account
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata["account"],
+domainid=cls.domain.id
+)
+cls._cleanup.append(cls.account)
+
+# Create user api client of the account
+cls.userapiclient = testClient.getUserApiClient(
+UserName=cls.account.name,
+DomainName=cls.account.domain
+)
+
+# Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata["service_offering"],
+)
+cls._cleanup.append(cls.service_offering)
+
+cls.disk_offering = DiskOffering.create(
+cls.apiclient,
+cls.testdata["disk_offering"],
+)
+
+cls._cleanup.append(cls.disk_offering)
+
+cls.vm = VirtualMachine.create(
+cls.userapiclient,
+cls.testdata["small"],
+templateid=cls.template.id,
+accountid=cls.account.name,
+domainid=cls.account.domainid,
+serviceofferingid=cls.service_offering.id,
+zoneid=cls.zone.id,
+mode=cls.zone.networktype
+)
+
+except Exception as e:
+cls.tearDownClass()
+raise e
+return
+
+@classmethod
+def tearDownClass(cls):
+try:
+cleanup_resources(cls.apiclient, cls._cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+
+def setUp(self):
+self.apiclient = self.testClient.getApiClient()
+self.dbclient = self.testClient.getDbConnection()
+self.cleanup = []
+
+def tearDown(self):
+try:
+cleanup_resources(self.apiclient, self.cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+return
+

[GitHub] cloudstack pull request: CLOUDSTACK-8308-Adding-automation-test-ca...

[gauravaradhye]

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

https://github.com/apache/cloudstack/pull/170#discussion_r28407426
  
--- Diff: test/integration/testpaths/testpath_snapshot_limits.py ---
@@ -0,0 +1,355 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+""" Test cases for VM/Volume snapshot Test Path
+"""
+
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources)
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ DiskOffering,
+ Volume,
+ Resources,
+ VirtualMachine,
+ Snapshot
+ )
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template
+   )
+
+
+class TestStorageSnapshotsLimits(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestStorageSnapshotsLimits, 
cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata["ostype"])
+
+cls._cleanup = []
+
+try:
+
+# Create an account
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata["account"],
+domainid=cls.domain.id
+)
+cls._cleanup.append(cls.account)
+
+# Create user api client of the account
+cls.userapiclient = testClient.getUserApiClient(
+UserName=cls.account.name,
+DomainName=cls.account.domain
+)
+
+# Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata["service_offering"],
+)
+cls._cleanup.append(cls.service_offering)
+
+cls.disk_offering = DiskOffering.create(
+cls.apiclient,
+cls.testdata["disk_offering"],
+)
+
+cls._cleanup.append(cls.disk_offering)
+
+cls.vm = VirtualMachine.create(
+cls.userapiclient,
+cls.testdata["small"],
+templateid=cls.template.id,
+accountid=cls.account.name,
+domainid=cls.account.domainid,
+serviceofferingid=cls.service_offering.id,
+zoneid=cls.zone.id,
+mode=cls.zone.networktype
--- End diff --

Pass "mode" parameter only in case when you want to SSH to VM. If you are 
not SSHing, remove it. It will unnecessarily acquire public IP in the network 
and create network rules on it to make VM reachable through SSH.


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

[GitHub] cloudstack pull request: CLOUDSTACK-8308-Adding-automation-test-ca...

[gauravaradhye]

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

https://github.com/apache/cloudstack/pull/170#discussion_r28407493
  
--- Diff: test/integration/testpaths/testpath_snapshot_limits.py ---
@@ -0,0 +1,355 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+""" Test cases for VM/Volume snapshot Test Path
+"""
+
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase
+from marvin.lib.utils import (cleanup_resources)
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ DiskOffering,
+ Volume,
+ Resources,
+ VirtualMachine,
+ Snapshot
+ )
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template
+   )
+
+
+class TestStorageSnapshotsLimits(cloudstackTestCase):
+
+@classmethod
+def setUpClass(cls):
+testClient = super(TestStorageSnapshotsLimits, 
cls).getClsTestClient()
+cls.apiclient = testClient.getApiClient()
+cls.testdata = testClient.getParsedTestDataConfig()
+cls.hypervisor = cls.testClient.getHypervisorInfo()
+
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.apiclient)
+cls.zone = get_zone(cls.apiclient, testClient.getZoneForTests())
+
+cls.template = get_template(
+cls.apiclient,
+cls.zone.id,
+cls.testdata["ostype"])
+
+cls._cleanup = []
+
+try:
+
+# Create an account
+cls.account = Account.create(
+cls.apiclient,
+cls.testdata["account"],
+domainid=cls.domain.id
+)
+cls._cleanup.append(cls.account)
+
+# Create user api client of the account
+cls.userapiclient = testClient.getUserApiClient(
+UserName=cls.account.name,
+DomainName=cls.account.domain
+)
+
+# Create Service offering
+cls.service_offering = ServiceOffering.create(
+cls.apiclient,
+cls.testdata["service_offering"],
+)
+cls._cleanup.append(cls.service_offering)
+
+cls.disk_offering = DiskOffering.create(
+cls.apiclient,
+cls.testdata["disk_offering"],
+)
+
+cls._cleanup.append(cls.disk_offering)
+
+cls.vm = VirtualMachine.create(
+cls.userapiclient,
+cls.testdata["small"],
+templateid=cls.template.id,
+accountid=cls.account.name,
+domainid=cls.account.domainid,
+serviceofferingid=cls.service_offering.id,
+zoneid=cls.zone.id,
+mode=cls.zone.networktype
+)
+
+except Exception as e:
+cls.tearDownClass()
+raise e
+return
+
+@classmethod
+def tearDownClass(cls):
+try:
+cleanup_resources(cls.apiclient, cls._cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+
+def setUp(self):
+self.apiclient = self.testClient.getApiClient()
+self.dbclient = self.testClient.getDbConnection()
+self.cleanup = []
+
+def tearDown(self):
+try:
+cleanup_resources(self.apiclient, self.cleanup)
+except Exception as e:
+raise Exception("Warning: Exception during cleanup : %s" % e)
+return
+

[GitHub] cloudstack pull request: CLOUDSTACK-8308-Adding-automation-test-ca...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/170#issuecomment-93303686
  
Adding Snapshot Test cases that are relating secondary
storage limits and snapshot limits

Test Result:

Storage and Snapshot Limit ... === TestName: 
test_01_storage_snapshots_limits | Status : SUCCESS ===
ok

--
Ran 1 test in 228.596s

OK


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

Re: {HELP-NEEDED] Replace Root CA etc, for CPVM and SSVM

[Andrija Panic]

Hi guys,

just to update - issue solved:

Deleted the 5th row, so only 4 additional rows left (as original keystore
table layout prior to replacing certificate)

The problem was actually, while URL encoding ROOT CA and Intermediate CA,
the plus sign ( + ) was replaced by SPACE...

Thanks for all the help everybody


On 7 April 2015 at 20:10, Suresh Sadhu  wrote:

>  If you have taken backup of  your  table(keystore) before upload then you
> revert to previous state then upload the certificates again.
>
> Encode(url ecode) the  root and intermediate keys while uploading through
> api
> Root - seq 1
> Intermediate  seq 2
>
>  And while uploading server certificate  through UI  don 't  encode the
> keys  ,enter only  server certificate and private key(it should be PKCS#8
> format) and domain name  because you have already uploaded root and
> intermediate through API.( how to check certificate uploaded correctly or
> not on system vms ,just run the keytool  -list on system vms --for
> syntax/description   ref this blog it might useful to you :
> http://sadhusuresh.blogspot.in/2015/01/t-hings-you-should-consider-while.html
> )
>
> Regards
> Sadhu
>
>
> -Original Message-
> From: Andrija Panic [mailto:andrija.pa...@gmail.com]
> Sent: 07 April 2015 23:19
> To: dev@cloudstack.apache.org
> Cc: us...@cloudstack.apache.org
> Subject: Re: {HELP-NEEDED] Replace Root CA etc, for CPVM and SSVM
>
> Thanks Suresh.
>
> 2 identical sequence numbers means:  first occurence is OLD Intermediate
> CA(from 1 year ago), and the second occurence is the new one just uploaded
> (it happened I used different names)
>
> for ROOT CA - it happened I used the same name "ROOT1" so the old one got
> overwriten with seq number 1
>
> Do you expect I should delete the old Intermediate1 CA manually (and leave
> only the new one) ?
> Or am I expected to upload again ROOT/intermediate with exact same names
> and seq numbers ?
>
> Thanks
>
> On 7 April 2015 at 19:43, Suresh Sadhu  wrote:
>
> > I see same sequence number for 2 intermediate certificates. does your
> > certificate has multiple  intermediate certificate or it has only one.
> >
> > The reason for getting realhost ip is . your certificate is not
> > applied correctly  that is reason it's still refer the old certificate.
> >
> >
> > Regards
> > sadhu
> >
> > -Original Message-
> > From: Andrija Panic [mailto:andrija.pa...@gmail.com]
> > Sent: 07 April 2015 22:56
> > To: us...@cloudstack.apache.org
> > Cc: dev@cloudstack.apache.org
> > Subject: Re: {HELP-NEEDED] Replace Root CA etc, for CPVM and SSVM
> >
> > Hi Lucian
> >
> > yes it is *.domain.com (from 4.3.1 onwards)...
> >
> > If you can check my attached image, keystore tableseems messed a
> > little bit
> > :)
> > http://snag.gy/LMA4h.jpg
> >
> >
> > On 7 April 2015 at 19:12, Nux!  wrote:
> >
> > > Can you check secstorage.ssl.cert.domain in global settings and see
> > > if it's the correct one?
> > > Should be *.blah.tld or whatever your domain is.
> > >
> > >
> > > HTH
> > > Lucian
> > >
> > > --
> > > Sent from the Delta quadrant using Borg technology!
> > >
> > > Nux!
> > > www.nux.ro
> > >
> > > - Original Message -
> > > > From: "Andrija Panic" 
> > > > To: us...@cloudstack.apache.org, dev@cloudstack.apache.org
> > > > Sent: Tuesday, 7 April, 2015 17:42:35
> > > > Subject: {HELP-NEEDED] Replace Root CA etc, for CPVM and SSVM
> > >
> > > > Hi guys,
> > > >
> > > > our SSL just expired, and I needed to upload new ROOT CA,
> > > > Intemediata
> > > ROOT
> > > > CA, and at the end SSL for sever and a private key.
> > > >
> > > > I uploaded new ROOT CA, and after CPVM rebooted, also uploaded
> > > Intermediate
> > > > ROOT CA, via API, with URL encoded stuff - checked in database all
> > > > seems
> > > OK.
> > > >
> > > > But after uploading new SSL and private key, destroyed CPVM and
> > > > SSVM
> > > > - my Console Proxy shows *.realiphost.com as the domain for the
> > > > SSL wjen I access
> > > >
> > > > Any clues what I did wrong ?
> > > > Should I have somehow removed first old ROOT CA and old
> > > > Intermediate CA, and upload new ones ?
> > > >
> > > > Here is database content from cloud.keystore:
> > > > http://snag.gy/LMA4h.jpg
> > > >
> > > > This means that for some reason, original realiphost.com SSL is
> > > > now used inside CPVM...
> > > >
> > > > Any help greatly appreciated, since this is live system...
> > > >
> > > > Thanks,
> > > >
> > > >
> > > >
> > > > --
> > > >
> > > > Andrija Panić
> > >
> >
> >
> >
> > --
> >
> > Andrija Panić
> >
>
>
>
> --
>
> Andrija Panić
>



-- 

Andrija Panić

[GitHub] cloudstack pull request: CLOUDSTACK-8388: test_accounts.py - Fixin...

[gauravaradhye]

GitHub user gauravaradhye opened a pull request:

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

CLOUDSTACK-8388: test_accounts.py - Fixing pep8 issues

Fixed all pep8 issues.

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

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

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

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


commit 2280d16e060d4e3704acfa2c6da2933771fa0f68
Author: Gaurav Aradhye 
Date:   2015-04-15T11:51:15Z

CLOUDSTACK-8388: test_accounts.py - Fixing pep8 issues




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

snapshot's not honouring the policy.

[Jeronimo Garcia]

Hi List.

Cloudstack doesn't seem to be honouring the snapshot policy , it never
deletes older snapshots when the retention is only set to 2.

I'm running 4.3.0.1 ,  anyone experienced this?

Thanks!

[GitHub] cloudstack pull request: CLOUDSTACK-8335: use libvirt from central...

[K0zka]

Github user K0zka commented on the pull request:

https://github.com/apache/cloudstack/pull/163#issuecomment-93350242
  
Hi @wilderrodrigues 

Patch 9a38b2f7de79888cdbbb2dca2305d6ec10f81ee0 should be a solution for 
*most* of the problem. The objective here is to completely get rid of the 
libvirt.org (and other 3rd party) repository as a step towards sharing CS 
artifacts in maven central. I will send you PRs!

Best regards,
Laszlo


---
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-8388: test_accounts.py - Fixin...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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-8335: use libvirt from central...

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/163#issuecomment-93363759
  
Awesome!

Thanks, @K0zka :)

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

Build failed in Jenkins: simulator-singlerun #1110

[jenkins]

See 

Changes:

[Gaurav Aradhye] CLOUDSTACK-8388: test_accounts.py - Fixing pep8 issues

--
[...truncated 10413 lines...]
> Running query: drop database if exists `cloud_usage`
> Running query: create database `cloud_usage`
> Running query: GRANT ALL ON cloud_usage.* to 'cloud'@`localhost` 
identified by 'cloud'
> Running query: GRANT ALL ON cloud_usage.* to 'cloud'@`%` 
identified by 'cloud'
> Initializing database=cloudbridge with host=localhost port=3306 
username=cloud password=cloud
> Running query: drop database if exists `cloudbridge`
> Running query: create database `cloudbridge`
> Running query: GRANT ALL ON cloudbridge.* to 'cloud'@`localhost` 
identified by 'cloud'
> Running query: GRANT ALL ON cloudbridge.* to 'cloud'@`%` 
identified by 'cloud'
> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing upgrade: com.cloud.upgrade.DatabaseUpgradeChecker
[INFO] 
[INFO] --- maven-site-plugin:3.3:attach-descriptor (attach-descriptor) @ 
cloud-developer ---
[INFO] 
[INFO] --- maven-install-plugin:2.3.1:install (default-install) @ 
cloud-developer ---
[INFO] Installing 
 
to 
/var/lib/jenkins/.m2/repository/org/apache/cloudstack/cloud-developer/4.6.0-SNAPSHOT/cloud-developer-4.6.0-SNAPSHOT.pom
[INFO] 
[INFO] BUILD SUCCESS
[INFO] 
[INFO] Total time: 1:56.130s
[INFO] Finished at: Wed Apr 15 08:37:19 EDT 2015
[INFO] Final Memory: 43M/193M
[INFO] 
[WARNING] The requested profile "simulator" could not be activated because it 
does not exist.
[simulator-singlerun] $ mvn -P developer -pl developer -Ddeploydb-simulator
[INFO] Scanning for projects...
[INFO] 
[INFO] 
[INFO] Building Apache CloudStack Developer Mode 4.6.0-SNAPSHOT
[INFO] 
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-developer ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- properties-maven-plugin:1.0-alpha-2:read-project-properties 
(default) @ cloud-developer ---
[WARNING] Ignoring missing properties file: 

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-developer ---
[INFO] 
[INFO] --- maven-antrun-plugin:1.8:run (default) @ cloud-developer ---
[INFO] Executing tasks

main:
[INFO] Executed tasks
[INFO] 
[INFO] >>> exec-maven-plugin:1.2.1:java (create-schema-simulator) @ 
cloud-developer >>>
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-developer ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] <<< exec-maven-plugin:1.2.1:java (create-schema-simulator) @ 
cloud-developer <<<
[INFO] 
[INFO] --- exec-maven-plugin:

[GitHub] cloudstack pull request: Make the routers iptables config persiste...

[resmo]

Github user resmo commented on the pull request:

https://github.com/apache/cloudstack/pull/167#issuecomment-93387973
  
Okay, I see. In this case cherry-picking does not make any sense.
I may implement a workaround by saving the iptables periodically on the 
routers in our env. This is not a big overhead. Thanks guys @bhaisaab 
@wilderrodrigues.


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

CLOUDSTACK-8335

[Roger Crerie]

Did this make it back into the master repository?  I have done a pull from the 
master and am seeing the following errors

[INFO] 
[INFO] Building Apache CloudStack Plugin - Hypervisor KVM 4.6.0-SNAPSHOT
[INFO] 
Downloading: http://ceph.com/maven/joda-time/joda-time/maven-metadata.xml
[WARNING] Could not transfer metadata joda-time:joda-time/maven-metadata.xml 
from/to ceph-com (http://ceph.com/maven): Failed to transfer file: 
http://ceph.com/maven/joda-time/joda-time/maven-metadata.xml. Return code is: 
503 , ReasonPhrase:Service Unavailable.
Downloading: http://ceph.com/maven/org/libvirt/libvirt/0.5.1/libvirt-0.5.1.pom

Re: [ANNOUNCE] pull request builder

[Daan Hoogland]

On Wed, Apr 15, 2015 at 12:06 AM, Abhinandan Prateek
 wrote:
> I have been fiddling with simulator and provisioning tests recently, ping me 
> in case I can take up some related tasks.
> -abhi


thanks, at the moment we don't have mysql, so we need to device a
docker based test env that can be deployed on builds.a.o. other
tactics are in the making.

-- 
Daan

Re: CLOUDSTACK-8335

[Wido den Hollander]

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1



On 04/15/2015 05:07 PM, Roger Crerie wrote:
> Did this make it back into the master repository?  I have done a
> pull from the master and am seeing the following errors
> 

The issue is currently a external repo which is down. (ceph.com). It's
not related to your pull request.

Wido

> [INFO]
> --
- --
>
> 
[INFO] Building Apache CloudStack Plugin - Hypervisor KVM 4.6.0-SNAPSHOT
> [INFO]
> --
- --
>
> 
Downloading: http://ceph.com/maven/joda-time/joda-time/maven-metadata.xm
l
> [WARNING] Could not transfer metadata
> joda-time:joda-time/maven-metadata.xml from/to ceph-com
> (http://ceph.com/maven): Failed to transfer file:
> http://ceph.com/maven/joda-time/joda-time/maven-metadata.xml.
> Return code is: 503 , ReasonPhrase:Service Unavailable. 
> Downloading:
> http://ceph.com/maven/org/libvirt/libvirt/0.5.1/libvirt-0.5.1.pom
> 
> 
-BEGIN PGP SIGNATURE-
Version: GnuPG v1

iQIcBAEBAgAGBQJVLn93AAoJEAGbWC3bPspCtzgP/iMTruC+OrdNWu3aBBCh8x0p
Nk6bd7p+6mdNhnNdWMIKKDiTz7cllmVp75tpb66PxWW50XuQu/e0nSjw364LBcaX
vn0bOD5szQu0sDkef5nXRRvDWRsGounzLQgcVXwR3YJ4jC2t+oYrApcscobECMCm
fPsRxQ8Xuvoqf3ASxUDGc298cgJ0u6VhsuaSDGiDahabowyQoyoAkTy9+GT4B0mV
8fpuy4RfQURxwdP1p1iogjS6XmERrYGCffEDEjrTkQkGc4r+bb1/PEAi8sxJKOI8
uMqYw8K/W/mEwGMBxeMGKFHUkMkrKW97/OQK17QpLwuTbZbsRZlqrI0jg0RpKVk0
xEiumSEj6IJYPIBLAzqoaoSZGfES9WL5huTTKrPs3yuJzvz5IJterthlUq7Q1L7K
EMW7shmrTdU6EZ/bPVGn35ZbLwBBLn+1F5NJMXHqlugGW7977DvqnjutJZ2kZkzW
N2JeDx2lwq2O0xFgceEMMa7D6JSDvWaq5BBi7PCz1HHROhfsKsRrZsafamqON7nu
vBqZzzkK0K9XgRf0OL2kG1/pZ4THHEGwp6o1U9zN5As1hS/XJPtQbTIzAFbIXkIJ
PwsXJlLmXpeMFRu6QyvK//ol+wSf4rbuLsCrScg54iM2UXWesrq2vUDzdMdqn040
otb5otgcqPoP4NVEMqqW
=onvz
-END PGP SIGNATURE-

RE: CLOUDSTACK-8335

[Roger Crerie]

Is there a workaround for this issue so I can continue to build and work :)?  A 
secondary location or an estimate when this repo will be back available?  
Thanks for the quick reply.


Roger

-Original Message-
From: Wido den Hollander [mailto:w...@widodh.nl] 
Sent: Wednesday, April 15, 2015 11:11 AM
To: dev@cloudstack.apache.org
Subject: Re: CLOUDSTACK-8335

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1



On 04/15/2015 05:07 PM, Roger Crerie wrote:
> Did this make it back into the master repository?  I have done a pull 
> from the master and am seeing the following errors
> 

The issue is currently a external repo which is down. (ceph.com). It's not 
related to your pull request.

Wido

> [INFO]
> --
- --
>
> 
[INFO] Building Apache CloudStack Plugin - Hypervisor KVM 4.6.0-SNAPSHOT
> [INFO]
> --
- --
>
> 
Downloading: http://ceph.com/maven/joda-time/joda-time/maven-metadata.xm
l
> [WARNING] Could not transfer metadata
> joda-time:joda-time/maven-metadata.xml from/to ceph-com
> (http://ceph.com/maven): Failed to transfer file:
> http://ceph.com/maven/joda-time/joda-time/maven-metadata.xml.
> Return code is: 503 , ReasonPhrase:Service Unavailable. 
> Downloading:
> http://ceph.com/maven/org/libvirt/libvirt/0.5.1/libvirt-0.5.1.pom
> 
> 
-BEGIN PGP SIGNATURE-
Version: GnuPG v1

iQIcBAEBAgAGBQJVLn93AAoJEAGbWC3bPspCtzgP/iMTruC+OrdNWu3aBBCh8x0p
Nk6bd7p+6mdNhnNdWMIKKDiTz7cllmVp75tpb66PxWW50XuQu/e0nSjw364LBcaX
vn0bOD5szQu0sDkef5nXRRvDWRsGounzLQgcVXwR3YJ4jC2t+oYrApcscobECMCm
fPsRxQ8Xuvoqf3ASxUDGc298cgJ0u6VhsuaSDGiDahabowyQoyoAkTy9+GT4B0mV
8fpuy4RfQURxwdP1p1iogjS6XmERrYGCffEDEjrTkQkGc4r+bb1/PEAi8sxJKOI8
uMqYw8K/W/mEwGMBxeMGKFHUkMkrKW97/OQK17QpLwuTbZbsRZlqrI0jg0RpKVk0
xEiumSEj6IJYPIBLAzqoaoSZGfES9WL5huTTKrPs3yuJzvz5IJterthlUq7Q1L7K
EMW7shmrTdU6EZ/bPVGn35ZbLwBBLn+1F5NJMXHqlugGW7977DvqnjutJZ2kZkzW
N2JeDx2lwq2O0xFgceEMMa7D6JSDvWaq5BBi7PCz1HHROhfsKsRrZsafamqON7nu
vBqZzzkK0K9XgRf0OL2kG1/pZ4THHEGwp6o1U9zN5As1hS/XJPtQbTIzAFbIXkIJ
PwsXJlLmXpeMFRu6QyvK//ol+wSf4rbuLsCrScg54iM2UXWesrq2vUDzdMdqn040
otb5otgcqPoP4NVEMqqW
=onvz
-END PGP SIGNATURE-

Re: CLOUDSTACK-8335

[Wido den Hollander]

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 04/15/2015 05:17 PM, Roger Crerie wrote:
> Is there a workaround for this issue so I can continue to build and
> work :)?  A secondary location or an estimate when this repo will
> be back available?  Thanks for the quick reply.
> 
> 

Yes, change ceph.com to eu.ceph.com in pom.xml

That should get all the files from a secondary location.

Wido

> Roger
> 
> -Original Message- From: Wido den Hollander
> [mailto:w...@widodh.nl] Sent: Wednesday, April 15, 2015 11:11 AM 
> To: dev@cloudstack.apache.org Subject: Re: CLOUDSTACK-8335
> 
> 
> 
> On 04/15/2015 05:07 PM, Roger Crerie wrote:
>> Did this make it back into the master repository?  I have done a
>> pull from the master and am seeing the following errors
> 
> 
> The issue is currently a external repo which is down. (ceph.com).
> It's not related to your pull request.
> 
> Wido
> 
>> [INFO] 
>> -
- -
>
-BEGIN PGP SIGNATURE-
Version: GnuPG v1

iQIcBAEBAgAGBQJVLoQgAAoJEAGbWC3bPspCZygP/0x1PhKp6lb5ra6/3M67ShVC
B5gV/pUMqTkRpvNw4j+4v/n1yJ9+sRUtr747pgikb9R6wrHKUYXH6Bs9dFHTY/Lq
7TR0UkKHKtDcTvwktB7eKYYq/1VYYm4L773kdGR4AEGyjRZBIsqXl4iXoEqCf3OY
LVF9toPZux+aOPrJPJYnz8HnVPhTpL1b9bRJtV8srdnJjU6kuIEP3qpSixFhXy2V
PRTL7MHahbZ2x1b295ZZmpQKbE6jzz6O0iSrnvFJChse7CzsxVMgIo9x0SzebI8G
cRoHngDzLJ8rsqzpwl9HWs3jHfHBoME7Ia11Ps9Vsp/Q8tJzOL2PWc6/qsmZdc6S
J5uXn6O1yffdR8Bz8LiLmkLLu0Z5dLUjjgNIJaMCeqJBr7J78WOQyxvCISpPDnZV
2NH9500XJ/Rp99l8ZzKXrSJRXCjSsT435XM1aOofRiz+H0haxkX1tq8G23AYXgeE
MaqmFyleur+obK+FLf8FvJV5CsdUzOZgWwee58US/pS8qGW4R2F6lMFSLnejYUmI
+7EvqDdymQ1Hcy+79aZLc6yzCEAMbxaGUYSpCxW+l+0SmRrAENtte7/lKBrHy2Rn
PPfZQk/E9DFPf8hwjNeCoGZba4suNS5mrRGP79GILqIcG34Gn5S20rG8Z6ssXGEj
rvSBNXAn5Iy996peMoE/
=YSl2
-END PGP SIGNATURE-

Build failed in Jenkins: build-4.5 #439

[jenkins]

See 

Changes:

[marcus_sorensen] CLOUDSTACK-8387 - Close mgmt server peer socket on failure, 
without relying on autoclose

--
[...truncated 1672 lines...]
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-engine-orchestration ---
[INFO] Deleting  
(includes = [target, dist], excludes = [])
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-server ---
[INFO] Starting audit...
Audit done.

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

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ cloud-usage ---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-engine-network ---
[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:2.5.1:compile (default-compile) @ 
cloud-engine-network ---
[INFO] Compiling 1 source file to 

[INFO] 
[INFO] --- maven-antrun-plugin:1.7:run (generate-resource) @ cloud-usage ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-engine-orchestration ---
[INFO] Executing tasks

main:
 [copy] Copying 2 files to 

[INFO] Executed tasks
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-usage ---
[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:2.5.1:compile (default-compile) @ cloud-usage 
---
[INFO] Compiling 19 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-engine-orchestration ---
[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:2.5.1:compile (default-compile) @ 
cloud-engine-orchestration ---
[INFO] Compiling 68 source files to 

[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-engine-network ---
[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:2.5.1:testCompile (default-testCompile) @ 
cloud-engine-network ---
[INFO] No sources to compile
[INFO] 
[INFO] --- maven-surefire-plugin:2.12:test (default-test) @ 
cloud-engine-network ---
[INFO] Surefire report directory: 


---
 T E S T S
---

Results :

Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ cloud-server 
---
[INFO] 
[INFO] --- maven-resources-plugin:2.5:testResources (default-testResources) @ 
cloud-usage ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 8 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-usage ---
[INFO] Compiling 4 source files to 

[INFO] 
[INFO] --- maven-antrun-plugin:1.7:run (generate-resource) @ cloud-server ---
[INFO] Executing tasks

main:
 [copy] Copying 3 files to 

 [copy] Copying 1 file to 

[INFO] Executed tasks
[INFO] 
[INFO] --- maven-resources-plugin:2.5:resources (default-resources) @ 
cloud-server ---
[debug] execute contextualize
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 30 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:compile (default-compile) @ cloud-server 
---
[INFO] Compiling 362 source files to 

[INFO] 
[INFO] --- maven-surefire-p

Jenkins build is still unstable: simulator-4.5-singlerun #205

[jenkins]

See 

Build failed in Jenkins: simulator-singlerun #1111

[jenkins]

See 

Changes:

[marcus_sorensen] CLOUDSTACK-8387 - Don't autoclose new mgmt server peer 
connections as soon as they open

[marcus_sorensen] CLOUDSTACK-8387 - Close mgmt server peer socket on failure, 
without relying on autoclose

--
[...truncated 2049 lines...]
[INFO] Copying slf4j-api-1.7.7.jar to 

[INFO] Copying spring-core-3.2.12.RELEASE.jar to 

[INFO] Copying aws-java-sdk-cloudsearch-1.9.8.jar to 

[INFO] Copying trilead-ssh2-1.0.0-build217.jar to 

[INFO] Copying jackson-annotations-2.4.4.jar to 

[INFO] Copying log4j-1.2.17.jar to 

[INFO] Copying aws-java-sdk-1.9.8.jar to 

[INFO] Copying commons-digester-1.8.jar to 

[INFO] Copying apache-log4j-extras-1.2.17.jar to 

[INFO] Copying activation-1.1.jar to 

[INFO] Copying aws-java-sdk-cognitosync-1.9.8.jar to 

[INFO] Copying commons-net-3.3.jar to 

[INFO] Copying httpcore-4.3.3.jar to 

[INFO] Copying xalan-2.7.1.jar to 

[INFO] Copying cloud-framework-events-4.6.0-SNAPSHOT.jar to 

[INFO] Copying aws-java-sdk-sns-1.9.8.jar to 

[INFO] Copying jetty-io-8.1.15.v20140411.jar to 

[INFO] Copying aopalliance-1.0.jar to 

[INFO] Copying aws-java-sdk-glacier-1.9.8.jar to 

[INFO] Copying jetty-util-8.1.15.v20140411.jar to 

[INFO] Copying aws-java-sdk-cloudwatch-1.9.8.jar to 

[INFO] Copying bcprov-jdk16-1.46.jar to 

[INFO] Copying nekohtml-1.9.12.jar to 

[INFO] Copying aws-java-sdk-cloudfront-1.9.8.jar to 

[INFO] Copying aws-java-sdk-kinesis-1.9.8.jar to 

[INFO] Copying batik-util-1.7.jar to 

[INFO] Copying xml-resolver-1.2.jar to 

[INFO] Copying aws-java-sdk-storagegateway-1.9.8.jar to 

[INFO] Copying stax2-api-3.1.4.jar to 

[INFO] Copying commons-fileuploa

Jenkins build is back to normal : build-4.5 #440

[jenkins]

See 

RE: listUsageRecord only for Admins? Why?

[Somesh Naidu]

As a workaround, we could just update the permissions for the API in the 
commands.properties to allow normal users to make that API call.

Default Contents:
 usage commands
generateUsageRecords=1
*listUsageRecords=7*
listUsageTypes=1

Change it to:
 usage commands
generateUsageRecords=1
*listUsageRecords=15*
listUsageTypes=1

Somesh
CloudPlatform Escalations
Citrix Systems, Inc.

-Original Message-
From: Koushik Das [mailto:koushik@citrix.com] 
Sent: Wednesday, April 15, 2015 5:41 AM
To: dev
Subject: RE: listUsageRecord only for Admins? Why?

Agree. I don't think it's possible currently. The listUsageRecords needs to be 
modified. 

-Original Message-
From: Ramzi Youssefi [mailto:ryouss...@gmail.com] 
Sent: Wednesday, 15 April 2015 13:28
To: dev
Subject: listUsageRecord only for Admins? Why?

Hello developers!

is there any reason why listUsageRecords api call is only limited to root and 
admin users?
I think this use case is very common and it would be nice if clients can see 
their usage records in their projects.
Is there any workaround for that?

Best,

--
Ramzi Youssefi
Görlitzer Str. 49
01099 Dresden, Germany.
Cell: +49 173 1965465
Email: ryouss...@gmail.com

Re: Missing storage label in XenCenter after migrate

[raja sekhar]

Iam talking about xencenter storage labels.you have asked that one right?

Regards,
Rajasekhar.

On Wed, Apr 15, 2015 at 1:25 PM, S. Brüseke - proIO GmbH <
s.brues...@proio.com> wrote:

> Cannot confirm this! Are you talking about XenServer "name-label" or CS?
>
> Mit freundlichen Grüßen / With kind regards,
>
> Swen Brüseke
>
>
> Von: raja sekhar [mailto:rajsekhar@gmail.com]
> Gesendet: Mittwoch, 15. April 2015 09:43
> An: S. Brüseke - proIO GmbH
> Cc: dev@cloudstack.apache.org
> Betreff: Re: Missing storage label in XenCenter after migrate
>
> I have tested in ACS 4.3.1 with Xenserver 6.2 as hyperviser.
> When vm is down the name label remains same,but when the vm is live name
> label changes.
>
> On Wed, Apr 15, 2015 at 1:01 PM, S. Brüseke - proIO GmbH <
> s.brues...@proio.com> wrote:
> Hi,
>
> thank you for testing. In 4.3 the name label is empty even when the vm is
> down.
>
> Mit freundlichen Grüßen / With kind regards,
>
> Swen Brüseke
>
> Von: raja sekhar [mailto:rajsekhar@gmail.com]
> Gesendet: Dienstag, 14. April 2015 18:48
> An: dev@cloudstack.apache.org; S. Brüseke - proIO GmbH
> Betreff: Re: Missing storage label in XenCenter after migrate
>
> Bruseke,
>
> When the disks are live migrated from one primary to another this problem
> is arising, i'am also getting same problem, the disk labels are changed to
> "Cloud-".it seems to be a bug.
> how ever it doesn't happens when the vm is down.
>
> On Tue, Apr 14, 2015 at 6:11 PM, S. Brüseke - proIO GmbH <
> s.brues...@proio.com> wrote:
> Hi,
>
> can somebody please test this behavior in 4.5.1-rc2?
>
> 1. Create instance on XenServer hypervisor via CS.
> 2. Check "Name" of storage in XenCenter's storage tab for this instance.
> (should be ROOT-XXX)
> 3. Migrate primary storage to different storage via CS.
> 4. Take a look at "Name" of storage in XenCenter's storage tab again. (It
> is empty now)
>
> If this is the case on your installation I consider this a bug. Can please
> someone verify?
> Thank you very much!
>
> Mit freundlichen Grüßen / With kind regards,
>
> Swen Brüseke
>
>
>
>
> - proIO GmbH -
> Geschäftsführer: Swen Brüseke
> Sitz der Gesellschaft: Frankfurt am Main
>
> USt-IdNr. DE 267 075 918
> Registergericht: Frankfurt am Main - HRB 86239
>
> Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte
> Informationen.
> Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich
> erhalten haben,
> informieren Sie bitte sofort den Absender und vernichten Sie diese Mail.
> Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieser Mail sind
> nicht gestattet.
>
> This e-mail may contain confidential and/or privileged information.
> If you are not the intended recipient (or have received this e-mail in
> error) please notify
> the sender immediately and destroy this e-mail.
> Any unauthorized copying, disclosure or distribution of the material in
> this e-mail is strictly forbidden.
>
>
>
>
> - proIO GmbH -
> Geschäftsführer: Swen Brüseke
> Sitz der Gesellschaft: Frankfurt am Main
>
> USt-IdNr. DE 267 075 918
> Registergericht: Frankfurt am Main - HRB 86239
>
> Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte
> Informationen.
> Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich
> erhalten haben,
> informieren Sie bitte sofort den Absender und vernichten Sie diese Mail.
> Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieser Mail sind
> nicht gestattet.
>
> This e-mail may contain confidential and/or privileged information.
> If you are not the intended recipient (or have received this e-mail in
> error) please notify
> the sender immediately and destroy this e-mail.
> Any unauthorized copying, disclosure or distribution of the material in
> this e-mail is strictly forbidden.
>
>
> - proIO GmbH -
> Geschäftsführer: Swen Brüseke
> Sitz der Gesellschaft: Frankfurt am Main
>
> USt-IdNr. DE 267 075 918
> Registergericht: Frankfurt am Main - HRB 86239
>
> Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte
> Informationen.
> Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich
> erhalten haben,
> informieren Sie bitte sofort den Absender und vernichten Sie diese Mail.
> Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieser Mail sind
> nicht gestattet.
>
> This e-mail may contain confidential and/or privileged information.
> If you are not the intended recipient (or have received this e-mail in
> error) please notify
> the sender immediately and destroy this e-mail.
> Any unauthorized copying, disclosure or distribution of the material in
> this e-mail is strictly forbidden.
>
>

Jenkins build is still unstable: simulator-4.5-singlerun #206

[jenkins]

See 

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

[K0zka]

GitHub user K0zka opened a pull request:

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

CLOUDSTACK-8304: disable snapshots from non-snapshot repositories

Backport from master to 4.4

Signed-off-by: Laszlo Hornyak 

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

$ git pull https://github.com/K0zka/cloudstack libvirt_4.4_2

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

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


commit 4375b374432947148c6238c164e2d8a77c93c154
Author: Laszlo Hornyak 
Date:   2015-03-06T16:41:32Z

CLOUDSTACK-8304: disable snapshots from non-snapshot repositories

Signed-off-by: Laszlo Hornyak 




---
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: simulator-singlerun #1112

[jenkins]

See 

Changes:

[marcus_sorensen] CLOUDSTACK-8387 - Close mgmt server peer socket on failure, 
without relying on autoclose

--
[...truncated 10447 lines...]
> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing upgrade: com.cloud.upgrade.DatabaseUpgradeChecker
[INFO] 
[INFO] --- maven-site-plugin:3.3:attach-descriptor (attach-descriptor) @ 
cloud-developer ---
[INFO] 
[INFO] --- maven-install-plugin:2.3.1:install (default-install) @ 
cloud-developer ---
[INFO] Installing 
 
to 
/var/lib/jenkins/.m2/repository/org/apache/cloudstack/cloud-developer/4.6.0-SNAPSHOT/cloud-developer-4.6.0-SNAPSHOT.pom
[INFO] 
[INFO] BUILD SUCCESS
[INFO] 
[INFO] Total time: 1:49.560s
[INFO] Finished at: Wed Apr 15 14:22:30 EDT 2015
[INFO] Final Memory: 45M/269M
[INFO] 
[WARNING] The requested profile "simulator" could not be activated because it 
does not exist.
[simulator-singlerun] $ mvn -P developer -pl developer -Ddeploydb-simulator
[INFO] Scanning for projects...
[INFO] 
[INFO] 
[INFO] Building Apache CloudStack Developer Mode 4.6.0-SNAPSHOT
[INFO] 
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-developer ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- properties-maven-plugin:1.0-alpha-2:read-project-properties 
(default) @ cloud-developer ---
[WARNING] Ignoring missing properties file: 

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-developer ---
[INFO] 
[INFO] --- maven-antrun-plugin:1.8:run (default) @ cloud-developer ---
[INFO] Executing tasks

main:
[INFO] Executed tasks
[INFO] 
[INFO] >>> exec-maven-plugin:1.2.1:java (create-schema-simulator) @ 
cloud-developer >>>
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-developer ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] <<< exec-maven-plugin:1.2.1:java (create-schema-simulator) @ 
cloud-developer <<<
[INFO] 
[INFO] --- exec-maven-plugin:1.2.1:java (create-schema-simulator) @ 
cloud-developer ---
log4j:WARN No appenders could be found for logger 
(org.springframework.core.env.StandardEnvironment).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more 
info.
> WARNING: Provided file does not exist: 

> Initializing database=simulator with host=localhost port=3306 
username=cloud password=cloud
> Running query: drop database if exists `simulator`
> Running query: create database `simulator`
> Running query: GRANT ALL ON simulator.* to 'cloud'@`localhost` 
identified by 'cloud'
> Running query: GRANT ALL ON simulator.* to 'cloud'@`%` identified 
by 'cloud'
> Processing SQL file at 

> Processing SQL file at 

Re: List of freelance developers (Was: Re: searching for a cloudstack developer)

[Daan Hoogland]

Swen, create an account in jura and i will give you access to create it.

On Wed, Apr 15, 2015 at 3:58 AM, S. Brüseke - proIO GmbH
 wrote:
> That is a great idea! Who can create this page?
>
> Mit freundlichen Grüßen / With kind regards,
>
> Swen Brüseke
>
> -Ursprüngliche Nachricht-
> Von: Sebastien Goasguen [mailto:run...@gmail.com]
> Gesendet: Mittwoch, 15. April 2015 10:45
> An: dev@cloudstack.apache.org
> Betreff: Re: List of freelance developers (Was: Re: searching for a 
> cloudstack developer)
>
>
>> On Apr 15, 2015, at 10:20 AM, Erik Weber  wrote:
>>
>> On Wed, Apr 15, 2015 at 10:05 AM, S. Brüseke - proIO GmbH <
>> s.brues...@proio.com> wrote:
>>
>>> Hi,
>>>
>>> we are searching for a cloudstack developer! We found and reported
>>> some bugs in CS, which are really annoying for our workflow, but
>>> minor to the opensource project.
>>> The problem is that we do not have the knowlegde and manpower to
>>> solve these bugs, but we are willing to pay for a solution and
>>> provide the code to the opensource project.
>>>
>>> Please contact me, if you need more information.
>>>
>>>
>> Renaming the thread for my input.
>>
>> We also face some issues from time to time that is of low importance
>> to the project.
>>
>> Would it make sense, and be possible, to compile a list of people /
>> companies / other resources that are willing to do per case development?
>>
>
> I think you could definitely start a wiki page and list folks there.
>
>
>> --
>> Erik
>
>
>
>
> - proIO GmbH -
> Geschäftsführer: Swen Brüseke
> Sitz der Gesellschaft: Frankfurt am Main
>
> USt-IdNr. DE 267 075 918
> Registergericht: Frankfurt am Main - HRB 86239
>
> Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte Informationen.
> Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich 
> erhalten haben,
> informieren Sie bitte sofort den Absender und vernichten Sie diese Mail.
> Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieser Mail sind nicht 
> gestattet.
>
> This e-mail may contain confidential and/or privileged information.
> If you are not the intended recipient (or have received this e-mail in error) 
> please notify
> the sender immediately and destroy this e-mail.
> Any unauthorized copying, disclosure or distribution of the material in this 
> e-mail is strictly forbidden.
>
>



-- 
Daan

Re: List of freelance developers (Was: Re: searching for a cloudstack developer)

[Nux!]

Great idea, everyone could use a hired gun every now and then!

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Erik Weber" 
> To: "dev" 
> Sent: Wednesday, 15 April, 2015 09:20:39
> Subject: List of freelance developers (Was: Re: searching for a cloudstack 
> developer)

> On Wed, Apr 15, 2015 at 10:05 AM, S. Brüseke - proIO GmbH <
> s.brues...@proio.com> wrote:
> 
>> Hi,
>>
>> we are searching for a cloudstack developer! We found and reported some
>> bugs in CS, which are really annoying for our workflow, but  minor to the
>> opensource project.
>> The problem is that we do not have the knowlegde and manpower to solve
>> these bugs, but we are willing to pay for a solution and provide the code
>> to the opensource project.
>>
>> Please contact me, if you need more information.
>>
>>
> Renaming the thread for my input.
> 
> We also face some issues from time to time that is of low importance to the
> project.
> 
> Would it make sense, and be possible, to compile a list of people /
> companies / other resources that are willing to do per case development?
> 
> --
> Erik

Re: List of freelance developers (Was: Re: searching for a cloudstack developer)

[Erik Weber]

On Wed, Apr 15, 2015 at 9:13 PM, Daan Hoogland 
wrote:

> Swen, create an account in jura and i will give you access to create it.
>


Would you mind adding me, username terbolous, as well? :-)

-- 
Erik

Re: List of freelance developers (Was: Re: searching for a cloudstack developer)

[Daan Hoogland]

On Wed, Apr 15, 2015 at 2:50 PM, Erik Weber  wrote:
> terbolous


is added

-- 
Daan

Fwd: Additional Travis-CI Capacity

[David Nalley]

FYI


-- Forwarded message --
From: David Nalley 
Date: Wed, Apr 15, 2015 at 4:33 PM
Subject: Additional Travis-CI Capacity
To: "bui...@apache.org" 


FYI:

https://blogs.apache.org/infra/entry/apache_gains_additional_travis_ci

Re: KVM plugs public nic twice in non-VPC router

[Simon Weller]

Remi,

We haven't experienced exactly what you're describing, but we did have a 
similar issue that caused multiple interfaces to be built back when we upgraded 
to 4.3 from 4.1.1 on KVM.
Have a look at https://issues.apache.org/jira/browse/CLOUDSTACK-6464.
The notes also reference another couple of issues that might have been related.

This particular issue was apparently fixed in 4.4, but nevertheless, it might 
be worth a look to see if anything jumps out for you.

- Si


From: Remi Bergsma 
Sent: Tuesday, April 14, 2015 4:19 PM
To: dev@cloudstack.apache.org
Subject: KVM plugs public nic twice in non-VPC router

Hi,

We’re testing KVM and noticed that a virtual router (non-VPC) gets two pubic ip 
nics plugged. This is in 4.4.2 when using Open vSwitch on CentOS 7 (advanced 
zone).

The reason for this, is that the first nic gets plugged by provisioning the 
nic. Then, a second nic is plugged that gets the same ip address but a slightly 
different nic. This is done by IpAssocCommand. When diving into this, this 
seems to happen because the StartCommand contains a IpAssocCommand part with 
again the public ip. This is probably because this is how we plug the public 
nic on VPCs. I think we should remove it for non-VPC routers.

Here you see the second public nic getting plugged:
2015-04-14 21:19:20,099 DEBUG [cloud.agent.Agent] (agentRequest-Handler-5:null) 
Processing command: com.cloud.agent.api.routing.IpAssocCommand
2015-04-14 21:19:20,103 DEBUG [kvm.resource.OvsVifDriver] 
(agentRequest-Handler-5:null) plugging 
nic=[Nic:Public-null-vlan://50]

Notice the ip-address part of the object is empty.  This leads to this command 
being ran on the router:

Apr 14 19:19:20 r-31443-VM cloud: VR config: executing: 
/opt/cloud/bin/ipassoc.sh -A -s -f -l 1.2.3.4/24 -c eth3 -g 1.2.3.1 -n

>From a functional perspective this does work, but is not what it should be if 
>you ask me.

It looks like this issue:
https://issues.apache.org/jira/browse/CLOUDSTACK-4690

But the story about many many nics I cannot reproduce. I do nowever found that 
associating extra public ip addresses does not work. The management server 
reports everything fine, but I see nothing happening on the router nor the 
agent on the hypervisor:
2015-04-14 23:09:13,437 DEBUG [cloud.network.IpAddressManagerImpl] 
(API-Job-Executor-82:ctx-f4953f35 job-534671 ctx-2a536669) Associating ip 
Ip[1.2.3.5-2] to network Ntwk[a4960690-4a29-445e-b745-881c13bca270|Guest|14]
2015-04-14 23:09:13,449 DEBUG [cloud.network.IpAddressManagerImpl] 
(API-Job-Executor-82:ctx-f4953f35 job-534671 ctx-2a536669) Successfully 
associated ip
address 1.2.3.5 to network Ntwk[a4960690-4a29-445e-b745-881c13bca270|Guest|14]

Question:
Does anyone else with KVM running see this behaviour? Or am I missing something?

Thanks,
Remi

Re: Additional Travis-CI Capacity

[Sebastien Goasguen]

> On Apr 15, 2015, at 10:35 PM, David Nalley  wrote:
> 
> FYI
> 
> 
> -- Forwarded message --
> From: David Nalley 
> Date: Wed, Apr 15, 2015 at 4:33 PM
> Subject: Additional Travis-CI Capacity
> To: "bui...@apache.org" 
> 
> 
> FYI:
> 
> https://blogs.apache.org/infra/entry/apache_gains_additional_travis_ci

outstanding and thank you to the ASF for doing this

Re: [VOTE] Apache Cloudstack 4.4.3

[Daan Hoogland]

closing this vote because of the ifconfig on centos7 issue. new rc coming up.

On Tue, Apr 14, 2015 at 10:29 AM, Daan Hoogland  wrote:
> Hi All,
>
> I've created a 4.4.3 release, with the following artifacts up for a vote:
>
> Git Branch and Commit SH:
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.4
> Commit: f2826ac60af3d9aae508a505370e1b280cb6f599
>
> List of changes:
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.4
> https://issues.apache.org/jira/issues/?filter=12330007
>
> Source release (checksums and signatures are available at the same
> location):
> https://dist.apache.org/repos/dist/dev/cloudstack/4.4/
>
> PGP release keys (signed using 2048D/5AABEBEA):
> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>
> Vote will be open for 72 hours.
>
> For sanity in tallying the vote, can PMC members please be sure to
> indicate "(binding)" with their vote?
>
> [ ] +1  approve
> [ ] +0  no opinion
> [ ] -1  disapprove (and reason why)
>
>
> @Rohit: can you facilitate rpm/deb builds again, thanks
> --
> Daan



-- 
Daan

[VOTE] Apache Cloudstack 4.4.3

[Daan Hoogland]

Hi All,

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

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

List of changes:
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.4
https://issues.apache.org/jira/issues/?filter=12330007

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

PGP release keys (signed using 2048D/5AABEBEA):
https://dist.apache.org/repos/dist/release/cloudstack/KEYS

Vote will be open for 72 hours.

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

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

-- 
Daan

Re: [VOTE] Apache Cloudstack 4.4.3

[Daan Hoogland]

fyi the release branch is 4.4-RC20150415T1700

On Tue, Apr 14, 2015 at 10:29 AM, Daan Hoogland  wrote:
> Hi All,
>
> I've created a 4.4.3 release, with the following artifacts up for a vote:
>
> Git Branch and Commit SH:
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.4
> Commit: f2826ac60af3d9aae508a505370e1b280cb6f599
>
> List of changes:
> https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.4
> https://issues.apache.org/jira/issues/?filter=12330007
>
> Source release (checksums and signatures are available at the same
> location):
> https://dist.apache.org/repos/dist/dev/cloudstack/4.4/
>
> PGP release keys (signed using 2048D/5AABEBEA):
> https://dist.apache.org/repos/dist/release/cloudstack/KEYS
>
> Vote will be open for 72 hours.
>
> For sanity in tallying the vote, can PMC members please be sure to
> indicate "(binding)" with their vote?
>
> [ ] +1  approve
> [ ] +0  no opinion
> [ ] -1  disapprove (and reason why)
>
>
> @Rohit: can you facilitate rpm/deb builds again, thanks
> --
> Daan



-- 
Daan

[GitHub] cloudstack pull request: CLOUDSTACK-8388: test_VirtualRouter_alert...

[gauravaradhye]

GitHub user gauravaradhye opened a pull request:

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

CLOUDSTACK-8388: test_VirtualRouter_alerts.py - Fix wait time and pep8 
issues

The wait period should be only 60 seconds more than 
"router.alerts.check.interval", it was 600 seconds before.
Also fixed pep8 issues.


Log:
test_01_VRServiceFailureAlerting 
(test_VirtualRouter_alerts.TestVRServiceFailureAlerting) ... === TestName: 
test_01_VRServiceFailureAlerting | Status : SUCCESS ===
ok

--
Ran 1 test in 1988.550s

OK

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

$ git pull https://github.com/gauravaradhye/cloudstack 8388-VirtualRouters

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

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


commit 270ba4828fd3d7d2ef34c6950c066b939d075ef7
Author: Gaurav Aradhye 
Date:   2015-04-16T03:49:17Z

CLOUDSTACK-8388: test_VirtualRouter_alerts.py - Fix wait time and pep8 
issues




---
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-8388: test_VirtualRouter_alert...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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-8304: disable snapshots from n...

[karuturi]

Github user karuturi commented on the pull request:

https://github.com/apache/cloudstack/pull/172#issuecomment-93635210
  
do we need this in 4.4? its a build time performance improvement. 


---
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-8304: disable snapshots from n...

[K0zka]

Github user K0zka commented on the pull request:

https://github.com/apache/cloudstack/pull/172#issuecomment-93639076
  
Backport was requested by @wilderrodrigues 
It is not really meant as build *performance* improvement, but rather a 
build stability patch.


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

Support for SecurityGroup in OpenVSwitch mode in Xenserver

[Suresh Ramamurthy]

Hi Security Group Experts,

I am trying to play with SecurityGroup in XenServer setup.

When I looked at the latest 4.5 code I found that the code expects Bridge
module to be present in Xenserver.

Is that true? Is Security Group supported using OpenVSwitch in Xenserver?

Thanks,
Suresh

Build failed in Jenkins: simulator-singlerun #1113

[jenkins]

See 

Changes:

[Gaurav Aradhye] CLOUDSTACK-8388: test_VirtualRouter_alerts.py - Fix wait time 
and pep8 issues

--
[...truncated 10490 lines...]
> Running query: drop database if exists `cloud_usage`
> Running query: create database `cloud_usage`
> Running query: GRANT ALL ON cloud_usage.* to 'cloud'@`localhost` 
identified by 'cloud'
> Running query: GRANT ALL ON cloud_usage.* to 'cloud'@`%` 
identified by 'cloud'
> Initializing database=cloudbridge with host=localhost port=3306 
username=cloud password=cloud
> Running query: drop database if exists `cloudbridge`
> Running query: create database `cloudbridge`
> Running query: GRANT ALL ON cloudbridge.* to 'cloud'@`localhost` 
identified by 'cloud'
> Running query: GRANT ALL ON cloudbridge.* to 'cloud'@`%` 
identified by 'cloud'
> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing SQL file at 

> Processing upgrade: com.cloud.upgrade.DatabaseUpgradeChecker
[INFO] 
[INFO] --- maven-site-plugin:3.3:attach-descriptor (attach-descriptor) @ 
cloud-developer ---
[INFO] 
[INFO] --- maven-install-plugin:2.3.1:install (default-install) @ 
cloud-developer ---
[INFO] Installing 
 
to 
/var/lib/jenkins/.m2/repository/org/apache/cloudstack/cloud-developer/4.6.0-SNAPSHOT/cloud-developer-4.6.0-SNAPSHOT.pom
[INFO] 
[INFO] BUILD SUCCESS
[INFO] 
[INFO] Total time: 1:57.004s
[INFO] Finished at: Thu Apr 16 01:21:33 EDT 2015
[INFO] Final Memory: 45M/174M
[INFO] 
[WARNING] The requested profile "simulator" could not be activated because it 
does not exist.
[simulator-singlerun] $ mvn -P developer -pl developer -Ddeploydb-simulator
[INFO] Scanning for projects...
[INFO] 
[INFO] 
[INFO] Building Apache CloudStack Developer Mode 4.6.0-SNAPSHOT
[INFO] 
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-developer ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] --- properties-maven-plugin:1.0-alpha-2:read-project-properties 
(default) @ cloud-developer ---
[WARNING] Ignoring missing properties file: 

[INFO] 
[INFO] --- maven-remote-resources-plugin:1.3:process (default) @ 
cloud-developer ---
[INFO] 
[INFO] --- maven-antrun-plugin:1.8:run (default) @ cloud-developer ---
[INFO] Executing tasks

main:
[INFO] Executed tasks
[INFO] 
[INFO] >>> exec-maven-plugin:1.2.1:java (create-schema-simulator) @ 
cloud-developer >>>
[INFO] 
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ 
cloud-developer ---
[INFO] Starting audit...
Audit done.

[INFO] 
[INFO] <<< exec-maven-plugin:1.2.1:java (create-schema-simulator) @ 
cloud-developer <<<
[INFO] 
[INFO

[GitHub] cloudstack pull request: CLOUDSTACK-8308-Adding-automation-test-ca...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/170#issuecomment-93653040
  
Adding Snapshot Test cases that are relating secondary
storage limits and snapshot limits

Test result after integrating review changes:

Storage and Snapshot Limit ... === TestName: 
test_01_storage_snapshots_limits | Status : SUCCESS ===
ok

--
Ran 1 test in 493.037s

OK



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

[GitHub] cloudstack pull request: CLOUDSTACK-8236-Test-case-storage-migrati...

[pritisarap12]

GitHub user pritisarap12 opened a pull request:

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

CLOUDSTACK-8236-Test-case-storage-migration-test-path

--Adding missing changes from storage migration testpath in codes.py

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

$ git pull https://github.com/pritisarap12/cloudstack 
CLOUDSTACK-8308-Adding-missing-changes-in-codes-for-storage-migration

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

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


commit 28cc61359cde8c3bea2a7dc5be7348b60a54554f
Author: pritisarap12 
Date:   2015-04-16T06:32:56Z

CLOUDSTACK-8236-Test-case-storage-migration-test-path
--Adding missing changes from storage migration testpath in codes.py




---
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-8236-Test-case-storage-migrati...

[pritisarap12]

Github user pritisarap12 commented on the pull request:

https://github.com/apache/cloudstack/pull/174#issuecomment-93656249
  
CLOUDSTACK-8236-Test-case-storage-migration-test-path:
   Adding missing changes from storage migration testpath in 
codes.py



---
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: Support for SecurityGroup in OpenVSwitch mode in Xenserver

[Jayapal Reddy Uradi]

Hi Suresh,

Yes, for security groups expects network mode 'bridge' for xenserver.
This is because the security group rules  iptables/ebatables in host filters on 
the bridge interfaces.

Please look at how we can achieve host level isolation of VM traffic for SG 
using openVswitch.

Thanks,
Jayapal

On 16-Apr-2015, at 10:14 AM, Suresh Ramamurthy 

 wrote:

> Hi Security Group Experts,
> 
> I am trying to play with SecurityGroup in XenServer setup.
> 
> When I looked at the latest 4.5 code I found that the code expects Bridge
> module to be present in Xenserver.
> 
> Is that true? Is Security Group supported using OpenVSwitch in Xenserver?
> 
> Thanks,
> Suresh

[GitHub] cloudstack pull request: CLOUDSTACK-8236-Test-case-storage-migrati...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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: List of freelance developers (Was: Re: searching for a cloudstack developer)

[Erik Weber]

On Wed, Apr 15, 2015 at 10:25 PM, Daan Hoogland 
wrote:

> On Wed, Apr 15, 2015 at 2:50 PM, Erik Weber  wrote:
> > terbolous
>
>
> is added
>
>
Thank you Daan

-- 
Erik