[GitHub] cloudstack issue #847: Bug-ID: CLOUDSTACK-8880: calculate free memory on hos...

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

https://github.com/apache/cloudstack/pull/847
  
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 #1917: CLOUDSTACK-9756: Configure to ignore the ipassoc fai...

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

https://github.com/apache/cloudstack/pull/1917
  
@jayapalu thanks, yes it seems an intermittent issue with `ipmisim`, 
`pyghmi` library that is used by the test to simulate a live ipmi server/board. 
I'll try to fix it soon, meanwhile you can push -f or close/open the PR to 
re-kick Travis.
Exception: Job failed: {jobprocstatus : 0, created : 
u'2017-02-09T13:17:54+', cmd : 
u'org.apache.cloudstack.api.command.admin.outofbandmanagement.ChangeOutOfBandManagementPasswordCmd',
 userid : u'9f69561f-eec8-11e6-8c34-42010a0a0806', jobstatus : 2, jobid : 
u'752cba20-e8f8-4b53-9675-8aa98186ee01', jobresultcode : 530, jobresulttype : 
u'object', jobresult : {errorcode : 530, errortext : u'Failed to change 
out-of-band management password for host (40328dc0-d290-481f-a6cc-f084ecf88661) 
due to driver error: Failed to find IPMI user to change password, error: > 
Error: no response from RAKP 1 message\n> Error: no response from RAKP 1 
message\n> Error: no response from RAKP 1 message\n> Error: no response from 
RAKP 1 message\nSet Session Privilege Level to ADMINISTRATOR failed\nError: 
Unable to establish IPMI v2 / RMCP+ session\n'}, accountid : 
u'9f693066-eec8-11e6-8c34-42010a0a0806'}


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #847: Bug-ID: CLOUDSTACK-8880: calculate free memory on hos...

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

https://github.com/apache/cloudstack/pull/847
  
@rhtyd Done


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


Re: [DISCUSS][FS] Host HA for CloudStack

2017-02-19 Thread Koushik Das
Rohit,

Thanks for the effort you have put in writing the FS. I have some questions 
based on my initial reading of the FS.

1. “Host HA” – In the FS you are talking about a generic HA framework but it is 
not clear what is meaning of “host HA”. Is it something like all or some VMs 
running on a host will be started on another host(s) in case of a failure or is 
it something else? How is it different from the existing “VM HA” that is 
already there?
2. You have mentioned that “Cloudstack lacks a way to reliably fence host”. 
Cloudstack considers VM as a 1st class object and so provides fencing for VM 
instead of host. There are hypervisor specific plugins that implement mechanism 
to fence a VM. I am not sure if it makes sense to expose host fencing as end 
user doesn’t care about it. Now the VM fencing implementation can use something 
like “host fencing” internally.
3. There is an existing HA framework which provides plugins for doing 
investigation if a VM is alive or not, host is alive or not, fencing of VM in 
case it is not alive. It will be good to understand the limitations of the 
existing framework and how the new framework helps in solving these problems. 
We also need to understand if the limitation is in the framework or some 
specific plugin implementation that is causing issues. Reference to JIRA issues 
would help.
4. You have mentioned about ipmi to investigate host failure. I would like to 
understand why same can’t be used in the existing framework.

Thanks,
Koushik

On 16/02/17, 4:48 PM, "Rohit Yadav"  wrote:

All,


I would like to start discussion on a new feature - Host HA for CloudStack.

CloudStack lacks a way to reliably fence a host, the idea of the host-ha 
feature is to provide a general purpose HA framework and HA provider 
implementation specific for hypervisor that can use additional mechanism such 
as OOBM (ipmi based power management) to reliably investigate, recover and 
fence a host. This feature can handle scenarios associated with server crash 
issues and reliable fencing of hosts and HA of VM. The first version will have 
HA provider implementation for KVM (and for simulator to test the framework 
implementation, and write marvin tests that can validate the feature on Travis 
and others).


Please have a look at the FS here:

https://cwiki.apache.org/confluence/display/CLOUDSTACK/Host+HA


Looking forward to your comments and questions.


Regards.

rohit.ya...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue
  
 






DISCLAIMER
==
This e-mail may contain privileged and confidential information which is the 
property of Accelerite, a Persistent Systems business. It is intended only for 
the use of the individual or entity to which it is addressed. If you are not 
the intended recipient, you are not authorized to read, retain, copy, print, 
distribute or use this message. If you have received this communication in 
error, please notify the sender and delete all copies of this message. 
Accelerite, a Persistent Systems business does not accept any liability for 
virus infected mails.


[GitHub] cloudstack issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

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

https://github.com/apache/cloudstack/pull/1847
  
@karuturi, Done


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


[GitHub] cloudstack issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

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

https://github.com/apache/cloudstack/pull/1847
  
Ok, Doing 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 #1379: CLOUDSTACK-8324: config drive data set/get scripts f...

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

https://github.com/apache/cloudstack/pull/1379
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 362
 Hypervisor xenserver
 NetworkType Advanced
 Passed=105
 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 #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

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

https://github.com/apache/cloudstack/pull/1847
  
Hi @anshul1886, I just observed that the related PR #1735 was for 4.9. 
Since this bug also exists in 4.9, can you change the base branch of this PR to 
4.9 and rebase?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

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

https://github.com/apache/cloudstack/pull/1847
  
@karuturi Test failing is unrelated to this PR.


=== TestName: test_add_user_to_project | Status : EXCEPTION ===
=== TestName: test_add_user_to_project | Status : EXCEPTION ===

 Test failing in travis is | ContextSuite con | exceptions.Excep | 36.025   
| test_project_lim |
| text=TestResourc | tion |  | its  
|
| eLimitsProject>: |  |  |  
|
| teardown 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1917: CLOUDSTACK-9756: Configure to ignore the ipassoc fai...

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

https://github.com/apache/cloudstack/pull/1917
  
Travis CI test_outofbandmanagement  is failing which is not related to this 
PR changes.  It seems there is problem with the test case.
@rhtyd  Do you have any idea on this ?
Running tests:  smoke/test_outofbandmanagement smoke/test_over_provisioning 
smoke/test_password_server smoke/test_portable_publicip 
smoke/test_primary_storage smoke/test_privategw_acl smoke/test_public_ip_range 
smoke/test_pvlan smoke/test_regions smoke/test_reset_vm_on_reboot 
smoke/test_resource_detail smoke/test_router_dhcphosts smoke/test_router_dns 
smoke/test_routers smoke/test_routers_iptables_default_policy 
smoke/test_routers_network_ops smoke/test_scale_vm smoke/test_secondary_storage 
smoke/test_service_offerings smoke/test_snapshots smoke/test_ssvm 
smoke/test_staticroles smoke/test_templates smoke/test_usage_events 
smoke/test_vm_life_cycle smoke/test_vm_snapshots smoke/test_volumes 
smoke/test_vpc_redundant smoke/test_vpc_router_nics smoke/test_vpc_vpn 
smoke/misc/test_deploy_vm smoke/misc/test_escalations_templates 
smoke/misc/test_vm_ha smoke/misc/test_vm_sync

Traceback (most recent call last):

  File "/opt/python/2.7.12/lib/python2.7/SocketServer.py", line 596, in 
process_request_thread

self.finish_request(request, client_address)

  File "/opt/python/2.7.12/lib/python2.7/SocketServer.py", line 331, in 
finish_request

self.RequestHandlerClass(request, client_address, self)

  File "/opt/python/2.7.12/lib/python2.7/SocketServer.py", line 652, in 
__init__

self.handle()

  File 
"/home/travis/.local/lib/python2.7/site-packages/ipmisim/ipmisim.py", line 436, 
in handle

return IpmiServerContext().handle(data, address, socket)

  File 
"/home/travis/.local/lib/python2.7/site-packages/ipmisim/ipmisim.py", line 112, 
in handle

self.initiate_session(data, address, self.session)

  File 
"/home/travis/.local/lib/python2.7/site-packages/ipmisim/ipmisim.py", line 143, 
in initiate_session

myaddr, netfnlun = struct.unpack('2B', data[14:16])

error: unpack requires a string argument of length 2

Traceback (most recent call last):

  File "/opt/python/2.7.12/lib/python2.7/SocketServer.py", line 596, in 
process_request_thread

self.finish_request(request, client_address)

  File "/opt/python/2.7.12/lib/python2.7/SocketServer.py", line 331, in 
finish_request

self.RequestHandlerClass(request, client_address, self)

  File "/opt/python/2.7.12/lib/python2.7/SocketServer.py", line 652, in 
__init__

self.handle()

  File 
"/home/travis/.local/lib/python2.7/site-packages/ipmisim/ipmisim.py", line 436, 
in handle

return IpmiServerContext().handle(data, address, socket)

  File 
"/home/travis/.local/lib/python2.7/site-packages/ipmisim/ipmisim.py", line 112, 
in handle

self.initiate_session(data, address, self.session)

  File 
"/home/travis/.local/lib/python2.7/site-packages/ipmisim/ipmisim.py", line 143, 
in initiate_session

myaddr, netfnlun = struct.unpack('2B', data[14:16])

error: unpack requires a string argument of length 2


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


[GitHub] cloudstack issue #1379: CLOUDSTACK-8324: config drive data set/get scripts f...

2017-02-19 Thread bvbharatk
Github user bvbharatk commented on the issue:

https://github.com/apache/cloudstack/pull/1379
  
Code changes look ok to me 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 #1874: CLOUDSTACK-9711: Fixed error reporting while adding ...

2017-02-19 Thread bvbharatk
Github user bvbharatk commented on the issue:

https://github.com/apache/cloudstack/pull/1874
  
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 #1922: CLOUDSTACK-9757: Fixed issue in traffic from additio...

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

https://github.com/apache/cloudstack/pull/1922
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 361
 Hypervisor xenserver
 NetworkType Advanced
 Passed=105
 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 #1953: CLOUDSTACK-9794: Unable to attach more than 14 devic...

2017-02-19 Thread remibergsma
Github user remibergsma commented on the issue:

https://github.com/apache/cloudstack/pull/1953
  
Thanks for the fix! 

Although I think that after id=25 stuff breaks on KVM, as the code 
generates a deviceName that is invalid. Id 25 results in vdz, after that it 
continues with caracters like `{` etc. The default for KVM seems to be 32 (as 
defined here: 
https://github.com/apache/cloudstack/blob/87ef8137534fa798101f65c6691fcf71513ac978/setup/db/db/schema-452to460.sql#L422).

See: 
https://github.com/apache/cloudstack/blob/master/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java#L591

So we either limit it to 25, or improve the code to generate valid device 
names above id 25. What do you think?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1917: CLOUDSTACK-9756: Configure to ignore the ipassoc fai...

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

https://github.com/apache/cloudstack/pull/1917
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 360
 Hypervisor xenserver
 NetworkType Advanced
 Passed=104
 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 #1952: CLOUDSTACK-9790: fix NPE in case of Basic zon...

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

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or 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 #1952: CLOUDSTACK-9790: fix NPE in case of Basic zone.

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

https://github.com/apache/cloudstack/pull/1952
  
we have review from @ustcweizhou and @mike-tutkowski tested the fix. 
merging this now.
Thanks you all.


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

~ Rajani

http://cloudplatform.accelerite.com/

On February 17, 2017 at 5:53 PM, Sigert GOEMINNE
(sigert.goemi...@nuagenetworks.net) wrote:

Hi all,

We have a new PR that fixes this issue:
https://github.com/apache/cloudstack/pull/1952

Kind regards,

*Sigert Goeminne*
Software Development Engineer

*nuage*networks.net
Copernicuslaan 50
2018 Antwerp
Belgium

On Fri, Feb 17, 2017 at 8:55 AM, Kris Sterckx <
kris.ster...@nuagenetworks.net> wrote:

Thanks Mike

Investigating.

Kris

On 17 February 2017 at 08:18, Tutkowski, Mike

wrote:

I added a comment on the code of the relevant PR, but the PR is
in the
Merged state, so I don't know if people receive e-mail
notifications

then.

On Feb 17, 2017, at 12:16 AM, Wei ZHOU 
wrote:

Maybe we need ask Nuage why the lines are added.

2017-02-17 7:54 GMT+01:00 Tutkowski, Mike
:

Hopefully we can now find someone with experience in CloudStack's
networking code to tell us if that fix is OK or if that was just
a

Band-Aid

and we should do something more thorough to correct the issue.

On Feb 16, 2017, at 11:48 PM, Rajani Karuturi 

wrote:

Thanks for digging it Mike.

~ Rajani

http://cloudplatform.accelerite.com/

On February 17, 2017 at 12:18 AM, Tutkowski, Mike
(mike.tutkow...@netapp.com) wrote:

Hi,

I opened a blocker for 4.10 regarding this issue:

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

Thanks,
Mike

On 2/16/17, 8:35 AM, "Tutkowski, Mike"
 wrote:

I believe the issue is here:

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

In ConfigurationManagerImpl.java, it appears the lines that led
to NPEs being thrown were added on lines 2924 – 2926:

https://github.com/apache/cloudstack/pull/1579/files

On 2/16/17, 2:42 AM, "Wido den Hollander" 
wrote:

Op 16 februari 2017 om 6:21 schreef Rajani Karuturi

:

unfortunately, running one set of tests itself is very time
consuming(multiplied by hypervisors, network modes etc. its
impossible to run all)

If I remember correctly, wido tested it with PR 1700 (IPV6 in
basic networking)

I highly doubt it's PR 1700 which is causing this. The PR I have
open to fix a issue ONLY applies if IPv6 has been set.

PR 1700 only does something when you have set a IPv6 subnet in
the 'vlan' table:
https://github.com/apache/cloudstack/commit/

c0e7766713b2631a167c2ceea7d42b574a5cd1b9#diff-f90fdac6810fd89f89e827309a41d180R231

So I don't think it's caused by this since you NIC table will
not be filled with IPv6 addresses afterwards.

Wido

Can you try a commit before and a commit after that PR?

Thanks,

~ Rajani

http://cloudplatform.accelerite.com/

On February 16, 2017 at 1:02 AM, Tutkowski, Mike
(mike.tutkow...@netapp.com) wrote:

Thanks for noting that PR, Rene. I tried building a Basic Zone
making use of code from that PR, but – unfortunately – I still
get an exception (below).

On a related note, does this mean we don’t run any integration
tests that build a Basic Zone before we OK a PR to be checked

in?

ERROR [c.c.a.ApiServer] (qtp555984412-267:ctx-a904cbd0
ctx-062a000a) (logid:30f5a3ec) unhandled exception executing

api

command: [Ljava.lang.String;@4732a98c
java.lang.NullPointerException
at
com.cloud.utils.net.NetUtils.getCidrNetmask(NetUtils.java:956)
at

com.cloud.configuration.ConfigurationManagerImpl.validateIpRange(

ConfigurationManagerImpl.java:2924)

at

com.cloud.configuration.ConfigurationManagerImpl.

createVlanAndPublicIpRange(ConfigurationManagerImpl.java:2822)

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.springframework.aop.support.AopUtils.

invokeJoinpointUsingReflection

(AopUtils.java:333)

at

org.springframework.aop.framework.ReflectiveMethodInvocation.

invokeJoinpoint(ReflectiveMethodInvocation.java:190)

at

org.springframework.aop.framework.ReflectiveMethodInvocation.

proceed(

ReflectiveMethodInvocation.java:157)

at

org.apache.cloudstack.network.contrail.management.

EventUtils$EventInterceptor.invoke(EventUtils.java:107)

at

org.springframework.aop.framework.ReflectiveMethodInvocation.

proceed(

ReflectiveMethodInvocation.java:168)

at

com.cloud.event.ActionEventInterceptor.invoke(

ActionEventInterceptor.java:51)

at

org.springframework.aop.framework.ReflectiveMethodInvocation.

proceed(

ReflectiveMethodInvocation.java:168)

at

org.springframework.aop.interceptor.ExposeInvocationInterceptor.

invoke(

ExposeInvocationInterceptor.java:92)

at

org.springframework.aop.framework.ReflectiveMethodInvocation.

proceed(

ReflectiveMethodInvocation.java:179)

at

org.springframework.aop.framework.JdkDynamicAopProxy.

invoke(JdkDynamicAopProxy.java:213)

at 

Re: PRs for 4.10

2017-02-19 Thread Rajani Karuturi
noted.

~ Rajani

http://cloudplatform.accelerite.com/

On February 17, 2017 at 6:03 PM, Frank Maximus
(frank.maxi...@nuagenetworks.net) wrote:

I have a couple of bugfixes on previous version outstanding,
which I would like to have merged to 4.10,
both still requiring review:
on 4.8: PR#1912
: which
fixes password service running on internal lb vms, making it
impossible to
do loadbalancing on port 8080
on 4.9: PR#1925
: Minor
plugin fix

Kind Regards,
Frank

On Tue, Feb 14, 2017 at 10:56 PM Syed Ahmed
 wrote:

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