Re: Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
Thanks for pointing this out.

I can incorporate those changes in the morning and see if it does the trick.

> On Feb 15, 2017, at 12:50 AM, Rene Moser  wrote:
> 
> Probably related to https://github.com/apache/cloudstack/pull/1927
> 
> 
> 
>> On 02/14/2017 10:44 PM, Tutkowski, Mike wrote:
>> Hi,
>> 
>> I’m getting a NullPointerException when trying to create a zone using master.
>> 
>> Below is the relevant code in ConfigurationManagerImpl.
>> 
>> In the else block, network.getCidr() returns null and NetUtil.getCidrNetmask 
>> then throws a NullPointerException.
>> 
>> I noticed that network.getGateway() also returns null (which seems odd).
>> 
>> Thoughts on this?
>> 
>> Thanks!
>> Mike
>> 
>> public Pair> validateIpRange(final String 
>> startIP, final String endIP, final String newVlanGateway, final String 
>> newVlanNetmask, final List vlans, final boolean ipv4,
>>final boolean ipv6, String ip6Gateway, String ip6Cidr, final String 
>> startIPv6, final String endIPv6, final Network network) {
>>String vlanGateway = null;
>>String vlanNetmask = null;
>>boolean sameSubnet = false;
>>if (CollectionUtils.isNotEmpty(vlans)) {
>>for (final VlanVO vlan : vlans) {
>>vlanGateway = vlan.getVlanGateway();
>>vlanNetmask = vlan.getVlanNetmask();
>>sameSubnet = hasSameSubnet(ipv4, vlanGateway, vlanNetmask, 
>> newVlanGateway, newVlanNetmask, startIP, endIP,
>>ipv6, ip6Gateway, ip6Cidr, startIPv6, endIPv6, network);
>>if (sameSubnet) break;
>>}
>>} else {
>>vlanGateway = network.getGateway();
>>vlanNetmask = NetUtils.getCidrNetmask(network.getCidr());
>> 


Re: Can't create a zone using master

2017-02-14 Thread Rene Moser
Probably related to https://github.com/apache/cloudstack/pull/1927



On 02/14/2017 10:44 PM, Tutkowski, Mike wrote:
> Hi,
> 
> I’m getting a NullPointerException when trying to create a zone using master.
> 
> Below is the relevant code in ConfigurationManagerImpl.
> 
> In the else block, network.getCidr() returns null and NetUtil.getCidrNetmask 
> then throws a NullPointerException.
> 
> I noticed that network.getGateway() also returns null (which seems odd).
> 
> Thoughts on this?
> 
> Thanks!
> Mike
> 
> public Pair> validateIpRange(final String 
> startIP, final String endIP, final String newVlanGateway, final String 
> newVlanNetmask, final List vlans, final boolean ipv4,
> final boolean ipv6, String ip6Gateway, String ip6Cidr, final String 
> startIPv6, final String endIPv6, final Network network) {
> String vlanGateway = null;
> String vlanNetmask = null;
> boolean sameSubnet = false;
> if (CollectionUtils.isNotEmpty(vlans)) {
> for (final VlanVO vlan : vlans) {
> vlanGateway = vlan.getVlanGateway();
> vlanNetmask = vlan.getVlanNetmask();
> sameSubnet = hasSameSubnet(ipv4, vlanGateway, vlanNetmask, 
> newVlanGateway, newVlanNetmask, startIP, endIP,
> ipv6, ip6Gateway, ip6Cidr, startIPv6, endIPv6, network);
> if (sameSubnet) break;
> }
> } else {
> vlanGateway = network.getGateway();
> vlanNetmask = NetUtils.getCidrNetmask(network.getCidr());
> 


[GitHub] cloudstack issue #672: Allow VM snapshots and volume snapshots to exist toge...

2017-02-14 Thread anshul1886
Github user anshul1886 commented on the issue:

https://github.com/apache/cloudstack/pull/672
  
@rhtyd @karuturi Should I close this PR then?


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


[GitHub] cloudstack issue #672: Allow VM snapshots and volume snapshots to exist toge...

2017-02-14 Thread anshul1886
Github user anshul1886 commented on the issue:

https://github.com/apache/cloudstack/pull/672
  
@rhtyd @karuturi @syed Raised #1941 against 4.9. 


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

2017-02-14 Thread anshul1886
GitHub user anshul1886 opened a pull request:

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

CLOUDSTACK-8663: Fixed various issues to allow VM snapshots and volume

snapshots to exist together

Reverting VM to disk only snapshot in Xenserver corrupts VM

Stale NFS secondary storage on XS leads to volume creation failure from 
snapshot

Fixed various concerns raised in #672 

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

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

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

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


commit ca84fd4ffd5d80aef9c7624365d29d7b2aeb3225
Author: Anshul Gangwar 
Date:   2015-07-24T09:15:20Z

CLOUDSTACK-8663: Fixed various issues to allow VM snapshots and volume
snapshots to exist together

Reverting VM to disk only snapshot in Xenserver corrupts VM

Stale NFS secondary storage on XS leads to volume creation failure from 
snapshot




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


[GitHub] cloudstack issue #672: Allow VM snapshots and volume snapshots to exist toge...

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/672
  
Thanks @anshul1886 kindly also squash your changes and fix the commit 
message/jira id etc


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #672: Allow VM snapshots and volume snapshots to exist toge...

2017-02-14 Thread anshul1886
Github user anshul1886 commented on the issue:

https://github.com/apache/cloudstack/pull/672
  
Resolving conflicts and rebasing the PR against 4.9


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1836: [4.10/master] Smoketest Health

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1836
  
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been 
kicked to run smoke tests


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


[GitHub] cloudstack issue #1836: [4.10/master] Smoketest Health

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1836
  
@blueorangutan test


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


[GitHub] cloudstack issue #1836: [4.10/master] Smoketest Health

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1836
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-486


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #672: Allow VM snapshots and volume snapshots to exist toge...

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/672
  
@anshul1886 I would suggest, if your PR already has @syed 's fix along with 
other useful fixes -- please rebase against 4.9, fix the conflicts and we can 
help review/test your PR if you've time/bandwidth? Alternatively, we can merge 
@syed 's PR first since it's a rather small change and already has 
review/tests, and we can merge/test your PR once that is ready for 
review/test/merge?



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


[GitHub] cloudstack issue #672: Allow VM snapshots and volume snapshots to exist toge...

2017-02-14 Thread anshul1886
Github user anshul1886 commented on the issue:

https://github.com/apache/cloudstack/pull/672
  
@karuturi @rhtyd @syed PR  #1829 is subset of this PR i.e. that contains 
the same fix as one commit of this PR. How to proceed with that?


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


[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2017-02-14 Thread swill
Github user swill commented on the issue:

https://github.com/apache/cloudstack/pull/1741
  
thank you sir.  :)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1836: [4.10/master] Smoketest Health

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1836
  
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1836: [4.10/master] Smoketest Health

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1836
  
@blueorangutan package


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


[GitHub] cloudstack issue #1928: Update hypervisor capabilities for xenserver 7

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1928
  
@syed simply moving this to a file is not enough. Please see 
Upgrade4910to4920.java and the class usage to see how upgrade paths are 
implemented. We'll need a Upgrade4920to4930 upgrade path, after wards you can 
include this sql file in that path/class.


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

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1741
  
Thanks @swill @jayapalu for the work. I'll build a new systemvmtemplate to 
use with upcoming Trilian tests on master/4.10+.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #672: Allow VM snapshots and volume snapshots to exist toge...

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/672
  
@anshul1886 can you retarget this PR for 4.9? Also rebase against 4.9, fix 
conflicts thanks. /cc @karuturi @syed @koushik-das 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/1829
  
LGTM. @karuturi this can be 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.
---


[GitHub] cloudstack issue #865: CLOUDSTACK-8856 Primary Storage Used(type tag with va...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue:

https://github.com/apache/cloudstack/pull/865
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 340
 Hypervisor xenserver
 NetworkType Advanced
 Passed=103
 Failed=1
 Skipped=7

_Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0


**Failed tests:**
* test_non_contigiousvlan.py

 * test_extendPhysicalNetworkVlan Failed


**Skipped tests:**
test_01_test_vm_volume_snapshot
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_11_ss_nfs_version_on_ssvm
test_nested_virtualization_vmware
test_3d_gpu_support
test_deploy_vgpu_enabled_vm

**Passed test suits:**
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_routers.py
test_reset_vm_on_reboot.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_login.py
test_deploy_vm_iso.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_volumes.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_routers_network_ops.py
test_disk_offerings.py


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


[GitHub] cloudstack pull request #1939: CLOUDSTACK-8886: Limitations is listUsageReco...

2017-02-14 Thread asfgit
Github user asfgit closed the pull request at:

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


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

2017-02-14 Thread asfgit
Github user asfgit closed the pull request at:

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


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

2017-02-14 Thread karuturi
Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1939
  
merging


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1838: CLOUDSTACK-9682: Block VM migration to a storage whi...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue:

https://github.com/apache/cloudstack/pull/1838
  
 ### ACS CI BVT Run
 **Sumarry:**
 Build Number 336
 Hypervisor xenserver
 NetworkType Advanced
 Passed=104
 Failed=0
 Skipped=7

_Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0


**Failed tests:**

**Skipped tests:**
test_01_test_vm_volume_snapshot
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_11_ss_nfs_version_on_ssvm
test_nested_virtualization_vmware
test_3d_gpu_support
test_deploy_vgpu_enabled_vm

**Passed test suits:**
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_routers.py
test_reset_vm_on_reboot.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_non_contigiousvlan.py
test_login.py
test_deploy_vm_iso.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_volumes.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_routers_network_ops.py
test_disk_offerings.py


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


[GitHub] cloudstack issue #1246: CLOUDSTACK-9165 unable to use reserved IP range in a...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue:

https://github.com/apache/cloudstack/pull/1246
  
 ### ACS CI BVT Run
 **Sumarry:**
 Build Number 328
 Hypervisor xenserver
 NetworkType Advanced
 Passed=103
 Failed=1
 Skipped=7

_Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0


**Failed tests:**
* test_routers_network_ops.py

 * test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failed


**Skipped tests:**
test_01_test_vm_volume_snapshot
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_11_ss_nfs_version_on_ssvm
test_nested_virtualization_vmware
test_3d_gpu_support
test_deploy_vgpu_enabled_vm

**Passed test suits:**
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_routers.py
test_reset_vm_on_reboot.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_non_contigiousvlan.py
test_login.py
test_deploy_vm_iso.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_volumes.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_disk_offerings.py


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


[GitHub] cloudstack issue #1252: CLOUDSTACK-9182: Some running VMs turned off on manu...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue:

https://github.com/apache/cloudstack/pull/1252
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 327
 Hypervisor xenserver
 NetworkType Advanced
 Passed=103
 Failed=1
 Skipped=7

_Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0


**Failed tests:**
* test_routers_network_ops.py

 * test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failed


**Skipped tests:**
test_01_test_vm_volume_snapshot
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_11_ss_nfs_version_on_ssvm
test_nested_virtualization_vmware
test_3d_gpu_support
test_deploy_vgpu_enabled_vm

**Passed test suits:**
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_routers.py
test_reset_vm_on_reboot.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_non_contigiousvlan.py
test_login.py
test_deploy_vm_iso.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_volumes.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_disk_offerings.py


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


[GitHub] cloudstack issue #872: Strongswan vpn feature

2017-02-14 Thread jayapalu
Github user jayapalu commented on the issue:

https://github.com/apache/cloudstack/pull/872
  
PR#1741 is closed. So closed this PR.


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


[GitHub] cloudstack pull request #872: Strongswan vpn feature

2017-02-14 Thread jayapalu
Github user jayapalu closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1900: CLOUDSTACK-8862: Introduced new state attaching for ...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue:

https://github.com/apache/cloudstack/pull/1900
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 324
 Hypervisor xenserver
 NetworkType Advanced
 Passed=103
 Failed=1
 Skipped=7

_Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0


**Failed tests:**
* test_routers_network_ops.py

 * test_03_RVR_Network_check_router_state Failed


**Skipped tests:**
test_01_test_vm_volume_snapshot
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_11_ss_nfs_version_on_ssvm
test_nested_virtualization_vmware
test_3d_gpu_support
test_deploy_vgpu_enabled_vm

**Passed test suits:**
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_routers.py
test_reset_vm_on_reboot.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_non_contigiousvlan.py
test_login.py
test_deploy_vm_iso.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_volumes.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_disk_offerings.py


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


[GitHub] cloudstack issue #843: Security group ingress/egress issues with xenserver 6...

2017-02-14 Thread jayapalu
Github user jayapalu commented on the issue:

https://github.com/apache/cloudstack/pull/843
  
Rebased against master.  This PR changes are for basic zone. CI runs test 
cases for advanced zone.
So tested manually.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1868: CLOUDSTACK-9707: While using hostid parameter, vm ge...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue:

https://github.com/apache/cloudstack/pull/1868
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 323
 Hypervisor xenserver
 NetworkType Advanced
 Passed=104
 Failed=0
 Skipped=7

_Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0


**Failed tests:**

**Skipped tests:**
test_01_test_vm_volume_snapshot
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_11_ss_nfs_version_on_ssvm
test_nested_virtualization_vmware
test_3d_gpu_support
test_deploy_vgpu_enabled_vm

**Passed test suits:**
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_routers.py
test_reset_vm_on_reboot.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_non_contigiousvlan.py
test_login.py
test_deploy_vm_iso.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_volumes.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_routers_network_ops.py
test_disk_offerings.py


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


[GitHub] cloudstack issue #1867: CLOUDSTACK-9706: Added snapshots cleanup in start an...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue:

https://github.com/apache/cloudstack/pull/1867
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 322
 Hypervisor xenserver
 NetworkType Advanced
 Passed=103
 Failed=1
 Skipped=7

_Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0


**Failed tests:**
* test_routers_network_ops.py

 * test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failed


**Skipped tests:**
test_01_test_vm_volume_snapshot
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_11_ss_nfs_version_on_ssvm
test_nested_virtualization_vmware
test_3d_gpu_support
test_deploy_vgpu_enabled_vm

**Passed test suits:**
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_routers.py
test_reset_vm_on_reboot.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_non_contigiousvlan.py
test_login.py
test_deploy_vm_iso.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_volumes.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_disk_offerings.py


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


[GitHub] cloudstack issue #1865: CLOUDSTACK-9705: Unauthenticated API allows Admin pa...

2017-02-14 Thread cloudmonger
Github user cloudmonger commented on the issue:

https://github.com/apache/cloudstack/pull/1865
  
 ### ACS CI BVT Run
 **Sumarry:**
 Build Number 321
 Hypervisor xenserver
 NetworkType Advanced
 Passed=104
 Failed=0
 Skipped=7

_Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0


**Failed tests:**

**Skipped tests:**
test_01_test_vm_volume_snapshot
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_11_ss_nfs_version_on_ssvm
test_nested_virtualization_vmware
test_3d_gpu_support
test_deploy_vgpu_enabled_vm

**Passed test suits:**
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_routers.py
test_reset_vm_on_reboot.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_non_contigiousvlan.py
test_login.py
test_deploy_vm_iso.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_volumes.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_routers_network_ops.py
test_disk_offerings.py


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


[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-14 Thread nvazquez
Github user nvazquez commented on the issue:

https://github.com/apache/cloudstack/pull/1918
  
@rafaelweingartner please ommit my last comment, service offering didn't 
use reserve CPU, so it didn't limit CPU speed to 500Mhz, it used all the 
available frecuency of the CPU instead.


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


[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-14 Thread nvazquez
Github user nvazquez commented on the issue:

https://github.com/apache/cloudstack/pull/1918
  
@rafaelweingartner sorry for the last example, actually I think we've found 
a bug on change service offering, as I've been using that instance for changing 
its service offering when it had vm snapshots for PR #1727. Looking at 
`/proc/cpuinfo` I could see that CPU speed was 2.6Ghz, as vCenter shows, and 
not 500Mhz as listed on Management Server, so CPU speed was not properly set 
after changing service offering. I'll work on this.


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


Re: Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
If it helps in debugging, I typically assign 10.117.40.20 - 10.117.40.25 for 
Management IPs and 10.117.40.26 - 10.117.40.99 for Guest IPs.

On the network in my lab, I am assigned the following range (by our IT people):

10.117.40.0 /25 (10.117.40.0 - 10.117.40.127)

Subnet Mask: 255.255.255.128
Gateway: 10.117.40.126

This worked fine up until I rebased against master recently.

Thanks!
Mike

On 2/14/17, 6:20 PM, "Tutkowski, Mike"  wrote:

Also, for any CloudStack networking person out there who is curious about 
this, I see the following in cloud.log for my SSVM:

Wed Feb 15 01:07:50 UTC 2017 Incompleted parameters STORAGE_IP:, 
STORAGE_NETMASK:, STORAGE_CIDR:. Cannot setup storage network
Wed Feb 15 01:07:50 UTC 2017 Not setting up route of RFC1918 space to 
10.117.40.126 befause 10.117.40.93 is RFC1918.

On 2/14/17, 6:14 PM, "Tutkowski, Mike"  wrote:

I’m not very familiar with the networking code that’s failing, but I 
worked around it with these changes:


https://github.com/mike-tutkowski/cloudstack/commit/834b7ea7df18f278cbc9eb1bfaa9578fad2debeb

I’m not sure if those are viable changes in this case, but perhaps 
someone who works in the networking area of CloudStack can say.

The problem now, however (and maybe this is unrelated), is that neither 
my CPVM nor my SSVM report their agent as Running.

Those system VMs do show as each having public, private, and link local 
IP addresses (as well as an IP address for their gateway).

Is anyone else having these kinds of issues with recent master code?

Thanks!
Mike

On 2/14/17, 3:02 PM, "Tutkowski, Mike"  
wrote:

By the way, this is for a Basic Zone.

On 2/14/17, 2:44 PM, "Tutkowski, Mike"  
wrote:

Hi,

I’m getting a NullPointerException when trying to create a zone 
using master.

Below is the relevant code in ConfigurationManagerImpl.

In the else block, network.getCidr() returns null and 
NetUtil.getCidrNetmask then throws a NullPointerException.

I noticed that network.getGateway() also returns null (which 
seems odd).

Thoughts on this?

Thanks!
Mike

public Pair> 
validateIpRange(final String startIP, final String endIP, final String 
newVlanGateway, final String newVlanNetmask, final List vlans, final 
boolean ipv4,
final boolean ipv6, String ip6Gateway, String ip6Cidr, 
final String startIPv6, final String endIPv6, final Network network) {
String vlanGateway = null;
String vlanNetmask = null;
boolean sameSubnet = false;
if (CollectionUtils.isNotEmpty(vlans)) {
for (final VlanVO vlan : vlans) {
vlanGateway = vlan.getVlanGateway();
vlanNetmask = vlan.getVlanNetmask();
sameSubnet = hasSameSubnet(ipv4, vlanGateway, 
vlanNetmask, newVlanGateway, newVlanNetmask, startIP, endIP,
ipv6, ip6Gateway, ip6Cidr, startIPv6, 
endIPv6, network);
if (sameSubnet) break;
}
} else {
vlanGateway = network.getGateway();
vlanNetmask = 
NetUtils.getCidrNetmask(network.getCidr());











Re: Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
Also, for any CloudStack networking person out there who is curious about this, 
I see the following in cloud.log for my SSVM:

Wed Feb 15 01:07:50 UTC 2017 Incompleted parameters STORAGE_IP:, 
STORAGE_NETMASK:, STORAGE_CIDR:. Cannot setup storage network
Wed Feb 15 01:07:50 UTC 2017 Not setting up route of RFC1918 space to 
10.117.40.126 befause 10.117.40.93 is RFC1918.

On 2/14/17, 6:14 PM, "Tutkowski, Mike"  wrote:

I’m not very familiar with the networking code that’s failing, but I worked 
around it with these changes:


https://github.com/mike-tutkowski/cloudstack/commit/834b7ea7df18f278cbc9eb1bfaa9578fad2debeb

I’m not sure if those are viable changes in this case, but perhaps someone 
who works in the networking area of CloudStack can say.

The problem now, however (and maybe this is unrelated), is that neither my 
CPVM nor my SSVM report their agent as Running.

Those system VMs do show as each having public, private, and link local IP 
addresses (as well as an IP address for their gateway).

Is anyone else having these kinds of issues with recent master code?

Thanks!
Mike

On 2/14/17, 3:02 PM, "Tutkowski, Mike"  wrote:

By the way, this is for a Basic Zone.

On 2/14/17, 2:44 PM, "Tutkowski, Mike"  
wrote:

Hi,

I’m getting a NullPointerException when trying to create a zone 
using master.

Below is the relevant code in ConfigurationManagerImpl.

In the else block, network.getCidr() returns null and 
NetUtil.getCidrNetmask then throws a NullPointerException.

I noticed that network.getGateway() also returns null (which seems 
odd).

Thoughts on this?

Thanks!
Mike

public Pair> validateIpRange(final 
String startIP, final String endIP, final String newVlanGateway, final String 
newVlanNetmask, final List vlans, final boolean ipv4,
final boolean ipv6, String ip6Gateway, String ip6Cidr, 
final String startIPv6, final String endIPv6, final Network network) {
String vlanGateway = null;
String vlanNetmask = null;
boolean sameSubnet = false;
if (CollectionUtils.isNotEmpty(vlans)) {
for (final VlanVO vlan : vlans) {
vlanGateway = vlan.getVlanGateway();
vlanNetmask = vlan.getVlanNetmask();
sameSubnet = hasSameSubnet(ipv4, vlanGateway, 
vlanNetmask, newVlanGateway, newVlanNetmask, startIP, endIP,
ipv6, ip6Gateway, ip6Cidr, startIPv6, endIPv6, 
network);
if (sameSubnet) break;
}
} else {
vlanGateway = network.getGateway();
vlanNetmask = NetUtils.getCidrNetmask(network.getCidr());









Re: Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
I’m not very familiar with the networking code that’s failing, but I worked 
around it with these changes:

https://github.com/mike-tutkowski/cloudstack/commit/834b7ea7df18f278cbc9eb1bfaa9578fad2debeb

I’m not sure if those are viable changes in this case, but perhaps someone who 
works in the networking area of CloudStack can say.

The problem now, however (and maybe this is unrelated), is that neither my CPVM 
nor my SSVM report their agent as Running.

Those system VMs do show as each having public, private, and link local IP 
addresses (as well as an IP address for their gateway).

Is anyone else having these kinds of issues with recent master code?

Thanks!
Mike

On 2/14/17, 3:02 PM, "Tutkowski, Mike"  wrote:

By the way, this is for a Basic Zone.

On 2/14/17, 2:44 PM, "Tutkowski, Mike"  wrote:

Hi,

I’m getting a NullPointerException when trying to create a zone using 
master.

Below is the relevant code in ConfigurationManagerImpl.

In the else block, network.getCidr() returns null and 
NetUtil.getCidrNetmask then throws a NullPointerException.

I noticed that network.getGateway() also returns null (which seems odd).

Thoughts on this?

Thanks!
Mike

public Pair> validateIpRange(final String 
startIP, final String endIP, final String newVlanGateway, final String 
newVlanNetmask, final List vlans, final boolean ipv4,
final boolean ipv6, String ip6Gateway, String ip6Cidr, final 
String startIPv6, final String endIPv6, final Network network) {
String vlanGateway = null;
String vlanNetmask = null;
boolean sameSubnet = false;
if (CollectionUtils.isNotEmpty(vlans)) {
for (final VlanVO vlan : vlans) {
vlanGateway = vlan.getVlanGateway();
vlanNetmask = vlan.getVlanNetmask();
sameSubnet = hasSameSubnet(ipv4, vlanGateway, vlanNetmask, 
newVlanGateway, newVlanNetmask, startIP, endIP,
ipv6, ip6Gateway, ip6Cidr, startIPv6, endIPv6, 
network);
if (sameSubnet) break;
}
} else {
vlanGateway = network.getGateway();
vlanNetmask = NetUtils.getCidrNetmask(network.getCidr());







[GitHub] cloudstack-www pull request #25: Updated list with makewholist.py

2017-02-14 Thread asfgit
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1829
  
Trillian test result (tid-821)
Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7
Total time taken: 38820 seconds
Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1829-t821-xenserver-65sp1.zip
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Test completed. 46 look ok, 2 have error(s)


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_05_rvpc_multi_tiers | `Failure` | 527.66 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | `Failure` | 1345.35 | 
test_vpc_redundant.py
test_02_redundant_VPC_default_routes | `Failure` | 353.82 | 
test_vpc_redundant.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 555.85 
| test_vpc_redundant.py
test_04_rvpc_privategw_static_routes | `Failure` | 736.79 | 
test_privategw_acl.py
test_01_vpc_site2site_vpn | Success | 320.48 | test_vpc_vpn.py
test_01_vpc_remote_access_vpn | Success | 151.41 | test_vpc_vpn.py
test_01_redundant_vpc_site2site_vpn | Success | 506.02 | test_vpc_vpn.py
test_02_VPC_default_routes | Success | 313.39 | test_vpc_router_nics.py
test_01_VPC_nics_after_destroy | Success | 661.28 | test_vpc_router_nics.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 867.14 | test_vpc_redundant.py
test_09_delete_detached_volume | Success | 15.53 | test_volumes.py
test_08_resize_volume | Success | 100.66 | test_volumes.py
test_07_resize_fail | Success | 105.69 | test_volumes.py
test_06_download_detached_volume | Success | 20.24 | test_volumes.py
test_05_detach_volume | Success | 100.21 | test_volumes.py
test_04_delete_attached_volume | Success | 10.14 | test_volumes.py
test_03_download_attached_volume | Success | 15.19 | test_volumes.py
test_02_attach_volume | Success | 10.64 | test_volumes.py
test_01_create_volume | Success | 392.06 | test_volumes.py
test_03_delete_vm_snapshots | Success | 280.17 | test_vm_snapshots.py
test_02_revert_vm_snapshots | Success | 175.94 | test_vm_snapshots.py
test_01_create_vm_snapshots | Success | 130.74 | test_vm_snapshots.py
test_deploy_vm_multiple | Success | 191.61 | test_vm_life_cycle.py
test_deploy_vm | Success | 0.02 | test_vm_life_cycle.py
test_advZoneVirtualRouter | Success | 0.02 | test_vm_life_cycle.py
test_10_attachAndDetach_iso | Success | 26.57 | test_vm_life_cycle.py
test_09_expunge_vm | Success | 125.15 | test_vm_life_cycle.py
test_08_migrate_vm | Success | 70.78 | test_vm_life_cycle.py
test_07_restore_vm | Success | 0.07 | test_vm_life_cycle.py
test_06_destroy_vm | Success | 10.10 | test_vm_life_cycle.py
test_03_reboot_vm | Success | 10.12 | test_vm_life_cycle.py
test_02_start_vm | Success | 20.16 | test_vm_life_cycle.py
test_01_stop_vm | Success | 25.18 | test_vm_life_cycle.py
test_CreateTemplateWithDuplicateName | Success | 80.49 | test_templates.py
test_08_list_system_templates | Success | 0.02 | test_templates.py
test_07_list_public_templates | Success | 0.03 | test_templates.py
test_05_template_permissions | Success | 0.04 | test_templates.py
test_04_extract_template | Success | 5.10 | test_templates.py
test_03_delete_template | Success | 5.08 | test_templates.py
test_02_edit_template | Success | 90.13 | test_templates.py
test_01_create_template | Success | 45.33 | test_templates.py
test_10_destroy_cpvm | Success | 191.53 | test_ssvm.py
test_09_destroy_ssvm | Success | 198.64 | test_ssvm.py
test_08_reboot_cpvm | Success | 111.43 | test_ssvm.py
test_07_reboot_ssvm | Success | 143.68 | test_ssvm.py
test_06_stop_cpvm | Success | 161.51 | test_ssvm.py
test_05_stop_ssvm | Success | 138.77 | test_ssvm.py
test_04_cpvm_internals | Success | 1.15 | test_ssvm.py
test_03_ssvm_internals | Success | 3.47 | test_ssvm.py
test_02_list_cpvm_vm | Success | 0.09 | test_ssvm.py
test_01_list_sec_storage_vm | Success | 0.09 | test_ssvm.py
test_01_snapshot_root_disk | Success | 20.94 | test_snapshots.py
test_04_change_offering_small | Success | 128.93 | test_service_offerings.py
test_03_delete_service_offering | Success | 0.03 | test_service_offerings.py
test_02_edit_service_offering | Success | 0.04 | test_service_offerings.py
test_01_create_service_offering | Success | 0.06 | test_service_offerings.py
test_02_sys_template_ready | Success | 0.09 | test_secondary_storage.py
test_01_sys_vm_start | Success | 0.12 | test_secondary_storage.py
test_01_scale_vm | Success | 5.12 | test_scale_vm.py
test_09_reboot_router | Success | 60.46 | test_routers.py
test_08_start_router | Success | 45.27 | test_routers.py

Re: Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
By the way, this is for a Basic Zone.

On 2/14/17, 2:44 PM, "Tutkowski, Mike"  wrote:

Hi,

I’m getting a NullPointerException when trying to create a zone using 
master.

Below is the relevant code in ConfigurationManagerImpl.

In the else block, network.getCidr() returns null and 
NetUtil.getCidrNetmask then throws a NullPointerException.

I noticed that network.getGateway() also returns null (which seems odd).

Thoughts on this?

Thanks!
Mike

public Pair> validateIpRange(final String 
startIP, final String endIP, final String newVlanGateway, final String 
newVlanNetmask, final List vlans, final boolean ipv4,
final boolean ipv6, String ip6Gateway, String ip6Cidr, final String 
startIPv6, final String endIPv6, final Network network) {
String vlanGateway = null;
String vlanNetmask = null;
boolean sameSubnet = false;
if (CollectionUtils.isNotEmpty(vlans)) {
for (final VlanVO vlan : vlans) {
vlanGateway = vlan.getVlanGateway();
vlanNetmask = vlan.getVlanNetmask();
sameSubnet = hasSameSubnet(ipv4, vlanGateway, vlanNetmask, 
newVlanGateway, newVlanNetmask, startIP, endIP,
ipv6, ip6Gateway, ip6Cidr, startIPv6, endIPv6, network);
if (sameSubnet) break;
}
} else {
vlanGateway = network.getGateway();
vlanNetmask = NetUtils.getCidrNetmask(network.getCidr());





Re: PRs for 4.10

2017-02-14 Thread Syed Ahmed
I'd like to include https://github.com/apache/cloudstack/pull/1928 to 4.10
as well. This is a simple fix that adds hypervisor capabilities for
XenServer 7

Thanks,
-Syed

On Tue, Feb 14, 2017 at 12:06 AM, Will Stevens 
wrote:

> Not sure, I will see if I can find some time tomorrow to look at this.
> Thanks...
>
> *Will STEVENS*
> Lead Developer
>
> 
>
> On Mon, Feb 13, 2017 at 11:58 PM, Rajani Karuturi 
> wrote:
>
> > Thanks Will. I will take a look at this today and merge.
> >
> > Is it possible to add a systemvmtemplate job on
> > builds.apachecloudstack.net (https://builds.cloudstack.org/)
> > similar to
> > http://jenkins.buildacloud.org/job/build-systemvm64-master/ ?
> >
> > Thanks,
> >
> > ~ Rajani
> >
> > http://cloudplatform.accelerite.com/
> >
> > On February 13, 2017 at 10:46 PM, Will Stevens
> > (wstev...@cloudops.com) wrote:
> >
> > At this point, I think 1741 is ready to merge. I think we need
> > one more
> > LGTM though...
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > 
> >
> > On Wed, Feb 8, 2017 at 11:32 AM, Will Stevens
> >  wrote:
> >
> > @Daan: In order for the tests to pass with 1741, the tests have
> > to be run
> > with the SystemVM from the PR being used.
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > 
> >
> > On Wed, Feb 8, 2017 at 2:46 AM, Daan Hoogland
> > 
> > wrote:
> >
> > About 1741, I started the packaging, Will but saw Boris making a
> > remark about the system VMs. I'll look into that and start the
> > tests.
> >
> > On Tue, Feb 7, 2017 at 8:19 PM, Will Stevens
> > 
> > wrote:
> >
> > I would like to get 1741(strongswan) in, but I need to get
> > Trillian to
> >
> > run
> >
> > against it with a system vm built from my pr. I have done a CI in
> > bubble
> > and it came back clean, but would like confirmation through
> > Trillian as
> > well.
> >
> > On Feb 7, 2017 2:17 PM, "Milamber"  wrote:
> >
> > And this PR for 4.9 but need to be foward to 4.10 too.
> >
> > https://github.com/apache/cloudstack/pull/1929
> >
> > 2 LGTM
> > CI green
> >
> > On 07/02/2017 13:07, Rajani Karuturi wrote:
> >
> > Hi all,
> > If you know of any PRs which meet the below criteria, please
> > mail
> >
> > them to
> >
> > me.
> > 1. No open questions.
> > 2. Has two LGTMs
> > 3. Has CI test results.
> >
> > ~Rajani
> > http://cloudplatform.accelerite.com
> >
> > --
> > Daan
> >
>


[GitHub] cloudstack issue #1928: Update hypervisor capabilities for xenserver 7

2017-02-14 Thread syed
Github user syed commented on the issue:

https://github.com/apache/cloudstack/pull/1928
  
Thanks @rhtyd for the review. I've moved it to the schema4920to4930.sql 
file. @karuturi Can we also push this for 4.10? 



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


Can't create a zone using master

2017-02-14 Thread Tutkowski, Mike
Hi,

I’m getting a NullPointerException when trying to create a zone using master.

Below is the relevant code in ConfigurationManagerImpl.

In the else block, network.getCidr() returns null and NetUtil.getCidrNetmask 
then throws a NullPointerException.

I noticed that network.getGateway() also returns null (which seems odd).

Thoughts on this?

Thanks!
Mike

public Pair> validateIpRange(final String 
startIP, final String endIP, final String newVlanGateway, final String 
newVlanNetmask, final List vlans, final boolean ipv4,
final boolean ipv6, String ip6Gateway, String ip6Cidr, final String 
startIPv6, final String endIPv6, final Network network) {
String vlanGateway = null;
String vlanNetmask = null;
boolean sameSubnet = false;
if (CollectionUtils.isNotEmpty(vlans)) {
for (final VlanVO vlan : vlans) {
vlanGateway = vlan.getVlanGateway();
vlanNetmask = vlan.getVlanNetmask();
sameSubnet = hasSameSubnet(ipv4, vlanGateway, vlanNetmask, 
newVlanGateway, newVlanNetmask, startIP, endIP,
ipv6, ip6Gateway, ip6Cidr, startIPv6, endIPv6, network);
if (sameSubnet) break;
}
} else {
vlanGateway = network.getGateway();
vlanNetmask = NetUtils.getCidrNetmask(network.getCidr());



Re: Annotations Exception

2017-02-14 Thread Syed Ahmed
It's been happening to me recently as well Mike. I think there was some
commit after which the management server takes way too long to start
spending time scanning annotations.

On Tue, Feb 14, 2017 at 4:07 PM, Tutkowski, Mike 
wrote:

> In case this happens to anyone else, here is how I solved it (maxWait
> parameter):
>
> mvn –pl client jetty:run –Dnoredist –Dorg.eclipse.jetty.
> annotations.maxWait=240
>
> On 2/14/17, 1:20 PM, "Tutkowski, Mike"  wrote:
>
> Hi,
>
> Anyone know why I might be getting this exception starting up the
> management server (with master)?
>
> java.lang.Exception: Timeout scanning annotations
> at org.eclipse.jetty.annotations.AnnotationConfiguration.
> scanForAnnotations(AnnotationConfiguration.java:578)
> at org.eclipse.jetty.annotations.AnnotationConfiguration.
> configure(AnnotationConfiguration.java:448)
> at org.eclipse.jetty.webapp.WebAppContext.configure(
> WebAppContext.java:479)
> at org.eclipse.jetty.webapp.WebAppContext.startContext(
> WebAppContext.java:1337)
> at org.eclipse.jetty.server.handler.ContextHandler.
> doStart(ContextHandler.java:741)
> at org.eclipse.jetty.webapp.WebAppContext.doStart(
> WebAppContext.java:505)
> at org.eclipse.jetty.maven.plugin.JettyWebAppContext.
> doStart(JettyWebAppContext.java:366)
> at org.eclipse.jetty.util.component.AbstractLifeCycle.
> start(AbstractLifeCycle.java:68)
> at org.eclipse.jetty.util.component.ContainerLifeCycle.
> start(ContainerLifeCycle.java:132)
> at org.eclipse.jetty.util.component.ContainerLifeCycle.
> doStart(ContainerLifeCycle.java:114)
> at org.eclipse.jetty.server.handler.AbstractHandler.
> doStart(AbstractHandler.java:61)
> at org.eclipse.jetty.server.handler.ContextHandlerCollection.
> doStart(ContextHandlerCollection.java:163)
> at org.eclipse.jetty.util.component.AbstractLifeCycle.
> start(AbstractLifeCycle.java:68)
> at org.eclipse.jetty.util.component.ContainerLifeCycle.
> start(ContainerLifeCycle.java:132)
> at org.eclipse.jetty.util.component.ContainerLifeCycle.
> doStart(ContainerLifeCycle.java:114)
> at org.eclipse.jetty.server.handler.AbstractHandler.
> doStart(AbstractHandler.java:61)
> at org.eclipse.jetty.util.component.AbstractLifeCycle.
> start(AbstractLifeCycle.java:68)
> at org.eclipse.jetty.util.component.ContainerLifeCycle.
> start(ContainerLifeCycle.java:132)
> at org.eclipse.jetty.server.Server.start(Server.java:387)
> at org.eclipse.jetty.util.component.ContainerLifeCycle.
> doStart(ContainerLifeCycle.java:114)
> at org.eclipse.jetty.server.handler.AbstractHandler.
> doStart(AbstractHandler.java:61)
> at org.eclipse.jetty.server.Server.doStart(Server.java:354)
> at org.eclipse.jetty.maven.plugin.JettyServer.doStart(
> JettyServer.java:73)
> at org.eclipse.jetty.util.component.AbstractLifeCycle.
> start(AbstractLifeCycle.java:68)
> at org.eclipse.jetty.maven.plugin.AbstractJettyMojo.
> startJetty(AbstractJettyMojo.java:535)
> at org.eclipse.jetty.maven.plugin.AbstractJettyMojo.
> execute(AbstractJettyMojo.java:358)
> at org.eclipse.jetty.maven.plugin.JettyRunMojo.execute(
> JettyRunMojo.java:167)
> at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(
> DefaultBuildPluginManager.java:133)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute(
> MojoExecutor.java:208)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute(
> MojoExecutor.java:153)
> at org.apache.maven.lifecycle.internal.MojoExecutor.execute(
> MojoExecutor.java:145)
> at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.
> buildProject(LifecycleModuleBuilder.java:108)
> at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.
> buildProject(LifecycleModuleBuilder.java:76)
> at org.apache.maven.lifecycle.internal.builder.singlethreaded.
> SingleThreadedBuilder.build(SingleThreadedBuilder.java:51)
> at org.apache.maven.lifecycle.internal.LifecycleStarter.
> execute(LifecycleStarter.java:116)
> at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:361)
> at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:155)
> at org.apache.maven.cli.MavenCli.execute(MavenCli.java:584)
> at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:213)
> at org.apache.maven.cli.MavenCli.main(MavenCli.java:157)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at sun.reflect.NativeMethodAccessorImpl.invoke(
> NativeMethodAccessorImpl.java:62)
> at sun.reflect.DelegatingMethodAccessorImpl.invoke(
> DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at org.codehaus.plexus.classworlds.launcher.Launcher.
> 

Re: Annotations Exception

2017-02-14 Thread Tutkowski, Mike
In case this happens to anyone else, here is how I solved it (maxWait 
parameter):

mvn –pl client jetty:run –Dnoredist –Dorg.eclipse.jetty.annotations.maxWait=240

On 2/14/17, 1:20 PM, "Tutkowski, Mike"  wrote:

Hi,

Anyone know why I might be getting this exception starting up the 
management server (with master)?

java.lang.Exception: Timeout scanning annotations
at 
org.eclipse.jetty.annotations.AnnotationConfiguration.scanForAnnotations(AnnotationConfiguration.java:578)
at 
org.eclipse.jetty.annotations.AnnotationConfiguration.configure(AnnotationConfiguration.java:448)
at 
org.eclipse.jetty.webapp.WebAppContext.configure(WebAppContext.java:479)
at 
org.eclipse.jetty.webapp.WebAppContext.startContext(WebAppContext.java:1337)
at 
org.eclipse.jetty.server.handler.ContextHandler.doStart(ContextHandler.java:741)
at 
org.eclipse.jetty.webapp.WebAppContext.doStart(WebAppContext.java:505)
at 
org.eclipse.jetty.maven.plugin.JettyWebAppContext.doStart(JettyWebAppContext.java:366)
at 
org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:68)
at 
org.eclipse.jetty.util.component.ContainerLifeCycle.start(ContainerLifeCycle.java:132)
at 
org.eclipse.jetty.util.component.ContainerLifeCycle.doStart(ContainerLifeCycle.java:114)
at 
org.eclipse.jetty.server.handler.AbstractHandler.doStart(AbstractHandler.java:61)
at 
org.eclipse.jetty.server.handler.ContextHandlerCollection.doStart(ContextHandlerCollection.java:163)
at 
org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:68)
at 
org.eclipse.jetty.util.component.ContainerLifeCycle.start(ContainerLifeCycle.java:132)
at 
org.eclipse.jetty.util.component.ContainerLifeCycle.doStart(ContainerLifeCycle.java:114)
at 
org.eclipse.jetty.server.handler.AbstractHandler.doStart(AbstractHandler.java:61)
at 
org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:68)
at 
org.eclipse.jetty.util.component.ContainerLifeCycle.start(ContainerLifeCycle.java:132)
at org.eclipse.jetty.server.Server.start(Server.java:387)
at 
org.eclipse.jetty.util.component.ContainerLifeCycle.doStart(ContainerLifeCycle.java:114)
at 
org.eclipse.jetty.server.handler.AbstractHandler.doStart(AbstractHandler.java:61)
at org.eclipse.jetty.server.Server.doStart(Server.java:354)
at 
org.eclipse.jetty.maven.plugin.JettyServer.doStart(JettyServer.java:73)
at 
org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:68)
at 
org.eclipse.jetty.maven.plugin.AbstractJettyMojo.startJetty(AbstractJettyMojo.java:535)
at 
org.eclipse.jetty.maven.plugin.AbstractJettyMojo.execute(AbstractJettyMojo.java:358)
at 
org.eclipse.jetty.maven.plugin.JettyRunMojo.execute(JettyRunMojo.java:167)
at 
org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:133)
at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:208)
at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:153)
at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:145)
at 
org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:108)
at 
org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:76)
at 
org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build(SingleThreadedBuilder.java:51)
at 
org.apache.maven.lifecycle.internal.LifecycleStarter.execute(LifecycleStarter.java:116)
at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:361)
at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:155)
at org.apache.maven.cli.MavenCli.execute(MavenCli.java:584)
at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:213)
at org.apache.maven.cli.MavenCli.main(MavenCli.java:157)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at 
org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:289)
at 
org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:229)
at 
org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:415)
at 
org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:356)

Thanks!
Mike




Annotations Exception

2017-02-14 Thread Tutkowski, Mike
Hi,

Anyone know why I might be getting this exception starting up the management 
server (with master)?

java.lang.Exception: Timeout scanning annotations
at 
org.eclipse.jetty.annotations.AnnotationConfiguration.scanForAnnotations(AnnotationConfiguration.java:578)
at 
org.eclipse.jetty.annotations.AnnotationConfiguration.configure(AnnotationConfiguration.java:448)
at org.eclipse.jetty.webapp.WebAppContext.configure(WebAppContext.java:479)
at 
org.eclipse.jetty.webapp.WebAppContext.startContext(WebAppContext.java:1337)
at 
org.eclipse.jetty.server.handler.ContextHandler.doStart(ContextHandler.java:741)
at org.eclipse.jetty.webapp.WebAppContext.doStart(WebAppContext.java:505)
at 
org.eclipse.jetty.maven.plugin.JettyWebAppContext.doStart(JettyWebAppContext.java:366)
at 
org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:68)
at 
org.eclipse.jetty.util.component.ContainerLifeCycle.start(ContainerLifeCycle.java:132)
at 
org.eclipse.jetty.util.component.ContainerLifeCycle.doStart(ContainerLifeCycle.java:114)
at 
org.eclipse.jetty.server.handler.AbstractHandler.doStart(AbstractHandler.java:61)
at 
org.eclipse.jetty.server.handler.ContextHandlerCollection.doStart(ContextHandlerCollection.java:163)
at 
org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:68)
at 
org.eclipse.jetty.util.component.ContainerLifeCycle.start(ContainerLifeCycle.java:132)
at 
org.eclipse.jetty.util.component.ContainerLifeCycle.doStart(ContainerLifeCycle.java:114)
at 
org.eclipse.jetty.server.handler.AbstractHandler.doStart(AbstractHandler.java:61)
at 
org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:68)
at 
org.eclipse.jetty.util.component.ContainerLifeCycle.start(ContainerLifeCycle.java:132)
at org.eclipse.jetty.server.Server.start(Server.java:387)
at 
org.eclipse.jetty.util.component.ContainerLifeCycle.doStart(ContainerLifeCycle.java:114)
at 
org.eclipse.jetty.server.handler.AbstractHandler.doStart(AbstractHandler.java:61)
at org.eclipse.jetty.server.Server.doStart(Server.java:354)
at org.eclipse.jetty.maven.plugin.JettyServer.doStart(JettyServer.java:73)
at 
org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:68)
at 
org.eclipse.jetty.maven.plugin.AbstractJettyMojo.startJetty(AbstractJettyMojo.java:535)
at 
org.eclipse.jetty.maven.plugin.AbstractJettyMojo.execute(AbstractJettyMojo.java:358)
at 
org.eclipse.jetty.maven.plugin.JettyRunMojo.execute(JettyRunMojo.java:167)
at 
org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:133)
at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:208)
at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:153)
at 
org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:145)
at 
org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:108)
at 
org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:76)
at 
org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build(SingleThreadedBuilder.java:51)
at 
org.apache.maven.lifecycle.internal.LifecycleStarter.execute(LifecycleStarter.java:116)
at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:361)
at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:155)
at org.apache.maven.cli.MavenCli.execute(MavenCli.java:584)
at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:213)
at org.apache.maven.cli.MavenCli.main(MavenCli.java:157)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at 
org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:289)
at 
org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:229)
at 
org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:415)
at org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:356)

Thanks!
Mike


[GitHub] cloudstack issue #1792: CLOUDSTACK-9623: Deploying virtual machine fails due...

2017-02-14 Thread nitin-maharana
Github user nitin-maharana commented on the issue:

https://github.com/apache/cloudstack/pull/1792
  
ping @sateesh-chodapuneedi @karuturi 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1786: CLOUDSTACK-9618: Load Balancer configuration page do...

2017-02-14 Thread nitin-maharana
Github user nitin-maharana commented on the issue:

https://github.com/apache/cloudstack/pull/1786
  
ping @sateesh-chodapuneedi @rajesh-battala @karuturi 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1771: CLOUDSTACK-9611: Dedicating a Guest VLAN range to Pr...

2017-02-14 Thread nitin-maharana
Github user nitin-maharana commented on the issue:

https://github.com/apache/cloudstack/pull/1771
  
ping @karuturi @sateesh-chodapuneedi 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1758: CLOUDSTACK-9588: Add Load Balancer functionality in ...

2017-02-14 Thread nitin-maharana
Github user nitin-maharana commented on the issue:

https://github.com/apache/cloudstack/pull/1758
  
ping @rajesh-battala @karuturi 


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

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1939
  
Trillian test result (tid-818)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 33181 seconds
Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1939-t818-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Test completed. 47 look ok, 2 have error(s)


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_02_redundant_VPC_default_routes | `Failure` | 858.87 | 
test_vpc_redundant.py
test_04_rvpc_privategw_static_routes | `Failure` | 350.74 | 
test_privategw_acl.py
test_01_vpc_site2site_vpn | Success | 165.29 | test_vpc_vpn.py
test_01_vpc_remote_access_vpn | Success | 66.22 | test_vpc_vpn.py
test_01_redundant_vpc_site2site_vpn | Success | 255.96 | test_vpc_vpn.py
test_02_VPC_default_routes | Success | 277.32 | test_vpc_router_nics.py
test_01_VPC_nics_after_destroy | Success | 537.76 | test_vpc_router_nics.py
test_05_rvpc_multi_tiers | Success | 511.00 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | Success | 1454.05 | 
test_vpc_redundant.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 543.71 | test_vpc_redundant.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1276.54 | 
test_vpc_redundant.py
test_09_delete_detached_volume | Success | 151.44 | test_volumes.py
test_08_resize_volume | Success | 156.39 | test_volumes.py
test_07_resize_fail | Success | 161.44 | test_volumes.py
test_06_download_detached_volume | Success | 156.31 | test_volumes.py
test_05_detach_volume | Success | 155.78 | test_volumes.py
test_04_delete_attached_volume | Success | 151.19 | test_volumes.py
test_03_download_attached_volume | Success | 156.20 | test_volumes.py
test_02_attach_volume | Success | 95.37 | test_volumes.py
test_01_create_volume | Success | 711.56 | test_volumes.py
test_03_delete_vm_snapshots | Success | 275.19 | test_vm_snapshots.py
test_02_revert_vm_snapshots | Success | 95.73 | test_vm_snapshots.py
test_01_create_vm_snapshots | Success | 163.73 | test_vm_snapshots.py
test_deploy_vm_multiple | Success | 267.62 | test_vm_life_cycle.py
test_deploy_vm | Success | 0.03 | test_vm_life_cycle.py
test_advZoneVirtualRouter | Success | 0.02 | test_vm_life_cycle.py
test_10_attachAndDetach_iso | Success | 26.56 | test_vm_life_cycle.py
test_09_expunge_vm | Success | 125.25 | test_vm_life_cycle.py
test_08_migrate_vm | Success | 40.89 | test_vm_life_cycle.py
test_07_restore_vm | Success | 0.13 | test_vm_life_cycle.py
test_06_destroy_vm | Success | 130.79 | test_vm_life_cycle.py
test_03_reboot_vm | Success | 125.88 | test_vm_life_cycle.py
test_02_start_vm | Success | 10.16 | test_vm_life_cycle.py
test_01_stop_vm | Success | 40.31 | test_vm_life_cycle.py
test_CreateTemplateWithDuplicateName | Success | 40.55 | test_templates.py
test_08_list_system_templates | Success | 0.03 | test_templates.py
test_07_list_public_templates | Success | 0.04 | test_templates.py
test_05_template_permissions | Success | 0.05 | test_templates.py
test_04_extract_template | Success | 5.16 | test_templates.py
test_03_delete_template | Success | 5.10 | test_templates.py
test_02_edit_template | Success | 90.12 | test_templates.py
test_01_create_template | Success | 45.46 | test_templates.py
test_10_destroy_cpvm | Success | 161.57 | test_ssvm.py
test_09_destroy_ssvm | Success | 163.59 | test_ssvm.py
test_08_reboot_cpvm | Success | 101.46 | test_ssvm.py
test_07_reboot_ssvm | Success | 133.56 | test_ssvm.py
test_06_stop_cpvm | Success | 161.70 | test_ssvm.py
test_05_stop_ssvm | Success | 138.81 | test_ssvm.py
test_04_cpvm_internals | Success | 1.13 | test_ssvm.py
test_03_ssvm_internals | Success | 3.23 | test_ssvm.py
test_02_list_cpvm_vm | Success | 0.12 | test_ssvm.py
test_01_list_sec_storage_vm | Success | 0.13 | test_ssvm.py
test_01_snapshot_root_disk | Success | 11.26 | test_snapshots.py
test_04_change_offering_small | Success | 240.79 | test_service_offerings.py
test_03_delete_service_offering | Success | 0.04 | test_service_offerings.py
test_02_edit_service_offering | Success | 0.08 | test_service_offerings.py
test_01_create_service_offering | Success | 0.17 | test_service_offerings.py
test_02_sys_template_ready | Success | 0.12 | test_secondary_storage.py
test_01_sys_vm_start | Success | 0.17 | test_secondary_storage.py
test_09_reboot_router | Success | 35.29 | test_routers.py
test_08_start_router | Success | 30.28 | test_routers.py
test_07_stop_router | Success | 10.16 | test_routers.py

[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-14 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue:

https://github.com/apache/cloudstack/pull/1918
  
@nvazquez, this helped.

I checked the “VmwareResource.java” at line 5113 is obtained the CPU 
usage of VM. From @jayakarteek explanations I understood that the usage does 
not come as %; VMware systems show the value as a continuous value of CPU used 
in MHz.  Does not the object “objProps” (created at line 5092) contain the 
configured CPU for the VM? So, the proper conversion could be applied on 
“VmwareResource.java” at line 5186, making the POJO (VmStatsEntry) 
consistent; otherwise, there is a variable called “cpuUtilization” that 
gives the idea of percentage/ratio of usage, but in reality, it is carrying the 
real used valued of CPU in MHz.

I also understood the issue regarding the CPU cap, but do not you think 
that if users select a service offering of 500MHz and their VMs are consuming 
1GHz, it would be interesting to show that the VM is using more than they would 
expect? The cloud provider is being very generous in this case :)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 #1907: Fix public IPs not being removed from the VR ...

2017-02-14 Thread swill
Github user swill commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1907#discussion_r101062037
  
--- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py ---
@@ -659,7 +663,10 @@ def is_guest_gateway(self, bag, ip):
 gw = interface.get_gateway()
 logging.info("Interface has the following gateway ==> %s", gw)
 
-if bag['nw_type'] == "guest" and rip == gw:
+guest_gw = self.config.cmdline().get_guest_gw()
+logging.info("Interface has the following gateway ==> %s", 
guest_gw)
+
+if bag['nw_type'] == "guest" and (rip == gw or rip == guest_gw):
--- End diff --

@ustcweizhou I am not sure that changes removes the need for this, but it 
looks like it could potentially.  Maybe I can run some tests to see if it can 
be removed given #1871 has been 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.
---


[GitHub] cloudstack issue #1919: CLOUDSTACK-9763: set ssh public key when creating vm...

2017-02-14 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue:

https://github.com/apache/cloudstack/pull/1919
  
Thanks for the explanation @serbaut. That is exactly why I asked; so, it is 
not a reboot/restart per se. It is a re-deploy; the old virtual machine is 
destroyed and a new one is deployed (or maybe a reset of the VHD of the VM). By 
VM here I mean VR (which at the end of the day is a VM). 

@serbaut, could you add these explanations on the PR description and Jira 
ticket (https://issues.apache.org/jira/browse/CLOUDSTACK-9763)? At least for 
me, this was not clear.

Thanks for the fix ;)
Code LGTM


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


[GitHub] cloudstack issue #1884: CLOUDSTACK-9699: Add global setting for enable/disab...

2017-02-14 Thread rashmidixit
Github user rashmidixit commented on the issue:

https://github.com/apache/cloudstack/pull/1884
  
@rhtyd Thanks for the update. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1855: CLOUDSTACK-9697: Better error message if user tries ...

2017-02-14 Thread rashmidixit
Github user rashmidixit commented on the issue:

https://github.com/apache/cloudstack/pull/1855
  
@sadhugit I have updated the bug description based on your comments.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1829
  
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) 
has been kicked to run smoke tests


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


[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread borisstoyanov
Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1829
  
@blueorangutan test centos7 xenserver-65sp1


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1829
  
@borisstoyanov unsupported parameters provided. Supported mgmt server os 
are: `centos6, centos7, ubuntu`. Supported hypervisors are: `kvm-centos6, 
kvm-centos7, kvm-ubuntu, xenserver-65sp1, xenserver-62sp1, vmware-60u2, 
vmware-55u3, vmware-51u1, vmware-50u1`


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread borisstoyanov
Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1829
  
@blueorangutan test centos7 xenserver-65sp2


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

2017-02-14 Thread asfgit
Github user asfgit closed the pull request at:

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


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

2017-02-14 Thread karuturi
Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1741
  
It has required LGTMs and tests. I am merging this.


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


[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1829
  
Packaging result: ✔centos6 ✔centos7 ✖debian. JID-485


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

2017-02-14 Thread swill
Github user swill commented on the issue:

https://github.com/apache/cloudstack/pull/1741
  
@deepthimachiraju remember that you need a new system vm built from this PR 
for this functionality to work. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1829
  
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep 
you posted as I make progress.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-14 Thread borisstoyanov
Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1829
  
@blueorangutan package


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


Re: Handling of DB migrations on forks

2017-02-14 Thread Wei ZHOU
Then you have to create your own branch forked from 4.10.0

In our branch, I moved some table changes (eg ALTER TABLE, CREATE TABLE)
from schema-.sql
to engine/schema/src/com/cloud/upgrade/dao/UpgradeXXXtoYYY.java.
If SQLException is throwed, then show a warning message instead upgrade
interruption..
By this way, the database will not be broken in the upgrade or fresh
installation.

-Wei


2017-02-14 11:52 GMT+01:00 Jeff Hair :

> Hi all,
>
> Many people in the CS community maintain forks of CloudStack, and might
> have implemented features or bug fixes long before they get into mainline.
> I'm curious as to how people handle database migrations with their forks.
> To make a DB migration, the CS version must be updated. If a developer adds
> a migration to their fork on say, version 4.8.5. Later, they decide to
> upgrade to 4.10.0 which has their migration in the schema upgrade to
> 4.10.0.
>
> How do people handle this? As far as I know, CS will crash on the DB
> upgrade due to SQL errors. Do people just sanitize migrations when they
> pull from downstream or somehting?
>
> Jeff
>


Handling of DB migrations on forks

2017-02-14 Thread Jeff Hair
Hi all,

Many people in the CS community maintain forks of CloudStack, and might
have implemented features or bug fixes long before they get into mainline.
I'm curious as to how people handle database migrations with their forks.
To make a DB migration, the CS version must be updated. If a developer adds
a migration to their fork on say, version 4.8.5. Later, they decide to
upgrade to 4.10.0 which has their migration in the schema upgrade to 4.10.0.

How do people handle this? As far as I know, CS will crash on the DB
upgrade due to SQL errors. Do people just sanitize migrations when they
pull from downstream or somehting?

Jeff


[GitHub] cloudstack issue #1940: CLOUDSTACK-9781:ACS records ID in events tables inst...

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1940
  
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests


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


[GitHub] cloudstack issue #1940: CLOUDSTACK-9781:ACS records ID in events tables inst...

2017-02-14 Thread borisstoyanov
Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1940
  
@blueorangutan test


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


[GitHub] cloudstack issue #1940: CLOUDSTACK-9781:ACS records ID in events tables inst...

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1940
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-484


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1919: CLOUDSTACK-9763: set ssh public key when creating vm...

2017-02-14 Thread ustcweizhou
Github user ustcweizhou commented on the issue:

https://github.com/apache/cloudstack/pull/1919
  
I agree with @serbaut , different from password which is applied to VR only 
once, the public keys should be set in VR each time when we recreate a VR.

this LGTM+1


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


[GitHub] cloudstack issue #1940: CLOUDSTACK-9781:ACS records ID in events tables inst...

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1940
  
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep 
you posted as I make progress.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1940: CLOUDSTACK-9781:ACS records ID in events tables inst...

2017-02-14 Thread borisstoyanov
Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1940
  
@blueorangutan package


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


[GitHub] cloudstack issue #1919: CLOUDSTACK-9763: set ssh public key when creating vm...

2017-02-14 Thread serbaut
Github user serbaut commented on the issue:

https://github.com/apache/cloudstack/pull/1919
  
The VPC VR maintains metadata 
(http://docs.cloudstack.apache.org/projects/cloudstack-administration/en/4.8/virtual_machines/user-data.html)
 as static files in /var/www/html/metadata. When a VR is destroyed and 
recreated (by e.g. "restart with cleanup") this metadata is rebuilt by 
createVmDataCommandForVMs(). public-keys is missing from that function so it 
becomes empty after the rebuild and a request for latest/meta-data/public-keys 
no longer returns the correct key.

This PR adds public-key to the rebuild.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 #1940: CLOUDSTACK-9781:ACS records ID in events tabl...

2017-02-14 Thread jayantpatil1234
GitHub user jayantpatil1234 opened a pull request:

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

CLOUDSTACK-9781:ACS records ID in events tables instead of UUID.

ISSUE
=
Wrong presentation of volume id in ASC events.
While creating a snapshot, only volume ID is mentioned in the events. For 
example, “Scheduled async job for creating snapshot for volume Id:270". On 
looking into the notification, user is not able to identify the volume. So 
modified event description with UUID.

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

$ git pull https://github.com/Accelerite/cloudstack CS-48313

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

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


commit 7f216e8960e466acbd610fd90d44583bb8b6c15e
Author: Jayant Patil 
Date:   2017-02-14T08:51:30Z

CLOUDSTACK-9781:ACS records ID in events tables instead of UUID.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1818: CLOUDSTACK-9655 The template which is registered in ...

2017-02-14 Thread ustcweizhou
Github user ustcweizhou commented on the issue:

https://github.com/apache/cloudstack/pull/1818
  
@karuturi in my understanding, the zoneid is not necessary if template is 
not cross-zone.
If template is cross-zone, the zoneid should be passed if template is 
deleted from the 'Zones' tab (it means template will be deleted from the zone), 
Otherwise, the template will be deleted from all zones (this is why I suggest 
to add the new button in template details page, then the zoneid will not be set)
Please correct me if I am wrong.



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


[GitHub] cloudstack issue #1881: CLOUDSTACK-9721: Remove deprecated/unused global con...

2017-02-14 Thread karuturi
Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1881
  
configuration cleanup. needs BVT


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1882: CLOUDSTACK-8737: Removed the missed out-of-band VR r...

2017-02-14 Thread karuturi
Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1882
  
unused class is removed. needs BVT


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

2017-02-14 Thread asfgit
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1818: CLOUDSTACK-9655 The template which is registered in ...

2017-02-14 Thread karuturi
Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1818
  
@ustcweizhou in the above snippet, its adding the zoneid if its not 
cross-zone. if no zoneid is provided, it will default to -1 which means cross 
zone. So, this check is required. 
This fix is just enhancing the message if its a cross-zone template. Its 
not trying to do the delete only in that zone. That is not possible.


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

2017-02-14 Thread jayapalu
Github user jayapalu commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1885#discussion_r100985099
  
--- Diff: server/src/com/cloud/network/IpAddressManagerImpl.java ---
@@ -460,6 +460,12 @@ boolean checkIfIpAssocRequired(Network network, 
boolean postApplyRules, List 0) {
+if (network.getVpcId() != null) {
--- End diff --

Improved. Added more details


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

2017-02-14 Thread asfgit
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 #1244: Bug-ID: CLOUDSTACK-8654: Added guest_os mappi...

2017-02-14 Thread asfgit
Github user asfgit closed the pull request at:

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


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

2017-02-14 Thread kishankavala
Github user kishankavala commented on the issue:

https://github.com/apache/cloudstack/pull/1939
  
LGTM

API response:


admin
8c91c04e-f282-11e6-8a09-d4ae52cb9a54
8c91ab6a-f282-11e6-8a09-d4ae52cb9a54
ROOT
a6401f5b-b090-4a64-9d73-c04369d15ca8

VM-5e458101-b6a7-477e-9086-ee659ce0a700 running time (ServiceOffering: 1) 
(Template: 111)

0.12778 Hrs
1
0.12778
5e458101-b6a7-477e-9086-ee659ce0a700
VM-5e458101-b6a7-477e-9086-ee659ce0a700
2b10fa98-c167-444f-b9c0-752611a9f267
02567d08-f283-11e6-8a09-d4ae52cb9a54
5e458101-b6a7-477e-9086-ee659ce0a700
Simulator
2017-02-14'T'07:07:15+00:00
2017-02-14'T'07:15:00+00:00



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes 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 issue #1934: [CLOUDSTACK-9772] Template: perform a HEAD request t...

2017-02-14 Thread marcaurele
Github user marcaurele commented on the issue:

https://github.com/apache/cloudstack/pull/1934
  
@remibergsma Good point, I was aware of that difference, which I think 
doesn't help to make systems reliable.
Another improvement would be to remove this function and refactor the code 
to read sizes from the template objects in the DB or on the GET requests when 
downloading them.


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

2017-02-14 Thread borisstoyanov
Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1939
  
@blueorangutan test


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


[GitHub] cloudstack issue #1939: CLOUDSTACK-8886: Limitations is listUsageRecords out...

2017-02-14 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/1939
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-483


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