Code review comment for lp:~jtv/maas/bug-1042877

Revision history for this message
Gavin Panella (allenap) wrote :

Bootiful, as they say in Norfolk (means: jolly spiffing).

[1]

-  # root: /var/lib/tftpboot
+  root: /var/lib/maas/tftp

The approach I've been taking with this file is to include the default
value, commented out. This is the expectation that Andres has now. So,
the value change here is fine, but it should still be commented. I see
that you've updated the ConfigTFTP.root default.

[2]

In tftp.py:

-        ^/?
...
+        ^/*

I think this should stay as zero-or-one rather than zero-or-more. The
latter speaks of slapdashery. The zero-or-one is there to accomodate
ambiguity in the TFTP spec, iirc.

review: Approve

« Back to merge proposal