Hugo, I just changed all the *NetworkGuru :: implement() to first try to retrieve the physicalNetworkId from the NetworkVO. If it is null, then they can call the regular code to derive it using _networkModel.findPhysicalNetworkId.
May be you need to change the unit test mocking due to this? -Prachi -----Original Message----- From: Chip Childers [mailto:chip.child...@sungard.com] Sent: Wednesday, February 20, 2013 12:35 PM To: cloudstack-dev@incubator.apache.org Cc: Hugo Trippaers Subject: Re: Jenkins build became unstable: cloudstack-marvin #517 On Wed, Feb 20, 2013 at 08:24:46PM +0000, Apache Jenkins Server wrote: > See <https://builds.apache.org/job/cloudstack-marvin/517/changes> Looks like we hit some unit test failures that caused this: https://builds.apache.org/job/cloudstack-marvin/517/org.apache.cloudstack$cloud-plugin-network-nvp/testReport/ Hugo, this is in the NVP plugin. Can you take a look please? -chip