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

=== [asc] is now known as [-asc-]
mborzeckimorning05:54
mwhudsonhttps://buildd.debian.org/status/fetch.php?pkg=snapd&arch=i386&ver=2.30-3&stamp=1515462684&file=log <- is deviceMgrSuite.TestDoRequestSerialErrorsOnNoHost known flaky?06:39
mupPR snapd#4453 closed: Update README.md <Created by popey> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4453>06:53
mborzeckimwhudson: sort of, the test is supposed to use an invalid domain name, though there's been some back and forth on what an invalid domain name that would not be resolved is06:57
mwhudsonhmm06:58
mwhudsontime to remember how to retry builds on debian buildds i guess06:58
mborzeckimwhudson: i think there might be something special here though, we've used *.invalid, then some bogus name.com and finally settled with nowhere.test, but it seems like the resolver is actually trying to resolve it :/06:59
mwhudsonmborzecki: it only failed on i38606:59
mwhudsonbut maybe that builder is different somehow06:59
zyga-ubuntugood morning07:01
mwhudsonmorning zyga-ubuntu07:01
mborzeckizyga-ubuntu: hey07:04
mvohey zyga-ubuntu and mwhudson and mborzecki07:04
mborzeckimvo: morning07:05
mwhudsonlaunchpad's "click a button" certainly seems clearer than https://release.debian.org/wanna-build.txt07:06
mupPR snapd#4451 closed: cmd/snap-update-ns: improve mocking for tests <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4451>07:06
zyga-ubuntuI'll be here soon, just finishing breakfast for kids07:10
* zyga-ubuntu finishes coffee checking maikl07:28
mborzeckidoing any text transformation in `snap_confine_snap_confine_debug_LDADD = $(something .. $(snap_confine_snap_confine_LDADD))` results in `snap_confine_snap_confine_debug_DEPENDENCIES = $(something .. $(snap_confine_snap_confine_LDADD))`, so the DEPENDENCIES includes -ludev -lfoo now :/07:41
kalikianagood mornin07:58
mborzeckikalikiana: morning07:59
mvomborzecki: 4449 fails with a link error on debian, could this be related to your makefile changes or is that an unrelated issue?08:13
mborzeckimvo: related, i'm working on it, one fugly workaround for automake :/08:14
mvomborzecki: uff, ok. good luck!08:15
mborzeckimvo: pushed just now, feel free to take a look and review :)08:15
mvogreat08:15
mborzeckipstolowski: morning08:22
pstolowskigood morning!08:22
mvogood morning pstolowski !08:26
* pstolowski reboots08:27
mborzeckimvo: there's a conflict in #435608:36
mupPR #4356: many: add new `snap refresh --amend <snap>` command <Created by mvo5> <https://github.com/snapcore/snapd/pull/4356>08:36
mborzeckiis #3998 actionable now?08:45
mupPR #3998: snap-confine, snap-seccomp: utilize new seccomp logging features <Decaying> <Created by tyhicks> <https://github.com/snapcore/snapd/pull/3998>08:45
zyga-ubuntumborzecki: I think it should be in the kernel by now08:45
zyga-ubuntumborzecki: but we need to connect the dots08:45
mvozyga-ubuntu, mborzecki iirc the problem is/was that fedora has no updated libseccomp08:48
zyga-ubuntumvo: indeed, we may need to always allow an older version of the library to stay realistically deployable08:50
mborzeckiok, let me quickly ccheck what's the latest version in updates08:50
mvozyga-ubuntu: yeah, iirc that is the missing bit08:51
mborzeckiby the looks of it, it's 2.3.2 in f26 and f2708:51
mborzeckiunless there's some specific patch that's missing08:51
zyga-ubuntumvo: altough having said that we should not differ in behaviour for apps, we should switch to EPERM universally, just enable logging if that is implemented08:51
zyga-ubuntumborzecki: ^08:52
mborzeckiright, makes sense08:52
mupPR snapd#4456 opened:  snap: rename `snap advise-command` to `snap advise-snap --command`  <Created by mvo5> <https://github.com/snapcore/snapd/pull/4456>08:54
mupPR snapd#4444 closed: tests: use "quiet" helper instead of "dnf -q" to get errors on failures <Created by mvo5> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/4444>09:12
=== __chip__ is now known as Chipaca
* kalikiana coffee09:29
* zyga-ubuntu tries to untangle an abstraction09:33
zyga-ubuntusigh09:33
zyga-ubuntuso close yet a bit frustratingly not there09:33
Chipacazyga-ubuntu: another security issue just to brighten your day: https://twitter.com/chris__martin/status/95051857458992333109:33
zyga-ubuntuwhat!09:33
zyga-ubuntuLOL!09:34
zyga-ubuntubeautiful09:34
Chipaca:) you're welcome09:34
mupPR snapd#4457 opened: spread: trying to re-enable tests on Fedora <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/4457>09:37
mborzeckiChipaca: you mentioned that you ran into some non-dnf related issues on fedora, can you describe what that was?09:38
Chipacamborzecki: "snap: command not found", and systemctl saying snapd.service was not found09:38
Chipacamborzecki: IOW something that looked suspiciously like snapd not being installed09:39
mborzeckiinteresting, i ran some tests on fedora today, haven't observed anything like this, maybe it will come up in the PR i just opened ^^09:41
mborzeckiduh. wonder what conditions would have to be met to get a latex error message that makes sense09:44
Chipacamborzecki: maybe the conditions need to exist in the observer's mind09:45
mborzeckirandomly hitting x09:46
Chipacamaybe latex was all about psychoquantumdynamics09:46
mborzeckihalf of the 'exploit' was getting your latex toolchain to work with proper fonts and language09:47
ackkmvo, hi, do you have any suggestion on how to debug those errors on my snap install?10:05
zyga-ubuntucoffee10:06
mvoackk: not right now, do you see apparmor or seccomp denials when you run the install? maybe zyga-ubuntu has an idea about the issue. another thing to try is see if using maas without base-18 gives the same error (removing the base: line)10:08
zyga-ubuntuhmm hmm hmm hmm10:10
ackkmvo, no, nothing10:10
ackkmhm, so there's this thing: the first time I snap try I get this error: https://paste.ubuntu.com/26352275/10:13
ackkbut now I have the prime directory mounted10:13
ackk/dev/sda3 on /snap/maas/x1 type btrfs (rw,relatime,space_cache,subvolid=263,subvol=/@home/ack/canonical/src/maas/build/dev-snap/prime)10:13
ackkand I can't unmount it10:13
ackkI suspect that's messing up snapd10:13
zyga-ubuntuoh10:13
zyga-ubuntubtrfs10:13
mvoackk: ohh, that is indeed interessting and a useful clue10:14
zyga-ubuntuI don't know if eve ever test that it works with snapd10:14
mvoand it would explain why I was not able to reproduce!10:14
ackkzyga-ubuntu, I've always used it10:14
ackkwith no issue so far10:14
zyga-ubuntuackk: sure but we never (AFAIK) use it and it's an extra layer of complexity10:14
mvoackk: does "systemctl  status snap-maas-x1.mount" and "journalctl  -xe"" give you anything useful why it fails?10:14
ackk$ umount /snap/maas/x110:14
ackkumount: /snap/maas/x1: umount failed: Operation not permitted10:14
zyga-ubuntuackk: I'm not saying it doesn't work, just that it is an extra variable10:14
zyga-ubuntuackk: anything in audit log:?10:15
ackkProcess: 20922 ExecUnmount=/bin/umount /snap/maas/x1 -c (code=exited, status=32)10:15
* zyga-ubuntu waits for people to try snaps on ZFS10:15
ackkbut status is active (mounted)10:15
ackkmvo, nothing relevant in journalctl, just ^^ in status output10:16
ackkI'll try to reboot the container10:17
zyga-ubuntuoh,. it's a container?10:17
zyga-ubuntuackk: wait10:17
mvoackk: hm, exit status 32 just means "mount-failure" not very helpful, especially without any extra info10:17
zyga-ubuntuackk: cat /proc/self/mountinfo10:17
zyga-ubuntumvo: is it not just the "snapd fails in containers or places without rshare /" case?10:18
mvozyga-ubuntu: it might be, I thought we had this under control :(10:18
zyga-ubuntuno, it's not fixed yet10:18
zyga-ubuntumvo: after several attempts it's still broken10:18
zyga-ubuntumvo: sorry about that :/10:18
ackkoh, after reboot /var/lib/snapd/snaps/maas_x1.snap became a dir :)10:18
ackkzyga-ubuntu, mvo ok, after reboot I don't have that mountpoint anymore, but same error10:19
mvozyga-ubuntu: no worries, thanks for remembering more than I do - yeah, then its probably this10:19
ackkfwiw that error from the paste is totally reproducible the first time I "snap try"10:19
ackkzyga-ubuntu, https://paste.ubuntu.com/26352304/10:20
mvoackk: and from installing, right? not just try?10:20
ackkmvo, snap try10:20
mvoackk: so just try?10:20
ackkmvo, correct10:21
* mvo nods10:21
ackkzyga-ubuntu, so that mountpoint was left behind somehow10:21
zyga-ubuntu-10:21
zyga-ubuntuso not shared10:21
zyga-ubuntuso proably that's the same cause as before10:21
zyga-ubuntuackk: I cannot explain how maas_x1.snap became a directory10:22
ackkso the thing that makes me thing that snapd is confused by that mountpoint is "2018/01/09 10:23:35.075389 task.go:303: DEBUG: 2018-01-09T10:23:35Z ERROR cannot find installed snap "maas" at revision x1"10:24
ackkthere's no installed maas snap10:24
ackkbut it seems snapd thinks it's actually updating it10:24
zyga-ubuntuackk: I think this is https://bugs.launchpad.net/snapd/+bug/171293010:24
mupBug #1712930: snap-confine: mounts happen in the wrong order <snapd:Confirmed for zyga> <https://launchpad.net/bugs/1712930>10:24
ackkah the old issue with snap in lxd10:26
zyga-ubuntuackk: indeed10:26
zyga-ubuntuackk: can you (perhaps) try if this goes away without the container in place?10:26
zyga-ubuntuackk: it woudl re-affirm where we are10:26
zyga-ubuntuackk: and I promise to attack that bug again later today10:26
ackkzyga-ubuntu, sure, lemme test10:27
ackkzyga-ubuntu, ah! so, somehow now umount worked on that dir10:27
ackkand snap try went further along10:27
ackkso that seems to be the issue10:28
ackk(I'm still testing in the container)10:28
* zyga-ubuntu makes some progress actually10:29
ackkzyga-ubuntu, so there's something weird, there10:33
ackk$ mount | grep snap/maas10:33
ackk/dev/sda3 on /snap/maas/x1 type btrfs (rw,relatime,space_cache,subvolid=263,subvol=/@home/ack/canonical/src/maas/build/dev-snap/prime)10:33
ackkzyga-ubuntu, ^ that's not actually a btrfs subvolume10:33
ackkis snap being confused by the underlying btrfs and trying to mount it as such?10:33
zyga-ubuntuackk: I doubt it, snapd doesn't mount that, this is systemd, systemd is not confused either (probably); mount is confusing, this is a bind mount and mount cannot represent those10:34
zyga-ubuntulook at mountinfo10:34
* kalikiana more coffee10:34
zyga-ubuntuthis is a bit annoying about bind mounts, they suck for UX10:34
zyga-ubuntunobody can make heads or tails about what's where10:34
zyga-ubuntu(and how)10:34
zyga-ubuntubecause bind mount history is logical to humans but not preserved10:34
zyga-ubuntuand bind mount effects are highly confusing but that's what the kernel models and stors10:34
zyga-ubuntu*stores10:34
ackkzyga-ubuntu, so /snap/maas/x1 is a bind-mount?10:35
zyga-ubuntuackk: look at mountinfo10:35
zyga-ubuntuackk: and we'll see10:35
zyga-ubuntuackk: but 99% yes10:35
* zyga-ubuntu will design a bind mount t-shirt one day10:35
ackkzyga-ubuntu, https://paste.ubuntu.com/26352351/10:36
Chipacais it bad that last night i learned how to use a package-private function from outside the package, in go?10:36
Chipacathis will make testing some things a lot simpler :-D10:36
zyga-ubuntuso /snap/maas/x1 is actually a btffs filesystem from /dev/sda3, specifically the fragment located at /@home/ack/canonical/src/maas/build/dev-snap/prime relative to that filesytem10:37
zyga-ubuntuyep, that's a bind mount10:37
zyga-ubuntuat some point I think mount stopped being useful for humans10:38
jameshChipaca: without modifying the package?10:38
Chipacajamesh: yes10:38
jameshChipaca: care to share then?10:38
Chipacajamesh: https://pastebin.ubuntu.com/26352358/10:39
zyga-ubuntujamesh: hey! sorry for lagging; I'd like to arrange a call later this week between you me and gustavo; I need to do some prep work but I think we can try the day after tomorrow (assuming schedule allows)10:39
zyga-ubuntujamesh: do you have preference for your evening vs your morning?10:39
zyga-ubuntujamesh: gustavo and I will adapt10:39
Chipacajamesh: the only caveat is that the package from which you do this needs to have a .s file (it can be empty)10:39
Chipacaand you need to import unsafe even if you don't use it, otherwise go:linkname refuses to woork10:39
Chipacaand you need to import the package, even if you don't use it :-)10:40
jameshzyga-ubuntu: my evening is probably going to be the easiest to cover Australia, Europe, and Brazil10:40
mupPR snapd#4458 opened: overlord/snapstate: no refresh just for hints if there was a recent regular full refresh <Created by pedronis> <https://github.com/snapcore/snapd/pull/4458>10:40
Chipacajamesh: (if you want to run that as is, note modeFromTriplet isn't in master yet; grab #4394 for that)10:41
mupPR #4394: snap: give the snap.Container interface a Walk method <Created by chipaca> <https://github.com/snapcore/snapd/pull/4394>10:41
zyga-ubuntujamesh: ok, so today I'm trying to wrap up last year's PRsr and will jump into LXD issue but we can do day after tomorrow because I could focus on getting in-depth understanding of your work tomorrow10:41
ackkzyga-ubuntu, I get https://paste.ubuntu.com/26352371/ at the first install on the physical machine10:41
ackkactually, every time10:42
zyga-ubuntujamesh: gustvo would like to get better understanding of where we are in anticipation of the upcoming sprint next week10:42
jameshokay10:42
zyga-ubuntuackk: interesting, is /var/snap present?10:42
zyga-ubuntumaybe it's a packaging issue10:42
zyga-ubuntusnap-confine doesn't create that directory10:42
ackkzyga-ubuntu, yes it is, i have also other snaps installed10:42
Chipacamvo: i've added some tests to #4394, i think you'll like it now10:42
mupPR #4394: snap: give the snap.Container interface a Walk method <Created by chipaca> <https://github.com/snapcore/snapd/pull/4394>10:42
zyga-ubuntuackk: bummer, no idea then :/10:43
ackkzyga-ubuntu, FTR this is on artful10:43
zyga-ubuntuackk: noted10:43
zyga-ubuntuackk: ahh wait10:43
zyga-ubuntuackk: this is with the base 18 snap, right?10:43
ackkyeah10:43
zyga-ubuntuackk: so can you quickly look in /snap/base-18/current/var/snap10:44
ackkI get the same error on the first try on bionic, so maybe that's the root cause?10:44
zyga-ubuntudoes that exist?10:44
zyga-ubuntuif no that's the problem10:44
zyga-ubuntu(needs to be in the base snap)10:44
ackknope10:44
ackkbingo :)10:44
zyga-ubuntugood catch!10:44
ackkmvo, ^^10:44
ackkzyga-ubuntu, so just creating var/snap inside the base should work?10:46
zyga-ubuntuyes10:46
zyga-ubuntuwell10:46
zyga-ubuntuone step forward10:46
zyga-ubuntubut yes10:46
mupPR snapd#4459 opened: snap: add support for `snap advice-snap pkgName` <Created by mvo5> <https://github.com/snapcore/snapd/pull/4459>10:46
mvoackk: thanks! a shame that LP building is not working, master is ok https://github.com/snapcore/base-18/blob/master/hooks/20-extra-files.chroot#L18 :/ I can give you an updated snap or you can hack it yourself?10:48
zyga-ubuntuwoot10:48
zyga-ubuntuprogress10:48
zyga-ubuntu5 errors, but the code builds and has good structure10:48
zyga-ubuntuand errors look like string tweaks I did don't match errors in tests10:48
ackkmvo, if you have an updated one that'd be great10:51
ackkmvo, have you asked for whitelisting?10:52
mvoackk: I asked but maybe not forceful enough. I was told things will be reenabled today once the kernel is updated10:54
mvoackk: I updated http://people.canonical.com/~mvo/tmp/base-18_very-unstable_amd64.snap to include the dirs from master10:56
mupPR snapd#4455 closed: store, daemon/api: Rename MyAppsServer, point to dashboard.snapcraft.io instead <Created by sparkiegeek> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4455>10:59
ackkmvo, thanks, will try that one11:10
ackkmvo, there's no /var/snap inthat snap11:15
* zyga-ubuntu sees green tests11:19
mvoackk: meh, I'm sorry, pushed the wrong one :/ I repushed the right one to the same location: https://paste.ubuntu.com/26352614/11:20
ackkmvo, mvo thanks11:20
zyga-ubuntulet's see coverage]11:20
BjornT_zyga-ubuntu: if you have some time, i added some more information to bug #174146311:22
mupBug #1741463: Failure to install maas snap in a container on a host using nvidia drivers <Snappy:New> <https://launchpad.net/bugs/1741463>11:22
ackkmvo, next error I got is: - Run configure hook of "maas" snap if present (run hook "configure": cannot perform operation: mount --bind /snap/base-18/current//etc/alternatives /tmp/snap.rootfs_9Cushn/etc/alternatives: Permission denied)11:25
zyga-ubuntuBjornT_: thanks, this looks like a simple apparmor profile tweak11:26
zyga-ubuntuBjornT_: looking now11:26
zyga-ubuntuhmmm11:27
zyga-ubuntuBjornT_: so looking at snap-confine.apparmor.in I see a rule that looks like this:11:27
zyga-ubuntu    # Vulkan support11:27
zyga-ubuntu    /tmp/snap.rootfs_*/var/lib/snapd/lib/vulkan/* w,11:27
zyga-ubuntu    mount fstype=tmpfs options=(rw nodev noexec) none -> /tmp/snap.rootfs_*/var/lib/snapd/lib/vulkan/,11:27
zyga-ubuntu    mount options=(remount ro) -> /tmp/snap.rootfs_*/var/lib/snapd/lib/vulkan/,11:27
zyga-ubuntucan you look into /etc/apparmor.d/ for snap-confine.real (there may be a few) and see if those rules exist there?11:28
zyga-ubuntu(I want to check if this code is released)11:28
mupPR snapd#4460 opened: daemon: store email, ID and macaroon when creating a new user <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/4460>11:30
mvoackk: aha, let me fix that and update11:30
mvoackk: updated on the same location as before, please let me know if that gets you further11:32
ackkthanks11:33
ackkmvo, it takes me a whilte to test since that mountpoint prevents me from reinstalling the maas snap, so i have to purge snapd, reboot and reinstall every time11:34
mvoackk: that sounds really painful :(11:35
zyga-ubuntuackk: inside a container or in general?11:35
ackkzyga-ubuntu, inside a container11:35
ackkbecause of that mount bug11:35
ackkif I try to remove a snap it's pending forever11:35
zyga-ubuntuok11:36
BjornT_zyga-ubuntu: there's nothing about 'vulkan' in usr.lib.snapd.snap-confine.real11:39
zyga-ubuntuBjornT_: are you on 2.30/11:39
zyga-ubuntuBjornT_: that file may be older but you should be getting reexec from the core snap11:39
zyga-ubuntuso snap.core.*.usr.lib.snapd.snap-confine should have those rules11:40
BjornT_zyga-ubuntu: i'm on xenial, so 2.29.4.211:40
zyga-ubuntuBjornT_: that's ok, but you should see 2.30 in snap version (because of core snap)11:41
zyga-ubuntuBjornT_: as a sanity check, can you switch to edge11:41
zyga-ubuntuBjornT_: and see if that fixes it11:42
zyga-ubuntuBjornT_: I believe this is fixed in master but may just not be released yet11:42
mupPR snapd#4461 opened: snap: fix missing error check when multiple snaps are refreshed <Created by mvo5> <https://github.com/snapcore/snapd/pull/4461>11:43
BjornT_zyga-ubuntu: ah right, yes 2.30. i'll try the edge version11:44
ackkmvo, progress, I'm hitting that ubuntu.csv issue now, do you still have the change to snapcraft.yaml handy?11:47
ackkI think it's just adding distro-info-data to stage-packages11:47
BjornT_zyga-ubuntu: same error with edge core snap. or do i have to restart apparmor as well?11:48
zyga-ubuntuBjornT_: no, you shouldn't have to restart anything11:49
ackkmvo, or actually, wasn't that added to the base?11:49
zyga-ubuntuBjornT_: hmm11:51
zyga-ubuntuinfo="failed flags match"11:51
zyga-ubuntuthis is curious11:51
zyga-ubuntuperhaps (just perhaps) the system is buggy and remount is not a real flag that works11:51
zyga-ubuntuBjornT_: can you please edit the /etc/apparmor.d/snap.core.$LATEST.usr.lib.snapd.snap-confine file11:52
zyga-ubuntuBjornT_: go to the line that describes the three vulcan entries11:52
zyga-ubuntuBjornT_: and remove the "ro" there from (remount ro)11:53
zyga-ubuntuto just (remount)11:53
zyga-ubuntuBjornT_: then use: sudo apparmor_parser -r /etc/apparmor.d/snap.core.$LATEST.usr.lib.snapd.snap-confine11:53
zyga-ubuntuand see if that fixes it11:53
mvoackk: it was not added, there is a open pr, I can add it for you right after lunch for this one test snap11:56
ackkmvo, thanks, and enjoy lunch :)11:57
* zyga-ubuntu keeps fingers crossed11:58
zyga-ubuntuand...11:58
zyga-ubuntuYESSSS11:58
zyga-ubuntu:D11:58
zyga-ubuntuhttps://github.com/snapcore/snapd/pull/4452 enables all of the mechanics of layouts11:59
mupPR #4452: cmd/snap-update-ns: enable writable mimic, allow content sharing to $SNAP <Created by zyga> <https://github.com/snapcore/snapd/pull/4452>11:59
zyga-ubuntunow to rebase the new content interface syntax PR12:00
zyga-ubuntuand to open a new PR that takes use of the layout yaml syntax and puts it into use12:00
zyga-ubuntuat just 310 new lines and 63 removed lines12:01
zyga-ubuntujamesh: ^^ this should be directly applicable to themes now12:01
zyga-ubuntujamesh: I'll focus on the source syntax for content because it also enables the new union mechanics where many entries can contribute to one (spool like) directory12:01
zyga-ubuntumvo: I added 4452 to 2.31, it's my most important addtion now12:02
kalikianapopey: ping wrt bug 174175212:02
mupBug #1741752: yaml.constructor.ConstructorError: could not determine a constructor for the tag '!ExtractedMetadata' <Snapcraft:New> <https://launchpad.net/bugs/1741752>12:02
zyga-ubuntuthere are some unit tests missing but spread tests cover everything and I'll fill the gaps quickly (just a few lines)12:03
ackkmvo, I hacked the image adding distro-info, it works now12:03
zyga-ubuntuBjornT_: any luck, I can help if you don't know what to change12:04
ackkzyga-ubuntu, can you depend on a base from a specific channel? like base: base-18/edge?12:11
mupPR snapd#4313 closed: timeutil: refresh timer take 2 <Created by bboozzoo> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/4313>12:11
zyga-ubuntuackk: no12:11
mupPR snapd#4373 closed: snap: app startup after/before validation <Created by bboozzoo> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/4373>12:17
BjornT_zyga-ubuntu: i removed the 'ro', but i still get the same error12:21
zyga-ubuntuBjornT_: I assume you re-compiled the apparmor profile12:21
zyga-ubuntuBjornT_: and that the revision in the file name matches the revision of the core snap12:21
zyga-ubuntuBjornT_: can you remove all of (remount, ro) next?12:22
BjornT_zyga-ubuntu: yeah, using apparmor_parser -r12:22
zyga-ubuntuok, just checking12:22
mupPR snapd#4462 opened: progress: switch ansimeter's Spin() to use a spinner <Created by chipaca> <https://github.com/snapcore/snapd/pull/4462>12:24
Chipacaniemeyer: mvo: just for you guys ^-^12:24
zyga-ubuntuok, feels like moment to make some food12:28
zyga-ubuntuI'll see you at the standup12:28
BjornT_zyga-ubuntu: still the same error12:32
zyga-ubuntuBjornT_: I wonder how I can test this, what setup do you use12:32
zyga-ubuntuBjornT_: if I could reproduce this locally I could fix it easier12:33
BjornT_zyga-ubuntu: the host is xenial and i've tried with both xenial and bionic containers. not sure if having the nvidia drivers is enough trigger it, though12:34
zyga-ubuntuBjornT_: which drivrs are you on ?12:36
zyga-ubuntuBjornT_: I have an artful system with old nvidia (without vulcan) but I can fake some things (perhaps) to trigger it12:36
zyga-ubuntuBjornT_: are you on xenial kernel or on some newer one?12:36
BjornT_zyga-ubuntu: i have the nvidia-384 package installed. still xenial kernel12:37
zyga-ubuntuI could use a single slot non-outdated nvidia GPU12:38
zyga-ubuntucould use it for testing12:38
zyga-ubuntuI'm all AMD on my desktop12:38
mvoackk: updated13:00
mvoackk: aha, reading backlog you added it already13:01
mvoackk: and it all works? maas init too?13:01
ackkmvo, does base-18 have timezone information?13:01
ackkmvo, no, I get the following error at maas init: psycopg2.DataError: invalid value for parameter "TimeZone": "UTC"13:01
ackkmvo, still investigating if that could be caused by something missing in the base13:02
ackkmvo, are zoneinfo data there?13:02
ackkmvo, ok "SET TIME ZONE 'UTC'" fails in postgres13:03
mvoackk: indeed, no zoneinfo13:04
ackkoh13:04
ackkso that's why13:04
ackkmvo, could those be added as well?13:04
ackkmvo, I'm sorry for this back-and-forth...13:04
mvoackk: sure, one minute13:05
mvoackk: please try now, I added tzdata - I have a meeting now so replies might be a bit slow13:07
ackkmvo, thanks a lot13:07
mvoackk: thanks for all your feedback!13:09
greybackjdstrand_: hey, when you get time could you look at https://github.com/snapcore/snapd/pull/4365 - hopefully it's good to go13:29
mupPR #4365: interfaces/mir: allow Wayland socket and non-root sockets <Created by gerboland> <https://github.com/snapcore/snapd/pull/4365>13:29
ackkzyga-ubuntu, out of curiosity, is it known what's the fix for the mount order issue causing the snap removal bug?13:38
zyga-ubuntuackk: yes, very quite so13:38
ackkzyga-ubuntu, ah cool, can't wait to have that fixed :)13:39
mvoackk: how are things looking with the updated base?13:39
zyga-ubuntuackk: we iterated on that issue a few times (well, I did) and we are zeroing on something that both works and is security sane13:39
Chipacamvo: so, i'll grab title and summary and put it in a bucket indexed by snap name, ok? (anything else you want in there?)13:39
ackkmvo, reinstalling now (had to break for lunch)13:39
zyga-ubuntuackk: I added an action item for myself to look at that with some urgency, we should have some time next week as many people get locked at the sprint and I can work on that13:39
ackkzyga-ubuntu, great13:39
mvoChipaca: iirc that is already (well, summary) done in my latest pr, no?13:43
Chipacamvo: ah, i missed that, perfect13:44
mvoChipaca: i.e. it should work end-to-end (except it does not spelling fixes yet)13:45
Chipacamvo: yep, looks fine (almost exactly what i would've done)13:45
Chipacai'll review in depth in a bit13:46
* Chipaca -> break, first13:46
zyga-ubuntusun is about to set and I dind't have lunch yet13:46
zyga-ubuntutomorrow I'll schedule a walk while it's still bright13:46
zyga-ubuntuand move lunch to post-call13:47
zyga-ubuntunow food :)13:47
ackkmvo, success! init works13:48
zyga-ubuntugreat work guys!13:48
ackkmaas doesn't, but that's another issue13:48
zyga-ubuntu:-)13:48
ackkmvo, zyga-ubuntu, thanks for the support :)13:49
zyga-ubuntufirst base-18 snap that almost runs ;)13:49
ackkmvo, do you think you can push to edge once the distro-info and zoneinfo changes are merged?13:49
ackkzyga-ubuntu, yeah :)13:49
ackkis /sbin/ip in the xenial snap?13:50
zyga-ubuntuno13:50
mborzeckiguys, is there a valid case when a local client is doing a POST to /v2/login with user data *and* sets an Authorization header?13:53
zyga-ubuntuhmmm13:53
* zyga-ubuntu doesn't know what authorization header does13:53
zyga-ubuntumaybe to login as someone else?13:53
mborzeckipedronis: mvo: that's probably for you ^^13:53
pedronisthat's how we send the local macaroon13:54
pedronisit would help detect a re-login13:54
pedronisI suppose13:54
mborzeckihmm then the user.Email from state should be the same as in body login data, right?13:54
mborzeckisome context maybe, so the user is logging in, there's a macaroon, username and an email, say foo@bar.com in ~/.snap/auth.json, then the user does snap login --email bar@bar.com, gets a macaroon, but the email in the response is foo@bar.com instead of bar@bar.com13:57
pedronisthat can happen I suppose if one doesn't use their main SSO email13:58
pedronismatiasb might tell more about that13:59
matiasbmborzecki, pedronis, right, I think the email in the response is the preferred SSO email address14:00
ackkzyga-ubuntu, is something different WRT /bin and /sbin if you use a base?14:00
ackkzyga-ubuntu, what I see in the /sbin and /bin in my snap (from the prime dir) is not there if I snap run --shell14:01
* pstolowski lunch14:01
zyga-ubuntuackk: /bin and /sbin should come directly from the designated base snap (here base-18)14:01
zyga-ubuntuackk: what do you see?14:01
zyga-ubuntuackk: more precisely14:01
ogra_if it is in your prime dir it sould be in §SNAP/bin and $SNAP/sbin14:01
zyga-ubuntuackk: / should be from your base snap14:01
ackkzyga-ubuntu, but can a snap add stuff to them?14:01
zyga-ubuntuackk: with layouts, that's coming soon but not in master yet14:02
ogra_god forbid !14:02
ogra_:)14:02
zyga-ubuntuackk: https://forum.snapcraft.io/t/layouts-re-mapping-snap-directories/147114:02
ackkso what we have for xenial can't work with the base :/14:02
zyga-ubuntuackk: can you expand/explain?14:02
zyga-ubuntuackk: you get to choose the base you want14:02
zyga-ubuntuackk: you switched to bionic, is that not good?14:02
mvoyeah, curious as well what is missing in base-1814:03
zyga-ubuntumvo: I suspect ackk will want /sbin/ip14:03
ackkzyga-ubuntu, mvo yes, the maas snap installs/symlinks a lot of stuff in /bin/ and /sbin, which works in xenial14:03
mborzeckimatiasb: pedronis: sorry, i probably didn't make that clear, i have a local macaroon, id and email in ~/.snap/auth.json (local for snapd), now i login with a different email address assigned to a different sso account, what I see in snapd is the local email not getting updated for the user with that ID in state.json and thus the ~/.snap/auth.json email does not get updated either14:03
zyga-ubuntuackk, mvo: note that the base apparmor template and all the interfaces don't care (know) about bases yet so we may need to do more work to allow -18 specific or -16 specific functionality14:03
zyga-ubuntuackk: how can it work in xenial and not in bionic based snap (I assume it's still a snap and not something else)14:04
ogra_classic ?14:04
ackkzyga-ubuntu, xenial without base and bionic with base yes14:04
zyga-ubuntuackk: is it a classic snap?14:04
ackkno, it's --devmode14:04
zyga-ubuntuackk: in devmode /usr/bin is still a read only squashfs14:04
zyga-ubuntuackk: can you show me what you do?14:04
ackkzyga-ubuntu, https://git.launchpad.net/~ack/maas/tree/snap/snapcraft.yaml?h=snap-bionic-fixes14:05
zyga-ubuntuackk: and where do you put anything in /usr/bin or /bin?14:05
pedronismborzecki: well,   updating makes sense only if you have a valid ID as determined by the local macaroon14:05
ackkzyga-ubuntu, stage-packages: does (specifically because iproute2)14:05
zyga-ubuntuackk: stage packages are unpacked to $SNAP/ no to /14:06
zyga-ubuntuackk: that hasn't changed at all14:06
ackkzyga-ubuntu, https://paste.ubuntu.com/26353268/ (from the prime/ dir)14:06
zyga-ubuntuackk: maybe some path settings are wrong, this should not behave any different than base-18-based snap14:06
zyga-ubuntuackk: but all of the prime dir is mounted at $SNAP (/snap/maas/1234)14:06
zyga-ubuntuackk: you cannot add anything to real /usr/bin14:07
pedronismborzecki: otherwise updating has no clear target14:07
pedronisbut I don't know how that code looks like at all14:07
zyga-ubuntuackk: all I'm saying is that you don't have any new limitations14:07
ackkzyga-ubuntu, mhm, then it might be a path issue14:07
zyga-ubuntuackk: look at $PATH14:07
ackkzyga-ubuntu, I see, then I'll dig further14:07
zyga-ubuntuackk: maybe snapcraft did something before and now you need to do it manualyl14:07
zyga-ubuntu*manually14:07
* ackk checks the xenial snap14:08
ackkzyga-ubuntu, /sbin/ip *is* there in  xenaial core14:14
zyga-ubuntuackk: indeed, my find was ... weird?14:15
ackkzyga-ubuntu, I get permission denied trying to run it with snap run --shell hello-world, but maas can since it's a devmode snap14:15
* zyga-ubuntu wonders why: find /snap-core-current -name ip 14:15
zyga-ubuntudidn't see it14:15
zyga-ubuntuwehh14:15
* zyga-ubuntu wonders why: find /snap/core/current -name ip 14:15
sergiusenshello14:16
ackkzyga-ubuntu, I guess we should really use $SNAP/bin/ip14:17
zyga-ubuntuyes14:17
ackkzyga-ubuntu, IOW prepend $SNAP/sbin $SNAP/bin to our paths14:17
zyga-ubuntuackk: until you can splay $SNAP/bin/ip to /bin/ip with layouts that is14:17
* mborzecki off to pick up the gkids14:21
kalikianasergiusens: o/14:31
* kalikiana going on lunch break shortly14:32
matiasbmborzecki, after taking a look, I see your point: it seems the local/state email information is not updated when you switch to a different SSO account without logout, only the auth bits are updated (ie. the macaroons); pedronis, fyi too <-14:34
mupPR core#71 opened: live-build: make /lib64/ld-linux-x86-64.so.2 a relative link <Created by mvo5> <https://github.com/snapcore/core/pull/71>14:48
elopiogood morning14:53
mborzeckimatiasb: right, i've made a change to update the users's email, the whole diff is something in the lines of https://paste.ubuntu.com/26353503/15:01
mupPR snapd#4460 closed: daemon: store email, ID and macaroon when creating a new user <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4460>15:01
mupIssue snapcraft#1816 closed: snapcraft update won't fetch from `SNAPCRAFT_PARTS_URI` <bug> <Created by sergiusens> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/issue/1816>15:02
mupPR snapcraft#1856 closed: remote_parts: Use hashed folder based on parts URI <Created by kalikiana> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1856>15:02
ackkzyga-ubuntu, mvo  after fixing hardcoded path issues with /sbin/ip, maas works!15:04
ackkquick, ship base-18! :)15:04
matiasbmborzecki, yeah, makes sense to me15:04
mvoackk: ha! lets hope the builders get back online soon15:04
mborzeckimatiasb: thanks15:05
zyga-ubuntuackk: ahead of 18.0415:05
zyga-ubuntuthat'd be a first15:05
zyga-ubuntugood work :)15:05
mvoackk: yeah, nice job!15:06
tedgHmm, okay. I guess builders being down is why the discord snap isn't getting built.15:22
kalikianare15:33
zyga-ubuntuI need a break, I'm so sleepy15:34
zyga-ubuntu16:30 - night15:34
zyga-ubuntuI hate north15:34
kalikianazyga-ubuntu: never go to Finland :-P15:35
zyga-ubuntukalikiana: or just in the summer15:36
* kalikiana likes Finland but struggled with the lack of sun in winter15:36
sergiusenszyga-ubuntu come to South America; this was my weeked -> https://www.instagram.com/p/BdpvFoUFBVH/?taken-by=sergiusens15:44
zyga-ubuntusergiusens: I would never leave and my wife would hate my and my kids would cry15:51
Chipacamvo: that progress spinner doesn't look too good on a linux vt :-(16:01
zyga-ubuntuChipaca: linux vt is a bit limited16:14
zyga-ubuntuChipaca: you get to have 256 characters in two sets16:14
Chipacazyga-ubuntu: i know16:14
Chipacazyga-ubuntu: looks like if it's important, it needs to be ascii for the terminal16:14
Chipacavt i mean16:14
* Chipaca unsure what to do16:15
zyga-ubuntuChipaca: curious, can you tweet a photo please?16:16
zyga-ubuntuChipaca: one more question is "how do you know where you are" (which is super hard to answer actually)16:17
mupIssue snapcraft#1712 closed: Tutorial for circle-ci <Created by sergiusens> <Closed by kyrofa> <https://github.com/snapcore/snapcraft/issue/1712>16:17
mupPR snapcraft#1857 opened: grammar: make on statement work with host arch <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1857>16:17
Chipacazyga-ubuntu: it depends on what font you're using on the terminal; you might get a square, *or*, a rhombus16:17
zyga-ubuntuChipaca: but my recommendation is to forgo any fanciness16:17
* Chipaca takes his bowtie off16:17
zyga-ubuntuChipaca: yes, though on the terminal we could actually ask and even load the right font16:17
Chipacazyga-ubuntu: there is no font with braille :-)16:18
zyga-ubuntuChipaca: ioctl_console(2)16:18
zyga-ubuntuChipaca: my pet project desires me to explore that man page16:18
Chipacazyga-ubuntu: console_ioctl but yes16:18
zyga-ubuntuChipaca: no, it's actually ioctl_console, I have it open16:18
Chipacazyga-ubuntu: in xenial it's console_ioctl(4)16:19
zyga-ubuntuoh, interesting16:19
Chipacazyga-ubuntu: and there is no ioctl_console(2)16:19
zyga-ubuntuI wonder if that's just a rename16:19
zyga-ubuntuat some point systemd wanted to provide a KMS based console with real support for everything16:19
zyga-ubuntubut I think that died or at least isn't deployed16:19
=== [-asc-] is now known as [asc]
kalikianakyrofa: elopio FYI snapcraft#1639 is green now, with the revert - also snapcraft#1857 is the next piece of the puzzle for updated "on" semantics16:20
mupPR snapcraft#1639: grammar: to statement <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1639>16:20
mupPR snapcraft#1857: grammar: make on statement work with host arch <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1857>16:20
kyrofakalikiana, so you don't see the weird test waffling locally anymore?16:20
Chipacazyga-ubuntu: we could make snapd require jfbterm16:20
zyga-ubuntuChipaca: a hard sell for servers16:21
kalikianakyrofa: Neither locally nor on Travis (both hit it before)16:21
zyga-ubuntuChipaca: can we just do /|\- style animation?16:21
Chipacazyga-ubuntu: i prefer .oOo, but, yes16:21
Chipaca(mostly because it bothers me how / and \ don't have the same width)16:22
zyga-ubuntuChipaca: the consequence of that 7 bit space with one bit of unspecified glyphs16:22
zyga-ubuntuChipaca: I wonder if the B-set is populated?16:22
Chipacazyga-ubuntu: showconsolefont16:23
Chipacazyga-ubuntu: answer: yes, but `dpkg-reconfigure console-setup` is all about changing that second half16:23
zyga-ubuntuChipaca: second half as in 9th bit?16:23
zyga-ubuntuChipaca: I don't mean 128..25516:24
zyga-ubuntubut 256..51216:24
zyga-ubuntuI meant greetings BTW16:25
Chipacazyga-ubuntu: it depends on what font you're loading, if you have that or not16:25
zyga-ubuntuhmm16:25
Chipacazyga-ubuntu: but showconsolefont shows you16:25
zyga-ubuntuI'm confused now16:25
zyga-ubuntulook at the thing I tweeted16:25
sergiusensChipaca zyga-ubuntu we took a lot of criticism for our progress bar not working correctly on emacs16:26
mupPR snapcraft#1858 opened: Import run so bin/snapcraft can work <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1858>16:26
Chipacasergiusens: whose progress bar? when?16:26
Chipacasergiusens: in the emacs terminal, or the emacs shell?16:26
zyga-ubuntusergiusens: interesting, so more limitations of the vty thing that emacs supplies16:26
kalikianaelopio: Hey! Can you have a look at snapcraft#1858 ? I found myself very confused to find integration tests don't run against git anymore16:27
mupPR snapcraft#1858: Import run so bin/snapcraft can work <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1858>16:27
Chipacazyga-ubuntu: i did test ansimeter in emacs :-)16:27
zyga-ubuntuit's good that we don't have to test it in vim ;-)16:28
Chipacaah, dunno how to do that :-)16:28
zyga-ubuntuChipaca: since I'm not a emacs person, how do I run emacs and open a terminal to test my stuff?16:28
Chipacazyga-ubuntu: in emacs, M-x term opens a terminal16:28
Chipacazyga-ubuntu: M-x shell runs a shell16:28
sparkiegeekzyga-ubuntu: snap install emacs-tealeg16:29
mborzeckiM-x eshell ftw16:29
sparkiegeekTMTOWTDI :)16:29
mvoogra_: thanks for your suggestion about arm64, what does the ld symlink looks like there?16:29
zyga-ubuntumvo: I think snapcraft has a list of those16:30
Chipacamborzecki: ouch, the ansimeter doesn't fare too well in eshell16:30
Chipacait chooses to do reverse as a colour instead of reverse16:30
sergiusenszyga-ubuntu mvo linkers? yes indeed16:30
zyga-ubuntuyes16:30
zyga-ubuntusergiusens: and learned the hard way for each arch16:31
ogra_mvo, ogra@dragonboard:~$ ls -l /lib/ld-linux-aarch64.so.116:31
ogra_lrwxrwxrwx 1 root root 28 Jun 16  2017 /lib/ld-linux-aarch64.so.1 -> aarch64-linux-gnu/ld-2.23.so16:31
ogra_ogra@dragonboard:~$16:31
sparkiegeekshow_fancy_progress = False if 'EMACS' in os.environ else show_fancy_progress16:31
Chipacamborzecki: eshell also doesn't respect the "turn off the cursor" escape16:31
sergiusenszyga-ubuntu mvo https://github.com/snapcore/snapcraft/blob/master/snapcraft/_options.py#L3116:31
Chipacamborzecki: booo16:31
Chipaca(neither does eterm, fwiw)16:32
ChipacaANYhow16:32
sergiusenszyga-ubuntu mvo and here is the logic to unpack it https://github.com/snapcore/snapcraft/blob/master/snapcraft/_options.py#L22616:32
zyga-ubuntuman, I'm running emacs16:32
* Chipaca hugs the braille spinner, and sends it into the void16:32
mvoogra_: oh, this link is already relative?16:33
sparkiegeekzyga-ubuntu: you think you're running emacs, in reality, emacs is running you16:33
mborzeckiwell, it does have some killer features, org-mode, magit, elisp16:34
mborzeckiwish it was more stable with ediff though, sometimes it just starts spinning and takes 100% cpu :/16:35
sparkiegeekmborzecki: not forgetting the builtin psychotherapist, and  M-x butterfly16:36
zyga-ubuntuwhat is that, I just ran it16:45
Chipacazyga-ubuntu: it randomizes your files, for extra productivity16:46
Chipacathe butterfly effect and all that16:46
elopiokalikiana: yes, weird. I will try to bisect it to see what caused the problem.16:47
mupPR snapcraft#1859 opened: adding option to decompress tar.lzma cleanly <Created by heesen3> <https://github.com/snapcore/snapcraft/pull/1859>16:53
kalikianaelopio: Thanks!16:56
* kalikiana wrapping up for today16:57
ackkmvo, I'm getting a "setlocale: No such file or directory" error message when I run virsh (although the program itself seems to works)16:58
ackkmvo, is it something that also needs adding to the base?16:58
zyga-ubuntuackk: base snaps don't ship any locale16:58
zyga-ubuntuackk: I'd ignore that _for now_; it's a big topic for us to explore IMO16:58
zyga-ubuntuackk: with no (apparently) easy answers16:58
ackkI see16:59
zyga-ubuntuPharaoh_Atem: selinux-policy now builds straight from github17:25
zyga-ubuntumaybe we could merge the snapd policy there and only keep the package for any unmerged TODOs17:25
Pharaoh_Atemmaybe17:25
Pharaoh_Atembut it has to work first17:26
Pharaoh_Atemand it doesn't17:26
Pharaoh_Atemsnapd still needs SELinux awareness17:26
zyga-ubuntuwell, little by little, most software isn't selinux aware even if there's a selinux policy covering it17:27
zyga-ubuntuI think those are orthogonal17:27
Pharaoh_Atemin most cases, they are17:27
Pharaoh_Atemin this case, no17:27
Pharaoh_Atemsnapd itself does security things itself17:27
Pharaoh_Atemmy hope was that the selinux policy would be augmented by snapd itself generating policies to apply with the defined labels17:27
zyga-ubuntuyes but not with selinux so it's not a blocker in my eyes17:27
zyga-ubuntuPharaoh_Atem: apart form the policy we need to figure out FS relableling and nobody is working on that17:28
zyga-ubuntuPharaoh_Atem: lots of things to explore17:28
Pharaoh_Atemwell, I can talk to Lukas about it17:31
zyga-ubuntuworth a try17:32
zyga-ubuntunot saying we have to17:32
zyga-ubuntubut looks like one less thing to package (eventually)17:32
* Pharaoh_Atem shrugs17:32
Pharaoh_Atemwe'd have to keep it for CI testing anyway17:32
zyga-ubuntuyes, I suppose so17:33
zyga-ubuntuPharaoh_Atem: I'm feeling excited today17:35
zyga-ubuntuPharaoh_Atem: my eternal task looks close to being done17:35
zyga-ubuntuPharaoh_Atem: I actually wonder what I'll do next (plenty of things to do but not sure as I haven't discusse that yet017:35
* Chipaca <- stupid poo-face18:03
zyga-ubuntuwhat happened?18:03
mupPR snapd#4450 closed: snap: support `command-not-found` symlink for `snap advise-command` <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4450>18:03
Chipacazyga-ubuntu: twice i brought up an ubuntu-core in linode to test something, before realising that the reason it was failing in a different place than in travis was because i had some stuff lying around here that wasn't in git18:04
zyga-ubuntudoh ;)18:05
Chipacaquite18:05
zyga-ubuntuit's good that we lave linode though18:06
zyga-ubuntuimagine doing that on our laptops 24/718:06
Chipacazyga-ubuntu: http://i.imgur.com/eVNrdel.gifv18:08
mupPR snapd#4462 closed: progress: switch ansimeter's Spin() to use a spinner <Created by chipaca> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4462>18:08
zyga-ubuntulol18:09
zyga-ubuntuI wonder if movie designers have a meme-ist among permanent staff18:09
zyga-ubuntuok, I should _probably_ break now18:10
zyga-ubuntuttyl18:11
Chipacazyga-ubuntu: I know that a lot of regulars on /r/HighQualityGifs work in the industry18:11
zyga-ubuntuI'm glad I don't do reddit18:13
zyga-ubuntuif I did I would have even less live18:14
zyga-ubuntulife18:14
zyga-ubuntuman, see what kind of mistakes I make18:14
zyga-ubuntuChipaca: question, do you have any magic command that shows me coverage in a browser and that updates as I edit code?18:14
zyga-ubuntuI run: ls *.go | entr -c go test18:15
zyga-ubuntubut I'd like that to do live coverage updates18:15
zyga-ubuntuI also have an alias: ggg18:15
zyga-ubuntugo test -cover -coverprofile=coverage.out && go tool cover -html=coverage.out18:15
zyga-ubuntubut that opens new tab in firefox18:15
zyga-ubuntuideally it'd overlay in $editor (but that's fancy)18:15
Chipacazyga-ubuntu: uh18:17
Chipacazyga-ubuntu: maybe? not very magic though18:17
Chipacazyga-ubuntu: go test -coverprofile .coverage/profile.out -v ./snap/squashfs/ && GOPATH=~/canonical/snappy go tool cover -o /tmp/coverage.html -html=.coverage/profile.out18:18
Chipacazyga-ubuntu: and then i open /tmp/coverage.html in the browser18:18
zyga-ubuntuyeah but what would keep it live?18:18
Chipacazyga-ubuntu: "keep it live" no, but it's usually fast enough18:19
Chipacaand because i'm explicitly telling it to write to that file it doesn't open a new page every time, which means it stays on the same file18:19
Chipacabeyond that, no18:20
Chipacazyga-ubuntu: maybe goland does that?18:20
Chipacazyga-ubuntu: (snap install goland?)18:20
zyga-ubuntuoh18:20
zyga-ubuntuI'll look, just curious18:21
Chipacazyga-ubuntu: https://www.jetbrains.com/go/features/ says "If you run your code with a coverage instruction, the IDE collects the data and displays it in both the aggregated view and per statement in the Editor."18:21
zyga-ubuntuooooh18:21
zyga-ubuntuman18:21
zyga-ubuntuI will probably sell vim18:22
zyga-ubuntuand get into one of those18:22
Chipacapedronis: hah! i was wondering about that failure18:24
Chipacapedronis: couldn't you also have fixed it by feeding jq from stdin instead of asking it to open the file?18:24
zyga-ubuntuChipaca: which failure?:18:37
sergiusenskyrofa elopio hey, I have updated snapcraft#1850 can you please take another look?18:37
mupPR snapcraft#1850: pluginhandler: patch and handle elf files on glibc mismatch <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1850>18:37
pedronisChipaca: yes, but we do install with --devmode elsewhere18:43
pedroniswhat I see true is that given the change I could have simplified some other lines18:49
pedronisChipaca: prepare.sh ,  auto-refresh/task.yaml and this have that choice to make18:51
sergiusenselopio would be nice if you follow up on snapcraft#184918:58
mupPR snapcraft#1849: tests: add snap not found tests <codein> <Created by daniellimws> <https://github.com/snapcore/snapcraft/pull/1849>18:58
=== devil is now known as Guest15211
naccshould /var/lib/snapd/hostfs have contents, generally?19:15
zyga-ubuntunacc: it dpeends on one's perspective19:18
zyga-ubuntunacc: on your regular host it should be empty19:18
zyga-ubuntunacc: from a snap's POV it should contain your hosts' root filesystem (with some exceptions)19:19
zyga-ubuntunacc: naturally confinement is in the way so you won't see that unless in devmode19:19
zyga-ubuntunacc: but some things are "brought over" from there (notably fonts)19:19
nacczyga-ubuntu: got it19:22
zyga-ubuntunacc: :-)19:50
kyrofasergiusens, I'm testing out this libc branch20:03
sergiusenskyrofa great; I tried with https://pastebin.ubuntu.com/26355121/ from bionic20:04
kyrofaAnd I'm realizing is_linker_compatible isn't quite doing what I thought it was doing. It's comparing the version of the linker against the glibc version, which is vastly different, is it not?20:05
sergiusenskyrofa the linker is part of glibc20:05
kyrofaI ran it on zesty, which uses gcc 2.4, and it compared that against 2.23 and decided it was happy with it20:05
kyrofaNot gcc, sorry20:05
kyrofaI expected it to grab the newer glibc, but it didn't20:06
sergiusenskyrofa zesty didn't bump the glibc requirement, that's why all our adt tests still pass there :-)20:06
sergiusensyou need artful or bionic to trigger this20:06
kyrofaArgh20:06
sergiusenskyrofa 2.23 linker is still good for the linker in zesty (even if the version is greater).20:07
sergiusenskyrofa if everything is correct, adt for bionic and artful will pass with glaring colours20:08
kyrofaMan, my connection to the image server is brutal today. I'm getting like 20k20:09
zyga-ubuntucurious, I saw uber-slow 90K to archive.u.c20:11
sergiusenskyrofa tough luck I guess; hope you like my approach to testing with runnables btw; I think I want us to move down that path for every other thing we have20:31
kyrofaSeems python-apt requires libdpkg-perl on artful20:31
kyrofasergiusens, I do indeed. More work upfront, but definitely testing more20:33
sergiusenskyrofa heh, that dep rings a bell; oh, when installing from debs; probably a bad dep as well; I saw that too20:49
* sergiusens will bbl later in the evening20:49
=== Guest15211 is now known as devil
kyrofasergiusens, I was using a venv, so we'll just need to add it to HACKING.md20:49
=== devil is now known as Guest96253
kyrofaSeems that it needs to access /usr/share/dpkg/no-pie-compile.specs20:50
=== Guest96253 is now known as devil_
=== devil_ is now known as devilz
sergiusenskyrofa yes; I ran into that as well, I recall now21:08
* sergiusens decided to not leave21:09
sergiusenskyrofa the only reason I didn't move it into fixtures_setup is because it is so big and I believe it has the wrong name21:10
sergiusenskyrofa I suspect travis has kernel patches for Meltdown and is the reason for the timeouts we see now?21:14
kyrofasergiusens, hmm, you think it has that much of an impact? That would suck21:15
kyrofaI didn't anticipate that it would be much21:15
kyrofaAlso... WE don't even have mitigation yet. How do they?21:15
sergiusenskyrofa I thought GCE already patched and java took the biggest hit; coincidentally we timeout while building with ant in one task and dotnet in another (which I suspect would take a similar hit)21:18
sergiusensI haven't done any deep analysis or anything though so refuting what I say should be rather easy ;-)21:19
kyrofaInnnteresting...21:20
=== blunden_ is now known as blunden
mupPR snapcraft#1858 closed: Import run so bin/snapcraft can work <Created by kalikiana> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/1858>21:54
mupPR snapcraft#1859 closed: adding option to decompress tar.lzma cleanly <Created by heesen3> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/1859>21:57
nacci forget who suggested it to me, but the idea of bind-mounting writable directories over a classic snap's filesystem is going to speed up debugging incredibly for me22:59
mupPR snapcraft#1860 opened: setup: simplify bin/snapcraft and correct tests <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1860>23:12
sergiusenskyrofa elopio ^23:13
sergiusenskyrofa snapcraft.tests.integration.plugins.test_ant_plugin.AntPluginTestCase.test_build_ant_plugin takes 15 minutes for me locally. Mind checking?23:14

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!