=== benfrancis8 is now known as benfrancis
mupPR snapd#9279 closed: github: run tests also on push to release branches <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9279>05:12
mvohey mborzecki - good morning05:18
mborzeckimvo: hey05:18
mborzeckiheh, my son got fever today, and it's only been like 4 days of school, didn't take long for him to catch something05:19
mvomborzecki: oh no!05:25
mvomborzecki: good luck05:25
mvomborzecki: let's hope it goes away again quickly05:26
mborzeckiand i can reproduce the problem with govendor sync on TW: https://paste.ubuntu.com/p/mDHNQV7kTD/05:59
mborzeckistill, makes very little sense05:59
mborzeckiquick errand, need to drive my daughter to school06:22
mvopstolowski: good morning07:03
mvomborzecki: hm, memory exaution or something?07:03
mborzeckipstolowski:  hey07:03
mborzeckimvo:  idk, running with strace now07:03
mvomborzecki: \o/07:03
mborzeckimvo: even `if ! govendor sync -v; then echo failed; fi` fails and shuts down the connection07:04
mborzeckimvo: oh, and it does not fail when i add exit 1 before the line in prepare-restore, and then run govendor sync from the debug shell07:05
mvomborzecki: you run it with or without quiet?07:08
mvomborzecki: I wonder if that has anything to do with it :/07:08
mborzeckimvo: with or without has same effect07:08
zygagood morning07:23
mvogood morning zyga07:24
zygahow are you doing? it's very warm for autumn this year07:24
zygaI'm still looking after Lucy, just getting ready for our call07:25
mvozyga we can postpone if you want07:28
mborzeckizyga: ehy07:33
zygamvo: that's fine, lets's have the call07:37
* zyga needs to restart avahi after setting up some bridges last night07:38
mvozyga sure thing07:42
* mvo is happy that zyga build bridges instead of burning them07:43
zygaI have a routable riscv64 VM now07:43
zygasomething that was in the back of my head for a while07:43
mborzeckiSep 07 07:49:52 sep070746-567919 kernel: sshd[1445]: segfault at 278 ip 000055e17401a553 sp 00007ffc9aba67c0 error 4 in sshd[55e173fd8000+74000]07:50
mborzeckiSep 07 07:49:52 sep070746-567919 kernel: Code: 1f 84 00 00 00 00 00 48 85 ff 0f 84 d7 00 00 00 41 54 55 48 89 fd 48 83 ec 08 48 8b 87 20 01 00 00 48 85 c0 0f 84 95 00 00 00 <8b> 40 18 a8 02 707:50
mborzecki5 46 a8 04 0f 85 8e 00 00 00 48 8b 7d 08 e8 05 f007:50
mborzeckion tumbleweed when disconnecting spread07:50
mborzeckimaybe that's why debug shell cannot be connected?07:50
zygaseg.vfault() :(07:50
zygayeah, this could be a problem07:51
zygamaybe we need to reboot after upgrades?07:51
mborzeckizyga: looking at zypper log there's  no updates pulled in spread stats07:52
zygamaybe that's something exploiting an unknown bug in ssh?07:53
mborzeckioh wow, kernel bug? https://paste.ubuntu.com/p/bDGGb7BhWX/07:53
zygathose machines are under constant barrage of ssh connections07:53
mborzeckithe network is down at this point07:53
mborzeckiidk, maybe this snapshot is just bad :/07:53
zygathat's 5.8?07:54
mborzeckithere's a bunch of updates (kernel among those)07:55
mborzeckimaybe we should ask cachio to update the images and we'll try again07:56
zygagood idea07:56
zygaI updated my TW install and didn't see any issues07:56
zygaand I released 2.46 there, waiting for .1 tarballs07:56
zygamvo: do you remember if 2.46.1 tarball contains the libexec patch from Maciej?07:57
mborzeckizyga: it does not, the PR is still up https://github.com/snapcore/snapd/pull/9280 and we cannot merge it because the spread job on TW does not run07:58
mupPR #9280: packaging/opensuse: fix for /usr/libexec on TW, do not hardcode AppArmor profile path <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9280>07:58
zygamborzecki: aha07:58
zygaok, I'll cherry pick that as distro patch07:58
zygait's interesting that openSUSE is the redhat-like distro but with apparmor07:58
zygacan we do libexec in ubuntu 14.04+ and remove all patches now ;-) ?07:59
zyga(no, we cannot because debian stable)07:59
mupPR core#118 opened: hooks: update motd handling and remove 50-motd-news <Created by mvo5> <https://github.com/snapcore/core/pull/118>07:59
* pstolowski errand08:24
zyga-kaveriin the office08:43
mupPR core#38 closed: Add another pi-config option <Created by sergey-borovkov> <Closed by pedronis> <https://github.com/snapcore/core/pull/38>08:44
mupPR core#110 closed: Disable dynamic motd <Created by vorlonofportland> <Closed by mvo5> <https://github.com/snapcore/core/pull/110>08:44
mupPR core#111 closed: live-build/hooks/motd: disable dynamic motd, disable motd services <Created by bboozzoo> <Closed by mvo5> <https://github.com/snapcore/core/pull/111>08:44
* zyga-kaveri goes upstairs for a pullover09:11
zyga-kaverimvo: when convenient please provide 2.46.1 tarballs09:11
mvozyga-kaveri: will do, sorry!09:11
zyga-kaverimvo: no problem09:27
mvozyga-kaveri: it's up now09:38
zyga-kaverimvo: thanks09:38
mupPR snapd#9289 opened: github: run tests also on push to release branches <Simple 😃> <Skip spread> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9289>09:43
mupPR snapd#9290 opened: release: 2.46.1 <Skip spread> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9290>09:48
mupPR snapd#9289 closed: github: run tests also on push to release branches <Simple 😃> <Skip spread> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9289>10:18
mupPR snapd#9210 closed: daemon: add /v2/systems "reboot" action API <UC20> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9210>10:38
=== pedronis_ is now known as pedronis
* zyga-kaveri breaks for coffee and then goes to write manager hooks11:03
mvoI updated 9021, hopefully a relatively simple review. for snap reboot11:12
mborzeckihmm `snapd   2.46.1-1-dirty` mkversion getting confused by something there?11:24
pedronismvo: I answered your question there, but didn't do a review yet11:35
mborzeckiquick errand, back in 3011:40
mborzeckicachio: hi, can you take a look whether we can update opensuse tumbleweed images maybe?12:07
cachiomborzecki, hi, yes12:07
mborzeckicachio: looks like the current image is broken, and that's what is causing issues with govendor sync12:08
* zyga-kaveri writes more tests12:08
mborzeckicachio: i was able to observe a kernel panic when govendor sync runs, breaking the host12:08
pedronispstolowski: is #9270 ready for reviews ?12:08
mupPR #9270: [RFC] wrappers, systemd: allow empty root dir and conditionally do not pass --root to systemctl <Run nested> <Services ⚙️> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9270>12:08
pstolowskipedronis: yes but i'd wait for spread tests, so far i saw many failures which seemed unrelated12:10
pstolowskibut maybe not, hard too tell, too much noise there12:10
pstolowskii'm keeping an eye on that one12:11
pstolowskipedronis: if you could advise on error code for #9259 that would be great12:12
mupPR #9259: client, api: handle insufficient space error <Disk space awareness> <Needs Samuele review> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9259>12:12
pedronisI plan to review that soon12:12
mupPR snapd#9285 closed: many: seed.Model panics now if called before LoadAssertions <UC20> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9285>12:33
mborzeckipedronis: hm building on sid with nosecboot must have been failing for a while now12:38
pedronismborzecki: it passes in other PRs12:39
mborzeckipedronis: maybe go is smart to figure out what it needs during the build, anyways, had to fix debian packaging to not remove secboot/encrypt.go (added earlier i believe) and gadget/install/params.go (added in claudio's PR)12:40
mborzeckiand yeah, the fun code copying over stuff to _build12:40
pedronismborzecki: something is wrong12:41
pedronismborzecki: why params and encrypt?12:43
mborzeckipedronis: install_dummy references Options (defined in params.go), and SystemInstallObserver.ChosenEncryptionKey() takes secboot.EncryptionKey (defined in secboot/encrypt.go)12:44
pedronisyes, but secboot/encrypt.go doesn't refer to anything interesting12:44
pedroniswhat am I missing12:44
pedronisI don't think you can build devicestate without params12:45
mborzeckipedronis: hm maybe i wasn't clear, afaict secboot/encrypt.go is needed to get secboot.EncryptionKey, used by gadget/install/params.go12:45
pedronisah, not remove12:46
mborzeckipedronis: and the code that sets up _build would remove everything from gadget/install and secboot but a few select files12:46
pedronissorry, I misread your comment12:46
mborzeckipedronis: cool, all is clear now ;)12:46
mborzeckicachio: let me know when you have a new opensuse tumbleweed image to try, wonder if that kernel bug is no longer happening12:49
cachiomborzecki,  it is in the queue12:49
cachioto be updated, i think in 20 minutes should be ready12:50
mvopedronis: thankyou!12:53
mupPR snapcraft#3279 opened: Set VDPAU_DRIVER_PATH appropriately <Created by flexiondotorg> <https://github.com/snapcore/snapcraft/pull/3279>12:56
mborzeckicachio: can you take a look at the console of spread node `sep071325-531734`?13:32
cachiomborzecki, sure13:33
cachiomborzecki, can't see that instances13:34
mborzeckicachio: hmm ok, i died when i tried to run the tests13:34
mborzeckicachio: let me start another one13:35
mborzeckicachio: can you observe the console of this one `sep071335-735209` ?13:35
mborzecki**** fought a bit with Sid packaging in Claudio's PR https://github.com/snapcore/snapd/pull/926513:38
mupPR #9265: many: move seal code from gadget/install to boot <Run nested> <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/9265>13:38
mborzecki     - we have some special code that sets up a separate source tree under _build13:38
cachiomborzecki, kernel BUG at mm/filemap.c:1315!13:38
mborzecki     - it would remove all unnecessary files, but a few selected ones13:38
mborzecki     - with the changes from the PR, files that were removed are actually needed13:38
mborzecki       to build with nosecboot tag13:38
mborzeckianother bug? https://paste.ubuntu.com/p/273tJhBxVd/13:38
pedronispstolowski: I reviewed #925913:38
mupPR #9259: client, api: handle insufficient space error <Disk space awareness> <Needs Samuele review> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9259>13:38
mborzeckicachio: yeah, and now it's dead isn't it?13:38
mborzeckiduh, secondary clipboard13:39
cachiomborzecki, https://paste.ubuntu.com/p/GXsBpHsmD7/13:40
cachiostill running13:40
mborzeckicachio: hmm BUG: workqueue lockup - pool cpus=0 node=0 flags=0x0 nice=0 stuck for 60s!13:43
mborzeckizyga-kaveri: ^^13:43
mborzeckicachio: tumbleweed is not an official image, we build it ourselves right?13:44
cachiomborzecki, yes13:45
cachiono official cloud image13:45
cachiomborzecki, this is hte script we use for that https://github.com/snapcore/spread-images/blob/master/tasks/google/add-opensuse-tumbleweed-64/task.yaml13:45
mupPR snapd#9290 closed: release: 2.46.1 <Skip spread> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9290>13:54
pstolowskipedronis: ty14:02
mborzeckimvo: i can try a reproduce the problem ian had on rpi4, got 1.2rev here14:33
mvomborzecki: cool, if it's not too much hassle, that would be great14:35
mupPR snapd#9291 opened: client: implement RebootToSystem <Created by mvo5> <https://github.com/snapcore/snapd/pull/9291>14:44
* cachio lunch15:04
mvodid anyone look into the 503 we get from the store for the main/searching test ? has anyone more info on that? or contacted the store team?16:16
cachiomvo, I'll do it16:21
mvothanks cachio16:22
pstolowskii'm seeing a lot of that too16:27
cachiopstolowski, can you reproduce the isuse doing -> snap find --section=photo-and-video vlc ?16:29
pstolowskicachio: no16:30
cachiopstolowski, tx16:30
mvome neither16:31
pstolowskihowever, mistyping section name gives 403, was it always like that?16:46
zygaback from PT18:33
zygaissues with pi?18:34
zygaijohnson, cachio: anything new?18:35
cachiozyga, sorry, issues?19:07
cachiobeta validation was ok for rpi19:07
cachiodidn't see anything weird19:08
* zyga finishes supper and will look at writing one more function19:10

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!