sergiusensthomi I just did a run through the commit logs, nothing stands out that would prevent this, plugin loading code has stayed unchanged02:27
=== chihchun_afk is now known as chihchun
TheMusoIf I want to file a bug against snapd, what project do I file the bug under in lp? Is it snappy?04:49
=== JanC is now known as Guest89378
=== JanC_ is now known as JanC
TheMusonvm think I have my answer.04:58
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
dholbachgood morning06:32
netphreakmorning :)06:35
mupPR snapd#1491 opened: use more consistent "snap %q (rev %s)" task descriptions <Created by mvo5> <https://github.com/snapcore/snapd/pull/1491>06:58
dholbachkyrofa, sergiusens: does https://github.com/ubuntu/snappy-playpen/issues/145 look like a snapcraft bug to you?07:02
mupPR snapd#1437 closed: snapstate: use snapstate.Type in backend.RemoveSnapFiles <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/1437>07:39
mupPR snapd#1480 closed: many: rename SideInfo.OfficialName to SideInfo.RealName <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/1480>07:44
mupPR snapd#1492 opened: overlord/auth: add Device/SetDevice to persist device identity in state <Created by wgrant> <https://github.com/snapcore/snapd/pull/1492>07:49
mupPR snapd#1493 opened: store/auth: add SnapUbuntuStoreAuthService with device identity methods <Created by wgrant> <https://github.com/snapcore/snapd/pull/1493>07:51
mupPR snapd#1494 opened: tests: add content sharing interface spread test <Created by fgimenez> <https://github.com/snapcore/snapd/pull/1494>08:00
mupPR snapd#1495 opened: snapstate: rename OfficialName to RealName in the new tests <Created by mvo5> <https://github.com/snapcore/snapd/pull/1495>08:19
didrocksjosepht: hey! it seems that the cronjob imported my new part definition. However, it didn't refresh with the new description, any idea what's happening?08:20
mupPR snapd#1496 opened: tests: add spread test for tried snaps removal <Created by fgimenez> <https://github.com/snapcore/snapd/pull/1496>08:36
mupPR snapd#1495 closed: snapstate: rename OfficialName to RealName in the new tests <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/1495>08:41
pachulo /buffer 708:45
mupPR snapd#1497 opened: tests, integration-tests: port auth errors test to spread <Created by fgimenez> <https://github.com/snapcore/snapd/pull/1497>09:37
loologra_: hey, how goes?09:54
loologra_: would you know if our future rpi3 image will be armhf or arm64 based?09:54
loologra_: if armhf, how hard would it be to do an arm64 one for a third-party?09:54
ogra_lool, we'll have both i think09:54
loologra_: both officially supported?09:55
ogra_(at least i plan to have snaps for both in the store, not sure which we'll make official09:55
ogra_i'll bring that up at the sprint09:55
ogra_we'll be most likely missing firmware blobs for BT and wifi for arm6409:56
ogra_i havent played with that yet09:56
jamiebennettogra_: How much work will it be to have both?09:57
ogra_not much i guess, once i have a snapcraft.yaml for the unofficial side it is just a snapcraft run away to build a kernel snap ... and gadgets change really rarely09:58
jamiebennettogra_: and on the testing and support side?09:58
ogra_you mean when both are official ? well, twice as much :)09:58
ogra_(manual smoke testing etc)09:59
* jamiebennett nods09:59
ogra_i think it is fine to have the arm64 build unsupported09:59
ogra_for official arm64 we have the dragonboard09:59
oSoMoNis there anything special to do to enable a snapped app to run translated (translations are correctly shipped with the package under /usr/share/locale/)?10:08
oSoMoNis bug #1576282 relevant here?10:10
mupBug #1576282: Snaps built from deb can't be gettext translated <snap-desktop-issue> <Snapcraft:New> <snapcraft (Ubuntu):Confirmed> <https://launchpad.net/bugs/1576282>10:10
ogra_oSoMoN, yeah, you need to hack something up ... if your app uses gettext you should preload something that allows the bindtextdomain function to use a custom path10:11
oSoMoNhugh, that’s ugly!10:11
oSoMoNogra_, looking at your suggestion in the bug report10:11
ogra_for plain libc translations LOCPATH should help10:12
ogra_afaik niemeyer plans to have some generic preloader thing builtin in snapd at some point ... that will make it easier10:12
oSoMoNogra_, I’m working on webbrowser-app, which uses the i18n.tr() mechanism from the UITK, which I believe uses bindtextdomain under the hood10:12
ogra_yeah, most likely10:13
joc_morphis: i notice the tpm part on the wiki page has a source-type field which i think shouldnt be there10:17
morphisjoc_: it should be origin-type10:18
morphisdidrocks added that10:18
morphisjoc_: fixed that10:18
joc_morphis: yep just added a part myself, maybe should remove it so it doesnt get copied by others10:18
joc_good :)10:19
morphisI talked with didrocks yesterday and thought he added origin-type and not source-type10:19
morphisdidrocks: was there any reason to use source-type instead?10:19
didrocksmorphis: hum, sorry, I'm lost context?10:20
didrocksI didn't touch the tpm part10:20
didrocksor not intentionally :p10:20
morphisdidrocks: or was it dholbach? :-)10:20
morphisdon't remember10:20
ogra_the "d" guys :)10:20
didrocksthis is the only changed I did: https://wiki.ubuntu.com/snapcraft/parts?action=diff&rev2=16&rev1=1510:21
didrocksseems it's dholbach, indeed: https://wiki.ubuntu.com/snapcraft/parts?action=diff&rev2=15&rev1=1110:21
oSoMoNogra_, actually, it seems the UITK exposes a i18n.bindtextdomain(domainName, dirName) method to QML10:21
ogra_oSoMoN, oh ! thats neat ...10:21
oSoMoNI’ll test it10:22
ogra_if it works, you should perhaps mention it in the bug10:22
oSoMoNwill definitely doo10:22
morphisdidrocks: yeah10:23
oSoMoNdidrocks, hey, I’m looking at https://github.com/ubuntu/snapcraft-desktop-helpers/blob/master/qt/launcher-specific#L29 , is this known to work correctly?10:47
oSoMoNdidrocks, I made a snap of webbrowser-app, the packages includes the mo files under $SNAP/share/locale/, but the app is not getting translated10:49
oSoMoNlooking at UbuntuI18n::setDomain(…), this should work, as it calls bindtextdomain with $APP_DIR/share/locale/10:50
didrocksoSoMoN: are you sure bindtextdomain works? I remember that seb128 looked at this closer than I did10:54
didrocksIIRC, there was an issue if you don't set it to prefix10:54
seb128didrocks, well, if the uitk look into $APP_DIR whatever that is...10:57
oSoMoNdidrocks, what do you mean by set it to prefix?10:57
seb128doesn't work for non-UbuntuUITK code10:57
seb128since most upstream dp $datadir/locales10:58
seb128which is set from the prefix at buildtime10:58
oSoMoNseb128, but for QML apps using the UITK, that should work, right? see my comment at https://bugs.launchpad.net/ubuntu/+source/snapcraft/+bug/1576282/comments/310:58
mupBug #1576282: Snaps built from deb can't be gettext translated <snap-desktop-issue> <Snapcraft:New> <snapcraft (Ubuntu):Confirmed> <https://launchpad.net/bugs/1576282>10:58
ogra_oSoMoN, i guess it depends ... if they use the same bindtextdomain in the backend you will have the exact same problem10:59
ogra_sadly the issue is in the lowest level of the stack11:01
=== hikiko is now known as hikiko|bbi
=== hikiko|bbi is now known as hikiko|bbl
oSoMoNogra_, I’m not sure I understand, I thought the issue was that apps normally had calls to bindtextdomain with a hardcoded path set at build time11:08
* oSoMoN is not very familiar with the inners of libintl11:09
seb128oSoMoN, yeah, which is what I wrote no?11:15
seb128oSoMoN, non-UbuntuUITK C code use $datadir/locales which is set according to prefix at buildtime11:16
oSoMoNseb128, yes, that’s why I’m confused by ogra_’s comments, are there two separate issues maybe?11:16
seb128oSoMoN, dunno about uitk, I guess it they handle it it's handled11:16
seb128no idea11:16
ogra_if UITK just uses the bindtextdomain fromm gettext any setting of a path will be ignored ... thats what i mean11:20
ogra_wrapping the UITK around the funbction wont change its behaviour11:20
seb128expect if the uitk read the env and do something like bindtextdomain(getenv($somevar))+"usr/share/locale")11:21
ogra_try stracing it :)11:21
ogra_seb128, with its own bindtextdomain ?11:22
ogra_that would indeed work11:22
ogra_if it just hooks into gettext there wont be a getenv ...11:22
kyrofadholbach, man... I have no idea what's going on with that qmake thing11:27
kyrofadholbach, I wonder if it means qmake's support for out-of-source builds is not that great11:31
seb128oSoMoN, you might want to open a new bug rather than commenting on this one btw11:33
seb128oSoMoN, that bug is specifically about code hardcoding a path to bindtextdomain() at buildttime11:33
seb128oSoMoN, your issue is different and with the uitk, that should work11:33
oSoMoNseb128, agreed, will do11:34
seb128oSoMoN, thanks11:35
oSoMoNseb128, could the first statement of your bug report («the core image doesn't have locales definition») be the actual issue I’m facing?11:39
seb128oSoMoN, I guess it's possible yes11:39
oSoMoNseb128, how would I go about shipping the locale definitions in my snap?11:40
seb128not sure there is a standard way, fwded you an email discussing the freecad example which had some hacks for that11:44
mupPR snapd#1497 closed: tests, integration-tests: port auth errors test to spread <Created by fgimenez> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/1497>11:50
mupPR snapd#1496 closed: tests: add spread test for tried snaps removal <Created by fgimenez> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/1496>11:54
ogra_oSoMoN, see  http://bazaar.launchpad.net/~ogra/+junk/nethack/view/head:/nethack.sh and http://bazaar.launchpad.net/~ogra/+junk/nethack/view/head:/snapcraft.yaml for plain locale stuff11:56
ogra_(or install freecad and take a look at thw wraper in /snap/freecad/current/ )11:57
mupPR snapd#1484 closed: tests, integration-tests: port unity test to spread <Created by fgimenez> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/1484>12:02
mhall119sergiusens: are your pkg-config changes in snapcraft 2.12?12:40
mupPR snapcraft#631 closed: Fix the store integration tests <Created by elopio> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/631>12:46
mupPR snapd#1498 opened: tests, integration-tests: port systemd service check test to spread <Created by fgimenez> <https://github.com/snapcore/snapd/pull/1498>12:53
sergiusensmhall119 no, 2.12 was released before I fixed it12:53
=== hikiko|bbl is now known as hikiko
josephtdidrocks: which part?12:59
kyrofasergiusens, elopio is jenkins having problems today? I can't reach results, and now they don't seem to be triggering13:01
sergiusenskyrofa we might need to telegram elopio13:10
kyrofasergiusens, you're seeing that too then, eh?13:10
mhall119sergiusens: how can I try your patch in a cleanbuild then?13:12
sergiusenskyrofa not really13:12
kyrofasergiusens, ah, I triggered them manually... we'll see what happens13:13
kyrofaWhoa, and failed13:13
sergiusenskyrofa I've been working on all this red https://github.com/snapcore/snapcraft/compare/master...sergiusens:refactor/push13:14
kyrofasergiusens, looooovely!13:14
kyrofaI love that kind of red13:14
sergiusenskyrofa and it is now sort of readable too ;-) less spaghettiesh13:15
kyrofaYeah, very nice13:15
josephtsergiusens: nice!13:15
didrocksjosepht: desktop/*13:27
josephtdidrocks: they look the same to me on the wiki[1] and in the parsed output[2]   [1] https://wiki.ubuntu.com/snapcraft/parts [2] https://parts.snapcraft.io/v1/parts.yaml13:30
josephtdidrocks: did you do a 'snapcraft update' to the get the latest parsed output locally?13:30
didrocksjosepht: oh, I thought that was fetched from the description in the git repo13:31
didrocksok, it's copied there as well13:31
didrocksjosepht: any chance to have a different one per "parts" from the same namespace?13:32
didrocks(like desktop/qt5 have a different description from desktop/gtk3)13:32
josephtdidrocks: it might be possible if we allow a 'description' field for a part in snapcraft.yaml.  Can you file a bug please and we'll see if we can add that?13:34
josephtdidrocks: for the time being you could have the general description include a list of specific descriptions (it would be a lot of description for each part)13:35
didrocksjosepht: yeah, that's what I did with the new one :)13:35
didrocksI'll file a bug13:36
josephtdidrocks: thank you13:36
didrocksyw, thanks to you :)13:36
josephtyw as well :)13:37
sergiusenskyrofa maybe fgimenez can help out13:46
sergiusenskyrofa I see on that "other" channel that elopio redeployed da jenks13:46
kyrofafgimenez, snapcraft's jenkins is super broken (not sure about snapd's)13:47
kyrofafgimenez, e.g.
fgimenezkyrofa, sergiusens yep, the redeploy has just finished, all should be working now13:48
fgimenezkyrofa, let me check...13:48
kyrofafgimenez, ah, okay I just retriggered them, see if things are working13:48
kyrofaNope, immediate failures13:48
kyrofafgimenez, the one just ran:
fgimenezkyrofa, it seems to complain about the available images, let me see what is it looking for...13:52
kyrofafgimenez, alright, thanks for checking it out :)13:52
fgimenezkyrofa, np :) i'll let you know how it goes13:52
sergiusenskyrofa my refactor only has 6 unit test errors!13:55
sergiusenskyrofa and only because it wants to mock stuff that I made dissappear :-)13:55
kyrofasergiusens, nice!13:56
fgimenezkyrofa, this one seems to be fine, there was a problem with a variable expansion14:01
kyrofafgimenez, yeah that one looks like it's running okay. Still seeing image errors here:
kyrofafgimenez, yeah, autopkgtests seem to be running now14:05
kyrofafgimenez, just example tests now14:05
fgimenezkyrofa, ok on it14:05
fgimenezkyrofa, examples was pointing to the wrong region, should be fixed now
ogra_do we have any snapcraft.yaml example for SW that uses sourceforge ?14:10
ogra_especially for downloading the source14:10
fgimenezkyrofa, this PR will make the fixes permanent https://github.com/ubuntu-core/snappy-jenkins/pull/186, when landed these errors won't raise again in the next redeploys14:15
mupPR ubuntu-core/snappy-jenkins#186: fixed regions and variable expansion <Created by fgimenez> <https://github.com/ubuntu-core/snappy-jenkins/pull/186>14:15
fgimenezlet me know if anything else is missing14:16
kyrofafgimenez, thank you! Looks great :)14:16
fgimenezkyrofa, yw :)14:17
=== tyhicks` is now known as tyhicks
sergiusenskyrofa before I move on, I would really like an initial review on https://github.com/snapcore/snapcraft/pull/63215:03
mupPR snapcraft#632: WIP Refactor/push <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/632>15:03
mupPR snapcraft#632 opened: WIP Refactor/push <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/632>15:04
kyrofasergiusens, alrighty15:07
kyrofasergiusens, yeah the direction looks good. Missing some good error handling, but I assume that stuff is coming15:14
sergiusenskyrofa except KeyError?15:14
sergiusenskyrofa yeah, that is missing ;-)15:15
kyrofaYeah, that :P15:15
sergiusenskyrofa going to go the exception route and create a StoreReviewError or something like that15:15
sergiusensgetting rid of so many if/else on keywords really made me happy15:15
kyrofaYeah I bet15:16
sergiusenskyrofa so happy I lost track of time last night and went to bed really late :-P15:17
niemeyerogra_: Regarding "afaik niemeyer plans to have some generic preloader thing builtin in snapd at some point ... that will make it easier"15:18
niemeyerogra_: I actually have it pretty much ready15:18
niemeyerogra_: It's not inside snapd though, but a library on its own15:19
niemeyerogra_: Worked on that over the weekend while fighting an app15:19
mupPR snapd#1482 closed: store, many: start using the new details endpoint <Reviewed> <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/1482>15:25
mupPR snapd#1488 closed: store: switch search to new snap-specific endpoint <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/1488>15:26
ogra_niemeyer, cool15:26
ogra_thanks for cllearifyin15:26
mupPR snapd#1499 opened: tests: use systemd-run for starting and stopping the unity app <Created by fgimenez> <https://github.com/snapcore/snapd/pull/1499>15:54
* kyrofa bashes head on desk and goes to make more coffee15:56
ogra_nessita, help ... i'm trying to upload a new nethack snap ... the old one was for 15.04 and it seems the store thinks the old one is a click ... when i upload a new revision i'm told click and snap can't be mixed ... trying to register a new package tells me the name is taken16:02
ogra_and unpublishing doesnt clean that up16:02
nessitaogra_, let me check!16:02
nessitaogra_, I see 3 nethacks: nethack-amd64 nethack-armhf and nethack -- is the last one?16:02
ogra_(and renaming obviously only renames the representation in the store, not the packagename)16:02
nessitalooks like is busted somehow, let me check16:03
ogra_i dont mind if you remove it completely16:03
ogra_in case thats needed16:04
nessitaogra_, I may do that, but let me fully check16:04
nessitaogra_, let me delete the package and you can register the name right after16:04
ogra_nessita, btw, ircproxy, upnp-server and packageproxy will have the same issue (and i plan fesh uploads for them)16:05
kyrofaelopio, this is successful, right?
kyrofaelopio, looks like it just didn't clean up right?16:05
ogra_so it might make sense to remove them too16:05
nessitaogra_, nethack deleted, feel free to re-register16:05
kyrofasergiusens, short of the (incorrectly failing) tests, does #622 look good?16:06
nessitaogra_, let me check details for ircproxy, upnp-server and packageproxy16:06
ogra_they can all go as well16:06
ogra_nope ... "new snap" still tells me the namespace is taken16:06
ogra_ah, because i wown it already :P16:08
ogra_direct upload works16:08
sergiusenskyrofa what test is failing? care to record that in a comment with the traceback?16:10
kyrofasergiusens, I'm actually not sure what's happening on autopkg16:18
kyrofaIt looks like a testbed issue, but I'm not certain16:18
sergiusenskyrofa good thing I told elopio to keep an eye on jenkins :-P16:22
tsimonq2key kyrofa, Jenkins passes in snapcraft#619 - what's next? I think I need a unit test, how would I go about doing that? (I mean, in the Zip and Tar sources or in a separate class?)16:28
mupPR snapcraft#619: Add source-checksum option for tar and zip sources (bug 1585913) <Created by tsimonq2> <Conflict> <https://github.com/snapcore/snapcraft/pull/619>16:28
tsimonq2oh great, my FAVORITE spam bot just commented on my PR twice again.... :P16:29
mupPR snapd#1498 closed: tests, integration-tests: port systemd service check test to spread <Created by fgimenez> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/1498>16:50
mupPR snapd#1499 closed: tests: use systemd-run for starting and stopping the unity app <Created by fgimenez> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/1499>16:52
mupPR snapcraft#633 opened: qmake plugin: Stop doing out-of-source builds <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/633>16:52
=== dpm is now known as dpm-afk
jdstrandzyga_: hey, so, snap-confine has a 1.0.34 tag but the debian/changelog still says UNRELEASED for 1.0.3417:37
kyrofasergiusens, any idea what's happening here? https://travis-ci.org/snapcore/snapcraft/jobs/14282072617:40
kyrofaNot the most descriptive failure I've seen17:40
ssweenyjdstrand, I'm trying to snap a python app (checkbox/plainbox if that's important) and I keep getting errors where python is trying to run /sbin/ldconfig. I think it's some internal python thing that's doing it. Any advice on how to proceed?17:40
sergiusenskyrofa I asked josepht to look at it, the wiki is probably broken and our tests rely on it17:41
kyrofasergiusens, ah17:41
sergiusenskyrofa I think work is already in progress to move to fake servers17:41
kyrofaGood deal17:41
josephtsergiusens, kyrofa: yes, the desktop entry is broken17:41
kyrofaI love it-- an errant wiki entry takes down our CI17:42
sergiusensjosepht do you know how to fix it?17:46
josephtsergiusens: yeah, I think so17:46
jdstrandssweeny: for now install in devmode. you can also file a bug stracing the process and seeing what it is doing. You can add '/sbin/ldconfig ixr,' to /var/lib/snapd/apparmor/profiles/snap.your.snap then run 'sudo apparmor_parser -r /var/lib/snapd/apparmor/profiles/snap.your.snap' and see if that work for you (please mention that in the bug)17:48
jdstrandssweeny: when filing a bug, please add the 'snapd-interface' tag17:48
ssweenyjdstrand, will try, thanks! (it works perfectly in dev mode and I'm working through confinement issues)17:49
ssweenyjdstrand, also it's trying to write to $HOME/snap/${SNAP}/${REV}/.cache and it's being denied. Are hidden files in the user writable area blocked by default?17:51
ssweenyi think that's what's being resolved for $XDG_CACHE_DIR17:52
ssweenyor $XDG_CACHE_HOME rather17:53
josephtsergiusens: the parser seems to be getting a stale/cached version of the wiki17:54
kyrofajdstrand, you remember that overlayfs script you had a while back? Just for ease of use I decided to throw it in a snap, but it seems the private mount space won't let me actually use it, even in devmode18:02
kyrofajdstrand, is there any way around that?18:02
jdstrandssweeny: hmm,  $HOME/snap/${SNAP}/${REV}/.cache should be allowed. can you paste the denial?18:05
kyrofajdstrand, oh nevermind... it's denying access to mount, haha18:06
kyrofaWait no... that's allowed18:06
kyrofaI stand by the mount space theory18:07
ssweenyjdstrand, it turns out I installed the snap as root and it created the rev directory with root ownership18:07
ssweenyeven under /home/ubuntu/snap/...18:07
jdstrandkyrofa: I can't so for sure why it would not work, but the plethora of bind mounts that we have, it wouldn't surprise me. it might need a different invocation from within a snap (haven't tried)18:08
jdstrandssweeny: yes that is an annoying bug18:08
jdstrandinstalling as root shouldn't be the issue. running as root before running as yourself would18:08
jdstrandor running another program (eg, snappy-debug) as root on a system that doesn't have ~/snap already will do it18:09
jdstrand(since ~/snap is root owned then)18:09
ssweenyI think I did run as root first18:12
niemeyerjdstrand: Would you mind to have a quick look on snapd#1405 when you have a moment?  Should be an easy walk18:16
mupPR snapd#1405: interfaces: add zigbee-dongle interface <Blocked> <Reviewed> <Created by jocave> <https://github.com/snapcore/snapd/pull/1405>18:16
slo_Hi, is there any HW matrix support available for Snappy? So which CPUs are supported or which HW platforms (rpi, artik, etc...) ?18:16
kyrofajdstrand, indeed, if you install hello-world in --devmode and bindmount $HOME/foo to $HOME/bar in hello-world.sh, only hello-world.sh can see is. If you open another session it's not mounted at all18:20
kyrofas/see is/see it/18:21
jdstrandniemeyer: I'll add it to my list (though note I'm working on a semi-urgent issue and may not get to it today, but should be able to tomorrow if not today)18:21
jdstrandzyga_, tyhicks: fyi, https://github.com/snapcore/snap-confine/pull/72. I think this should be part of the /ubuntu-core-launcher/snap-confine SRU that is being worked on18:23
mupPR snap-confine#72: use execle() with empty environment when calling snappy-app-dev <Created by jdstrand> <https://github.com/snapcore/snap-confine/pull/72>18:23
kyrofajdstrand, I figure it's the slave mount namespace18:23
kyrofajdstrand, is there any nifty mount instantiations that will allow me to hop out of that in devmode?18:24
jdstrandkyrofa: that wouldn't surprise me. that said, you can deliver it as snap and execute it from /snap/name/current/script (yucky, but... ;)18:24
kyrofajdstrand, haha, that's exactly what I'm doing but... yuck :P18:25
jdstrandkyrofa: otoh, idk. atm I'm working on said semi-urgent issue and don't have time to look right now18:25
kyrofajdstrand, that's alright, no problem18:25
niemeyerjdstrand: Thanks!18:25
jdstrandniemeyer: sure thing18:26
zygajdstrand: ack, looking now18:26
zygajdstrand: ack, I'll merge this soon but please add a spread test18:27
zygajdstrand: ?18:27
jdstrandI'll have to think about that. this is a pretty convulated attack18:28
zygajdstrand: after this lands I'll release .3518:28
niemeyerjoc_: That PR is yours isn't it?18:28
zygajdstrand: is there something we could do to just show that PATH is reset?18:28
zygajdstrand: what is NULL environment in practice? empty18:28
jdstrandzyga: yes, empty18:28
zygajdstrand: or inherited from some safe default18:28
zygaso PATH won't resolve at all, is that what we want?18:29
niemeyerjoc_: snapd#1405 that is18:29
mupPR snapd#1405: interfaces: add zigbee-dongle interface <Blocked> <Reviewed> <Created by jocave> <https://github.com/snapcore/snapd/pull/1405>18:29
jdstrandzyga: the problem is that only snappy-app-dev will see that environment18:29
zygajdstrand: that's right, so as long as it can cope, it is okay18:29
jdstrandzyga: will spread allow me to replace /lib/udev/snappy-app-dev? I guess not since it is in read only part?18:29
zygait will18:29
zygayou build the package, install it and run anything as root18:30
zyga(as long as you clean up in restore you are good)18:30
jdstrandok, then I can come up with a test18:30
jdstrandzyga: we want an empty environment, coping in a shell script is difficult. the next thing I'm going to do is to rewrite the shell script, but that doesn't need to be part of 1.0.3518:31
jdstrandzyga: let me add a comment like tyhicks suggested and then work on spread. it will take me a bit-- this is my first spread test and I haven't set up the spread env yet18:31
jdstrandI may have some questions18:32
zygajdstrand: I'll be here18:32
zygajdstrand: hint: spread -list18:32
zygajdstrand: then spread $idofthetest18:33
zygajdstrand: that will get you started faster18:33
jdstrandzyga: oh, by cope maybe you meant if snappy-app-dev can handled not having PATH set, for example. yes-- anything the shell needs, if it isn't there, the shell will provide (PATH, HOME, PWD, etc), but passing a NULL environment means the user can't control those things18:39
kyrofanessita, why do automated review warnings result in a manual review being required?18:44
kyrofanessita, like this one: "Could not find compiled binaries for architecture 'arm64' lint-snap-v2_architecture_specified_needed (arm64)" at worst it's simply a mistake, at best (like my case) on purpose18:45
jdstrandthat is an ongoing conversation18:47
jdstrandbottom line is, each check needs to be re-reviewed for what should block and what shouldn't18:48
jdstrandand we need to think about how to present to the user and the uploader18:49
jdstrandwe probably need to do that, have errors block and warnings display but not block18:49
jdstrandfeel free to file a bug against the review tools18:50
jdstrandkyrofa (cc nessita): ^18:50
kyrofajdstrand, alright, thank you18:51
kyrofajdstrand, any chance you could approve that snap?18:55
jdstrandkyrofa: done18:59
* jdstrand leaves sergiusens' nil-snap alone18:59
sergiusensjdstrand please do :-)19:00
kyrofajdstrand, thank you!19:01
nessitajdstrand, thanks for answering19:05
nessitakyrofa, we want good packages in the store, so why would your package not have a binary? (besides what jamie said)19:06
kyrofanessita, because snappy doesn't support it being a binary19:10
kyrofanessita, so one must run the script straight out of the snap19:10
kyrofanessita, but the snap is still a useful deployment mechanism19:10
kyrofa /sharing mechanism19:11
kyrofajdstrand, ah, an armhf arrived late if you could push the button on that one as well19:13
kyrofaThank you!19:14
kgunnogra_: yo, for stitching a core 16 edge image....should i still be using the u-d-f on mvo's file share?19:22
kgunnhe has one there listed "experimental"19:23
kgunni think i'm on cirica may 2nd19:23
zygajdstrand: about the exec bug, I'll do something with suse but then release .35 with tests or without tests before I EOD20:01
jdstrandzyga: I've almost got a test verifying cgroups work correctly. then the next test for regressing execle should be fast20:02
zygajdstrand: fantastic!20:03
zygajdstrand: I'll work on running spread tests on sid this week20:03
jdstrandzyga (cc niemeyer): gotta say, spread was really easy to get started with20:03
zygajdstrand: and suse/fedora/arch - depending on what is the easiest one20:03
zygajdstrand: not sure if you already know about it, -reuse and -keep accelerate iteratio cycle20:06
zygayou just re-run on the same host all the time, no allocation required20:06
jdstrandoh yes20:06
jdstrandusing -reuse a lot20:06
jdstrandzyga: ok, added preliminary cgroup test20:09
jdstrandzyga: so, debian/changelog still has 1.0.34 UNRELEASED20:09
jdstrandzyga: can you fix that up? I can then add an entry for this20:10
sawdogHi folks, I was redirected here via the Ubuntu channel; I’m asking:  so I’m running on a Gigabyte IOT gateway box, have Core installed on a USB stick and am trying to get it installed on the internal MMC. I installed curl via snappy so I can grab the image and download it so that I can dd the image to the internal MMC; but using curl to redirect the output to a file, I get a permission denied. I’m assuming this is because curl20:11
sawdogdoesn’t have the appropriate access granted to write to my disk?20:11
zygajdstrand: hmm, that's complicated, this is release to debian with separate packaging now20:11
zygajdstrand: (non-native)20:11
sawdogI did do a snappy hw-assign curl.tetor /dev/sda520:11
zygajdstrand: the packaging here is just used for spread testing20:12
sawdogwhich granted access to the device, but same behavior; —> 0Warning: Failed to create the file ubuntu-core-15.04-intel-nuc.img.xz: Warning: Permission denied curl: (23) Failed writing body (0 != 9922)20:12
zygajdstrand: add the changelog entry anywhere you want, it is not essential20:12
zyga(for packaging)20:12
jdstrandzyga: ok, then I'll let you handle the debian/changelog entry?20:12
zygajdstrand: yep20:12
jdstrandzyga: I tell you waht, I'll add it here so you know what to add later20:13
zygajdstrand: I will do the rest and coordinate releases20:13
sawdogI’m assuming that the curl app doesn’t have proper access…20:14
zygasawdog: I'm not sure what you are doing is what you really want20:15
zygasawdog: snappy has a few bits that need to understand boot and this will likely break20:16
zygasawdog: I'd recommend asking on the mailing list, it will probably require a longer and more comprehensive answer20:16
sawdogzyga: Ah, it seemed like from the docs I could download an .img.xz - xzcat that pipped to dd and write it to the internal MMC20:17
sawdoge.g this is from the docs online:20:18
sawdogDecompress the image and write to the disk downloaded:20:18
sawdogUsing the NUC eMMC board:20:18
sawdogxzcat ubuntu-core-15.04-intel-nuc.img.xz | sudo dd of=/dev/mmcblk0 bs=32M; sync20:18
zygasawdog: it depends on finer details20:18
sawdogbut I need to download the compressed image - which is where I’m at; writing it to my usb drive (e.g. /dev/sda5)20:18
zygasawdog: anyway, I need to work on something, I', sorry but I cannot help you now20:19
sawdoglooks like I’m facing an AppArmor violation with curl20:19
thomiHi all, while building a snap with more than one python-based part, I'm getting this error when trying to build the snap: https://pastebin.canonical.com/160364/ Does anyone know how to fix this?20:19
zygathomi: using a public pastebin would be easier for others to read20:21
thomizyga: good point20:21
thomizyga: http://pastebin.ubuntu.com/18656156/20:22
zygajdstrand: ready?20:23
zygathomi: just add all the python bits to one shared part20:23
zygathomi: that's what I did before20:23
zygathomi: I mean all the stage-packages20:23
jdstrandzyga: I pushed a couple more commits. almost have a regression test20:23
thomizyga: unfortunately that's not do-able here. One of the parts isn't using the python build plugin, as it's a custom build system20:24
thomizyga: also, I'll want to share one of the parts for others to re-use20:24
zygajdstrand: k, I'll wait20:28
* zyga packages $Nth go package for suse20:28
zygathomi: that's okay, my point is that you can move *all* stage-packages to a dedicated part20:28
zygathomi: then use after to build when those are ready20:28
zygathomi: as for sharing, not sure, just a snapcraft design or bug20:29
thomizyga: I don't understand, sorry - my WIP snapcraft.yaml is http://paste.ubuntu.com/18656658/ - what are you suggesting?20:30
zygathomi: I don't know what wxpython plugin is doing20:31
zygathomi: ask sergiusens perhaps, sorry20:31
thomiok, thanks20:31
=== chihchun is now known as chihchun_afk
thomisergiusens: any ideas on how to resolve http://paste.ubuntu.com/18656658/ ?20:38
thomizyga: hey - do you have any objection if I take your excellent pmr tool and turn it into a proper lp project and work on it? We're using it in onlineservices, and want to make a few modifications20:41
zygathomi: woot, not at all20:44
zygathomi: I'm very glad someone found it interesting! :)20:44
thomizyga: awesome, thanks20:44
zygathomi: I guess it spread through roadmr :D20:44
zygathomi: I may send some pull requests your way :)20:44
roadmrzyga: yes, I am the virus :)20:44
zygahaha, that's fantastic roadmr :)20:44
zygahow are you doing? :)20:44
zygait'd be really cool if pmr was a snap20:45
roadmrgreat! how about you? having snap-fun?20:45
zygaso it can use confinement to protect user data from malicious pull requests20:45
zygaand would be available everywhere20:45
roadmrahh, great :) though that may interfere with e.g. test scripts that use fancy stuff, lxc and so on, right?20:45
zygaroadmr: yes, though all will work in good time :)20:46
roadmrawesome \o/20:46
zygaI'm deep in various distributions, not running ubuntu much anymore :)20:47
zygaI'm in suse world now :)20:47
roadmrhaha :)20:51
roadmrcross-distro snap support is awesome, thanks for that \o/20:52
slo_anyone runs eclipse kura on snappy?21:27
mupPR snapd#1500 opened: many: remove snapstate.Candidate <Created by mvo5> <https://github.com/snapcore/snapd/pull/1500>21:28
zygaslangasek: hey, can you please release snap-confine to debian21:33
zygaslangasek: if you are around, I just published the upstream tarabll21:33
zygaslangasek: I sent a patch to mwhudson, not sure if it got applied yet (I didn't check)21:33
mwhudsonzyga: i've done both of those things21:33
zygamwhudson: oh, fantastic21:33
zygamwhudson: can you do 1.0.35-1 please?21:34
mwhudsonyou guys don't stop do you21:34
zygamwhudson: btw, glad to see you around :) how have you been?21:34
zygamwhudson: if you add me to comitters/uploaders I can co-maintain the package21:34
zygamwhudson: we don't stop to fix security issues :)21:35
* zyga goes to push gentoo and fedora bis21:35
mwhudsonzyga: oh are you a DM?21:36
mwhudsoni've been OK, had my head in go toolchain land for most of the last year21:36
zygamwhudson: no :-(21:36
zygamwhudson: well, not sure actually, but I haven't been updating my debian packages lately21:36
slangasekzyga: you don't appear to be a DM, and I imagine you would know if you were :)21:37
zygaslangasek: yeah, I got lost in the paperwork21:38
zygaslangasek: I think I should apply, it's super annoying sometimes21:38
mwhudsonyeah, it's not overly hard21:38
mwhudsonif you have a key signed by a DD anyway21:38
mwhudsonthe waiting is the hardest part21:38
zygamwhudson: hmm, I don't know if I do, I had that many years ago but I lost that key with a laptop hdd years ago21:39
mwhudsonzyga: so, step 1) ;-)21:39
mwhudsonzyga: for reference https://wiki.debian.org/DebianMaintainer21:40
* mwhudson hits gbp with a stick21:41
zygamwhudson: hehe, I might just do it now :)21:42
mwhudsonslangasek: can you just say pristine-tar = True in gbp.conf?21:42
slangasekmwhudson: should be possible, yes21:44
mwhudsonzyga: you should fix that VENDOR_ARGS=--disable-confinement thing in master :-)21:45
* zyga checks if hrw is a DD21:45
zygamwhudson: in upstream master?21:45
zygamwhudson: yes, I plan to change that entirely21:45
zygamwhudson: by dropping packaging upstream21:45
mwhudsonah ha, yes that would solve that problem21:45
zygamwhudson: and adjusting spread tests to fetch debian packaging, generate a new tarball, building *that* package and then merry-carrying-on21:46
zygamwhudson: just bigger than what I wanted to tackle now21:46
mwhudsonfair enough21:46
zyga(I'm discovering suse packaging the hard/practical way)21:46
mwhudsonah nope, screwed up gbp again :-)21:47
mwhudsondifferently this time21:47
zygapackaging tools are so diverse and different21:48
mwhudsononce more around ...21:51
zygasigh, not signed :/21:51
zygaok, time to call it a day21:52
zygajust fedora bump and time to sleep21:52
mwhudsonzyga: 1.0.35-1 will be in debian soon21:53
mwhudsonzyga: are there release notes i can crib for debian/changelog?21:53
mwhudsonor i can just say "New upstream release" and leave it at that21:53
zygamwhudson: there are, let me show you21:54
zygamwhudson: btw, how did you end up packaging snap-confine? :-)21:54
mwhudsonzyga: um well i ended up on the foundations team and helping with snapd packaging made sense cause it's in go21:55
mwhudsonand then snap-confine is related to snapd?21:55
zygabtw, do you think golang shared libraries will hit yakkety?21:56
mwhudsonthey have!21:56
zygaand if so, how does that work in practice21:56
zygais that used in any other distro?21:56
mwhudsonnot sure, fedora might have something?21:56
mwhudsonit's still very much in its infancy, i got about halfway to getting lxd to use them and then hit a grotty upstream circular dependency problem21:57
zygahow can I check that shared libraries are in use?21:57
mwhudsonzyga: https://docs.google.com/document/d/1IOlBWWgcDeB9PfRORENESYj8iJt4W2EwsbYcpg4akBE/edit#heading=h.j590j9v4qbxg21:57
zygaldd will say something?21:57
zygaadded to my reading list, I bet this will end up all over all the distros soon21:58
zygaI need to spend some time tomorrow to build my VM host :/22:00
zygaso many distros22:00
mwhudsonok, uploaded to debian22:00
mwhudsonoh and i'm patch pilot today22:01
zygamwhudson: good night, talk to you soon I bet ;)22:04
mwhudsonvery probably22:05
mwhudsongood night22:05
mhall119sergiusens: did you ever try to build pantheon-mail snap with your changes? If so, how far did you get?22:08
mcphailIf my snapcraft.yaml has 2 parts containing the same build-packages, can I safely remove the duplicates?22:25
mupPR snapd#1439 closed: daemon, overlord: add buy endpoint to REST API <Reviewed> <Created by pete-woods> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/1439>22:32
jdstrandzyga: hrmm22:33
jdstrandzyga: the apparmor profile for snap-confine is wrong. you moved the binary to /usr/lib/snapd/snap-confine but /etc/apparmor.d/usr.bin.snap-confine doesn't reflect that22:34
zygajdstrand: running spread tests with the change locally22:42
zygajdstrand: let's see what fails in practice22:42
thomiCan anyone point me at some docs that explain how snapcraft decides what should get copied from 'stage/' to 'prime/' ? I have files in 'stage' that aren't getting copied into the snap, and they're required for the snap to run correctly22:45
thomiev: any idea on the above? ^^23:08
evthomi: have you tried including these missing files in a "snap:” stanza?23:21
evthomi: if you haven’t seen it yet, https://github.com/ubuntu/snappy-playpen/wiki/Examples is handy23:22
thomiev: I haven't - I assumed there'd be something like that, but was looking for docs https://github.com/ubuntu/snappy-playpen/wiki/Examples looks perfect, thanks23:23
thomiev: is there a way we can link to that from snapcraft.io/create?23:23
thomiactually, it looks like I want something in organize: awesome23:24
thomiso many things I didn't know existed :D23:24
mupPR snapcraft#634 opened: Run the parser integration tests with debug flag <Created by elopio> <https://github.com/snapcore/snapcraft/pull/634>23:50

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