/srv/irclogs.ubuntu.com/2018/08/20/#snappy.txt

=== pbek_ is now known as pbek
mborzeckimorning05:08
=== chihchun_afk is now known as chihchun
mborzeckimvo: hey06:14
mvohey mborzecki - good morning06:14
mborzeckimvo: do you plan to release 2.35 today?06:14
mvomborzecki: yes06:15
mborzeckiack06:15
mvomborzecki: reviews for 2.35 prs would be great, iirc its not that many left06:19
mvomborzecki: and 5583 can be ignored and 5340 as well, those will go to 2.3606:19
mvomborzecki: oh, I see there are reviews :) yay06:19
mborzeckimvo: yeah, tried to go through 2.35 ones first06:22
zygaGood morning07:03
mborzeckizyga: hey07:09
zyga*yawn* :-)07:11
zygaI should make a double espresso07:11
mvozyga: heh, yeah, I make a cup of tea as well07:26
zyga:-)07:30
mborzeckizyga: quick question, can you take a look at https://paste.ubuntu.com/p/xgQDvNqmtb/ ? i haven't updated apparmor profile yet, so it's expected to fail, the thing i'm a bit suprised is that failure in snap-update-ns does not make the whole thing exit with non-0 status07:30
zygaYes that is expected”07:30
zygaOnly layout failures are “loud”07:30
zygaIt is an old decision we took07:32
mborzeckizyga: ok, so i guess i should make those changes be of 'layout' origin07:33
=== pstolowski|afk is now known as pstolowski
mborzeckipstolowski: hey07:36
pstolowskiheyas07:36
zygaOr change the default07:38
zygamborzecki: I mean, the decision was made because we only had the content interface using it07:43
zygamborzecki: I would argue that we could reverse the decision now07:44
* zyga goes to find 2fa tokens07:59
Chipacamoin moin08:06
zygahey Chipaca :)08:06
mborzeckiChipaca: hey08:07
Chipacamborzecki: is 2.34 really *that* old, or is something weird going on there?08:07
mborzeckiChipaca: hm?08:07
zygamborzecki: btw, I prepared 2.35 for opensuse08:08
Chipacamborzecki: this arch bug on the forum08:08
zygabut then my opensuse box removed grub08:08
zygaso I need to recover it but the packaging is ready08:08
zygaI tested it for about a day08:08
mborzeckiChipaca: i think it's something odd08:08
mvoChipaca: once 5678 has a second review I can release 2.35 for real ;)08:20
mvoChipaca: hm, and 5677 it seems08:20
mvoChipaca: meh, the later one can go in08:21
Chipacamvo: how any +1s do you want on it?08:21
* mvo does that now08:21
Chipacamvo: 5678 has a +1 from samuele from 3 days ago and one from maciej from 2 hours ago08:21
mvoChipaca: aha, sorry then!08:22
mvoChipaca: I was trying to debug this grub xz failure, I think that messed with my brain :(08:22
Chipacamvo: good thing the week is nearly over08:22
Chipacamvo: the comment about the error message is on point though (but fine for a follow-up imo)08:23
Chipacaniemeyer: dunno if you saw my question about the limitation in spread about not mixing absolute and relative 'systems' statements08:24
mborzeckiChipaca: yeah, that forum user has a messed up snapd install from before antergos fixes imo08:24
Chipacaniemeyer: I'd like to relax it so that you can mix them as long as the absolute come before the relative08:25
niemeyerMornings!08:25
mborzeckiChipaca: at that time we'd just default to /snap instead of /var/lib/snapd/snap and after an update to proper version things go sideways08:25
Chipacaniemeyer: that way one can say: systems: [ubuntu-*, -ubuntu-14*]08:25
niemeyerChipaca: Sounds reasonable08:25
Chipacaniemeyer: PR coming your way in a bit08:25
niemeyerThanks!08:25
Chipacaniemeyer: 'go test' is failing with errors in humbox, am i holding it wrong?08:37
niemeyerChipaca: No, I see them as well, sorry.. humbox is not yet properly finished.. the other end of it, the spread backend itself, still sits on my tree08:39
Chipacaniemeyer: no worries08:40
niemeyerChipaca: Just to make sure, the errors you see are from vet, right?08:40
Chipacaniemeyer: yes, about printf08:40
niemeyerChipaca: COol.. let me fix those quickly08:40
niemeyerChipaca: Done08:44
Chipacaniemeyer: mucho bettero08:45
mborzeckizyga: hm asking for user.Current() to work in snap-update-ns is probably too much right?08:48
zygayes, it will run inside the mount namespace and it will not necessarily see the right /etc08:49
zygabrb08:50
zygare09:01
zygapstolowski: I'm going over all of the hot plug branches09:04
pstolowskizyga: great, thank you! btw, last friday we're discussing the need for a helper such as SystemSnapInfo(), i need to talk yo you about that09:06
zygapstolowski: tell me more, what would that do09:06
pstolowskizyga: can we have a quick HO?09:06
pstolowskiit's related to hotplug09:07
zygayes09:12
zygasure09:12
zygastandup?09:12
pstolowskiok, coming09:13
zygaeh09:13
zygawhy is google so moronic09:13
zygaI personally cannot wait until we move off google services :/09:14
niemeyerI'm particularly frustrated with the new interface of Meet.. it now feels like we're speaking to one or two people, even if there are 10 on the call09:17
mborzeckizyga: https://paste.ubuntu.com/p/F2hNNVx8HJ/09:33
mborzeckizyga: it's still a bit ugly on the inside as I need to guess the user's home dir while not beign able to use user.Current() https://paste.ubuntu.com/p/h8RPVFm4RH/09:34
zygaone sec, in a call09:34
niemeyerpedronis: Commented on the store scope doc09:38
pedronisniemeyer: I'm a bit confused by some of the comments09:39
zygamborzecki: looking09:46
zygamborzecki: hmm, curious09:48
zygamborzecki: since we do user mounts, we have to know the user id at least, and presumably more, in a robust way09:48
zygamborzecki: how does snap-confine get that today?09:49
zygamborzecki: is it a bit of environment that we trust?09:49
mborzeckizyga: uid is known, but the paths are from snap run (?)09:49
zygaaha09:49
mborzeckizyga: s/paths/env vars/09:49
zygaright09:50
zygamborzecki: well, we could do one more thing09:50
zygasince this is a profile written by snapd09:50
zygawe can just ask snapd to tell us09:50
zygathat is, we can bake the variables into the per-user profile09:50
mborzeckizyga: per user profile?09:51
zygayeah09:51
zygaI mean per user mount profile09:51
zygaah09:51
zygaI'm dumb, sorry09:51
zygano we cannot :P09:51
zygadrat :)09:51
mborzeckizyga: but you don't know the user at this point09:51
zygayeah09:51
zyga...09:51
zygawe could have snapd write authoritative user data per-user09:52
zygabut ...09:52
mborzeckibit of chicken and egg :)09:52
zyganot sure if sensible09:52
zygaI mean something like /var/lib/snapd/users/zyga09:52
zygawith authoritative data09:52
zygaand it would be true inside mount namespaces and what not09:52
zygaso no need to rely on /etc09:52
mborzeckizyga: you'd need to know the users upfront09:52
zygawell, snapd already needs to know09:53
zygabut yeah, more sprawling complexity there09:53
=== chihchun is now known as chihchun_afk
photexmoin moin! Looking into using snap as an internal app format. Is it possible to run a private snap repo/service?10:23
zygaphotex: hey, you can send private snaps to the store that only you can download, you can also copy a snap and just install it from a file as well; we don't support running a private repository just yet but for bigger deployments you can redirect your device to a private store (which is a commercial offering I believe) where you can locally host snaps that are private to your organisation (locally) and also filter access to the10:25
zygamain store10:25
photexThanks for the info zyga10:28
photexAnd can snaps contain services?10:29
zygayes10:30
photexwe're specifically looking at snaps as an alternative to deb10:30
photexwhich I'd assume you may have guessed :D10:30
mborzeckizyga: so the spread test that runs snaps, writes some data to SNAP_{DATA,COMMON,USER_DATA,USER_COMMON} works now10:31
zygamborzecki: that's great :-)10:31
mborzeckizyga: obviously non ubuntu host :P now i have to update autogenerated apparmor profile whihc will be a bit of fun i presume10:31
zygamborzecki: yeah, it should be doable :-)10:51
mborzeckiis mup quiet again?10:54
mborzeckihttps://github.com/snapcore/snapd/pull/568410:55
zygapstolowski: hey, I put hotplug reviews on hold for now, I will fix my PRs so that they land; once I do another pass we should be closer to the key decision wrt system snap11:10
zygamvo: we should discuss the system snap concept again11:11
zygamvo: I have a feeling that pawel's work needs to solve that11:11
pstolowskizyga: sure & thanks11:13
mvozyga: yeah, I have free cycles one 2.35 is out11:21
mvosimple review: 568511:23
mvo(but important for core18)11:23
zygamvo: done11:27
zygaand back to hacking11:27
mvozyga: thank you!11:28
mvoogra: I think the latest (edge) snapd fixes the reboot hang, kudos to pedronis11:32
ogramvo, will try !11:40
=== pstolowski is now known as pstolowski|lunch
ogra(VM is running but it might take a while until it refreshes)11:46
ograpedronis, mvo, seems to have helped, thanks (havent seen any timeouts this time)11:53
* Chipaca ~> lunch11:55
mborzeckiwhat part is actually skipped when doing spread -reuse ?12:22
zygaI think chunks of prepare are skipped12:22
zygabut also allocation (obviously)12:23
mborzeckihm got a case when s-c was not being rebuilt when using -reuse12:23
zygathat's prepare that I mentioned12:23
mborzeckii'd guess that package is rebuilt but well, maybe not then12:24
mborzeckizyga: https://github.com/snapcore/snapd/pull/5687 your turf12:25
zygahmmm12:26
zygawill this work if we disable reexecution12:26
zygathat is12:27
zygaif future snapd sets SNAP_NAME and SNAP_INSTANCE_NAME12:27
zygaand somehow old snap-confine is ran12:27
zygaI know that the answer is "it doesn't work"12:27
zygabut I wonder how to guard against that12:27
zygaor if it will happen in practice12:27
mborzeckizyga: i remember talking to mvo that snap/s-c go in sync12:28
mborzeckiso you either reexec both or none12:28
zygaanyway, I will think and review12:28
mborzeckizyga: thanks!12:40
Chipacamvo: did we have a problem with x/sys/unix? something about powerpc?12:44
=== pstolowski|lunch is now known as pstolowski
mvoChipaca: we had a problem there in 2.35. do we have a new one=12:47
mvoChipaca: ?12:47
mvoChipaca: do you have more details? 2.35 should be happy everywhere now12:47
Chipacamvo: I'm asking because I was wondering whether I could use it or not12:48
mvoChipaca: let me try12:49
Chipacawhy did I add the 'or not' there12:49
* Chipaca grabs the dunce hat12:49
mvoChipaca: did I mention that my brain got a bit damaged by grub?12:49
mvoChipaca: anyway, let me try to login into a ppc machine to test this12:49
Chipacawe use syscall all over the place :-/12:50
mvoChipaca: which package? "syscall"? or "x/sys/unix"?12:56
Chipacamvo: x/sys/unix is the one I'd like to use12:56
mvoChipaca: hm, there is a bugreport that x/sys/unix is broken with go1.612:58
Chipacamvo: link?12:58
mvoChipaca: https://github.com/golang/go/issues/2657612:59
mvoChipaca: on powerpc I have proxy issues right now, looking into it12:59
Chipaca:-(13:00
zygamvo: it's interesting because of how that snap (classic) works;13:16
zygamvo: should we keep all the variant deltas (core18, core16) inside?13:17
zygashould those be separate snaps (classic/18)13:17
zygaetc13:17
mvozyga: good questions13:22
zygamvo: and to take the question further, what about fedora29 is that is a boot base and you install classic, then what13:23
zygamvo: perhaps classic should be a special snap, that can talk to snapd (snapd-control) figure out where it is (which boot base, which environment (classic or core)) and DTRT13:24
zyga(e.g. download the delta or do something similar)13:24
zygamvo: but being executable in the classic snap itself feels like a good idea, then we could do the right thing without having all that logic in snapd13:25
* zyga goes to make coffee while spread runs13:32
=== Son_Goku is now known as Conan_Kudo
=== Conan_Kudo is now known as Son_Goku
mborzeckiChipaca: could you take a look at https://github.com/snapcore/snapd/pull/5670 later on?13:48
Chipacamborzecki: yep13:48
mborzeckiChipaca: thanks!13:48
zygadh_clean fails in spread? https://www.irccloud.com/pastebin/Sp1cvxRD/13:56
mvozyga: I have not seen this yet14:05
* zyga -> afk14:21
mborzeckizyga: when you're back, please take a look at https://github.com/snapcore/snapd/compare/master...bboozzoo:bboozzoo/parallel-install-snap-namespace-mapping just to do a sanity check of the changes14:23
alexlarssonjamesh: I'm updating the flatpak PPA to 1.0, which includes the xdg-desktop-portal 1.0 release14:23
alexlarssonI'm not sure if updating it for bionic is a good idea though14:24
alexlarssonDo you want to do that update eventually?14:24
mborzeckizyga: this works on ubuntu 18.04 running from spread, i've pushed it to the integration branch as well so we'll know if other releases are good too14:25
zygaLooking14:34
zygaMborzecki: there may be one issue there14:36
zygaMborzecki is envp big enough? I doubt it is14:36
Chipacamvo: can i show you something funny?14:37
Chipacamvo: funny but not urgent :-) so no worries if you'd rather be ignorant for a bit longer14:37
sergiusensChipaca: is funny the thing we are talking about? 😅14:38
Chipacasergiusens: yes14:38
mvoChipaca: sure14:38
Chipacamvo: try "snap install --channel=latest/stable bofh" (or any snap) (or refresh instead of install)14:39
mvoChipaca: heh, it tells me this channel is closed14:39
Chipacamvo: :-)14:40
Chipacai'll look into fixing it later14:40
mvoChipaca: thank you14:40
Chipacaas soon as this uname thing is green :-)14:40
Chipacathanks to sergiusens fwiw14:40
* mvo hugs sergiusens 14:40
Chipacarefreshing between stable and latest/stable is also fun14:40
Chipacawe need a "normalize channel name" thing before comparing them14:40
mvoChipaca: I think we have something like this14:42
mvoChipaca: iirc Channel.Clean() will do that14:42
sergiusensChipaca: I have normalizing logic in snapcraft for build snaps because the api does not normalize them for me ;-)14:43
sergiusensChipaca: LP: #178796714:44
Chipacasergiusens: ta14:59
* Chipaca pokes mup14:59
Chipacaniemeyer: ^15:00
niemeyerChipaca: Thanks, must have reconnected .. I still haven't managed to fix the nickserv behavior properly15:00
ograthe spam wave seems to be over though ... perhaps we could remove +r again15:01
niemeyermup: Ok now?15:02
mupniemeyer: Roses are red, violets are blue, and I don't understand what you just said.15:02
niemeyerChipaca: ^15:03
Chipacamup: <315:03
niemeyerogra: Worth trying, otherwise I'll need to spend some time this week actually fixing that15:03
ograniemeyer, actually, looks like someone did that already15:05
ograseeems to only be +cnt ... i see no +r15:06
Chipacaogra: what does the global +R do?15:06
Chipaca(block messages from unidentified users)15:08
ograChipaca, +r means registered channel +R means you also need to use an registered nick ... but neither seems to be set here anymore15:08
ograso it should be fine for mup for the moment15:08
ograChipaca, for details https://freenode.net/kb/answer/channelmodes15:14
Chipaca#178796715:15
mupPR snapd#5689 opened: many: move Uname to osutil, for more DRY and easier porting <Created by chipaca> <https://github.com/snapcore/snapd/pull/5689>15:15
mupBug #1787967: installing from latest/stable tells you latest/stable is closed and tracking stable <snapd:Triaged by chipaca> <https://launchpad.net/bugs/1787967>15:15
Chipacak15:15
kyrofaHey Chipaca, any chance you could take a look at https://github.com/snapcore/snapd/pull/5655 ?15:37
mupPR #5655: snap,snap-exec: support command-chain for hooks <Created by kyrofa> <https://github.com/snapcore/snapd/pull/5655>15:37
=== sparkieg` is now known as sparkiegeek
=== pstolowski is now known as pstolowski|afk
jdstrandroadmr: hi! would you mind pulling r1115 of the review-tools? it's all overrides16:19
roadmrjdstrand: ohai! sure thing, coming up16:21
jdstrandroadmr: thanks!16:23
Chipacakyrofa: tomorrow, i'm afraid17:23
kyrofaChipaca, no problem, thanks!17:35
smoserhey all. someone suggeseted i enable the 'removable-media' plug. is there any reason to not do that?17:56
smoser https://github.com/smoser/pdftk/issues/117:56
zygasmoser: no, just go ahead and do it17:56
popeyogra: finally tried wmx, but it doesn't start. https://paste.ubuntu.com/p/PjHX72f2Bx/19:35
jdstrandroadmr, nessita: hey, I've had two LP generated uploads fail. Eg: https://launchpad.net/~myapps-reviewers/+snap/review-tools/+build/31638619:41
jdstrand"Store upload failed: Piston/0.2.4 (Django 1.11.13) crash report: Method signature does not match. Resource does not expect any parameters. "19:41
jdstrandI looked on status.snapcraft.io and...19:42
jdstrandoh, there it is "19:42
jdstrandWe are experiencing some problems with the Snapstore hosting infrastructure, and are investigating."19:42
nessitajdstrand, is all the rebootarama on every server19:42
jdstrandI see, ok19:43
nessitajdstrand, but I'm not sure your error is related19:43
jdstrandit is a weird error19:43
roadmr\o/19:43
roadmrit's piston, it's all weird19:44
nessitajdstrand, if you click retry, does it fail consistently?19:44
jdstrandnessita: I did one retry and it failed. let me try again19:45
jdstrandnessita: it failed again19:46
nessitajdstrand, hum, I will try to dig logs19:53
=== marosg_ is now known as marosg
* zyga fixed a very stupid network bug that was driving him crazy20:01
zygaI had network sharing on one machine and two conflicting dhcp servers20:01
zygastuff _sometimes_ worked, depending on luck and off/on status of the other box20:01
zygaman that was silly and hard to track down20:01
roadmrHeads up folks - snap store is having some trouble with some operations like some snap pushes, releases, addition of developer accounts. We're looking into it.20:06
=== Dmitrii-Sh_ is now known as Dmitrii-Sh
=== gurmble is now known as grumble
popeyogra: seems if I reboot, wmx is running and I see a mouse pointer, but nothing more.20:11
roadmrjdstrand: see my latest. Some stuff is broken indeed :( we're looking into it20:17
nessitajdstrand, hey, can you please retry the review-tools?20:19
nessitajdstrand, or I can click on the button (I see it, at least :-))20:19
* nessita clicks20:19
nessitajdstrand, issue fixed20:28
ograpopey, a left click should bring up a one-entry-menu ... "New" ...20:32
ograclicking it starts an xterm20:32
ogra(and makes that xterm show up in that menu)20:33
popeyno, it's a blank desktop20:33
popeylike, doesn't even look like a desktop is running.20:33
ograand nothing if you click ?20:33
popeynope20:33
ograweird20:33
ograyou did install in devmode ?20:33
popeyyes20:34
ograand connected the wayland-socket-dir and and x11-plug interfaces20:34
popeyyes20:34
ogravery weird20:34
ograwhcih snap did you use ?20:34
ogra(it runs fine here in my qemu instance and on a pi)20:35
ograi uploaded the one i'm running to the store but the review is stuck atm20:35
roadmrogra: stuck or waiting for manual review?20:36
popeyi used the one you linked to20:36
roadmrogra: stuck I can help with, waiting for manual, I probably can't :(20:36
popeyhttps://paste.ubuntu.com/p/XKJfG99Cxr/  ogra20:37
ograroadmr, the latter ... the snap uses a llopback x11 plug/scket setup ...20:37
popeyi see no snap called wmx in the store20:37
ograthe review tools arent happy with that20:37
popeyhttps://dashboard.snapcraft.io/snaps/wmx/20:37
popeythat shows 40420:37
ograpopey, no, it is called wmx-kiosk-session20:38
popeyoh20:38
jdstrandnessita: thanks! sorry, someone was at the door20:38
popeyok, well it doesn't work here. I have a mouse cursor and that's it20:38
popeyin fact, if i touch the mouse, the cursor disappears20:38
ograthats ok ... thats a mir bug20:39
ograbut you should also get a menu20:39
popeyooh!20:39
popeynow I do20:39
roadmrogra: oh then :( sorry :( would need to wait for someone who knows what they're doing to review it20:39
ogra\o/20:39
ograroadmr, yeah, i know jamie knows how to deal with that ... but it isnt that urgent that i want to push it :)20:40
popeyogra: is there some graphical thing I should be able to run?20:40
popeylike a desktop snap?20:40
ograi havent tried desktop apps ... but htop definitely works20:40
ograso should irssi and friends20:41
popeyhm, i have to "snap run" things20:41
ograi'm not really sure what happens with desktop apps ... you'd likely need to connect them somehow to the x11 slot20:41
ograsu popey20:42
ogra;)20:42
ograswitch to the actual user20:42
ograthat should give you the needed path entry (root doesnt have /snap7bin in path)20:42
popeycore:)20:42
ograha !20:42
popeyhttps://usercontent.irccloud-cdn.com/file/PQV1cfQk/IMG_20180820_214251.jpg20:43
ograyeah ...20:43
popeyNice one, thanks ogra :)20:43
ograwith another "New" you can open mre terminals20:43
ogra*more20:43
ograto run more apps :)20:44
Chipacasergiusens: how do I tell mypy to chill?20:47
sergiusenschipaca: # type: ignore20:49
Chipacasergiusens: and, where should I put a yaml module, if I were to write it?20:50
sergiusenschipaca I think kyrofa might take over your PR20:50
Chipacasergiusens: that's fair20:50
sergiusenschipaca if that speeds up a review for his he might be happier too :-)20:51
Chipacasergiusens: that sounds efficient20:52
ChipacaOTOH, i've checked out of go, and was only looking at mypy because I was curious :-D20:52
sergiusensoh, add the proper type check then :-)20:54
zygajdstrand: re20:55
Chipacasergiusens: it looks  like I'll dive into mypy over my staycation20:55
Chipacabah20:55
Chipacaa friend is getting married, also20:56
zygajdstrand: not sure if you remember the interface for sharing /mnt, I just updated the PR for that with a simpler approach that just reuses the removable-media; added some more tests and replied / applied all feedback.20:56
Chipacaso there might be a lot of alcohol involved20:56
jdstrandzyga: I do and I saw and it is already on my TODO to review20:58
jdstrand:)20:58
zygathank you20:58
zygahttps://twitter.com/zygoon/status/1031646427611578370 :-)20:58
* zyga tries to switch to vim hjkl for navigation20:58
Chipacag'night y'all21:05
jdstrandzyga: nice :) and hjkl is worth it I think (I switched a year or so ago)21:06
zygajdstrand: it's nicer in real life, my phone camera doesn't capture the contrast21:07
zygaplus most keys leave ghosting trail as I type21:07
zygait's pure bling but it looks great :D21:07
jdstrandhehe21:07
zygabrb, shower time21:07
jdstrandnessita: fyi, snapcraft release had the same issue as before: bad-request: Piston/0.2.4 (Django 1.11.13) crash report:21:18
jdstrandMethod signature does not match.21:18
jdstrandResource does not expect any parameters.21:18
jdstrandnessita: different from before, retries seemed to help. I am not blocked21:18
roadmrjdstrand: yes, we just noticed we missed restarting some processes :)21:26
* jdstrand nods21:26
roadmrjdstrand: done, hopefully solved now but we'll keep an eye on things21:31
sergiusensjdstrand: zyga hjkl on vi(m) or something else?21:44
sergiusensI only ever used that, but am on vscode now... contemplating switching back just for navigational purposes21:45
roadmrI was going to say hjkl sucks for me because I'm left-handed, but the arrow keys which I use just fine *are* on the right side :/ so I guess it's just never getting used to using letters if there are perfectly fine, pointy arrows for that purpose21:46
jdstrandsergiusens: I was assuming vim21:50
jdstrandroadmr: hehe :)21:51
cjwatsonogra: I think I set +q $~a here rather than +r23:42
cjwatsonwhich is slightly gentler23:43

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