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

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

I'm going to review this in detail, but time has run out for me today; I'll have to get back to it tomorrow. Smaller branches have less of this problem!

For the record, I agree with Jason that it's better to fail early than tohide potentially problematic settings. But of course that's assuming that we can reasonably catch the problem at the right stage. Once we're trying to boot a node, it's probably too late.

As for the get_boot_purpose call where the label is None, I'd suggest passing it as a keyword for clarity: label=None.

« Back to merge proposal