[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1145#issuecomment-161337904
  
Okay... no panic! I might have been some expunge global settings that is 
deployed on all of our test  environments. It has been reverted and I will run 
the tests again.

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-9074: Support shared networkin...

2015-12-02 Thread nvazquez
Github user nvazquez commented on the pull request:

https://github.com/apache/cloudstack/pull/1094#issuecomment-161320788
  
Sure, I couldn't find you mail address, mine is in my profile


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


[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/943#discussion_r46436064
  
--- Diff: server/test/async-job-component.xml ---
@@ -108,11 +108,7 @@
 
 
   
--- End diff --

Not sure about this, anyone using this?


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


[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1145#issuecomment-161310807
  
Ping @remibergsma @DaanHoogland @ustcweizhou @bhaisaab 

The test fails on latest 4.6 branch!

```
=== TestName: test_01_stop_vm | Status : SUCCESS ===

=== TestName: test_02_start_vm | Status : SUCCESS ===

=== TestName: test_03_reboot_vm | Status : SUCCESS ===

=== TestName: test_06_destroy_vm | Status : SUCCESS ===

=== TestName: test_07_restore_vm | Status : SUCCESS ===

=== TestName: test_09_expunge_vm | Status : FAILED ===
```

And here is the failure:

```
2015-12-02 14:12:47,260 - CRITICAL - FAILED: test_09_expunge_vm: 
['Traceback (most recent call last):\n', '  File 
"/usr/lib64/python2.7/unittest/case.py", line 369, in run\ntestMethod()\n', 
'  File 
"/data/git/cs2/cloudstack/test/integration/smoke/test_vm_life_cycle.py", line 
646, in test_09_expunge_vm\nself.assertEqual(list_vm_response,None,"Check 
Expunged virtual machine is in listVirtualMachines response")\n', '  File 
"/usr/lib64/python2.7/unittest/case.py", line 553, in assertEqual\n
assertion_func(first, second, msg=msg)\n', '  File 
"/usr/lib64/python2.7/unittest/case.py", line 546, in _baseAssertEqual\n
raise self.failureException(msg)\n', 'AssertionError: Check Expunged virtual 
machine is in listVirtualMachines response\n']

/tmp//MarvinLogs//Dec_02_2015_13_33_23_C9WHUF/failed_plus_exceptions.txt 
(END)
```


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-9074: Support shared networkin...

2015-12-02 Thread nvazquez
Github user nvazquez commented on the pull request:

https://github.com/apache/cloudstack/pull/1094#issuecomment-161315034
  
Thanks @miguelaferreira I really appreciate that! Thanks a lot for your 
help!
I'll try to make it before the deadline. I'm not that into Python but I'll 
do my best with Marvin tests


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


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

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/855#issuecomment-161352903
  
LGTM (only code review), but we need to test it and verify console proxy 
functionality


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


[GitHub] cloudstack pull request: Removed the PlannerBase class because it ...

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1108#issuecomment-161353281
  
We've used _var for global or class variables throughout the codebase, 
should we keep it?
LGTM (only code review) otherwise


---
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: Create test cases to getPatchFilePath met...

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/944#issuecomment-161352035
  
LGTM (just code review)


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


[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-12-02 Thread miguelaferreira
Github user miguelaferreira commented on the pull request:

https://github.com/apache/cloudstack/pull/1094#issuecomment-161317452
  
I'm working on adding a new test case to the test file we already have, but 
unfortunately I don't know how to cover your change. What I do know is how 
marvin works, and how a test case should look like, so please do ping me (here, 
or by mail) so that we can setup a chat somewhere and I'll be happy to help you.


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


[GitHub] cloudstack pull request: Removed unused code from the EngineHostDa...

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/942#issuecomment-161352291
  
LGTM (only code review)


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


[GitHub] cloudstack pull request: Add marvin test to verify if DomainAdmin ...

2015-12-02 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1122#issuecomment-161362360
  
```
=== TestName: test_24_public_IP_tag | Status : SUCCESS ===

Cleanup resources used
```

LGTM


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


Re: [VOTE] Apache CloudStack 4.6.1

2015-12-02 Thread Daan Hoogland
I am having a problem, When trying to add a vpc from  the UI the zone
dropdown is empty. when trying to add an isolated network the dropdown does
contain the wanted choice. Any body else seeing this?


On Mon, Nov 30, 2015 at 11:12 PM, Remi Bergsma 
wrote:

> Hi all,
>
> I've created a 4.6.1 release candidate, 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=4.6.1-RC20151130T2258
>
> Commit: beb840fc1cdfa5ba3c76ce4071cf5c4b775362cd
>
> Source release (checksums and signatures are available at the same
> location):
> https://dist.apache.org/repos/dist/dev/cloudstack/4.6.1/
>
> PGP release keys (signed using A47DDC4F):
> 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


[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-9004: Add features to Hy...

2015-12-02 Thread jharshman
Github user jharshman commented on the pull request:

https://github.com/apache/cloudstack/pull/1013#issuecomment-161367693
  
@DaanHoogland from reading the mailing lists.  Looks like there is a freeze 
for 4.7 features starting Dec 7.  Can we get this change merged in before then? 
 The change is very simplistic but it lays some ground work for making 
CloudStack a bit more stable for windows guests.


---
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-8988: Removed unused adapters ...

2015-12-02 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/943#discussion_r46455242
  
--- Diff: server/test/async-job-component.xml ---
@@ -108,11 +108,7 @@
 
 
   
--- End diff --

@bhaisaab , probably note (I would say that no one uses it) that class does 
not exist.


---
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: Add marvin test to verify if DomainAdmin ...

2015-12-02 Thread asfgit
Github user asfgit closed the pull request at:

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


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


Build failed in Jenkins: build-master-jdk18 #418

2015-12-02 Thread jenkins
See 

Changes:

[sanjeev] Add marvin test for bug CS-38356 Bug-Id: CS-38356 Reviewed-By: Self

[sanjeev] Incorporated review comments from the PR

--
[...truncated 398 lines...]
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.039 sec - in 
com.cloud.utils.UuidUtilsTest
Running com.cloud.utils.crypto.RSAHelperTest
2015-12-02 17:08:09,638 INFO  [utils.crypt.RSAHelper] (main:) [ignored]error 
during public key encryption: Unsupported format
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.118 sec - in 
com.cloud.utils.crypto.RSAHelperTest
Running com.cloud.utils.crypto.EncryptionSecretKeyCheckerTest
2015-12-02 17:08:09,803 DEBUG [utils.crypt.EncryptionSecretKeyChecker] (main:) 
Encryption Type: null
2015-12-02 17:08:09,804 DEBUG [utils.crypt.EncryptionSecretKeyChecker] (main:) 
Encryption Type: file
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.054 sec - in 
com.cloud.utils.crypto.EncryptionSecretKeyCheckerTest
Running com.cloud.utils.PropertiesUtilsTest
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.033 sec - in 
com.cloud.utils.PropertiesUtilsTest
Running com.cloud.utils.exception.ExceptionUtilTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0 sec - in 
com.cloud.utils.exception.ExceptionUtilTest
Running com.cloud.utils.storage.QCOW2UtilsTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.002 sec - in 
com.cloud.utils.storage.QCOW2UtilsTest
Running com.cloud.utils.encoding.UrlEncoderTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.001 sec - in 
com.cloud.utils.encoding.UrlEncoderTest
Running com.cloud.utils.UriUtilsTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.057 sec - in 
com.cloud.utils.UriUtilsTest
Running com.cloud.utils.HttpUtilsTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.12 sec - in 
com.cloud.utils.HttpUtilsTest
Running com.cloud.utils.backoff.impl.ConstantTimeBackoffTest
2015-12-02 17:08:10,165 INFO  [backoff.impl.ConstantTimeBackoff] (Thread-1:) 
Thread Thread-1 interrupted while waiting for retry
2015-12-02 17:08:10,268 DEBUG [backoff.impl.ConstantTimeBackoffTest] (main:) 
thread started
2015-12-02 17:08:10,269 DEBUG [backoff.impl.ConstantTimeBackoffTest] 
(Thread-2:) before
2015-12-02 17:08:10,370 DEBUG [backoff.impl.ConstantTimeBackoffTest] (main:) 
testing wakeup
Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.333 sec - in 
com.cloud.utils.backoff.impl.ConstantTimeBackoffTest
Running com.cloud.utils.ProcessUtilTest
2015-12-02 17:08:10,370 INFO  [backoff.impl.ConstantTimeBackoff] (Thread-2:) 
Thread Thread-2 interrupted while waiting for retry
2015-12-02 17:08:10,371 DEBUG [backoff.impl.ConstantTimeBackoffTest] 
(Thread-2:) after
2015-12-02 17:08:10,379 DEBUG [cloud.utils.ProcessUtil] (main:) 
environment.properties could not be opened
2015-12-02 17:08:10,385 DEBUG [cloud.utils.ProcessUtil] (main:) 
environment.properties could not be opened
2015-12-02 17:08:10,385 DEBUG [cloud.utils.ProcessUtil] (main:) Executing: bash 
-c ps -p 123456 
2015-12-02 17:08:10,472 DEBUG [cloud.utils.ProcessUtil] (main:) Exit value is 1
2015-12-02 17:08:10,473 DEBUG [cloud.utils.ProcessUtil] (main:)   PID TTY   
   TIME CMD
2015-12-02 17:08:10,475 DEBUG [cloud.utils.ProcessUtil] (main:) Executing: bash 
-c echo $PPID 
2015-12-02 17:08:10,478 DEBUG [cloud.utils.ProcessUtil] (main:) Execution is 
successful.
2015-12-02 17:08:10,487 DEBUG [cloud.utils.ProcessUtil] (main:) 
environment.properties could not be opened
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.117 sec - in 
com.cloud.utils.ProcessUtilTest
Running com.cloud.utils.PasswordGeneratorTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.008 sec - in 
com.cloud.utils.PasswordGeneratorTest
Running com.cloud.utils.rest.HttpUriRequestBuilderTest
Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.058 sec - in 
com.cloud.utils.rest.HttpUriRequestBuilderTest
Running com.cloud.utils.rest.HttpStatusCodeHelperTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.01 sec - in 
com.cloud.utils.rest.HttpStatusCodeHelperTest
Running com.cloud.utils.rest.RESTServiceConnectorTest
2015-12-02 17:08:10,854 DEBUG [utils.rest.RESTServiceConnector] (main:) 
Executing retrieve object on /somepath
2015-12-02 17:08:10,854 DEBUG [utils.rest.BasicRestClient] (main:) Executig GET 
request on https://localhost:443/somepath
2015-12-02 17:08:10,862 DEBUG [utils.rest.RESTServiceConnector] (main:) 
Executed request: GET /somepath HTTP/1.1 status was HTTP/1.1 200 OK
2015-12-02 17:08:10,862 DEBUG [utils.rest.RESTServiceConnector] (main:) 
Response entity: [{somethig_not_type : "WrongType"}]
2015-12-02 17:08:10,904 DEBUG [utils.rest.BasicRestClient] (main:) Closing HTTP 
connection
2015-12-02 17:08:10,924 DEBUG 

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

2015-12-02 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/855#issuecomment-161389994
  
@bhaisaab, we definitely should test that. Since there were none references 
of that class, we suggested removing it. ACS uses spring, and since there is 
neither XML configuration nor annotation in the class to instantiate an object, 
we believe it is not used. Moreover, we did not find any docs describing its 
difference from the other ConsoleProxy class that is configured as a Spring 
bean. We also did not find any tutorial describing how to use it. 


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


[GitHub] cloudstack pull request: Removed the PlannerBase class because it ...

2015-12-02 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request:

https://github.com/apache/cloudstack/pull/1108#issuecomment-161388418
  
@bhaisaab, I assume the use of _ was introduced probably by someone that 
comes from a C/C++ background. IMHO that does not make much sense when coding 
in Java, I can dive deeper in an explanation if you like. Additionally, in most 
of the new codes, no one seems to be using the “_” anymore.


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


Build failed in Jenkins: build-4.5 #545

2015-12-02 Thread jenkins
See 

Changes:

[Rohit Yadav] debian: Partially pick change from #626 for getting version

--
[...truncated 2951 lines...]
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 3 resources
[INFO] Copying 3 resources
[INFO] 
[INFO] --- maven-compiler-plugin:2.5.1:testCompile (default-testCompile) @ 
cloud-plugin-network-internallb ---
[INFO] Compiling 6 source files to 

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


---
 T E S T S
---
Build timed out (after 11 minutes). Marking the build as failed.
Build was aborted
Recording test results
[INFO] 
[INFO] 
[INFO] Skipping Apache CloudStack Plugin - Network Netscaler
[INFO] This project has been banned from the build due to previous failures.
[INFO] 
[INFO] 
[INFO] 
[INFO] Skipping Apache CloudStack Plugin - Network Nicira NVP
[INFO] This project has been banned from the build due to previous failures.
[INFO] 
[INFO] 
[INFO] 
[INFO] Skipping Apache CloudStack Plugin - BigSwitch Virtual Network Segment
[INFO] This project has been banned from the build due to previous failures.
[INFO] 
[INFO] 
[INFO] 
[INFO] Skipping Apache CloudStack Plugin - Network Brocade VCS
[INFO] This project has been banned from the build due to previous failures.
[INFO] 
[INFO] 
[INFO] 
[INFO] Skipping Apache CloudStack Plugin - Stratosphere SSP
[INFO] This project has been banned from the build due to previous failures.
[INFO] 
[INFO] 
[INFO] 
[INFO] Skipping Apache CloudStack Plugin - Network Opendaylight
[INFO] This project has been banned from the build due to previous failures.
[INFO] 
[INFO] 
[INFO] 
[INFO] Skipping Apache CloudStack Plugin - User Authenticator LDAP
[INFO] This project has been banned from the build due to previous failures.
[INFO] 
[INFO] 
[INFO] 
[INFO] Skipping Apache CloudStack Plugin - User Authenticator MD5
[INFO] This project has been banned from the build due to previous failures.
[INFO] 
[INFO] 
[INFO] 
[INFO] Skipping Apache CloudStack Plugin - User Authenticator PBKDF2-SHA-256
[INFO] This project has been banned from the build due to previous failures.
[INFO] 
[INFO] 
[INFO] 
[INFO] Skipping Apache CloudStack Plugin - User Authenticator Plain Text
[INFO] This project has been banned from the build due to previous failures.
[INFO] 
[INFO] 

[4.7] Compiling with -Dnoredist fails for me

2015-12-02 Thread Mike Tutkowski
Hi,

Did I miss an e-mail about manually acquiring new VMware dependencies for
4.7 or something?

[INFO]

[INFO] Building Apache CloudStack VMware Base 4.7.0-SNAPSHOT
[INFO]

Downloading:
http://repo.maven.apache.org/maven2/com/cloud/com/vmware/vmware-vim25/5.5/vmware-vim25-5.5.pom
[WARNING] The POM for com.cloud.com.vmware:vmware-vim25:jar:5.5 is missing,
no dependency information available
Downloading:
http://repo.maven.apache.org/maven2/com/cloud/com/vmware/vmware-vim25/5.5/vmware-vim25-5.5.jar

Thanks!

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


[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1145#issuecomment-161430026
  
LGTM based on these tests:

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

```

Result:

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

--
Ran 33 tests in 16226.348s

OK
```


And:

```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a 
tags=advanced,required_hardware=false \
smoke/test_routers.py \
smoke/test_network_acl.py \
smoke/test_privategw_acl.py \
smoke/test_reset_vm_on_reboot.py \
smoke/test_vm_life_cycle.py \
smoke/test_vpc_vpn.py \
smoke/test_service_offerings.py \

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1134#issuecomment-161431929
  
@pdube will run some integration tests against this branch.


---
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: [DISCUSS] ACS 4.5.3 release

2015-12-02 Thread Rohit Yadav

On 03-Dec-2015, at 12:19 AM, Rohit Yadav 
> wrote:

So we got some things done on 4.5 lately;

Just to be clear - I meant cherry-picking, backporting commits, or 
reviewing/merging 4.5 PRs lately :)

Rohit Yadav
Software Architect

[cid:image003.png@01D122E8.F6EFE910]



S: +44 20 3603 0540 | M: +91 88 262 30892

rohit.ya...@shapeblue.com | 
www.shapeblue.com | 
Twitter:@ShapeBlue

ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS




Find out more about ShapeBlue and our range of CloudStack related services

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

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


[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1151#issuecomment-161428530
  
@wilderrodrigues Will start some tests!


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


[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-9022: keep Destroyed vol...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1029#issuecomment-161431453
  
@ustcweizhou PR #1145 is merged to 4.6, please close this PR. Thanks!


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


[GitHub] cloudstack pull request: [4.6/4.6.1] IMPORTANT for 4.6.1 release -...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1149#issuecomment-161429236
  
Makes sense, LGTM also based on the successful job above.


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


[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1145#issuecomment-161429404
  
@ustcweizhou Took me a while, but the issue is not in this PR. Will post 
test results soon.


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


[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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: [DISCUSS] ACS 4.5.3 release

2015-12-02 Thread Rohit Yadav
Hi all,

So we got some things done on 4.5 lately;

- L10N updates
- Fix missing DHCP entries
- Allow users to create multiple Guest Shared Networks in same Vlan ID
- Ensure power state is up to date when handling missing VMs in powerReport
- We got UI enhancements
- Security enhancements
- Packaging enhancements

@Stephan - we’ll try to get 
https://issues.apache.org/jira/browse/CLOUDSTACK-8936 fixed
@Bruno - applied the diff, thanks
@Rene - applied PRs, thanks

I’m aiming to start helping with the 4.5.3 release process after 4.7.0 RC1; 
possibly around 15Dec.
So, we have about 2 more weeks to fix bugs and test. Thanks.

Regards.

On 22-Nov-2015, at 3:59 AM, Milamber 
> wrote:

Hello Rohit,

I think that the PR 626 [1] on the debian/rules must be apply on the 4.5 branch 
to release 4.5.3 version to allow the good building of CS 4.53 on Ubuntu. 
(snapshot suffix issue on cloud-agent package)


$ git diff
diff --git a/debian/rules b/debian/rules
index 45461bb..78dd487 100755
--- a/debian/rules
+++ b/debian/rules
@@ -11,7 +11,7 @@
# build-arch and build-indep targets  by Bill Allombert 2001

DEBVERS := $(shell dpkg-parsechangelog | sed -n -e 's/^Version: //p')
-VERSION := $(shell echo '$(DEBVERS)' | sed -e 's/^[[:digit:]]*://' -e 
's/[~-].*//')
+VERSION := $(shell cat pom.xml |sed '22!d'| cut -d'>' -f2 |cut -d'<' -f1)
MVNADD := $(shell if echo '$(DEBVERS)' | grep -q snapshot; then echo ; fi )
PACKAGE = $(shell dh_listpackages|head -n 1|cut -d '-' -f 1)
SYSCONFDIR = "/etc"

Milamber

[1] https://github.com/apache/cloudstack/pull/626/files



On 20/11/2015 15:16, Stephan Seitz wrote:
Hi Rohit,

I've reported one Bug[1], that affects the network.throttling.rate. I
assume, this is just some bad calculation.

Maybe this could be fixed for 4.5.3? :)

[1] https://issues.apache.org/jira/browse/CLOUDSTACK-8936


Am Freitag, den 20.11.2015, 06:27 + schrieb Rohit Yadav:
Hi all,


I want to ask how happy people are with the last 4.5.2 release and if
there are any issues they want to report or want to be fixed in a
future minor release. If we’ve enough demand, we can work towards a
last 4.5 minor release. Thanks.


Rohit Yadav
Software Architect





D: +44 20 3642 6102 | S: +44 20 3603 0540 | M: +91 88 262 30892

 rohit.ya...@shapeblue.com | 
www.shapeblue.com | Twitter:@ShapeBlue
 ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS


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.




Rohit Yadav
Software Architect

[cid:image003.png@01D122E8.F6EFE910]



S: +44 20 3603 0540 | M: +91 88 262 30892

rohit.ya...@shapeblue.com | 
www.shapeblue.com | 
Twitter:@ShapeBlue

ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS




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 

[GitHub] cloudstack pull request: debian: allow rules to pick ACS_BUILD_OPT...

2015-12-02 Thread bhaisaab
GitHub user bhaisaab opened a pull request:

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

debian: allow rules to pick ACS_BUILD_OPTS from env

Only now debian builds can be noredist etc.

cc @remibergsma @wido  - I think the apt-get.eu 4.6.0 deb repo was oss and 
not noredist one. I also checked the jenkins job it does not "export 
ACS_BUILD_OPTS='-Dnoredist -Dnonoss'" before running dpkg-buildpackage

I discovered this while testing 4.6.1 RC1 today, nothing major just a build 
profile issue.

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

$ git pull https://github.com/shapeblue/cloudstack 4.6-debian-noredist-fix

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

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


commit 433a79f989a51f88713b130cd51182373139577e
Author: Rohit Yadav 
Date:   2015-12-02T08:15:18Z

debian: allow rules to pick ACS_BUILD_OPTS from env

Only now debian builds can be noredist etc.

Signed-off-by: Rohit Yadav 




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


[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1145#issuecomment-161223389
  
@ustcweizhou @bhaisaab @wido 

Did you actually test this PR?

Cheers,
Wilder


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


[GitHub] cloudstack pull request: Add marvin test to verify if DomainAdmin ...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1122#issuecomment-161225032
  
Thanks for writing this test, @sanju1010. It's much appreciated!

@remibergsma and you executed the test and I went through the code. So, it 
LGTM :+1: 

@remibergsma: feel free to merge whenever you have time. :)

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

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1086#issuecomment-161226000
  
@ustcweizhou @DaanHoogland 

I was on holidays last week... just got back to the PR-Pool. I saw that 
@DaanHoogland already tested, so I will do a code review and give my vote 
depending on the outcome.

Talk to you later.

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: [4.6/4.6.1] IMPORTANT for 4.6.1 release -...

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1149#issuecomment-161216838
  
Test jenkins build job: 
http://sb.bhaisaab.org:/job/deb-shapeblue-build-patch-noredist/103/console


---
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: [VOTE] Apache CloudStack 4.6.1

2015-12-02 Thread Rohit Yadav
+1 (binding)

Tested with debian build against Ubuntu based KVM host, and fresh installation:
- Basic and Adv zone deployment (without SG)
- SystemVMs came up, with 4.6.0 systemvm template (used)
- Normal VM life cycle in both isolated and VPC networks
- PF, Firewall rules tested
- Console proxy, secondary storage VM operations tested (template related)
- UI tested - sortable tables, wider UI width

Notes:
- Found and fixed a debian packaging issue, because of we could not build 
noredist debian pkgs: https://github.com/apache/cloudstack/pull/1149 (Remi/Wido 
- please review and merge; while we don’t need to break the voting we can 
include this fix if the vote passes)

Repo used:
http://packages.shapeblue.com/cloudstack/testing/4.6.1/ (debian repo has the 
above PR fix, hence pkgs are noredist ones)

Systemvm template used:
http://packages.shapeblue.com/systemvmtemplate/4.6/

Regards.

On 01-Dec-2015, at 3:42 AM, Remi Bergsma 
> wrote:

Hi all,

I've created a 4.6.1 release candidate, 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=4.6.1-RC20151130T2258

Commit: beb840fc1cdfa5ba3c76ce4071cf5c4b775362cd

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

PGP release keys (signed using A47DDC4F):
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 Yadav
Software Architect

[cid:image003.png@01D122E8.F6EFE910]



S: +44 20 3603 0540 | M: +91 88 262 30892

rohit.ya...@shapeblue.com | 
www.shapeblue.com | 
Twitter:@ShapeBlue

ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS




Find out more about ShapeBlue and our range of CloudStack related services

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

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


[GitHub] cloudstack pull request: Strongswan vpn feature

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/872#issuecomment-161222526
  
@remibergsma no promises but I'll try to meet that made up deadline :)


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


[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1145#issuecomment-161224861
  
@wilderrodrigues no, just code review


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


[GitHub] cloudstack pull request: Add marvin test to verify if DomainAdmin ...

2015-12-02 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1122#discussion_r46391214
  
--- Diff: test/integration/component/test_tags.py ---
@@ -2401,3 +2403,143 @@ def 
test_23_list_untagged_host_for_vm_migration(self):
 if not id in host_ids_for_migration:
 self.fail("Not all hosts are available for vm migration")
 return
+
+@attr(tags=["advanced"], required_hardware="false")
+def test_24_public_IP_tag(self):
+""" Testcreation, adding and removing tag on public IP address
+"""
+# Validate the following
+# 1. Create a domain and admin account under the new domain
+# 2. Create  a tag on acquired public IP address using createTags 
API
+# 3. Delete above created tag using deleteTags API
+# 4. Perform steps 3&4 using domain-admin
--- End diff --

:) will you update it?


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


[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-12-02 Thread miguelaferreira
Github user miguelaferreira commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1094#discussion_r46391389
  
--- Diff: 
plugins/network-elements/nicira-nvp/src/main/java/com/cloud/network/element/NiciraNvpElement.java
 ---
@@ -239,8 +244,51 @@ public boolean implement(Network network, 
NetworkOffering offering, DeployDestin
  * multiple operations that should be done only once.
  */
 
-// Implement SourceNat immediately as we have al the info already
-if 
(networkModel.isProviderSupportServiceInNetwork(network.getId(), 
Service.SourceNat, Provider.NiciraNvp)) {
+if (network.getGuestType().equals(GuestType.Shared)){
--- End diff --

Yes! Much better. Thanks!


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


[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-12-02 Thread miguelaferreira
Github user miguelaferreira commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1094#discussion_r46392407
  
--- Diff: 
plugins/network-elements/nicira-nvp/src/main/java/com/cloud/network/resource/wrapper/NiciraNvpConfigureSharedNetworkUuidCommandWrapper.java
 ---
@@ -0,0 +1,110 @@
+//
+// 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.
+//
+
+package com.cloud.network.resource.wrapper;
+
+import static com.cloud.network.resource.NiciraNvpResource.NAME_MAX_LEN;
+
+import java.util.ArrayList;
+import java.util.List;
+
+import org.apache.log4j.Logger;
+
+import com.cloud.agent.api.Answer;
+import com.cloud.agent.api.ConfigureSharedNetworkUuidAnswer;
+import com.cloud.agent.api.ConfigureSharedNetworkUuidCommand;
+import com.cloud.network.nicira.LogicalRouterPort;
+import com.cloud.network.nicira.LogicalSwitchPort;
+import com.cloud.network.nicira.NiciraNvpApi;
+import com.cloud.network.nicira.NiciraNvpApiException;
+import com.cloud.network.nicira.NiciraNvpTag;
+import com.cloud.network.nicira.PatchAttachment;
+import com.cloud.network.resource.NiciraNvpResource;
+import com.cloud.resource.CommandWrapper;
+import com.cloud.resource.ResourceWrapper;
+
+@ResourceWrapper(handles =  ConfigureSharedNetworkUuidCommand.class)
+public final class NiciraNvpConfigureSharedNetworkUuidCommandWrapper 
extends CommandWrapper{
+
+private static final Logger s_logger = 
Logger.getLogger(NiciraNvpConfigureSharedNetworkUuidCommandWrapper.class);
+
+@Override
+public Answer execute(ConfigureSharedNetworkUuidCommand command, 
NiciraNvpResource niciraNvpResource) {
+final String logicalRouterUuid = command.getLogicalRouterUuid();
+final String logicalSwitchUuid = command.getLogicalSwitchUuid();
+final String portIpAddress = command.getPortIpAddress();
+final List tags = new ArrayList();
+tags.add(new NiciraNvpTag("cs_account", command.getOwnerName()));
+final long networkId = command.getNetworkId();
+
+final NiciraNvpApi niciraNvpApi = 
niciraNvpResource.getNiciraNvpApi();
+
+s_logger.debug("Attaching Logical Switch " + logicalSwitchUuid + " 
on Logical Router " + logicalRouterUuid + " for Shared Network" + networkId);
+
+//Stored for rollback
+LogicalRouterPort lrpi = null;
+LogicalSwitchPort lsp = null;
+try {
+// Create the inside port for the router
+lrpi = new LogicalRouterPort();
+lrpi.setAdminStatusEnabled(true);
+lrpi.setDisplayName(niciraNvpResource.truncate(networkId + 
"-shared-att-port", NAME_MAX_LEN));
+lrpi.setTags(tags);
+final List ipAddresses = new ArrayList();
+ipAddresses.add(portIpAddress);
+lrpi.setIpAddresses(ipAddresses);
+lrpi = niciraNvpApi.createLogicalRouterPort(logicalRouterUuid, 
lrpi);
+
+try {
+// Create the inside port on the lswitch
+lsp = new 
LogicalSwitchPort(niciraNvpResource.truncate(networkId + "-shared-att-port", 
NAME_MAX_LEN), tags, true);
+lsp = 
niciraNvpApi.createLogicalSwitchPort(logicalSwitchUuid, lsp);
+
+// Attach the inside router port to the lswitch port with 
a PatchAttachment
+
niciraNvpApi.updateLogicalRouterPortAttachment(logicalRouterUuid, 
lrpi.getUuid(), new PatchAttachment(lsp.getUuid()));
+
+// Attach the inside lswitch port to the router with a 
PatchAttachment
+
niciraNvpApi.updateLogicalSwitchPortAttachment(logicalSwitchUuid, 
lsp.getUuid(), new PatchAttachment(lrpi.getUuid()));
+}
+catch (final NiciraNvpApiException e){
+try {
+

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1151#issuecomment-161434770
  
Ping @DaanHoogland @remibergsma @borisroman @bhaisaab 

More tests!

* Tests executed

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

* Results

```
Test router internal advanced zone ... === TestName: 
test_02_router_internal_adv | Status : SUCCESS ===
ok
Test restart network ... === TestName: test_03_restart_network_cleanup | 
Status : SUCCESS ===
ok
Test router basic setup ... === TestName: test_05_router_basic | Status : 
SUCCESS ===
ok
Test router advanced setup ... === TestName: test_06_router_advanced | 
Status : SUCCESS ===
ok
Test stop router ... === TestName: test_07_stop_router | Status : SUCCESS 
===
ok
Test start router ... === TestName: test_08_start_router | Status : SUCCESS 
===
ok
Test reboot router ... === TestName: test_09_reboot_router | Status : 
SUCCESS ===
ok
Test reset virtual machine on reboot ... === TestName: 
test_01_reset_vm_on_reboot | Status : SUCCESS ===
ok
Test advanced zone virtual router ... === TestName: 
test_advZoneVirtualRouter | Status : SUCCESS ===
ok
Test Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : 
SUCCESS ===
ok
Test Multiple Deploy Virtual Machine ... === TestName: 
test_deploy_vm_multiple | Status : SUCCESS ===
ok
Test Stop Virtual Machine ... === TestName: test_01_stop_vm | Status : 
SUCCESS ===
ok
Test Start Virtual Machine ... === TestName: test_02_start_vm | Status : 
SUCCESS ===
ok
Test Reboot Virtual Machine ... === TestName: test_03_reboot_vm | Status : 
SUCCESS ===
ok
Test destroy Virtual Machine ... === TestName: test_06_destroy_vm | Status 
: SUCCESS ===
ok
Test recover Virtual Machine ... === TestName: test_07_restore_vm | Status 
: SUCCESS ===
ok
Test migrate VM ... SKIP: At least two hosts should be present in the zone 
for migration
Test destroy(expunge) Virtual Machine ... === TestName: test_09_expunge_vm 
| Status : SUCCESS ===
ok
Test start/stop of router after addition of one guest network ... === 
TestName: test_01_start_stop_router_after_addition_of_one_guest_network | 
Status : SUCCESS ===
ok
Test reboot of router after addition of one guest network ... === TestName: 
test_02_reboot_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to change service offering of router after addition of one guest 
network ... === TestName: 
test_04_chg_srv_off_router_after_addition_of_one_guest_network | Status : 
SUCCESS ===
ok
Test destroy of router after addition of one guest network ... === 
TestName: test_05_destroy_router_after_addition_of_one_guest_network | Status : 
SUCCESS ===
ok
Test to stop and start router after creation of VPC ... === TestName: 
test_01_stop_start_router_after_creating_vpc | Status : SUCCESS ===
ok
Test to reboot the router after creating a VPC ... === TestName: 
test_02_reboot_router_after_creating_vpc | Status : SUCCESS ===
ok
Tests to change service offering of the Router after ... === TestName: 
test_04_change_service_offerring_vpc | Status : SUCCESS ===
ok
Test to destroy the router after creating a VPC ... === TestName: 
test_05_destroy_router_after_creating_vpc | Status : SUCCESS ===
ok
Test to create service offering ... === TestName: 
test_01_create_service_offering | Status : SUCCESS ===
ok
Test to update existing service offering ... === TestName: 
test_02_edit_service_offering | Status : SUCCESS ===
ok
Test to delete service offering ... === TestName: 
test_03_delete_service_offering | Status : SUCCESS ===
ok
Test create VPC offering ... === TestName: test_01_create_vpc_offering | 
Status : SUCCESS ===
ok
Test VPC offering without load balancing service ... === TestName: 
test_03_vpc_off_without_lb | Status : SUCCESS ===
ok
Test VPC offering without static NAT service ... === TestName: 
test_04_vpc_off_without_static_nat | Status : SUCCESS ===
ok
Test VPC offering without port forwarding service ... === TestName: 
test_05_vpc_off_without_pf | Status : SUCCESS ===
ok
Test VPC offering with invalid services ... === TestName: 
test_06_vpc_off_invalid_services | Status : SUCCESS ===
ok
Test update VPC offering ... === TestName: test_07_update_vpc_off | Status 
: SUCCESS ===
ok
Test list VPC offering ... === TestName: test_08_list_vpc_off | Status : 
SUCCESS ===
ok

[GitHub] cloudstack pull request: Quota

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/768#issuecomment-161449429
  
Had a first look at the feature and have some questions / remarks:

The quota icon is the only one in color and doesn't really fit the rest. Is 
that done on purpose (color/bigger)?

https://cloud.githubusercontent.com/assets/1630096/11545626/88f626b4-9948-11e5-9ab1-483d5d329938.png;>

Global setting `quota.enable.service` is `false` by default, but then it 
does display in the menu. One assumes it works, but you get errors like this:

https://cloud.githubusercontent.com/assets/1630096/11545666/d3db8e9e-9948-11e5-8297-57910d1b6c15.png;>

Flipping the setting to `true` makes this error go away. Maybe it is better 
to not display anything when the global setting is false? Or at least make 
clear it is not supposed to work.

Setting `quota.currency.symbol` defaults to `R`. Is that a sane default?

Now that the plugin is enabled I get some SQL errors so I probably miss 
some tables or such. I built a new environment from this branch. Tomorrow I'll 
investigate and read some more documentation on what I should to do set this up.

Nice feature, looking forward to try the rest.


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


[GitHub] cloudstack pull request: Removed unused code from the EngineHostDa...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/942#issuecomment-161433986
  
@DaanHoogland That expunge issue is now resolved in the integration tests, 
my bad :-s


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


[GitHub] cloudstack pull request: Quota

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/768#issuecomment-161451821
  
@bhaisaab The SQL changes required are implemented in 
`schema-461to470.sql`. When I build from scratch, I hit `schema-460to470.sql` 
and end up with a db without the tables I need. Do we need the SQL in there, 
too? Thanks for the help!


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


[GitHub] cloudstack pull request: Removed unused code from the EngineHostDa...

2015-12-02 Thread asfgit
Github user asfgit closed the pull request at:

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


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


[GitHub] cloudstack pull request: Quota

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/768#issuecomment-161436020
  
Sounds great guys!


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


[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-9022: keep Destroyed vol...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1029#issuecomment-161439739
  
@remibergsma
#1145 has conflict with master branch. you can merge this PR instead.


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


[GitHub] cloudstack pull request: [4.6/4.6.1] IMPORTANT for 4.6.1 release -...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1149#issuecomment-161437830
  
Ping @borisroman to have a look


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


[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1151#issuecomment-161484253
  
given the number of tests done I am fine with this. I also reviewed the 
code, lgtm, I am not confident to put it in capitals but am floating on my 
experience ;)
The exception is a known problem in jenkins and has no correlation 
whatsoever with the proposed code change:
```
org.apache.maven.InternalErrorException: Internal error: 
java.lang.IllegalStateException: Invalid object ID 14 iota=593
```


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


[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-9022: keep Destroyed vol...

2015-12-02 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1029#issuecomment-161484620
  
@ustcweizhou this PR is not needed as the code from 4.6 will be merged into 
master.


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


[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-9004: Add features to Hy...

2015-12-02 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1013#issuecomment-161480457
  
sure, that is the goal.
@borisroman @remibergsma @wilderrodrigues @wido @ustcweizhou Can you review?


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


CloudMonkey Import error in cloudmonkey.requester : cannot import name connection

2015-12-02 Thread Kenny Ho
Hi,

I am trying out CloudStack and trying to do some BareMetal deployment.
>From Jira, looks like the web UI is not ready for that so I turn to command
line option.  I found cloudmonkey, but on fresh install on first run, I got
this error:

$ sudo apt-get install python-pip
$ sudo pip install cloudmonkey
$ cloudmonkey
Import error in cloudmonkey.requester : cannot import name connection

Is this a known issue?  Is there a work around for this?

Regards,
Kenny


[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-9022: keep Destroyed vol...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou closed the pull request at:

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


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


[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-9022: keep Destroyed vol...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1029#issuecomment-161535590
  
@DaanHoogland @remibergsma OK, I close this PR, You need to fix the 
conflict manually when you merge 4.6 to master.


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


Re: [4.7] Compiling with -Dnoredist fails for me

2015-12-02 Thread Mike Tutkowski
Got it. It worked great - thanks!

On Wed, Dec 2, 2015 at 10:36 PM, Rohit Yadav 
wrote:

>
> On 03-Dec-2015, at 10:23 AM, Mike Tutkowski 
> wrote:
>
>
> Do you think I just need to copy vim25_55.jar
> 
> to
> deps and then run ./install-non-oss.sh?
>
>
> Whatever works for you, the install-non-oss.sh in the repo I shared will
> install all the non-oss deps you need.
>
> Rohit Yadav
> *Software Architect*
>
>
>
>
>
>
>
>
>
> * S: +44 20 3603 0540 <+442036030540> | M: +91 88 262 30892
> <+447770745036>  rohit.ya...@shapeblue.com
>  | www.shapeblue.com
>  | Twitter:@ShapeBlue
>  ShapeBlue Ltd, 53 Chandos Place, Covent
> Garden, London, WC2N 4HS *
>
> 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.
>



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


RE: [VOTE] Apache CloudStack 4.6.1

2015-12-02 Thread Paul Angus
@Daan, the zone dropdown is populated for me.
@Glenn, can you confirm whether is ok for you pls.

Regards,
Paul Angus
VP Technology/Cloud Architect
ShapeBlue Ltd
S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus
paul.an...@shapeblue.com | www.shapeblue.com | Twitter:@shapeblue
ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS





[ShapeBlue]
Paul Angus
VP Technology   ,   ShapeBlue


d:  +44 2036170528 |  t:  
@cloudyangus   |  m:  +44 
7711418784

e:  paul.an...@shapeblue.com|  
w:  www.shapeblue.com

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


[cid:image217c82.png@49558a73.46b52678]


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




-Original Message-
From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
Sent: Wednesday, December 2, 2015 5:13 PM
To: dev 
Subject: Re: [VOTE] Apache CloudStack 4.6.1

I am having a problem, When trying to add a vpc from the UI the zone dropdown 
is empty. when trying to add an isolated network the dropdown does contain the 
wanted choice. Any body else seeing this?


On Mon, Nov 30, 2015 at 11:12 PM, Remi Bergsma 
wrote:

> Hi all,
>
> I've created a 4.6.1 release candidate, 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=
> 4.6.1-RC20151130T2258
>
> Commit: beb840fc1cdfa5ba3c76ce4071cf5c4b775362cd
>
> Source release (checksums and signatures are available at the same
> location):
> https://dist.apache.org/repos/dist/dev/cloudstack/4.6.1/
>
> PGP release keys (signed using A47DDC4F):
> 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
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.


Re: [4.7] Compiling with -Dnoredist fails for me

2015-12-02 Thread Rohit Yadav
You may use this — https://github.com/bhaisaab/cloudstack-nonoss

git clone <>
./install-non-oss.sh

Regards.

On 03-Dec-2015, at 12:43 AM, Mike Tutkowski 
> wrote:

Hi,

Did I miss an e-mail about manually acquiring new VMware dependencies for
4.7 or something?

[INFO]

[INFO] Building Apache CloudStack VMware Base 4.7.0-SNAPSHOT
[INFO]

Downloading:
http://repo.maven.apache.org/maven2/com/cloud/com/vmware/vmware-vim25/5.5/vmware-vim25-5.5.pom
[WARNING] The POM for com.cloud.com.vmware:vmware-vim25:jar:5.5 is missing,
no dependency information available
Downloading:
http://repo.maven.apache.org/maven2/com/cloud/com/vmware/vmware-vim25/5.5/vmware-vim25-5.5.jar

Thanks!

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

Rohit Yadav
Software Architect

[cid:image003.png@01D122E8.F6EFE910]



S: +44 20 3603 0540 | M: +91 88 262 30892

rohit.ya...@shapeblue.com | 
www.shapeblue.com | 
Twitter:@ShapeBlue

ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS




Find out more about ShapeBlue and our range of CloudStack related services

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

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


[GitHub] cloudstack pull request: Quota

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/768#issuecomment-161530260
  
Let's ask @bhaisaab, we probably need to have the sql in both files to 
support both upgrade paths. Is there a better way?


---
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: CloudMonkey Import error in cloudmonkey.requester : cannot import name connection

2015-12-02 Thread Rohit Yadav
Hi Kenny,

On 03-Dec-2015, at 5:25 AM, Kenny Ho 
> wrote:

I am trying out CloudStack and trying to do some BareMetal deployment.
>From Jira, looks like the web UI is not ready for that so I turn to command
line option.  I found cloudmonkey, but on fresh install on first run, I got
this error:

$ sudo apt-get install python-pip
$ sudo pip install cloudmonkey
$ cloudmonkey
Import error in cloudmonkey.requester : cannot import name connection

I tested on Ubuntu 14.04.3 and it works for me. Can you try this:
(as root, or with sudo)

pip uninstall cloudmonkey
pip uninstall requests
apt-get update
apt-get upgrade
pip install --upgrade pip
pip install --upgrade cloudmonkey

In my case pip version was 7.1.2, cloudmonkey version was 5.3.2, python version 
was 2.7.6.

Regards.

Is this a known issue?  Is there a work around for this?  I am running
Ubuntu 14:
$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description: Ubuntu 14.04.3 LTS
Release: 14.04
Codename: trusty

$python --version
Python 2.7.6

Rohit Yadav
Software Architect

[cid:image003.png@01D122E8.F6EFE910]



S: +44 20 3603 0540 | M: +91 88 262 30892

rohit.ya...@shapeblue.com | 
www.shapeblue.com | 
Twitter:@ShapeBlue

ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS




Find out more about ShapeBlue and our range of CloudStack related services

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

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


[GitHub] cloudstack pull request: Quota

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/768#issuecomment-161539545
  
@remibergsma I'll test the upgrade/sql issue today to check for any issues 
and fix if needed.


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


[GitHub] cloudstack pull request: Quota

2015-12-02 Thread abhinandanprateek
Github user abhinandanprateek commented on the pull request:

https://github.com/apache/cloudstack/pull/768#issuecomment-161502434
  
@remibergsma since the feature was going in 4.7 the schema changes were put 
in 4.6.1. Moving them to 460-470 path will cater to any upgrade from 4.6.* to 
4.7, will make this change. 


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


[GitHub] cloudstack pull request: CLOUDSTACK-9087:adding projectid paramete...

2015-12-02 Thread nitt10prashant
GitHub user nitt10prashant opened a pull request:

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

CLOUDSTACK-9087:adding projectid parameter to create method of class 
VmSnapshot

Adding projectid parameter to class VmSnapshot of base.py .

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

$ git pull https://github.com/nitt10prashant/cloudstack mvr

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

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


commit bf24ad8099366a3f17bc4e3a94236a75a2e673de
Author: nitt10prashant 
Date:   2015-11-26T10:57:15Z

CLOUDSTACK-9087:adding projectid parameter to create method of class 
VmSnapshot




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


[GitHub] cloudstack pull request: Quota

2015-12-02 Thread abhinandanprateek
Github user abhinandanprateek commented on the pull request:

https://github.com/apache/cloudstack/pull/768#issuecomment-161534340
  
If the upgrade to 4.7, includes an upgrade to 4.6.1 then it can only be at 
one place and that seems to be a better option as it seems there are schema 
changes from 4.6-4.6.1.


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


[GitHub] cloudstack pull request: Quota

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/768#issuecomment-161539427
  
@remibergsma we should not have the alter statement both in sql file and 
the Java upgrade path. Since the putting it in SQL might throw exceptions 
causing the upgrade path execution to abort (in case of an existing column), we 
decided to put in the Java upgrade class with a try-catch (in case of an 
existing column, we simply ignore it).


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


[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1151#issuecomment-161528070
  
Ping @remibergsma @DaanHoogland @borisroman 

Before testing, please note that the smoke/test_privategw_acl.py now 
requires hardware!

* Environment:
  * ACS 4.6 branch
  * Hardware required: TRUE
  * Management Server + MySQL on CentOS 7.1
  * One KVM Host on CentOS 7.1
  * Agent + Common RPMs built from source

* Tests executed

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

* Results

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

Re: [4.7] Compiling with -Dnoredist fails for me

2015-12-02 Thread Mike Tutkowski
Thanks, Rohit

Do you think I just need to copy vim25_55.jar
 to
deps and then run ./install-non-oss.sh?

On Wed, Dec 2, 2015 at 9:49 PM, Rohit Yadav 
wrote:

> You may use this — https://github.com/bhaisaab/cloudstack-nonoss
>
> git clone <>
> ./install-non-oss.sh
>
> Regards.
>
> On 03-Dec-2015, at 12:43 AM, Mike Tutkowski 
> wrote:
>
> Hi,
>
> Did I miss an e-mail about manually acquiring new VMware dependencies for
> 4.7 or something?
>
> [INFO]
> 
> [INFO] Building Apache CloudStack VMware Base 4.7.0-SNAPSHOT
> [INFO]
> 
> Downloading:
>
> http://repo.maven.apache.org/maven2/com/cloud/com/vmware/vmware-vim25/5.5/vmware-vim25-5.5.pom
> [WARNING] The POM for com.cloud.com.vmware:vmware-vim25:jar:5.5 is missing,
> no dependency information available
> Downloading:
>
> http://repo.maven.apache.org/maven2/com/cloud/com/vmware/vmware-vim25/5.5/vmware-vim25-5.5.jar
>
> Thanks!
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkow...@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the cloud
> *™*
>
>
> Rohit Yadav
> *Software Architect*
>
>
>
>
>
>
>
>
>
> * S: +44 20 3603 0540 <+442036030540> | M: +91 88 262 30892
> <+447770745036>  rohit.ya...@shapeblue.com
>  | www.shapeblue.com
>  | Twitter:@ShapeBlue
>  ShapeBlue Ltd, 53 Chandos Place, Covent
> Garden, London, WC2N 4HS *
>
> 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.
>



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


[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1134#discussion_r46519549
  
--- Diff: 
server/src/org/apache/cloudstack/affinity/AffinityGroupServiceImpl.java ---
@@ -229,59 +205,99 @@ public AffinityGroupVO 
doInTransaction(TransactionStatus status) {
 return group;
 }
 });
+}
 
-if (s_logger.isDebugEnabled()) {
-s_logger.debug("Created affinity group =" + affinityGroupName);
+private DomainVO getDomain(Long domainId) {
+DomainVO domain = _domainDao.findById(domainId);
+if (domain == null) {
+throw new InvalidParameterValueException("Unable to find 
domain by specified id");
 }
+return domain;
+}
 
-return group;
+private void verifyAffinityGroupNameInUse(long accountId, long 
domainId, String affinityGroupName) {
+if (_affinityGroupDao.isNameInUse(accountId, domainId, 
affinityGroupName)) {
+throw new InvalidParameterValueException("Unable to create 
affinity group, a group with name " + affinityGroupName + " already exists.");
+}
+}
+
+private void verifyDomainLevelAffinityGroupName(boolean domainLevel, 
long domainId, String affinityGroupName) {
+if (domainLevel && 
_affinityGroupDao.findDomainLevelGroupByName(domainId, affinityGroupName) != 
null) {
+throw new InvalidParameterValueException("Unable to create 
affinity group, a group with name " + affinityGroupName + " already exists 
under the domain.");
+}
 }
 
 @DB
-@Override
 @ActionEvent(eventType = EventTypes.EVENT_AFFINITY_GROUP_DELETE, 
eventDescription = "Deleting affinity group")
-public boolean deleteAffinityGroup(Long affinityGroupId, String 
account, Long domainId, String affinityGroupName) {
+public boolean deleteAffinityGroup(Long affinityGroupId, String 
account, Long projectId, Long domainId, String affinityGroupName) {
+
+AffinityGroupVO group = getAffinityGroup(affinityGroupId, account, 
projectId, domainId, affinityGroupName);
 
+// check permissions
 Account caller = CallContext.current().getCallingAccount();
-Account owner = _accountMgr.finalizeOwner(caller, account, 
domainId, null);
+_accountMgr.checkAccess(caller, AccessType.OperateEntry, true, 
group);
--- End diff --

why not AccessType.ModifyProject ?
If AccessType.OperateEntry, all normal uses (not only admin) can delete the 
affinity group.


---
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: [VOTE] Apache CloudStack 4.6.1

2015-12-02 Thread Daan Hoogland
Ok, thanks guys, I'll keep looking at my installation.

It is a 4.6.0 upgraded to 4.6.1 for those who asked.

On Thu, Dec 3, 2015 at 8:23 AM, Paul Angus  wrote:

> @Daan, the zone dropdown is populated for me.
> @Glenn, can you confirm whether is ok for you pls.
>
> Regards,
> Paul Angus
> VP Technology/Cloud Architect
> ShapeBlue Ltd
> S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus
> paul.an...@shapeblue.com | www.shapeblue.com | Twitter:@shapeblue
> ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS
>
>
>
>
>
> [image: ShapeBlue]  Paul Angus VP Technology ,
> ShapeBlue d:  *+44 2036170528* <+44%202036170528>  |  t:  *@cloudyangus*
> <@cloudyangus>  |  m:  *+44 7711418784* <+44%207711418784> e:
> *paul.an...@shapeblue.com*   |  w:
> *www.shapeblue.com* 
> 53 Chandos Place, Covent Garden, London WC2N 4HS UK Shape Blue Ltd is a
> company incorporated in England & Wales. ShapeBlue Services India LLP is a
> company incorporated in India and is operated under license from Shape Blue
> Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil
> and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is
> a company registered by The Republic of South Africa and is traded under
> license from Shape Blue Ltd. ShapeBlue is a registered trademark.
> This email and any attachments to it may be confidential and are intended
> solely for the use of the individual to whom it is addressed. Any views or
> opinions expressed are solely those of the author and do not necessarily
> represent those of Shape Blue Ltd or related companies. If you are not the
> intended recipient of this email, you must neither take any action based
> upon its contents, nor copy or show it to anyone. Please contact the sender
> if you believe you have received this email in error.
>
>
> -Original Message-
> From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
> Sent: Wednesday, December 2, 2015 5:13 PM
> To: dev 
> Subject: Re: [VOTE] Apache CloudStack 4.6.1
>
> I am having a problem, When trying to add a vpc from the UI the zone
> dropdown is empty. when trying to add an isolated network the dropdown does
> contain the wanted choice. Any body else seeing this?
>
>
> On Mon, Nov 30, 2015 at 11:12 PM, Remi Bergsma <
> rberg...@schubergphilis.com>
> wrote:
>
> > Hi all,
> >
> > I've created a 4.6.1 release candidate, 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=
> > 4.6.1-RC20151130T2258
> >
> > Commit: beb840fc1cdfa5ba3c76ce4071cf5c4b775362cd
> >
> > Source release (checksums and signatures are available at the same
> > location):
> > https://dist.apache.org/repos/dist/dev/cloudstack/4.6.1/
> >
> > PGP release keys (signed using A47DDC4F):
> > 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
> 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.
>



-- 
Daan


Re: [4.7] Compiling with -Dnoredist fails for me

2015-12-02 Thread Rohit Yadav

On 03-Dec-2015, at 10:23 AM, Mike Tutkowski 
> wrote:


Do you think I just need to copy vim25_55.jar
 to
deps and then run ./install-non-oss.sh?

Whatever works for you, the install-non-oss.sh in the repo I shared will 
install all the non-oss deps you need.

Rohit Yadav
Software Architect

[cid:image003.png@01D122E8.F6EFE910]



S: +44 20 3603 0540 | M: +91 88 262 30892

rohit.ya...@shapeblue.com | 
www.shapeblue.com | 
Twitter:@ShapeBlue

ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS




Find out more about ShapeBlue and our range of CloudStack related services

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

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


[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1134#issuecomment-161537378
  
LGTM. tested 
(1) create affinity group in project
(2) deploy vm with affinity group
(3) update vm affinity group
(4) delete affinity group.

one suggestion,
```
diff --git 
a/server/src/org/apache/cloudstack/affinity/AffinityGroupServiceImpl.java 
b/server/src/org/apache/cloudstack/affinity/AffinityGroupServiceImpl.java
index 5da1f88..e2de220 100644
--- 
a/server/src/org/apache/cloudstack/affinity/AffinityGroupServiceImpl.java
+++ 
b/server/src/org/apache/cloudstack/affinity/AffinityGroupServiceImpl.java
@@ -434,7 +434,7 @@ public class AffinityGroupServiceImpl extends 
ManagerBase implements AffinityGro
 throw new InvalidParameterValueException("Unable to find 
affinity group by id " + affinityGroupId);
 } else {
 // verify permissions
-_accountMgr.checkAccess(caller, null, true, owner, ag);
+_accountMgr.checkAccess(caller, AccessType.OperateEntry, 
true, owner, ag);
 // Root admin has access to both VM and AG by default, but 
make sure the
 // owner of these entities is same
 if (caller.getId() == Account.ACCOUNT_ID_SYSTEM || 
_accountMgr.isRootAdmin(caller.getId())) {
```


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


CloudMonkey Import error in cloudmonkey.requester : cannot import name connection

2015-12-02 Thread Kenny Ho
Hi,

I am trying out CloudStack and trying to do some BareMetal deployment.
>From Jira, looks like the web UI is not ready for that so I turn to command
line option.  I found cloudmonkey, but on fresh install on first run, I got
this error:

$ sudo apt-get install python-pip
$ sudo pip install cloudmonkey
$ cloudmonkey
Import error in cloudmonkey.requester : cannot import name connection

Is this a known issue?  Is there a work around for this?  I am running
Ubuntu 14:
$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description: Ubuntu 14.04.3 LTS
Release: 14.04
Codename: trusty

$python --version
Python 2.7.6

Regards,
Kenny


[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-12-02 Thread miguelaferreira
Github user miguelaferreira commented on the pull request:

https://github.com/apache/cloudstack/pull/1094#issuecomment-161235643
  
@nvazquez I have one more request for you, and (if you agree, of course) 
that should be the end of it on a code level. With the marvin test then this 
would be ready for 4.7, but be aware that feature freeze is planned for 
December 7, so we need to work fast.
However, if we do not make that deadline we can always include this feature 
in 4.8 (as you have probably seen, we are releaseing more often these days 
:smile:)

I have to say I've never really liked reviewing code for other people, but 
it has been a pleasure to do it with you :+1:.


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


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

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1073#issuecomment-161242475
  
today one of my nodes was rebooted  unexpectedly, I got the same error

2015-12-02 09:59:53,224 ERROR [c.c.n.r.VirtualNetworkApplianceManagerImpl] 
(RedundantRouterStatusMonitor-9:ctx-417ced5f) Fail to complete the 
RvRStatusUpdateTask!
java.lang.NullPointerException
at 
com.cloud.network.router.VirtualNetworkApplianceManagerImpl.updateRoutersRedundantState(VirtualNetworkApplianceManagerImpl.java:1019)
at 
com.cloud.network.router.VirtualNetworkApplianceManagerImpl$RvRStatusUpdateTask.runInContext(VirtualNetworkApplianceManagerImpl.java:1201)
at 
org.apache.cloudstack.managed.context.ManagedContextRunnable$1.run(ManagedContextRunnable.java:49)
at 
org.apache.cloudstack.managed.context.impl.DefaultManagedContext$1.call(DefaultManagedContext.java:56)
at 
org.apache.cloudstack.managed.context.impl.DefaultManagedContext.callWithContext(DefaultManagedContext.java:103)
at 
org.apache.cloudstack.managed.context.impl.DefaultManagedContext.runWithContext(DefaultManagedContext.java:53)
at 
org.apache.cloudstack.managed.context.ManagedContextRunnable.run(ManagedContextRunnable.java:46)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:745)


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


[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1145#issuecomment-161243997
  
@wilderrodrigues I just tested. 

I set the storage.cleanup.interval to 60s, the storage.cleanup.delay to 
120s. 

I restored a VM, then the old volume of the vm will be in 'Destroy' state

```
mysql> select id,name,uuid,created,updated,removed,state from volumes where 
id=7574\G
*** 1. row ***
 id: 7574
   name: ROOT-7558
   uuid: 82c5991d-5211-4fd8-bd8b-f1285cc50574
created: 2015-11-30 08:27:36
updated: 2015-12-02 09:50:34
removed: NULL
  state: Destroy
1 row in set (0.00 sec)
```

The volume is expunged after 2-3 minutes.

```
mysql> select id,name,uuid,created,updated,removed,state from volumes where 
id=7574\G
*** 1. row ***
 id: 7574
   name: ROOT-7558
   uuid: 82c5991d-5211-4fd8-bd8b-f1285cc50574
created: 2015-11-30 08:27:36
updated: 2015-12-02 09:53:31
removed: 2015-12-02 09:53:31
  state: Expunged
1 row in set (0.00 sec)
```


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


[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1145#issuecomment-161249455
  
@remibergsma @wilderrodrigues checked the code (java and 
test_vm_life_cycle.py), I think this PR is not related to the failure in 
integration test. Can you please run testing again?




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


[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1145#issuecomment-161250671
  
test/integration/component/test_browse_volumes.py
this test might be impacted.



---
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: [GitHub] cloudstack pull request: Add marvin test to verify if DomainAdmin ...

2015-12-02 Thread Sanjeev N
@daanhoogland I have updated the validation steps.

On Wed, Dec 2, 2015 at 2:46 PM, DaanHoogland  wrote:

> Github user DaanHoogland commented on a diff in the pull request:
>
> https://github.com/apache/cloudstack/pull/1122#discussion_r46391214
>
> --- Diff: test/integration/component/test_tags.py ---
> @@ -2401,3 +2403,143 @@ def
> test_23_list_untagged_host_for_vm_migration(self):
>  if not id in host_ids_for_migration:
>  self.fail("Not all hosts are available for vm
> migration")
>  return
> +
> +@attr(tags=["advanced"], required_hardware="false")
> +def test_24_public_IP_tag(self):
> +""" Testcreation, adding and removing tag on public IP address
> +"""
> +# Validate the following
> +# 1. Create a domain and admin account under the new domain
> +# 2. Create  a tag on acquired public IP address using
> createTags API
> +# 3. Delete above created tag using deleteTags API
> +# 4. Perform steps 3&4 using domain-admin
> --- End diff --
>
> :) will you update it?
>
>
> ---
> If your project is set up for it, you can reply to this email and have your
> reply appear on GitHub as well. If your project does not have this feature
> enabled and wishes so, or if the feature is enabled but not working, please
> contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
> with INFRA.
> ---
>


[GitHub] cloudstack pull request: CLOUDSTACK-8968: UI icon over VM snapshot...

2015-12-02 Thread nitin-maharana
GitHub user nitin-maharana opened a pull request:

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

CLOUDSTACK-8968: UI icon over VM snapshot to deploy user instance

Added a new Icon in Instance page to launch the VM from the snapshot.

A new icon over VM snapshot object, which upon invoked will open a deploy 
user instance wizard, where there is no choice to choose zone and ISO or 
template to deploy the user instance from.

1) A new icon to indicate deploy user instance from VM snapshot
2) The new icon is placed as an operation over each VM snapshot object.
3) Clicking this icon will show the wizard to deploy user instance, which 
is same as wizard for normal user instance except that zone selection page and 
ISO/template selection page would not be present in this wizard.

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

$ git pull https://github.com/nitin-maharana/CloudStack CloudStack-Nitin12

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

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


commit 0fddcc663e201edaff7ed80eb09cf2e1197ae08c
Author: Nitin Kumar Maharana 
Date:   2015-10-19T19:07:56Z

CLOUDSTACK-8968: UI icon over VM snapshot to deploy user instance

Added a new Icon in Instance page to launch the VM from the snapshot.




---
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-9069: Newly added project is n...

2015-12-02 Thread nitin-maharana
Github user nitin-maharana commented on the pull request:

https://github.com/apache/cloudstack/pull/1082#issuecomment-161256224
  
I checked in Internet Explorer and Chromium browser also. It is working 
fine.


---
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: Add marvin test to verify if DomainAdmin ...

2015-12-02 Thread DaanHoogland
Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1122#issuecomment-161260733
  
thanks, i'll retest after lunch and merge


---
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-8968: UI icon over VM snapshot...

2015-12-02 Thread nitin-maharana
Github user nitin-maharana commented on the pull request:

https://github.com/apache/cloudstack/pull/953#issuecomment-161262674
  
Made a pull request to merge in 4.6. So closing this PR. Thanks.


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


[GitHub] cloudstack pull request: CLOUDSTACK-8968: UI icon over VM snapshot...

2015-12-02 Thread nitin-maharana
Github user nitin-maharana closed the pull request at:

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


---
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-9069: Newly added project is n...

2015-12-02 Thread nitin-maharana
Github user nitin-maharana closed the pull request at:

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


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


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

2015-12-02 Thread jenkins
See 

--
[...truncated 797 lines...]
2015-12-02 08:12:07,164 DEBUG [utils.script.Script] (main:) System resource: 
null
2015-12-02 08:12:07,165 DEBUG [utils.script.Script] (main:) Classpath resource: 
null
2015-12-02 08:12:07,166 DEBUG [utils.script.Script] (main:) Executing: 
/bin/bash -c echo 'hello world!' 
2015-12-02 08:12:07,168 DEBUG [utils.script.Script] (main:) Execution is 
successful.
2015-12-02 08:12:07,169 WARN  [utils.script.Script] (main:) Exception: 
/bin/bash -c echo 'hello world!' 
java.lang.IllegalArgumentException
at com.cloud.utils.ScriptTest$1.interpret(ScriptTest.java:107)
at com.cloud.utils.script.Script.execute(Script.java:223)
at 
com.cloud.utils.ScriptTest.executeWithOutputInterpreter(ScriptTest.java:103)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:606)
at 
org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:47)
at 
org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at 
org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:44)
at 
org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:271)
at 
org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:70)
at 
org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:50)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:238)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:63)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:236)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:53)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:229)
at org.junit.runners.ParentRunner.run(ParentRunner.java:309)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:283)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.executeWithRerun(JUnit4Provider.java:173)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:153)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:128)
at 
org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:203)
at 
org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:155)
at 
org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:103)
2015-12-02 08:12:07,172 DEBUG [utils.script.Script] (main:) Executing: 
/bin/bash -c /not/existing/scripts/1449043927172 
2015-12-02 08:12:07,174 DEBUG [utils.script.Script] (main:) Exit value is 127
2015-12-02 08:12:07,175 DEBUG [utils.script.Script] (main:) /bin/bash: 
/not/existing/scripts/1449043927172: No such file or directory
2015-12-02 08:12:07,176 DEBUG [utils.script.Script] (main:) Executing: 
/bin/bash -c echo 'hello world!' 
2015-12-02 08:12:07,178 DEBUG [utils.script.Script] (main:) Execution is 
successful.
2015-12-02 08:12:07,179 DEBUG [utils.script.Script] (main:) Executing: 
/bin/bash -c echo 'hello world!' 
2015-12-02 08:12:07,181 DEBUG [utils.script.Script] (main:) Execution is 
successful.
Tests run: 10, Failures: 0, Errors: 0, Skipped: 1, Time elapsed: 0.044 sec - in 
com.cloud.utils.ScriptTest
Running com.cloud.utils.TestProfiler
Configure log4j with default properties
2015-12-02 08:12:07,207 INFO  [cloud.utils.TestProfiler] (main:) testProfiler() 
started
2015-12-02 08:12:08,207 INFO  [cloud.utils.TestProfiler] (main:) Duration in 
Millis: 1000
2015-12-02 08:12:08,207 INFO  [cloud.utils.TestProfiler] (main:) testProfiler() 
stopped
Configure log4j with default properties
Configure log4j with default properties
2015-12-02 08:12:09,229 INFO  [cloud.utils.TestProfiler] (main:) Duration in 
Nano: 1000138018
Configure log4j with default properties
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 2.047 sec - in 
com.cloud.utils.TestProfiler
Running com.cloud.utils.net.NetUtilsTest
2015-12-02 08:12:09,273 INFO  [utils.net.NetUtils] (main:) Invalid value of 
cidr 10.3.6.5/50
2015-12-02 08:12:09,280 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::2
2015-12-02 08:12:09,282 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::1
2015-12-02 08:12:09,283 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::2
2015-12-02 08:12:09,284 INFO  [utils.net.NetUtilsTest] (main:) IP is 
1234:5678::1
2015-12-02 08:12:09,285 INFO  [utils.net.NetUtilsTest] (main:) IP is 

[GitHub] cloudstack pull request: CLOUDSTACK-9069: Newly added project is n...

2015-12-02 Thread nitin-maharana
Github user nitin-maharana commented on the pull request:

https://github.com/apache/cloudstack/pull/1077#issuecomment-161263377
  
Closing this PR. As #1082 is there to merge in 4.6. Thanks.


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


[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1145#issuecomment-161270573
  
@wilderrodrigues I'm approaching EOD today, but can help test master 
tomorrow.
I tested vm lifecycle manually with 4.6.1 RC1 and KVM, so I suppose 4.6 
should be stable.


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


[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread wilderrodrigues
GitHub user wilderrodrigues opened a pull request:

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

[4.6] CLOUDSTACK-9075 - As a Developer I want the Private GW feature fixed 
on single VPCs

This PR fixes the issue we faced with Private Gateways on single VPC when 
using ACS 4.6.0 and onwards.

The root cause: during the VR refactor, the static routes configuration was 
left unimplemented.

This PR also improves the existing Replace ACL test and adds a new test, 
that cover the Private Gateway in a more complete way.

The new test does the following:

1. Create 2 VPCs
2. Create 2 Tiers - 1 per VPC
3. Deploy 2 VMs - 1 per Tier
4. Acquire 2 pub IPs - 1 per VPC
5. Create 2 PF rules - 1 per pub IP
6. Create 2 ACLs + rules - 1 per VPC
7. Assign new ACLs to Tiers
8. Create 2 Private GWs - 1 per VPC
9. Replace the Pvt GWs ACLs
10. Create 2 Static routes - 1 per Pvt GW
11. SSH into VM1 (VPC1) and from there ping VM2 (VPC2)

There is also a test for Private Gateways on Redundant VPCs. But I found 
out that the feature is broken in when used with rVPCs. It will be addressed in 
a separate Issue/PR.

I'm running the tests. Will post results as soon as they are ready.

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

$ git pull https://github.com/ekholabs/cloudstack 
fix/private_gw_rVPC-CLOUDSTACK-9075

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

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


commit 4ea4e7e687527cc8c06489d4deaedf4ed1c3c91c
Author: Wilder Rodrigues 
Date:   2015-11-19T11:03:32Z

CLOUDSTACK-9075 - Add method to get list of Physical Networks per zone

commit 3e02b8999bb7f1f79d99b32955c842fbda4d29a9
Author: Wilder Rodrigues 
Date:   2015-11-19T11:04:01Z

CLOUDSTACK-9075 - Covers Private GW ACL with Redundant VPCs

commit a17fa48de1cfa4f0f4425f90b0f435d6cf8e6540
Author: Wilder Rodrigues 
Date:   2015-11-19T11:28:32Z

CLOUDSTACK-9075 - Adds VPC static routes test

   - Adds redundant VPC tests
   - Adds support to Static Routes on VPC private gatways
   - Removes the route configuration in case static route is deleted.

commit 6d9a3d82f9b617d40d0ee1472bef87cb630595d6
Author: Wilder Rodrigues 
Date:   2015-12-02T06:30:06Z

CLOUDSTACK-9075 - Uses the same vlan since it should have been already 
released

- After the first test is done, the clean up will delete the whole VPC, 
also releasing the VLAN that was in use.




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


[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1145#issuecomment-161266056
  
@remibergsma  @DaanHoogland @wido @bhaisaab @ustcweizhou @karuturi @runseb 
@borisroman @jburwell 

If test_vm_life_cycle.py is failing because the current 4.6 and/or master 
is broken/unstable, we have to get it working again before any other PR gets 
merged. 

We cannot, for any imaginable reason, get into the situation we were at few 
months ago.

I will run tests against current 4.6 branch and Master to see if that's 
actually broken. Who can help me on testing as well?

Cheers,
Wilder


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


[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1145#issuecomment-161272059
  
Thanks, @bhaisaab!

I will test it and and also see if I have time tomorrow to run the same 
tests agains this PR. We need to be 100% sure that it works on both branches 
and with this PR.

Cheers,
Wilder


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


[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1151#issuecomment-161274671
  
Ping @remibergsma @DaanHoogland @borisroman 

You can start testing now... just a matter of running the following:

```
cd test/integration

nosetests --with-marvin --marvin-config=[your configuration here]-s -a 
tags=advanced,required_hardware=true smoke/test_privategw_acl.py
```

I will post my results soon.

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-9099: SecretKey is returned fr...

2015-12-02 Thread kansal
GitHub user kansal opened a pull request:

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

CLOUDSTACK-9099: SecretKey is returned from the APIs - Fixed

The current implementation of User and account management API (in general) 
return the secret key as a user or account response. 
Fix: Added a new API to explicitly return the secretKey and removed it from 
the user and account response.

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

$ git pull https://github.com/kansal/cloudstack CLOUDSTACK-9099

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

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


commit 410045a97a75fd1e43972c66bc4882a30a5098bf
Author: Kshitij Kansal 
Date:   2015-12-02T10:43:45Z

CLOUDSTACK-9099: SecretKey is returned from the APIs - Fixed




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


[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1151#issuecomment-161290470
  
Ping @remibergsma @DaanHoogland @borisroman 

Since I touched the router code, I also executed the rVPC tests to make 
sure those are working fine! I will now run more tests.

* Environment:
  * ACS 4.6 branch
  * Hardware required: TRUE
  * Management Server + MySQL on CentOS 7.1
  * One KVM Host on CentOS 7.1
  * Agent + Common RPMs built from source

* Test results:

```
test_01_vpc_privategw_acl 
(integration.smoke.test_privategw_acl.TestPrivateGwACL) ... === TestName: 
test_01_vpc_privategw_acl | Status : SUCCESS ===
ok
test_02_vpc_privategw_static_routes 
(integration.smoke.test_privategw_acl.TestPrivateGwACL) ... === TestName: 
test_02_vpc_privategw_static_routes | Status : SUCCESS ===
ok
test_03_rvpc_privategw_static_routes 
(integration.smoke.test_privategw_acl.TestPrivateGwACL) ... SKIP: Redundant VPC 
Routers have to be fixed. Private Gateway not working yet.
Create a redundant VPC with two networks with two VMs in each network ... 
=== TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : 
SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network and 
check default routes ... === TestName: test_02_redundant_VPC_default_routes | 
Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network ... 
=== TestName: 
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Status : 
SUCCESS ===
ok

--
Ran 6 tests in 5188.110s

OK (SKIP=1)
/tmp//MarvinLogs/test_vpc_redundant_2QYZ59/results.txt (END)
```


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


[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1151#issuecomment-161291156
  
Jenkins error is related to this:


![image](https://cloud.githubusercontent.com/assets/5129209/11532121/fa1f7274-9901-11e5-8e80-23c199c96a71.png)

Which doesn't ring any bell and seems completely unrelated.

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-6276 Fixing affinity groups fo...

2015-12-02 Thread resmo
Github user resmo commented on the pull request:

https://github.com/apache/cloudstack/pull/1134#issuecomment-161286193
  
Code LGTM, not yet tested though.


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


Jenkins build is back to normal : build-master-jdk18 #419

2015-12-02 Thread jenkins
See 



Re: [VOTE] Apache CloudStack 4.6.1

2015-12-02 Thread Pierre-Luc Dion
@daan, I don't see the problem on 2 env, could it be possible the zone is
disabled or in basic zone ?

on my side I'm having a glitch where creating a VM that support password in
a VPC does not get is password updated. but the same template in a Isolated
network does have is password set. not sure if it's related to 4.6.1 as I
have another 4.6.1 installed and does behave correctly... still testing...




On Wed, Dec 2, 2015 at 12:12 PM, Daan Hoogland 
wrote:

> I am having a problem, When trying to add a vpc from  the UI the zone
> dropdown is empty. when trying to add an isolated network the dropdown does
> contain the wanted choice. Any body else seeing this?
>
>
> On Mon, Nov 30, 2015 at 11:12 PM, Remi Bergsma <
> rberg...@schubergphilis.com>
> wrote:
>
> > Hi all,
> >
> > I've created a 4.6.1 release candidate, 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=4.6.1-RC20151130T2258
> >
> > Commit: beb840fc1cdfa5ba3c76ce4071cf5c4b775362cd
> >
> > Source release (checksums and signatures are available at the same
> > location):
> > https://dist.apache.org/repos/dist/dev/cloudstack/4.6.1/
> >
> > PGP release keys (signed using A47DDC4F):
> > 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
>