Page MenuHomePhabricator

Beta puppet patch "prometheus: make ferm DNS record type configurable"
Closed, ResolvedPublic



This patch has been cherry-picked on Beta for many months now:

If it is still needed, please re-open the change set and consider merging it or asking for help with finishing and/or reviewing it.

If it is no longer needed, consider removing it from Beta's puppetmaster and removing the hashtag.


Prometheus exports have a ferm rule that resolves hosts using AAAA DNS
records. Labs lacks IPv6 and the class is installed on the
deploymemt-memc* hosts among others which causes ferm to fail loading.

Make it possible to change the record type used by ferm when resolving.
Default remains 'AAAA'.

Set it to 'A' for labs via hiera:

profile::prometheus::dns_record_type: 'A'

Bug: T153468
Bug: T176314

Event Timeline

jijiki triaged this task as Medium priority.Feb 10 2020, 10:39 AM

ferm has been fixed in stretch-wikimedia and buster-wikimedia to properly resolve AAAA records with a fallback, if all jessie instances are gone from deployment-prep, this patch be be removed (if all stretch/buster hosts are running ferm 2.4-1+wmf2+deb10u1 or 2.4-1+wmf2+deb9u1)

We still have some Jessie instances which are in need of migration. I think
production still has some jessie instances.

This local patch no longer merges, due to cleanups of the use of hiera and type hints. I dropped it when rebasing to un-break beta puppet.

Feel free to rewrite and re-cherry-pick if still needed.

dpifke claimed this task.

Calling this resolved, as it's been a week and nobody has missed the associated patch.