/srv/irclogs.ubuntu.com/2019/01/09/#snappy.txt

=== Caelum is now known as fishies
=== ^arcade_droid is now known as zarcade_droid
=== chihchun is now known as chihchun_afk
mborzeckimorning06:14
zygagood morning07:32
mborzeckizyga: hey07:39
zygagood morning :)07:39
zygaI'm staying away from the office till spring07:39
liuxg1hi, did anyone make the bluetooth working on ubuntu core, raspberry pi3b+ board? I have a problem in getting it working07:46
zygaI have that board but I've never tried using BT yet07:46
liuxg1zyga, 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
mupBug #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
zygaseems like that bug is well under way07:49
liuxg1I 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
mborzeckimaybe something bluez related07:50
liuxg1zyga, are you able to pair a bluetooth device using your device? yes, it is related to the bluez.07:50
zygaas I said I have never tried07:51
zygathe device is on a shelf now, I'm powering them down to avoid cable mess and power drain when unused07:51
mborzeckiliuxg1: can you run bluetootctl devices? is the paired device listed there?07:52
liuxg1I 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. thanks07:52
liuxg1mborzecki, there is nothing there.07:52
mborzeckiliuxg1: and bluetootctl list?07:53
liuxg1I have to run this command: sudo /snap/bin/hciattach /dev/ttyAMA0 bcm43xx 921600 noflow - to see something there.07:53
liuxg1otherwise, there is no controller for it.07:54
liuxg1also, it seems that I have to install the bluez in --devmode to get it working.07:58
mborzeckiintersting, looking at some rspi3 block diagrams, bt contoller is indeed wired over uart08:00
mborzeckiliuxg1: anything related to apparmor in dmesg?08:00
=== chihchun_afk is now known as chihchun
=== pstolowski|afk is now known as pstolowski
pstolowskimornings08:01
mborzeckipstolowski: hey, while you're here got some questions about pre and post refresh hooks, figured you might know :)08:02
mborzeckipstolowski: pre refresh runs before the previous revision has been disabled, so the services will be up at this point?08:04
liuxg1mborzecki, these are the errors I got from dmesg. I am not sure whether they are related.  https://paste.ubuntu.com/p/kXhssPJnSH/08:04
pstolowskimborzecki: yes, pre-refresh runs before stopping the services; stop-snap-services is next08:06
mborzeckipstolowski: great, then post-refresh is run before the services are started, so I could snapctl stop --disable a service inside?08:07
pstolowskimborzecki: yes you could. we don't have spread tests for something like this though..08:11
mborzeckipstolowski: 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 it08:13
mborzeckipstolowski: relevant post: https://forum.snapcraft.io/t/how-to-manage-services-with-sockets-timers/7904/308:13
mborzeckipstolowski: 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 out08:14
pstolowskimborzecki: i'll take a look at this in a bit and get back to you, need to prepare for the meeting i've soon08:17
mborzeckipstolowski: thanks08:17
zygahey pawel08:29
mborzeckizyga: fontconfig fonts.conf parser https://paste.ubuntu.com/p/2KJDd6xdff/08:39
mupBug #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
mborzeckizyga: 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
mupBug #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
mupPR 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
pedronismvo: hi, I commented on #633108:56
mupPR #6331: systemd: allow only a single daemon-reload at the same time <Created by mvo5> <https://github.com/snapcore/snapd/pull/6331>08:56
mvopedronis: excellent, thank you!08:56
Chipacao/08:59
pedronisChipaca: hi09:00
Chipacapedronis: 'sup09:01
mvoChipaca: good morning!09:02
pstolowskipedronis: 1 sec, i'm coming, have little touble with google meet09:02
mvoChipaca: oh is the released-at stuff coming along :) ?09:02
Chipacamvo: good morning boss09:02
Chipaca:-)09:02
mvoChipaca: *cough* I need to think about the right kind of title - maybe MASTER is appropriate?09:02
pedronispstolowski: I added a meet to the event btw09:03
mvoChipaca: I guess we need to discuss while having a beer in malta ;)09:03
Chipacamvo: https://i.imgur.com/x846GwL.png09:03
Chipacapedronis: also ^09:03
Chipacathat's me playing with some other stuff as well09:03
mvoChipaca: \o/ so cool09:03
Chipacabecause in the sample output  requesting this data, there's no up arrows09:04
Chipacait's all compacted09:04
pedronisChipaca: anyway I still have the feeling that we maybe should go with just the iso dates for this09:04
Chipacabut i don't know if that was just for didactic reasons or not09:04
Chipacapedronis: yeah, i think so09:04
Chipacapedronis: and use the human date or full iso date in verbose09:05
mvopedronis: 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 today09:10
zygamborzecki: nice09:11
Chipacapedronis: there's a problem with released-at currently in the store, where it's not the right format09:14
Chipacapedronis: easy to work around, but the fix is on master of the store already09:14
Chipacapedronis: dunno if i should propose without the workaround and mark blocked, or not09:15
mvoChipaca: when will the store release the fix? like roughly?09:18
Chipacamvo: A fair question and one that in recent [days] 'as been much on my mind.09:18
Chipacamvo: yesterday somebody was off sick, so it didn't happen09:19
mvoChipaca: aha, but that sounds like "really-soon" (i.e. days not weeks?)09:19
Chipacayeah09:20
mvocool09:20
Chipacayeah09:20
Chipacaright now I have the workaround because I like poking at things, but I think I'll drop it before proposing the PR09:21
mborzeckizyga: want to chat quickly about the fontconfig thing?09:22
zygasure09:22
zygahere or HO?09:22
mvoChipaca: 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
mborzeckizyga: https://meet.google.com/koc-gdjf-qve09:23
mvoChipaca: or we just drop it from 2.37 :)09:23
mvoChipaca: but I really like the feature so that would be a bit sad09:23
Chipacamvo: I'd rather add the workaround than have the feature slip09:26
mvoChipaca: *nod*09:27
mborzeckizyga: let's try again https://meet.google.com/dwz-hyza-nwc?authuser=109:28
Chipacapedronis: mvo: https://forum.snapcraft.io/t/upcoming-change-to-the-output-of-snap-info/9368 fwiw09:37
Chipacapedronis: mvo: or should I say https://forum.snapcraft.io/t/upcoming-change-to-the-output-of-snap-info/9368?u=chipaca09:37
* Chipaca hoards the fake internet points09:37
pedronisChipaca: was in a meeting with pawel09:38
Chipacapedronis: poor pawel09:38
* Chipaca hides09:38
pedronisChipaca: heh09:38
pedronisChipaca: I think we want the workaround, we can backport dropping it if that makes sense09:39
ChipacaI'm told the fix will land before EOD today09:40
Chipacas/will/should/09:40
pedronisChipaca: ok09:40
Chipacabut as i say, the workaround exists already :-)09:40
Chipacaanyway, fixing the tests and should have it up before standup09:40
pedronisyea09:40
pedronisChipaca: do you have a PR for the other change?  (unicode off if no tty)09:41
Chipacapedronis: not yet, do you want that one first?09:41
pedronisChipaca: not first, but would like them to go out together somehow09:41
pedronis(disrupt things in one go)09:41
Chipaca:-)09:41
ChipacaI could stack 'em09:42
Chipacabut reviewers hate that :-)09:42
mvoChipaca: \o/09:42
Chipacamvo: does a 2.37 branch exist already?09:42
mvoChipaca: not yet, will be created today09:42
Chipaca'cause if so i could get the individual things on master, and stack them for 2.3709:42
mvoChipaca: if all goes well :)09:43
Chipacathere's also the alignment change09:43
Chipacabah09:43
ChipacaI could do this: add released-at in one pr, but use it in the same pr as the other two09:43
Chipacaseems messy conceptually tho09:44
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
pedronisdoes this rings any bells https://forum.snapcraft.io/t/installation-fails-as-core-snap-not-found/9367/3 ?10:47
pedronismvo: do snaps that require core18 actually also need core ? but we install only the former automatically11:01
pstolowskimborzecki: 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
mborzeckipstolowski: we know the names, but we don't know whether A rev 1 is the same as A rev 211:03
mborzeckipstolowski: all we know is that the names are the same11:04
mvopedronis: 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 history11:04
mborzeckipstolowski: that's why i think just looking at name is fragile11:04
pedronismvo: that bug is probably still in 18.04 I suppose because no SRU yet?11:05
mvopedronis: that is possible, is this about the forum post you mentioned earlier?11:05
pedronismvo: no, see PM but maybe also11:06
pedronismvo: where do we mount snapctl from for those?  the host until there is a core snap available?11:06
Chipacapedronis: fwiw, it's snap-confine11:06
mvopedronis: aha, yes, I think this is the SRU :/11:06
Chipacain 2.34.211:06
pstolowskimborzecki: ah, because it's about knowing what to keep enabled/disabled?11:06
mborzeckipstolowski: yup11:06
pedronisChipaca: yes, that's why me asking these questions11:06
mvopedronis: yeah, the host (95% certain)11:06
pedronisChipaca: does juju-db has base: core18 ?  or is there something else going on in the forum case11:07
pedronisI could check myself, but need to go have lunch11:08
Chipacapedronis: I mean, the forum says it's core1811:08
pedronisah11:08
pedronisyes11:08
Chipacapedronis: and the snap does say base:core1811:08
pedronisyea, ok11:08
pedronisso it's fixed bug11:08
pedronisbut no SRU11:08
Chipacaif they installed the core snap, they'd re-exec, and get the fix11:08
mvoyeah, I need to chase the SRU, let me do that now11:08
pedronis(also this will be saner when we have snapd snap on classic but also means more work)11:09
pstolowskimborzecki: i see, right. i agree with your assesment and the idea of using post-refresh hook11:09
zygare11:12
mupPR snapd#6337 opened: store, snap, cmd/snap: channels have released-at <Created by chipaca> <https://github.com/snapcore/snapd/pull/6337>11:22
Chipacapedronis, mvo: ^11:22
mvoChipaca: cool!11:23
Chipacawhoops, forgot to undo a for-testing change i'd made11:27
* Chipaca pushes11:27
mupPR 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
mupPR 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
pedronisChipaca: I moved the card about this to doing12:15
Chipacapedronis: thank you12:15
=== pstolowski is now known as pstolowski|lunch
Chipacasergiusens: does snapcraft have a homebrew plugin? 1.9.0 supports Linux, and it's been out for _hours_ already!12:27
sergiusensChipaca: a homebrew plugin to build from brews and make them snaps?12:27
Chipacasergiusens: clearly12:27
sergiusensmeta meta meta12:28
Chipaca:-)12:28
sergiusensI guess with overlays it could work12:28
sergiusensprobably worth looking into when more man power becomes available 🙂12:28
Chipacaalso 1.9.0 drops support for intel 32 bits12:28
mupPR 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
Chipacadunno if that affects zyga's lab12:28
zygaChipaca: not much12:29
zygaChipaca: you mean go 1.9 is x86_64 only?12:29
zygaChipaca: I have one 32bit (strict) machine left12:29
zygaas in i38612:29
Chipacazyga: "Homebrew 1.9.0 no longer runs on 32-bit Intel CPUs."12:29
zygaah12:29
zyganah that's ok12:29
zygathat entry point is not using macos :)12:29
Chipacafrom https://brew.sh/2019/01/09/homebrew-1.9.0/12:29
zygathere are no 32bit only macs left really12:30
zygaeven the watch is 64 bit12:30
Chipacait isn't 4 bits emulating 8?12:30
zyga:D ?12:31
zygaquick coffee break12:31
zygabtw, working from the kitchen has one clear advantage :)12:31
Chipacanice tweak to travis' ui there12:31
Chipacazyga: just keep your jitters under half of your keycap size and you'll be fine12:31
mupPR 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
Chipacamvo: three PRs up, that should be it for 2.37 from me :-)12:48
pedronisChipaca: you mean non-tty there?12:52
Chipacagah12:52
* Chipaca checks12:52
Chipacagah²12:52
ChipacaI had it right the first time :-)12:52
Chipacaand … "fixed" it12:52
Chipacapedronis: where it says don't, read only12:53
Chipaca:-)12:53
Chipacai'll fix it right back12:53
mborzeckioff to pick up the kids13:01
cachioniemeyer, hello, a new change pushed for the spread reboot PR13:01
cachiopstolowski|lunch, hey, if you have time could you please take a look to #5887 ?13:08
mupPR #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
mupPR 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
niemeyercachio: Thanks!13:20
=== pstolowski|lunch is now known as pstolowski
pedronisChipaca: the description seems also wrong13:34
pedronisI mean the PR description13:35
Chipacapedronis: yes13:35
Chipacahttps://www.dell.com/en-uk/work/shop/cty/pdp/spd/precision-17-7720-laptop/xctop7720emea13:40
Chipacaguess what laptop won't be my next laptop13:41
=== fishies is now known as Crocodillian
zygaouch13:48
zygaStarting at £7,001,726.2813:48
zygaChipaca: but hey, it's a "bargin"13:48
zygaOriginal Price13:48
zyga£10,002,466.1113:48
ograoh, is dell already establishing the post-brexit pricing ?13:49
mborzeckiIntel Core i5-6300HQ with Thunderbolt 3 is super expensive13:49
mborzeckimust be that thunderbolt 313:49
ogra"delivery by row-boat to the continent ..."13:50
ogra.. "included"13:50
Chipacamborzecki: the HQ stands for high quality, it's the audiophile edition13:53
mborzeckiintelophile13:53
zygamborzecki: wait, is that really i5-6300HQ? :)13:56
zygaoh my13:56
zygathat's garbage :D13:56
zyga(for that price)13:56
pedronisChipaca: could you push master into those PRs to reduce the diff?13:56
Chipacapedronis: I could, but that'd kick off another travis run :-)13:57
Chipacapedronis: should I?13:58
pedronisnot immediately13:59
Chipacapedronis: it's two commits, one of which is on master :-)13:59
mborzeckizyga: standup14:02
zygajoining!14:02
=== chihchun_afk is now known as chihchun
* pedronis break14:35
* zyga goes for lunch14:48
pedronismvo: can we answer something here https://forum.snapcraft.io/t/installation-fails-as-core-snap-not-found/9367/3 ?14:51
mupPR snapd#6340 opened: store: undo workaround for timezone-less released-at <Created by chipaca> <https://github.com/snapcore/snapd/pull/6340>14:52
mvopedronis: yes15:03
ChipacaSIGH15:22
* Chipaca fixes things15:22
zygaChipaca: found something fun?15:25
Chipacaspread tests looking for unicode output in things that no longer produce unicode output :-)15:25
Chipacafun level 3.7215:25
=== chihchun is now known as chihchun_afk
zygaquick coffee15:44
* genii 's ears perk up momentarily at the mention of coffee15:45
zygathe coffee machine is warming up15:46
pedronisChipaca: reviewed #633815:56
mupPR #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
Chipacapedronis: thanks15:57
Chipacapedronis: info is a lovely mess15:57
Chipacapedronis: I'll see if I can't clean it up a bit after epochs15:57
Chipacameanwhile, probably not worth it for this change15:57
pedronisChipaca: as I said, mostly wondering, because of the passing things in and out, and also formatting concerns are all a bit mixed up now15:59
* cachio lunch15:59
pedronismvo: will you write there?  you are probably the most familiar with the state, also afaik you did the fix :)16:22
mvopedronis: yes, I will write in the forum, sorry for hte delay, just finished the unit tests for protocol-error PR (6331)16:22
mvopedronis: 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
zyganamespaces and more namespaces16:28
pedronismvo: thank you16:35
pedronismvo: need to go for a walk, but will do more 2.37 reviews afterward16:35
mvopedronis: \o/16:36
mvopedronis: I will also make a cup of tea and look at the next steps16:36
pedronisjdstrand: hi, I finally got to look and comment on #5644 ,there are some questions there16:40
mupPR #5644: interfaces: add audio-playback/audio-record and make pulseaudio manually connect <Created by jdstrand> <https://github.com/snapcore/snapd/pull/5644>16:40
pedronisjdstrand: also, could you look at #6334 ?16:41
mupPR #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
jdstrandpedronis: yes, I saw the emails for both16:44
zygabrb17:07
zygaback to work :)17:23
=== pstolowski is now known as pstolowski|afk
zygatests looking good, writing some commit message for the PR17:56
zygasome spread IO issues18:37
zygabut restarted now18:37
* mvo hugs pedronis for his tireless reviews18:49
pedronisI think I looked at most 2.37 things, they need 2nd reviews and/or passing tests18:54
mupPR 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
mupPR 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 EODs20:01
mupPR 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
mupPR 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!