Marcus, I will but Joris is considering John Nielsen's
comment. It is on 4.4-forward and will be in by the end of the week I
think. 4.3 is open territory I think, isn't it?

regards,
Daan


On Mon, May 19, 2014 at 7:45 PM, Marcus Sorensen <shadow...@gmail.com>wrote:

>    This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/21431/
>
> On May 14th, 2014, 2:12 p.m. UTC, *daan Hoogland* wrote:
>
> 2ebeec43340f33e2a00553b171452c6516ccc946 on master
>
>  We should consider cherry-picking this for 4.3/4.4 as well
>
>
> - Marcus
>
> On May 14th, 2014, 1:58 p.m. UTC, Joris van Lieshout wrote:
>   Review request for cloudstack, daan Hoogland, Hugo Trippaers, and Sheng
> Yang.
> By Joris van Lieshout.
>
> *Updated May 14, 2014, 1:58 p.m.*
>  *Bugs: * 
> cloudstack-6665<https://issues.apache.org/jira/browse/cloudstack-6665>
>  *Repository: * cloudstack-git
> Description
>
> The releasing of dhcp leases on vpc routers fails because the network on 
> which the instance that holds the lease lives is not behind eth0, like with 
> normal router vms. The current code assumes eth0, this patch will make 
> interface selection dynamic based on the ipv4 address of the instance.
>
>   Testing
>
> Tested on vpc routers in our own environment. Manual workaround for vpc 
> experiencing this issue was to execute the command in this patch by hand.
>
>   Diffs
>
>    - systemvm/patches/debian/config/opt/cloud/bin/edithosts.sh (70c95a8)
>
> View Diff <https://reviews.apache.org/r/21431/diff/>
>



-- 
Daan

Reply via email to