[18:57:37] volans: https://gerrit.wikimedia.org/r/#/c/operations/dns/+/547773/ in case you have an opinion on these subjects [18:59:57] I did this: https://gerrit.wikimedia.org/r/c/operations/puppet/+/547767 dunno if it's the good way of doing it, but it's there for people to review [19:00:07] for the puppet and monitoring folks ^ [19:01:18] XioNoX: add those people as reviewers ;) [19:01:40] yup doing it right now [19:02:20] bblack: I did tthat at the time only because I saw it used in the repo and I needed it to properly parse it. I'm happy we can kill it now [19:03:31] yeah () and leading whitespace continuations are "features" of the legacy rfc1035 file format, but they're the features that make it a PITA to deterministically parse or grep it, I hate them. [19:03:40] today I ran into it again and just decided we should outlaw them heh [19:04:09] +10 :) [19:05:54] nice! yes [19:06:37] I've also checked that the zone_Validator reported warnings are the same between the first patch in your series and teh last one that changes zone_validator [19:07:05] :)