Code review comment for lp:~allenap/maas/tftp-rebind-on-netif-change

Looks good although I'm not familiar enough with twisted to understand 100% of it. At always when making important changes to a core piece of the infrastructure, I recommend building a package from this branch and testing it before landing that whole lot.

[0]

188 +class TestTFTPService(MAASTestCase):
189 +
190 + def test_tftp_service(self):

This test is a bit painful to digest: it checks a ton of stuff. Could you think of a way to split it into more focused tests?