/srv/irclogs.ubuntu.com/2018/10/23/#snappy.txt

mborzeckimorning05:01
zygaHey05:31
zygamborzecki: mvo asked for release reviews today05:31
mborzeckizyga: hey, ok05:32
mborzeckizyga: do you know what's the status of https://github.com/snapcore/snapd/pull/5845 ? iirc it was waiting for a review from jdstrand mostly05:33
zygaIt also needs a new name from Gustavo05:35
zygaI need to take Bit to the vet for surgery today but that should take a moment05:37
zygaIโ€™ll be in the office shortly05:37
mborzeckizyga: https://paste.ubuntu.com/p/3ZwGpF6ZyT/ something i found on the laptop06:17
mborzeckizyga: lrwxrwxrwx 1 root root 13 10-23 08:15 /var/lib/lxd -> /mnt/data/lxd06:17
mborzeckisince we test for access() removing the symlink makes it work06:18
zygaHmmm06:48
zygaYeah, the error message need improvements06:57
=== pstolowski|afk is now known as pstolowski
pstolowskimornings07:04
mborzeckizyga: can you take another look at https://github.com/snapcore/snapd/pull/6023 ?07:18
mborzeckipstolowski: pedronis left a suggestion in https://github.com/snapcore/snapd/pull/602707:24
pstolowskiyep, looking07:25
zygaOk07:34
zygaDog at vet07:34
zygaTwo hours after surgery for pickup07:34
zygaFingers crossed it goes well07:34
zygamborzecki: two, in 2 minutes, just going home07:35
jdrabpani, VCERA SME PREMESKALI MEDZINARODNY DEN CAPS LOCKU!07:39
zygare07:39
zygauh?07:39
zygamborzecki: looking now07:39
mborzeckiyday was the national day of cap lock?07:40
jdraboh sry, wrong channel :D07:41
zygamborzecki: thank you for the link about Khan's Algorithm07:42
zygaI wasn't aware of it, cool07:42
zygajdrab: nie szkodzi07:42
zyga:)07:42
jdrabahaha :)07:42
mborzeckizyga: yeah, and it's surprisingly simple once you understand it07:46
seb128hey there07:49
seb128$ snap changes07:49
seb128ID    Status  Spawn                      Ready  Summary07:49
seb1281462  Doing   4 days ago, at 09:21 CEST  -      Auto-refresh snaps "core", "core18", "gnome-logs"07:49
seb128is that "4 days ago" normal?07:49
seb128or is snapd stucked in some way?07:49
mborzeckipstolowski: ^^07:49
pstolowskiseb128: snap version?07:50
seb128$ snap version07:50
seb128snap    2.36~pre2+git965.966c1d4~ubuntu16.04.107:50
seb128snapd   2.36~pre2+git965.966c1d4~ubuntu16.04.107:50
seb128series  1607:50
seb128pstolowski, ^07:51
mborzeckipstolowski: you think that could be auto-connect?07:51
pstolowskiseb128: ok, and snap change 1462?07:51
seb128that takes a bit...07:52
seb128pstolowski, http://people.canonical.com/~seb128/log07:53
zygaabout that07:54
zygaI think we could use RW locks on state07:54
zygaso snap changes is fast07:54
zygaunless I misunderstand and the delay is because of the volume of data07:54
mborzeckicrazy thought, but should we try to detect that stuff is getting retried indefinitely and perhaps log && abort?07:55
zygaI think we do that in a way07:55
pstolowskiseb128: thanks. so yes, it's a known bug we've right now in master/edge, fix is being reviewed07:55
zygavery old tasks get aborted07:55
zygaor perhaps we meant to07:55
zygaI'm not super familiar with that code07:56
pstolowskiseb128: as a workaround you can snap abort the change, and then remove all unused revisons of gnome-logs (snap remove --revision=....) to only leave one. alternatively, switching core to stable (after aborting the change) might help07:57
zygamborzecki: I will follow up with a validator for topological sorting in the mount backend07:58
zygaat least we must detect cycles07:58
mborzeckizyga: ping me for review07:58
zygawith pleasure :)07:58
seb128pstolowski, ok, thanks, I didn't even remember that I changed the core to a non stable channel on that box :)07:58
ackkkyrofa, WRT --destructive-mode, it seems it currently only works if you don't pass a comand to snapcraft? if I try to "snapcraft prime --destructive-mode" it tries to use multipass08:02
ackkis that expected?08:02
mborzeckizyga: pstolowski: if i'm reading that right, the task would be auto aborted after 7 days08:04
zygathat's about right08:04
zygait's a lot of time but it was meant to be conservative08:04
pstolowskiseb128: which is good, thanks to one other report about this we avoided releasing it (it happens under specific circumstances)08:04
pstolowskimborzecki: zyga, right, we will not eat cpu indifinately or anything... but the problematic change will keep re-appearing and will never succeed (auto-refresh in this case)08:05
zygayeah08:06
zygawe may have a fallback mode08:06
mborzeckieat cpu and fill up the state08:06
zygaif we fail to refresh and abort after 7 days08:06
zygawe could do something special, like just refresh core - if installed, or snapd - if installed08:06
zygaand nothing else08:06
mborzeckiiff we do healtchecks, this could be part of it08:06
pstolowskiseb128: so, thanks for using of edge, please switch back to it if possible after the fix lands, you guys have more interesting use cases than anyone else, so better chances of finding potential problems08:09
mborzeckiif nobody minds i can push the fixes to https://github.com/snapcore/snapd/pull/5727 that jdstrand requested08:09
zygamborzecki: +108:10
seb128pstolowski, np, thx for replying/getting me unstuck, your step worked08:10
pstolowskizyga: yes, this sounds like a good idea08:14
zygamborzecki: can we land ? https://github.com/snapcore/snapd/pull/534608:27
mborzeckizyga: let me finish with the changes for breakpad and i'll take another look at it08:34
zygait's +2 and green08:34
zygabreakpad?08:34
zygaah08:34
zygamozilla08:34
zygasure08:34
mborzeckidamn, hate running tests in snap-seccomp08:35
ogra(assuming that GH behaves again i un-quietened mup again)08:35
zygamborzecki: on your laptop?08:43
mborzeckizyga: anywhere, it's making the modules for all those crazy AF_* loaded08:44
zygaah right08:44
mborzeckizyga: looking at the snap:// pr08:50
zygaI wonder if it's ok to just merge08:50
zygamorning Chipaca09:00
Chipacamorning zyga09:00
mupPR snapd#5985 closed: overlord/many: cleanup use of snapName vs. instanceName <Parallel installs โ›“> <Simple ๐Ÿ˜ƒ> <Created by bboozzoo> <Merged by zyga> <https://github.com/snapcore/snapd/pull/5985>09:02
zyganeed to go to get the dog from the vet09:15
zygasurgery over09:15
zygabrb09:15
pstolowskizyga: hmm, interesting issue from popey on the forum, 'snap interfaces' and conns state has the connection, but gnome snaps complain about lack of it; i've vague memory of something similiar before, do you remember?09:15
pstolowskizyga: and it's 2.35.409:16
pstolowskizyga: was there something about profile generation you were investigating some time ago?09:17
pstolowskiit's https://forum.snapcraft.io/t/cant-connect-interfaces-so-cant-run-snaps/812309:18
mupPR snapd#5346 closed: cmd/snap: gnome-software install via snap:// handler <Created by jhenstridge> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/5346>09:18
Chipacaupdate sequencing makes my head hurt, still09:24
Chipacamborzecki: the snapcraft static suite doesn't like spread-shellcheck :-(09:28
Chipacamborzecki: I reformatted it with black, which passed the first hurdle, but now its flake8 is saying it's all too complicated09:29
Chipacazyga: at what  point are cmd/snap-confine/spread-tests/ run?09:30
zygaNever, those are the old ones from split repo09:33
zygaProbably more stale than ever09:33
zygapstolowski: still afk at vet, will be home in 10 min09:33
mborzeckiChipaca: heh, we can feed spread-shellcheck to snapcraft, have them fix it to be nice python, and merge it back09:34
Chipacamborzecki: *cough* snapcraft#2381 *cough*09:34
mupPR snapcraft#2381: tools: copy in spread-shellcheck from snapd <Created by chipaca> <https://github.com/snapcore/snapcraft/pull/2381>09:34
zygapstolowski: can you please ask for mount profiles09:34
zygaCurrent and desired09:34
mborzeckibtw. are we having any talks about ubuntu core & snaps at ELC?09:36
mborzeckiChipaca: heh, checkfile is too complex, it's the cyclic complexity check right?09:37
Chipacamborzecki: "aaah this is too complicated i give up"09:37
Chipacamborzecki: I imagined flake8 slamming the program printing down and going off in a huff09:37
mborzeckiChipaca: i have some expierience working around gocyclo :P09:37
mborzeckiwonder if that applies to python too09:38
mborzeckiChipaca: probably dropping some ifs would help09:38
Chipacamborzecki: I was going to just leave it there and see if snapcrafters would push a fix :-)09:38
mborzeckiChipaca: hm ther's if > for > for > if sequence09:39
Chipacai'm going to have to figure out why flake8 locally complains about everything, making it pointless, aren't i09:40
Chipacasigh09:40
* Chipaca remembers the snapcraft venv, and tries that09:40
Chipacamborzecki: got it09:41
Chipacamborzecki: i'll fix09:41
mborzeckiChipaca: heh, it's fun when you apply an artificial metric and thresholds and then workaround to meet the them :)09:42
pstolowskipopey: allright, i'll ask you to collect a bunch of other info in a moment09:48
popeypstolowski: ok09:49
zygare09:50
zygadog safely home, still numb. but alive :)09:50
pstolowskizyga: are the *fstab files from the host os useful, or only those from the mount namespace?09:51
zygapstolowski: we want a pair of files: /var/lib/snapd/snap.$SNAP_NAME.fstab and /run/snapd/ns/snap.$SNAP_NAME.fstab09:52
Chipacamborzecki: http://paste.ubuntu.com/p/6hmN9bKR6j/09:53
pstolowskizyga: k, thanks09:54
zygathank you, let me know if you get more data09:55
mborzeckiChipaca: does that bring the complexity count down?10:00
Chipacamborzecki: it brings it down to 9, apparently10:03
Chipacano, 1010:03
mborzeckiChipaca: lgtm10:03
Chipacamborzecki: from 12 or 14 that it was10:03
Chipaca1210:04
Chipacamborzecki: ok, i'll push it10:04
Chipacaof course now black doesn't like it10:04
Chipaca:-)10:04
zygapstolowski: the .fstab files are available in all namespaces10:08
zygayou don't need to be in a specific spot to read them10:08
pstolowskizyga: yep, i figured by md5'suming them just to be sure ;)10:09
pstolowskipopey: ty! zyga, all the info there10:29
threshis there a way to make firefox in a snap to use the same mouse pointer as system-wide?10:30
zygapopey, pstolowski: responded10:32
zygathresh: kenvandine would know I suspect10:32
ograyou can surely fake it if you ship the cursor theme and force theme defaults for your app10:33
pstolowskizyga: thanks, interesting (whether the script does the right thing)10:33
ograbut thats indeed only making it a look-alike and only works as long as the default of the system has not been changed10:33
popeypstolowski: fyi. I just did ctrld to exit that thing you had me enter. The second I did that something segfaulted and I lost  my entire desktop10:35
zygapopey: !!10:35
pstolowskiwoot10:35
zygacan you please collect some logs10:35
zygaI saw something similar on my machine once10:35
zygaI was inside a mount namespace entered via nsenter10:35
threshwell, me being on KDE and using breeze/dark will only make it more complex with firefox being a gtk app10:35
zygaI hit exit and my desktop blinked10:36
zygaI was surprised because as far as we're concerned that's just bash quitting10:36
threshbut at least having the mouse pointer scaled 2x would already help10:36
pstolowskipopey: sorry! unintended and unexpected10:36
zygapopey: are you on Wayland?10:36
ograthresh, well, my suggestion above would also not scale it ... just set a shipped default so you dont end up with the x11 pointer10:36
popeyNo. X1110:37
zygahmmm10:37
zygacollect logs please, let's try to understand what crashed10:37
popeyogra: thresh mouse cursors are on kenvandine radar10:38
ogra(though you could probably write a complex wrapper, read the dpi, force the theme up in scale etc etc ... but thats super hackish and pretty overkill)10:38
ogra(pretty much what ken will eventually do in a clean way as popey says :) )10:38
threshgotcha10:39
threshthanks!10:39
popeyWe shouldn't work around this but fix it10:39
ograpopey, we should ... since 2 years :P10:39
ograif only the day had more hours10:39
popeyhttp://paste.ubuntu.com/p/sQtP7vmXPY/10:39
ogra:D10:39
popeyDmesg from my exploded session10:39
zygasome things crashed on X10:40
zygabut I suspect X disconnected10:41
zygaand those things died as a consequence10:41
zygapopey: can you look for logs of X10:41
zyganot sure how yet10:41
popeyYeah10:41
cachiopstolowski, hey,10:42
zygaTBH I'm not sure how X is started or managed nowadays10:42
pstolowskihey cachio !10:42
cachiopstolowski, did it work the change10:42
cachiocould you try it?10:42
pstolowskicachio: I think it didn't, i'm just giving it another try10:43
zygafor instance it _seems_ that gdb runs everything10:43
zygagdm then has gdm-session-worker as a child10:43
cachiopstolowski, do you have the command line you used to start the vm?10:43
zygathen gdm-x-session10:44
zygaperhaps X is hidden there10:44
pstolowskicachio: yes i've. let's talk in a private channel10:44
zygabut no idea really10:44
zygapopey: can you work with someone from the desktop team to help us understand this, if you can reproduce this the better10:44
zygapopey: do you have Nvidia drivers?10:44
popeyI do10:44
popeyhttp://paste.ubuntu.com/p/VxM3bjHq3K/10:45
popeySyslog10:45
zygapopey: I see10:45
popeyNeed to reboot to actually do some work10:45
zygaOct 23 11:30:43 hal /usr/lib/gdm3/gdm-x-session[2062]: (**) Option "fd" "84"10:46
zygaOct 23 11:30:43 hal /usr/lib/gdm3/gdm-x-session[2062]: (II) event3  - Power Button: device removed10:46
zygaI think this is udev input triggering10:46
zygabut no idea why it would happen10:46
ogrado interfaces still call udevadm tigger ?10:47
ograiirc we had some doing that10:47
zygaOct 23 11:30:39 hal org.gnome.Shell.desktop[2197]: Window manager warning: last_focus_time (773055215) is greater than comparison timestamp (773055214).  This most likely represents a buggy client sending inaccurate timestamps in messages such as _NET_ACTIVE_WINDOW.  Trying to work around...10:48
zygathen all hell breaks loose it seems10:48
zygaogra: regardless of what happens in udev - in this case we were not using interfaces10:48
zygapopey just closed bash10:48
ograouch, ok10:48
popeywell, i closed the command I had opened in bash10:48
zygapopey: right10:49
zygait should not have happened10:49
zygabtw, I have the same mouse :)10:49
popey:)10:49
zygaOct 23 11:30:46 hal /usr/lib/gdm3/gdm-x-session[2062]: (EE) NVIDIA(GPU-0): Failed to acquire modesetting permission.10:49
zygathis is interesting10:49
zygabut TBH I think the X + Nvidia combo is something someone needs to look into10:50
* zyga has no nvidia10:50
zygapopey: do you think it would help if I had a GPU at home?10:50
zygaI could stick it into my test desktop for cases like that :/10:50
popey๐Ÿคทโ€โ™‚๏ธ10:50
mborzeckiwhat what nvidia?10:50
zygapopey: Oct 23 11:30:46 hal /usr/lib/gdm3/gdm-x-session[2062]: (EE) Please also check the log file at "/var/log/Xorg.1.log" for additional information.10:50
zygado you still have that log files?10:50
zygaperhaps X log has more fun stuff10:50
zygaOct 23 11:30:46 hal /usr/lib/gdm3/gdm-x-session[2062]: (WW) NVIDIA(0):  - Setting a mode on head 0 failed: Insufficient permissions10:51
zygaOct 23 11:30:47 hal nautilus[13429]: nautilus: Fatal IO error 11 (Resource temporarily unavailable) on X server :1.10:51
zygaso my theory is as follows:10:51
zygasomething is broken in the desktop session and your ctlr+d went to X10:51
zygaX got nuts and went belly up10:51
zygaapps died making lots of noise when X disconnected10:52
zygaI think a  test case is to:10:52
zygaopen bash just like you had befor10:52
zyga(nsenter and such)10:52
zygaand ctrl+d10:52
zygaperhaps open the same apps10:52
zyganot sure if there's a catalyst that makes this happen10:52
zygaremember when ctrl-c was killing gdm?10:52
zygabecause overzealous input somewhere10:53
mborzeckipopey: what did you do?10:53
popeygood question10:54
mupPR snapd#6030 opened: cmd/snap: tweak `snap services` output when there is no services <Simple ๐Ÿ˜ƒ> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6030>10:55
mborzeckipopey: i mean, what command did you run? i don't see it in the log10:55
popeysudo nsenter -m/run/snapd/ns/gnome-calculator.mnt10:56
zygaso you were root10:56
popeyand now I have rebooted, the snaps that didn't work, now work10:56
ograzyga, popey https://bugs.launchpad.net/ubuntu/+source/gdm3/+bug/171063710:56
zyga(not that it would explode because you were root and wrote ctrl+d)10:56
popeyi cant see that ogra10:56
* zyga can10:57
zygabut that was fixed in 201710:57
ograthat was the "ctrl+c (or +d) kills gdm" bug10:57
zygathat's the bug where Ctrl-c goes to the wrong place10:57
zygabut10:57
zygapopey: I'll add you there10:57
zygacan you read the last few comments10:57
zygadone10:58
zygathere's a command10:58
zygawait10:58
zyga50310:58
zyga:/10:58
zygastill 50310:58
mborzeckiheh, pefect timing10:58
mborzeckizyga: btw. that PR ^^ hopefuly Chipaca and degville can help come up with a message that makes sense :)10:59
zygapopey: ok, as a quick reproduce test10:59
popeyNote, I haven't restarted my system for 15 days before this happened10:59
popeyI seem to see all these odd issues because I have "long" uptimes10:59
popeystuff seems to get itself well wedged as a result10:59
zygapopey: run as root: udevdm trigger; udevadm settle --timeout=1011:00
zygathis _may_ trigger bad stuff11:00
zygaso mind the abyss11:00
zygathen try the nsenter thing again11:00
zygaI have a feeling we should stop using udev rules for tagging11:01
zygaand use croups11:01
zygacgroups11:01
ogracroups ?11:01
zygamoronic spellchecker11:01
ogracroutons ?11:01
ogra:)11:01
zygacre^p du kernel ;)11:01
ogra:D11:02
popeyYes11:02
popeyThat did bad things11:02
popeyDesktop exploded11:02
zygaperfect11:02
zygapackage this in a bug report11:02
popeyThanks11:02
zygaalong with system version, the driver you had (version) and perhaps the make of the card11:02
zygawe will take it from there11:03
popeyAgainst gdm3?11:03
zygaand I'm very sorry for the inconvenience this is causing11:03
popeyOr snapd11:03
zygaagainst snapd first11:03
popeyKk11:03
zygaso that we can alter the status11:03
zygawe can affect others as we discover11:03
popeyWill do11:03
zygawoot!11:03
popeyNice one11:03
* zyga inserts mental model of a plane falling on fire with two engineers watching from the ground, congratulating each other, one saying "woot, you were right, it was the fuel injection pump" 11:04
popeyThis is separate from the original issue though, right :)11:04
zyga"reproduced"11:04
zygaI think so11:04
zygapopey: I don't have recent enough hardware but I will see if this happens without nvidia11:05
zygaand then buy a card for testing11:05
zygaand upcoming chunk of Nvidia work anwy11:05
zygawooot11:10
zygareproduced like hell11:10
zygapopey: no nvidia neede11:10
zygawe need to get off of udev11:10
zygatouching udev is bad11:10
zygahere's my login screen11:11
zygaI think willcooke is on a sprint now, right?11:11
popeyhttps://bugs.launchpad.net/snapd/+bug/179943311:12
mupBug #1799433: GNOME desktop on nvidia crashes when exiting nsenter <snapd:New> <https://launchpad.net/bugs/1799433>11:12
zygapopey: funny, in my case something crashed, desktop blinked but my programs were intact11:12
zygaI logged in and it was back11:12
popeyhah11:12
zygabut it's clearly broken11:12
popeyI am happy for you :)11:12
diddledanOT https://github.com/features/actions11:13
pstolowskizyga: see my comment re that problem11:14
zygapstolowski: replied11:16
pstolowskizyga: ah, sorry, let me remove this not to bring confusion11:16
mborzeckizyga: it does not reproduce here, i mean udevadm trigger && settle and nsenter11:21
mborzeckizyga: popey: nope, maybe i'm doing something wrong11:23
zygamborzecki: we're talking in the #ubuntu-desktop channel as well11:28
zygasame, I cannot reproduce it again11:28
zygamaybe I was just very lucky11:28
zyga(that one time)11:28
mborzeckizyga: popey: i'm looking through my irc logs, but i recall we discussed a similar case when udevadm trigger with nvidia would bring down the whole session11:31
popeyyeah11:31
zygamborzecki: note, I was not on nvidia11:31
popeythis is familiar11:31
mborzeckipopey: do you remember what that was?11:32
popeynot without grepping logs11:32
popeyhttps://forum.snapcraft.io/t/weird-udev-enumerate-error/2360/911:33
mborzeckipopey: heh, grepping is what i do, i really wish i had that indexed by xapian or sth11:33
=== chihchun is now known as chihchun_afk
* zyga lunches11:50
mborzeckipopey: zyga: i think last time we discussed this: https://bugs.launchpad.net/ubuntu/+source/nvidia-graphics-drivers/+bug/176010411:52
mupBug #1760104: Xorg crashed with SIGSEGV in InputReady() from ospoll_wait() from InputThreadDoWork() <amd64> <apport-crash> <bionic> <compiz-0.9> <single-occurrence>11:52
mup<ubuntu> <snapd:Invalid> <nvidia-graphics-drivers (Ubuntu):Confirmed> <xorg-server (Ubuntu):Confirmed> <https://launchpad.net/bugs/1760104>11:52
mborzeckioff to pick up the kids11:53
zygaindeed11:53
zygainteresting11:53
* pstolowski lunch12:21
dot-tobiasHi greyback, may I ask if you found the time to identify the cause for the rotation issue in chromium-mir-kiosk (https://discourse.ubuntu.com/t/snaps-to-develop-a-web-kiosk-on-ubuntu-core-using-wayland/6424/87) when rotating mir-kiosk to portrait mode (https://community.ubuntu.com/t/display-configuration-for-mir-kiosk/7815/3)? I'm currently building the snap with an altered i3 config which sets default_orientation = vertical. Does not seem to help,12:31
dot-tobias except I get a short i3 nagbar โ€œyou have an error in your configuration fileโ€, but I cannot figure out what the error is.12:31
ogradot-tobias, did you try shipping xrandr and use its rotation feature ?12:32
greybackdot-tobias: hey, my apologies but I've been focused elsewhere.12:32
ograalternatively i think you would actually need a mir config to force the rotation12:32
ograhttps://community.ubuntu.com/t/configuring-mir-kiosk-a-masterclass/815012:33
greybackogra's idea is worth a try, but I suspect it is i3 itself that isn't dealing with the X display rotation12:33
greybackalan_g looked into it and it did appear that Xwayland was being correctly told by mir that the display had rotated12:33
dot-tobiasogra: Not yet, was hoping I could achieve this through config files. Adding a layout for mir-kiosk rotates just fine, but chromium-mir-kiosk (which uses i3) does not pick up on it.12:33
ograouch12:33
greybackdot-tobias: as (lousy) workaround, could you make i3 size the chromium window in pixels at the size you desire?12:34
ograi'll hit that myself soon, working on https://snapcraft.io/magicmirror to build a pre-configured appliance image :)12:34
dot-tobiasgreyback: Yeah, that workaround is building right now on my Pi ๐Ÿ˜Š But good to hear that would actually work.12:35
ograuh, you build on your pi ?12:35
ogrause build.snapcraft.io ;)12:35
ogra(it might fail to release, but you can still directly download the bult snap from there)12:35
ogra*built12:35
dot-tobiasI use launchpad in parallel, but we're hosting everything on Gitlab.com which is currently not supported by build.snapcraft?12:36
greybackdot-tobias: it shoud work, afaics i3 just isn't sizing the chromium surface by flipping the width/height12:36
ograah, right12:36
ogralaunchpad offers git imports into local LP trees ... you might be able to use that with gitlab, not sure12:37
dot-tobiasogra: That's actually exactly what I used, but good to hear I went in the right direction ๐Ÿ˜Š12:39
ograheh12:39
mupPR snapd#6031 opened: snap/pack, cmd/snap: allow specifying the filename of 'snap pack' <Created by chipaca> <https://github.com/snapcore/snapd/pull/6031>12:44
mupPR # closed: snapd#5170, snapd#5644, snapd#5696, snapd#5712, snapd#5714, snapd#5727, snapd#5789, snapd#5792, snapd#5822, snapd#5845, snapd#5885, snapd#5887, snapd#5897, snapd#5915, snapd#5916, snapd#5946, snapd#5954, snapd#5955, snapd#5958, snapd#5962, snapd#5963, snapd#5972, snapd#5981,13:18
mupsnapd#5982, snapd#5987, snapd#5996, snapd#6000, snapd#6008, snapd#6010, snapd#6016, snapd#6019, snapd#6023, snapd#6025, snapd#6027, snapd#6028, snapd#6030, snapd#603113:18
mupPR # opened: snapd#5170, snapd#5644, snapd#5696, snapd#5712, snapd#5714, snapd#5727, snapd#5789, snapd#5792, snapd#5822, snapd#5845, snapd#5885, snapd#5887, snapd#5897, snapd#5915, snapd#5916, snapd#5946, snapd#5954, snapd#5955, snapd#5958, snapd#5962, snapd#5963, snapd#5972, snapd#5981,13:19
mupsnapd#5982, snapd#5987, snapd#5996, snapd#6000, snapd#6008, snapd#6010, snapd#6016, snapd#6019, snapd#6023, snapd#6025, snapd#6027, snapd#6028, snapd#6030, snapd#603113:19
* Chipaca hugs mup13:24
mupIssue core18#86 opened: Ubuntu 18.10 <Created by kravietz> <https://github.com/snapcore/core18/issue/86>13:29
diddledandid mup just vomit?13:58
diddledanlots of "PR # closed"13:58
ograit rather "released a constipation"13:59
diddledanaha. stinky mup!13:59
dot-tobiasogra: (late reply re: rotation & mirror applkiance after AFK) BTW using display_rotate and dtoverlay=vc4-fkms-v3d (https://github.com/michmich/magicmirror/wiki/configuring-the-raspberry-pi#enable-the-open-gl-driver-to-decrease-electrons-cpu-usage) works most of the time with mir-kiosk, but it produced some funny rendering issues and the Pi heats up considerably. Had two temp-induced reboots,14:10
mupPR snapd#5981 closed: interfaces/many: updates to support k8s worker nodes <Created by jdstrand> <Merged by zyga> <https://github.com/snapcore/snapd/pull/5981>14:26
Chipacahmmm14:27
Chipacai should probably have lunch14:27
ograisnt that linner already ?14:27
ogradot-tobias, fkms is 100% SW rendering, it completely turns off the GPU14:28
ograyour board will overheat if you run it for a while14:28
dot-tobiasogra: that's what I thought, and observed โ€ฆ I'll test the i3 resizing and/or xrandr route once I'm back at the office.14:29
ograeither use the kms driver and gallium or somehow get the closed source driver to work ... but i doubt thats possible without also patching the kernel14:29
ograsadly using kms means no rotation before mir comes up (i.e. splash screen)14:32
zygamborzecki: could you please review https://github.com/snapcore/snapd/pull/601014:33
mupPR #6010: cmd/snap-discard-ns: add support for per-user mount namespaces <Created by zyga> <https://github.com/snapcore/snapd/pull/6010>14:34
zygayou did in the past but not fully (vote vote vote)14:34
zygamborzecki: I'll review your ordering PR next, I stopped on Khan before and then it got lost in a tab swarm14:38
dot-tobiasogra: is that what you mentioned about the missing /dev/fb0 in your PR @ https://github.com/snapcore/pi3-gadget/pull/13 ? Or unrelated? I could live with an always-portrait splash for now ๐Ÿ˜Š14:43
mupPR pi3-gadget#13: add splash support <Created by ogra1> <https://github.com/snapcore/pi3-gadget/pull/13>14:43
ogradot-tobias, the missing framebuffer is caused by the vc4 module not being in the initrd ...14:44
ogradot-tobias, i have a workaround in my pi-kiosk gadget14:45
ograhttps://github.com/ogra1/pi-kiosk-gadget14:45
ograand the next kernel should actually even add the modules to the initrd by default so only the modprobe call might be needed (if at all) ... i'm not sure where we stand with that though ... ppisati is your man14:46
tomwardillhello! another question about pre-refresh hooks, which version of the hook do they execute? The installed version or the about-to-be-installed one?14:47
tomwardillthe docs here https://docs.snapcraft.io/supported-snap-hooks/3795 aren't entirely clear14:47
ograif you'd do the installed one, your fix to that hook would only apply in the next update ...14:47
ograso i guess you can assume it is the about-to-be-installed one14:48
dot-tobiasogra: I have your pi-kiosk gadget on my todo/inspiration list for exactly that usecase ๐Ÿ˜Š Thank you for your work btw!14:48
ogranp :)14:48
tomwardillogra: thanks :)14:51
GargoyleIs there any way to accelerate the launch of snap apps?14:51
ograbuy a faster disk ? :)14:52
zygaGargoyle: snaps launch very very quickly - what is slow are parts of the desktop stack, such as fontconfig, that is unique to each snap14:52
zygaGargoyle: work on fontconfig should improve that situation to the point where perhaps a shared cache can be used?14:52
ograspecifically at the foirst start of a desktop snap you have a lot of stuff being copied around14:53
zygaGargoyle: to see how fast snaps run install snapd-hacker-toolbelt and time it14:53
ogra(and generated etc)14:53
Gargoyleahh so the 15 seconds it takes atom to launch it doing things like updating the fontconfig cache each time?14:53
zygaI don't think it is each time14:54
diddledandon't you love when builds are long: https://usercontent.irccloud-cdn.com/file/u3SMBRXX/image.png14:54
mupPR snapcraft#2382 opened: unit tests: missing full adapter in extension test <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/2382>14:54
zygabut it should be per ... something14:54
zyga(per revision perhaps?)14:54
zygabut I really don't know14:54
zygait needs per-snap analysis14:54
zygathough I believe the overhead from desktop helpers is commonly known14:54
GargoyleOK. I think something might be screwy on my laptop then. It;s taking 15s every time. But it's also giving me the "first run" screen every time too14:55
zygaGargoyle: perhaps something to debug with kenvandine14:56
* zyga returns to reviews14:56
* Chipaca drags himself off to have something to eat before he keels over15:07
mupPR snapd#6030 closed: cmd/snap: tweak `snap services` output when there is no services <Simple ๐Ÿ˜ƒ> <Created by bboozzoo> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6030>15:14
Chipacazyga: notes := []string{"", ""}[:0]15:23
zygahaha15:24
zygayeah15:24
zygaI don't know if golang does any static arrays but perhaps :)15:24
Chipacazyga: OTOH i don't know why you care :-)15:24
zygawe seem to care about some trivial optimisations like that15:24
zygalike pre-computing array size15:24
zygaI heard that map resizing is slow15:25
zygabut nothing tangible I can recall15:25
Chipacazyga: it's a little bit bad if it leaves the function, and can be big15:25
Chipacazyga: and is in something long-lived15:26
Chipacaotherwise, eh15:26
kyrofazyga, Chipaca is core edge not on snapd master at the moment?15:53
kyrofaPerhaps I should have phrased that the other way around15:53
kyrofaAnd in reality, perhaps that's a question for cachio15:54
Chipacakyrofa: edge is 11 hours old15:54
Chipacaand  built from master15:54
mupPR snapd#6027 closed: overlord/ifacestate: don't conflict on own discard-snap tasks when refreshing & doing garbage collection <Squash-merge> <โš  Critical> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6027>15:54
Chipacaafaik :-)15:54
kyrofaChipaca, I can't find that commit hash though15:55
ChipacaI'd love to know why you even care,  but ok15:55
kyrofaHaha, Chipaca because it doesn't seem to contain what I WANT it to contain, so I'm trying to figure out what it DOES contain15:56
Chipacakyrofa: what do you want it to contain?15:56
kyrofaChipaca, https://github.com/snapcore/snapd/pull/602915:57
mupPR #6029: snapstate: add command-chain to supported featureset <Created by kyrofa> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6029>15:57
Chipacakyrofa: hmm16:00
ChipacaI wonder if we're mangling the tree before building the snap16:00
kyrofaMaking some sort of temporary commit?16:01
Chipacakyrofa: https://launchpadlibrarian.net/394484955/buildlog_snap_ubuntu_xenial_amd64_core_BUILDING.txt.gz16:01
=== chihchun_afk is now known as chihchun
kyrofaChipaca, hmm. That doesn't tell me much. Edge doesn't seem to contain the functionality I added there, so I'm trying to figure out if it's broken or just not there. Are you pretty sure it's the former?16:09
Chipacakyrofa: what time was that build started?16:10
kyrofaChipaca, ah, looks like yesterday in my early morning16:11
Chipacakyrofa: i thought it was today16:11
Chipacabut dunno where you are in the world16:11
kyrofaChipaca, Mon Sep 24 13:14:43 UTC16:11
Chipaca.... wat16:12
Chipacathat's a month ago16:12
kyrofaOh16:12
kyrofaUh16:12
Chipacawhere are you looking?16:12
kyrofaWhoops, ignore me16:12
Chipacaway ahead of you16:12
Chipaca:-รพ16:12
kyrofaChipaca, you're right, looks like it was today16:15
Chipaca4am utc is pretty much exactly 12 hours ago16:16
* zyga grabs dinner 16:16
Chipacakyrofa: so now the question is did the git tree in launchpad get the commit you wanted in time16:17
Chipacait's cutting it close :-)16:17
kyrofaYeah it is. It doesn't look like it, but there's no way to be sure unless snapd happens to shove a changelog in there16:18
kyrofaIt does!16:19
kyrofaNot sure how this works though. If this is to be believed it's just 2.36~pre216:21
Chipacakyrofa: i think the changelog has a manual step16:21
* Chipaca grabs cachio before he runs off again16:22
Chipacacachio: hi16:24
Chipacacachio: how can kyrofa figure out what commits of snapd are on core edge?16:24
zygare16:34
zygaback to reviews16:34
cachioChipaca, kyrofa there is a launchpad repo16:37
cachioon sed16:37
cachiosec16:37
cachiokyrofa, this is the repo https://launchpad.net/snapd-vendor16:38
cachioit takes a time to create the snap16:39
cachiokyrofa, what do you need to do with it?16:39
kyrofacachio, I was hoping folks could utilize a PR that landed yesterday16:40
kyrofaChipaca, ah, the commit hash comes from that ^16:40
kyrofacachio, so what, you just dump snapd master into that every few days?16:42
cachiokyrofa, we dump it always when master is in green16:44
cachioso, we run tests on master branch after each merge16:44
=== chihchun is now known as chihchun_afk
cachioif the tests pass, then we run a script to dump the master to this lp repo16:45
cachiothen, based on that, we build the core snap on edge16:46
cachioit takes some time16:46
kyrofaOkay so based on this, the edge snap is based on snapd master as of Friday?16:47
kyrofaAm I deducing this all correctly?16:48
Chipacakyrofa: BTW, you know assumes: is a last-ditch kind of thing, currently? it's not user friendly at all16:48
cachiokyrofa, let me check when was the last one16:49
Chipacakyrofa: it's fine for exploratory work or for super-early-adopters kind of thing, but not for end users16:49
Chipacakyrofa: it'll download the snap before even knowing about assumes16:49
Chipacaand throw a hissy fit at you16:49
kyrofaChipaca, yeah. It's either that or magically not work16:49
cachiokyrofa, edge snap is based on snapd master16:50
cachioit si right16:50
cachiobased on last master which has passed the tests16:51
cachiokyrofa, last one is from today16:52
kyrofacachio, I'm confused. The edge snap has this version: "16-2.36~pre2+git967.54466bd"16:54
mupPR snapcraft#2383 opened: ci: use more travis primitives for osx tests <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2383>16:54
kyrofacachio, that commit hash points to a commit you made on Friday: https://git.launchpad.net/snapd-vendor/commit/?id=54466bdcf1432fb7c2869a3abbf92751186aaa4216:54
kyrofaAlthough when I click on it it changes to Saturday... but I'll ignore that :P16:55
kyrofaTimezones I guess16:55
kyrofaAnd if I look at the file I'm interested in, it's definitely not from today-ish since it doesn't contain the commit that landed yesterday: https://git.launchpad.net/snapd-vendor/tree/overlord/snapstate/check_snap.go?id=54466bdcf1432fb7c2869a3abbf92751186aaa4216:56
=== pstolowski is now known as pstolowski|afk
koala_manmborzecki, Chipaca: I'm glad you're finding shellcheck useful! unfortunately the shellcheck snap build has been broken for a while due to https://bugs.launchpad.net/snapcraft/+bug/1797809 . Any suggestions?17:01
mupBug #1797809: Build fails on snapcraft.io, works locally (during `cabal update`) <Snapcraft:New> <https://launchpad.net/bugs/1797809>17:02
Chipacakoala_man: I didn't know, no! and yes, we're using it all over the place including in static checks of snapd itself17:04
Chipacakoala_man: I've got to run now, but I'll take a look later17:05
koala_manI'd appreciate it ^^17:05
Chipacakyrofa: sergiusens: if you have a bit of time, ^^^17:06
kyrofakoala_man, one of the better tools I've used17:07
kyrofaWe're using it in snapcraft as well17:07
kyrofakoala_man, do you know if cabal obeys http_proxy?17:08
koala_mankyrofa: yes, it does17:08
koala_manIf I run http_proxy='http://localhost:12345' cabal update  it fails with "cabal: Couldn't establish HTTP connection. Possible cause: HTTP proxy server is down."17:09
kyrofacjwatson, are you around? Curious to get your thoughts on https://bugs.launchpad.net/snapcraft/+bug/179780917:10
mupBug #1797809: Build fails on snapcraft.io, works locally (during `cabal update`) <Snapcraft:New> <https://launchpad.net/bugs/1797809>17:10
* zyga goes for last streak today17:13
cachiokyrofa, the commit summary has the commit in githb17:15
cachiokyrofa, the last commit on lp has a commit message which points to c712d09ac22ee65c48b520f462f8ddbaf1ee5efa commit on github17:15
cachiothis it the traceability17:15
kyrofacachio, right, but that one's not in edge17:17
kyrofaHow often is it built?17:17
mupPR snapcraft#2384 opened: project: do not install base is already installed <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2384>17:24
cachiokyrofa, once a day aprox17:27
cachioduring the night17:27
cachiokyrofa, 4 am was built the last one17:27
kyrofacachio, then why is the version referring to 54466bd?17:28
cachiowe have a sync job which takes the changes in snapd master and push it to launchpad17:29
cachiothe 54466bd... is the commit on launchpad which has the same content than whe commit id added in the summary17:30
cachioContent updated (b5e852ba093baad2624a8ecc32e9715c72631752)17:30
cachioso yoy know the commit in lp 54466bdcf1432fb7c2869a3abbf92751186aaa42 is the same then the commit in github b5e852ba093baad2624a8ecc32e9715c7263175217:31
cachiothen every night another job is executed which creates and publishes the core on edge17:31
cachioit takes the last change in the lp repo and use that to create the core snap17:32
cachioit is a bit complicated :)17:32
kyrofaAh, I'm just looking at the commit timestamps and thinking "13 hours ago was yesterday!" but I suppose not everywhere17:33
kyrofaOkay so the one that builds tomorrow will be what I want17:34
kyrofaThanks cachio, that traceability is handy17:34
cjwatsonkyrofa: should be reassigned over to launchpad-buildd - I guess either cabal ignores the proxy or they manage to confuse each other somehow17:35
kyrofacjwatson, will do, thanks :)17:36
cachiokyrofa, yes, I think tomorrow will be includede17:36
cjwatsonkyrofa: I think I saw another question about the same thing recently but it doesn't seem to have ended up as a launchpad-buildd bug, so ...17:36
cachiootherwise ping me17:36
kyrofakoala_man, cjwatson knows everything about build.snapcraft.io, he'll help get that working17:38
koala_manawesome! I'm happy to help in any way I can17:39
koala_mancjwatson: is there any way I can run the build locally with the same proxy?17:40
cjwatsonkoala_man: It's possible but complicated.  Requires setting up a xenial VM, installing launchpad-buildd in it from ppa:launchpad/ubuntu/ppa, and then sending some err not-very-well-documented XML-RPC commands to it to get it to do something ...17:48
cjwatsonOh and of course you need a proxy for it to talk to17:49
cjwatsonI suspect the problem here is the localhost-only proxy that launchpad-buildd runs to avoid builds needing to be able to cope with authenticated proxies17:50
koala_mancabal apparently has a --http-transport=curl flag that can be switched to wget or plain-http. is there a nice way to test this that doesn't involve me pushing upstream test commits?17:53
koala_manby the way, this used to work, and I can probably find the date it started breaking if that's helpful17:53
cjwatsonTesting it locally can be done by just dispatching a snap build, but I won't be able to get to it until later this week17:54
cjwatsonIt's quite likely that it started breaking in mid-June when we deployed the fix for bug 1690834 / bug 175334017:55
mupBug #1690834: snap builds in cleanbuild but fails on buildd <launchpad-buildd:Fix Released by cjwatson> <https://launchpad.net/bugs/1690834>17:55
mupBug #1753340: build failing on download by ant build  <ant> <python> <snapcraft> <launchpad-buildd:Fix Released by cjwatson> <https://launchpad.net/bugs/1753340>17:55
koala_manI just looked it up and you're right17:55
cjwatsonlaunchpad-buildd 16217:55
cjwatsonI'd rather we not work around it in shellcheck TBH since you're probably not the only affected project17:56
cjwatsonI'm slightly surprised I've only heard about this in relation to cabal17:56
cjwatsonBut there may be some subtle property of the HTTP connection that's confusing it17:57
koala_manof course, but it would be nice way to narrow down the problem17:57
cjwatsonIt would be possible to push test commits to a branch and create a test snap on LP that builds it17:57
cjwatsonbuild.snapcraft.io can't do that, but it can be done directly using the API17:57
cjwatsonhttps://launchpad.net/+apidoc/devel.html#snaps-new17:58
cjwatsonThough TBH I suspect actually fixing the bug is going to require more direct debugging methods anyway - strace/tcpdump, that sort of thing17:59
cjwatsonor a load of debugging prints sprinkled through the localhost-only proxy17:59
mupPR snapcraft#2383 closed: ci: use more travis primitives for osx tests <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2383>18:24
zygajdstrand: hey, I know you are sprinting but if you have a moment please consider looking at https://github.com/snapcore/snapd/pull/572718:54
mupPR #5727: interfaces/browser-support, cmd/snap-seccomp: Allow read-only ptrace, for the Breakpad crash reporter <Created by jld> <https://github.com/snapcore/snapd/pull/5727>18:54
mupPR snapd#6000 closed: snap,client: use a different exit code for retryable errors <Created by mvo5> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6000>18:55
* cachio afk18:59
koala_mancjwatson: I would be willing to look into that if I had a simple way to reproduce it19:06
mupPR snapcraft#2382 closed: unit tests: missing full adapter in extension test <Created by kyrofa> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2382>20:52
AuroraAvenueHi N00b question. Is it possible to Snap the google Api ?21:26
AuroraAvenuehttps://developers.google.com/+/web/api/rest/21:26
ijohnsonAuroraAvenue: if all you want to do is call the API via HTTP calls then yes you could have a snap which interfaces with the Google+ REST API21:27
AuroraAvenuecheers - I shall add it to the list.21:39
mupPR snapd#5727 closed: interfaces/browser-support, cmd/snap-seccomp: Allow read-only ptrace, for the Breakpad crash reporter <Created by jld> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/5727>22:39
mupPR snapd#6023 closed: overlord/snapstate, snap, wrappers: start services in the right order during install <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6023>22:46
mupPR snapd#6032 opened: overlord/snapstate, snap, wrappers: start services in the right order during install <Created by mvo5> <https://github.com/snapcore/snapd/pull/6032>22:49

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