Code review comment for lp:~jameinel/maas/1.2-node-get-effective-kernel-options

Revision history for this message
Martin Packman (gz) wrote :

Looks good. Worth noting, is that if the first tuple item is not None, the second must be a string, as we should only match tags that have kernel parameters.

review: Approve

« Back to merge proposal