| === chihchun_afk is now known as chihchun | ||
| mup | PR snapcraft#2353 closed: tests: remove dependency on snapcraft for integration tests <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2353> | 04:19 |
|---|---|---|
| === cpaelzer_ is now known as cpaelzer | ||
| mborzecki | morning | 05:11 |
| mborzecki | ugh, travis jobs unhappy again | 05:13 |
| zyga | good morning | 05:43 |
| zyga | mborzecki: the dreaded accounts service test | 05:43 |
| mborzecki | zyga: hey | 05:43 |
| mborzecki | yeah, not repeating in isolation :/ | 05:44 |
| zyga | I reproduced it but didn't noice and closed the session | 05:44 |
| mborzecki | ayy | 05:44 |
| mup | PR snapd#5990 opened: tests: do not run degraded test in autopkgtest env <Created by mvo5> <https://github.com/snapcore/snapd/pull/5990> | 05:44 |
| mvo | hey zyga | 05:45 |
| mvo | and hey mborzecki | 05:45 |
| mborzecki | mvo: hey | 05:45 |
| zyga | hey mvo | 05:45 |
| zyga | good morning | 05:45 |
| zyga | I haven't seen infinity yet | 05:45 |
| mvo | zyga: I just asked him in #ubuntu-release | 05:45 |
| zyga | I kicked off main again, I will try to hunt the accounts service failure | 05:45 |
| zyga | thanks! | 05:45 |
| mborzecki | hm maybe it's something about gce being slow or sth, that online accounts service is dbus activated, isn't it? | 05:46 |
| mvo | zyga: thank you - all autopkgtest are good except for the degraded one | 05:46 |
| mvo | zyga: I do the proper release now (without the umount fix sadly) | 05:46 |
| zyga | mborzecki: unlikely, bus activated services get the socket from systemd, right? | 05:54 |
| zyga | mborzecki: so you'd race with systemd | 05:54 |
| zyga | well - we shall see really | 05:54 |
| mup | PR snapd#5966 closed: snap: overhaul validation error messages <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/5966> | 06:12 |
| mup | PR snapd#5991 opened: release: 2.35.5 <Created by mvo5> <https://github.com/snapcore/snapd/pull/5991> | 06:16 |
| mup | PR snapd#5992 opened: release: merge 2.35.5 changes into the 2.36 branch <Created by mvo5> <https://github.com/snapcore/snapd/pull/5992> | 06:21 |
| zyga | https://blogs.gnome.org/tbernard/2018/10/15/restyling-apps-at-scale/ <- interesting for theming | 06:24 |
| mborzecki | heh pushed a commit with some additional debug to the PR that failed consistntly due to interfaces-accounts-service, and it's failing no more | 06:46 |
| mborzecki | why am i even suprised | 06:46 |
| zyga | brb | 06:48 |
| zyga | mborzecki: it's just an ordering issue | 06:48 |
| mborzecki | zyga: i'm not so sure, the testt that executed before this one do not seem reelvant | 06:57 |
| zyga | mborzecki: I sent a trivial PR, we'll see | 07:01 |
| mborzecki | zyga: are debug sections meged together? | 07:02 |
| mup | PR snapd#5993 opened: tests: show list of processes when ifaces-accounts-service fails <Created by zyga> <https://github.com/snapcore/snapd/pull/5993> | 07:02 |
| pstolowski | mornings | 07:03 |
| mborzecki | pstolowski: hey | 07:03 |
| zyga | Oh! No | 07:09 |
| mborzecki | pulled logs from 6 failed runs, dropped into python to check if there's a test common to all failed jobs that ran before, seems there's no connection there | 07:14 |
| mborzecki | maybe we shuld just switch this test to manual to unblock prs | 07:15 |
| mborzecki | zyga: this is what i'm trying with here https://paste.ubuntu.com/p/Ktcj47ZDqh/ | 07:23 |
| mup | PR snapd#5991 closed: release: 2.35.5 <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5991> | 07:29 |
| mup | PR snapd#5990 closed: tests: do not run degraded test in autopkgtest env <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5990> | 07:31 |
| pedronis | mborzecki: the failure mode of that test look like the are different services that provide similar but different apis for something, it's quite weird | 07:45 |
| mup | PR snapd#5994 opened: tests/main/interfaces-accounts-service: more debugging <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5994> | 07:55 |
| mborzecki | https://www.reddit.com/r/linux/comments/9oddtb/flatpaks_sandboxes_and_security/ another *pak & sandbox thread | 08:04 |
| mup | PR snapd#5984 closed: ifacestate/hooks: only create interface hook tasks if hooks exist (2.36) <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5984> | 08:10 |
| zyga | woot, that's great | 08:10 |
| zyga | thank you again pstolowski | 08:10 |
| zyga | I will try to optimise apparmor use next week | 08:10 |
| pstolowski | zyga: sounds awesome! | 08:10 |
| zyga | man, reading reddit is a waste of time | 08:17 |
| mborzecki | zyga: heh, usual bs in the comments | 08:18 |
| zyga | BS is one thing, I really cannot stand people that claim to know something but are clearly clueless about it | 08:18 |
| mborzecki | btw. on the topic of desktop themes, anyone running kde? do gtk apps from snaps have breeze properly applied to them? | 08:23 |
| zyga | mborzecki: sorry, I'm on gnome usually | 08:24 |
| zyga | but on the bright side, we will always have Adwaita | 08:24 |
| zyga | eh | 08:29 |
| zyga | not a great morning | 08:29 |
| mvo | sil2100: bad news, probert seems to be still crashing, I have a theory (only tested on my pi3-64) | 08:30 |
| zyga | question, what is probers? | 08:30 |
| zyga | probert | 08:30 |
| Chipaca | we need a "this topic provides no added value" thing for the forum :-| | 08:40 |
| * zyga hugs Chipaca | 08:40 | |
| zyga | what topic is that? | 08:41 |
| Chipaca | zyga: today, it's https://forum.snapcraft.io/t/gnome-calendar-is-not-in-spanish/7866/11 | 08:41 |
| Chipaca | if it were a support ticket, i wouldn't mind | 08:41 |
| Chipaca | also, ps, spanish is not a regional language :) | 08:43 |
| sil2100 | mvo: oh no | 08:45 |
| sil2100 | Let me check if the right version has been pulled in | 08:45 |
| sil2100 | Yeah, it looks like it | 08:46 |
| sil2100 | mvo: so it's still crashing at the same place in console-conf? On startup? | 08:46 |
| Chipaca | zyga: has anything reached master since yesterday? | 08:48 |
| zyga | not that I know of :/ | 08:48 |
| zyga | let me look | 08:48 |
| zyga | Chipaca: actually, I'm mistaken | 08:49 |
| zyga | 063f83e687d1fa2d339bc123ff5c37a2030afb50 landed | 08:49 |
| Chipaca | zyga: hello mistaken, i'm Chipaca | 08:49 |
| zyga | I'm Zygmunt if I'm not mistaken | 08:49 |
| zyga | ;-) | 08:49 |
| zyga | (or both, knowing me) | 08:49 |
| Chipaca | mborzecki: does 'gdbus monitor' work withouot --dest? | 08:50 |
| Chipaca | $ gdbus monitor --session | 08:51 |
| Chipaca | Error: Destination is not specified | 08:51 |
| sil2100 | zyga: did you try a new image on your dragonboard? | 08:51 |
| Chipaca | mborzecki: go with dbus-monitor --session | 08:51 |
| zyga | sil2100: I only have the image from Friday | 08:51 |
| zyga | shall I re-download http://people.canonical.com/~lzemczak/dragonboard.img.xz ? | 08:52 |
| mup | PR snapd#5995 opened: tests/main/interfaces-accounts-service: switch to manual while we debug the test <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5995> | 08:53 |
| mborzecki | Chipaca: see #5994 | 08:54 |
| mup | PR #5994: tests/main/interfaces-accounts-service: more debugging <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5994> | 08:54 |
| Chipaca | mborzecki: better :) | 08:55 |
| Chipaca | mborzecki: or was it always like that, and I'm in need of more coffee? | 08:55 |
| zyga | meh, now travis is not doing much | 08:55 |
| zyga | ohhh coffee | 08:55 |
| mborzecki | Chipaca: no, i mistyped it, had it `dbus monitor` before | 08:55 |
| zyga | I could use that | 08:55 |
| Chipaca | hah, even more fun | 08:55 |
| mborzecki | did eavesdrop need to be expliclitly enabled for session bus? | 08:56 |
| zyga | not that I recall | 08:57 |
| Chipaca | mborzecki: what about checking the package version of accountsservice? | 08:59 |
| mborzecki | Chipaca: that should be the same as if you spin up the test image | 09:00 |
| Chipaca | hm, tru | 09:01 |
| Chipaca | e | 09:01 |
| mborzecki | Chipaca: gnome-online-accounts 3.18.3-1ubuntu2 | 09:03 |
| mborzecki | btw. pushed a repo with a minimal spread setup i use to spin up images, faster and smaller than doing the same from snapd tree https://github.com/bboozzoo/spread-mini | 09:07 |
| === chihchun is now known as chihchun_afk | ||
| === cpaelzer_ is now known as cpaelzer | ||
| Chipaca | mborzecki: also: gdbus introspect --session --dest=org.gnome.OnlineAccounts --object-path=/org/gnome/OnlineAccounts/Manager | 09:12 |
| === chihchun_afk is now known as chihchun | ||
| Chipaca | WTF, travis | 09:35 |
| Chipaca | mborzecki: zyga: as travis is kinda stuck, would you mind merging #5994 and #5993 into a single one? | 09:43 |
| mup | PR #5994: tests/main/interfaces-accounts-service: more debugging <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5994> | 09:43 |
| mup | PR #5993: tests: show list of processes when ifaces-accounts-service fails <Created by zyga> <https://github.com/snapcore/snapd/pull/5993> | 09:43 |
| mborzecki | hahah | 09:43 |
| Chipaca | oh wait one of them is already running | 09:43 |
| Chipaca | ¯\_(ツ)_/¯ | 09:44 |
| mborzecki | ok, hm last time we were debugging this test the observation was that the signature returned as 'expected' is correct and matches that of AddAccount | 09:48 |
| sil2100 | zyga: I'll rebuild it in a moment | 09:49 |
| sil2100 | (the image) | 09:49 |
| zyga | sil2100: ok | 09:49 |
| mborzecki | looking at my notes i was digging into gdbus to see if there's some parsing error | 09:49 |
| sil2100 | I'm on the release sprint right now and we need to fix ubiquity first | 09:49 |
| zyga | sure | 09:49 |
| mvo | sil2100: sorry for the delay, had a call - my mistake, console-conf is working! | 09:54 |
| Chipaca | mborzecki: ISTR gdbus was funny about how it interpreted arguments | 09:57 |
| mborzecki | Chipaca: i switched to busctl now, will push a commit in a minute | 09:57 |
| Chipaca | mborzecki: but you can specify the type | 09:57 |
| mborzecki | at least this one is explicit about the types | 09:57 |
| Chipaca | mborzecki: ah, good | 09:58 |
| zyga | mborzecki: what is the chance that with more debugging it will no longer fail ;-) ? | 10:06 |
| mborzecki | zyga: high :) | 10:07 |
| zyga | still, I'd love to merge it | 10:07 |
| zyga | to see what is wrong | 10:07 |
| zyga | I'm wary of merging your branch with extra kills and what not just because we don't know the problem yet | 10:08 |
| mborzecki | the upside is we'll know more the next time it appears | 10:08 |
| zyga | yes, exactly | 10:08 |
| mborzecki | zyga: i'll drop pkills | 10:08 |
| sil2100 | mvo: \o/ | 10:15 |
| sil2100 | mvo: sweet! | 10:15 |
| sil2100 | I'll prep a quick image for zyga in a moment for the db and upload it somewhere, since I'm at the office it should be a fast upload | 10:16 |
| zyga | cool | 10:16 |
| mborzecki | zyga: while pipe2 instead of socketpair in your pr? | 10:18 |
| zyga | no preference really | 10:18 |
| zyga | also no need to use sockets :-) | 10:19 |
| mborzecki | zyga: Chipaca: please take a look at #5994 | 10:20 |
| mup | PR #5994: tests/main/interfaces-accounts-service: more debugging <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5994> | 10:20 |
| zyga | what is in dbus.env? | 10:21 |
| mborzecki | zyga: when you dbus-launch it spits some stuff you can eval in the shell to set the environment variables, now it's dumped to a file in case i need to use it in debug section | 10:22 |
| zyga | right | 10:24 |
| zyga | I know that, | 10:24 |
| zyga | what is there in practice :) | 10:24 |
| zyga | is it just a variable definition? | 10:24 |
| Chipaca | DBUS_SESSION_BUS_ADDRESS='unix:abstract=/tmp/dbus-QvYL1sZnrl,guid=78c09efa6f3ce976cbffb0a35bc5bc72'; | 10:24 |
| Chipaca | export DBUS_SESSION_BUS_ADDRESS; | 10:24 |
| Chipaca | DBUS_SESSION_BUS_PID=25966; | 10:24 |
| Chipaca | DBUS_SESSION_BUS_WINDOWID=119537665; | 10:24 |
| mborzecki | zyga: https://paste.ubuntu.com/p/TTh392C3Bb/ | 10:25 |
| zyga | aha, I see | 10:25 |
| zyga | thanks! | 10:25 |
| mborzecki | oh, windowid? | 10:25 |
| mborzecki | wonder why i don't have it | 10:25 |
| Chipaca | mborzecki: probably because I did it from within X and you didn't | 10:26 |
| zyga | mborzecki: my PR passed | 10:26 |
| zyga | let's merge it | 10:26 |
| zyga | man, I knew this would happen | 10:26 |
| mborzecki | Chipaca: hm did it in tmux, which was in turn started by systemd user session, maybe that's why | 10:27 |
| zyga | sil2100: flashing now | 10:27 |
| Chipaca | an extra downside of this accounts service silliness is that it took me a day to see a trivial bug in my shutdown pr | 10:27 |
| zyga | now let's reproduce the f*r | 10:28 |
| mup | PR snapd#5993 closed: tests: show list of processes when ifaces-accounts-service fails <Created by zyga> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/5993> | 10:28 |
| sil2100 | zyga: thanks! Fingers crossed it's all resolved here as well | 10:28 |
| zyga | we will know soon | 10:30 |
| mup | PR snapd#5996 opened: [RFC] daemon: support `snap logs snapd` to get snapd logs <Created by mvo5> <https://github.com/snapcore/snapd/pull/5996> | 10:33 |
| mvo | cachio: good morning! new snapd 2.35.5 in beta - sorry for yet another dot release :/ | 10:35 |
| mvo | cachio: but .4 is fine, this is just yet another fix on top | 10:35 |
| mup | PR snapd#5997 opened: ifacestate: simplify task chaining in ifacestate.Connect <Created by stolowski> <https://github.com/snapcore/snapd/pull/5997> | 10:37 |
| pstolowski | mborzecki: ^ the simplification you suggested in previous PRs | 10:38 |
| pstolowski | *PR | 10:38 |
| zyga | nice pstolowski :) | 10:38 |
| mborzecki | pstolowski: nice | 10:38 |
| zyga | sil2100, mvo: booting ... | 10:40 |
| zyga | fingers crossed | 10:40 |
| zyga | sil2100, mvo: no more crashes! | 10:43 |
| zyga | woot | 10:43 |
| mvo | zyga: yay | 10:43 |
| zyga | sil2100: what was the issue? | 10:43 |
| zyga | sil2100, mvo: although it does not crash I cannot actually connect to wifi | 10:47 |
| zyga | it times out all the time | 10:47 |
| zyga | (wifi dit not change) | 10:47 |
| mvo | zyga: same for me on my pi3-64 | 10:47 |
| mvo | sil2100: -^ | 10:47 |
| zyga | bor0en | 10:48 |
| zyga | or just broken | 10:48 |
| === prime is now known as Guest4938 | ||
| Chipaca | mvo: want me to give the appInfosFor a stab? I'm blocked waiting for spread otherwise :-) | 10:51 |
| Chipaca | bah, i've got other things i could do | 10:51 |
| Chipaca | like lunhc | 10:52 |
| Chipaca | lunch | 10:52 |
| Chipaca | but that's boring | 10:52 |
| zyga | sil2100: do you need that reported? | 10:57 |
| mup | PR snapd#5998 opened: osutil: tweak handling of error adduser errors <Created by mvo5> <https://github.com/snapcore/snapd/pull/5998> | 10:59 |
| mvo | Chipaca: app infos for as in 5996? | 11:01 |
| Chipaca | mvo: ah, yes :-) | 11:01 |
| mvo | Chipaca: I would not mind - but note its still an RFC PR that needs approval from gustavo | 11:02 |
| Chipaca | mvo: actually we could expose a pseudo-snap 'snapd' in /v2/snaps | 11:03 |
| mvo | Chipaca: that works for me | 11:04 |
| mvo | Chipaca: no strong opinion either way | 11:04 |
| Chipaca | mvo: snapInfosFor calls allLocalSnapInfos, if we make that one pseudo-snapd-aware, we get a consistent pseudo-snapd in the api (except you can't remove/refresh/etc it sometimes) | 11:04 |
| Chipaca | (the latter makes it awkward, i guess) | 11:04 |
| Chipaca | bah :) | 11:04 |
| Chipaca | it depends on what we want | 11:05 |
| Chipaca | i'll comment on the rfc | 11:05 |
| * mvo nods | 11:05 | |
| mvo | thank you! | 11:05 |
| pedronis | Chipaca: we have system and snapd (as pseudo but not quite things), should snap logs system do the same, do something? | 11:07 |
| pedronis | all of this is cute but without a plan it will get messy | 11:07 |
| zyga | snap logs system giving just snapd logs feels weird though | 11:07 |
| pedronis | yea, agreed | 11:08 |
| pedronis | also snapd will be a snap | 11:08 |
| zyga | but without services | 11:08 |
| zyga | so all kinds of weirdness | 11:08 |
| sil2100 | zyga: yeah, not sure what's the reason for that not working, I think I also had wifi issues on my regular armhf rpi3 core18 last time | 11:09 |
| Chipaca | pedronis: zyga: FC on the RFC | 11:09 |
| sil2100 | Not sure if that got reported somewhere | 11:09 |
| sil2100 | I guess Paolo was looking at something like that? | 11:09 |
| pedronis | Chipaca: my C is mostly that it probably needs a forum topic | 11:10 |
| Chipaca | sergiusens: what was the env var that dropped snap/snapcraft.yaml and manifest.yaml into the snap? | 11:16 |
| pedronis | Chipaca: I think SNAPCRAFT_BUILD_INFO=1 from here: https://forum.snapcraft.io/t/notifications-for-out-of-date-stage-packages/5161 | 11:18 |
| Chipaca | thanks | 11:18 |
| sergiusens | Chipaca: I see you found it | 11:27 |
| Chipaca | sergiusens: yes, thanks | 11:28 |
| pstolowski | we're at the brink of opening 3rd page of PRs... would be good to land some | 11:28 |
| Chipaca | grep took me to the tests, and then samuele confirmed it :) | 11:28 |
| Chipaca | pstolowski: har har | 11:28 |
| sergiusens | Chipaca: hah, I thought you would have done a forum search first 😉 | 11:29 |
| Chipaca | pstolowski: dunno if you noticed but we've been fighting the spread tests since at least yesterday | 11:29 |
| mborzecki | pstolowski: well, stuff is blocked by travis atm | 11:30 |
| pstolowski | Chipaca: yes i'm aware | 11:30 |
| Chipaca | pstolowski: almost everything in the first page is red | 11:30 |
| mborzecki | hopefully we can land #5994 soon | 11:30 |
| Chipaca | so, i mean, sure it'd be nice to land stuff | 11:30 |
| mup | PR #5994: tests/main/interfaces-accounts-service: more debugging <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5994> | 11:30 |
| zyga | mborzecki: it failed in my PR | 11:32 |
| mborzecki | zyga: hm, which one? | 11:32 |
| zyga | https://www.irccloud.com/pastebin/OuTgkBZz/ | 11:32 |
| zyga | https://github.com/snapcore/snapd/pull/5988 | 11:33 |
| mup | PR #5988: cmd: rename ns_group to mount_ns <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/5988> | 11:33 |
| mborzecki | zyga: is that goa using 100MB of mem/ | 11:33 |
| zyga | apparently | 11:34 |
| mborzecki | anyone willing to look at #5959 while we wait for PRs to be green again? | 11:37 |
| mup | PR #5959: systemd: extend Status() to work for socket and timer units <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5959> | 11:37 |
| === juergh_ is now known as juergh | ||
| zyga | mborzecki: done | 11:41 |
| mborzecki | zyga: thanks! | 11:41 |
| cachio | mvo, hi | 11:44 |
| cachio | sure | 11:44 |
| cachio | mvo, I'll make a run | 11:44 |
| zyga | mborzecki: I pushed https://github.com/snapcore/snapd/pull/5999 | 11:44 |
| mup | PR #5999: tests: ensure that goa-daemon is off <Created by zyga> <https://github.com/snapcore/snapd/pull/5999> | 11:44 |
| zyga | not sure if this is a red herring or real | 11:44 |
| cachio | mvo, should we promote 35.4 to stable? | 11:44 |
| mup | PR snapd#5999 opened: tests: ensure that goa-daemon is off <Created by zyga> <https://github.com/snapcore/snapd/pull/5999> | 11:45 |
| pstolowski | Chipaca: #5961 gtg | 11:47 |
| mup | PR #5961: snap/pack, snap/squashfs: use type to determine mksquashfs args <Created by chipaca> <https://github.com/snapcore/snapd/pull/5961> | 11:47 |
| mborzecki | zyga: i'm not sure it's a problem though, the original error is Error: GDBus.Error:org.freedesktop.DBus.Error.InvalidArgs: Type of message, '(ssssa{ss})', does not match expected type '(sssa{sv}a{ss})', so the expected type is correct and matches what busctl introspect shows | 11:48 |
| mborzecki | zyga: the mystery is why gdbus came up with (ssssa{ss}) | 11:49 |
| mborzecki | zyga: it's like it's treating '{}' as s instead ov a{..} | 11:50 |
| mborzecki | and only occasionally | 11:50 |
| mborzecki | off to pick up the kids | 11:51 |
| dot-tobias | Hi everyone | 11:52 |
| zyga | hey | 11:53 |
| sil2100 | mvo, zyga: another good news - we seem to have everything for enabling core18 dailies, so I'll do that once we spin out latest RC images for cosmic | 12:00 |
| sil2100 | As I don't want to eat up precious resources before we're done with those | 12:00 |
| zyga | sil2100: ok let me know if you need any more testing | 12:00 |
| mup | PR snapcraft#2355 closed: extensions: cleanup and generic tests <Created by kyrofa> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2355> | 12:10 |
| mup | PR snapd#6000 opened: snap,client: use a different exit code for retryable errors <Created by mvo5> <https://github.com/snapcore/snapd/pull/6000> | 12:12 |
| mvo | cachio: yeah, 2.35.4 can go to stable | 12:14 |
| Chipaca | mvo: do you need #5986 squashed? | 12:14 |
| mup | PR #5986: data/systemd, wrappers: tweak system-shutdown helper for core18 <Simple 😃> <⚠ Critical> <Created by chipaca> <https://github.com/snapcore/snapd/pull/5986> | 12:14 |
| mvo | sil2100: nice, thanks! | 12:15 |
| mvo | sil2100: did our 2.35.5 snapd upload from cosmisc-proposed made it? | 12:15 |
| Chipaca | also dunno what makes gdbus go ssssa{ss} instead of sssa{sv}a{ss} but it looks like it's over it for a bit | 12:15 |
| Chipaca | :-( | 12:15 |
| mvo | Chipaca: yes, please squash 5986 it will be needed for 2.36 | 12:16 |
| mvo | Chipaca: and thanks for this PR, nice job! | 12:16 |
| Chipaca | mvo: should i retarget it? (it's currently targetted to 2.35) | 12:16 |
| Chipaca | targeted* | 12:16 |
| mvo | Chipaca: yeah, I really really job we are done with 2.35 | 12:16 |
| Chipaca | :) | 12:16 |
| mvo | pedronis: I pushed a PR for the retryable errors, iirc field was askign for this in the context of auto-connect (just fyi) | 12:17 |
| dot-tobias | ogra: I'm trying to rotate the HDMI display connected to my Pi 3B running Core edge (edge because I need the mir-kiosk snap). setting pi-config.display-rotate=1 and then rebooting seems to bog the system, it does not boot up at all anymore. Removing the option from config.txt (SD inserted into the reader on my main machine) allows proper boot again. This does not occur on Core stable. Is this known or should I open a forum thread? Couldn't find | 12:18 |
| dot-tobias | anything there. | 12:18 |
| mup | PR snapd#5986 closed: data/systemd, wrappers: tweak system-shutdown helper for core18 <Simple 😃> <⚠ Critical> <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/5986> | 12:18 |
| mup | PR snapcraft#2356 opened: nodejs plugin: add support for bases <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2356> | 12:19 |
| Chipaca | pstolowski: thanks for the review! I'll add a comment to Alike in a followup (but note it's only there for tests) | 12:20 |
| pstolowski | Chipaca: k, yes, i know it's for test | 12:21 |
| pedronis | mvo: ah, I see, was confused for a second (too many problems with similar terms) | 12:21 |
| mup | PR snapd#5961 closed: snap/pack, snap/squashfs: use type to determine mksquashfs args <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/5961> | 12:21 |
| jdstrand | mvo: hey, do you want a 2.36 branch for PR 5989? | 12:29 |
| zyga | error: cannot communicate with server: Post http://localhost/v2/snaps: write unix @->/run/snapd.socket: write: broken pipe | 12:29 |
| mup | PR #5989: interfaces/system-key: add parser mtime and only discover features on write <Created by jdstrand> <https://github.com/snapcore/snapd/pull/5989> | 12:29 |
| zyga | man, my trivial PR is red for 40 hours over random stuff | 12:29 |
| jdstrand | zyga: hey, I know the feeling | 12:29 |
| zyga | yeah :| | 12:29 |
| jdstrand | "do I really want to fix that typo?" ;) | 12:30 |
| zyga | mvo: the graceful restart thing, is that a thing? | 12:30 |
| zyga | graceful restart breaking snap install (perhaps) https://www.irccloud.com/pastebin/6cblPVl4/ | 12:30 |
| zyga | jdstrand: "let's start with this simple PR" | 12:31 |
| jdstrand | zyga: "I'll just merge from trunk" | 12:32 |
| zyga | jdstrand: I actually tried to debug various failures | 12:32 |
| jdstrand | zyga: I see you are looking at the accounts-service ones. thanks! | 12:32 |
| zyga | but this is not the very best of Tuesdays | 12:32 |
| jdstrand | been seeing those a lot | 12:33 |
| mvo | jdstrand: yes please - 2.36 for 5989 | 12:35 |
| mvo | zyga: hm, the graceful stop breaks? or start? the log you pasted looks harmless? | 12:36 |
| mborzecki | re | 12:36 |
| zyga | snap install died on unix socket hangup | 12:36 |
| zyga | error: cannot communicate with server: Post http://localhost/v2/snaps: write unix @->/run/snapd.socket: write: broken pipe | 12:37 |
| mvo | zyga: oh? do you have more details? | 12:37 |
| mborzecki | zyga: got journactl of snapd? | 12:37 |
| zyga | doesn't look transparent | 12:37 |
| zyga | https://api.travis-ci.org/v3/job/441691823/log.txt | 12:37 |
| mvo | zyga: thanks, let me look | 12:37 |
| zyga | look for "error executing" please | 12:37 |
| zyga | save the log guys, I really want to land and iterate :/ | 12:37 |
| zyga | so I want to restart that job | 12:37 |
| zyga | just tell me when please | 12:37 |
| mvo | zyga: right, I see the error | 12:38 |
| zyga | mborzecki: ready? | 12:40 |
| mborzecki | zyga: yeah | 12:40 |
| zyga | restarting job | 12:40 |
| mborzecki | mvo: isn't that just snap (the client) polling and hitting /run/snapd.sock with some unfortunate timing? | 12:40 |
| mvo | mborzecki: the log looks odd, it looks like core got installed | 12:41 |
| mvo | mborzecki: and with a snap snapd should not go into stop mode | 12:41 |
| mvo | mborzecki: aha, no - just the core configureation, hm | 12:42 |
| mvo | mborzecki: maybe it is just unfortunate timing | 12:42 |
| mvo | mborzecki: which still sucks :/ | 12:42 |
| zyga | mvo: I'm more and more convinced we should do something with systemd to ensure we can shut down gracefully | 12:45 |
| zyga | (that is, pass the fd back and exit) | 12:45 |
| zyga | and then let systemd handle the grace period where we are going down | 12:45 |
| mvo | zyga: right, I'm not sure how to do that race-free yet, I am looking into a test to reproduce the current issue now | 12:48 |
| mborzecki | zyga: afaiu this is a remote end doing shutdown on a connection | 12:51 |
| pedronis | niemeyer: mvo: hi, reminder that I have a conflict with the standup | 12:56 |
| zyga | mvo: did you look at sd_pid_notify_with_fds ? | 12:57 |
| niemeyer | pedronis: Ack, thanks | 12:57 |
| mvo | zyga: I did not | 12:57 |
| mvo | zyga: thank you, I have a look now | 12:58 |
| mvo | zyga: aha, neat, that looks interessting | 12:59 |
| mborzecki | https://github.com/snapcore/snapd/pull/5994 is green, land it? | 13:01 |
| mup | PR #5994: tests/main/interfaces-accounts-service: more debugging <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/5994> | 13:01 |
| mup | PR snapcraft#2357 opened: scons plugin: add support for bases <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2357> | 13:05 |
| mup | PR snapd#6001 opened: interfaces: typo 'allows' for consistency with other ifaces <Created by zyga> <https://github.com/snapcore/snapd/pull/6001> | 13:18 |
| mup | PR snapd#5994 closed: tests/main/interfaces-accounts-service: more debugging <Created by bboozzoo> <Merged by zyga> <https://github.com/snapcore/snapd/pull/5994> | 13:29 |
| mup | PR snapd#5988 closed: cmd: rename ns_group to mount_ns <Simple 😃> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/5988> | 13:37 |
| mborzecki | https://paste.ubuntu.com/p/3D95h34pXs/ | 13:42 |
| mborzecki | heh, so i broke dbus/gdbus/goa https://paste.ubuntu.com/p/4bF95krMWf/ | 13:44 |
| mup | PR snapd#5995 closed: tests/main/interfaces-accounts-service: switch to manual while we debug the test <Created by bboozzoo> <Closed by bboozzoo> <https://github.com/snapcore/snapd/pull/5995> | 13:47 |
| mup | PR snapd#6002 opened: interfaces/system-key: add parser mtime and only discover features on write - 2.36 <Created by jdstrand> <https://github.com/snapcore/snapd/pull/6002> | 13:48 |
| Chipaca | mborzecki: the dbus snakes are attacking | 13:52 |
| pstolowski | zyga: might be worth re-running the tests on 5170, they were last run 19 days ago | 13:52 |
| zyga | pstolowski: great point | 13:52 |
| zyga | pstolowski: they will fail on accounts service ;-) | 13:53 |
| pstolowski | :) | 13:53 |
| mup | PR snapd#5998 closed: osutil: tweak handling of error adduser errors <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5998> | 13:55 |
| jdstrand | mvo: fyi, https://github.com/snapcore/snapd/pull/6002/ | 14:05 |
| mvo | jdstrand: thank you! | 14:05 |
| mup | PR #6002: interfaces/system-key: add parser mtime and only discover features on write - 2.36 <Created by jdstrand> <https://github.com/snapcore/snapd/pull/6002> | 14:05 |
| jdstrand | mvo: fyi, I'll let you decide on PR 5980, PR 5981 and PR 5982 for 2.36 (I'm not pushing for it-- might be nice to have more testing), but it would be nice if they were in trunk by SLC | 14:07 |
| mup | PR #5980: interfaces/apparmor: conditionally add explicit deny rules for ptrace <Created by jdstrand> <https://github.com/snapcore/snapd/pull/5980> | 14:07 |
| mup | PR #5981: interfaces/many: updates to support k8s worker nodes <Created by jdstrand> <https://github.com/snapcore/snapd/pull/5981> | 14:07 |
| mup | PR #5982: interfaces/many: conditionally use 'unsafe' with docker-support change_profile rules <Created by jdstrand> <https://github.com/snapcore/snapd/pull/5982> | 14:07 |
| jdstrand | mvo: that's just an fyi for what I'm thinking about. I know you're busy, so whatever you decide | 14:08 |
| sergiusens | mvo: cachio are are your travis pull requests just stalled? A quick check on your PR run tells me so, our are too 😢 | 14:08 |
| cachio | sergiusens, we have several on the queue | 14:11 |
| sergiusens | but none are running right? just waiting? | 14:11 |
| cachio | 2 running | 14:11 |
| cachio | the rest waiting | 14:12 |
| cachio | 3 running | 14:12 |
| sergiusens | ah, I see, closer to the bottom https://travis-ci.org/snapcore/snapd/pull_requests | 14:13 |
| cachio | :) | 14:14 |
| sergiusens | mvo: what is the ETA for core16 coming to stable? Not rushing, just getting my ducks aligned 🙂 | 14:26 |
| zyga | Chipaca, mvo: shall we land the .po update? | 14:26 |
| Chipaca | zyga: I'm not sure I can +1 it at this point :) | 14:28 |
| mvo | sergiusens: thats a sil2100 question now :) | 14:28 |
| mvo | sergiusens: foundations works on this | 14:28 |
| mvo | zyga: yes | 14:28 |
| mvo | Chipaca: no? why not? | 14:28 |
| Chipaca | zyga: mvo: but I +1 the ones mvo committed | 14:28 |
| Chipaca | mvo: 5 commits, 4 by me | 14:29 |
| mvo | Chipaca: haha | 14:29 |
| mvo | Chipaca: ok | 14:29 |
| Chipaca | there, I +1'ed it :) | 14:29 |
| mup | PR snapd#5935 closed: po: sync translations from launchpad <Created by mvo5> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/5935> | 14:31 |
| mvo | thanks zyga | 14:31 |
| mborzecki | hm i can reproduce the problem with dbus when goa-daemon is in the process of dying and a call is made, at least in that situation busctl delivers more meaningful error message | 14:31 |
| mborzecki | still, we do not kill goa-daemon in the test | 14:32 |
| mvo | mborzecki: nice | 14:32 |
| zyga | mborzecki: my PR has kills it | 14:35 |
| Chipaca | zyga: mvo: #5980 has two +1's (from you two), is green, but jdstrand said it was ready for re-review. Seems to me to be a quick one :) | 14:37 |
| mup | PR #5980: interfaces/apparmor: conditionally add explicit deny rules for ptrace <Created by jdstrand> <https://github.com/snapcore/snapd/pull/5980> | 14:37 |
| sil2100 | mvo, sergiusens: aaaaa, for core16 it's well, not foundations as far as I know | 14:38 |
| sil2100 | mvo, sergiusens: since I don't have upload access for core16 in the store or even snap recipe access (AFAIK) | 14:38 |
| mvo | sil2100: for core16 it is - for "core" (without the 16) we are still in charge | 14:39 |
| sergiusens | mvo feels like it might still be your problem if only to transition correctly 😅 | 14:39 |
| mvo | sil2100: sounds like we need to fix the access :) | 14:39 |
| Chipaca | I could use reviews on #5957 (it unblocks #5955) | 14:40 |
| mup | PR #5957: overlord/snapshotstate/backend: fall back on sudo when no runuser <Snapshots 📸> <Created by chipaca> <https://github.com/snapcore/snapd/pull/5957> | 14:40 |
| mup | PR #5955: cmd/snap, tests: snapshots for all <Snapshots 📸> <⛔ Blocked> <Created by chipaca> <https://github.com/snapcore/snapd/pull/5955> | 14:40 |
| mvo | sil2100: snap recipe is here https://github.com/snapcore/core16 | 14:40 |
| mvo | sil2100: and you should have store access now :) | 14:41 |
| sil2100 | mvo: thank you! | 14:41 |
| mvo | sil2100: let me know if I can help, sorry if my earlier reply to sergiusens sounded strange, I think for core16 we need the same mechanisms as for core18, i.e. a watcher that scans for package changes and that then triggers a core16 build plus some (auto) testing | 14:42 |
| sil2100 | mvo: sure! The script for core18 is ready btw., just waiting for some time to poke Steve about a nice comfy place for it to be on | 14:43 |
| sil2100 | Hopefully I won't have to create a new instance for it, my mojo mojo is weak | 14:44 |
| mvo | sil2100: cool | 14:44 |
| mvo | sil2100: yeah, I had the same problem with my cnf-extractor-charm - fortunately I was rescued by IS | 14:44 |
| mvo | sil2100: speaking of this, thats something I need to talk to steve about as well :) | 14:44 |
| mvo | sil2100: anyway, none of your concerns (most likely) | 14:45 |
| tomwardill | hello! A question, what happens if a snap pre-refresh hook fails? Can it abort the refresh and continue with the existing installed version? | 14:56 |
| Chipaca | tomwardill: yes | 14:59 |
| pstolowski | tomwardill: it should work that way yes, are you experiencing something else? | 14:59 |
| tomwardill | Chipaca, pstolowski: We've not tried yet, was just checking if it worked that way | 15:00 |
| tomwardill | is it just a case of exit(1) from the hook? | 15:00 |
| Chipaca | tomwardill: yep | 15:01 |
| tomwardill | excellent, thanks. | 15:02 |
| Chipaca | at least that's what the integration test uses :-) | 15:02 |
| sil2100 | sergiusens: anyway, we weren't looking at core16 recently yet | 15:03 |
| sil2100 | sergiusens: so I guess it | 15:03 |
| sil2100 | argh, I guess it | 15:03 |
| sil2100 | (keyboard errors) | 15:03 |
| sil2100 | I guess it'll still take a bit | 15:03 |
| sil2100 | Since you mentioned it now, I'm doing an image test build to see what's the current state | 15:05 |
| mup | PR snapd#6003 opened: cmd/snap: block 'snap help <cmd> --all' <Simple 😃> <Created by chipaca> <https://github.com/snapcore/snapd/pull/6003> | 15:14 |
| === chihchun is now known as chihchun_afk | ||
| Chipaca | hmm, hmm, snap refresh --offline | 16:09 |
| pedronis | Chipaca: what would that do ? | 16:13 |
| Chipaca | pedronis: remember apt-zip? | 16:13 |
| pedronis | no | 16:14 |
| Chipaca | https://wiki.debian.org/AptZip | 16:14 |
| pedronis | I see | 16:14 |
| Chipaca | pedronis: context is a person in the forum in 'the heart of africa' (says their profile), 3rd world with intermittent energy and connectivity, wanting to only talk to the store once and get everything for multiple devices | 16:18 |
| Chipaca | pedronis: I tagged you in one of the topics on this, there's a second (linked to that one) | 16:18 |
| Chipaca | anyway, i'm off for a bit | 16:19 |
| mup | PR snapd#6004 opened: cmd/snap-confine: reduce verbosity of debug and error messages <Created by zyga> <https://github.com/snapcore/snapd/pull/6004> | 16:20 |
| jdstrand | mvo: I think this is what sergiusens may have been reporting. it looks like my 5981, 5989 and 6002 PRs are 'stuck'. 'Details' show all green for 5981, shows spread at 6 minutes for 5989 (did that restart?) and show nothing started for 6002 | 16:20 |
| jdstrand | mvo: let me know if I should do something | 16:21 |
| zyga | dpkg-deb: error: <compress> from tar -cf subprocess was killed by signal (Illegal instruction), core dumped | 16:36 |
| zyga | that's a new error | 16:36 |
| === chihchun_afk is now known as chihchun | ||
| mup | PR snapd#5992 closed: release: merge 2.35.5 changes into the 2.36 branch <Created by mvo5> <Merged by zyga> <https://github.com/snapcore/snapd/pull/5992> | 17:11 |
| mvo | jdstrand: sorry for the slow reply - "stuck> not sure what we can do, I poke around a bit | 17:21 |
| zyga | hey mvo | 17:22 |
| zyga | back from sports? | 17:22 |
| mvo | zyga: yes | 17:22 |
| zyga | mvo: I wonder if I can and 6001 today | 17:22 |
| zyga | mvo: there's a deadlock in one of the tests: https://api.travis-ci.org/v3/job/442145572/log.txt | 17:24 |
| mvo | zyga: urgh | 17:24 |
| zyga | look for PC=0x5626a1bf1aa1 | 17:24 |
| mvo | zyga: hrm, hrm, its not even clear where it hangs | 17:26 |
| jdstrand | mvo: no worries. let me know if I need to mash any buttons or anything | 17:35 |
| zyga | mvo: if still here can I land 5999 | 18:01 |
| zyga | jdstrand: I know there are other priority things on your plate but gentle ping about the oldest open PR: https://github.com/snapcore/snapd/pull/5170 | 18:02 |
| mup | PR #5170: interfaces/builtin: add adb interface <Created by zyga> <https://github.com/snapcore/snapd/pull/5170> | 18:02 |
| mup | PR snapcraft#2357 closed: scons plugin: add support for bases <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2357> | 18:02 |
| jdstrand | yes. it seems like it, but it isn't forgotten | 18:02 |
| zyga | that's great, I'll try to review your PRs tomorrow | 18:02 |
| zyga | jdstrand: I sent a rewording PR for snap-confine but it's just about making the debug and error messages more consistent so I didn't request your review on anything | 18:04 |
| zyga | there's a 2nd PR about IPC tweak (upgrading eventfd to a full pipe) so that the helper can do more | 18:04 |
| zyga | I don't think this is security critical either but if you want I can request your review | 18:04 |
| zyga | though I'd rather not really since it's just the start and I have more after that and it would probably not land this week | 18:05 |
| jdstrand | zyga: are they all static messages or is there string handling in the debug/error messages that changed? | 18:05 |
| jdstrand | s/is there/are there/ | 18:05 |
| zyga | nothing new | 18:05 |
| zyga | just changing existing strings | 18:05 |
| zyga | no more string manipulation | 18:05 |
| zyga | (actually less, I remove some of the repeating arguments we keep saying over and over, like snap name) | 18:05 |
| zyga | In general I tried to make it more terse and more useful | 18:06 |
| mup | PR snapd#5980 closed: interfaces/apparmor: conditionally add explicit deny rules for ptrace <Created by jdstrand> <Merged by zyga> <https://github.com/snapcore/snapd/pull/5980> | 18:13 |
| zyga | jdstrand: can you please merge master into 5982 | 18:13 |
| zyga | mvo: FYI: another PR had a deadlock in cmd/snap, something to be wary of | 18:14 |
| === chihchun is now known as chihchun_afk | ||
| zyga | jdstrand: and 5981 please | 18:16 |
| zyga | ok, I gotta go | 18:16 |
| zyga | promised to watch Han Solo with my son :) | 18:16 |
| zyga | ttyl | 18:16 |
| jdstrand | zyga: nice, thanks! :) | 18:19 |
| jdstrand | zyga: enjoy :) | 18:19 |
| * jdstrand wonders why 5981 is showing changes to master that were merged in 5980 | 18:32 | |
| jdstrand | after merging master | 18:32 |
| jdstrand | oh, it idn't have it yet | 18:36 |
| jdstrand | weird | 18:36 |
| mup | PR snapcraft#2358 opened: dotnet plugin: add support for bases <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2358> | 18:44 |
| mvo | zyga: same package the deadlock? | 19:10 |
| mvo | zyga: might be worth running it with go test -count 1000 | 19:10 |
| Chipaca | zyga: deadlock? | 19:11 |
| Chipaca | ah | 19:11 |
| Chipaca | han solo wins | 19:11 |
| slvn_ | Hey, my 10 snaps games are published for a while. Only two of them appear in the listing of snapcraft.io | 20:25 |
| mup | PR snapd#6005 opened: snap/pack, snap/squashfs: remove extra copy before mksquashfs <Created by chipaca> <https://github.com/snapcore/snapd/pull/6005> | 22:35 |
| mup | PR snapd#6006 opened: tests: move spread-shellcheck to run-checks --static <Created by chipaca> <https://github.com/snapcore/snapd/pull/6006> | 23:10 |
| mup | PR snapcraft#2359 opened: [WIP] extensions: add glib <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/2359> | 23:27 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!