[00:09] RAOF: did that change help mir? [00:34] mwhudson: Still finishing it to test! [00:34] Turns out there's quite a lot of places where we assumed the 3-parameter open(), and it's annoying to forward varargs calls. [01:44] mwhudson: For more in the β€œhow did that ever work?” department - we were only ever implementing the 3-parameter open() call, but virtually all of the calls we were trying to intercept were of the 2-parameter open(). [02:46] Seriously, gcc? β€˜β€™ may be used unitialised? [02:46] On ppc64el, but not x86-64? [02:46] C was a mistake πŸ”₯ [04:04] RAOF: yay C [04:04] Actually, this is on C++. [04:04] "it's annoying to forward varargs calls" i think it's impossible generically but i guess in this case it's not very generic [04:05] Yeah, impossible in the general case and merely annoying in this case. :) [04:05] Apparently gcc (just on ppc64el!) thinks the default constructor for a std::optional accesses an anonymous uninitialised variable. [04:05] that's... special [04:05] and smells like a gcc bug tbh [04:06] I think it must be. [04:06] Oh! It's going to be an interaction between the optimiser and the C++ frontend, isn't it. That's how it can be ppc64 specific. [04:12] * mwhudson types things into gcc bugzilla [04:13] Helpfully, the diagnostic identifies it in entirely the wrong function, too. [04:13] RAOF: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80635 [04:13] gcc.gnu.org bug 80635 in gcc "[9/10 regression] std::optional and bogus -Wmaybe-uninitialized warning" [Normal, Assigned] [04:14] although that's old [04:15] looks similar though [04:18] It does, yes. [04:19] It also suggests that we might have hit the problem on a !ppc64 architecture, essentially at random? [05:09] πŸŽ‰ That fixes the stack smashing! [05:10] Leaving only a bunch of failing tests 😬 [06:24] o/ [06:25] We bootstrap a bunch of containers every night and this was working well until this morning, when I woke up to: [06:25] Setting up ubuntu-advantage-tools (27.2.1~18.04.1) ... [06:25] ERROR: File not found '/run/cloud-init/instance-data.json'. Provide a path to instance data json file using --instance-data [06:25] dpkg: error processing package ubuntu-advantage-tools (--configure): [06:25] installed ubuntu-advantage-tools package post-installation script subprocess returned error exit status 1 [06:27] Which apparently gets pulled in by ubuntu-minimal. A bit annoying. [06:40] * RAOF checks. [06:40] Hm. Is that a recent SRU? [06:40] Yup! Looks like that was published 14 hours ago. [06:40] blackboxsw: Around? It looks like your recent ubuntu-advantage-tools upload introduced a regression ^^^ [06:42] tobikoch: Have you filed a bug about this yet? [06:43] No yet, but can do. [06:44] It's easier to link to :) [06:44] k [06:50] RAOF: https://bugs.launchpad.net/ubuntu-advantage/+bug/1938128 [06:50] rcj: this might bite you, as well^ [07:04] tobikoch: Thanks. Did you deliberately mark that bug as proprietary? It means that I can't re-assign it to the correct place :) [07:04] If it is meant to be proprietary I can work around that. [07:04] I need to head off now, so I hand this over to… [07:04] !regression-release [07:04] Oh, no ubotu in here? Have I forgotten that? [07:04] RAOF: I thought I had changed it, but maybe it is private because UA is proprietary? [07:05] Maybe. I can just copy the information out into the non-proprietary place and subscribe you to the new bug if that's ok. [07:06] Sure, sorry for the hassle :| [07:07] No problem! [07:08] rbasak, you should be on soon. If you're back before I am back from collecting my daughter, please see above! [09:48] vorlon: Thanks for fixing gtk4 on i386! [10:15] huh i fixed that bug in git [10:15] the ubuntu-advantage-tools one [10:16] https://bugs.launchpad.net/ubuntu/+source/ubuntu-advantage-tools/+bug/1936833 [10:16] Launchpad bug 1936833 in ubuntu-advantage-tools (Ubuntu) "package ubuntu-advantage-tools 27.2.1~21.10.1 failed to install/upgrade: installed ubuntu-advantage-tools package post-installation script subprocess returned error exit status 1" [Medium, Fix Committed] [10:20] Thanks [10:20] Which series are affected? [10:26] sil2100: could you halt phasing of ubuntu-advantage-tools please? Bug 1936833 and the discussion above. [10:26] Bug 1936833 in ubuntu-advantage-tools (Ubuntu) "postinst fails when cloud-init is installed but never ran" [Medium, Fix Committed] https://launchpad.net/bugs/1936833 [10:26] What version of apt started doing phasing again? [10:26] But still, desktops may be affected too. [10:42] rbasak: trying o/ [10:48] rbasak: all of them, i think [10:48] i thought the SRU just landed but it looks like it was a week ago? [10:49] ah no was about 16 hours ago [10:50] Ok, overrode the phasing, but that's only for update manager [10:50] apt started phasing in hirsute but image builds generally ignore that anyway [10:52] any reason not to roll it back? [10:58] I had thought it had been released for a while, but perhaps not. [10:59] Ah that's what Michael said :) [11:03] the date on https://launchpad.net/ubuntu/+source/ubuntu-advantage-tools is confusing [11:03] oh is it showing the date it was published in proposed? [11:03] I don't have much context re: this upload, but we can indeed pull it from -updates if needed [11:04] I would [11:06] sil2100: OK. Please pull it from -updates. [11:10] sergiodj: ^ [11:23] ACK [11:23] Will take a moment [11:24] Since I need to remove the current package and reinstate the previous version in -updates [11:24] rbasak: can you mention on the regression bug that we're pulling this update back to -proposed? [11:25] you'll want to un-fixed-released the original SRU bugs too I guess [11:26] fix-released* [11:43] sil2100: done [11:50] un-fixed-released the original SRU bugs too> also done [11:55] otp now for the next hour [13:08] If anyone is interested in reviewing some update-notifier, I'm working on an interesting issue where a crash happens, you get the dialog to report or not, click the report button, and nothing happens. https://code.launchpad.net/~dbungert/update-notifier/+git/update-notifier/+merge/405628 === genii-core is now known as genii === JanC_ is now known as JanC === TheRedQueen is now known as Drone === genii is now known as genii-core === franksmcb_ is now known as franksmcb