[03:56:50] <wikibugs>	 10Traffic, 10Gerrit, 10Operations, 10Patch-For-Review: Enable avatars in gerrit - https://phabricator.wikimedia.org/T191183 (10Krinkle) The prior conversation at T179212 may be relevant here. The two options discussed so far were 1) Use Gerrit repo with static file server, or 2) Use Phabricator.  ##### Ger...
[12:40:33] <wikibugs>	 10Traffic, 10Beta-Cluster-Infrastructure, 10DNS, 10Operations, and 3 others: Ferm's upstream Net::DNS Perl library bad handling of NOERROR responses without records causing puppet errors when we try to @resolve AAAA in labs - https://phabricator.wikimedia.org/T153468 (10Krenair) Ferm PR: https://github.com...
[12:50:03] <wikibugs>	 10Traffic, 10Gerrit, 10Operations, 10Patch-For-Review: Enable avatars in gerrit - https://phabricator.wikimedia.org/T191183 (10Dzahn) I don't think that cloning / committing / pushing is a big hurdle in this case. After all we are talking about Gerrit users not Phabricator. People who already explicitly co...
[13:19:51] <paravoid>	 Krenair: if you're looking into ferm changes, one of the long-standing ones that I had on my TODO has been to fix @resolve to work automagically cross-domain
[13:20:04] <paravoid>	 Krenair: right now @resolve always does A queries IIRC, even under "domain ip6"
[13:20:14] <paravoid>	 Krenair: so that's why we're passing it 'AAAA' in some cases, I believe
[13:20:27] <paravoid>	 but it's been 3 years since I last looked at all that, don't take my word for it
[13:21:01] <paravoid>	 so it really should do A for domain ip and AAAA for domain ip6, and if the rrset is empty, just treat it as an empty list
[13:30:47] <wikibugs>	 10Traffic, 10Operations, 10ops-eqiad: cp1076 hardware failure - https://phabricator.wikimedia.org/T206394 (10BBlack) p:05Triage>03Normal
[13:43:26] <wikibugs>	 10Traffic, 10Operations, 10ops-eqiad: cp1076 hardware failure - https://phabricator.wikimedia.org/T206394 (10BBlack)
[13:43:40] <wikibugs>	 10Traffic, 10Operations, 10ops-eqiad: cp1076 hardware failure - https://phabricator.wikimedia.org/T206394 (10BBlack) Note to future self on a weekday: we should probably dig further via the nvme-cli commands, as there's lots of queryable hardware errorlog/state/status that might give more insight.
[13:43:43] <bblack>	 and in case it's lost in later spam: cp1076 had some kind of nvme-disk/pci issue.  It's back to normal live state now, but I've left it depooled in case things go south with it again.  Details in: https://phabricator.wikimedia.org/T206394 
[14:21:05] <Krenair>	 paravoid, so I feel like maybe that should be a separate @command thing
[14:21:07] <Krenair>	 though I'm not a ferm developer
[14:21:09] <Krenair>	 the PR I've submitted today should handle the "if the rrset is empty, just treat it as an empty list"
[14:57:11] <Krenair>	 anyway
[14:57:16] <Krenair>	 part of the problem is the type param passed to @resolve is not just A/AAAA
[14:57:23] <Krenair>	 it can be NS, MX, and I imagine other stuff like SRV
[14:57:58] <Krenair>	 https://github.com/MaxKellermann/ferm/blob/master/src/ferm#L1172-L1185
[17:45:54] <wikibugs>	 10Traffic, 10Operations: Update certspotter - https://phabricator.wikimedia.org/T204993 (10Krenair) So now we just pin the certspotter package to `release a=stretch-backports`?
[17:47:25] <wikibugs>	 10Traffic, 10Operations: Update certspotter - https://phabricator.wikimedia.org/T204993 (10Krenair) Actually it looks like it wasn't in stretch to stretch-backports has highest priority anyway. So the host just needs package updates..?
[18:08:48] <paravoid>	 Krenair: yeah, it's just that it defaults to 'A'
[18:10:40] <paravoid>	 lin 1314: my $type = $params[1] || 'A';
[18:10:49] <paravoid>	 so that should be conditional to the domain
[18:56:33] <Krenair>	 paravoid, conditional to the domain?