Code review comment for lp:~jtv/maas/dns-disable-ipv4

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

Looks good, but you need more tests for the SQL changes, particularly all the scenarios around mixing v4/v6 and same/different MACs. I trust you will get those right since you're probably the best test writer on the team, which pains me to write this review :)

review: Approve

« Back to merge proposal