[08:19:21] 10serviceops, 10Prod-Kubernetes, 10Kubernetes, 10Patch-For-Review: Update Kubernetes clusters to v1.23 - https://phabricator.wikimedia.org/T307943 (10JMeybohm) a:03JMeybohm [08:38:23] 10serviceops, 10Machine-Learning-Team, 10Patch-For-Review: Fix calico, cfssl-issuer and knative-serving Helm dependencies - https://phabricator.wikimedia.org/T303279 (10JMeybohm) Fixed for calico with v3.23.3 [08:43:04] 10serviceops, 10Prod-Kubernetes, 10Kubernetes, 10Patch-For-Review: Update Kubernetes clusters to v1.23 - https://phabricator.wikimedia.org/T307943 (10JMeybohm) [09:26:20] I'd like to merge a small change for docker registry fixing the jwt config for Trusted Runners. I'll make sure puppet and nginx are happy after merging. This is mostly fyi, but feel free to ping/stop me :) [09:26:20] I'm talking about https://gerrit.wikimedia.org/r/836135 and https://gerrit.wikimedia.org/r/836139 [09:35:31] 10serviceops, 10Data-Engineering, 10SRE, 10Event-Platform Value Stream (Sprint 02), 10Patch-For-Review: eventstreams chart should use latest common_templates - https://phabricator.wikimedia.org/T310721 (10JMeybohm) This needs SRE support to depool eventstreams from one DC. helmfile destroy/helmfile appy... [09:51:25] <_joe_> jelto: let me take a look [09:53:33] thanks! [11:02:43] 10serviceops, 10Dumps-Generation, 10Patch-For-Review, 10Performance-Team (Radar): Remove php 7.2 from production - https://phabricator.wikimedia.org/T318894 (10Joe) 05Open→03In progress [11:02:49] 10serviceops, 10Dumps-Generation, 10Patch-For-Review, 10Performance-Team (Radar): Migrate WMF production from PHP 7.2 to PHP 7.4 - https://phabricator.wikimedia.org/T271736 (10Joe) [11:04:31] 10serviceops, 10Release-Engineering-Team, 10docker-pkg, 10Patch-For-Review: docker-pkg / docker downloads all versions of parent image upon building - https://phabricator.wikimedia.org/T310458 (10Clement_Goubert) p:05Triage→03Medium a:03Clement_Goubert [11:08:06] 10serviceops, 10Prod-Kubernetes, 10Kubernetes, 10Patch-For-Review: Update Kubernetes clusters to v1.23 - https://phabricator.wikimedia.org/T307943 (10JMeybohm) [12:13:10] 10serviceops, 10SRE, 10Continuous-Integration-Config, 10Release-Engineering-Team (CI & Testing services), 10Test-Coverage: Add pcov PHP extension to wikimedia apt so it can be used in Wikimedia CI - https://phabricator.wikimedia.org/T243847 (10Daimona) Now that the php72 images were dropped, I think we o... [12:53:46] 10serviceops, 10Release-Engineering-Team, 10docker-pkg: docker-pkg / docker downloads all versions of parent image upon building - https://phabricator.wikimedia.org/T310458 (10hashar) 05Open→03Resolved Should be good now. I will report back if I notice the problem hasn't been solved. [12:54:04] 10serviceops, 10Release-Engineering-Team, 10docker-pkg: docker-pkg / docker downloads all versions of parent image upon building - https://phabricator.wikimedia.org/T310458 (10Clement_Goubert) Thanks for your help solving this! [13:04:25] good morning inflatador [13:16:52] 10serviceops, 10Machine-Learning-Team, 10Observability-Metrics, 10Kubernetes: Don't scrape every containerPort for metrics - https://phabricator.wikimedia.org/T318707 (10bking) a:05bking→03None Unassigning for now, will circle back later this week or next week to discuss further. [13:16:52] greetings _joe_ ! [13:17:45] If you have any other tickets for me to look at LMK, I'm just going thru the SRE training checklist at the moment [13:26:15] 10serviceops, 10Prod-Kubernetes, 10Kubernetes, 10Patch-For-Review: Update Kubernetes clusters to v1.23 - https://phabricator.wikimedia.org/T307943 (10JMeybohm) [13:43:51] Hola! As promised, I have a CR to add a spark container image to operations/docker-images/production-images - I'd love some reviews if anyone would care to take a look: https://gerrit.wikimedia.org/r/c/operations/docker-images/production-images/+/838151 - Thanks. [13:49:27] <_joe_> btullis: very basic q: why a separate build image? [13:49:58] <_joe_> I would expect to see a multi-stage container image usually in these cases (so basically the two dockerfiles unified) [13:53:28] 10serviceops, 10Parsoid, 10Patch-For-Review, 10Performance-Team (Radar), 10Performance-Team-publish: Parsoid migration to php 7.4 - https://phabricator.wikimedia.org/T312638 (10Clement_Goubert) [13:53:32] 10serviceops, 10Parsoid (Tracking): Parsoid migration: Cleanup - https://phabricator.wikimedia.org/T318946 (10Clement_Goubert) 05Open→03Resolved `php 7.2` removed from all codfw parse servers. [14:01:42] 10serviceops, 10Prod-Kubernetes, 10Kubernetes, 10Patch-For-Review: Update Kubernetes clusters to v1.23 - https://phabricator.wikimedia.org/T307943 (10JMeybohm) [14:08:31] _joe_: Hmm. Maybe you're onto something there. There's no long-term value in a published spark/build image. It's only used as an input to the spark/spark image. [14:09:26] So I should try to be more clever about that intermediate stage and put it into an earlier stage in a single Dockerfile, right? [14:10:40] I will be going to to make a spark/operator image, which is why I put it into a spark/ subdirectory in the first place (referencing the istio example). [14:12:40] 10serviceops, 10Observability-Metrics, 10Kubernetes: Limit the envoy metrics scraped from k8s - https://phabricator.wikimedia.org/T318705 (10bking) 05Open→03Resolved This appears to be working based on [[ https://phabricator.wikimedia.org/P35177#145844 | the grafana dashboard query I ran ]] . I'm resolv... [14:30:22] btullis: but spark and spark operator are just loosely coupled, right? They probably don't share code/repos like it is with istio/cert-manager where we generate N containers out of the same codebase [14:38:45] jayme: Also a very good point that I hadn't considered. Many thanks. So perhaps it would be better to put have spark and spark-on-k8s-operator as normal entries beneath images/ ? [14:41:56] I'd say yes. But I'm really just passing by :D [14:42:25] Can I still override the seed_image if it's in /images ? I was looking to build spark from an openjdk-11-jre image [14:43:44] Apologies for all the questions :-) [14:52:57] I suppose all I do is add another FROM into the Dockerfile.template for the execution stage. There's nothing sacrosanct about use of `seed_image` in these files, is there? [15:00:04] no, I don't think there is. Apart from that it can be changed in a central place [15:54:46] quick workout, back in ~40 [16:31:59] 10serviceops, 10MediaWiki-Core-JobQueue, 10WMF-JobQueue, 10Patch-For-Review, 10Platform Team Workboards (Platform Engineering Reliability): Enable MW REST API on job runners and video scalers (for the new rest.php job executor) - https://phabricator.wikimedia.org/T246389 (10hnowlan) [20:42:47] 10serviceops, 10Performance-Team: Rewrite mw-warmup.js in Python - https://phabricator.wikimedia.org/T288867 (10Krinkle) 05Open→03Declined [21:01:54] 10serviceops, 10Performance-Team (Radar): Consider socket files for MW-to-mcrouter connection - https://phabricator.wikimedia.org/T235216 (10Krinkle)