[07:18:56] morning [07:22:19] hello from the hackathon!! [07:26:39] :-) [07:26:49] I hope you have a good time, and decent food [09:13:38] dhinus: thanks for the feedback on https://gitlab.wikimedia.org/repos/cloud/toolforge/maintain-kubeusers/-/merge_requests/19 [09:29:51] arturo: yw. what was the error from shellcheck? [09:30:20] regarding shell expansion [09:30:24] you can see the details here [09:30:24] https://gitlab.wikimedia.org/repos/cloud/toolforge/maintain-kubeusers/-/jobs/250658 [09:31:19] so if you give it a final try, maybe we can get the patch merged now [09:36:03] the shellcheck seems to suggest that escaping the $ with \$ might fix it [09:36:08] *shellcheck docs [09:37:11] and there's an even easier way: https://github.com/lima-vm/lima/issues/251#issuecomment-2091428507 [09:55:49] ok, going with that instead [09:57:20] LGTM, if the pipeline is green I think you can merge! [09:57:54] thanks! [10:20:07] taavi: I put a +1 on a couple of new Cloud VPS project request tickets [10:20:26] bd808: ack, looking at those now [10:20:45] you're awesome :) [10:21:23] bd808: hello from a conveniently close timezone [10:22:11] hola arturo :) I'm very jet lagged, so not sure what timezone I may be in. [10:22:58] I can imagine [10:25:20] do we have a ticket for the failure in the cookbook <-> WMCS sal integration? [11:20:14] arturo: which failure? [11:22:34] the "unknown project" SAL errors I can see in -feed? [11:25:41] I think that happens when you add users to a freshly-created project. I cannot find any phab task about it, maybe we should create one [11:33:46] I think you are areight [11:33:51] are right* [11:55:08] Stashbot caches the list of project from LDAP for like 5 minutes or something which is probably the trigger [12:02:29] yeah, 5 minute cache is in -- https://gerrit.wikimedia.org/r/plugins/gitiles/labs/tools/stashbot/+/refs/heads/master/stashbot/sal.py#258 [12:30:34] andrewbogott: when you are awake, could please verify if this edit is true or fake news https://wikitech.wikimedia.org/w/index.php?title=Portal%3ACloud_VPS%2FAdmin%2FTrove&diff=2174850&oldid=2158028 [13:42:34] arturo: that seems right, was probably just a c/p error [14:09:28] thanks! [14:09:52] it was in the context of https://phabricator.wikimedia.org/T363901 [14:10:11] I figured we'd want to also add a RAM quota [14:10:45] yeah, seems right [14:39:09] slyngs: cloudidm2001 is still a work in progress, right? We have some notices popping up about puppet thrashing there, I'm just making sure I can ignore them :)