launchpad-reviewers team mailing list archive

["Jeroen T. Vermeulen" <jtv@xxxxxxxxxxxxx>]

The proposal to merge lp:~jtv/maas/bug-1116700 into lp:maas has been updated.

Description changed to:

We're taking a bit of a risk with this one, because we've lost the reason for the original behaviour.  We do know that we had to do things like this for the original Cobbler-based MAAS, where things worked the way they did simply because that's how they worked in Cobbler, or because it was the way we ended up interfacing with Cobbler.  DHCPLease was bolted on later, so there may have been problems with incorporating it nicely.

Raphaël sees no problem with the change.  It seems more sensible than what we did before.

For more details, see:
https://code.launchpad.net/~jtv/maas/bug-1116700/+merge/146835
-- 
https://code.launchpad.net/~jtv/maas/bug-1116700/+merge/146835
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~jtv/maas/bug-1116700 into lp:maas.