[06:14] <mborzecki> morning
[07:32] <zyga> good morning
[07:39] <mborzecki> zyga: hey
[07:39] <zyga> good morning :)
[07:39] <zyga> I'm staying away from the office till spring
[07:46] <liuxg1> hi, did anyone make the bluetooth working on ubuntu core, raspberry pi3b+ board? I have a problem in getting it working
[07:46] <zyga> I have that board but I've never tried using BT yet
[07:48] <liuxg1> zyga, thanks for your reply. https://bugs.launchpad.net/snappy/+bug/1674509 this seems the bug. I tried to power on it, however, after reboot, the setting is going.
[07:48] <mup> Bug #1674509: Unable to find bluetooth device on RPi3 running Ubuntu Core 16 <snapd-interface> <Snappy:Confirmed> <linux-raspi2 (Ubuntu):Invalid by p-pisati> <https://launchpad.net/bugs/1674509>
[07:49] <zyga> seems like that bug is well under way
[07:49] <liuxg1> I used the steps described in the link https://docs.ubuntu.com/core/en/stacks/bluetooth/bluez/docs/reference/pairing/introduction. I used to connect it with bluetooth long long time ago. I do not know why it was broken.
[07:50] <mborzecki> maybe something bluez related
[07:50] <liuxg1> zyga, are you able to pair a bluetooth device using your device? yes, it is related to the bluez.
[07:51] <zyga> as I said I have never tried
[07:51] <zyga> the device is on a shelf now, I'm powering them down to avoid cable mess and power drain when unused
[07:52] <mborzecki> liuxg1: can you run bluetootctl devices? is the paired device listed there?
[07:52] <liuxg1> I see. when you are free, would you please have a try if it is OK with you. I need to use the bluetooth to connect a device. thanks
[07:52] <liuxg1> mborzecki, there is nothing there.
[07:53] <mborzecki> liuxg1: and bluetootctl list?
[07:53] <liuxg1> I have to run this command: sudo /snap/bin/hciattach /dev/ttyAMA0 bcm43xx 921600 noflow - to see something there.
[07:54] <liuxg1> otherwise, there is no controller for it.
[07:58] <liuxg1> also, it seems that I have to install the bluez in --devmode to get it working.
[08:00] <mborzecki> intersting, looking at some rspi3 block diagrams, bt contoller is indeed wired over uart
[08:00] <mborzecki> liuxg1: anything related to apparmor in dmesg?
[08:01] <pstolowski> mornings
[08:02] <mborzecki> pstolowski: hey, while you're here got some questions about pre and post refresh hooks, figured you might know :)
[08:04] <mborzecki> pstolowski: pre refresh runs before the previous revision has been disabled, so the services will be up at this point?
[08:04] <liuxg1> mborzecki, these are the errors I got from dmesg. I am not sure whether they are related.  https://paste.ubuntu.com/p/kXhssPJnSH/
[08:06] <pstolowski> mborzecki: yes, pre-refresh runs before stopping the services; stop-snap-services is next
[08:07] <mborzecki> pstolowski: great, then post-refresh is run before the services are started, so I could snapctl stop --disable a service inside?
[08:11] <pstolowski> mborzecki: yes you could. we don't have spread tests for something like this though..
[08:13] <mborzecki> pstolowski: i'm asking bc ijohnson had some issues with services getting enabled after a snap refresh, and I suggested using {pre,post}-refresh hooks to grab the current state and restore it
[08:13] <mborzecki> pstolowski: relevant post: https://forum.snapcraft.io/t/how-to-manage-services-with-sockets-timers/7904/3
[08:14] <mborzecki> pstolowski: i think snapd cannot second guess whether a service name is true across revisions of a snap, so it falls down to snap creator to figure this out
[08:17] <pstolowski> mborzecki: i'll take a look at this in a bit and get back to you, need to prepare for the meeting i've soon
[08:17] <mborzecki> pstolowski: thanks
[08:29] <zyga> hey pawel
[08:39] <mborzecki> zyga: fontconfig fonts.conf parser https://paste.ubuntu.com/p/2KJDd6xdff/
[08:40] <mup> Bug #1811063 opened: "snap refresh" does not report failure to update because snap switched to classic confinement <Snappy:New> <https://launchpad.net/bugs/1811063>
[08:41] <mborzecki> zyga: wrote a little tool to dump my local config, no more guessing: dirs: &{SystemFonts:[/usr/share/fonts /usr/local/share/fonts] SystemCache:[/var/cache/fontconfig] UserFonts:[$XDG_USER_DATA_DIR/fonts $HOME/.fonts] UserCache:[$XDG_USER_DATA_DIR/fontconfig $HOME/.fontconfig]}
[08:43] <mup> Bug #1811063 changed: "snap refresh" does not report failure to update because snap switched to classic confinement <snapd:New> <Snappy:Invalid> <https://launchpad.net/bugs/1811063>
[08:54] <mup> PR snapd#6336 opened: snap-confine: fix incorrect use "src" var in mount-support.c <Created by mvo5> <https://github.com/snapcore/snapd/pull/6336>
[08:56] <pedronis> mvo: hi, I commented on #6331
[08:56] <mup> PR #6331: systemd: allow only a single daemon-reload at the same time <Created by mvo5> <https://github.com/snapcore/snapd/pull/6331>
[08:56] <mvo> pedronis: excellent, thank you!
[08:59] <Chipaca> o/
[09:00] <pedronis> Chipaca: hi
[09:01] <Chipaca> pedronis: 'sup
[09:02] <mvo> Chipaca: good morning!
[09:02] <pstolowski> pedronis: 1 sec, i'm coming, have little touble with google meet
[09:02] <mvo> Chipaca: oh is the released-at stuff coming along :) ?
[09:02] <Chipaca> mvo: good morning boss
[09:02] <Chipaca> :-)
[09:02] <mvo> Chipaca: *cough* I need to think about the right kind of title - maybe MASTER is appropriate?
[09:03] <pedronis> pstolowski: I added a meet to the event btw
[09:03] <mvo> Chipaca: I guess we need to discuss while having a beer in malta ;)
[09:03] <Chipaca> mvo: https://i.imgur.com/x846GwL.png
[09:03] <Chipaca> pedronis: also ^
[09:03] <Chipaca> that's me playing with some other stuff as well
[09:03] <mvo> Chipaca: \o/ so cool
[09:04] <Chipaca> because in the sample output  requesting this data, there's no up arrows
[09:04] <Chipaca> it's all compacted
[09:04] <pedronis> Chipaca: anyway I still have the feeling that we maybe should go with just the iso dates for this
[09:04] <Chipaca> but i don't know if that was just for didactic reasons or not
[09:04] <Chipaca> pedronis: yeah, i think so
[09:05] <Chipaca> pedronis: and use the human date or full iso date in verbose
[09:10] <mvo> pedronis: thanks for the review on the systemd workaround - I will expand the unit tests, sorry I should have mentioned this in the PR actually. I wanted to make sure we are happy with the general approach before spending too much time on this. I will do so today
[09:11] <zyga> mborzecki: nice
[09:14] <Chipaca> pedronis: there's a problem with released-at currently in the store, where it's not the right format
[09:14] <Chipaca> pedronis: easy to work around, but the fix is on master of the store already
[09:15] <Chipaca> pedronis: dunno if i should propose without the workaround and mark blocked, or not
[09:18] <mvo> Chipaca: when will the store release the fix? like roughly?
[09:18] <Chipaca> mvo: A fair question and one that in recent [days] 'as been much on my mind.
[09:19] <Chipaca> mvo: yesterday somebody was off sick, so it didn't happen
[09:19] <mvo> Chipaca: aha, but that sounds like "really-soon" (i.e. days not weeks?)
[09:20] <Chipaca> yeah
[09:20] <mvo> cool
[09:20] <Chipaca> yeah
[09:21] <Chipaca> right now I have the workaround because I like poking at things, but I think I'll drop it before proposing the PR
[09:22] <mborzecki> zyga: want to chat quickly about the fontconfig thing?
[09:22] <zyga> sure
[09:22] <zyga> here or HO?
[09:23] <mvo> Chipaca: we want this for 2.37 so if the store does not release the fix by tomorrow I think we should add the workaround. or 2.37 will be late for CPT :/
[09:23] <mborzecki> zyga: https://meet.google.com/koc-gdjf-qve
[09:23] <mvo> Chipaca: or we just drop it from 2.37 :)
[09:23] <mvo> Chipaca: but I really like the feature so that would be a bit sad
[09:26] <Chipaca> mvo: I'd rather add the workaround than have the feature slip
[09:27] <mvo> Chipaca: *nod*
[09:28] <mborzecki> zyga: let's try again https://meet.google.com/dwz-hyza-nwc?authuser=1
[09:37] <Chipaca> pedronis: mvo: https://forum.snapcraft.io/t/upcoming-change-to-the-output-of-snap-info/9368 fwiw
[09:37] <Chipaca> pedronis: mvo: or should I say https://forum.snapcraft.io/t/upcoming-change-to-the-output-of-snap-info/9368?u=chipaca
[09:37]  * Chipaca hoards the fake internet points
[09:38] <pedronis> Chipaca: was in a meeting with pawel
[09:38] <Chipaca> pedronis: poor pawel
[09:38]  * Chipaca hides
[09:38] <pedronis> Chipaca: heh
[09:39] <pedronis> Chipaca: I think we want the workaround, we can backport dropping it if that makes sense
[09:40] <Chipaca> I'm told the fix will land before EOD today
[09:40] <Chipaca> s/will/should/
[09:40] <pedronis> Chipaca: ok
[09:40] <Chipaca> but as i say, the workaround exists already :-)
[09:40] <Chipaca> anyway, fixing the tests and should have it up before standup
[09:40] <pedronis> yea
[09:41] <pedronis> Chipaca: do you have a PR for the other change?  (unicode off if no tty)
[09:41] <Chipaca> pedronis: not yet, do you want that one first?
[09:41] <pedronis> Chipaca: not first, but would like them to go out together somehow
[09:41] <pedronis> (disrupt things in one go)
[09:41] <Chipaca> :-)
[09:42] <Chipaca> I could stack 'em
[09:42] <Chipaca> but reviewers hate that :-)
[09:42] <mvo> Chipaca: \o/
[09:42] <Chipaca> mvo: does a 2.37 branch exist already?
[09:42] <mvo> Chipaca: not yet, will be created today
[09:42] <Chipaca> 'cause if so i could get the individual things on master, and stack them for 2.37
[09:43] <mvo> Chipaca: if all goes well :)
[09:43] <Chipaca> there's also the alignment change
[09:43] <Chipaca> bah
[09:43] <Chipaca> I could do this: add released-at in one pr, but use it in the same pr as the other two
[09:44] <Chipaca> seems messy conceptually tho
[10:47] <pedronis> does this rings any bells https://forum.snapcraft.io/t/installation-fails-as-core-snap-not-found/9367/3 ?
[11:01] <pedronis> mvo: do snaps that require core18 actually also need core ? but we install only the former automatically
[11:03] <pstolowski> mborzecki: re "i think snapd cannot second guess whether a service name is true across revisions of a snap, so it falls down to snap creator to figure this out": we know the service names of the old revsion, and services of the new revision, no?
[11:03] <mborzecki> pstolowski: we know the names, but we don't know whether A rev 1 is the same as A rev 2
[11:04] <mborzecki> pstolowski: all we know is that the names are the same
[11:04] <mvo> pedronis: snaps that require core18 do not need core, there was a bug a while ago though where it was confused about this but we fixed that, let me look at the git history
[11:04] <mborzecki> pstolowski: that's why i think just looking at name is fragile
[11:05] <pedronis> mvo: that bug is probably still in 18.04 I suppose because no SRU yet?
[11:05] <mvo> pedronis: that is possible, is this about the forum post you mentioned earlier?
[11:06] <pedronis> mvo: no, see PM but maybe also
[11:06] <pedronis> mvo: where do we mount snapctl from for those?  the host until there is a core snap available?
[11:06] <Chipaca> pedronis: fwiw, it's snap-confine
[11:06] <mvo> pedronis: aha, yes, I think this is the SRU :/
[11:06] <Chipaca> in 2.34.2
[11:06] <pstolowski> mborzecki: ah, because it's about knowing what to keep enabled/disabled?
[11:06] <mborzecki> pstolowski: yup
[11:06] <pedronis> Chipaca: yes, that's why me asking these questions
[11:06] <mvo> pedronis: yeah, the host (95% certain)
[11:07] <pedronis> Chipaca: does juju-db has base: core18 ?  or is there something else going on in the forum case
[11:08] <pedronis> I could check myself, but need to go have lunch
[11:08] <Chipaca> pedronis: I mean, the forum says it's core18
[11:08] <pedronis> ah
[11:08] <pedronis> yes
[11:08] <Chipaca> pedronis: and the snap does say base:core18
[11:08] <pedronis> yea, ok
[11:08] <pedronis> so it's fixed bug
[11:08] <pedronis> but no SRU
[11:08] <Chipaca> if they installed the core snap, they'd re-exec, and get the fix
[11:08] <mvo> yeah, I need to chase the SRU, let me do that now
[11:09] <pedronis> (also this will be saner when we have snapd snap on classic but also means more work)
[11:09] <pstolowski> mborzecki: i see, right. i agree with your assesment and the idea of using post-refresh hook
[11:12] <zyga> re
[11:22] <mup> PR snapd#6337 opened: store, snap, cmd/snap: channels have released-at <Created by chipaca> <https://github.com/snapcore/snapd/pull/6337>
[11:22] <Chipaca> pedronis, mvo: ^
[11:23] <mvo> Chipaca: cool!
[11:27] <Chipaca> whoops, forgot to undo a for-testing change i'd made
[11:27]  * Chipaca pushes
[11:42] <mup> PR snapd#6286 closed: release: support probing SELinux state <SELinux> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6286>
[11:58] <mup> PR snapd#6338 opened: cmd/snap: right-align revision and size in info's channel map <Created by chipaca> <https://github.com/snapcore/snapd/pull/6338>
[12:15] <pedronis> Chipaca: I moved the card about this to doing
[12:15] <Chipaca> pedronis: thank you
[12:27] <Chipaca> sergiusens: does snapcraft have a homebrew plugin? 1.9.0 supports Linux, and it's been out for _hours_ already!
[12:27] <sergiusens> Chipaca: a homebrew plugin to build from brews and make them snaps?
[12:27] <Chipaca> sergiusens: clearly
[12:28] <sergiusens> meta meta meta
[12:28] <Chipaca> :-)
[12:28] <sergiusens> I guess with overlays it could work
[12:28] <sergiusens> probably worth looking into when more man power becomes available 🙂
[12:28] <Chipaca> also 1.9.0 drops support for intel 32 bits
[12:28] <mup> PR snapd#6336 closed: snap-confine: fix incorrect use "src" var in mount-support.c <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6336>
[12:28] <Chipaca> dunno if that affects zyga's lab
[12:29] <zyga> Chipaca: not much
[12:29] <zyga> Chipaca: you mean go 1.9 is x86_64 only?
[12:29] <zyga> Chipaca: I have one 32bit (strict) machine left
[12:29] <zyga> as in i386
[12:29] <Chipaca> zyga: "Homebrew 1.9.0 no longer runs on 32-bit Intel CPUs."
[12:29] <zyga> ah
[12:29] <zyga> nah that's ok
[12:29] <zyga> that entry point is not using macos :)
[12:29] <Chipaca> from https://brew.sh/2019/01/09/homebrew-1.9.0/
[12:30] <zyga> there are no 32bit only macs left really
[12:30] <zyga> even the watch is 64 bit
[12:30] <Chipaca> it isn't 4 bits emulating 8?
[12:31] <zyga> :D ?
[12:31] <zyga> quick coffee break
[12:31] <zyga> btw, working from the kitchen has one clear advantage :)
[12:31] <Chipaca> nice tweak to travis' ui there
[12:31] <Chipaca> zyga: just keep your jitters under half of your keycap size and you'll be fine
[12:47] <mup> PR snapd#6339 opened: cmd/snap: don't auto-enable unicode to a tty <Created by chipaca> <https://github.com/snapcore/snapd/pull/6339>
[12:48] <Chipaca> mvo: three PRs up, that should be it for 2.37 from me :-)
[12:52] <pedronis> Chipaca: you mean non-tty there?
[12:52] <Chipaca> gah
[12:52]  * Chipaca checks
[12:52] <Chipaca> gah²
[12:52] <Chipaca> I had it right the first time :-)
[12:52] <Chipaca> and … "fixed" it
[12:53] <Chipaca> pedronis: where it says don't, read only
[12:53] <Chipaca> :-)
[12:53] <Chipaca> i'll fix it right back
[13:01] <mborzecki> off to pick up the kids
[13:01] <cachio> niemeyer, hello, a new change pushed for the spread reboot PR
[13:08] <cachio> pstolowski|lunch, hey, if you have time could you please take a look to #5887 ?
[13:08] <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:13] <mup> PR snapd#6337 closed: store, snap, cmd/snap: channels have released-at <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/6337>
[13:20] <niemeyer> cachio: Thanks!
[13:34] <pedronis> Chipaca: the description seems also wrong
[13:35] <pedronis> I mean the PR description
[13:35] <Chipaca> pedronis: yes
[13:40] <Chipaca> https://www.dell.com/en-uk/work/shop/cty/pdp/spd/precision-17-7720-laptop/xctop7720emea
[13:41] <Chipaca> guess what laptop won't be my next laptop
[13:48] <zyga> ouch
[13:48] <zyga> Starting at £7,001,726.28
[13:48] <zyga> Chipaca: but hey, it's a "bargin"
[13:48] <zyga> Original Price
[13:48] <zyga> £10,002,466.11
[13:49] <ogra> oh, is dell already establishing the post-brexit pricing ?
[13:49] <mborzecki> Intel Core i5-6300HQ with Thunderbolt 3 is super expensive
[13:49] <mborzecki> must be that thunderbolt 3
[13:50] <ogra> "delivery by row-boat to the continent ..."
[13:50] <ogra> .. "included"
[13:53] <Chipaca> mborzecki: the HQ stands for high quality, it's the audiophile edition
[13:53] <mborzecki> intelophile
[13:56] <zyga> mborzecki: wait, is that really i5-6300HQ? :)
[13:56] <zyga> oh my
[13:56] <zyga> that's garbage :D
[13:56] <zyga> (for that price)
[13:56] <pedronis> Chipaca: could you push master into those PRs to reduce the diff?
[13:57] <Chipaca> pedronis: I could, but that'd kick off another travis run :-)
[13:58] <Chipaca> pedronis: should I?
[13:59] <pedronis> not immediately
[13:59] <Chipaca> pedronis: it's two commits, one of which is on master :-)
[14:02] <mborzecki> zyga: standup
[14:02] <zyga> joining!
[14:35]  * pedronis break
[14:48]  * zyga goes for lunch
[14:51] <pedronis> mvo: can we answer something here https://forum.snapcraft.io/t/installation-fails-as-core-snap-not-found/9367/3 ?
[14:52] <mup> PR snapd#6340 opened: store: undo workaround for timezone-less released-at <Created by chipaca> <https://github.com/snapcore/snapd/pull/6340>
[15:03] <mvo> pedronis: yes
[15:22] <Chipaca> SIGH
[15:22]  * Chipaca fixes things
[15:25] <zyga> Chipaca: found something fun?
[15:25] <Chipaca> spread tests looking for unicode output in things that no longer produce unicode output :-)
[15:25] <Chipaca> fun level 3.72
[15:44] <zyga> quick coffee
[15:45]  * genii 's ears perk up momentarily at the mention of coffee
[15:46] <zyga> the coffee machine is warming up
[15:56] <pedronis> Chipaca: reviewed #6338
[15:56] <mup> PR #6338: cmd/snap: right-align revision and size in info's channel map <Created by chipaca> <https://github.com/snapcore/snapd/pull/6338>
[15:57] <Chipaca> pedronis: thanks
[15:57] <Chipaca> pedronis: info is a lovely mess
[15:57] <Chipaca> pedronis: I'll see if I can't clean it up a bit after epochs
[15:57] <Chipaca> meanwhile, probably not worth it for this change
[15:59] <pedronis> Chipaca: as I said, mostly wondering, because of the passing things in and out, and also formatting concerns are all a bit mixed up now
[15:59]  * cachio lunch
[16:22] <pedronis> mvo: will you write there?  you are probably the most familiar with the state, also afaik you did the fix :)
[16:22] <mvo> pedronis: yes, I will write in the forum, sorry for hte delay, just finished the unit tests for protocol-error PR (6331)
[16:22] <mvo> pedronis: feedback welcome (as always :)
[16:23]  * mvo goes and writes about " Installation fails as ‘core snap’ not found "
[16:28]  * zyga goes to read that 
[16:28] <zyga> namespaces and more namespaces
[16:35] <pedronis> mvo: thank you
[16:35] <pedronis> mvo: need to go for a walk, but will do more 2.37 reviews afterward
[16:36] <mvo> pedronis: \o/
[16:36] <mvo> pedronis: I will also make a cup of tea and look at the next steps
[16:40] <pedronis> jdstrand: hi, I finally got to look and comment on #5644 ,there are some questions there
[16:40] <mup> PR #5644: interfaces: add audio-playback/audio-record and make pulseaudio manually connect <Created by jdstrand> <https://github.com/snapcore/snapd/pull/5644>
[16:41] <pedronis> jdstrand: also, could you look at #6334 ?
[16:41] <mup> PR #6334: dirs, interfaces/builtin/desktop: system fontconfig cache path is different on Fedora <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6334>
[16:44] <jdstrand> pedronis: yes, I saw the emails for both
[17:07] <zyga> brb
[17:23] <zyga> back to work :)
[17:56] <zyga> tests looking good, writing some commit message for the PR
[18:37] <zyga> some spread IO issues
[18:37] <zyga> but restarted now
[18:49]  * mvo hugs pedronis for his tireless reviews
[18:54] <pedronis> I think I looked at most 2.37 things, they need 2nd reviews and/or passing tests
[19:32] <mup> PR snapd#6340 closed: store: undo workaround for timezone-less released-at <Created by chipaca> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6340>
[19:38] <mup> PR snapd#6341 opened: cmd/snap-update-ns: persist per-user mount namespace profile <Created by zyga> <https://github.com/snapcore/snapd/pull/6341>
[20:01]  * zyga EODs
[20:08] <mup> PR snapcraft#2431 closed: rust plugin: update to use latest rustup command options <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2431>
[21:12] <mup> PR snapcraft#2434 opened: rust plugin: refactor to use the latest rustup <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2434>