[02:48] Is there a way to utilizr a bz2 file in the source directive? [02:48] utilize* [02:50] I'm working on a snap for restic in which the author prefers that the snap use the "official" binary (hope to change that later). But his releases come out as a bz2 file. When targeting the file I get an error stating that there is no handler for the bz2 file. [02:51] I'm in the middle of working on it - so was hoping for a quick ping type answer, otherwise I will move this over to the forum. [02:51] Thanks all. [03:53] bashfulrobot: atm, there's no bz2 support in snapcraft, iirc [03:53] debian doesn't generally encounter bz2 that much anymore, so I would guess they didn't think about it [04:32] Son_Goku: thanks for the info. I had pretty much came to that same answer. I guess I'll have to figure out the best way to maybe wget the file in the build, and extract, then just use `organize` to getit where it needs to go. [06:07] morning [06:11] morning [06:11] * Son_Goku needs to sleep [06:12] anyway, if anyone sees robert-ancell, can someone poke him about this? https://github.com/snapcore/snapd-glib/issues/32 [06:24] o/ [06:25] Son_Goku: zyga-ubuntu: morning guys [06:27] hey, good morning [06:27] uefi greeted me today with "ubuntu (disk not present)" [06:27] I think this is not a happy day [06:29] * zyga-ubuntu boots from usb and hopes the disk isn't dead yet [06:32] zyga-ubuntu: whad did you do? another dead disk? [06:37] mborzecki: no, just booted it again today [06:37] man, I'm not lucky [06:37] it shows up and mounts from live media [06:38] drive not present... [06:38] sounds like bad news [06:39] Hey guys (morning to those of you in Europte, etc) [06:39] I have a snap that I'm getting ready for snapcrafters. [06:40] It is going to need classic confinment as it is a backup program. [06:41] Going through the checklist it states "Convert the snap to `strict` confinement, or `classic` confinement if it qualifies". Is there some sort of qualifier process to allow a classic confiment for hte store? [06:42] And then a few lines down it states to move it into beta once it has been made either strict or classic. Do others generally just do locally testing in devmode nad hten call the comminuity in for review? [06:42] once in the beta channel? [06:43] * bashfulrobot getting sleepy - almost bed time. [06:50] bashfulrobot: hey [06:50] hello [06:50] bashfulrobot: you can request classic confinement on the forum (forum.snapcraft.io), there is a process to follow there [06:51] I'll search it up then. [06:51] bashfulrobot: if you can become a strictly confined snap it is much easier, obviously, but for a backup program it may be challenging, depending on the use-cases [06:51] bashfulrobot: you can test in classic mode as well, though devmode and classic are quite different [06:51] Well basically you need ot be able to backup any location on the file system [06:52] I was under the impression classic was the onbly way to do that [06:52] backup and restore, which is read and write anywhere [06:52] yes, I think so [06:52] exactly [06:52] you *can* do that in devmode [06:52] but it's more tricky [06:52] ok, I'm going to apply for classic then [06:52] and there's no interface for backup programs yet [06:52] ah ok. [06:52] sorry for brief answers, I think monday doesn't like me (my computer is not working) [06:52] hey - nooo worries [06:53] It's almost 11 pm Sunday for me... so my brain is giving up on me. :-) [06:54] "logical sector size is zero" [06:54] my ESP partition is broken [06:54] ok, that's it [06:54] I'm turning this machine off and I'll go and buy a new computer today [06:54] I'm so so so fed up with it [06:54] bashfulrobot: enjoy your evening and thank you for making snaps! [06:56] no worries! Last questoin - I see that the requests for classic seem to be in the store category. I'll put my request in there. Appreciate the direction! [06:56] zyga-ubuntu: hope the day gets better! [06:58] yes, the store category is right [07:06] good mornin mvo [07:08] hey zyga-ubuntu - good monring [07:08] morning even [07:08] zyga-ubuntu: shall we continue with this systemd shared unit today? [07:08] zyga-ubuntu: appreciate the help. Classic has been requested. Have a good day. I'm off to bed. [07:09] * bashfulrobot zzzzzzz [07:12] mvo: yes, let's do that; though please forgive me for being grumpy today as my computer has a broken ESP partition and I'm on the backup laptop without my work [07:12] mvo: I'm throwing the towel on my desktop today [07:13] zyga-ubuntu: "esp"? [07:13] zyga-ubuntu: sorry to hear that things are broken :( [07:13] mvo: the system boot partiotion on efi systems [07:13] zyga-ubuntu: ohhh, ok [07:13] my disks are failing on me :( [07:14] zyga-ubuntu: get ssds :P [07:14] I can boot the live media but I cannot boot the disk anymore [07:14] zyga-ubuntu: when they fail, they fail faaaaster [07:14] mvo: yeah, I'm considering just getting out and going to buy a new computer now [07:14] hahaha [07:16] PR snapd#4558 opened: cmd/snap-mgmt: fix out of source tree build [07:16] mvo: morning [07:17] zyga-ubuntu: get intel ssds, they got crazy 5y warranty [07:17] already had 2 disks replaced under warranty :) not a single complaint [07:18] mborzecki: goooood morning [07:20] yocto segfault thing, i can reproduce it on 2.31 too [07:21] a simple fix that resolves/masks the problem is moving progress bar Finished() call in store dowload to happen after checking the errors [07:21] good run log with breadcrumbs; https://paste.ubuntu.com/26482312/ [07:22] otherwise the crash looks like this: https://paste.ubuntu.com/26482304/ [07:22] 2018/01/29 07:16:11.641340 progress.go:71: progress adapter--- &{task:0x967d8780 unlocked:true label:core total:8.0797696e+07 current:2.784677e+06} [07:22] 2018/01/29 07:16:11.641850 task.go:248: -- in task 0xb77006fc set progress, state 0xe3f [07:24] these 2 lines are interesting, the first one is pbar.Finished() in store.go, the second is a call to task.SetProgress() inside pbar.Finished() [07:24] somehow the pointer to the task changed from 0x967d8780 to 0xb77006fc [07:25] mborzecki: which golang is used on that yocto build? [07:26] 1.9 [07:26] mborzecki: is it a compiler bug maybe? [07:26] mborzecki: any patches in ubuntu or debian or any patches in yocto that may be relevant? [07:26] update to 1.9.3 is on my list, but the interaction cycle is long [07:27] looked through commits between 1.9 and 1.9.3 nothing that looks relevant [07:27] mborzecki: do you think this is something in our code? [07:27] not sure yet [07:28] for instance, no clue why we try to do Kill() on the tomb twice, but the same sequence is visible in the 'good' run :/ [07:28] albeit in the good run there's no call to pbar.Finished() [07:29] mborzecki: oh, doing the Kill twice sounds wrong, do you have more info? [07:29] mborzecki: also no call to Finish sounds wrong :( [07:29] mvo: take a look at the logs i posted [07:30] this is the diiff i'm working with: https://paste.ubuntu.com/26482348/ [07:32] mvo: in taskrunner.Ensure() there's a call to tomb.Kill() for the running task [07:33] aiui that kill will trigger cancel on the context [07:36] mborzecki: interessting [07:43] mborzecki: I guess I'm stating the obvious, it looks like task.state is corrupted, might be interessting (no idea how painful it is to do one of those runs) to output the value of the task.state pointer to see when it becomes invalid [07:45] morning snappy [07:46] hey kalikiana [07:46] kalikiana: morning [07:54] zyga-ubuntu: mvo: forcing a static build of snapd makes the problem go away https://paste.ubuntu.com/26482426/ [07:56] mborzecki: oh, this is also interessting. how dynmaic is the dynamic build? i.e. what is linked? [07:57] hmm ... [08:01] mvo: nothing special, libc and libstd (that's go) [08:01] good morning! [08:01] pstolowski: morning [08:03] mborzecki: I wonder if that would fail if you used a ubuntu-build of snapd [08:03] btw. also according to the forum post the problem is only on x86, does not appear on x86-64 [08:03] mborzecki: I read about some peculiar crash of golang when gentoo used new kernel optimization flag and broke the vsdo [08:04] i've tried rebuilding snapd locally, with same flags but no success [08:04] mborzecki: that was a golang bug, assuming too much about kernel behavior [08:05] i mean, GOARCH=386 GO386=387 CGO_ENABLED={0,1} all worked fine when deployed to the vm [08:05] hoped to use -race but it's 386 ;) so no race for me [08:06] mborzecki: can you try a i386 kernel from ubuntu there? [08:06] and see if the same binary of snapd tests wokrs [08:06] *works [08:06] just a long shot but somethig that could help bisect the area that is broken [08:18] zyga-ubuntu: can you take a look at #4558? this is an easy one [08:18] PR #4558: cmd/snap-mgmt: fix out of source tree build [08:18] yep [08:19] mborzecki: you could use | there [08:19] to make it an ordering-only rule [08:19] mborzecki: just add a | before the new dependency [08:19] either way +1 [08:21] yocto does out of the source builds by default, and this came up there :) [08:22] there's actually a separate class for packages that have broken separation [09:03] zyga-ubuntu: mvo: could you take another look at 4487? [09:07] mborzecki: sure [09:08] looking [09:08] mborzecki: heh, sorry that my suggestion to use "internal" error was wrong [09:11] mborzecki: looks good to me! zyga-ubuntu can merge once he finished (unless he has more suggestions of course) [09:11] mvo: thanks [09:12] yes, I'm just re-reading [09:16] mborzecki: done, have a look please [09:16] brrr, cold, let me make some more tea [09:17] zyga-ubuntu: thanks, good catch [09:20] mvo: I'm juming into lxd issue now [09:20] mvo: I don't have anything to do before 4471 lands [09:21] zyga-ubuntu: cool I'm working on xdg-settings final touches and then ~rc2 [09:21] zyga-ubuntu: but let me know if I can do anything to help with that, happy to tweak the packaging [09:21] mvo: land 4471 [09:21] zyga-ubuntu: but my feeling is right now the solution is not clear [09:21] I think we're wasting time now [09:21] it's blocking everything related to mount work [09:22] zyga-ubuntu: don't we need gustavo for this? [09:22] well, I am waiting [09:22] I'm just saying it's a bit unreasonable now [09:23] (that branch hasn't been touched in weeks) [09:27] * mvo nods [09:38] moin moin [09:39] ho ho [09:39] zyga-ubuntu: http://i.imgur.com/AAGJvD1.png [09:40] Chipaca: \o/ you made my morning [09:40] mvo: woo! [09:40] mvo: how tho :-) [09:40] Chipaca: the subway map [09:41] it's rather brilliant, isn't it [09:41] neat [09:41] Chipaca: oh yes [09:41] love it [10:02] * kalikiana going for a short walk, need a little fresh air [10:04] PR snapd#4559 opened: snap-confine/nvidia: Support legacy biarch trees for GLVND systems [10:04] * ikey now unblocks solus sync. :p [10:16] ikey: [10:16] pretty much lol [10:16] we were gonna sync last night, but i couldnt get the snapd patch (above) working [10:17] turns out i was missing a comma -_- [10:22] mvo, hi, it seems base-18/edge is still missing the distro-info csv that the version you gave me a while ago has. any chance you could push that fix? [10:23] ikey: it's better to miss a comma than to miss the point [10:24] hah [10:24] * zyga-ubuntu gets back to being reasonable [10:24] zyga-ubuntu: I apologize for the delay. My #1 priority last week was to get our CI in good shape, as that unblocks and improves everybody's daily workflow. [10:24] zyga-ubuntu: This week I should get back to reviews and discussions [10:24] niemeyer: I understand that, no apologies needed; I think that code is critical enough to warrant the wait [10:24] niemeyer: I'm trying to get useful in other areas [10:25] niemeyer: and I think the new spread work is brilliant and we will benefit tremendously mid/long term [10:25] niemeyer: (and rest of canonical and anyone who uses it as well) [10:25] I always wished spread either did queueing or do dynamic allocation and now it does [10:25] zyga-ubuntu: I hope the benefit is immediate.. [10:26] niemeyer: it is, I meant that it's not just short term benefit [10:26] zyga-ubuntu: The practical consequence is ~23 minutes runs, and no more limiting on the number of runs, so everybody gets more productive.. [10:26] I postponed that for quite a while last year, but now it was getting a bit unreasonable.. [10:27] and less issues hand holiding systems and re-starting failed tests due to overuse [10:27] Anyway, I hope that's now done [10:27] i dont understand how or why my PR failed [10:28] ikey: concurrent non-deterministic software that is connected to the internet has that property a lot [10:28] :D [10:36] * kalikiana re [10:37] ohh [10:37] * zyga-ubuntu has an insight [10:38] .ubuntu.com? [10:38] >_> [10:39] no, maybe a fix for the bug mvo and me were fighting for a long time [10:39] mvo: no double mounts! [10:40] zyga-ubuntu: tell me more! [10:40] mvo: hold on, just a teaser, let me do more experiments first [10:43] * zyga-ubuntu fiddles thumbs while package builds [10:56] if I could get a review for 4342 that would be great. its the last bit of polish I would like to get in for ~rc2 [11:03] ikey: you drank some punny water this morning? :-P [11:04] perhaps. :p [11:08] mvo: still no double mounts, things look good so far, let me do some more and then try a spread run [11:09] zyga-ubuntu: what is the gist of the alternative idea? [11:09] zyga-ubuntu: I'm really curious [11:09] mvo: well, ... it's silly really [11:09] mvo: one char less than yours probably [11:09] zyga-ubuntu: hu? [11:10] zyga-ubuntu: was there a typo in my unit? [11:10] not really, I don't quite know if you got the double entries because of that [11:10] so so far so good, I'll try to cause the problem by doing what you initially did to confirm [11:10] zyga-ubuntu: so this is a mount unit, we still need all the scaffolding ? i.e. write out from snapd.postinst etc? [11:10] mvo: yes [11:11] zyga-ubuntu: ok - what does this one look like :) please, I'm really curious [11:11] mvo: hold on, one test will tell me if this makes sense, [11:12] hmmmmm [11:12] mvo: so, no double mounts but now I did what you did [11:12] unless lxd is doing something magic [11:12] ok, to be sure I'll just reboot [11:12] brb [11:12] mvo: sorry to keep you waiting, I don't want to say "got it" before it really works and I know *why* [11:13] zyga-ubuntu: no double mounts inside lxd? or no double mounts inside *and* outside of lxd? [11:16] mvo: so, let me check something, I'm on artful now, the container is xenial [11:16] mvo: I disabled reexec to test my build easily [11:16] mvo: (and it works so far) [11:17] mvo: did you do anything different? [11:17] mvo: I'll try to remove the package now, wonder if that will crash like it did before [11:17] but no double mounts, right sharing and snaps can be removed allright [11:17] so that counts as an improvement [11:17] now the last bit is packaging [11:18] zyga-ubuntu: no double mounts outside as well? [11:20] nope [11:20] mvo: actually, I didn't run many snaps and now I see that: [11:20] (though I doubt this is related, probably a bug) [11:20] ubuntu@experiments:~$ hello [11:20] snap-confine has elevated permissions and is not confined but should be. Refusing to continue to avoid permission escalation attacks [11:21] but I heard that there was a bug in lxd+apparmor about recently [11:21] * zyga-ubuntu looks at loaded profiles [11:21] ha [11:21] indeed [11:21] only snapd-managed profiles are active [11:22] we didn't load the profile for snap confine [11:22] mvo: that's a separate bug manifested if you disable re-exec [11:22] Jan 29 11:15:49 experiments apparmor[85]: * Not starting AppArmor in container [11:22] Jan 29 11:15:49 experiments apparmor[85]: ...done. [11:22] jdstrand, stgraber: ^ [11:23] is this reported or am I dreaming things? [11:23] mvo: once I load snap-confine's profile manually things are ok [11:23] mvo: also inside process namespace [11:24] mvo: ok, let me get back and make this into a proper patch I can share with you [11:24] mvo: and see if I can run the test I wrote a while back against it [11:24] jdstrand, stgraber: this is with apparmor 2.10.95-0ubuntu2.7 [11:27] mvo: this still makes little sense to me, it works but the difference between your branch seems meaningless [11:33] hmmm [11:33] mvo: wow, it even removed the package allright [11:33] * zyga-ubuntu is so confused now [11:33] mvo: it only printed tihs [11:33] dpkg: warning: while removing snapd, unable to remove directory '/snap': Device or resource busy - directory may be a mount point? [11:34] and purged cleanly [11:34] :-)) [11:34] * zyga-ubuntu has a happy monday now [11:34] zyga-ubuntu, https://www.youtube.com/watch?v=SsmVgoXDq2w :) [11:35] zyga-ubuntu: did it actually stop your mount unit? [11:35] zyga-ubuntu: it sounds a bit dubious :) [11:36] zyga-ubuntu: but I'm keen to see your PR [11:36] mvo: yes [11:36] zyga-ubuntu: woah, ok [11:36] mvo: it's totally clean [11:36] mvo: it's so weird [11:36] zyga-ubuntu: looking forward to the diff [11:36] zyga-ubuntu: I have a small test in my PR that will be fun to test against [11:36] zyga-ubuntu: by all means, if we have a solution and I will sing and dance and hug you :) [11:41] me too :) [11:42] I'll build another package that doesn't involve any hand-holding anymore [11:42] and re-test [11:42] and then send a PR [11:42] and then find and cherry pick those tests we wrote [11:42] you're so unromantic ! [11:42] ogra_: ? [11:43] I'll build another package that doesn't involve any hand-holding anymore [11:43] but first, my tea has run out [11:43] mvo: btw I have t otell you [11:43] haha [11:43] banning hand holding, and we're coming up to the 14th soon enough [11:43] sheesh [11:43] ogra_: but there will be hugging instead [11:43] ah ! [11:43] mvo: my favourite tea is in stock again, it is called ... [11:43] thats an improvement then :) [11:43] sir adalbert's tea, soursop green tea [11:44] it has a big elephan on a decorative green box, not sure if you know it [11:44] zyga-ubuntu: I don't let me search for it [11:45] zyga-ubuntu: the downside of not sharing an office, I would love to try it :) [11:45] it's awesome, I can bring you a bag next time we meet [11:48] mvo: the trick is that fruit they added, soursop [11:49] maybe if you like traditional tea it's "meh flavoured" but there's something really tasty about it [11:49] zyga-ubuntu: I'm a bit of a traditionalist :) but also open so I'm certainly curious [11:49] zyga-ubuntu: *cough* diff *cough* ;) [11:50] zyga-ubuntu: else I'm gone for lunch and will be curious all the time [11:51] mvo: here it is [11:51] https://github.com/zyga/snapd/tree/fix/fix-snap-share-v3 [11:51] don't open PR yet please, I didn't touch packaging [11:52] mvo: if you build that and run it, does it show duplicates or misbehaves for you? [11:52] I'll proceed with proper tests now [11:53] zyga-ubuntu: heh [11:53] zyga-ubuntu: ConditionVirtualization=lxc [11:53] that's just making it easy to ship [11:53] zyga-ubuntu: I was thinking something like this this morning, if we should make it conditional [11:53] zyga-ubuntu: and it works fine inside? [11:53] yes [11:53] just try it [11:53] zyga-ubuntu: awesome [11:53] I'll fix packaging and get that spread test that tried it merged [11:53] mvo: note [11:53] mvo: the conditional part is irrelevant [11:53] zyga-ubuntu: thank you [11:54] zyga-ubuntu: hu? how so? [11:54] mvo: there's no difference from what you wrote [11:54] mvo: (your unit was started the same way in the end) [11:54] mvo: I really don't know why this works yet [11:54] zyga-ubuntu: it was, but mine ran on classic as well [11:54] mvo: I used a different wants [11:54] ahhhhh [11:54] zyga-ubuntu: and caused all sorts of problems there [11:54] oh, that's it then [11:54] zyga-ubuntu: yeah, thats it :) [11:54] on classic it's not sane [11:54] heh [11:54] zyga-ubuntu: (I think at least) [11:54] sounds like a good day then :) [11:54] zyga-ubuntu: very nice, thank you! [11:54] ok, let me finish the polish on this [11:55] zyga-ubuntu: good luck, I get dinner now [11:55] enjoy! [12:04] mvo, should I file a bug about that missing file? [12:05] spread running [12:05] kyrofa: IT'S HAPPENING [12:08] * zyga-ubuntu thinks about lunch [12:08] PR snapd#4560 opened: many: fix removal of snaps inside LXD [12:12] * pstolowski lunch [12:22] hmmm [12:22] - Download snap "lxd" (5522) from channel "stable" (sha3-384 mismatch for "lxd": got 3ed1910ba0d678da4820c74e35f288db4feb6dcad637466ab6c2bee26fca363b1cdb42e6b81e3f4fb834b672c64457c2 but expected 05ca4d0b5cd936beff0323dd786f2787fdf1783fbb7ffb89c09143c49a6e2269aca4a76e12d34422ea59826129f6009f) [12:22] Chipaca: ^ [12:22] is this store being funky or do I have a bad day? [12:25] is there a way to switch all of snapd to a different default channel? (e.g. install all future snaps and refresh all existing everything from edge) [12:25] zyga-ubuntu: mvo: disabling optimizations and inlining resolves the problem [12:32] sitter: no, not at present [12:40] mvo: things passed in my spread [12:40] mvo: I think it's a winner, let' see how the PR fares [12:52] I got called out of physio to go pick up one of the boys who's being sick all over school (apparently) [12:52] will miss the standup [12:52] ttfn [12:53] mvo: 4560 is ready for review [12:54] brb, see you at the call [12:56] ls [12:58] mvo: that pr needs one more patch, to give the mount unit the right name (following snap mount dir) [12:58] I'll do that [13:00] zyga-ubuntu: ok [13:28] * kalikiana going for lunch break in a few minutes [13:33] PR snapd#4561 opened: tests: generic detection of gadget and kernel snaps [13:52] * kalikiana off for lunch now [13:52] mvo, hey, any specific reason why we abandoned these build stamp / system key branches? [13:54] mvo, hey [14:04] hey cachio [14:04] pstolowski: no specific reason, happy to pick them up again and/or work with you on them. the main issue was that we had too many open PRs and a lack of reviewers so I closed to postpone [14:04] mvo, is the rc2 comming today? [14:05] cachio: thats my goal, however I need a review for the xdg-settings UI branch [14:06] cachio: https://github.com/snapcore/snapd/pull/4342 [14:06] PR #4342: userd: add support for a simple UI that can be used from userd [14:07] mvo, good, I'll take a look [14:08] mvo, I am also working on the test i18n which is failing on master for debian 9 and trusty [14:13] mvo, i see. okay, it seems like a requirement now [14:14] cachio: thank you [14:15] pstolowski: yeah, let me finish my current task, then I can merge master and see how bad the conflicts are :) [14:15] pstolowski: there is also a forum discussion - have you seen that one? [14:17] mvo, not yet, but I think it's linked in one of the PRs [14:18] mvo: another random failure [14:18] (on lxd / 4560) [14:18] I re-started [14:18] mvo, has the ship sailed for 2.31 yet? [14:18] timeout [14:19] kenvandine: no, just initial RCs [14:19] great [14:19] * zyga-ubuntu goes to cook some lunch and then gets back to mount magic [14:19] i thought jamesh's per-user mounts PR would be included [14:19] any chance of that? [14:19] kenvandine: that's possible but unlikely [14:19] :( [14:19] kenvandine: and even if so it won't be "live" [14:19] kenvandine: it will only be useful once we have interfaces using that [14:19] yeah [14:19] kenvandine: I asked jdstrand to review it [14:20] can we get a milestone set on the PR? [14:20] kenvandine: and I will jump onto it soon as now (just this minute) I have my review on my only pending work [14:20] kenvandine: no, I think it's premature [14:20] kenvandine: there are things to do there and it's not something that benefits users yet [14:20] yeah, but we need it in place for other things that we need for 18.04 [14:20] kenvandine: as it will only be used by interfaces and it's not yet, so I can merge it now and it's not changing anything yet [14:21] quickly running out of time [14:21] yeah [14:21] kenvandine: sure but that's not related to merging [14:21] kenvandine: development, sure [14:21] but i'd like to get that stuff in for 2.32 [14:21] kenvandine: I will work on that branch (and related branches) this week for sure [14:21] kenvandine: it's aggressive but we'll see what needs to be done [14:21] i'll talk to james as well, but I think he's already working on stuff that depends on it [14:22] kenvandine: yes, I'm in contact with james [14:22] kenvandine: and we discussed it (james and me) with niemeyer [14:23] kenvandine: for what? session startup? [14:23] freeze is march 1, so we'd really like to land his desktop portal work before then [14:23] mvo, this is for portals [14:23] okay [14:23] * mvo reads backlog [14:24] i'm just trying to make sure all the pieces we need fall into place before feature freeze [14:24] or soon after if we can get a freeze exception [14:24] kenvandine: understood [14:24] kenvandine: are those on halt now? [14:24] or can they be deveoped while this gets ready? [14:24] but we really don't want an LTS release to pass without the portal work [14:24] they can still be worked on [14:25] it's just over a month away though, so i'm getting stressed :) [14:25] niemeyer: ^ FYI [14:25] we can still fix bugs in anything related after of course [14:26] but we prefer not to beg for freeze exceptions for features :) [14:26] which are harder to get for an LTS [14:27] what's the $PWD a "daemon: simple" app is run from? is it $SNAP? [14:28] ackk: I think it could be /root but ... I forgot, sorry [14:29] * zyga-ubuntu -> kitchen [14:33] a "command: bin/mybinary" works so I had thought it was $$NAP [14:34] ackk: no, those are reslved and that doesn't matter [14:35] ah I see [14:35] ackk: those would work regardless of working directory [14:35] zyga-ubuntu, so if snap run --shell gives you the same env I guess it's /root [14:37] ackk: ok, that's good then [14:37] we tried to be consistent [14:37] but for services we wanted to be a bit special [14:37] HOME depends on user [14:38] anyway, [14:38] lunch is ready now [14:40] re [14:40] $HOME is where the <3 is [14:40] * ikey ducks [14:40] zyga-ubuntu: but are you ready for it? ;-) [14:41] ikey: me $HOME es su $HOME [14:41] d'aww [14:41] lol [14:41] that shoul've been "mi...", my Spanish spelling is terrible [14:44] HOME is when you don't have to change your namespace [14:44] * zyga-ubuntu gets back to lunch [14:58] mvo, how should I make to reproduce the scenario with the ui for xdg-settings? [14:58] mvo, I would like to manually test it [14:59] mvo, is there any snap that I could use? [14:59] mvo: man [14:59] luck hates me today [14:59] /dev/stdin: line 2: `504 Gateway Time-out [14:59] maybe you can hit the restart button [15:05] PR snapcraft#1889 opened: tests: remove plainbox part from plugin tests [15:11] cachio: sorry for the delay. you can run "$ go build && ./snap userd" and then install the brave snap [15:11] cachio: and brave will ask you [15:11] cachio: to make itself default browser [15:12] mvo, great, tx [15:12] cachio: you may need to killall snap first if there is one already running in your session [15:16] mvo, ok [15:16] kenvandine: you do realize that you got the review of that PR preempted by another PR, right? ;) [15:16] I guess now that the reprioritized one is done, it is time to have the other one preempt things :P [15:17] anyway, it is on the list. it isn't for 2.31 at this point, so I put it behind the steam one for the moment, but I'll get to it soon [15:18] * jdstrand notes that he will be at the snapcraft sprint this week so will likely not get to everything this week [15:18] jdstrand: o/ [15:19] jdstrand: did you notice my earlier ping about an apparmor bug in apparmor inside lxdd? [15:20] zyga-ubuntu, kenvandine: What are the dependencies for the user mount PR? [15:20] niemeyer: I want jdstrand's ack on it, then we merge and iterate on small chunks [15:21] niemeyer: on specific points to iterate on: maybe handling /media, handling root user, handling updates, etc [15:21] I started to look at that pr on friday, but it is massive [15:21] Yeah, that's not a good thing [15:21] (in terms of interelated changes) [15:22] hmm yocto uses `-linkshared ink against installed Go shared libraries (experimental).` [15:23] taking snapd and linux_386_dynlink/libstd.so to artful i386 i can reproduce the segfault [15:24] the address that ends up replacing task pointer is within a region like: [15:24] b770f000-b7753000 rw-p 012cc000 fd:00 1371 /usr/lib/go/pkg/linux_386_dynlink/libstd.so [15:24] zyga-ubuntu: I saw your comment re lxd. it sounds like the thing that I mentioned last week and alerted mvo to: https://bugs.launchpad.net/ubuntu/+source/snapd/+bug/1744738/comments/2 [15:24] Bug #1744738: snapd 2.29.4.2 ADT test tests/main/lxd failure with linux-hwe 4.13.0-30.33~16.04.1 [15:25] PR snapd#4562 opened: debian: add a zenity|kdialog suggests [15:25] zyga-ubuntu: ie https://irclogs.ubuntu.com/2018/01/09/%23ubuntu-devel.html#t03:34. my understanding is that jjohansen sent up a pr for the hwe kernel [15:25] zyga-ubuntu: is what you're seeing on an hwe kernel? [15:32] jdstrand: I saw this on artful kernel [15:33] jdstrand: with xenial userspace [15:33] zyga-ubuntu: that is essentially the same thing, yes. the hwe kernel is 4.13, like artful [15:34] zyga-ubuntu: it should just naturally fix itself once the patch makes it into the Ubuntu kernel [15:36] zyga-ubuntu: that said, I don't have the timing of this work. jj is traveling. tyhicks, do you happen to have details on the 4.13 kernel fix for apparmor profiles not loading inside lxd containers? [15:39] jdstrand: ack, thank you! [15:44] jdstrand, indeed, totally understand [15:45] jdstrand, just getting things lined up so we have no surprises as feature freeze approaches [15:48] zyga-ubuntu: I added some comments on 4560, would be awesome to have for 2.31 [15:49] zyga-ubuntu: do we need jamie for 4559? [15:50] can someone please review 4342? it soft blocks 2.31~rc2 [15:51] mvo: I saw, thanks [15:52] jdstrand: no, I'm not sure how far along the 4.13 kernel fix is [15:52] kenvandine: note that snapd is not bound by feature freeze [15:52] kenvandine: I'm not sure if the portals side is, but a ffe would certainly be possible [15:53] jdstrand, exactly... but the portal work is [15:53] yeah, possible [15:53] kenvandine: is the portal work something that is ubuntu specific [15:53] and I think that the portals side could continue without it [15:53] jdstrand, just trying to plan ahead :) [15:53] kenvandine: we also have to have portals and snaps work on fedora and debian and opensuse and elsewhere [15:53] zyga-ubuntu, we're doing it with upstream [15:53] kenvandine: sure, I understand. it it just more of the 'everything it critical priority, so nothing is' syndrome [15:53] but it's enabling snaps [15:53] kenvandine: there are no freezes upstream! [15:54] zyga-ubuntu, but to get it in ubuntu :) [15:54] (kudos to upstream work) [15:54] thx [15:57] PR snapd#4517 closed: data: add systemd unit that unshares /snap [15:58] mvo: yes [15:58] jdstrand: low hanging fruit on https://github.com/snapcore/snapd/pull/4559 [15:58] PR #4559: snap-confine/nvidia: Support legacy biarch trees for GLVND systems [15:58] literally one line of lstat of unlink to review [15:58] disablink -linkshared makes the segfault problem go away [15:58] * zyga-ubuntu feels ... sleepy... must coffee [15:59] everything else is enabled [16:00] mborzecki: that's something mwhudson may want to know [16:00] he likely wrote it [16:00] niemeyer: please look at https://travis-ci.org/snapcore/snapd/builds/334699893?utm_source=github_status&utm_medium=notification [16:00] niemeyer: linode hicckup [16:00] niemeyer: returning HTML on api request [16:00] Our development team has been alerted to this error. If you continue to have problems please contact support: [16:01] eee [16:01] gee [16:01] they could _at least_ return that in a RPC response [16:01] mvo hi, around? [16:01] ackk: yes [16:01] with X-Linode-Coupon-Code: header [16:02] mvo any chance you could push the changes to include the distro-info csvs in base-18/edge ? [16:02] mvo, when I set the default browser on brave, then I open a url, it is opened on chrome [16:03] zyga-ubuntu: Thanks, this is the issue I mentioned earlier.. [16:03] zyga-ubuntu: Shouldn't affect the run [16:03] haha once linshared is disabled even remote gdb magically started to work [16:03] But needs fixing as the machines stay around [16:04] ackk: yes, its tiny, let me add it [16:04] cachio: oh? what do you get with "xdg-settings get default-web-browser" ? [16:05] google-chrome.desktop [16:05] mvo, [16:05] cachio: did you get the dialog from snap userd asking for confirmation? it sounds for some reason it did not work. are you on the edge core? [16:06] mvo, beta [16:07] mvo, using the one I just built [16:07] cachio: hm, you *might* need edge, I'm not 100% certain [16:07] mvo, updating [16:07] cachio: you can check if you have /snap/core/current/usr/bin/xdg-settings - it should consists of a bunch of dbus- calls [16:07] refreshing [16:07] cachio: but updating is easier :) [16:07] ackk: pushed, let me trigger a built [16:07] mvo, in progress [16:08] mvo, tx [16:08] mvo, thanks! [16:10] ackk: I pulled in distro-info-data (so just the csv) should be availalbe in ~30min (once the build is finished) [16:10] mvo, is it automatically published? [16:10] ackk: it should be, yes [16:10] ah, cool [16:10] thanks [16:15] kyrofa: https://github.com/snapcore/snapd/pull/4560 [16:15] PR #4560: many: fix removal of snaps inside LXD [16:23] mvo, it seems some builds failed [16:29] ackk: yeah, the ports nameserver lookup failed, looks very internal to me [16:29] ackk: meh, all failed [16:30] ackk: looking at it now [16:32] * zyga-ubuntu afk but will be back, needs to refresh [16:52] lxd pr reviewed [17:00] PR snapcraft#1889 closed: tests: remove plainbox part from plugin tests [17:00] zyga-ubuntu, yeeeeesssssss [17:01] kyrofa: indeed [17:01] kyrofa: we're just polishing the details [17:01] but it works for real [17:02] zyga-ubuntu, no security review yet, does it have security implications? [17:02] kyrofa: nope [17:02] kyrofa: it's all good [17:02] So happy to hear it, thanks for sticking with it zyga-ubuntu [17:02] kyrofa: it will be in 2.31 for sure (next RC) [17:02] * kyrofa subscribes to PR [17:02] kyrofa: I only did that because I had a terrible morning where another HDD broke and it was monday and it was raining and kids were crazy about returning back to school [17:03] kyrofa: it turned out to be a good day after all :) [17:03] Haha, good! [17:03] zyga-ubuntu: there is a bug with apparmor in containers when running on some > 4.4 ubuntu kernel, tyhicks is aware and jj was supposed to look into it [17:04] stgraber: that's the bug that he's referring to [17:04] stgraber: I pointed him to the relevant conversation [17:04] yes, thank you everyone! [17:05] perfect :) [17:05] jdstrand: now that I think about it, didn't jj say that he had test kernels available for testing? [17:05] it's not an immediate probem because snapd manages its own profiles and reexecs but it will bite some people sooner or later [17:05] jdstrand: does that ring any bells for you? if so, I can try to dig up the conversation [17:09] tyhicks: well, yes, it does, but I couldn't remember the deatils so came to you :) [17:09] tyhicks: I don't even know if there is a bug tbh [17:11] jdstrand: the error message seems like it's a deliberate thing [17:11] jdstrand: but unexpected [17:11] jdstrand: it looks like apparmor thinks the container is privileged and doesn't do apparmor stacking [17:11] * kalikiana wrapping up after a frustrating day of debugging symlink issues [17:12] kalikiana: ln -s kalikiana /dev/bed [17:12] or was it the other way around [17:12] I hate "ln -s" syntax [17:13] zyga-ubuntu: haha. you know, I've written a love poem about it, because that command annoys me so much (not a joke in fact) [17:13] zyga-ubuntu, tyhicks: what I recall is that lxd is using a funky way to detect things because libapparmor doesn't provide something better. the kernel never meant to have what lxd is currently checking be depended on. the kernel changed this for some other reason and the check broke [17:13] jdstrand: the problem is that the broken kernel return an empty string at /sys/kernel/security/apparmor/.ns_name rather than the profile name [17:13] jdstrand: and the apparmor init script uses that string being non-empty as a way to detect stacking and so fails [17:14] zyga-ubuntu, tyhicks: so jj was going to fix the kernel for now. I suspect he is going to also provide a libapparmor change, but I don't know that for a fact [17:14] stgraber: right [17:14] jdstrand: the LXD side of the detection actually does work fine, LXD does setup stacking and everything but the container itself can't detect it [17:14] and that was never meant to be 'the right check' [17:14] but it is all you have [17:15] ok, right, it was /lib/apparmor/functions that was wrong [17:15] jdstrand: so it's either a kernel bug that this string is now empty or it's an apparmor init script bug that it's relying on something which is meant to be empty :) [17:15] but it was changed for lxd. anyway... [17:15] stgraber: I think I said as much [17:15] yep [17:19] niemeyer, pedronis I'd be useful to conclude #4401 (but #4400 first) as it will clear the way for interface hooks PR (and will help me a bit with resolving the issue of running hooks on refresh). AFAIR the most problematic point in 4401 was conflict resoluton, I've improved it and looking forward for feedback [17:19] PR #4401: snapstate/ifacestate: auto-connect tasks [17:19] PR #4400: tests/main/searching: handle changes in featured snaps list [17:20] jdstrand: quoting linus, "fu... nvidia"^H^H "never break userspace" :-) [17:20] I'm sure it will be resolved quickly [17:20] niemeyer, pedronis ah, mistake, not after 4400, but after #4440 [17:20] PR #4440: state: unknown tasks handler [17:27] pstolowski: #4440 LGTM [17:27] PR #4440: state: unknown tasks handler [17:32] niemeyer, thanks [17:40] zyga-ubuntu, have you messed with snapd in docker lately? [17:47] kyrofa: no, never [17:47] kyrofa: like literally never ever [17:47] Yeah me neither, but I seem to remember other people running into issues [17:47] kyrofa: is docker the deb something that I can try to use? or should I use docker the snap [17:47] guess I should try today [17:48] zyga-ubuntu, no idea :D [17:48] * zyga-ubuntu handles some important errand at home and will be back to snapd in a moment [18:36] PR snapcraft#1890 opened: Fixed typo [18:49] niemeyer: can you please look at 4560 again please? [18:49] too much pleasure ;-) [18:49] niemeyer: I changed how I generate the unit and made the name dynamic [18:49] and fixed some silly things in that makefile while I was at it [18:49] spread is in progress, local testing was ok [18:50] man, where is everyone, no chipaca, no mvo :/ I should sleep at night and work during the day more [19:01] Question - staged packages in a confined snap.. are they updated when apt is run? Or only when the snap package is refreshed/ [19:05] bashfulrobot, anything in the snap is only updated when the snap is updated [19:05] Remember the snap is read-only [19:07] I suspected. I have a use case for a snap at my work, but I know it will likely have to be classic (display related - remoting protocol). But the base software does not need to be updated often. So I was trying to figure out if there si a negative aspect to snapping it. [19:08] zyga-ubuntu, about the hidraw interface [19:08] zyga-ubuntu, I cannot see it as part of the interfaces in my machine [19:08] zyga-ubuntu, do you [19:08] ? [19:08] zyga-ubuntu, I got disconnected [19:16] cachio: re [19:17] cachio: AFAIK it's juts added in gadget snaps, it's not implicit [19:17] bashfulrobot: snaps can be updated as often as you want [19:17] bashfulrobot: you just have to do the update [19:17] bashfulrobot: snapcraft.io can help, you can just rebuild your pacakge [19:17] *package [19:18] zyga-ubuntu, ok, tx [19:18] Ah ok, so you almost need to rebuild regularly in this case regardless of the packaged software itself. [19:38] * ikey issued "normal" updates for the lsi edge snaps.. [19:54] Thanks for the info zyga-ubuntu [20:09] PR snapcraft#1891 opened: lifecycle: use in-snap mksquashfs if running from snap [20:12] jdstrand: hi there! would it be OK for me to update the reviewer tools to r978? or would that need the extra packages for the resquash work? my reason for asking: there's at least one snap using a 2.30 interface which is failing review because said interface is not yet known by tools r950 which is what the store has [20:13] there is a typo in base-18 Makefile: s,curren/t,current/ [20:13] * s,curren/t,current/, [20:13] mvo ^ in case you're around [20:18] FTR: https://github.com/snapcore/base-18/pull/36 [20:18] PR base-18#36: Makefile: fix typo in base URL [20:23] ackk: looking [20:25] ackk: I cannot merge it, tests failed, I'm looking at why but this looks like something mvo knows better [20:31] zyga-ubuntu, ok. it's probably not related to this change, though [20:33] zyga-ubuntu, can you kick a build on current master to see if it hits the same error? [20:35] ackk: I restarted that test [20:36] chroot: failed to run command ‘/tmp/999-clean-resolv-conf.chroot’: Permission denied [20:37] hmm hmm [20:37] ackk: is this https://github.com/snapcore/base-18/blob/master/hooks/999-clean-resolv-conf.chroot file +x in your branch? [20:37] cna you please check that [20:41] zyga-ubuntu, it's not [20:41] nor in master [20:41] zyga-ubuntu, should I chmod it? [20:42] * ackk tries [20:42] let's see if tests pass now [20:45] yes [20:48] zyga-ubuntu: First comment wasn't addressed or responded [20:50] zyga-ubuntu, tests passed [20:50] niemeyer: oh, let me check [20:51] hmm, that's odd, I responded but my response is gone now [20:52] niemeyer: let me respond again, I certainly didn't ignore it [20:52] and odd [20:52] I cannot respnd to the thread there [20:52] * zyga-ubuntu reloads [20:52] niemeyer: anyway, let's do it here so that we can reach an agreement [20:53] my initial response was that I'd prefer to fail close and early rather than notice the situation we don't support and fail open later when snap is refreshed or removed [20:53] if you think this is too grave concern we can reomve that whole section now, it's not doing anything apart from checking and failing when the externally satisfied precondition is not met [20:57] niemeyer: let me know what you think [20:57] zyga-ubuntu: Yeah, I found it awkward that there's no way to add comments there indeed [20:58] zyga-ubuntu: My concern (and mvo's I think) is not that it's better to fail one way or the other, but rather that we have existing logic and no clue whether people are depending on it today or not [20:58] niemeyer: yes, that's a valid concern, as I said I responded to the comment (somehow) and I wanted to discuss it more hence no changes [20:59] zyga-ubuntu: Before, there was nothing there.. then, in an automatic update, we go from "no worries!" to "BOOM!" [20:59] niemeyer: I think for now we can make the die go away and land this, to the happiness of LXD uses [20:59] and we could look at a way for snapd to measure the environment say "gee, I cnanot work here" perhaps [21:00] zyga-ubuntu: And we need some kind of heads up before we do hard changes [21:00] zyga-ubuntu: So we have a clue if we're breaking any existing environment for which we don't currently have tests for [21:01] niemeyer: shall we printf something or just do nothing at all now? [21:02] (as in rip that code path out) [21:06] zyga-ubuntu: I would not touch that kind of logic right now [21:06] zyga-ubuntu: There's just too much going on [21:06] zyga-ubuntu: Just add a note of what the intention is for the near future and why [21:06] zyga-ubuntu: Then, when everything else settles, we can poke at it and see what happens [21:07] zyga-ubuntu: Certainly with a warning before we do anything aggressive [21:07] ok [21:14] niemeyer: updated [21:16] * zyga-ubuntu doesn't like old make :/ [21:18] zyga-ubuntu, I see the error 'runtime/cgo: ' when we run the i18n test [21:18] zyga-ubuntu: Looking again [21:19] zyga-ubuntu, any idea if it is an error or a message that could be discarded? [21:19] cachio: that looks like one of those go bugs we had [21:19] I thought those were fixed [21:19] are those on 14.04? [21:20] zyga-ubuntu: Can we please not touch that logic right now instead? [21:20] zyga-ubuntu, yes, it is making the i18n test fail [21:20] niemeyer: no [21:20] niemeyer: that logic as it was was broken [21:20] niemeyer: it was either inactive or it was broken [21:20] niemeyer: (see the original bug description from stgraber) [21:20] zyga-ubuntu: Why? [21:20] niemeyer: it was running to late, creating mirror entries [21:20] zyga-ubuntu, it is also failing on debian-9 [21:21] zyga-ubuntu: I won't re-read the *whole* bug right now trying to guess hints of this :) [21:21] cachio: so on 14.04 and on debian-9 we probably have the bug still, I don't know really [21:21] niemeyer: essentially the self-bind mount would duplicate existing mount entries [21:21] niemeyer: and then apply sharing over them [21:21] niemeyer: but keep the old entries intact [21:21] zyga-ubuntu, do you remember in which PR it was fixed for the other systems? so I can take a look [21:21] niemeyer: there were two lines there: a rbind and a rshare [21:22] niemeyer: the rbind gives us a thing that can control sharing (sharing is per mount entry not per directory) [21:22] niemeyer: and the sharing is obvious [21:22] niemeyer: but at that time /snap was already populated [21:22] niemeyer: the result confuses systemd and mount units cannot stop correctly [21:22] niemeyer: that's about it [21:23] cachio: it was a (perhaps more than one) update to the golang compiler, but perhaps I don't recall correctly; mwhudson would know [21:23] * zyga-ubuntu will snap ancient make for self-punishment and testing [21:24] zyga-ubuntu: Yes, I get that.. I thought that was exactly what was fixed [21:24] zyga-ubuntu: and this path shouldn't be reachable given that [21:24] niemeyer: yes! [21:24] niemeyer: that's why I used die [21:24] and argued we could rip it out [21:24] zyga-ubuntu: Well.. now you are saying that if we preserve the logic we have a bug.. [21:25] no no, sorry, we have a comms problem over irc [21:25] if we keep the old bind mount + share code there we will have problems on systems that the new snap.mount unit doesn't fix (e.g. on a non-container with that behavior) [21:25] essentially, that whole code was wrong and it should be removed regardless [21:26] but for initial version of this patch in the morning I used die to see if it would fail anywhere [21:26] one place we could perhaps explore is 14.04 as a non-container [21:26] zyga-ubuntu: If we have code that the change doens't fix, then you'd be blowing it up with die() [21:26] yes [21:26] zyga-ubuntu: hmmm [21:27] niemeyer: but 14.04 is using a different mechanism to fix it [21:27] niemeyer: so nothing (apart from build failure now with make) was broken [21:27] niemeyer: either way, I would leave: printf (as it is now) or nothing; I would not keep that mount rbind, mount rshare code as we used to have [21:28] we *could* keep it [21:28] and it would not alter behavior [21:28] *behaviour [21:28] but I think it's just buggy [21:28] zyga-ubuntu: +? [21:28] we can do that on another iteration if you are worried about something unanticipated [21:28] mwhudson: hmm? [21:28] zyga-ubuntu: "mwhudson should know" -- know what? :) [21:28] zyga-ubuntu: I've been saying that for some time :) [21:28] mwhudson: ah, sorry, cachio saw "cgo runtime" thing again [21:29] zyga-ubuntu: cgo runtime? [21:29] mwhudson: and asked if we there was a patch for that somewhere [21:29] 22:18 < cachio> zyga-ubuntu, I see the error 'runtime/cgo: ' when we run the i18n test [21:29] zyga-ubuntu: i'm just back from leave so have no context at all [21:29] zyga-ubuntu: We have several concurrent ideas of what's supposed to happen: [21:29] 1. Nobody will ever reach this logic [21:29] 2. We want to kill those who reach this logic [21:29] niemeyer: sure, I just wanted to explain the motivation of the patches I sent so far so that my ideas are clear [21:29] 3. We want to preserve the behavior of those reaching this logic [21:30] Those are all incompatible with each other [21:30] I agree [21:30] mwhudson, hey [21:30] mwhudson, https://travis-ci.org/snapcore/snapd/builds/334699893#L2129 [21:30] PR snapcraft#1890 closed: Fixed typo [21:30] niemeyer: 1 can still happen on systems that don't use systemd to boot but we don't support that [21:31] zyga-ubuntu: i don't know what that means [21:31] mwhudson, when we run a test that tries the different languages with the command snap [21:31] mwhudson: I was under the impression that we ran into a bug in golang where exec + fork were racing [21:31] mwhudson: but perhaps that was fixed in the kernel, I don't recall [21:31] zyga-ubuntu: I don't want to go hunting that sort of bugs *right now* [21:31] mwhudson, we can see an error "runtime/cgo:" [21:31] zyga-ubuntu: Or to burn mvo with late releases to fix serious issues in unexpected environments [21:31] niemeyer: understood [21:32] zyga-ubuntu: We need to focus on delivering the few things which are high prio at the moment.. layouts, interface hooks, etc [21:32] I'll restore the code and leave a todo to remove this after the release [21:32] Thanks! [21:32] cachio: oh well i18n is disabled entirely in debian [21:32] niemeyer: thank you! :) [21:32] mwhudson, it is just happening on debian-9 and ubuntu-14.04 [21:32] zyga-ubuntu: aaah the fork/exec thing [21:33] wait one version of that is just a warning [21:33] cachio: was it failing or just printing and you noticed? [21:33] zyga-ubuntu, it is printing [21:33] zyga-ubuntu: Why was the unit file embedded in the makefile? [21:33] zyga-ubuntu, If I reexec it doesn't fail [21:33] hmmm i thought that was fixed in debian though [21:34] zyga-ubuntu: That sounds a bit less nice than what you had there earlier [21:34] oh wait, probably only in buster [21:34] niemeyer: it was nice when it worked, no need to sed, just expand variables [21:34] mwhudson, zyga-ubuntu if I exec the command that failed in a debug session, it works well [21:34] niemeyer: but given that old make sucks I think I must go back to sed [21:34] cachio: it's only ever a 1 in a 1000 type thing, it's a race [21:34] niemeyer: the advantage was that we didn't have to duplicate a rule for a file with variable target name [21:35] niemeyer: anyway, I'll sort it out [21:35] mwhudson, it seems so [21:35] there is also the kernel bug where a setuid executable doesn't get uid=0, no idea if that's fixed in debian yet [21:35] zyga-ubuntu: I don't see the relationship [21:35] zyga-ubuntu: Embedding everything in the Makefile isn't so nice, whether make rocks or not [21:36] niemeyer: it was nice for this case IMO, because I could didn't have to duplicate rules that sed the vars in and get the new filename I had to use [21:38] almost there... [21:39] mwhudson, something that I already tried is to add a sleep between different tries and the issue also happened [21:40] mwhudson, any idea where I could start looking at to fix it? [21:40] cachio: it's a race within the go start up routines [21:41] mwhudson, ah, ok [21:43] zyga-ubuntu, is it ok if we ignore these kind of errors or set this to manual? [21:46] zyga-ubuntu, also we could skip the failing systems until the issue is resolved [21:46] mwhudson, how did you deal with these kind of issues in the past? [21:47] cachio: well, it's not really honest, we should get the relevant fixes to those systems if that makes snapd work better [21:48] roadmr: it is probably ok but I haven't fully verified it [21:48] jdstrand: oh ok... I'd rather wait until you greenlight it properly then [21:48] zyga-ubuntu, of course, but the fixes should go in snapd? [21:48] roadmr: right now most of that is behind env vars [21:49] jdstrand: what can cachio do in the meanwhile, with that interface he wants to use that the old tools don't grok? [21:49] roadmr: but we/I can be responsive to the queue for now too [21:49] cachio: no [21:49] jdstrand: ok... hm this snap was marked as "failed review", not as "held for manual review", whic was odd [21:49] cachio: to kernel/golang AFAIK [21:49] roadmr: he can request a manual review [21:50] cachio: ^^ oh yay! [21:50] cachio: hm the fix for this is https://go-review.googlesource.com/c/go/+/33894 which is in go 1.8 which i thought was what stretch had... [21:51] ah no 1.7 [21:51] mwhudson, ok [21:52] niemeyer: I think this is it [21:52] lets see if tests pass [21:53] roadmr, sure, I'll do [21:53] roadmr, done [21:53] whee :0 cachio that way no need to wait for the tools upgrade because it might be a bit longer than expected [21:53] cachio: great, that puts it in the queue and someone will look shortly. [21:54] roadmr, I'll have to request also for all the other arquitectures [21:54] architectures [21:54] roadmr, np, thanks [21:55] cachio: if it is test-snapd-gpio-memory-control, I just approved it, but you'll need to publish [21:55] jdstrand, yes, thanks [21:56] zyga-ubuntu, i18n is making snapd fail 1/3 executions, so, what do you think is the best idea to deal with this one [21:56] cachio: well, it depends, we could just ignore that result, ideally we'd try to get the fix released if the frequency is that high people will also hit that bug often [21:58] zyga-ubuntu, it is often because the test is going through all the langs and trying LANG=xxx snap [21:58] I see [21:58] well [21:58] I don't think that changes anything [21:59] * zyga-ubuntu reads about dell vmware merger [22:00] imagine a laptop with "vmware" logo on it :) [22:00] on the other hand vmware metal servers would be fun [22:00] zyga-ubuntu: LGTM assuming it works :) [22:00] thank you! [22:01] I'll get the other branch working now [22:02] and in some alternative universe we're all running dell VMs using dell workstation which is a program [22:02] .... [22:02] dell and vmware watch too many mirror universe trek episodes [22:03] haha [22:03] good one [22:07] but, at least dell fusion makes sense, with vmware, that is [22:24] niemeyer: tests are green :) [22:24] we need a 2nd review [22:24] I'll ask mvo for one in the morning [22:36] hmm, no vmware would probably not be affected [22:36] that's curious, I wonder if I could reproduce that [22:37] kyrofa: it looks like I will have clean plate soon (I will work on per-user mounts with jamesh and on layout policy with jdstrand) but this can be my backburner task [22:38] zyga-ubuntu, no problem, just comment on that issue when you're able, I'll leave it open [22:39] kyrofa: actually, with my gazillion tabs lost on my desktop [22:39] do you have a link? [22:39] zyga-ubuntu: fwiw, I'm in UTC-8 this week (rather than UTC+8) [22:39] zyga-ubuntu, https://github.com/nextcloud/nextcloud-snap/issues/425 [22:39] Haha, no problem [22:39] jamesh: oh what happened :( who multiplied you by -1? [22:39] jamesh: oh, cool, sprinting? [22:39] kyrofa: thank you [22:40] roadmr: LOL [22:40] zyga-ubuntu: yeah. At the Snapcraft Summit with kyrofa and others [22:42] cool, is jdstrand there too? [22:42] jamesh please talk to jdstrand about your PR, if he acks it I'll merge and we can iterate [22:42] zyga-ubuntu: I think he's arriving later in the week [22:43] excellent, don't let him leave without that :) [22:43] ;-) [22:43] zyga-ubuntu, jamesh: fyi, I just reviewed it [22:43] jamesh, yeah, Thursday [22:43] ooooh [22:43] * zyga-ubuntu is looking [22:44] I'll be there Wed and Thu [22:44] (leaving tomorrow) [22:44] what https://paste.ubuntu.com/26486123/ [22:45] (tldr snapcraft.storeapi.errors.StoreReleaseError: ) [22:45] jdstrand: nice, thank you! [22:45] kyrofa: ^ [22:45] Oh right, was thinking Thursday/Friday [22:45] Glad you said something :P [22:47] hm i tried running snapcraft login again and now it's just hanging [22:48] mwhudson, hmm... what version [22:48] ? [22:48] 2.35 (stable channel) [22:49] ... because snap install --edge --classic snapcraft was failing too :) [22:49] clearly i shouldn't be near computers today [22:49] Huh, sounds like the store is having issues then [22:49] mwhudson: snap install weekend -> EMONDAY [22:50] kyrofa: could be [22:50] where is that apt thing [22:50] where it prints something odd once in blue moon [22:50] when something else fails [22:50] it's a bit insane [22:50] ah, past midnight something [22:51] zyga-ubuntu: it's a 'man after midnight' think i think [22:51] (and cj-watson's fault) [22:51] ah :D [22:51] it was man [22:52] https://unix.stackexchange.com/questions/405783/why-does-man-print-gimme-gimme-gimme-at-0030 [22:53] kyrofa: it's working now [22:55] Weird