[GitHub] cloudstack pull request: [UI] fix bug: Cannot delete SSH keypairs ...

[ustcweizhou]

GitHub user ustcweizhou opened a pull request:

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

[UI] fix bug: Cannot delete SSH keypairs in projects

Before change:
error message: ProjectId and account/domainId can't be specified together

After change:
SSH keypairs can be removed

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

$ git pull https://github.com/ustcweizhou/cloudstack 
fix-ui-delete-sshkey-projects

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

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


commit e9cc5551d310c288a4b5ff748eee00951020f9a2
Author: Wei Zhou 
Date:   2015-12-03T08:07:48Z

[UI] fix bug: Cannot delete SSH keypairs in projects

Before change:
error message: ProjectId and account/domainId can't be specified together

After change:
SSH keypairs can be removed




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

[bhaisaab]

Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/768#issuecomment-161547150
  
@remibergsma I hope I've fixed the db issues and code, the db upgrade path 
implementation from 4.6.0/4.6.1 to 4.7.0 (on master) duplicates code. May be we 
can de-dupe it, just keep one set of class and sql scripts and use the other 
class to extend of the previous class (just changing the version, from to 
values).


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

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

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

Please, be aware that there is a change in the 
```tools/marvin/marvin/lib/common.py``` file. It means that you have to 
build/install Marvin from before running the new tests.

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

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/943#issuecomment-161551497
  
LGTM as these integration tests still pass:

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

```

Result:

```
Create a redundant VPC with two networks with two VMs in each network ... 
=== TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | 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
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

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

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/943#issuecomment-161552200
  
@DaanHoogland @rafaelweingartner The failed expunge test was a false alarm 
due to our testing environment. It passes again now. Please ping me when 
there's another review so I can merge. 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-9080: Resource limits for Prim...

[remibergsma]

Github user remibergsma commented on the pull request:

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

```
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
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 
===

[GitHub] cloudstack pull request: CLOUDSTACK-9080: Resource limits for Prim...

[asfgit]

Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-8868: use PasswordGenerator.ge...

[remibergsma]

Github user remibergsma commented on the pull request:

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

```
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
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 
===
 

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

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/943#issuecomment-161555776
  
LGTM, ran the tests as well and reviewed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-8868: use PasswordGenerator.ge...

[asfgit]

Github user asfgit closed the pull request at:

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


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

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1151#issuecomment-161557066
  
The test passes fine here, also on a 2 node cluster:


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.

--
Ran 3 tests in 842.201s
```

Will post integration 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] CLOUDSTACK-9075 - As a Developer I ...

[remibergsma]

Github user remibergsma commented on the pull request:

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

```
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
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 
===

[GitHub] cloudstack pull request: Create test cases to getPatchFilePath met...

[remibergsma]

Github user remibergsma commented on the pull request:

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

```
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
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 
===
 

[GitHub] cloudstack pull request: Create test cases to getPatchFilePath met...

[asfgit]

Github user asfgit closed the pull request at:

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


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

[remibergsma]

Github user remibergsma commented on the pull request:

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

```
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
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 
===

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

[asfgit]

Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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: Merge 4.6

[DaanHoogland]

GitHub user DaanHoogland opened a pull request:

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

Merge 4.6

there is one (simple) conflict so for formailty's sake I make this a pull 
request.
@utscweizhou en @remibergsma please have a look/test-run

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

$ git pull https://github.com/DaanHoogland/cloudstack merge-4.6

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

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


commit 9077c9a5b47224b2d00978ff67e9af6957ec589b
Author: Wei Zhou 
Date:   2015-11-03T16:04:55Z

CLOUDSTACK-9022: keep Destroyed volumes for sometime

commit 4ed1e0d5f809d1bb2845b6e076d55eeb1ce3f0f4
Author: Wei Zhou 
Date:   2015-11-04T08:27:32Z

CLOUDSTACK-9022: move storage.cleanup related global configurations to 
StorageManager

commit 5b7d935ab0b35bec936785e44023ac0d8552e72b
Author: Remi Bergsma 
Date:   2015-12-02T20:52:46Z

Merge pull request #1145 from ustcweizhou/storage-cleanup-delay-4.6

[4.6.1] CLOUDSTACK-9022: keep Destroyed volumes for sometimefor now, the 
Destroyed volumes will be expunged in Storage cleanup thread, no matter when 
they are destroyed.
In Expunging vm thread, we expunge the Destroyed vms which have been 
destroyed at least 'expunge.delay' seconds. We add the similar configuration 
for volumes.

same to #1029 , for 4.6

* pr/1145:
  CLOUDSTACK-9022: move storage.cleanup related global configurations to 
StorageManager
  CLOUDSTACK-9022: keep Destroyed volumes for sometime

This closes #1029

Signed-off-by: Remi Bergsma 

commit f9775de8ffcd0c07bfd7e670819e0142663cfde1
Author: Daan Hoogland 
Date:   2015-12-03T09:35:00Z

Merge release branch 4.6 to master

* 4.6:
  CLOUDSTACK-9022: move storage.cleanup related global configurations to 
StorageManager
  CLOUDSTACK-9022: keep Destroyed volumes for sometime

 Conflicts:
server/src/com/cloud/storage/StorageManagerImpl.java




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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: Merge 4.6

[ustcweizhou]

Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1155#issuecomment-161568869
  
LGTM, 
code are almost same to #1029 , 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: Merge 4.6

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1155#issuecomment-161569317
  
Will run some tests to double check.


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

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/768#issuecomment-161570202
  
@bhaisaab That sounds like a good solution. I will now test this PR again 
and let you know.


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

[wilderrodrigues]

Github user wilderrodrigues commented on the pull request:

https://github.com/apache/cloudstack/pull/1151#issuecomment-161595328
  
@remibergsma time to merge! It also got a LGTM from @DaanHoogland :)

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: Fixed issues with test_vpc_vpn script

[sanju1010]

GitHub user sanju1010 opened a pull request:

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

Fixed issues with test_vpc_vpn script

There are few issues with this test file. Please refer to JIRA Ticket 
https://issues.apache.org/jira/browse/CLOUDSTACK-9102 for details on the issues 
that have been fixed.

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

$ git pull https://github.com/sanju1010/cloudstack testvpn

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

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


commit 09409a7ac264e864c45146927b86229c8e67fbc8
Author: sanjeevn 
Date:   2015-12-03T10:26:08Z

Fixed issues with test_vpc_vpn script




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

[GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

[sanju1010]

Github user sanju1010 commented on the pull request:

https://github.com/apache/cloudstack/pull/1156#issuecomment-161599503
  
Test Remote Access VPN in VPC ... === TestName: test_vpc_remote_access_vpn 
| Status : SUCCESS ===
ok

--
Ran 1 test in 169.692s

OK
Test VPN in VPC ... === TestName: test_vpc_site2site_vpn | Status : FAILED 
===
FAIL

==
FAIL: Test VPN in VPC
--
Traceback (most recent call last):
  File 
"D:\CCPRepo\internal-cloudstack\test\integration\smoke\test_vpc_vpn.py", line 
695, in test_vpc_site2site_vpn
self.assert_(int(packet_loss) == 0, "Ping did not succeed")
AssertionError: Ping did not succeed
 >> begin captured stdout << -
=== TestName: test_vpc_site2site_vpn | Status : FAILED ===


The second test failure is not related to these changes. It is something to 
do with the connectivity issues between the vms.


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

[abhinandanprateek]

Github user abhinandanprateek commented on the pull request:

https://github.com/apache/cloudstack/pull/768#issuecomment-161604884
  
@remibergsma the new UI changes improve the UI and also take care of the 
issue you have noted above. If quota.enable.service is set to false then the 
quota icon will not appear on the UI.
Can make the icon black and white if that is the preference.


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

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

[SudharmaJain]

GitHub user SudharmaJain opened a pull request:

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

CLOUDSTACK-9100: ISO.CREATE/TEMPLATE.CREATE event missing for usage_e…

…vent by template sync thread

If there is a Management server restart while template is in downloading or 
installing state. Template Sync does not push event into usage_event table. 

I have verified the fix manually. Here is a snapshot.

![image](https://cloud.githubusercontent.com/assets/12229259/11559714/0989748c-99e1-11e5-88e0-48b4dbec0224.png)

 I have registered 4 templates. template id 207 and 208(ISO.CREATE event is 
missing) before applying the fix. and template id 209 and 210 after applying 
the fix.

Repro Steps (3 cases)
==
Case - 1 (private template)
-
1. register a private template/iso.
2. restart management server when template is in downloading state.
3. After management server restart, template_store_ref entry is removed if  
download was not yet completed.
4. on next management server restart, if download would have completed, 
template_store_ref entry will get populated, but TEMPLATE.CREATE event is 
missing in usage_event.

Case - 2 (Public template)

1.  register public template.
2. restart management server when in downloading state.
3. after restart template download reinitiates.
4. template goes to ready state, but there is no usage event.

case -3 (public/private template)
- 
1. register a template 
2. restart management server when template is in installing state.
3. after restart template goes to ready, but there is no usage event.



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

$ git pull https://github.com/SudharmaJain/cloudstack cs-9100

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

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


commit 912735a03086056e8e1e0d049492ead1bc0dfd3d
Author: SudharmaJain 
Date:   2015-12-03T08:41:14Z

CLOUDSTACK-9100: ISO.CREATE/TEMPLATE.CREATE event missing for usage_event 
by template sync thread




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-8892:If VR enters out-of band ...

[maneesha-p]

GitHub user maneesha-p opened a pull request:

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

CLOUDSTACK-8892:If VR enters out-of band context, routers lose their 
private/link-local IP

If router is powered-on out-of-band , we try to reboot it n times with some 
retry interval where n and the interval are configurable existing parameters 
whose default values are 5 and 10 min resp.

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

$ git pull https://github.com/maneesha-p/cloudstack CLOUDSTACK-8892-branch

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

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


commit 58faab09e337b0a953f69ee816a6d06fde444ad6
Author: Maneesha.P 
Date:   2015-09-16T04:47:51Z

CLOUDSTACK-8892:If VR enters out-of band context, routers lose their 
private/link-local IP




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

[bhaisaab]

Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/768#issuecomment-161622406
  
@remibergsma completed my internal tests around UI and backend 
implementation against a small cloud usage db (with a live environment), along 
with db upgrades (should work for both 4.6.0 and 4.6.1). It looks good to me.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-9103 : Missing OS Mappings for...

[maneesha-p]

GitHub user maneesha-p opened a pull request:

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

CLOUDSTACK-9103 : Missing OS Mappings for VMware 6.0

Added suitable db entries in tables hypervisor_capabilities and 
guest_os_hypervisor to support VMware 6.0 by copying from 5.5 and excluding few 
depricated guest os.

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

$ git pull https://github.com/maneesha-p/cloudstack CLOUDSTACK-9103

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

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


commit 9da64221d815825dad5d7b361ebb08584228b12b
Author: Maneesha.P 
Date:   2015-12-03T12:42:50Z

CLOUDSTACK-9103: Missing OS Mappings for VMware 6.0




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-8892:If VR enters out-of band ...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1158#issuecomment-161641914
  
-1

I don't think it is a good idea to reboot routers when out-of-band 
operation is detected. We discussed this before, let's not do it again. Please 
search the archives on this subject.

We use out-of-band migrations when patching and this way I can migrate 
1000s of routers out-of-band without issues. This is on XenServer and KVM. Your 
code change seems to make them reboot all, which would be a disaster. It's not 
even 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

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/768#issuecomment-161645283
  
@abhinandanprateek I will test it soon, thanks. Personally, I think a black 
& white icon is better in line with the other icons.


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

Job opening

[Simon Weller]

All,


We're currently hiring for a Product Manager position dedicated to our Cloud 
Product (based on CloudStack). Where would it be appropriate to post the ad?


Thanks,


- Si

[GitHub] cloudstack pull request: CLOUDSTACK-9101: fix some issues in resiz...

[ustcweizhou]

GitHub user ustcweizhou opened a pull request:

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

CLOUDSTACK-9101: fix some issues in resize volume

(1) fix issue: volume size is not updated even if the operation succeed
(2) Add ui support for root volume resize
(3) resize on qcow2 type ROOT volume of stopped vm does not really work
see https://issues.apache.org/jira/browse/CLOUDSTACK-9101

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

$ git pull https://github.com/ustcweizhou/cloudstack resize-volume-issues

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

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


commit 9077c9a5b47224b2d00978ff67e9af6957ec589b
Author: Wei Zhou 
Date:   2015-11-03T16:04:55Z

CLOUDSTACK-9022: keep Destroyed volumes for sometime

commit 4ed1e0d5f809d1bb2845b6e076d55eeb1ce3f0f4
Author: Wei Zhou 
Date:   2015-11-04T08:27:32Z

CLOUDSTACK-9022: move storage.cleanup related global configurations to 
StorageManager

commit 5b7d935ab0b35bec936785e44023ac0d8552e72b
Author: Remi Bergsma 
Date:   2015-12-02T20:52:46Z

Merge pull request #1145 from ustcweizhou/storage-cleanup-delay-4.6

[4.6.1] CLOUDSTACK-9022: keep Destroyed volumes for sometimefor now, the 
Destroyed volumes will be expunged in Storage cleanup thread, no matter when 
they are destroyed.
In Expunging vm thread, we expunge the Destroyed vms which have been 
destroyed at least 'expunge.delay' seconds. We add the similar configuration 
for volumes.

same to #1029 , for 4.6

* pr/1145:
  CLOUDSTACK-9022: move storage.cleanup related global configurations to 
StorageManager
  CLOUDSTACK-9022: keep Destroyed volumes for sometime

This closes #1029

Signed-off-by: Remi Bergsma 

commit 119b27b2c6c366949b574fbca1574f15d67a3af3
Author: Wei Zhou 
Date:   2015-12-03T10:54:20Z

CLOUDSTACK-9101: update volume size after resizevolume

commit d6e21f74163212b198731ddf23dd48bc4c787b84
Author: Wei Zhou 
Date:   2015-12-03T10:53:42Z

CLOUDSTACK-9101: add UI support for root volume resize

commit 9221cb3e0d0e5a0ea374f10e889f7e32c2a3eda1
Author: Wei Zhou 
Date:   2015-12-03T13:31:55Z

CLOUDSTACK-9101: resize root volume of stopped vm on KVM




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-9101: fix some issues in resiz...

[ustcweizhou]

Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1160#issuecomment-161647136
  
wrong 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.
---

[GitHub] cloudstack pull request: CLOUDSTACK-9101: fix some issues in resiz...

[ustcweizhou]

Github user ustcweizhou closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-9101: fix some issues in resiz...

[ustcweizhou]

GitHub user ustcweizhou opened a pull request:

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

CLOUDSTACK-9101: fix some issues in resize volume

(1) fix issue: volume size is not updated even if the operation succeed
(2) Add ui support for root volume resize
(3) resize on qcow2 type ROOT volume of stopped vm does not really work
see https://issues.apache.org/jira/browse/CLOUDSTACK-9101

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

$ git pull https://github.com/ustcweizhou/cloudstack resize-volume-issues

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

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


commit 119b27b2c6c366949b574fbca1574f15d67a3af3
Author: Wei Zhou 
Date:   2015-12-03T10:54:20Z

CLOUDSTACK-9101: update volume size after resizevolume

commit d6e21f74163212b198731ddf23dd48bc4c787b84
Author: Wei Zhou 
Date:   2015-12-03T10:53:42Z

CLOUDSTACK-9101: add UI support for root volume resize

commit 9221cb3e0d0e5a0ea374f10e889f7e32c2a3eda1
Author: Wei Zhou 
Date:   2015-12-03T13:31:55Z

CLOUDSTACK-9101: resize root volume of stopped vm on KVM




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

[pdube]

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

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

I think that if a user is able to create an AG, then he should be able to 
delete 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.
---

CS slack channel

[Simon Weller]

Daan,


Could you send me an invite to the CloudStack Slack Team please?


Thanks,


- Si

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

[pdube]

Github user pdube commented on the pull request:

https://github.com/apache/cloudstack/pull/1153#issuecomment-161650382
  
Isn't the project inferred from the instance? The 
[CreateVmSnapshot](http://cloudstack.apache.org/docs/api/apidocs-4.6/root_admin/createVMSnapshot.html)
 API doesn't even take a projectId


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

[ustcweizhou]

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

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

Checked the cloudstack, the difference of project admin and project user are


http://docs.cloudstack.apache.org/projects/cloudstack-administration/en/4.6/projects.html
The project administrator can pass on the role to another project member. 
The project administrator can also add more members, remove members from the 
project, set new resource limits (as long as they are below the global defaults 
set by the CloudStack administrator), and delete the project. When the 
administrator removes a member from the project, resources created by that 
user, such as VM instances, remain with the project. This brings us to the 
subject of resource ownership and which resources can be used by a project.


so, @pdube you can ignore this comment.


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

[GitHub] cloudstack pull request: Quota

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/768#issuecomment-161653555
  
@bhaisaab The SQL issue is indeed resolved. I still see the icon when the 
plugin is disabled (and it of course gives errors then). Tried clearing the 
cache and built from scratch but still the same.


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

[Remi Bergsma]

Hi all,

FYI run some integration tests against 4.6.1 and they all pass.

Talked to Daan and I cannot reproduce his UI issue.

Test results:

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

```

Result:

```
Create a redundant VPC with two networks with two VMs in each network ... === 
TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | 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
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 | Status : SUCCESS 
===
ok

--
Ran 34 tests in 16542.754s

OK
```


And:

```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a 
tags=advanc

Re: CS slack channel

[Nguyen Anh Tu]

Great to hear CS is now happening in Slack. Could you send me an invitation
as well, Daan ?


On Thu, Dec 3, 2015 at 9:08 PM, Simon Weller  wrote:

> Daan,
>
>
> Could you send me an invite to the CloudStack Slack Team please?
>
>
> Thanks,
>
>
> - Si
>

Re: CS slack channel

[Daan Hoogland]

very welcome guys (and potentially dolls)

On Thu, Dec 3, 2015 at 3:35 PM, Nguyen Anh Tu  wrote:

> Great to hear CS is now happening in Slack. Could you send me an invitation
> as well, Daan ?
>
>
> On Thu, Dec 3, 2015 at 9:08 PM, Simon Weller  wrote:
>
> > Daan,
> >
> >
> > Could you send me an invite to the CloudStack Slack Team please?
> >
> >
> > Thanks,
> >
> >
> > - Si
> >
>



-- 
Daan

Re: CS slack channel

[Nux!]

https://drewdevault.com/2015/11/01/Please-stop-using-slack.html

--
Sent from the Delta quadrant using Borg technology!

Nux!
www.nux.ro

- Original Message -
> From: "Nguyen Anh Tu" 
> To: dev@cloudstack.apache.org
> Sent: Thursday, 3 December, 2015 14:35:24
> Subject: Re: CS slack channel

> Great to hear CS is now happening in Slack. Could you send me an invitation
> as well, Daan ?
> 
> 
> On Thu, Dec 3, 2015 at 9:08 PM, Simon Weller  wrote:
> 
>> Daan,
>>
>>
>> Could you send me an invite to the CloudStack Slack Team please?
>>
>>
>> Thanks,
>>
>>
>> - Si

[GitHub] cloudstack-www pull request: developer page: fixed some repo links...

[swill]

GitHub user swill opened a pull request:

https://github.com/apache/cloudstack-www/pull/21

developer page: fixed some repo links and new tab for external links

Fixed repo links as they were broken before.

I also changed the external links on this page to be opened in new tabs so 
it preserves the flow for the developer as they go through the guide and click 
to learn more about different topics as they work their way through the guide.

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

$ git pull https://github.com/swill/cloudstack-www master

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

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


commit 9bc1391770d4cdb6c38a9b2a0ebb2c9bd7573526
Author: Will Stevens 
Date:   2015-12-03T14:46:08Z

developer page: fixed some repo links and now opening external links in a 
new tab for better flow




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

[pdube]

Github user pdube commented on the pull request:

https://github.com/apache/cloudstack/pull/1134#issuecomment-161657343
  
Hey guys, there are enough LGTMs, can we get this merged?


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

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

[Kenny Ho]

Thanks Rohit.  I think the pip upgrade fixed it.  Ubuntu 14 python comes
with python requests by default at version 2.2.1 and looks like cloudmonkey
need requests at 2.8.1

Regards,
Kenny

On Thu, Dec 3, 2015 at 1:25 AM, Rohit Yadav 
wrote:

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

Re: [VOTE] Apache CloudStack 4.6.1

[Daan Hoogland]

To be on the record:
+0
I have an issue but cannot lay the blame with cloudstack. I will keep
looking and as I saw lots of reasons to go through with this version
reported by others.

On Thu, Dec 3, 2015 at 3:31 PM, Remi Bergsma 
wrote:

> Hi all,
>
> FYI run some integration tests against 4.6.1 and they all pass.
>
> Talked to Daan and I cannot reproduce his UI issue.
>
> Test results:
>
> ```
> nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
> tags=advanced,required_hardware=true \
> component/test_vpc_redundant.py \
> component/test_routers_iptables_default_policy.py \
> component/test_routers_network_ops.py \
> component/test_vpc_router_nics.py \
> smoke/test_loadbalance.py \
> smoke/test_internal_lb.py \
> smoke/test_ssvm.py \
> smoke/test_network.py
>
> ```
>
> Result:
>
> ```
> Create a redundant VPC with two networks with two VMs in each network ...
> === TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL |
> 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
> 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 acqu

Re: [VOTE] Apache CloudStack 4.6.1

[Daan Hoogland]

should be closed today

On Thu, Dec 3, 2015 at 4:40 PM, Paul Angus  wrote:

> I've lost track - how long has the vote got left?
>
>
> [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: 03 December 2015 15:23
> To: dev 
> Cc: Glenn Wagner 
> Subject: Re: [VOTE] Apache CloudStack 4.6.1
>
> To be on the record:
> +0
> I have an issue but cannot lay the blame with cloudstack. I will keep
> looking and as I saw lots of reasons to go through with this version
> reported by others.
>
> On Thu, Dec 3, 2015 at 3:31 PM, Remi Bergsma 
> wrote:
>
> > Hi all,
> >
> > FYI run some integration tests against 4.6.1 and they all pass.
> >
> > Talked to Daan and I cannot reproduce his UI issue.
> >
> > Test results:
> >
> > ```
> > nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
> > tags=advanced,required_hardware=true \ component/test_vpc_redundant.py
> > \ component/test_routers_iptables_default_policy.py \
> > component/test_routers_network_ops.py \
> > component/test_vpc_router_nics.py \ smoke/test_loadbalance.py \
> > smoke/test_internal_lb.py \ smoke/test_ssvm.py \ smoke/test_network.py
> >
> > ```
> >
> > Result:
> >
> > ```
> > Create a redundant VPC with two networks with two VMs in each network ...
> > === TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL |
> > 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 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

RE: [VOTE] Apache CloudStack 4.6.1

[Paul Angus]

Thanks Daan,

I've only had time for some very basic testing, but nothing failed.

+0


[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:image14f217.png@d4c39fd3.409691bf]


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: 03 December 2015 15:42
To: dev 
Cc: Glenn Wagner 
Subject: Re: [VOTE] Apache CloudStack 4.6.1

should be closed today

On Thu, Dec 3, 2015 at 4:40 PM, Paul Angus  wrote:

> I've lost track - how long has the vote got left?
>
>
> [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: 03 December 2015 15:23
> To: dev 
> Cc: Glenn Wagner 
> Subject: Re: [VOTE] Apache CloudStack 4.6.1
>
> To be on the record:
> +0
> I have an issue but cannot lay the blame with cloudstack. I will keep
> looking and as I saw lots of reasons to go through with this version
> reported by others.
>
> On Thu, Dec 3, 2015 at 3:31 PM, Remi Bergsma
> 
> wrote:
>
> > Hi all,
> >
> > FYI run some integration tests against 4.6.1 and they all pass.
> >
> > Talked to Daan and I cannot reproduce his UI issue.
> >
> > Test results:
> >
> > ```
> > nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
> > tags=advanced,required_hardware=true \
> > component/test_vpc_redundant.py \
> > component/test_routers_iptables_default_policy.py \
> > component/test_routers_network_ops.py \
> > component/test_vpc_router_nics.py \ smoke/test_loadbalance.py \
> > smoke/test_internal_lb.py \ smoke/test_ssvm.py \
> > smoke/test_network.py
> >
> > ```
> >
> > Result:
> >
> > ```
> > Create a redundant VPC with two networks with two VMs in each network ...
> > === TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL
> > | 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 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_sin

RE: [VOTE] Apache CloudStack 4.6.1

[Paul Angus]

I've lost track - how long has the vote got left?


[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:image74bf9b.png@b982afad.42ae7273]


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: 03 December 2015 15:23
To: dev 
Cc: Glenn Wagner 
Subject: Re: [VOTE] Apache CloudStack 4.6.1

To be on the record:
+0
I have an issue but cannot lay the blame with cloudstack. I will keep looking 
and as I saw lots of reasons to go through with this version reported by others.

On Thu, Dec 3, 2015 at 3:31 PM, Remi Bergsma 
wrote:

> Hi all,
>
> FYI run some integration tests against 4.6.1 and they all pass.
>
> Talked to Daan and I cannot reproduce his UI issue.
>
> Test results:
>
> ```
> nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
> tags=advanced,required_hardware=true \ component/test_vpc_redundant.py
> \ component/test_routers_iptables_default_policy.py \
> component/test_routers_network_ops.py \
> component/test_vpc_router_nics.py \ smoke/test_loadbalance.py \
> smoke/test_internal_lb.py \ smoke/test_ssvm.py \ smoke/test_network.py
>
> ```
>
> Result:
>
> ```
> Create a redundant VPC with two networks with two VMs in each network ...
> === TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL |
> 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 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 

RE: [VOTE] Apache CloudStack 4.6.1

[Paul Angus]

Sorry should say 4.6.1 on Centos6.7 mgmt server with ESXi 5.5 hosts/vCenter.



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





From: Paul Angus [mailto:paul.an...@shapeblue.com]
Sent: 03 December 2015 15:47
To: dev@cloudstack.apache.org
Cc: Glenn Wagner 
Subject: RE: [VOTE] Apache CloudStack 4.6.1

Thanks Daan,

I've only had time for some very basic testing, but nothing failed.

+0

[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:image14f217.png@d4c39fd3.409691bf]



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: 03 December 2015 15:42
To: dev mailto:dev@cloudstack.apache.org>>
Cc: Glenn Wagner mailto:glenn.wag...@shapeblue.com>>
Subject: Re: [VOTE] Apache CloudStack 4.6.1

should be closed today

On Thu, Dec 3, 2015 at 4:40 PM, Paul Angus 
mailto:paul.an...@shapeblue.com>> wrote:

> I've lost track - how long has the vote got left?
>
>
> [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* 
> mailto: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: 03 December 2015 15:23
> To: dev mailto:dev@cloudstack.apache.org>>
> Cc: Glenn Wagner 
> mailto:glenn.wag...@shapeblue.com>>
> Subject: Re: [VOTE] Apache CloudStack 4.6.1
>
> To be on the record:
> +0
> I have an issue but cannot lay the blame with cloudstack. I will keep
> looking and as I saw lots of reasons to go through with this version
> reported by others.
>
> On Thu, Dec 3, 2015 at 3:31 PM, Remi Bergsma
> mailto:rberg...@schubergphilis.com>>
> wrote:
>
> > Hi all,
> >
> > FYI run some integration tests against 4.6.1 and they all pass.
> >
> > Talked to Daan and I cannot reproduce his UI issue.
> >
> > Test results:
> >
> > ```
> > 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

[GitHub] cloudstack pull request: CLOUDSTACK-8892:If VR enters out-of band ...

[resmo]

Github user resmo commented on the pull request:

https://github.com/apache/cloudstack/pull/1158#issuecomment-161684501
  
big -1 
We tested out of band migrations on VR on VMware and they work without 
reboots. 

A VM/VR does not "loose" the IP just because it gets migrated. Look for the 
root cause.


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

[wido]

Github user wido commented on the pull request:

https://github.com/apache/cloudstack/pull/1149#issuecomment-161709051
  
Seems good, 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

[Milamber]

Hello,

My vote +1 (binding)

My tests :
1/ Fresh install of 4.6.1 (adv net) on Ubuntu 14.04 + KVM + NFS : OK
Some standard tests (create vm, migration, HA, create network, create 
user, create ssh key, destroy vm, register template, create snapshot, 
restore snapshot, create template, ip association, ip release, static 
nat, firewall rule)
Some tests with cloudstack ansible module (create network, register 
templates, create vm, ip, firewall rule)


2/ On fresh install of 4.5.2 (adv net) on Ubuntu 14.04 + KVM + NFS, 
migration to 4.6.1 : OK
Some standard tests (create vm, migration, HA, create network, create 
user, create ssh key, destroy vm, register template, create snapshot, 
restore snapshot, create template)
Some tests with cloudstack ansible module (create network, register 
templates, create vm, ip, firewall rule)


3/ Tests all localizations of Web UI (with a complete installation of 
CS) : local strings works wells except Spanish and Arabic (expected, 
some strings needs to be fixes on Transifex)


Thanks to RM team.

Milamber


On 30/11/2015 22:12, 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)

Re: [VOTE] Apache CloudStack 4.6.1

[Glenn Wagner]

Good day

+1

Centos 6,7 management server
Xenserver 6,2
NFS storage

Tests done ,
Standard VM tests
Template uploads
ISO uploads
Basic network functions , add and remove IPs
VM Snapshots

Didn't get to vpc testing

Regards
Glenn Wagner
Shape blue. Pty ltd



On Thu, Dec 3, 2015 at 8:48 AM -0800, "Milamber" 
mailto:milam...@apache.org>> wrote:

Hello,

My vote +1 (binding)

My tests :
1/ Fresh install of 4.6.1 (adv net) on Ubuntu 14.04 + KVM + NFS : OK
Some standard tests (create vm, migration, HA, create network, create
user, create ssh key, destroy vm, register template, create snapshot,
restore snapshot, create template, ip association, ip release, static
nat, firewall rule)
Some tests with cloudstack ansible module (create network, register
templates, create vm, ip, firewall rule)

2/ On fresh install of 4.5.2 (adv net) on Ubuntu 14.04 + KVM + NFS,
migration to 4.6.1 : OK
Some standard tests (create vm, migration, HA, create network, create
user, create ssh key, destroy vm, register template, create snapshot,
restore snapshot, create template)
Some tests with cloudstack ansible module (create network, register
templates, create vm, ip, firewall rule)

3/ Tests all localizations of Web UI (with a complete installation of
CS) : local strings works wells except Spanish and Arabic (expected,
some strings needs to be fixes on Transifex)

Thanks to RM team.

Milamber


On 30/11/2015 22:12, 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)
>

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: Add support for not (re)starting server a...

[ProjectMoon]

GitHub user ProjectMoon opened a pull request:

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

Add support for not (re)starting server after cloud-setup-management.

This adds an option to the cloud-setup-management script to not start
the management server after a successful configuration of it. 

The primary motivation for this is to avoid circular dependency issues on 
systems that use systemd. When calling cloud-setup-management from a unit with 
a Before= directive on a service depending on
cloudstack-management, the process will deadlock because /usr/bin/service 
will delegate to systemd, which is waiting for the Before service to start.

Executing the cloud-setup-management script with this new `--no-start` 
option will simply leave the management server stopped after a successful 
configuration. systemd can then be bypassed with `export 
_SYSTEMCTL_SKIP_REDIRECT=1` and using the init.d script.

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

$ git pull https://github.com/greenqloud/cloudstack pr-no-start

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

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


commit 1f2357dc22f78acc8a5e6a1cb3e9269488aa499f
Author: jeff 
Date:   2015-12-03T18:36:10Z

Add support for not (re)starting server after cloud-setup-management.

This adds an option to the cloud-setup-management script to not start
the management server after a successful configuration of it. The
primary motivation for this is to avoid circular dependency issues on
systems that use systemd. When calling cloud-setup-management from a
unit with a Before= directive on a service depending on
cloudstack-management, the process will deadlock because
/usr/bin/service will delegate to systemd, which is waiting for the
Before service to start.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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: Send arping to the gateway instead of our...

[remibergsma]

GitHub user remibergsma opened a pull request:

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

Send arping to the gateway instead of our own address

We need to send an Unsolicited ARP to the gateway, instead of our own 
address. We now encounter problems when people deploy/destroy/deploy and get 
the same public ip.

Packets arrive, but with incorrect / cached mac and are ignored by the 
routervm kernel.
Run arping manually to update the arp-cache on the gateway and things start 
to work.

Then we discovered the `arping` is actually done, but sent to its own 
address. Therefore the gateway doesn't pick it up. We only saw this happening 
when rapid deploy tools are used, like Terraform that do deploy/destroy/deploy 
and might get the same ip but on a new router having a new mac.

```
2015-12-03 18:07:25,589  CsHelper.py execute:160 Executing: arping -c 1 -I 
eth1 -A -U -s 192.168.23.8 192.168.23.1
```

The integration tests seem happy, although the full run is still ongoing:

```
=== TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | 
Status : SUCCESS ===
```

Thanks @sspans for helping trouble shoot this. Ping @wilderrodrigues can 
you review please?

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

$ git pull https://github.com/remibergsma/cloudstack arping-to-gw

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

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


commit 90e01c95a2db3d9b7c6586b2b228e0a9c5b9e415
Author: Remi Bergsma 
Date:   2015-12-03T17:15:31Z

CLOUDSTACK-9097 Make public ip work immediately

We need to send an Unsolicited ARP to the gateway, instead of our own 
address. We now encounter problems when people deploy/destroy/deploy and get 
the same public ip.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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: Merge 4.6

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1155#issuecomment-161745983
  
LGTM, run some tests and they are all fine:

```
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
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 | Status : 
SUCCESS ===
ok

--
Ran 34 tests in 16622.001s

OK
```


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w

[GitHub] cloudstack pull request: Merge 4.6

[asfgit]

Github user asfgit closed the pull request at:

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


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

[asfgit]

Github user asfgit closed the pull request at:

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


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

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1149#issuecomment-161748134
  
@bhaisaab Should I merge this to master, or should it be merged to 4.6?


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

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

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1134#issuecomment-161750205
  
@pdube indeed, I waited because of the comments but that has been addressed 
now :-)


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

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

[asfgit]

Github user asfgit closed the pull request at:

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


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

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1013#issuecomment-161751305
  
@jharshman I will run some tests and report back. Can you please squash the 
commits?


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

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

[asfgit]

Github user asfgit closed the pull request at:

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


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

[pdube]

Github user pdube commented on the pull request:

https://github.com/apache/cloudstack/pull/1134#issuecomment-161775836
  
@remibergsma  awesome :)


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

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

[PaulAngus]

Github user PaulAngus commented on the pull request:

https://github.com/apache/cloudstack/pull/1087#issuecomment-161780789
  
So looking at the rpm spec options using +%config(noreplace) will cause a 
new version of cloudstack-catalina (cloudstack-catalina.rpmnew) to be created 
if cloudstack-catalina has been edited.
However as the contents of /etc/logrotate.d/ are enumerated and run this 
would result in both the new and old behaviour both being applied - which seems 
the worst of all worlds.
Therefore I suggest the orginal PR (without commit 
ec698f5377ad98ac2a34488a0102c45c2408d089) which will overwrite if the new rpm 
has an updated value in it [1]; is the lesser of two evils.

[1] http://www-uxsup.csx.cam.ac.uk/~jw35/docs/rpm_config.html


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 support for not (re)starting server a...

[DaanHoogland]

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

https://github.com/apache/cloudstack/pull/1162#discussion_r46613321
  
--- Diff: client/bindir/cloud-setup-management.in ---
@@ -24,20 +24,23 @@ from cloudutils.globalEnv import globalEnv
 from cloudutils.serviceConfigServer import cloudManagementConfig
 from optparse import OptionParser
 if __name__ == '__main__':
-initLoging("@MSLOGDIR@/setupManagement.log")
+initLoging("/var/log/cloudstack/management/setupManagement.log")
--- End diff --

Will this hardcoded value fly on windows? (yes it is used as ms platform by 
some)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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: Send arping to the gateway instead of our...

[DaanHoogland]

Github user DaanHoogland commented on the pull request:

https://github.com/apache/cloudstack/pull/1163#issuecomment-161788201
  
Makes sense @remibergsma Is that test an example of the failure before the 
fix?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 support for not (re)starting server a...

[ProjectMoon]

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

https://github.com/apache/cloudstack/pull/1162#discussion_r46618985
  
--- Diff: client/bindir/cloud-setup-management.in ---
@@ -24,20 +24,23 @@ from cloudutils.globalEnv import globalEnv
 from cloudutils.serviceConfigServer import cloudManagementConfig
 from optparse import OptionParser
 if __name__ == '__main__':
-initLoging("@MSLOGDIR@/setupManagement.log")
+initLoging("/var/log/cloudstack/management/setupManagement.log")
--- End diff --

Ah yes this was an unintended change. I will put it back to the original 
@MSLOGDIR@ tomorrow.


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

[jharshman]

Github user jharshman commented on the pull request:

https://github.com/apache/cloudstack/pull/1013#issuecomment-161840311
  
@remibergsma just squashed the commits, let me know if there is anything 
else I can do to assist testing.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-9103 : Missing OS Mappings for...

[jburwell]

Github user jburwell commented on the pull request:

https://github.com/apache/cloudstack/pull/1159#issuecomment-161876783
  
Master is now 4.7.0.  Therefore, these database changes should be in 
``schema-460to470.sql`` and ``schema-461to470.sql`` files.


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

[jburwell]

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

https://github.com/apache/cloudstack/pull/1152#discussion_r46649429
  
--- Diff: 
api/src/org/apache/cloudstack/api/command/admin/user/ListKeysCmd.java ---
@@ -0,0 +1,72 @@
+// 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 org.apache.cloudstack.api.command.admin.user;
+
+
+import com.cloud.user.Account;
+import com.cloud.user.User;
+import org.apache.cloudstack.api.APICommand;
+import org.apache.cloudstack.api.ApiConstants;
+import org.apache.cloudstack.api.BaseCmd;
+import org.apache.cloudstack.api.Parameter;
+import org.apache.cloudstack.api.response.RegisterResponse;
+import org.apache.cloudstack.api.response.UserResponse;
+
+import java.util.logging.Logger;
+
+@APICommand(name = "listUserKeys",
+description = "This command allows the user to query the 
seceret and API keys for the account",
+responseObject = RegisterResponse.class,
+requestHasSensitiveInfo = false,
+responseHasSensitiveInfo = true)
+
+public class ListKeysCmd extends BaseCmd{
+
+@Parameter(name= ApiConstants.ID, type = CommandType.UUID, entityType 
= UserResponse.class, required = true, description = "ID of the user whose keys 
are required")
+private Long id;
+
+public static final Logger s_logger = 
Logger.getLogger(RegisterCmd.class.getName());
+public static final String s_name = "listuserkeysresponse";
+
+public Long getID(){
+return id;
+}
+
+public String getCommandName(){
+return s_name;
+}
+
+public long getEntityOwnerId(){
+User user = _entityMgr.findById(User.class, getID());
+if(user != null){
+return user.getAccountId();
+}
+else return Account.ACCOUNT_ID_SYSTEM;
+}
+public void execute(){
+String[] keys = _accountService.getKeys(this);
+RegisterResponse response = new RegisterResponse();
+if(keys != null){
--- End diff --

Add an check before setting the keys to check that ``keys`` has a length = 
2 to avoid an ``ArrayIndexOutOfBoundsException``.  If the length is not equal 
to 2, throw an ``IllegalStateException``.


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

[jburwell]

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

https://github.com/apache/cloudstack/pull/1152#discussion_r46649508
  
--- Diff: api/src/com/cloud/user/AccountService.java ---
@@ -136,4 +140,6 @@ void checkAccess(Account account, AccessType 
accessType, boolean sameOwner, Stri
  */
 UserAccount getUserAccountById(Long userId);
 
+public String[] getKeys(ListKeysCmd cmd);
--- End diff --

Why is the return type defined as an array and not a ``List``?


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

[4.6.1/4.7.0] Database upgrade path issues/forks between 4.6.0/4.6.1 and 4.7.0/master

[Rohit Yadav]

There seems to be a db view related change between 4.6.0 and 4.6.1 (DB) 
versions; and there exists separate DB upgrade paths from 4.6.0 to 4.7.0 and 
4.6.1 to 4.7.0 on master branch which shares the same changes. The issue is 
that the upgrade path from 4.6.0 to 4.7.0 does not include the db view related 
changes that exists in the 4.6.0 to 4.6.1 upgrade path.

While unlikely, but please share if this was done on purpose (i.e. we don’t 
want that db view related change if some user is upgrading from 4.6.0 to 
4.7.0), in which case it’s a non-issue.

I’ll try to send a PR that *ought* to be merged on master before we freeze 
master (for 4.7.0) or cut a RC, but if I’m unable to do so today — someone 
please help fix this, and also refactor code (since there is a lot of common 
code), and don’t proceed without fixing this issue.

Thanks.

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: Removed the PlannerBase class because it ...

[jburwell]

Github user jburwell commented on the pull request:

https://github.com/apache/cloudstack/pull/1108#issuecomment-161877685
  
@rafaelweingartner I agree.  The use of "_" has no place in modern Java 
code.  Constants should be all caps with _ separators.  Everything else should 
be camelCase.


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

[bhaisaab]

Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1149#issuecomment-161877977
  
@remibergsma sorry for the source/target branches; this needs to go into 
both 4.6 and master branches. So merge on 4.6, and fwd-merge 4.6 on master. 
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 -...

[bhaisaab]

Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1149#issuecomment-161878014
  
@remibergsma let me know if you want me to open a new PR against 4.6 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.
---

[GitHub] cloudstack pull request: CLOUDSTACK-9103 : Missing OS Mappings for...

[bhaisaab]

Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1159#issuecomment-161878161
  
@maneesha-p thanks for the PR, please fix as John suggests. We also support 
vmware 6.0 on 4.5 branch, so do you think we need this in 4.5.2 to 4.5.3 
upgrade path as well?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-9095 : Hypervisor changes to s...

[jburwell]

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

https://github.com/apache/cloudstack/pull/1142#discussion_r46650119
  
--- Diff: 
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java 
---
@@ -1276,6 +1277,53 @@ public String toString() {
 }
 }
 
+public static class MetadataDef {
+Map customNodes = new HashMap<>();
+
+public  T getMetadataNode(Class fieldClass) {
+T field = (T) customNodes.get(fieldClass.getName());
+if (field == null) {
+try {
+field = fieldClass.newInstance();
+customNodes.put(field.getClass().getName(), field);
+} catch (InstantiationException e) {
+} catch (IllegalAccessException e) {
+}
+}
+return field;
+}
+
+@Override
+public String toString() {
+StringBuilder fsBuilder = new StringBuilder();
+fsBuilder.append("\n");
+for (Object field : customNodes.values()) {
+fsBuilder.append(field.toString());
+}
+fsBuilder.append("\n");
+return fsBuilder.toString();
+}
+}
+
+public static class NuageExtensionDef {
+private Map addresses = Maps.newHashMap();
+
+public void addNuageExtension(String macAddress, String vrIp) {
+addresses.put(macAddress, vrIp);
+}
+
+@Override
+public String toString() {
+StringBuilder fsBuilder = new StringBuilder();
+for (Map.Entry address : addresses.entrySet()) 
{
+fsBuilder.append("\n");
+fsBuilder.append("  \n");
--- End diff --

Performing string append operations when using a ``StringBuilder`` defeats 
purpose of the class.  Please change to the following to avoid unnecessary 
string re-allocation:
```
fsBuilder.append("  \n");
```


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-9095 : Hypervisor changes to s...

[jburwell]

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

https://github.com/apache/cloudstack/pull/1142#discussion_r46650221
  
--- Diff: 
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java 
---
@@ -1276,6 +1277,53 @@ public String toString() {
 }
 }
 
+public static class MetadataDef {
+Map customNodes = new HashMap<>();
+
+public  T getMetadataNode(Class fieldClass) {
+T field = (T) customNodes.get(fieldClass.getName());
+if (field == null) {
+try {
+field = fieldClass.newInstance();
+customNodes.put(field.getClass().getName(), field);
+} catch (InstantiationException e) {
+} catch (IllegalAccessException e) {
--- End diff --

Consolidate to multi-catch and log the exception details to ``DEBUG`` 
explaining why the exceptions were ignored.


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

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1149#issuecomment-161885184
  
@bhaisaab Check! This I cannot merge to 4.6 with this PR so if you make one 
against 4.6 that would be easy. Mention this one so it can go through faster. 
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-8845: Fixed list snapshots com...

[anshul1886]

GitHub user anshul1886 opened a pull request:

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

CLOUDSTACK-8845: Fixed list snapshots command throwing exception

https://issues.apache.org/jira/browse/CLOUDSTACK-8845

Fixed list snapshots command throwing exception if any one of snapshot is 
not present on primary.

Bug got introduced because of KVM specific code.

Testing done

List snapashots working fine.

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

$ git pull https://github.com/anshul1886/cloudstack-1 CLOUDSTACK-8845

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

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


commit 91578903e1c687991329aac1518ca6717ac81285
Author: Anshul Gangwar 
Date:   2015-11-23T09:26:53Z

CLOUDSTACK-8845: Fixed list snapshots command throwing exception if any one 
of snapshot is not present on primary
Bug got introduced because of KVM specific code




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-8845: Fixed list snapshots com...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1164#issuecomment-161889769
  
@anshul1886 Thanks for the fix! Could you please make this PR against 4.6? 
Once merged in 4.6, it will be fwd-merged to master. 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-9104: VM naming convention in ...

[priyankparihar]

GitHub user priyankparihar opened a pull request:

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

CLOUDSTACK-9104: VM naming convention in case vmware is used

I have reverted all the changes. Now functionality is same as it was  in 
earlier version. 

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

$ git pull https://github.com/priyankparihar/cloudstack CLOUDSTACK-9104

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

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


commit 2588925e9c82a9914b76e06204a04f4a5ff1ddb1
Author: Priyank Parihar 
Date:   2015-11-05T15:18:00Z

CLOUDSTACK-9104: VM naming convention in case vmware is used




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

[GitHub] cloudstack pull request: CLOUDSTACK-8845: Fixed list snapshots com...

[anshul1886]

GitHub user anshul1886 opened a pull request:

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

CLOUDSTACK-8845: Fixed list snapshots command throwing exception

https://issues.apache.org/jira/browse/CLOUDSTACK-8845

CLOUDSTACK-8845: Fixed list snapshots command throwing exception if any one 
of snapshot is not present on primary

Bug got introduced because of KVM specific code

Testing done

List snapshots is working fine.

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

$ git pull https://github.com/anshul1886/cloudstack-1 CLOUDSTACK-8845-4.6

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

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


commit e41f61194ae275f9d90b778899fe2ed352b6db46
Author: Anshul Gangwar 
Date:   2015-11-23T09:26:53Z

CLOUDSTACK-8845: Fixed list snapshots command throwing exception if any one 
of snapshot is not present on primary
Bug got introduced because of KVM specific code




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-8845: Fixed list snapshots com...

[anshul1886]

Github user anshul1886 commented on the pull request:

https://github.com/apache/cloudstack/pull/1164#issuecomment-161891307
  
Created PR https://github.com/apache/cloudstack/pull/1166 for 4.6.


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

[GitHub] cloudstack pull request: CLOUDSTACK-8845: Fixed list snapshots com...

[remibergsma]

Github user remibergsma commented on the pull request:

https://github.com/apache/cloudstack/pull/1164#issuecomment-161893553
  
@anshul1886 Thanks, you can close this PR as the other one will go to both 
4.6 and 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: CLOUDSTACK-9105: Logging enhancement: Han...

[koushik-das]

GitHub user koushik-das opened a pull request:

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

CLOUDSTACK-9105: Logging enhancement: Handle/reference to track API c…

…alls end to end in the MS logs

Added logid to logging framework, now all API call logs can be tracked with 
this id end to end

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

$ git pull https://github.com/koushik-das/cloudstack CLOUDSTACK-9105

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

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


commit a4dc271253be5d6a504f644bf457cea38070857e
Author: Koushik Das 
Date:   2015-12-04T07:15:57Z

CLOUDSTACK-9105: Logging enhancement: Handle/reference to track API calls 
end to end in the MS logs
Added logid to logging framework, now all API call logs can be tracked with 
this id end to end




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

[GitHub] cloudstack pull request: CLOUDSTACK-9105: Logging enhancement: Han...

[koushik-das]

Github user koushik-das commented on the pull request:

https://github.com/apache/cloudstack/pull/1167#issuecomment-161897298
  
Now all logs related to an operation can be traced by running something 
like "grep logid:62314271 ".

Sample logs
2015-12-02 12:40:27,628 DEBUG [o.a.c.f.j.i.AsyncJobManagerImpl] 
(API-Job-Executor-1:ctx-9fb051eb job-939) (logid:62314271) Executing AsyncJobVO 
{id:939, userId: 2, accountId: 2, instanceType: VirtualMachine, instanceId: 
144, cmd: org.apache.cloudstack.api.command.admin.vm.DeployVMCmdByAdmin, 
cmdInfo: 
{"id":"144","response":"json","serviceofferingid":"0381afb2-50ab-40ca-9a57-e5d12482b5a3","ctxDetails":"{\"interface
 com.cloud.dc.DataCenter\":\"4cea85aa-3dda-4b03-8caf-391198777fb3\",\"interface 
com.cloud.vm.VirtualMachine\":\"5fb88c3e-3879-45b0-9a34-47dc1b82b8a5\",\"interface
 
com.cloud.offering.ServiceOffering\":\"0381afb2-50ab-40ca-9a57-e5d12482b5a3\",\"interface
 
com.cloud.template.VirtualMachineTemplate\":\"f74be2e8-73be-11e5-8882-249684d59f9c\"}","cmdEventType":"VM.CREATE","ctxUserId":"2","zoneid":"4cea85aa-3dda-4b03-8caf-391198777fb3","hypervisor":"Simulator","iptonetworklist[0].networkid":"4156249e-2b1f-49cb-b333-c2032c0792c5","templateid":"f74be2e8-73be-11e5-8882-249684d59f9c"
 
,"httpmethod":"GET","_":"1449040227133","uuid":"5fb88c3e-3879-45b0-9a34-47dc1b82b8a5","ctxAccountId":"2","ctxStartEventId":"1420"},
 cmdVersion: 0, status: IN_PROGRESS, processStatus: 0, resultCode: 0, result: 
null, initMsid: 1, completeMsid: null, lastUpdated: null, lastPolled: null, 
created: null}

Even the HV resource logs are tagged using the same "logid" identifier.

2015-12-02 12:40:30,263 DEBUG [c.c.a.m.DirectAgentAttache] 
(DirectAgent-15:ctx-99c2667b) (logid:62314271) Seq 2-852024754503155726: 
Response Received: 
2015-12-02 12:40:30,266 DEBUG [c.c.a.t.Request] 
(DirectAgent-15:ctx-99c2667b) (logid:62314271) Seq 2-852024754503155726: 
Processing:  { Ans: , MgmtId: 1, via: 
2(SimulatedAgent.40265aab-4037-41f7-a971-230b2cf50730), Ver: v1, Flags: 10, 
[{"com.cloud.agent.api.StartAnswer":{"vm":{"id":144,"name":"i-2-144-QA","type":"User","cpus":1,"minSpeed":500,"maxSpeed":500,"minRam":536870912,"maxRam":536870912,"arch":"x86_64","os":"CentOS
 5.3 
(32-bit)","bootArgs":"","enableHA":false,"limitCpuUse":false,"enableDynamicallyScaleVm":false,"vncPassword":"fE8JPrJ_58va5JFGCvzpYg","params":{"deployvm":"true"},"uuid":"5fb88c3e-3879-45b0-9a34-47dc1b82b8a5","disks":[{"data":{"org.apache.cloudstack.storage.to.VolumeObjectTO":{"uuid":"55ca5f13-f7e3-4e04-8cfe-e35e1c4f6e7f","volumeType":"ROOT","dataStore":{"org.apache.cloudstack.storage.to.PrimaryDataStoreTO":{"uuid":"7c07ec9b-a3c6-3466-ab5a-f5669ead0b22","id":1,"poolType":"NetworkFilesystem","host":"10.147.28.6","path":"/export/home/sandbox/primary0","port":2
 
049,"url":"NetworkFilesystem://10.147.28.6/export/home/sandbox/primary0/?ROLE=Primary&STOREUUID=7c07ec9b-a3c6-3466-ab5a-f5669ead0b22"}},"name":"ROOT-144","size":100,"path":"8f44da97-efa5-400e-8b9c-fcef0399c9e4","volumeId":150,"vmName":"i-2-144-QA","accountId":2,"format":"RAW","provisioningType":"THIN","id":150,"deviceId":0,"hypervisorType":"Simulator"}},"diskSeq":0,"path":"8f44da97-efa5-400e-8b9c-fcef0399c9e4","type":"ROOT","_details":{"managed":"false","storagePort":"2049","storageHost":"10.147.28.6","volumeSize":"100"}},{"data":{"org.apache.cloudstack.storage.to.TemplateObjectTO":{"id":0,"format":"ISO","accountId":0,"hvm":false}},"diskSeq":3,"type":"ISO"}],"nics":[{"deviceId":0,"networkRateMbps":200,"defaultNic":true,"pxeDisable":false,"nicUuid":"f6da6502-7971-4b34-903b-15ba11247338","uuid":"4156249e-2b1f-49cb-b333-c2032c0792c5","ip":"10.1.1.172","netmask":"255.255.255.0","gateway":"10.1.1.1","mac":"02:00:04:52:00:8a","dns1":"10.147.28.6","broadcastType":"Vlan","trafficId":1,"type
 
":"Guest","broadcastUri":"vlan://108","isolationUri":"vlan://108","isSecurityGroupEnabled":false}]},"result":true,"wait":0}}]
 }
2015-12-02 12:40:30,266 DEBUG [c.c.a.t.Request] 
(Work-Job-Executor-1:ctx-5ea59ec0 job-939/job-940 ctx-73b5d0fa) 
(logid:62314271) Seq 2-852024754503155726: Received:  { Ans: , MgmtId: 1, via: 
2(SimulatedAgent.40265aab-4037-41f7-a971-230b2cf50730), Ver: v1, Flags: 10, { 
StartAnswer } }
2015-12-02 12:40:30,302 DEBUG [c.c.c.CapacityManagerImpl] 
(Work-Job-Executor-1:ctx-5ea59ec0 job-939/job-940 ctx-73b5d0fa) 
(logid:62314271) VM state transitted from :Starting to Running with event: 
OperationSucceededvm's original host id: 2 new host id: 2 host id before state 
transition: 2


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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-8845: set isRevertable of snap...

[ustcweizhou]

GitHub user ustcweizhou opened a pull request:

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

CLOUDSTACK-8845: set isRevertable of snapshot to false if the volume is 
removed

Some users encounter an exception when listSnapshots.
We should set the isRevertable of snapshot to false if the original volume 
is removed, without checking if the snapshot is stored in primary store (the 
exception was thowned during the checking).

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

$ git pull https://github.com/ustcweizhou/cloudstack listSnapshots-Exception

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

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


commit 52412286c657d384f86441095f5cbef510dd8763
Author: Wei Zhou 
Date:   2015-12-04T07:18:09Z

CLOUDSTACK-8845: set isRevertable of snapshot to false if the volume is 
removed




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

[bhaisaab]

Github user bhaisaab closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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: debian: allow rules to pick ACS_BUILD_OPT...

[bhaisaab]

GitHub user bhaisaab opened a pull request:

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

debian: allow rules to pick ACS_BUILD_OPTS from env

Only now debian builds can be noredist etc.

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

cc @remibergsma 

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/1169.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 #1169


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

[bhaisaab]

Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1149#issuecomment-161898597
  
@remibergsma thanks, opened another PR #1169  please merge that. Closing 
this one, as once the other PR will be merged on 4.6 and 4.6 on master; we 
won't need this PR anyway.


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

[GitHub] cloudstack pull request: CLOUDSTACK-8845: Fixed list snapshots com...

[ustcweizhou]

Github user ustcweizhou commented on the pull request:

https://github.com/apache/cloudstack/pull/1166#issuecomment-161898618
  
I just saw this PR, sorry, I created #1168  for 4.6


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

[GitHub] cloudstack pull request: CLOUDSTACK-8845: Fixed list snapshots com...

[anshul1886]

Github user anshul1886 closed the pull request at:

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


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

[Wilder Rodrigues]

Sorry for the absence… testing the RC now!

Cheers,
Wilder

> On 03 Dec 2015, at 18:52, Glenn Wagner  wrote:
> 
> Good day
> 
> +1
> 
> Centos 6,7 management server
> Xenserver 6,2
> NFS storage
> 
> Tests done ,
> Standard VM tests
> Template uploads
> ISO uploads
> Basic network functions , add and remove IPs
> VM Snapshots
> 
> Didn't get to vpc testing
> 
> Regards
> Glenn Wagner
> Shape blue. Pty ltd
> 
> 
> 
> On Thu, Dec 3, 2015 at 8:48 AM -0800, "Milamber" 
> mailto:milam...@apache.org>> wrote:
> 
> Hello,
> 
> My vote +1 (binding)
> 
> My tests :
> 1/ Fresh install of 4.6.1 (adv net) on Ubuntu 14.04 + KVM + NFS : OK
> Some standard tests (create vm, migration, HA, create network, create
> user, create ssh key, destroy vm, register template, create snapshot,
> restore snapshot, create template, ip association, ip release, static
> nat, firewall rule)
> Some tests with cloudstack ansible module (create network, register
> templates, create vm, ip, firewall rule)
> 
> 2/ On fresh install of 4.5.2 (adv net) on Ubuntu 14.04 + KVM + NFS,
> migration to 4.6.1 : OK
> Some standard tests (create vm, migration, HA, create network, create
> user, create ssh key, destroy vm, register template, create snapshot,
> restore snapshot, create template)
> Some tests with cloudstack ansible module (create network, register
> templates, create vm, ip, firewall rule)
> 
> 3/ Tests all localizations of Web UI (with a complete installation of
> CS) : local strings works wells except Spanish and Arabic (expected,
> some strings needs to be fixes on Transifex)
> 
> Thanks to RM team.
> 
> Milamber
> 
> 
> On 30/11/2015 22:12, 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)
>> 
> 
> 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.