| mborzecki | morning | 06:39 |
|---|---|---|
| pstolowski | morning | 07:29 |
| pstolowski | jamesh: hi, i pushed one more commit to https://github.com/snapcore/snapd/pull/10867, there was a test failure on 21.10 and i think that fixed it. i think we need to unregister dbus name (mostly important for test) | 07:30 |
| mup | PR #10867: desktop, usersession: observe notifications <Created by stolowski> <https://github.com/snapcore/snapd/pull/10867> | 07:30 |
| jamesh | pstolowski: hmm. So that's due to reusing the same bus between tests? | 08:00 |
| pstolowski | jamesh: i think so, yes. i don't have the detailed error anymore, but it was about not being able to register the name; it would fail 9/10 times | 08:03 |
| pstolowski | only 21.10 for some reason, and also i couldn't reproduce it locally on 21.10, but could easily reproduce with spread in gc | 08:04 |
| jamesh | Maybe a race condition between one connection being closed and the next opened? | 08:07 |
| jamesh | pstolowski: either way, manually releasing the bus name on clean up sounds fine. | 08:07 |
| pstolowski | jamesh: good, ty | 08:17 |
| mvo | fun! looks like the syscalls in libseccomp need another update | 08:21 |
| mborzecki | hmm something broke on centos between 2.53.1 and master | 08:22 |
| mborzecki | actually may be something | 08:22 |
| mborzecki | more recent | 08:22 |
| pstolowski | mvo: hi, i saw it a failure on 18.04 i think, is that it? | 08:28 |
| pstolowski | hey mborzecki | 08:28 |
| mborzecki | mvo: pstolowski hey | 08:34 |
| mborzecki | fun, i see that snapd got the http request, so it's not selinux that's breaking this, but nothing is being sent back | 08:37 |
| mborzecki | mvo: lp builds are failing | 08:46 |
| mvo | mborzecki: uh, ok - let me check | 08:57 |
| mvo | pstolowski: yeah, the 18.04 failure is the canary for the new syscalls | 08:58 |
| mborzecki | mvo: this is what's failing: https://pastebin.ubuntu.com/p/NhQGK225fv/ | 08:58 |
| mborzecki | seems to be consistent | 08:58 |
| mvo | mborzecki: ok | 08:59 |
| mborzecki | hmm ok, i'm confused, so curl works the same, i see the request on the snapd side, there's no response | 09:03 |
| mborzecki | hmm reverting https://github.com/snapcore/snapd/commit/d550f860a23625c53bfce624949a76c458269d80 makes it work again | 09:05 |
| mborzecki | mvo: ^^ hmm wonder if maybe it's the dbus bits, but i don't understand how that PR would block responding over the api socket | 09:06 |
| mborzecki | so it's dbus, somehow the code is stuc receiving from timesyncd, https://pastebin.ubuntu.com/p/x2SJddTDDG/ after taking the state lock, so all other requests through the API are stuck now waiting for state.Lock() | 09:30 |
| mup | PR snapd#11007 closed: o/snapstate: test relink remodel helpers do a proper subset of doInstall and rework the verify*Tasks helpers <Skip spread> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/11007> | 09:47 |
| mborzecki | pstolowski: can you take a look at https://github.com/snapcore/snapd/pull/11017 ? | 09:58 |
| mup | PR #11017: data/selinux: update the policy to allow snapd to talk to org.freedesktop.timedate1 <⚠ Critical> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/11017> | 09:58 |
| mup | PR snapd#11017 opened: data/selinux: update the policy to allow snapd to talk to org.freedesktop.timedate1 <⚠ Critical> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/11017> | 10:02 |
| pstolowski | mborzecki: +1 with a question | 10:03 |
| pstolowski | mborzecki: can you re-review https://github.com/snapcore/snapd/pull/10962 ? | 10:05 |
| mup | PR #10962: o/assertstate: functions for handling validation sets tracking history <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10962> | 10:05 |
| mborzecki | pstolowski: sure | 10:08 |
| abeato | hey, I'm getting some errors while building snap-bootstrap for arm64: https://paste.ubuntu.com/p/2HQpZRyNR3/ (amd64 builds fine) - any hint on what might be going wrong here? | 10:19 |
| pstolowski | abeato: hi, have you run ./get-deps.sh ? | 10:28 |
| abeato | pstolowski, not in a long time, let me try. Does it install something needed for cross-compilation? | 10:28 |
| pstolowski | abeato: ah, i missed GOARCH=arm64 there | 10:29 |
| pstolowski | no idea, but worth doing anywy | 10:29 |
| pstolowski | abeato: yeah i'm getting the same with GOARCH=arm64 | 10:30 |
| abeato | hm, unfortunately that didn't help... I guess some bits in the dependency are calling some C code | 10:30 |
| abeato | I'll build on a Pi, that should be fine | 10:31 |
| abeato | and yeah, that worked | 10:35 |
| pstolowski | abeato: yeah there is cdefs.go in that package that needs #include <linux/input.h> | 10:36 |
| mborzecki | abeato: you can also set CC= to the right compiler | 10:38 |
| abeato | mborzecki, right, that should work | 10:39 |
| mup | Issue core18#56 closed: Sensible version number <Created by mvo5> <Closed by Meulengracht> <https://github.com/snapcore/core18/issues/56> | 11:05 |
| mup | Issue core18#86 closed: Ubuntu 18.10 <Created by kravietz> <Closed by Meulengracht> <https://github.com/snapcore/core18/issues/86> | 11:10 |
| mup | Issue core18#117 closed: update-motd is installed but cannot operate on read-only / <bug> <Created by zyga> <Closed by Meulengracht> <https://github.com/snapcore/core18/issues/117> | 11:25 |
| mup | Issue core18#129 closed: Multiarch isn't working <Created by xordspar0> <Closed by Meulengracht> <https://github.com/snapcore/core18/issues/129> | 11:35 |
| mup | Issue core18#142 closed: Stuck with blank screen on snap-core 18 Raspi 3 B+ install <Created by bernermic> <Closed by Meulengracht> <https://github.com/snapcore/core18/issues/142> | 11:35 |
| mup | Issue core18#153 closed: missing /etc/cloud/cloud.cfg.d/99-snappy-azure.cfg <Created by anonymouse64> <Closed by Meulengracht> <https://github.com/snapcore/core18/issues/153> | 11:50 |
| mup | Issue core18#158 closed: quoting fix for timedatectl wrapper got lost between core16 and core18 <Created by ogra1> <Closed by Meulengracht> <https://github.com/snapcore/core18/issues/158> | 11:55 |
| mup | PR snapd#11018 opened: cmd/snap-seccomp/syscalls: update syscalls to match libseccomp abad8a8f4 <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/11018> | 12:17 |
| mborzecki | mvo: ^^ | 12:18 |
| mup | PR snapd#10998 closed: o/snapstate: do not prune refresh-candidates if gate-auto-refresh-hook feature is not enabled <Bug> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10998> | 12:28 |
| mvo | mborzecki++ | 12:30 |
| mup | PR snapd#11019 opened: overlord/state: add a unit test for a kernel+base refresh like sequence <Simple 😃> <Skip spread> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/11019> | 13:53 |
| mborzecki | oh wow Failed to download action 'https://api.github.com/repos/actions/checkout/tarball/ec3a7ce113134d7a93b817d10a8272cb61118579'. Error: Resource temporarily unavailable | 14:24 |
| mup | PR snapd#10974 closed: docs: add run-checks dependency install instruction <Created by flotter> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10974> | 15:28 |
| mup | PR snapd#11020 opened: timeutil,deviceutil: fix unit tests on systems without dbus <⚠ Critical> <Created by mvo5> <https://github.com/snapcore/snapd/pull/11020> | 16:08 |
| === Kamilion|ZNC is now known as Kamilion | ||
| mup | PR snapd#7586 closed: interfaces/optical-drive: additional permission needed by mir-kiosk-kodi <⛔ Blocked> <:birthday:> <Needs security review> <Created by AlanGriffiths> <Closed by AlanGriffiths> <https://github.com/snapcore/snapd/pull/7586> | 16:49 |
| mup | PR snapd#11009 closed: cmd/snap-bootstrap/README: explain all the things (well most of them anyways) <Documentation> <Skip spread> <Created by anonymouse64> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/11009> | 17:54 |
| mup | PR snapd#11021 opened: gadget: replace ondisk implementation with disks package, refactor part calcs <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/11021> | 19:59 |
| mup | PR snapd#11022 opened: gadget, many: add model param to Update() <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/11022> | 23:35 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!