Code review comment for lp:~racb/maas/fix-ipmi-enlistment

Revision history for this message
Julian Edwards (julian-edwards) wrote :

Is there any chance you can make it more obvious that the second invocation is conditional on the first? It's pretty easy to misread given that the first line is very long and ends with the critical "||\" characters. Perhaps indent the second maas-enlist line?

Looks ok otherwise.

review: Approve

« Back to merge proposal