> -----Original Message----- > From: Daan Hoogland [mailto:[email protected]] > Sent: Thursday, February 06, 2014 9:56 PM > To: Alena Prokharchyk > Cc: [email protected]; [email protected] > Subject: Re: commit 6523c068695d0431070060667c222eb40d54b14d breaks > network removal > > second thought, > > Soheils mail bounces and the commit does not refer a ticket from jira. > I am going to revert. I should have been more vigilant. sorry. > [Animesh] Daan was this change introduced because of being flagged during findbug run?
> On Fri, Feb 7, 2014 at 6:49 AM, Daan Hoogland <[email protected]> > wrote: > > will do Alena, > > > > thanks for the headsup > > > > On Thu, Feb 6, 2014 at 10:42 PM, Alena Prokharchyk > > <[email protected]> wrote: > >> Soheil/Daan, > >> > >> The commit in the subject breaks network System vms destroy (VR, > >> SSVM, CPVM), resulting in the network removal failures. Following > >> line replacement causes the failure: > >> > >> - if (vm.getType() == Type.User && > >> isDhcpAccrossMultipleSubnetsSupported(network) && > >> isLastNicInSubnet(nic) && > >> network.getTrafficType() == TrafficType.Guest > >> > >> With > >> > >> + DhcpServiceProvider dhcpServiceProvider = > >> getDhcpServiceProvider(network); > >> > >> > >> When you try to call getDhcpServiceProvider(network), it throws an > >> exception because DHCP service is not enabled in Public/Control > >> networks of system vms nics. So system vm always fails to expunge. > >> > >> Could you please fix it by checking if DHCP service is enabled on the > >> network, before getting the DHCP service provider? > >> > >> Thanks, > >> Alena. > >> > >> > >> > > > > > > > > -- > > Daan > > > > -- > Daan
