[10:05:52] I am trying to push a DNS change but CI is failing but I don't think it is related to my change: https://integration.wikimedia.org/ci/job/operations-dns-lint/2022/console and my change: https://gerrit.wikimedia.org/r/c/operations/dns/+/1320111 [10:05:54] Any thoughts? [10:06:32] It looks related to those eqsin addresses that do not have PTR? [10:06:42] topranks XioNoX does this ring any bell? ^ [10:07:03] maybe from netbox integration? [10:08:22] marostegui: looks to be filling on v6 entries added for new links in eqsin [10:08:48] pa. paul was working on those but seems to have omitted the required dns patch [10:08:57] give me 5 mins I can sort it out [10:10:28] topranks: thank you no rush :* [10:52:02] marostegui: there was more tidying up that I expected, but I think you should be good now [10:52:16] if you force a re-check on your patch it should clear CI [10:55:35] let me check! [10:57:20] topranks: all good now thank you! [10:57:29] np :) [14:09:01] Would anyone who knows Puppet/Cumin be available to review https://gerrit.wikimedia.org/r/c/operations/puppet/+/1319891 ? Most of my team is out. Mainly I just wanna make sure the idea of cumin targeting based on a defined type is reasonable [14:12:38] I haven't looked at it in depth, but isn't possible to have a hierarchy of cumin aliases to do the same? [14:19:30] jynus I'm not sure...I went for the same concept as the mariadb aliases (ref https://gerrit.wikimedia.org/r/plugins/gitiles/operations/puppet/+/refs/heads/production/modules/profile/manifests/mariadb/section.pp ) but I'm using a defined type instead of a profile [14:20:55] I see, but in that case that is used by the puppet code itself, my understanding is that you only wanted to "tag" them for selection [14:21:33] I don't think it is super important, but it would be nice to understand usage before some guidance [14:22:23] how often those fact change, how dynamic they are, etc. [14:23:04] For the master-eligibles, they don't change much. Only when we are replacing a host [14:23:17] the idea being that puppet should only be used for static configuration [14:23:21] then that seems ok [14:23:52] a few times a year at most [14:24:23] maybe work with the person on clinic duty and they can delegate the decision for other SREs if needed [14:24:55] NP, thanks for taking a look