[00:09] sergiusens, snapcraft#1717 armhf had network problems during one of the ROS tests... so that won't tell us anything. Waiting on arm64 [00:09] PR snapcraft#1717: catkin plugin: check for pip packages in part only [00:41] * ikey is doing his last clean local builds ready for upload :3 [01:03] so uhm [01:03] Pushing solus-runtime-gaming_0.0.0_amd64.snap [===========================] 100% [01:03] [1] 9686 segmentation fault snapcraft push solus-runtime-gaming_0.0.0_amd64.snap [01:03] idk what happened [01:03] processing went kersplat [01:04] and i believe thats because it never expected to need a manual review [01:04] ikey it is python, I don't expect a segmentation fault but a traceback instead [01:04] python is special [01:04] this is much worse of a thing to track down I fear [01:05] yeah [01:05] I will look into it [01:05] so who manually reviews snaps? :D [01:05] ikey among them is jdstrand [01:05] oh ok [01:06] my assumption is i can't actually upload my lsi snap until the base snap is in as it depends on it [01:10] ikey so the one that failed to push is the one with a base decl? [01:10] yeah [01:10] (NEEDS REVIEW) type 'base' not allowed lint-snap-v2_snap_type_redflag [01:10] 0 Warnings [01:10] 22 Passes [01:11] kyrofa the arm64 test might take for ever and ever [01:11] full error: https://hastebin.com/raw/vayobosulu [01:11] setuid binaries won't be usable inside there anyway [01:11] it also dislikes symlinks :p === JoshStrobl is now known as JoshStrobl|zzz [01:14] * ikey will nuke them for the next build to clean it up [01:23] wtf i cant use bin in my snap [01:29] bin/linux-steam-integration.exec glxgears does not exist lint-snap-v2_command (glxgears) [01:29] ok the review system is dumb. [01:30] like clinically stupid [01:31] command: bin/linux-steam-integration.exec linux-steam-integration.settings [01:31] it won't accept that [01:33] * ikey is unable to upload snaps at all, woo. [01:54] * ikey made a workaround ^^ [01:55] ikey we in snapcraft wrap those in scripts [01:55] the `command` [01:55] yeah i just made it do the same [01:55] https://github.com/solus-project/runtime-snaps/commit/3cdf7f2d9f06187a76e26465551f470561be3de2 [01:55] "tada" [01:55] I don't think that review tool is geared toward reviewing base snaps [01:56] oh that rejection was for LSI itself [01:56] ive now got LSI actually uploaded and accepted [01:56] but yeah it hates base snaps and forbids them [01:56] so the solus-runtime-gaming is still pending [01:57] ive cleaned up my base runtime for those other errors so im actually going to abandon the initial review and upload the cleaner guy [01:57] * ikey is totally OCD [01:57] there, rejected [01:58] * ikey builds the new clean guy [01:58] kyrofa your arm64 adt run started! [01:58] PR snapcraft#1736 closed: recording: pass the build info flag to the container [01:58] sergiusens, yeah you're right... still waiting on that :( [01:58] Oh! Haha [01:58] Well that's good. Another four hours... [01:59] kyrofa I am going to go to bed though or I will be toast tomorrow [01:59] sergiusens, yeah not worth it [01:59] We'll pick it up again in the morning. Almost there [01:59] kyrofa I might wake up super early though [02:00] I forgot I need to prepare my presentation for pyconar this weekend too [02:00] oh, and Monday is a holiday [02:00] so many things to do, so little time [02:00] Yeah I hear you! [02:00] sergiusens, sleep well, talk tomorrow :) [02:00] o wat dashboard has download stats [02:02] i bet i could build a tiny bit of machinery for these lsi bits and have it automate a lot of stuff like the wrappers snapcraft style.. [02:03] * ikey creates more work for himself [02:08] ok linux-steam-integration rev3 is on edge now [02:08] devmode [02:13] PR snapcraft#1732 closed: tests: dotnet only works on 16.04 [02:14] kyrofa I think you looked at the execution location incorrectly snapcraft#1717 armhf -> https://pastebin.ubuntu.com/25964709/ this might be a thing for elopio [02:14] PR snapcraft#1717: catkin plugin: check for pip packages in part only === chihchun_afk is now known as chihchun [02:34] i fixed the remaining issues with the solus-runtime-gaming, has 0 warnings now [02:34] just 1 error [02:34] its a base snap [02:34] and it requested review by itself [03:31] kyrofa: sergiusens: found it: https://github.com/snapcore/snapcraft/pull/1717/commits/b1768cc851adc1c94e08a063b2479269805a4580 [03:31] PR snapcraft#1717: catkin plugin: check for pip packages in part only [03:31] snappy-m-o autopkgtest 1717 xenial:armhf xenial:arm64 [03:31] elopio: I've just triggered your test. [04:00] sergiusens, elopio nooooooooooooo [04:01] sergiusens, I didn't scroll back that far, just saw some errors in the middle of catkin tests [04:02] Well, should be done by morning-ish anyway [04:02] elopio, thanks for pushing a fix :) === JanC_ is now known as JanC === JanC is now known as Guest70158 === JanC_ is now known as JanC === JanC is now known as Guest67213 === JanC__ is now known as JanC === JanC__ is now known as JanC [05:00] snappy-m-o autopkgtest 1717 xenial:armhf xenial:arm64 [05:00] elopio: I've just triggered your test. [06:41] snappy-m-o autopkgtest 1717 xenial:arm64 [06:41] elopio: I've just triggered your test. [06:46] o/ [07:04] mvo: hey, good morning; I'm just waking up here, any fires related to debian image update? [07:07] zyga: good morning. all quiet on the debian image [07:07] zyga: at least afaict, not many PRs since last night it seems [07:08] zyga: but at least some from jamie that did succeed [07:08] interesting [07:08] I'll boot sid and see if anything was reverted [07:09] last night cachio helped me refresh the image so in theory we should be running apparmor now [07:09] moin moin [07:10] zyga: cool [07:10] zyga: yeah, I have some 2.30 tasks and need to chase why 2.29 is not in stable yet [07:11] mvo: any fires there or just unknown? [07:12] zyga: I'm trying to find this out now [07:16] zyga: looks like it was a test that was giving unexpected results but upon re-testing its all good and the original traces of the issue are no longer available (which is slightly sad) [07:24] hi, vlc from stable is no installable. I reported bug 1732359 in LP but I am not sure it's the right place. [07:24] Bug #1732359: installation snap vlc / stable fails with: ERROR open /snap/vlc/7/meta/gui/vlc.desktop: no such file or directory [07:24] not* installable [07:26] jibel: thanks, looking [07:26] jibel: and good morning :)! [07:27] jibel: I can reproduce, its a broken symlink in the snap [07:30] Good morning mvo [07:30] it's weird it reached stable and no one noticed it is not installable [07:32] jibel: yeah, also it seems like stable->edge all have the same revision of vlc [07:32] jibel: version says "daily" [07:32] jibel: I guess we need to reach out to them and suggest to use "daily" for "edge" and not stable :) [07:32] mvo: note the revision, 7, meaning it's not daily [07:33] mvo: unless they count days at pluto frequency [07:33] zyga: why not? [07:33] zyga: maybe they just started 7 days ago, I have no insight into this [07:33] mvo: it was "daily" and ~5 in heildenberg [07:33] (I honestly don't know when they started to publish it) [07:33] (sorry for spelling that wrong) [07:33] zyga: uhhhh, ok. so daily for a day long ago [07:33] it's just "not actively maintianed" [07:33] :( [07:50] hmm, debian seems to be working [07:50] I'll check if we get the right image for real [08:01] eh :) [08:01] mvo: so cachio or gustavo mixed up the two [08:01] mvo: and "sid" and "stretch" are swapped [08:02] zyga: heh [08:02] zyga: ok, mystery solved [08:02] mvo: hmm, actually, it's the same old image [08:02] mvo: I changed sources.list in both [08:02] we are not on the new images yet [08:02] I guess I'll just talk to cachio later today [08:09] zyga: so I feel slightly proud of myself [08:09] I delved into the world of ELF deps and didn't drown [08:09] I didn't really like messing around with ELF data, though :/ [08:10] top elf tip: alias readelf='readelf --wide' [08:12] mwhudson: oh if only it were that simple [08:12] I was using libelf directly [08:12] ah [08:13] i actually quite like go's debug/elf [08:13] it doesn't support reading all the attributes, but it's a nice API [08:14] Son_Goku: what are you trying to do? [08:14] do you really want to know? [08:14] yes [08:14] I like elf and things like tat [08:14] https://github.com/rpm-software-management/rpm/pull/360 [08:14] PR rpm-software-management/rpm#360: elfdeps: Add full multiarch deps support [08:14] that [08:15] is rpm going multiarch? [08:15] well, rpm is going to support it [08:16] it's up to distributions if they want to do a multiarch scheme of some kind [08:17] elfdeps is actually the only bit of rpm that wasn't multiarch-safe already [08:17] as it predates rpm's own multiarch support [08:19] thank you for pushing that in your spare time :) [08:21] the only thing I'm not sure about is if this handles x32 properly [08:22] Son_Goku: I played with x32 lately, I'm actually very fond of this idea [08:22] Son_Goku: object size is very compact and you have all the instructions and registers [08:22] I don't know what it looks like from ELF [08:22] Son_Goku: for vast majority of my work x32 is sufficient [08:22] Son_Goku: in which sense? [08:23] oh interesting [08:23] it comes up as x86-32 [08:23] that's probably not good [08:23] I think it's called x32 more often [08:24] but naming and off by one and so on :) [08:24] so here's the interesting bit [08:24] it is elfclass32 [08:24] but machine is x86_64 [08:25] right, because that probably defines various pointer sizes and what not [08:25] right [08:25] and I also suspect nobody thought of this when elf was designed [08:25] I think that's a pretty safe assumption [08:25] as people thought it would just grow and grow [08:25] oh god, this means I need to special-case this [08:25] offtopic, meld and gnome look really amazing [08:37] snappy-m-o autopkgtest 1717 xenial:arm64 [08:37] sergiusens: I've just triggered your test. [08:38] zyga: and now I added handling for x32 [08:39] Son_Goku: given that an average consumer PC has 4GB of ram or less, I'd love to see windows use x32 [08:39] not going to happen [08:39] Son_Goku: so that linux folks with their 32GB machines would notice [08:39] Son_Goku: never say never [08:39] I probably should add support for x32 to the regular elfdeps mode [08:39] Son_Goku: microsoft is quite a surprise source lately [08:39] since I am touching that code... [08:40] Son_Goku: and on that sense it would also make sense in arm phone world [08:40] there, absolutely [08:40] Son_Goku: and by extension in iot [08:41] 512MB board with 64bit CPU running in x32-like mode === JanC is now known as Guest41818 [08:42] brb [09:19] PR snapd#4204 closed: store: enable "base" field from the store [09:31] eh all phone cpus just implement aarch32 still don't they? [09:31] it seems noone can quite decide if arm64ilp32 is worth it yet [09:31] mwhudson: I think the trend is clear for aarch64 being used by most now [09:32] zyga: i mean, 64-bit phone cpus implement the old arm 32-bit instruction set as well [09:32] there are server cpus that don't but they also have gobs of ram [09:32] mwhudson: yes, that's true [09:45] :( my base snap is already outdated and needs redoing [09:46] PR snapd#4219 opened: snap/validate: extend socket validation tests [09:48] jdstrand, hi, just saw your messages from yesterday, I pushed ^^ with the changes you requested since the branch was merged already === __chip__ is now known as Chipaca [09:54] i've just logged on to not miss out on anything, but i've got to go off to the doc's for a 10:30 [09:54] so i won't be around until ~midday, assuming they're on time [09:59] Chipaca: good luck! === JoshStrobl|zzz is now known as JoshStrobl [10:51] PR snapd#4210 closed: many: add magic /snap/README file [10:53] snappy-m-o autopkgtest 1717 xenial:arm64 [10:53] sergiusens: I've just triggered your test. [10:53] Hi, I thought to look at the yaml file of the new "official" pulsemixer snap. Anyone can point me where to find it. There isn't any kinf of docker hub equivalent website for snaps, is there? [10:54] PR snapd#4217 closed: interfaces/browser-support: add shm path for nwjs [10:54] PR snapcraft#1735 closed: unit tests: reset log level after test [10:57] PR snapd#4207 closed: Flesh out NVIDIA support for biarch and multiarch systems [11:03] snappy-m-o autopkgtest 1717 xenial:i386 [11:03] sergiusens: I've just triggered your test. [11:11] hrm, asciicasting is harder than it looks, when you make stupid mistakes like leaving a folder around after the dry run [11:12] mvo: is master broken? I got a couple of emails about it, though recent merges seemed innocent enough [11:20] pedronis: yeah, it was a prepare problem that looked like a fluke but I just got the second error mail, I'm looking now [11:21] PR snapd#4220 opened: snapd: allow hooks to have slots [11:21] pedronis: it looks like the opensuse repo has a hickup, I will disable opensuse for the moment I think [11:36] 4214 needs a second review - should be easy peasy [11:38] mvo: done [11:41] zyga: \o/ [11:41] PR snapd#4214 closed: fakestore: add go-flags to prepare for `new-snap-declaration` cmd [11:54] pedronis: I am updating the roadmap right now, anything I need to add to 2.29 from your side except "make --ignore-validation sticky"? (I was looking for tags "pedronis" :) [11:55] mvo: for 2.29 that's it, and some improvement bits on that are in 2.30 [11:55] pedronis: thank you [11:56] a bit of a meta question, it would be nice to have a way to tag topics from upcoming to "done" (or done-2.29) this would make writing the roadmap much simpler [11:56] (or "done", "2.29") [11:56] "upcoming" "2.29" [11:57] interesting, I tend to put in topic "this will be avaliable in 2.xx+" when things have landed, but not easy to search on that [11:58] mvo: in theory we (Gustavo) put stuff in the roadmap topic that we know will come [11:58] but there's often more/smaller stuff too [11:58] also don't know how that process will change [11:58] mvo: I would be ok to have further tags personnaly [11:59] mvo: I think I wrote already that I fell we have too many things in upcoming and is a bit unclear what is really upcoming (next 2 releases or so) [11:59] s/fell/feel/ [12:00] pedronis: yes [12:00] Chipaca: silly question, do we have a forum topic describing the new ansimeter? [12:06] sergiusens: I sent you my gist. also note the bug I just found... I'll make a PR for it asap [12:12] mvo: btw there is something weird with roadmap topic links, many don't work for me but if I retype the last bits they work [12:13] pedronis: I have the same problem, when I click on them directly they give me an error but when I open them in a new tab thinks work as expected [12:17] mvo: wonder if it's the syntax or something about perms we don't understand [12:17] Chipaca, zyga, pstolowski please double check if https://forum.snapcraft.io/t/the-snapd-roadmap/1973 for 2.29 and 2.30 reflects reality or if I forgot anything (cc niemeyer :) [12:18] pedronis: yeah, worth raising during the standup I think, maybe gustavo knows (he knows the software best afaict) [12:20] pedronis: It's okay to have further tags.. the main concern (mine, at least) is always that when it gets too fiddly in general it gets forgotten over time [12:21] pedronis: As long as we can stick to it, more metadata is good [12:21] pedronis: As long as it's useful as well, obviousy [12:23] ... and on that note, I really need to get driving or we won't get there today [12:23] o/ [12:23] yea, I tend to be ok with process (within reason), but yes unclear we follow already current process [12:24] Ah, last note, per forum note, if you have some headshot/upper body pictures you're comfortable with (core team), please send it along so I can build a nice slide for the talk [12:33] mvo: this is also 2.30 https://forum.snapcraft.io/t/new-configure-snapd-task-and-reverting/2774 and needs more work ? [12:40] PR snapcraft#1737 opened: cli: pass remote from container_config to clean method [12:45] hi [12:46] would anyone know how to deal with https://paste.ubuntu.com/25967274/ ? [12:49] axino: hey there [12:49] did you manually tinker with the mount ? [12:49] axino: it's a known bug, I looked at it extensively though I don't know if there's something simple that would work around it for you [12:50] not that I know of [12:50] ogra_: it's a bug in 14.04 and inside containers [12:50] ah, 14.04 ... [12:50] (this is a xenial container yes) [12:50] ogra_: this is a container here actually [12:50] axino: not sure if this will help you, but: [12:50] yes, i noticed the container [12:51] axino: you can unmount those things manually (all the mounts in /snap/*/*) [12:51] axino: then you should be able to: sudo mount --bind /snap /snap; sudo mount --make-rshared /snap; [12:51] FWIW I found I could "snap changes" and "snap abort" but it doesn't help [12:51] axino: then you should be able to sudo systemctl start each of the mount units [12:51] axino: and then it should recover [12:51] axino: that's roughly what my in-progress fix is trying to do [12:52] zyga: should I stop snapd first ? [12:52] axino: you can but it doesn't (should not) matter [12:55] zyga: now the core snap is broken :x [12:55] axino: did you mount everything back? [12:55] axino: ideally paste the steps you took [12:56] axino: look at /etc/systemd/system and start all the mount units back (or mount them manually) [12:56] zyga: https://pastebin.ubuntu.com/25967312/ [12:57] (what I used to start the mounts back) [12:57] who can i beg or bribe for snap store review? :) [12:57] err [12:57] the units I started are gone [12:57] pedronis: yes [12:57] pedronis: thank you [12:57] axino: hmm [12:57] axino: what is actually mounted? [12:57] zyga: it was looking like this https://pastebin.ubuntu.com/25967315/ [12:58] no [12:58] I mean moubted [12:58] not what systemd things [12:58] thinks* [12:58] zyga: https://pastebin.ubuntu.com/25967318/ [12:58] niemeyer: how would you feel about adding a "2.30" tag (and same for subsequent releases)? then we could tag topics as "upcoming", "2.30" and the roadmap would be slightly simpler to build [12:58] axino: how about /proc/self/mountinfo [12:59] zyga: https://pastebin.ubuntu.com/25967323/ [12:59] mvo: he answered, I think he said ok if we manage to stick to it [12:59] pedronis: aha, cool. sorry missed the reply [13:00] axino: ok this looks good now [13:00] axino: now go to /etc/systemd/system and look at the mount units you see [13:00] niemeyer: thanks about your reply for the tags [13:00] start the core one [13:01] zyga: https://pastebin.ubuntu.com/25967332/ [13:01] OK starting the last core one [13:01] axino: ok, start each one one by one [13:01] axino: then start snapd [13:01] zyga: OK we're back in business [13:01] axino: and that should work [13:01] zyga: thanks o/ [13:01] axino: and my fix is doing essentially that, just working on some details [13:02] ok [13:02] axino: thank you for testing this! :) [13:02] zyga: will that impact every "snap remove" under LXD ? [13:02] axino: no [13:02] axino: well, the bug affects a lot of things [13:02] axino: the fix I made is changing the mount sharing for /snap before the very first app ever runs on a given system/container [13:02] axino: then it's all good [13:03] ok [13:03] axino: what we did here was the more costly/complex recovery process [13:04] mvo: i did not write about ansimeter [13:07] PR snapcraft#1738 opened: unit tests: make the check for output less strict [13:11] Chipaca: did you see this: https://forum.snapcraft.io/t/do-not-print-the-progress-bar-when-running-snap-commands/2816 ? [13:13] pedronis: i hadn't; commented [13:26] jdstrand: around? [13:31] mvo, PR #4218 [13:31] PR snapcraft#1739 opened: Lxd refresh remote [13:31] PR #4218: tests: adding tests for time*-control interfaces [13:31] this is the new one [13:31] I closed the other one before more people look at it [13:32] mvo, https://travis-ci.org/snapcore/snapd/builds/302294954#L4740 [13:32] this is the error I see in ubuntu core, with timezone = n/a [13:32] mvo: will you create the 2.30 2.31 tags ? [13:43] kalikiana how confident are you in your PRs? [13:44] kalikiana can you add notes on what you tested? [13:44] sergiusens: I tested only from git - building snaps for testing takes a while... even if theoretically it shouldn't be relevant here, it seems best to check that [13:45] lemme document what I did [13:47] kalikiana please check [13:52] tea [14:07] jdstrand, updated the PR, thanks for the review [14:15] flexiondotorg: around ? [14:34] zyga: hey, yes [14:34] ackk: thanks! [14:40] jdstrand: I updated the secure-file PR [14:41] pstolowski: what snap can i use to play with config? [14:41] was looking for 'test-snapd-conf*' but that's not it [14:43] pedronis: sorry for the slow reply, I am still trying to figure out how to define new tags [14:43] pedronis: where? [14:43] um [14:43] mvo: where? [14:43] :-) [14:44] Chipaca, do you need something for a spread test? [14:44] Chipaca: in the forum [14:44] pstolowski: for playing with snnapshots [14:45] mvo: you need admin [14:45] i think [14:45] Chipaca: I have this (I think) - maybe not enough of it [14:45] mvo: what tag do you want to create? mind if i do it for you? [14:45] Chipaca: sure, 2.30 [14:45] Chipaca: the tag should be "2.30", then we can tag topics as "upcoming", "2.30", "chipaca" [14:45] etc [14:47] mvo: so, there isn't an explicit CRUD for tags; you create the tag when you use it [14:47] mvo: is there anything i can tag for you? [14:47] mvo: also: no dots in tags [14:51] zyga: I saw. it is on my list for today [14:51] thanks! [14:52] om26er: Hello [14:53] Chipaca, ondra may have some examples for you. i think there are still very few of them. you can always package one of our tests/lib/snaps/ for local testing; if you only need to check that config is stored/restored, then you don't really need any logic in configure hook [14:54] pstolowski: right, that's what i was asking: what in tests/lib/snaps is a good one to use for playing with config [14:56] Chipaca, any of those I guess - http://pastebin.ubuntu.com/25967995/ - probably those with empty configure will be best [14:56] Chipaca, and looks like config-versions spread test may be an inspiration? [14:57] ok, i'll look, thank you [15:05] flexiondotorg: re my Android Studio snap, any update ? [15:06] https://forum.snapcraft.io/t/classic-confinement-for-android-studio/2634/9 [15:07] Chipaca: 2_30 then ? 2dot30 ? [15:07] can then start with a number? [15:07] *they [15:09] Hrm. `error: cannot refresh []: cannot refresh snap-declaration for "core": Get [15:09] https://api.snapcraft.io/api/v1/snaps/assertions/snap-declaration/16/99T7MUlRhtI3U0QFgl5mXXESAiSwt776?max-format=2: [15:09] dial tcp: lookup api.snapcraft.io on [::1]:53: read udp [::1]:35870->[::1]:53: [15:09] read: connection refused` [15:09] This is "snap refresh" in a LXD container [15:09] popey: flexiondotorg: can you please review https://github.com/snapcrafters/vault/pulls [15:10] Chipaca: I'm confused, afaict you cannot create them on the fly [15:10] kalikiana: is it just temporary? [15:10] elopio: looking now [15:11] zyga: I've reproduced it a few times now. [15:11] I'm checking if it has anything to do with the particular server [15:11] pedronis: you can't; I can [15:11] pedronis: neener neener [15:11] that's good [15:12] it's super dangerous, for fat-fingered me :-) [15:12] pedronis: 2-30 would work [15:12] Chipaca: this for example would need 2.30: https://forum.snapcraft.io/t/do-not-print-the-progress-bar-when-running-snap-commands/2816 [15:12] Chipaca: and the things in the roadmap under 2.30 [15:13] I'll tag that one [15:13] heh sorry [15:13] I miscopied [15:13] 2_30, or 2-30? [15:13] Chipaca: I meant this one: https://forum.snapcraft.io/t/new-configure-snapd-task-and-reverting/2774 [15:13] Chipaca: I'm partial to2_30 [15:13] boom [15:17] zyga: seems to be fine in one but not the other... now to wonder what's happening there [15:19] zyga: now it works [15:22] thanks Chipaca [15:23] sergiusens: I documented my steps for both PRs. First one ran into network issues on Travis so tests re-triggered. Second one still pending. [15:23] mvo: pedronis: ftr jdstrand can also do this [15:26] elopio: perhaps you could sanity-check my test code? snapcraft#1737 and snapcraft/pull/1739 [15:26] PR snapcraft#1737: cli: pass remote from container_config to clean method [15:26] PR snapcraft#1739: Lxd refresh remote [15:27] mvo: and so can you! :-) heh [15:27] mvo: and ev and noise [15:27] ¯\_(ツ)_/¯ [15:28] Chipaca: oh, we can create tags now? [15:28] mvo: you should've been able to do it before [15:28] Chipaca: what is the magic to do it? I just create one on the fly? [15:28] mvo: yes [15:31] ta [15:42] * kalikiana tea break [15:46] PR snapcraft#1738 closed: unit tests: make the check for output less strict [15:51] * sergiusens is having an excellent time with 39 Celsius and a power outage [15:52] sergiusens: ffffffantastic [15:57] i'd happily trade the 7°C for that [15:58] ogra_: nuh uh [15:59] ogra_: you can always put on more clothes [15:59] ogra_: you can't get more naked than naked [15:59] putting your feet into a bucket of water helps though ... [16:00] ogra_: especially if the bucket is in a pool [16:00] +1 [16:00] Chipaca: have you seen that gallery with skinless ppl? [16:00] zyga: i've had lunch in a museum with skinless people, does that count? [16:01] also babies in formaldehyde [16:03] skinless sounds great for hot days [16:03] Chipaca: does "cool" qualify as an answer? [16:03] zyga: ¯\_(ツ)_/¯ [16:04] zyga: my secondary school was next to a medical science school that had a museum you could just walk into [16:04] lots of weird stuff [16:04] anyhoo [16:05] Chipaca: "do your homework or you end up in a barrel like the rest of exhibits" the biology teacher said ;-) [16:05] pstolowski: config doesn't have the idea of per-user data, right? [16:05] Chipaca, correct [16:13] Chipaca: per user snaps would be interesting but also hard [16:14] zyga: yeah, i think i'm going to store snap's config only if i'm storing the system data, otherwise no [16:14] (there's a per-user aspect to snapshots, and a system aspect) [16:14] makes sense to me :-) [16:26] Chipaca ogra_ I'll take the 7 Celsius any day of the year [16:26] power just came back! \o/ [16:27] sergiusens, man we can't catch a break on snapcraft#1717 [16:27] PR snapcraft#1717: catkin plugin: check for pip packages in part only [16:29] sergiusens, why i386 though? I already got a local pass on amd64 [16:30] just to see if that log error showed up [16:30] Ah okay. Well I noticed you merged the PR that's supposed to fix that [16:30] But it looks like this needs to be merged with master first [16:31] kyrofa wait for it [16:34] grrr, network errors on pip on both PRs [16:35] FYI I need to hop on a train in a few minutes so I can't work late tonight- but I'll check back in for a bit later [16:35] kalikiana, want me to run one locally? [16:36] Assuming it's for 2.35 [16:36] kyrofa: the first one passed on the second attempt, second one pending... so, it works, it's just a waste of time... [16:36] Okay [16:37] kyrofa: this is concerning snapcraft#1737 and snapcraft/pull/1739 which are fixing an issue with remote lxd - if you can have a looksie that'd be appreciated [16:37] PR snapcraft#1737: cli: pass remote from container_config to clean method [16:37] PR snapcraft#1739: Lxd refresh remote [16:44] kalikiana, sure thing, reviewing now [16:45] * zyga made small progress on layouts :) [16:48] Bug #1732494 opened: Impossible to manage package after Trusty --> Xenial upgrade [16:48] elopio: can you help me with the CLA check failing for this pull req? [16:48] elopio: https://github.com/snapcore/snapcraft/pull/1720 [16:48] PR snapcraft#1720: kernel plugin: introduce kernel-channel to select from which channel … [16:48] kyrofa: Grand, thanks! [16:49] elopio: i think you are the keeper of the 7 keys^W^W^W... i mean unit tests :) [16:50] ppisati: heh, you just made me switch my music [16:51] mvo: at least someone got the joke :) [16:52] ppisati: :-D /me hums along [16:58] hahahaha [16:58] hhaa [16:58] hqa [16:58] dhe391edcf :flips table: [16:59] ppisati: how weird, it shows no error. I hate this CLA check, like a year ago I asked if it would be possible to move to the github integrated CLA check, but nothing happened :( [16:59] so... golang on 32-bit archs can only see half of the users on the system [16:59] of all the *possible* users i mean [17:00] ppisati: let me updated your branch, that at least will reduce the number of checks [17:02] Chipaca: ? [17:03] Chipaca: ?? [17:03] Chipaca: why is that? [17:03] getuid and co return an int [17:03] uid_t and co are 32 bit unsigned [17:05] PR snapd#4221 opened: tests: new test for interface network status [17:10] zyga: https://github.com/golang/go/issues/22739 [17:12] Chipaca: well that will show those 32 bit users ;) [17:12] zyga: we are going to be using uids somewhere in that space [17:12] for our per-snap uids [17:12] Chipaca: well, that ended quickly [17:12] Chipaca: can we use negative integers? ;-) [17:13] zyga: no, that's the problem [17:13] syscall.Getuid returns a negative number [17:13] and then that's looked up in /etc/passwd [17:13] guess how well that works [17:13] :-) [17:14] Chipaca, oops [17:14] I've got really weird behaviour. I have created a patch which I'm trying to apply in a snapcraft prepare script. Doing the steps manually works, but when I try to do it via snapcraft the git apply call just silently does nothing [17:14] and the reason you can't create a user with uid -1 is because that's a special flag value that means "don't change the uid" [17:14] (by -1 i mean 0xf...f obvs) [17:15] my prepare script is: git apply --stat $SNAPCRAFT_STAGE/unrar.patch [17:15] I've confirmed the patch is at $SNAPCRAFT_STAGE/unrar.patch, and I've confirmed that I'm in the build folder with the source I'm trying to patch [17:16] popey: one more, please https://github.com/snapcrafters/vault/pull/6 [17:16] PR snapcrafters/vault#6: Update the grade to stable [17:16] elopio: done [17:20] thank you [17:21] diddledan: are you in the directory you think you are when you try to do that? [17:21] yes, I've confirmed that [17:24] trying to replicate it in a minimal build doesn't recreate the problem. It's only on my complex build that's failing [17:24] diddledan, which plugin? [17:24] make [17:26] diddledan, remember that the prepare scriptlet runs in the build dir, not the source dir [17:26] it's this part that fails on my attempt to build calibre, but pulling it to it's own snapcraft.yaml doesn't fail: [17:26] https://www.irccloud.com/pastebin/wqlPNJXA/ [17:27] the patch is: [17:27] https://www.irccloud.com/pastebin/Mv9VK6TH/ [17:28] Hmm, yeah I don't see anything wrong there [17:28] full calibre snapcraft.yaml which FAILS (the above parts don't fail when isolated like that) is: [17:28] https://www.irccloud.com/pastebin/MTOl6x2y/ [17:28] diddledan, dumb question: do you have git installed? [17:28] (it's not a build-package) [17:30] PR snapd#4222 opened: tests: add new `fakestore new-snap-{declaration,revision}` helpers [17:30] ah, yeah that one will install git [17:31] snapcraft installs git itself automatically. it doesn't need to be manually specified. even if it did need to be specified it's working enough that git is responding in both scenarios [17:33] specifically the behaviour is : when the parts are isolated, the `git apply` works. when in the calibre full yaml the `git apply` silently returns with exit code 0 indicating success but the source is not patched [17:38] jdstrand, https://paste.ubuntu.com/25968840/ [17:38] Running the test to check network-status interface [17:38] https://github.com/snapcore/snapd/pull/4221/files [17:38] PR #4221: tests: new test for interface network status [17:38] jdstrand, any idea why it could happening? [17:41] diddledan, and you're checking the source in the build dir, not the src dir? === cachio is now known as cachio_afk [17:49] elopio kyrofa I am beginning to thing that we should cut where we are; that test is breaking on i386 as well [17:50] sergiusens, wait, no, that's the plainbox bug I logged [17:50] I thought I was only seeing it locally though [17:51] This is the first time I've seen it in the real adt [17:52] I'll investigate now [17:53] sergiusens, catkin tests look good there [17:53] kyrofa ok, thanks [17:54] ok, EOD'ing [17:54] I think that PR is probably good, then [17:55] kyrofa please rebase so that other one can get in [17:55] Of course [17:56] PR snapcraft#1717 closed: catkin plugin: check for pip packages in part only [17:59] PR snapcraft#1737 closed: cli: pass remote from container_config to clean method [17:59] kyrofa you are in charge of the boat, I need to go to physiotherapy now... my dream is to see those last remaining PRs merged and the changelog branch updated ;-) [17:59] sergiusens, you got it [18:14] snappy-m-o, autopkgtest 1739 xenial:i386 [18:14] kyrofa: I've just triggered your test. [18:14] elopio, I'm not clear on your comment-- do you feel like snapcraft#1739 needs to change? [18:14] PR snapcraft#1739: lxd: refresh remote container [18:22] elopio, hmm.... don't the debian/tests/* scripts run as root? [18:22] Which means /home/ubuntu/autopkgtest_tmp is owned by root? [18:22] Although you give it mode 777 I suppose [18:23] kyrofa: I'm happy if the message on 1739 is added later. [18:23] and yes, you wanred about that so it's 777 now. [18:23] we could set the owner to ubuntu if there's any problem. [18:23] elopio, wondering if the plainbox issue traces back to that, poking now [18:24] The message on 1739 being the one I mentioned as well? [18:24] kyrofa: yes. [18:24] Agreed [18:24] kyrofa: and this is what we do on integrationtests -> mkdir --parents /home/ubuntu/autopkgtest_tmp --mode 777 === rmescandon_ is now known as rmescandon [18:36] elopio, oh man... see all the old snapcraft xenial:amd64 PRs running now... === JoshStrobl is now known as JoshStrobl|Away [18:45] kyrofa: now the bottleneck is our fault ::( [18:46] Yeah that hurts [18:46] Why can we not cancel these?!?!?! [18:46] turn it off and on again. [18:48] Chipaca: around? [18:48] elopio, do you know how adt creates lxc containers? [18:49] It must be special, because I'm getting confinement issues trying to run the plainbox test [18:49] cachio_afk: what is the specific apparmor denial? [18:49] can I get a quick +1 for a trivial: https://github.com/snapcore/snapd/pull/4223 [18:49] PR #4223: cmd/snap-update-ns: misc cleanups [18:49] PR snapd#4223 opened: cmd/snap-update-ns: misc cleanups [18:49] elopio, although looking at the config of the running container, doesn't look special at all. Looks default [18:52] Alright, plan B [18:56] kyrofa: this is the provisioner https://anonscm.debian.org/git/autopkgtest/autopkgtest.git/tree/virt/autopkgtest-virt-lxd#n135 [18:58] elopio, interesting. Sucks, I can't reproduce the same error outside of the adt suite, so I've paired that suite down to only the single snapd test, but it still needs to run the units and build the deb, haha [18:58] That's alright, I'll get there [18:59] But yeah, that proves that there's nothing special about it [19:01] * zyga is hyped by https://teletype.atom.io/ [19:06] zyga: for a little bit [19:06] zyga: wassup [19:06] Chipaca: can you please +1 https://github.com/snapcore/snapd/pull/4223 [19:06] PR #4223: cmd/snap-update-ns: misc cleanups [19:06] kyrofa: my hack to do it faster is change setup.py. Also, use the ppa instead of building from source, but just if you are not changing the deb. [19:06] I'll merge it when green and push a feature PR on top [19:06] I just didn't want this noise in the same one [19:06] Oh the setup.py, duh [19:07] elopio, okay, tests disabled. But how do I tell it to use the PPA? [19:08] kyrofa: that's what I'm working on now. I think I'm close: https://github.com/snapcore/snapcraft/pull/1643/files#diff-bc66fbdb036f79e9f15bcbf47a6bc67cR44 [19:08] PR snapcraft#1643: [WIP] tests: run daily autopkgtest in travis [19:08] Ah yes, nice [19:11] jdstrand: FYI: a formal +1 on https://github.com/snapcore/snapd/pull/4170 would help :) [19:11] PR #4170: cmd/snap-update-ns: add planWritableMimic [19:12] zyga: +1 (beyond the "silly wrapper is silly" comment) [19:12] Chipaca: thanks, I just wanted to highligh the mocking aspect, I can reword that if you have ideas [19:12] highlight* [19:13] zyga: i understood it purpose, but the wrapper isn't needed [19:14] oh? [19:14] aaah [19:14] I see now [19:14] thanks, I'll do that [19:14] Chipaca: seeing this brings back classmethodwrapper horrors [19:15] mbuahahaa [19:15] zyga: it is also on the list [19:15] zyga: if it's any consolation, at least you're not fiddling with the glob table [19:15] Chipaca: oh, about that [19:15] Chipaca: a puzzle [19:16] Chipaca: I failed, maybe you can do better [19:16] https://twitter.com/fijall/status/930744285728735232 [19:16] Chipaca: ^ [19:16] jdstrand: thank you! [19:17] ah, the answer is in the responses below now [19:18] zyga: it's a hacky way of having a global that survives (re)imports [19:18] e.g. if you do reload() or if you import it with two different paths [19:18] I was trying the two different paths idea [19:19] I must have remembered this wrong [19:19] but inter-package imports did something funky [19:19] from .foo import bar [19:19] vs from pkg.foo import bar [19:20] wow, when did github start doing this: https://github.com/snapcore/snapd/blob/master/COPYING === cachio_afk is now known as cachio [19:22] zyga: it's probably just reload() if the importer doesn't trip up [19:22] zyga: or! it could be for re-exec [19:22] jdstrand, https://paste.ubuntu.com/25969363/ [19:22] these are the denials that I saw [19:25] Dang, now adt is failing with that new error as well. I'm starting to hate plainbox [19:26] Either it or the way we're using it makes it such a moving target [19:26] mmm [19:26] zyga: ¯\_(ツ)_/¯ can't make it work outside of a plain reload -- which might be enough [19:27] some apppservers reload() the modules of your app when you edit it [19:38] Chipaca: I recall using reload but very rarely === dontbeadick is now known as kennyloggins [19:52] * elopio <- lunch [19:59] kyrofa how are things going? [19:59] sergiusens, tests still running. I'm having trouble reproducing plainbox all of a sudden as well [19:59] It's a different error today :P [20:00] "cannot create freezer cgroup hierarchy for snap plainbox-simple" [20:00] Researching now [20:01] That's coming from snap-confine [20:01] zyga, any idea why I might be getting "cannot create freezer cgroup hierarchy for snap X" ? [20:01] zyga, it's in a normal lxd container [20:04] And then it's so broken I can't remove it [20:06] kyrofa: no, [20:06] kyrofa: any denials? [20:20] zyga, crap. Now it succeeded. What the heck... [20:21] post on the forum please [20:24] Software is so quantum. It does X until you look at it, then it does Y just because you're looking at it [20:31] zyga, did you recently release a new core snap by any chance? [20:32] yes [20:32] today [20:32] Yeah this was working yesterday [20:32] And it just happened again [20:32] sergiusens, getting THAT error in adt now ^ [20:32] I'll make a forum post [20:33] zyga: do you have any ideas for the debian breakage yet? [20:33] This is interesting: "kernel_os.go:192: cannot get boot vars: open /boot/grub/grubenv: no such file or directory" [20:36] mwhudson: I didn't look after that, jjohansen needs to say if this is a bug that we should fix in debian or in the kernel [20:36] ok [20:36] mwhudson: all I know is in the forum for that topic [20:36] i guess it still makes sense to get 2.29.3.1 ready for uploading to debian even if it's not going to work for a while [20:36] yes [20:37] zyga: --enable-nvidia-multiarch, do we want that on debian? [20:37] zyga: sorry I haven't figured that out yet. I'm working on it [20:38] zyga: also should we stop passing --disable-apparmor now? [20:38] PR snapd#4220 closed: snapd: allow hooks to have slots [20:38] jjohansen: thank you! [20:38] mwhudson: I think so [20:38] mwhudson: the core was ablt to handle that for some time now [20:38] mwhudson: (since ~august) [20:39] zyga: which of my two questions are you answering? :) [20:40] * mwhudson acquires coffee [20:41] mwhudson: [20:42] mwhudson: as for nvidia, ... not sure, perhaps yes but I never tried it because debian doesn't (AFAIK) ship proprietary drivers [20:42] mwhudson: probably doesn't hurt [20:43] mm ok [20:44] there was a guy in the forum complaining about nvidia problems i think [20:44] but i don't understand how any of that works :) [20:44] mwhudson: I can explain if you want [20:44] sergiusens, scratch that... I'm getting that error on _every single_ snapd test [20:45] * mwhudson considers if he cares [20:45] zyga: i've used intel graphics for years and have no intention of changing any time soon :) [20:50] mwhudson: fair enough :) [20:50] I think I should EOD, I'll check some PRs later and try to merge [20:55] snappy-m-o, autopkgtest 1719 xenial:i386 [20:55] kyrofa: I've just triggered your test. [20:56] zyga, to be clear, you released to stable today? [20:58] zyga: pushed something that might build to alioth [20:58] zyga: now go to bed :) [20:58] kyrofa: yes, stable was updated today, according to mvo in the forum [20:59] kyrofa: https://forum.snapcraft.io/t/2-29-release-cycle-started/2562/9 [21:01] kyrofa: sorry, had to pop out. the source isn't patched ANYWHERE. neither the src directory nor the build directory. The git apply simply silently exits without patching any files, despite checking that there is a copy of that file in the directory it's executed within. [21:02] kyrofa: the point I've been trying to state is that the exact same bits work correctly when I pull them into their own snapcraft yaml [21:02] kyrofa: not me personally but yes, we as a team pushed 2.29.3 to stable toda [21:02] zyga, forum post published [21:03] so I know that it works. it doesn't work when I use my full calibre build snapcraft yaml despite the parts being identical to those that work on their own [21:03] kyrofa: thank you [21:03] kyrofa: after lxd bug, which is close to being fixed, I will look at this [21:03] Thanks zyga [21:09] elopio, can you think of any reason why I would get that error when running the test, but not when installing the snap by hand? [21:09] in fact, even separated it doesn't patch anything, so I was wrong there [21:12] the following snapcraft.yaml doesn't patch anything [21:12] https://www.irccloud.com/pastebin/5rktodoh/ [21:12] same patch as before: https://www.irccloud.com/pastebin/Mv9VK6TH/ [21:13] patch lives in ./patches/unrar.patch and snapcraft.yaml lives in ./snap/snapcraft.yaml [21:14] "Instead of applying the patch, output diffstat for the input. Turns [21:14] off "apply"." [21:14] diddledan: why are you passing --stat ? [21:14] diddledan: seems like user error to do so :) [21:15] it doesn't work either way [21:15] jdstrand: hey, just a word of caution: the tun/tap test is failing randomly [21:15] jdstrand: we've seen it fail in lots of branches [21:15] diddledan: well, it definitley will not (and should not) do anything with --stat [21:15] jdstrand: it goes away on retry sometimes, not sure if you can think of anything in particular that would be specific to that test [21:16] https://www.irccloud.com/pastebin/MTOl6x2y/ is the full project which is failing [21:16] diddledan: but wait [21:16] diddledan: reading... [21:16] diddledan: (tbh, i fid it confusing to use a source that's not git with a git command) [21:17] diddledan: i don't think there's any guarantee provided that you actually have git [21:17] diddledan: it probably works in general, but just a note [21:18] diddledan: did you try to do the build interactively? e.g., stage patches, then try to stage unrar and see what happens? [21:19] I don't know what you mean "do the build interactively" [21:22] diddledan: like you can run `snapcraft` and it will build all the steps of the snap (I recommend in a LXD or vm) or you can do `snapcraft stage ` and just ahve it stage that part [21:22] diddledan: so maybe stage patches, take look at hte stage directory [21:23] diddledan: then stage unrar and see what it does [21:23] without --stat I get no feedback, which is expected. BUT the simplified testcase reverts to my assumption in that it works fine, but the complex calibre build fails [21:23] it's the same damned part definition in both casess! [21:24] diddledan: do you have a repository i can clone? [21:24] I can make one [21:25] diddledan: might be easier (although i'm not a snap expert myself -- i can at least try and recreat your issue) [21:26] https://github.com/diddledan/calibre-snap [21:26] diddledan: so i'm not sure if i follow -- is there a simple snapcraft yaml that does show the issue? [21:26] no [21:26] ok [21:26] the simplified testcase works fine [21:26] with the same bloomin part definitions!?! [21:27] I'm bashing my head against a wall here :-( [21:27] it doesn't make any sense whatsoever [21:30] zyga: it is only debian-unstable-64, right? [21:31] kyrofa subprocess.check_call by hand and see what happens [21:32] kyrofa why is adt running for snapcraft#1739 ? [21:32] PR snapcraft#1739: lxd: refresh remote container [21:32] sergiusens, I figured we wanted to get at least one pass on everything, no? [21:33] kyrofa travis already does that ;-) [21:33] sergiusens, haha, feel free to merge if you're happy with it, but the number of problems we built up because of exactly that is, well, not good [21:59] jdstrand: no, it also happened on 16.04-32 once [21:59] jdstrand: mvo thinks it is a race [22:00] jdstrand: or perhaps a sequence issue where earlier test is causing this one to fail [22:00] PR snapd#4223 closed: cmd/snap-update-ns: misc cleanups [22:01] zyga: if you want to disable the test and then assign me to fix it, I can look into it [22:01] PR snapd#4224 opened: cmd/snap-update-ns: teach update logic to handle synthetic changes [22:01] jdstrand: I think that's fine, I'll send a PR [22:02] jdstrand: https://github.com/snapcore/snapd/pull/4224 is interesting, there are a few lines of changes there and lots of tests [22:02] PR #4224: cmd/snap-update-ns: teach update logic to handle synthetic changes [22:02] jdstrand: reading the description will make the context for other layout PRs richer [22:02] diddledan: sorry, but your yaml makes very little sense to me :) [22:02] diddledan: how woudl `git apply` work if you're not i na git repo? [22:02] it works fine [22:02] diddledan: you are giving it a raw tarball and asking it to treat it as a git repo? you should be using patch [22:02] zyga: added to list [22:03] diddledan: i don't see how it can (in general) ... the source isn't a git repo [22:03] except in this particular case. if I pull everything except the patches and the unrar parts it works correctly. it only fails in situ with the rest of the yaml [22:03] diddledan: if you just use patch, does it work? [22:03] git apply doesn't require a git repo [22:04] I've proven the concept works. it is only in that full yaml that it fails [22:04] diddledan: ok [22:05] PR snapd#4225 opened: cmd/snap-update-ns: tweak changePerform [22:08] jdstrand: https://github.com/snapcore/snapd/pull/4226 this is the tuntap test, please merge that when green [22:08] PR #4226: tests: disable interfaces-network-control-tuntap [22:08] PR snapd#4216 closed: interfaces/time*_control: explicitly deny noisy read on /proc/1/environ [22:08] PR snapd#4226 opened: tests: disable interfaces-network-control-tuntap [22:09] PR snapcraft#1739 closed: lxd: refresh remote container [22:09] PR snapd#4202 closed: cmd: use a preinit_array function rather than parsing /proc/self/cmdline [22:10] jdstrand: if you want to please collect logs from https://travis-ci.org/snapcore/snapd/builds/302045926?utm_source=github_status&utm_medium=notification [22:10] jdstrand: this is where the test failed [22:10] jdstrand: the sequence number could be useful for reproducing this [22:11] jdstrand: once you collec that, please re-trigger the test [22:11] another instance is here: https://travis-ci.org/snapcore/snapd/builds/302478451?utm_source=github_status&utm_medium=notification on ubuntu-core this time [22:11] diddledan: so i staged everythinng, actually tried ot build it, got an error [22:11] bingo [22:11] i went into the parts/unrar/src directory [22:12] ran `git apply ../../patches/install/unrar.patch` and it did nothing [22:12] the denial here is [Wed Nov 15 13:41:22 2017] audit: type=1400 audit(1510753282.329:205): apparmor="DENIED" operation="open" profile="snap.test-snapd-tuntap.tuntap" name="/dev/net/tun" pid=9534 comm="tuntap.py" requested_mask="wr" denied_mask="wr" fsuid=0 ouid=0 [22:12] not a snapcraft problem, afaict, git doesn't want to apply the patch [22:12] diddledan: patch works [22:13] now, nuke the parts, stage and prime directories and try again with this yaml (which is the same yaml with everything except unrar and the patch removed): [22:13] https://www.irccloud.com/pastebin/to0fUZYt/ [22:14] and your git apply command was missing a ../ which is why your manual apply failed [22:15] diddledan: no it wasn't? [22:15] parts/patches/install/unrar.patch is what i wanted [22:15] oh maybe not missing a ../ - you're using the one copied into parts/ [22:15] diddledan: which is what snapcraft would be doing [22:15] nope, snapcraft is using the one in stage/ [22:15] well, in this case the same file [22:16] diddledan: so, i'm going to say it one last time, because i've wasted way too much time on this [22:16] diddledan: i jsut did exactly what you said [22:16] the patch is also not applied [22:16] the parts happen to build [22:17] but http://paste.ubuntu.com/25970392/ [22:17] git-apply fails to apply the patch [22:17] patch succeeds [22:17] stop using git-apply :) [22:17] seriously, git apply WORKS ON IT'S OWN! [22:17] no, it doesn't [22:18] look at the above output [22:18] ffs [22:18] diddledan: please, just try with plain patch [22:18] see if it works [22:19] diddledan: if `git-apply` worked, i can provide another paste for this, thenn running git-apply, then running patch would result in patch reporting an error. It does not [22:20] diddledan: i will admite freely, i didn't know you could `git-apply` outside of a git repository. I don't know why you would want to, or what it gains you, but I am seeing a difference in behavior between git-apply and patch. I trust patch. [22:20] +1 on patch [22:21] diddledan: http://paste.ubuntu.com/25970412/ ... that's the paste i mentioned [22:21] because you refuse to believe it works: https://paste.ubuntu.com/25970413/ [22:21] that is using the tarball snapcraft downloads and the patch I'm trying to apply [22:22] like I said, it works damned fine until I put it in the calibre snapcraft.yaml [22:22] diddledan: is the patch actually applied? [22:22] yes [22:23] or rather it is once I remove the --check because that kills apply like --stat does [22:24] from the source that I just applied the patch to which isn't in a git repo, and had the patch applied with git apply: [22:24] https://www.irccloud.com/pastebin/mXdBs4WB/ [22:24] that is what I want the patch to do, make those two lines look like that! [22:26] * zyga looks at Nov 15 12:32:38 Pandora kernel: [15491.386617] audit: type=1400 audit(1510777958.562:790): apparmor="DENIED" operation="capable" namespace="root//lxd-brave-muskrat_" profile="/snap/core/3440/usr/lib/snapd/snap-confine" pid=28464 comm="snap-confine" capability=2 capname="dac_read_search" [22:26] stgraber: (if around) does lxd/lxc do something special to systemd freezer cgroup? [22:27] diddledan: ok, i can see the bheavior you are seeing but I don't know why [22:27] diddledan: and i don't konw why in the snap directories, the same does not work [22:27] don't use --check [22:28] diddledan: i would still be curious if just patch would work, if it's a weird thing with git (corner case) [22:28] diddledan: i wasn't [22:28] diddledan: if you really can't get it to work, just fork it on github and patch your source? [22:29] oh frak me. I've just had a lightbulb and it's too damned bright it hurts [22:29] diddledan: what's that? [22:30] my snapcraft is in a git repo. git recurses to all your subdirectories. previous uses of git apply had been in their own git repo checked out by snapcraft which overrode the snapcraft repo. you're right in that it's because it's not a git repo, but not just that, it's because it's not a git repo and THEREFORE INSIDE MY SNAPCRAFT git repo [22:31] diddledan: ah [22:31] diddledan: so it seems like ither use a git repo as the source, or use patch? [22:32] so because it detected the .git dir way back up the chain it decided "I don't care that you were in that subdir, I'm going to apply your patch to the top-level where .git is!" [22:32] right [22:32] i thikn that's configurable -- i wonder if that should be set by snapcraft when it invokes git [22:32] since it can non-git nested inside git [22:33] should be relatively easy to make a testcase that shows the issue [22:42] this looks to be the correct incantation for such an occurance: git apply --directory=. --unsafe-paths ../../../stage/unrar.patch [22:43] or as you said, just use patch [22:45] diddledan: fun :) [22:45] fudge me, that was confusing [22:46] diddledan: we were both right and wrong :) [22:46] and of course it worked in my testcase because I made the testcase in a new dir that wasn't a git repo [22:46] right [22:51] Bug #1732555 opened: Installing bad snap has snapd crashing [22:56] PR snapd#4227 opened: tests: adding test for interface frame buffer [22:57] that incantation doesn't work actually. reverting to patch :-) [22:58] wow, I am glad I know what was happening now, but boy what an idiot was I!?! :-p [22:58] anywho, tis sleepytime methinks [23:17] elopio, do the bot subscriptions actually work for you? It never pings me on failures [23:18] snappy-m-o, github subscribe 1719 [23:18] kyrofa: I'll send you a message if a test fails in the pull request #1719 (many: account for python shebang args in rewrite). [23:18] PR #1719: firstboot: add firstboot assertions importing [23:18] I'll try again anyway [23:20] kyrofa: it worked last time I tried, but haven't in a long time [23:20] kyrofa: please let me know if you don't receive it, and I'll debug. [23:21] elopio, will do [23:22] elopio, is it technically possible for it to ping me when it finishes, regardless of success, and TELL me whether it succeeded or not? [23:22] Or is there no trigger for that? [23:22] (totally don't know how the bot works) [23:26] PR snapd#4226 closed: tests: disable interfaces-network-control-tuntap [23:30] kyrofa: the bot doesn't know how many tests are running. It just sees when one finishes. [23:30] so to report success, you would have to give it the number of tests you expect. Or something more clever than that. Currenly, we have nothing like that. [23:31] elopio, you mean like the number of tests in a single adt run? [23:31] * kyrofa makes a google code-in task to count all our tests [23:31] kyrofa: no, the number of tasks reported to the pr. [23:31] Haha, oh [23:32] elopio, so say I trigger both xenial:amd64 and xenial:i386 [23:32] currently, what it does is to listen on the updates to the PR. If one is a failure, it will ping you. Or something like that, I don't remember all the details [23:32] Regardless of success or failure, will it get two reports back from adt? [23:33] If so, I'd love to have it ping me twice, once with each result [23:34] kyrofa: well, you could subscribe also to successful results. Not implemented, but that is easy. [23:34] https://github.com/elopio/snappy-m-o/blob/master/plugins/snapcraft_github/snapcraft_github.py#L57 [23:34] elopio, ah, nice! Now I can make PRs when I want stuff? [23:35] kyrofa: do you mean, PRs to the bot? Of course! [23:36] Definitely [23:36] And it looks like it's a snap? [23:37] github_build is an interesting looking function [23:38] elopio, how do you test this, though? [23:46] elopio, first PR made [23:51] PR snapd#4228 opened: interfaces,tests: skip unknown plug/slot interfaces [23:51] PR snapd#4229 opened: interfaces,tests: skip unknown plug/slot interfaces [23:52] * zyga EODs [23:53] kyrofa elopio why have the tests restarted? Can you at least add notes to the PR? [23:54] sergiusens, which ones? I didn't restart any