launchpad-reviewers team mailing list archive

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

The proposal to merge lp:~jtv/maas/nodegroup-update-lease-api into lp:maas has been updated.

Description changed to:

This was discussed for pre-imp with Raphaël.  It builds on a previous, still unreviewed branch that created a very minimal NodeGroups API.  The new POST method I introduce here allows a node group worker to register its current DHCP leases with the server.

For some reason I can't get the changes from the predecessor branch to fall out of this diff.  It may be more convenient for the same person to review both.

There remains work to be done: we need proper worker authentication so that regular users can't call this API method (it does require /some/ authentication of course) and we still need to pass the news on to the master worker so that it can update the corresponding DNS zone file.

Jeroen

For more details, see:
https://code.launchpad.net/~jtv/maas/nodegroup-update-lease-api/+merge/114565
-- 
https://code.launchpad.net/~jtv/maas/nodegroup-update-lease-api/+merge/114565
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~jtv/maas/nodegroup-update-lease-api into lp:maas.