[06:51:01] greetings [08:06:35] morning! [08:14:18] the alerts about D processes on a non-nfs worker seem to be caused by one tool that's causing extra load [08:15:48] ack, so actually legitimate [08:19:18] yep [08:23:18] I think I'll skip non-nfs nodes from that alert, as it's meant to catch nfs-based issues [08:23:39] I'll send a patch for discussion [08:27:45] there you go :) https://gitlab.wikimedia.org/repos/cloud/toolforge/alerts/-/merge_requests/37 [08:29:07] LGTM! [08:31:05] thanks :) [09:47:57] what's the policy for hosts unreachable via cloudcumin's root key? i.e. https://phabricator.wikimedia.org/P81423 cfr https://phabricator.wikimedia.org/T401880#11093384 [09:53:58] a bunch of those are trove nodes (so that's ok, they use a different image), and most of the rest are magnum nodes (also ok, the ones with `-master-` and `-node-`), the rest if they are from our project, we should look into, otherwise users can change things at their own risk (like using puppet or not), unless there's reason to think that something is wrong (ex. bug, abuse or similar) I have not been enforcing it at least [09:55:16] though it's weird that VMs don't allow it, as the images we provide have it by default, so might be worth taking a look [09:55:45] dcaro: ack, thank you for the explanation [09:55:59] yes some VMs there I do recognize, e.g. the ones in o11y project which I'll take a look [10:16:53] * godog lunch [15:02:45] this should be a quick review for jobs-api https://gitlab.wikimedia.org/repos/cloud/toolforge/jobs-api/-/merge_requests/198 [15:03:09] (removing old __future__ imports and old syntax) [15:08:11] neat, +1 [16:12:31] this is ready for review, it's not big, but requires some context https://gitlab.wikimedia.org/repos/cloud/toolforge/jobs-api/-/merge_requests/196 [16:24:15] Raymond_Ndibe: reading the list of things to do from T350687, you removed any mention of tools-harbor-3/trixie, what is your plan? [16:24:16] T350687: [harbor] Move harbor data to object storage service - https://phabricator.wikimedia.org/T350687 [16:44:35] this one is an easy review that fixes a production bug https://gitlab.wikimedia.org/repos/cloud/toolforge/jobs-api/-/merge_requests/199 [16:54:56] another very easy review https://gitlab.wikimedia.org/repos/cloud/toolforge/jobs-api/-/merge_requests/200 fixing another bug [16:57:06] dhinus: thanks! I'll run a full set of tests on them and merge+deploy (probably tomorrow) [17:00:49] dcaro: I'm also reviewing !196 I hope to complete it today [17:01:36] awesome! :), let me know if you have any questions or anything (or in the review itself) [17:56:26] * dcaro off [17:56:28] cya tomorrow!