Code review comment for lp:~blake-rouse/maas/add-osystem-to-node-migration

Revision history for this message
Blake Rouse (blake-rouse) wrote :

[1,2] Fixed

[3] Using 20 as that was what was chosen by the core team for the release field in BootImages, whether that is the correct thing, idk.

[4] At this current state in the branch the field is not used for testing. This merge requests uses the field and provides test for it. If you would like to review that, :)

https://code.launchpad.net/~blake-rouse/maas/add-osystem-to-node-form-api

« Back to merge proposal