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.
On Fri, Feb 7, 2014 at 6:49 AM, Daan Hoogland <daan.hoogl...@gmail.com> wrote: > will do Alena, > > thanks for the headsup > > On Thu, Feb 6, 2014 at 10:42 PM, Alena Prokharchyk > <alena.prokharc...@citrix.com> 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