[05:02] morning [05:29] PR snapd#5177 closed: interfaces/builtin: allow access to libGLESv* too for opengl interface [05:29] PR snapd#5180 closed: tests/main/snap-service-timer: account for service timer being in the 'running' state [05:34] PR snapd#4497 closed: many: make rebooting of core on refresh immediate, refactor logic around it [05:34] 5181 looks like an easy win if someone wants to do a quick review [05:36] PR snapd#5181 closed: userd: add the "snap" scheme to the whitelist [05:37] yay, down to 25 PRs [05:37] * mvo hugs mborzecki [05:37] yup [05:40] wonder if we should have a `snap timers` command, or include some information in the output of `snap services` that some services are timer activated [06:24] mvo: hi. I was wondering if you could help me in registering another snap package for use in some snapd spread tests [06:26] mvo: this is to test some interfaces for evolution-data-server's contacts and calendar APIs: so it needs a binary client. [06:26] jamesh: sure, I just need a (possible empty) snap and I can do this [06:28] mvo: I've got a 13 MB test-snapd-eds_0.1_amd64.snap. Let me see if I can upload it somewhere [06:29] mvo: or can you just register the name and add me as a collaborator? [06:33] jamesh: I can register the name but canont add people before there is a snap. its a store thing [06:34] mvo: That changed a couple of weeks back. You can add collaborators from registration now. [06:34] wgrant: oh, cool [06:35] jamesh: I can add you without a snap now, thanks to wgrant for this tip [06:36] okay. Well the snap is currently "test-snapd-eds" -- I used a single test snap for both calendar and contacts, since they share much of the client code [06:36] jamesh: you should have access now [06:37] just as I finish uploading it to https://people.canonical.com/~jamesh/test-snapd-eds_0.1_amd64.snap :) [06:38] jamesh: heh, sorry [06:38] mvo: is there a need to have all *.deb and all *.a files in the all snap image for tests? [06:38] no problem. Blame my slow ADSL :) [06:39] I might actually get something faster this year, assuming the NBN rollout doesn't get delayed again [06:39] mborzecki: probably not, this is the integraton test? [06:39] mvo: yes in ubuntu-core prepare in #5134 [06:40] PR #5134: Shrink image generated with snap prepare [06:40] jamesh: nice, how much do you have now and how much will you get (if I may ask)? [06:41] mborzecki: hm, hm, lets kill it. is that triggering the space problem? [06:42] mvo: yeah, the built image is running out of space, and we seem to take 60-80MB of space just with the archives and debs [06:43] mvo: On ADSL currently, I've got about 7.6 Mb/s down, 0.9 up. NBN could potentially go as high as 100/40 (but probably won't hit that in practice) [06:44] jamesh: nice improvement! [06:52] jamesh: that's nice! is it expensive? [06:53] mvo: so the image is ~400MB, data partition is 346MB, once it's built we have 80MB left, /home/gopath (all of it is copied in prepare) is ~125MB [06:53] mborzecki: haven't actually looked at the pricing, since it has seemed so far off. There are a number of speed tiers for the NBN ranging from 12/1 up to 100/40. So I'll probably be constrained by price rather than technology [06:55] mborzecki: looks like my current ISP is offering AU$70/month for the 12 Mb/s plan, going up to AU$100 for 100 Mb/s [06:55] jamesh: i have some shitty adsl here, it's a bit outside of the city and i'm bit far from the concentrator, paying for 20mbps, actually getting ~12-13mbps, upside is it really costs pennies, ~9eur/month [06:56] jamesh: for comparison, if i were in the city, fiber 100/50 is ~10eur [06:58] mborzecki: my current ADSL is $80/month for unlimited data (roughly what the 50MB/s NBN plan costs). That's a "naked DSL" plan, so there is no line rental on top of that (which would usually be ~ $25/month) [06:59] actually, looks like I'm on a grandfathered plan at $70/month [06:59] anyway: a lot more than what you're paying :) [06:59] jamesh: yeah === pstolowski|afk is now known as pstolowski [07:03] morning [07:04] mvo: Okay, I've an amd64 version of the snap uploaded and requested manual review. Thank you for your help. [07:05] zyga, i understood the problem you have on your box, i'm starting working on a fix [07:07] jamesh: it is in but you should talk to jamie to ensure the review-tools gets updated [07:09] mvo: hey, i needed to try to reproduce an issue with an older revision of edge yesterday, but apparently i'm not allowed to specify --revision for core unless i'm in the core snap dev group. i think i don't need older edge anymore (the bug is present in current one as well), but perhaps it may be useful in the future [07:10] mvo: can you add me to the core devs? [07:12] pstolowski: if you need it to reproduce I'm happy to add you, otherwise I prefer to keep the list as small as possible [07:13] mvo: i see, fair enough [07:14] mvo: i don't think reproducing this particular problem is important anymore, i understand it now [07:14] mvo: it's more about future cases like this [07:16] mvo: so i guess i'm fine if it's on case-by-case basis as need arises in the future, and can be temporary membership [07:16] pstolowski: thank you [07:16] pstolowski: that sounds reasonable [07:16] great [07:18] PR snapd#5182 opened: snapstate/hooks: reorder autoconnect and reconnect hooks [07:19] mvo: heh, ok, so the amount of free space is probably some factor of the amount of data in the system partition, with ondra's change, that size has shrunk, so there is less free space and the whole image is ~60% of the old size, https://paste.ubuntu.com/p/ghdFfrT9yJ/ [07:20] mvo: I think I just have to live with the manual review until the new interfaces get merged to snapd: the review tools are driven off the current snapd version [07:20] jamesh: ok [07:21] mborzecki: oh, woah [07:23] mvo: well, it'd be nice if there was a way to pass desired amount of free space in the image [07:24] mborzecki: don't we calc the image size during the tests? or does our code have no control over that? (pardon my ignorance, I don't remember without looking to the test scripts) [07:25] mvo: we don't [07:26] mvo: well, for sure we don't need go *.a files and *.debs, this will save enough space, then we can think of hardcoding the image size or asking ubuntu-image guys how to force some amount of free space [07:27] mborzecki: hrm, hrm, I'm sure I miss something. the writable partition should be big. or are using something else here? [07:27] mborzecki: yeah, this all sounds dubious, iirc ubuntu-image has some code that calculates some free space area but there is iirc an option to override (and again, I think I miss something) [07:33] mvo: afaiu ondra's change, we no longer have 2 copies of the snaps, but instead /var/lib/snapd/snaps files are symlinked to ones from /var/lib/snapd/seed/snaps/ [07:36] mborzecki I don't think this has landed yet, there seem to be some mysterious lock down in tests [07:38] ondra: Could not get lock /var/lib/apt/lists/lock thing? iirc it was resolved by cachio or sth [07:38] mborzecki yeah that one [07:38] mborzecki what was the problem? and can somebody then restart test so we can re-check pull request [07:39] ondra: i'm looking in the no space left thing [07:40] ondra: your change seems to have triggered a problem elsewhere [07:41] ondra: i can keep looking into it and push a fix if you don't mind [07:45] mborzecki: isn't it "just" a matter of tweaking tests/lib/prepare.sh:348 (the ubuntu-image call) [07:46] mvo yeah that would great [07:46] mborzecki: and adding --image-size=1G or something? [07:46] mborzecki: or whatever we need (not sure if the file is sparse so we may need to ensure its not crazy :) [07:46] mvo: yes [07:47] mborzecki: cool, thank you! [07:47] mborzecki: and thanks for diving into it :) [07:48] * Son_Goku groans to life [07:57] moin moin [07:58] hey Chipaca ! good morning [07:58] mvo: welcome back! how're things? [07:58] Chipaca: good, good. how are you? [07:58] mvo: hm --image-size 700M, the file is 700MB, partition is still 360MB :/ === LarreaMikel1 is now known as LarreaMikel [07:58] mborzecki: :( booo [07:59] i'm without context, but resize was done from initramfs wasn't it? [07:59] mvo: sil2100 says we'd have to list the partition in gadget.yaml to set the desired size [08:00] Chipaca: https://github.com/snapcore/snapd/pull/5134#issuecomment-390898663 [08:00] PR #5134: Shrink image generated with snap prepare [08:00] mborzecki: don't we use writable? [08:00] i suspected it was related to that :-) [08:01] mvo: writable? [08:01] i'm getting a «Running task 3777 on Doing: Reconnect interfaces for snap "core"» every 5 seconds [08:01] Chipaca: talk to pstolowski [08:01] pstolowski: OHAI [08:02] mborzecki: we have two partitons (well, three) on core, system-boot and writable - I'm pretty sure that --image-size used to work to make writable bigger (but not system-boot which should be irrelevant for this test or so I hope) [08:04] mvo: hm maybe it does not work if the partition is not listed in gadget.yaml [08:04] Chipaca: yes, there is at least one critical bug around there that I'm fixing right now. can your try abort it [08:04] ? [08:04] Chipaca: i started looking into the snapd snap and hit bug #1772584 - I think I might have an idea for a workaround but its all a bit messy [08:04] Bug #1772584: Having a "snap" directory with actual content causes build failures [08:04] mvo: this is what we have now https://paste.ubuntu.com/p/qStWjg56NV/ [08:04] pstolowski: done [08:04] mborzecki: yeah, thats the default and this should work [08:05] mborzecki: I can look in a wee bit if that blocks you [08:05] mborzecki: got an error (not reported because I've got SNAPPY_TESTING=1) [08:05] um [08:05] pstolowski: ^ that one was for you [08:06] pstolowski: should I pastebin the report for you instead? [08:06] Chipaca: yes please, not sure what's the context [08:06] by error i mean an oops [08:07] pstolowski: I did "sudo snap abort --last=auto-refresh" [08:07] pstolowski: that succeeded [08:07] pstolowski: or more or less succeeded :-) [08:07] pstolowski: and that ended up in an error i think [08:07] pstolowski: I'll pastebin the whole thing, then you tell me [08:08] mvo: not really blocki, just wanted to push that PR a bit further [08:08] pstolowski: this is the log, including the oops at the end: https://pastebin.ubuntu.com/p/75RZSmVGmm/ [08:08] mborzecki: yeah, I think this is great (pusing it further) [08:09] pstolowski: let me know if you need anything further to figure it out [08:11] Chipaca: can you send me your state.json? [08:16] pstolowski: people.canonical.com:/home/john/state.json.bz2 [08:16] PR snapd#5183 opened: snapcraft.yaml: add minimal snapcraft.yaml with custom build [08:18] pstolowski: if you can't ssh to p.c.c i can put it elsewhere [08:18] Chipaca: indeed i can't [08:18] pstolowski: you need the vpn up, fwiw [08:18] i haven't used it for years [08:19] Chipaca: right. can you email it? [08:20] pstolowski: sent [08:20] it's 12MB uncompressed, so I bzipped [08:21] pstolowski: also i formatted it and removed secrets from it (i think :-) ) [08:21] Chipaca: good, thanks [08:23] mvo: core in edge is very broken because of reconnect bug, can we prevent people from using it (in fact last few revisions are broken)? [08:23] pstolowski: I think so, I can revert edge to a previous revision, if you tell me which one [08:24] mvo: problem is the PR landed 10 days aog [08:24] *ago [08:24] so you can hit the bug with any revision since then [08:27] pstolowski: ok [08:35] Chipaca: if at some point later you have time to talk about the snapd snap that would be great, it feels like we have a lot of work ahead of us :/ I will try to write up something about what we need to do to get there [08:35] Chipaca: i.e. what is missing etc [08:35] mvo: ok [08:35] mvo: right now i'm working on the conn check branch, beating tests into submission [08:36] Chipaca: sounds good [08:38] what is the interface to choose if i wish to expose the Ethernet port? [08:42] eraserpencil: https://docs.snapcraft.io/reference/interfaces [08:42] that's the full list. we don't have one for the ethernet device/port, it's "network" [08:42] (and network-bind if you want to run a server/service) [08:43] Chipaca: the bug you hit is same as zyga's [08:43] pstolowski: huzzah [08:44] High five [08:44] All 70000 of them [08:44] so that's what I'm fixing. there may also be a related problem with conflict checking but i haven't looked into that yet [08:46] basically the issue is that i iterate over all potential connections and create tasks as I go. however, if an error is detected during the iteration (such as conflict with another snap op) i error out (with Retry in some cases), and that leaves already created tasks and they are not assigned to a change. so the main task is retried and it all repeats [08:49] pstolowski: does 4646 2018-05-11T04:29:24Z arm64 16-2.32.6+git716.83cec93 edge looks reasonable to go back to? the next one we have is 2018-05-14T04:30:30Z [08:53] mvo: 1 moment [08:57] mvo: that wouldn't include https://github.com/snapcore/snapd/pull/5120 that landed on 05/11 right? [08:57] PR #5120: interfaces: interface hooks for refresh

[08:59] pstolowski: it seems it does not, the r4646 was build 2018-05-11 at 04:29 in the morning [09:00] mvo: good, that solves it for edge [09:01] mvo: i fear we have a potential to hit similar issue with autoconnect logic, and that's in stable already right? [09:02] pstolowski: autoconnect is in stable, yes [09:02] pstolowski: hm, at least iirc, let me double check [09:02] pstolowski: that is "Done 2018-05-22T09:02:31Z 2018-05-22T09:02:33Z Automatically connect eligible plugs and slots of snap "test-snapd-tools" [09:02] " ? [09:03] mvo: yes [09:03] pstolowski: thats in stable [09:04] right, that's what i remembered [09:07] mvo: feedback on my changes to your conncheck pr appreciated [09:09] * Chipaca off to physio [09:17] pstolowski: I was reading the logs a bit, what's the issue? never ending retry cycles? [09:18] pedronis: yes, and continuous adding of hook tasks because of that, and they have no change associated if error out on Retry [09:19] pstolowski: the latter is expected, we prune them but probably could be more aggressive, see state.go [09:20] pedronis: hmm that's interesting, i wasn't aware of that [09:20] pedronis: also while working on a fix i think i found another general flaw in reconnect logic [09:20] pstolowski: do we know why the we retry forever? some kind of recursive case we didn't consider? [09:20] but that's probably not directly involved in the issue [09:21] pedronis: i haven't actually got the retry logic yet, you found general problem with error handling and focues on that first [09:22] d/you/ [09:22] ok [09:22] let me know if I can help [09:23] thanks [09:24] * Chipaca really off to physio now [09:36] pedronis: would you like to take a look at John's state.json to see if you can spot the cause of retries? [09:37] nb, i think my remark about general flaw in reconnect logic may be premature [09:43] pstolowski: the issue might be related to the ordering of refreshes we do now [09:43] pedronis: you mean ordering of refresh vs autoconnect? [09:44] pstolowski: the fact that we refresh core first and then other snaps in a multi-refresh, I suppose auto-connect/or reconnect of core will never finish now [09:48] Chipaca: looks great, I will do a formal review now [09:50] pstolowski: I think we didn't notice with auto-connect because we don't do anything if the connection is already there which is the common case with core and another snap [09:53] pstolowski: you should try to write a test, have core and another snap that has a connected interface to core, then do a refresh (UpdateMany) that needs to refresh both core and this other one, I think you get the bug [09:53] pedronis: thanks, this sounds plausible [09:58] pedronis: btw, i also proposed https://github.com/snapcore/snapd/pull/5182 [09:58] PR #5182: snapstate/hooks: reorder autoconnect and reconnect hooks [09:59] popey: I was checking the documentation on that, and erm just double checking if there was an "ethernet" option. Thanks anyway, I'll try out network. [10:16] PR snapd#5184 opened: [WIP] interfaces: add desktop-{contacts,calendar}-service interfaces [10:49] PR snapd#5185 opened: tests: shellchecks part 2 [10:50] pedronis: thanks for the review [11:05] fwiw #5134 is green now [11:05] PR #5134: Shrink image generated with snap prepare [11:10] mborzecki that helped :) [11:10] mborzecki tests do pass now [11:10] mborzecki I added you there also comment how to actually change writable size, in case this ever comes back [11:12] ondra: i tried something like this and ubuntu-image seems to ignore any edits to gadget.yaml :/ is there anyhing else i'd need to do beside dropping a new partition entry? [11:13] mborzecki not really, that there is bug in u-i [11:13] mborzecki it should not ignore that section [11:14] ondra: maybe what i added made no sense (but then there was no error or whatever) [11:15] mborzecki you can paste it to me, and I will see if I can spot some problem [11:16] ondra: let me try your snippet [11:19] pstolowski: btw, are you working on writing the test I described? [11:20] not yet, but getting to it, addressed your feedback re reorder PR and finished part of the fix related to error handling. onto test now. [11:22] Chipaca: what was your terrible snap called again? [11:25] ah interdenominational-counterintelligences [11:26] mwhudson: yup [11:26] Chipaca: ah heh another non-maximal field! [11:26] Chipaca: publiser [11:26] *publisher [11:26] (or did you say that already) [11:27] mwhudson: I did not [11:27] mwhudson: good point. Want to register a maximal lp user? [11:27] i'm not sure what the upper limit is there [11:30] mwhudson: probably less than an exabyte [11:30] probably [11:34] https://staging.launchpad.net/~m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789 [11:34] Chipaca: ^ [11:35] * Chipaca hugs mwhudson [11:36] https://staging.launchpad.net/~ [11:36] m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123 [11:36] 456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m1234567 [11:36] 89m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m1 [11:36] 23456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m12345 [11:36] 6789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789 [11:36] m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123 [11:36] 456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m1234567 [11:36] 89m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789m123456789 [11:36] yeah ok so maybe not much less than an exabyte [11:36] (sorry!) [11:39] mwhudson: if it's just a text field, it might not be much less [11:39] ondra: looks like gadget.yaml comes from gadget snap which is downloaded on every build [11:39] Chipaca: that user has a 3600 char name so... [11:39] mwhudson: I would argue that it's a bug if it's longer than ~20 chars, myself [11:40] Good mornings! [11:40] although I might be convinced to thinking 32 isn't too bad either [11:41] Chipaca: should we s/url/host/ in a bunch of docs, variables in the connectivity check PR ? [11:41] pedronis: no [11:41] pedronis: accurate documentation is for the feeble-minded [11:41] ok :) [11:41] * Chipaca fixes [11:47] PR snapd#5080 closed: many: support 'system' nickname in interfaces [11:47] pedronis: fixed [11:49] Chipaca: I'm removing your +1 because it has changed so much by you since [11:49] pedronis: that seems fair to me :-) [11:49] I +1'ed it, and then rewrote it [11:49] :-D [11:49] "yeah that's fine, let's do something COMPLETELY DIFFERENT" [11:49] *merge* [11:49] I've been taking lessons from the US congress [11:50] let's add a random thing at the last minute before the vote? fix conflicts, non-sense laters? [11:51] seems it's a style there [11:51] > @niemeyer approved this pull request. [11:51] * Chipaca dances [11:52] i feel like i should bring out the champagne and the cigars [11:52] Chipaca: (snap)shots for everyone? [11:52] mborzecki: let me sneakily rename them to schnappshots before the merge [11:52] * niemeyer grabs the domain name [11:53] PR snapd#5066 closed: overlord/snapshotstate/backend: introducing the snapshot backend [12:08] off to school, back for standup [12:12] mwhudson ah if you are using gadget snap from store, then you are out of luck [12:12] mwhudson ups sorry wrong nick [12:13] mborzecki ah if you are using gadget snap from store, then you are out of luck [12:13] mwhudson you can always change u-i command and pass it own gadget snap instead, but that will make test a bit more static [12:13] ondra: you did it again! [12:14] mwhudson damn irc client, sorry for that [12:14] heh heh it's funny [12:14] no worries :) [12:14] * mwhudson goes to bed [12:14] mborzecki you can always change u-i command and pass it own gadget snap instead, but that will make test a bit more static [12:14] good night :) [12:38] Son_Goku, hey [12:38] yo [12:38] Son_Goku, I am working again with the fedora 28 for google [12:38] I just see the oslogin package [12:38] which ones do you need? [12:39] Son_Goku, https://paste.ubuntu.com/p/g9HByWRgbg/ [12:39] those [12:41] okay, so you need all three [12:41] I'll try to work on that today [12:42] Son_Goku, great, thanks a lot === pbek_ is now known as pbek [12:45] re [12:45] master unit tests are broken, i'll be opening a pr in a while [12:47] there should be a github feature that triggers a build on a PR whenever it's about to be merged into master [12:54] PR snapd#5186 opened: daemon: update unit tests to match current master [12:55] should fix master builds ^^ [13:14] joc, thresh, diddledan, popey, Wimpress https://forum.snapcraft.io/t/call-for-testing-snapcraft-2-42-1/5540 has the snapcraft you want or need (a quick validation, if time permits, would be nice) [13:14] hah [13:14] you always have the snapcraft I need [13:15] sergiusens: thanks, how quick were you hoping for? [13:15] sergiusens: looks like I'm already on edge, so have been using that for at least the last 24 hours [13:15] joc: the test can be quick, but it is not urgent ;-) [13:16] popey: thanks [13:17] sergiusens: do you want me to write a forum post about the "snap" directory isssue outlined in 1772584? [13:17] mvo: this bug plays nicely with the fact that we also want to make `parts` relocatable for build environments [13:18] but this one is a bit more special even [13:18] sergiusens: being able to put it all under .snapcraft might also be an option [13:18] because it is our main entry point into the source of truth to build [13:18] sergiusens: I don't really mind, but the hack I had to do to make it work makes my cry [13:18] mvo: yeah, let's start with a post to brainstomr [13:19] sergiusens: cool, will do [13:19] hacks always make us want to cry until they stand the test of time, from then on, we are proud of them :-P [13:22] sergiusens: I used it yesterday from edge (purposefully) to build gimp 2.10.2 - works great - fixes the argument length issue gimp build was encountering [13:23] yay [13:23] https://usercontent.irccloud-cdn.com/file/11QDTn3O/Good%20news%21 [13:24] sergiusens, hey I've tried the edge multiple times and it worked fine for me - no time to test the actual release atm, sorry [13:25] speaking of gimp - I've been building the world because glib needed to be newer than that of xenial, but because I'm now not using the themes and such from the ubuntu repo I'm missing icons that I can't work out how to get back - I'm building gnome-themes-extra and gnome-icon-theme and gnome-icon-theme-extra and adwaita-icon-theme but still there are missing icons :-( [13:25] oh, and hicolor-icon-theme, too [13:26] diddledan: is it possible to build using some of the magic kenvandine does to get newer gnome in desktop snaps? (or use 18 core) [13:27] using 18 core would probably fix it [13:27] how do I do that? [13:27] I installed a snap the other day that pulled in core 18, surprised people are using it already [13:27] https://forum.snapcraft.io/t/the-road-to-core18/5547 [13:27] it's not official yet is it? [13:27] oh, that's not what you want [13:27] "Sorry, you don't have access to that topic!" :-p [13:28] that's a secrit topic [13:28] ignore that then [13:28] sorry. [13:28] * diddledan haxx0rs it [13:28] mmm fresh warez [13:28] i think there's some work still to do in the store and snapcraft before core18 is commonplace [13:29] thresh: having used edge is good enough, thanks! [13:30] * diddledan surmises that popey has insider information from a secrit topic that he can't tell me about that makes popey "think there's some work still to do". 'the road to core18' is long and winding? :-p [13:30] PR snapd#5187 opened: overlord: introduce snapshotstate [13:48] PR snapd#5186 closed: daemon: update unit tests to match current master [13:54] cachio: can you post the *.timer file somewhere? [14:06] roadmr: hi! can you pull r1074 of the review tools? note that most of the changes are around snap usn support, but there are a few things for regular reviews [14:07] jdstrand: sure! [14:07] thanks! [14:07] jdstrand: we still haven't re-enabled resquash enforcement, right? [14:07] mborzecki, well, it fails to install and it doers not leave the .timer [14:07] note to channel: ^ that includes support for the new 'timer' yaml [14:07] mborzecki, is it any way to see the .timer in that case? [14:08] roadmr: that is correct. plan to do it the week of June 4th. electron upstream is the last thing and they are working on it (several commits already pushed) [14:08] cachio: no i think not, what's your snapd version? [14:08] 2.32.8 [14:10] cachio: w8, timers are in 2.33, any timer entry you have there will be ignored [14:11] cachio: note, I'm approving your gce-cleaner snap now [14:11] jdstrand, ok, thanks [14:11] mborzecki, weird, I tried with edge and I had same problem [14:11] cachio: all approved. you'll need to publish to a channel [14:11] mborzecki, let me check it again [14:12] jdstrand, perfect, tx [14:13] cachio: can you double check that there are no snap.*.timer files before you attempt to install that snap? [14:13] mborzecki, yes [14:13] mborzecki, I am refreshing core now, let me try once it is finished [14:15]