=== Caelum is now known as fishies | ||
=== ^arcade_droid is now known as zarcade_droid | ||
=== chihchun is now known as chihchun_afk | ||
mborzecki | morning | 06:14 |
---|---|---|
zyga | good morning | 07:32 |
mborzecki | zyga: hey | 07:39 |
zyga | good morning :) | 07:39 |
zyga | I'm staying away from the office till spring | 07:39 |
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:46 |
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:48 |
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:49 |
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:50 |
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:51 |
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:52 |
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:53 |
liuxg1 | otherwise, there is no controller for it. | 07:54 |
liuxg1 | also, it seems that I have to install the bluez in --devmode to get it working. | 07:58 |
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:00 |
=== chihchun_afk is now known as chihchun | ||
=== pstolowski|afk is now known as pstolowski | ||
pstolowski | mornings | 08:01 |
mborzecki | pstolowski: hey, while you're here got some questions about pre and post refresh hooks, figured you might know :) | 08:02 |
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:04 |
pstolowski | mborzecki: yes, pre-refresh runs before stopping the services; stop-snap-services is next | 08:06 |
mborzecki | pstolowski: great, then post-refresh is run before the services are started, so I could snapctl stop --disable a service inside? | 08:07 |
pstolowski | mborzecki: yes you could. we don't have spread tests for something like this though.. | 08:11 |
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:13 |
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:14 |
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:17 |
zyga | hey pawel | 08:29 |
mborzecki | zyga: fontconfig fonts.conf parser https://paste.ubuntu.com/p/2KJDd6xdff/ | 08:39 |
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:40 |
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:41 |
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:43 |
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:54 |
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:56 |
Chipaca | o/ | 08:59 |
pedronis | Chipaca: hi | 09:00 |
Chipaca | pedronis: 'sup | 09:01 |
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:02 |
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:03 |
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:04 |
Chipaca | pedronis: and use the human date or full iso date in verbose | 09:05 |
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:10 |
zyga | mborzecki: nice | 09:11 |
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:14 |
Chipaca | pedronis: dunno if i should propose without the workaround and mark blocked, or not | 09:15 |
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:18 |
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:19 |
Chipaca | yeah | 09:20 |
mvo | cool | 09:20 |
Chipaca | yeah | 09:20 |
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:21 |
mborzecki | zyga: want to chat quickly about the fontconfig thing? | 09:22 |
zyga | sure | 09:22 |
zyga | here or HO? | 09:22 |
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:23 |
Chipaca | mvo: I'd rather add the workaround than have the feature slip | 09:26 |
mvo | Chipaca: *nod* | 09:27 |
mborzecki | zyga: let's try again https://meet.google.com/dwz-hyza-nwc?authuser=1 | 09:28 |
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:37 | |
pedronis | Chipaca: was in a meeting with pawel | 09:38 |
Chipaca | pedronis: poor pawel | 09:38 |
* Chipaca hides | 09:38 | |
pedronis | Chipaca: heh | 09:38 |
pedronis | Chipaca: I think we want the workaround, we can backport dropping it if that makes sense | 09:39 |
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:40 |
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:41 |
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:42 |
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:43 |
Chipaca | seems messy conceptually tho | 09:44 |
=== chihchun is now known as chihchun_afk | ||
=== chihchun_afk is now known as chihchun | ||
pedronis | does this rings any bells https://forum.snapcraft.io/t/installation-fails-as-core-snap-not-found/9367/3 ? | 10:47 |
pedronis | mvo: do snaps that require core18 actually also need core ? but we install only the former automatically | 11:01 |
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:03 |
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:04 |
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:05 |
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:06 |
pedronis | Chipaca: does juju-db has base: core18 ? or is there something else going on in the forum case | 11:07 |
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:08 |
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:09 |
zyga | re | 11:12 |
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:22 |
mvo | Chipaca: cool! | 11:23 |
Chipaca | whoops, forgot to undo a for-testing change i'd made | 11:27 |
* Chipaca pushes | 11:27 | |
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:42 |
=== chihchun is now known as chihchun_afk | ||
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> | 11:58 |
pedronis | Chipaca: I moved the card about this to doing | 12:15 |
Chipaca | pedronis: thank you | 12:15 |
=== pstolowski is now known as pstolowski|lunch | ||
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:27 |
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:28 |
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:29 |
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:30 |
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:31 |
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:47 |
Chipaca | mvo: three PRs up, that should be it for 2.37 from me :-) | 12:48 |
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:52 |
Chipaca | pedronis: where it says don't, read only | 12:53 |
Chipaca | :-) | 12:53 |
Chipaca | i'll fix it right back | 12:53 |
mborzecki | off to pick up the kids | 13:01 |
cachio | niemeyer, hello, a new change pushed for the spread reboot PR | 13:01 |
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:08 |
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:13 |
niemeyer | cachio: Thanks! | 13:20 |
=== pstolowski|lunch is now known as pstolowski | ||
pedronis | Chipaca: the description seems also wrong | 13:34 |
pedronis | I mean the PR description | 13:35 |
Chipaca | pedronis: yes | 13:35 |
Chipaca | https://www.dell.com/en-uk/work/shop/cty/pdp/spd/precision-17-7720-laptop/xctop7720emea | 13:40 |
Chipaca | guess what laptop won't be my next laptop | 13:41 |
=== fishies is now known as Crocodillian | ||
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:48 |
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:49 |
ogra | "delivery by row-boat to the continent ..." | 13:50 |
ogra | .. "included" | 13:50 |
Chipaca | mborzecki: the HQ stands for high quality, it's the audiophile edition | 13:53 |
mborzecki | intelophile | 13:53 |
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:56 |
Chipaca | pedronis: I could, but that'd kick off another travis run :-) | 13:57 |
Chipaca | pedronis: should I? | 13:58 |
pedronis | not immediately | 13:59 |
Chipaca | pedronis: it's two commits, one of which is on master :-) | 13:59 |
mborzecki | zyga: standup | 14:02 |
zyga | joining! | 14:02 |
=== chihchun_afk is now known as chihchun | ||
* pedronis break | 14:35 | |
* zyga goes for lunch | 14:48 | |
pedronis | mvo: can we answer something here https://forum.snapcraft.io/t/installation-fails-as-core-snap-not-found/9367/3 ? | 14:51 |
mup | PR snapd#6340 opened: store: undo workaround for timezone-less released-at <Created by chipaca> <https://github.com/snapcore/snapd/pull/6340> | 14:52 |
mvo | pedronis: yes | 15:03 |
Chipaca | SIGH | 15:22 |
* Chipaca fixes things | 15:22 | |
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:25 |
=== chihchun is now known as chihchun_afk | ||
zyga | quick coffee | 15:44 |
* genii 's ears perk up momentarily at the mention of coffee | 15:45 | |
zyga | the coffee machine is warming up | 15:46 |
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:56 |
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:57 |
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 | 15:59 | |
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:22 |
* mvo goes and writes about " Installation fails as ‘core snap’ not found " | 16:23 | |
* zyga goes to read that | 16:28 | |
zyga | namespaces and more namespaces | 16:28 |
pedronis | mvo: thank you | 16:35 |
pedronis | mvo: need to go for a walk, but will do more 2.37 reviews afterward | 16:35 |
mvo | pedronis: \o/ | 16:36 |
mvo | pedronis: I will also make a cup of tea and look at the next steps | 16:36 |
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:40 |
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:41 |
jdstrand | pedronis: yes, I saw the emails for both | 16:44 |
zyga | brb | 17:07 |
zyga | back to work :) | 17:23 |
=== pstolowski is now known as pstolowski|afk | ||
zyga | tests looking good, writing some commit message for the PR | 17:56 |
zyga | some spread IO issues | 18:37 |
zyga | but restarted now | 18:37 |
* mvo hugs pedronis for his tireless reviews | 18:49 | |
pedronis | I think I looked at most 2.37 things, they need 2nd reviews and/or passing tests | 18:54 |
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:32 |
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> | 19:38 |
* zyga EODs | 20:01 | |
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> | 20:08 |
mup | PR snapcraft#2434 opened: rust plugin: refactor to use the latest rustup <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2434> | 21:12 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!