[16:00:34] marostegui, btullis, akosiaris: gentle reminder to not use cumin1001 for cumin/cookbooks but cumin1002/cumin2002 instead (see moritzm 's email from before the break) [16:00:46] k [16:01:42] Oops, thanks for the reminder. [16:06:24] <_joe_> volans: add a MOTD [16:07:36] _joe_: it's there, on the readability of it I'll let moritz speak ;) [16:09:41] <_joe_> volans: oh lol, I guess I just proved it's not very attention capturing [16:10:01] <_joe_> upon closer and careful inspection I could detect it [16:13:40] I can only cite myself :-P https://wm-bot.wmcloud.org/browser/index.php?start=12%2F19%2F2023&end=12%2F19%2F2023&display=%23wikimedia-sre-foundations [16:18:21] What's the ASCII terminal escape sequence for ? [16:22:19] $ echo -e "\033[31;1;4mHello\033[0m" [16:22:35] ^ example ANSI for making "Hello" in red,bold,underline (and then reset after) [16:22:56] cribbed from random SO answer [16:29:09] https://manpages.debian.org/bookworm/manpages/console_codes.4.en.html <-- if you want the full delights :) [16:34:30] https://gerrit.wikimedia.org/r/c/operations/puppet/+/987156 ? [16:34:48] something like that anyways [16:35:12] you could probably get fancier and probe for whether the terminal supports color escapes [16:36:53] it doesn't like the quoting, though [16:37:47] Do not use this server! [16:38:22] I think maybe PS2 would work [16:54:22] PS3 has it right now. I ran it through the compiler and then pasted it over the shellscript on cumin1001 and logged in to check [16:54:43] (turns out -e is for /bin/echo, built-in echo doesn't want it and still processes the escapes) [17:08:07] I think this is one of the reasons printf is sometimes preferred to echo :) [17:09:16] yeah one of our other examples in the repo uses printf [17:09:21] either way, same result in the end [17:09:43] modules/puppetmaster/files/motd/99-obsolete [17:11:41] which itself is kind of an ironic file [17:12:30] "99-obsolete" is itself obsolete. I believe the last reference that made use of it died in a commit from Ariel in 2014 ( 8ea4674c2a4ad0c6dd15951db6594d9e777b62ca ), when we retired the old puppetmaster from the now-long-dead Tampa datacenter :) [17:13:45] someone should write a lint tool that detected puppet files, templates, and/or manifests that are effectively dead code that isn't referenced from anywhere. [17:14:57] (while you're at it, solve the halting problem too of course) [17:17:42] :) [17:22:25] tbh if the halting problem isn't trivial on any of our Puppet, we should refactor it until it is [17:23:45] I had suggested to just use the motd of the inactive deploy host :-P [19:37:11] rats I had hoped the patch above was already deployed, hopped onto cumin1001 and ... boring :-P [20:03:49] apergos: it's live now :) [20:09:25] * apergos dashes over to check it [20:09:44] wow, definitely eye-catching! [20:11:51] the deployment hosts manage with a humongous banner which I'm sure you remember... [22:05:11] bblack: the typo was an intended pun? :-P [22:44:01] volans: ? [22:44:21] oh I see it now, cookcooks. [22:44:28] I didn't write that, I just injected the ANSI bits :) [22:45:05] :D