Alright, gotcha. Then just using the Debian-packaged version nicely saves us adding a repo.
Ah, ye, makes sense! Would just assign an IP then as that doesn't hurt and Lilly can choose to use it or not.
Ah, I see, yeah, okay. We would have the option now, as we change things anyway, to clean up that configuration, but I'm also fine with keeping it as is for legacy reasons.
Oh, that totally makes sense. I'm fine with the name, if others really like it, but personally I would prefer something like z9-router indeed.
I would rather just stick to only having tags at the playbook level. I don't think this kind granular control for the role is really needed as it the config files also shouldn't trigger a reload,…
Just a nit-pick, but having the indentation be consistent across the dhcp4 and dhcp6 config would be nice.
Core configuration looks good, tho I left some comments. Left some more comments on the roles as well.
This seems to be using tabs for some reason, probably best to replace with spaces.
It's fine, just feel like rt1 is one of those obscure names again, which gives more trouble than benefit.