[05:01] <zyga> good morning :)
[06:20] <mborzecki> morning
[06:20] <mvo> good morning mborzecki
[06:23] <mborzecki> mvo: so crng init struck core image now?
[06:23] <mvo> mborzecki: it seems to have done so yesterday - not so sure about today, there was a new kernel update
[06:23] <mvo> mborzecki: so maybe that fixed things I'm just looking
[06:28] <mvo> mborzecki: hm, the diff is gigantic, no idea
[06:29] <mborzecki> mvo: kernel diff?
[06:30] <mborzecki> mvo: do you have a link?
[06:31] <mvo> mborzecki: sure: https://launchpad.net/~canonical-kernel-team/+archive/ubuntu/ppa/+packages?field.name_filter=&field.status_filter=published&field.series_filter=bionic
[06:32] <mvo> mborzecki: the first one is linux-4.15…
[06:49] <mup> PR snapd#6382 closed: tests: use pc-kernel from stable instead of edge to build core-16 and core-18 <Created by sergiocazzolato> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/6382>
[06:52] <mup> PR snapd#6370 closed: interfaces/builtin/opengl: allow access to NVIDIA VDPAU library <Created by cgutman> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6370>
[06:53] <mup> PR snapd#6362 closed: cmd/snap-update-ns: explicitly check for return value from parse_arg_u <Simple 😃> <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6362>
[06:58] <mvo> degville: could you please do a final look/approval on pr#6369 (the one about PATH)? feel free to merge if you are happy
[07:18] <zyga> Hey
[07:26] <mvo> hey zyga
[07:42] <mborzecki> zyga: hey
[07:47] <mup> PR snapd#6372 closed: tests: define new "tests/smoke" suite and use that for autopkgtests <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6372>
[08:00] <mborzecki> mvo: PRs are turning green ;)
[08:00] <mup> PR snapd#6373 closed: overlord/ifacestate: helper API to obtain the state of connections <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6373>
[08:01] <mborzecki> mvo: wdyt about this one? https://github.com/snapcore/snapd/pull/6383
[08:01] <mup> PR #6383: tests: provide a fake random device to the core images <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6383>
[08:01] <pstolowski> mornings
[08:01] <mborzecki> pstolowski: hey
[08:05] <mvo> mborzecki: I was thinking about it this morning
[08:05] <mvo> hey pstolowski
[08:05] <mvo> mborzecki: I think its ok, we do the same on classic afterall
[08:05] <mborzecki> mvo: yes, we do
[08:05] <mvo> mborzecki: but it won't help with the entropy misisng on boot
[08:06] <mvo> mborzecki: so I think its ok to have it but we shouldn't expect much from it except that it speeds up key generation when we do gpg tests (which iirc we skip on core anyway because we have no gnupg snap)
[08:07] <mvo> mborzecki: wdyt?
[08:07] <mborzecki> mvo: yeah, gppg and maybe ssh would benefit
[08:07] <mvo> mborzecki: aha, ssh indeed
[08:07] <mvo> mborzecki: that might be a good reason, we generate ssh host keys on first boot
[08:08] <zyga> re
[08:09] <mvo> welcome back zyga
[08:11] <mborzecki> mvo: i'm looking at the prepare code, maybe w could generate the host keys while preparing the image and just copy them over to system-data/etc/ssh/
[08:11] <mborzecki> this would speed up the first boot
[08:11] <zyga> hey, how are things on your side guys?
[08:11] <zyga> I'm looking at debian package update now
[08:11] <mvo> mborzecki: yeah, that would work
[08:11] <degville> mvo: will do!
[08:11] <mvo> mborzecki: we should measure first though
[08:12] <mvo> mborzecki: I mean, it might be just so fast its not worth the work on our side :)
[08:22] <mup> PR snapd#6375 closed: tests: fix enable-disable-unit-gpio test on external boards <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6375>
[08:33] <mup> PR snapd#6326 closed: interface: raw-usb: Adding ttyACM ttyACA permissions <Created by kubiko> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6326>
[08:56] <klebers> mvo, hi, we have refreshed the pc-kernel snaps to add some fixes to the image. smb has more details about it, he's working on that
[08:57] <mvo> klebers: thanks for getting back to me. we had trouble with the previous kernel, it failed to boot in most tests but the most recent one is fine again.
[08:58] <klebers> mvo, we are planning to update soon the version on 18/stable, would that affect someone?
[08:59] <mvo> klebers: we don't have that many 18 users yet, but let me check
[09:10] <mvo> hey Chipaca ! good morning
[09:11] <Chipaca> Iä! Iä! mvo fhtagn!
[09:11] <mborzecki> haha
[09:11] <ogra> was that finnish ?
[09:12] <Chipaca> ogra: the long form is «Iä! Iä! mvo fhtagn! Ph'nglui mglw'nfah mvo R'lyeh wgah'nagl fhtagn!» if that gives you any more clues
[09:12] <mborzecki> cthulu language, rly'something
[09:13] <ogra> oh, chtulu !
[09:14]  * Chipaca practices for the post-brexit world
[09:14] <ogra> heh
[09:14] <Gargoyle> Chipaca: I think you were offline by the time I got back last night. My Atom looks like this :-( https://i.imgur.com/YVeUqls.png
[09:15] <Chipaca> Gargoyle: looks like a problem in the theme to me
[09:16] <Chipaca> Gargoyle: is that plain Ubuntu?
[09:17] <Gargoyle> Yup. 18.10 with Yaru theme
[09:17] <Gargoyle> I have been playing with builder a few months ago - wonder if some dev libraries are messing things up?
[09:18] <Chipaca> Gargoyle: is there a bug or forum post talking about the issue?
[09:19] <Gargoyle> Not that I have found. There is one relating to file open/save dialogs which are all messed up too.
[09:19] <mborzecki> Chipaca: isn't that some ubuntu specific gtk extension that moved the menu bar somewhere to the top bar under unity?
[09:20] <Chipaca> mborzecki: in 18.10? not that I'm aware
[09:21] <Chipaca> Gargoyle: so when you asked "is this issue fixed yet" you were assuming we had some kind of telepathic power? :-)
[09:21] <Gargoyle> OK. You are correct Chipaca. Switched "Applications" theme to Adwaita and atom's menu and file dialogs look normal again.
[09:22] <Gargoyle> Chipaca: It's related to the filesystem one
[09:22] <Gargoyle> *filesystem dialogs one
[09:22] <Chipaca> Gargoyle: how so?
[09:22] <mvo> Chipaca: haha - I read a lot of lovecraft but forgot this one
[09:24] <Chipaca> Gargoyle: in any case I can confirm in 18.10 with the default theme in atom the menu line is black-on-black or something
[09:24] <Chipaca> also it takes almost a minute to start the first time
[09:25] <Chipaca> (but this might be the snapd version)
[09:25] <Chipaca> s/almost/what feels like/
[09:26] <Gargoyle> It's something to do with the GTK themes and the switch to 18.10 and Yaru. I asked a while ago and someone like popey confirmed there was an ongoing issue. Didn't get a link to a forum post though.
[09:29] <Gargoyle> I can't find one now ether, so I'll grab some screenshots / video and post one up this morning.
[09:34] <Chipaca> Gargoyle: ta
[10:15] <mup> PR snapd#6369 closed: Add check for snap binaries dir not being in path <Simple 😃> <Squash-merge> <Created by liamg> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/6369>
[10:26] <mup> PR snapd#6384 opened: snapd: fix race in TestSanityFailGoesIntoDegradedMode test <Created by mvo5> <https://github.com/snapcore/snapd/pull/6384>
[10:36] <mborzecki> mvo: left a comment ^^
[10:40] <mvo> mborzecki: \o/ for that
[10:40] <mvo> mborzecki: excellent catch
[11:02] <mup> PR snapd#6385 opened: debian: fix silly typo in the spread test invocation <Created by mvo5> <https://github.com/snapcore/snapd/pull/6385>
[11:26] <Gargoyle> Chipaca: Found the underlying issue - it's electron and not specific to the snap. (https://github.com/electron/electron/issues/15194)
[11:52] <Gargoyle> dum di dum.... Electron api demo app works fine. Building atom from source...
[11:59] <mup> PR snapcraft#2437 closed: repo,baseplugin: support trusting repo keys <Created by kyrofa> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2437>
[12:09] <pstolowski> mborzecki: hey, can you take a look at https://github.com/snapcore/snapd/pull/6379 ?
[12:09] <mup> PR #6379: ifacestate/tests: extra test for hotplug-connect handler <Hotplug 🔌> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6379>
[12:09] <mup> PR snapd#6386 opened: tests: skip lp-1802591 on "official" images <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6386>
[12:10] <pstolowski> (the test you suggested in previous hotplug PR)
[12:10] <mborzecki> pstolowski: ack
[12:10] <pstolowski> thanks
[12:13] <greyback> hi all, can anyone interpret why spread failed for me: https://travis-ci.org/snapcore/snapd/builds/479834032 ?
[12:17] <mborzecki> Chipaca: i've updated #6333
[12:17] <mup> PR #6333: daemon: introduce /v2/connections snapd API endpoint <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6333>
[12:18] <mborzecki> greyback: sorry the link doesn't seem to work, which PR is this?
[12:18] <mborzecki> greyback: 6361?
[12:18] <greyback> mborzecki:  https://github.com/snapcore/snapd/pull/6361
[12:18] <mup> PR #6361: kvm: load required kernel modules if necessary <Created by gerboland> <https://github.com/snapcore/snapd/pull/6361>
[12:18] <greyback> yep
[12:21] <mborzecki> greyback: hmm, the link to travis job from the PR doesn't work either, wth?
[12:21] <zyga> mvo: hey, can we get the 2.37 release tarballs on the release page please
[12:22] <greyback> mborzecki: hmm, I just tried it now, broken here too ("We couldn't find repo snapcore/snapd") Worked ~10 mins ago
[12:23] <mborzecki> travis induced coffee break :)
[12:23] <greyback> :D the new https://www.xkcd.com/303/
[12:26] <zyga> mvo: I'm working on the 2.37 debian update
[12:26] <zyga> finally got over the hurdles there
[12:26] <zyga> (in the early prep of how-do-I-even-start)
[12:33] <mborzecki> greyback: i've restarted the spread job
[12:47] <greyback> mborzecki: thank you
[12:51] <zyga> or I guess I can do that too
[12:52] <Chipaca> mborzecki: is connectionJSON used outside of api_connections/
[12:52] <Chipaca> ?
[12:52] <Chipaca> mborzecki: in the whole picture i mean
[12:53] <mborzecki> Chipaca: yes, there's corresponding struct in the client package
[12:53] <mborzecki> planned to push the client bits to #6016
[12:53] <mup> PR #6016: [RFC] move various name validation helpers to snap/name package <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6016>
[12:53] <mborzecki> uh, #6079
[12:54] <mup> PR #6079: [RFC] `snap connections` command <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6079>
[12:59]  * pstolowski lunch
[13:03] <Chipaca> mborzecki: is there a reason to have the structs twice? I've been trying to avoid that unless there's a reason
[13:03] <Chipaca> bah, maybe it's "it still uses the old json stuff that's in daemon and i didn't want to refactor", that's a reason for example :-)
[13:16] <zyga> cachio: hey, what's the state of 2.37
[13:16] <zyga> I'm trying to determine if the debian update should go with 2.36.x or with 2.37
[13:16] <zyga> mvo: ^ CC please comment as well if you can
[13:16] <mvo> zyga: 2.37
[13:17] <zyga> mvo: hey :)
[13:17] <mvo> zyga: I'm working on it right now and we should (fingers crossed) have a 2.37 final today or tomorrow
[13:17] <zyga> I tried to get the tarballs and I was in a busy meeting
[13:17] <mvo> zyga: I really really want it today
[13:17] <zyga> is there a .0 release yet?
[13:17] <zyga> and you will go with .1
[13:17] <zyga> or is the work now focused on 2.37.0
[13:17] <mvo> zyga: its not out just yet, we are at 2.37~rc1
[13:17] <mvo> zyga: two more PRs
[13:17] <zyga> ok
[13:17] <mvo> zyga: one race fix, one autopkgtest fix
[13:18] <zyga> I'll continue to assess the debian side, we should have some actions there
[13:18] <zyga> for one
[13:18] <mvo> zyga: and  thanks for working on the update
[13:18] <zyga> I would like to go through all the build deps
[13:18] <mvo> zyga: \o/
[13:18] <zyga> (which are really runtime deps)
[13:18] <cachio> zyga, hey
[13:18] <zyga> and then check if we are comfortable with them being as old as they are
[13:18] <zyga> hey cachio, how are you doing?
[13:18] <mvo> zyga: there is one potential PR I might pull in (6378)
[13:18] <cachio> zyga, initial validation for pre~1 is completed
[13:18] <zyga> mvo: and if no, actually go to salsa, work on updates (despite most likely missing this freeze)
[13:19] <cachio> zyga, mvo is gonna generate a new release soon
[13:19] <zyga> mvo: and I would honestly put this on us each release
[13:19] <zyga> mvo: where if we pay the painful price now once
[13:19] <zyga> mvo: we should be in a good place for each release (it should be cheap)
[13:19] <cachio> mvo, next friday 25 is my last day before vacations
[13:20] <cachio> mvo, I mentioned that in the standup but I think you were already on vacations
[13:20] <cachio> it was on december
[13:21] <mvo> cachio: ok
[13:21] <mvo> cachio: should be enough time for the validation
[13:21] <mvo> zyga: yeah, maybe we need to do it
[13:21] <mvo> zyga: not great but better than the alternatives it seems
[13:21] <cachio> mvo, I hope so
[13:34] <mup> PR snapd#6387 opened: client: introduce helper for querying snapd API for the list of slot/plug connections <⛔ Blocked> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6387>
[13:34] <mborzecki> Chipaca: opened the client bits ^^
[13:35]  * Chipaca shakes fist at the internet at large, and goes for coffee
[13:36] <diddledan> ☕
[13:40] <cachio> pstolowski, hey
[13:41] <cachio> pstolowski, when you have time, could you please take a look to this one? #5887
[13:41] <mup> PR #5887: tests: moving core-snap-refresh-on-core test from main to nested suite <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/5887>
[13:41] <cachio> pstolowski, it is related to nested suite
[13:44] <pstolowski> cachio: will do
[13:44] <Gargoyle> How do we find out who is currently doing the packaging for Atom if it currently just says "snapcrafters" on the store page?
[13:44] <cachio> pstolowski, thanks
[13:45] <diddledan> Gargoyle: that's the snapcrafters, which means anyone on the snapcrafters org lead by popey and wimpress
[13:49] <mup> PR snapd#6385 closed: debian: fix silly typo in the spread test invocation <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6385>
[13:49] <Gargoyle> diddledan: OK. I've got a "working" build of atom. I say "working" because the menu and file dialogs are rendering properly but atom's own test suite fails horribly. But I can't get my head rount the branching on atom's github page! :/
[13:50] <mup> PR snapd#6359 closed: tests: fix listing tests to match "snap list --unicode=never" <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6359>
[13:51] <mup> PR snapd#6386 closed: tests: skip lp-1802591 on "official" images <Created by sergiocazzolato> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6386>
[13:52] <mvo> 6384 needs a second review
[13:56] <mup> PR snapcraft#2440 opened: meta: make hooks executable instead of complaining they're not <Created by Saviq> <https://github.com/snapcore/snapcraft/pull/2440>
[14:18] <diddledan> Gargoyle: the snap of atom is created from https://github.com/snapcrafters/atom
[14:36] <pstolowski> cachio: btw 5887 has conflicts
[14:50] <cachio> pstolowski, ohh, sorry
[14:52] <cachio> pstolowski, fixed
[14:52] <cachio> thanks
[14:57] <mborzecki> mvo: i think we could land #6368 as it is now, even with the typo, given the spread job on master has failed because of this problem
[14:57] <mup> PR #6368: tests: fix daemon-notify test checking denials considering all the log lines <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6368>
[14:58] <mvo> mborzecki: if you are happy with it: +1
[14:59] <mup> PR snapd#6368 closed: tests: fix daemon-notify test checking denials considering all the log lines <Created by sergiocazzolato> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6368>
[15:00] <mvo> mborzecki: thank you!
[15:17] <mup> PR snapd#6378 closed:  cmd: fix snap-device-helper to deal correctly with hooks  <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6378>
[15:53] <mvo> 6384 needs a second review
[15:55] <Chipaca> mvo: lgtm
[15:55] <mvo> ta!
[15:56] <mup> PR snapd#6384 closed: snapd: fix race in TestSanityFailGoesIntoDegradedMode test <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6384>
[16:19] <Gargoyle> Seems I'm trying all the broken snaps today! :/
[16:19]  * ogra hugs Saviq for https://bugs.launchpad.net/bugs/1812003
[16:19] <mup> Bug #1812003: hooks should be made executable instead of erroring out <Snapcraft:New> <https://launchpad.net/bugs/1812003>
[16:20] <Gargoyle> Jenkins can't use https (probably any SSL) due to a trust store error.
[16:20] <mup> PR snapd#6388 opened: tests: fix install-snaps test by changing the snap info regex <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6388>
[16:20] <Gargoyle> Does it need access to etc/ssl ?
[16:23]  * cachio lunch
[16:33]  * zyga resumes work on debian packaging
[16:35] <zyga> eh, quilt
[16:35] <zyga> how I love the
[16:35] <zyga> (not)
[16:35] <zyga> thee
[16:45] <Gargoyle> created report here:- https://forum.snapcraft.io/t/jenkins-cannot-access-update-server-via-https/9499
[16:51] <mup> PR snapd#6389 opened: cmd/snap: small refactor of cmd_info's channel handling <Created by chipaca> <https://github.com/snapcore/snapd/pull/6389>
[17:07] <mup> PR snapd#5887 closed: tests: moving core-snap-refresh-on-core test from main to nested suite <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/5887>
[18:01] <Chipaca> ok, EOD from me
[18:01] <Chipaca> ttfn
[18:48] <mup> PR snapd#6390 opened: release: 2.37 <Created by mvo5> <https://github.com/snapcore/snapd/pull/6390>
[19:23] <mvo> cachio: all arches have 2.37 in beta now, just fyi
[19:35] <cachio> mvo, great
[19:36] <cachio> starting right now
[19:47] <mvo> cachio: \o/
[19:52] <mup> PR snapd#6390 closed: release: 2.37 <Created by mvo5> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6390>
[21:08] <mup> PR snapd#6391 opened: tests: simplify interfaces-contacts-service test <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6391>