GitHub user bvbharatk opened a pull request:

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

    CLOUDSTACK-89027 Restart Network fails in EIP/ELB zone

    The restart network was failing when using external loadbalencer. The 
failure was because of a number format exception. When 
BroadcastDomainType.getValue(guestConfig.getBroadcastUri() is executed this 
returns a string untagged. We were trying to parse this as long so there was a 
number pointer exception.
    
    This happens only when the vlan uri is vlan://untagged. in other cases were 
there is a number instead of untagged (vlan tag) this used to succeed. Although 
we were trying to convert the number to long we were not really using it. we 
were converting the number to long and then back to string when creating the 
IpAddressTo. so I removed this unnecessary conversion in this case for fixing 
the issue at hand.
    
    
    I did a manual restart of the network and checked for this number format 
exception in a EIP/ELB setup.
    
     


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

    $ git pull https://github.com/bvbharatk/cloudstack CLOUDSTACK-8902

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

    https://github.com/apache/cloudstack/pull/898.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 #898
    
----
commit 3c61746dcec5025d10fda0f85c9157f69f58b15d
Author: Bharat Kumar <bharat.ku...@citrix.com>
Date:   2015-09-23T08:35:38Z

    CLOUDSTACK-89027 Restart Network fails in EIP/ELB zone

----


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

Reply via email to