sergiusenselopio I've updated 62900:04
sergiusenselopio 629 is the interesting one to be tested against the staging store00:12
MichaelTunnellif a non-official snap is added to the snapstore and an official team wants to take it over is it possible to transfer ownership to become official and if so can the suffix be removed from the snap name?01:28
MichaelTunnellniemeyer: ogra_: ?01:30
niemeyerMichaelTunnell: Yes to both01:38
elopiosergiusens: you can register your own user on staging, and pass the TEST_USER_NAME and TEST_USER_PASSWORD. Just make sure to accept the agreement first.01:42
elopioor I can send you the test user password.01:42
MichaelTunnellniemeyer: nice thanks for clarification01:43
mupBug #1598984 opened: snap list reports no snaps installed when invalid name passed <Snappy:New> <https://launchpad.net/bugs/1598984>02:28
TheMusoIs the latest version of snapd going into yakkety any time soon? I know we are targetting snaps for xenial right now, but it would be nice to be able to work with and test snaps in yakkety as well.02:45
mupPR snapcraft#629 changed: Proper message when registering an already registered snap <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/629>03:07
sergiusenselopio about lp: #1572399 ... I talked to cprov and he asked me to create a bug, mind just linking this to the store tracker?03:41
mupBug #1572399: [upload] Catch name registration issue and explain how <Snapcraft:In Progress by tsimonq2> <https://launchpad.net/bugs/1572399>03:41
dholbachhey hey05:26
=== chihchun_afk is now known as chihchun
mupPR snapd#1485 opened: snap: add `snap run --debug-shell` <Created by mvo5> <https://github.com/snapcore/snapd/pull/1485>07:07
mupPR snapd#1394 changed: debian: add snapd.postrm that purges <Created by mvo5> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/1394>07:16
mupPR snapd#1486 opened: interfaces: add wireless-control <Created by lpotter> <https://github.com/snapcore/snapd/pull/1486>07:51
mupPR snapd#1481 changed: overlord: make patch1_test more robust <Created by mvo5> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/1481>08:41
mupPR snapd#1487 opened: tests: set yaml indentation to 4 spaces <Created by fgimenez> <https://github.com/snapcore/snapd/pull/1487>08:43
=== chihchun is now known as chihchun_afk
=== seb128_ is now known as seb128
mupPR snapd#1462 closed: snapstate: cleanup downloaded temp snap files <Reviewed> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/1462>09:43
mupPR snapd#1487 closed: tests: set yaml indentation to 4 spaces <Created by fgimenez> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/1487>09:55
mwhudsonzyga, mvo: hey, i was looking at the snap-confine package in the yakkety queue10:47
mwhudsonwhere did the tarball come from? it doesn't seem to match the release on https://github.com/snapcore/snap-confine/releases10:47
mwhudson(i think maybe the one in the upload queue is a straight git export and the one on github is the output of some 'make dist' type thing?)10:47
mwhudsonalso i'm probably going to upload it to debian as 1.0.34-1 tomorrow so maybe you should use a version that won't be overwritten by that? (or maybe being overwritten is fine?)10:48
mwhudsonalso also i'm going to bed10:48
mwhudson(package for debian in http://anonscm.debian.org/cgit/collab-maint/snap-confine.git/log/?h=debian fwiw)10:50
zygamwhudson: hey10:51
zygamwhudson: I'm not sure, I didn't' do the yakkety release, please definitely use the make dist one10:52
mwhudsonzyga: the snap-confine-1.0.34.tar.gz is a make dist one?10:53
mwhudson(it looks like it to me)10:53
zygamwhudson: yes10:53
mwhudsonzyga: on https://github.com/snapcore/snap-confine/releases10:53
zygamwhudson: libexecdir should stay as /usr/lib/snapd10:53
zygamwhudson: (for various interesting reasons)10:53
willcookedidrocks, btw - did that issues detecting gtk3 vs gtk2 on 64bit get fixed?10:53
zygamwhudson: if you want I can make those changes and send you patches10:53
mwhudsonzyga: i took that change10:55
mwhudsonzyga: but thanks for confirming :-)10:56
zygamwhudson: in .34 the config option changed to --disable-seccomp and --disable-apparmor10:57
zygamwhudson: nice, thank you10:57
mwhudsonzyga: ah ok, please send a patch for that :)10:57
zygamwhudson: gladly10:57
zygagouchi: hey10:58
mwhudsonit still built with --disable-confinement though, unless i goofed my build testing10:58
gouchiI was wondering if those device will be part of opengl interface ?10:58
zygamwhudson: that's odd, it definitely should not10:58
zygamwhudson: cloning to test now10:58
* mwhudson really really going to bed now10:59
zygagouchi: let me check10:59
zygamwhudson: bye!10:59
=== hikiko is now known as hikiko|ln
mupPR snapd#1478 closed: many: Add device authentication skeleton <Created by wgrant> <Closed by niemeyer> <https://github.com/snapcore/snapd/pull/1478>11:01
mupPR snapd#1488 opened: store: switch search to new snap-specific endpoint <Created by chipaca> <https://github.com/snapcore/snapd/pull/1488>11:01
zygagouchi: /dev/dri/dri0 is not in the opengl interface11:03
zygagouchi: can you tell me more about the application you are trying to run as well as the GPU and driver you are using11:04
gouchizyga: using intel gpu I tried to run retroarch in drm/kms with egl11:05
gouchizyga: https://github.com/libretro/RetroArch/wiki/KMS-mode11:05
gouchizyga: As I couldn't make it work with x11 interface, I tried with opengl interface11:06
zygagouchi: this looks closer to the mir interface11:06
zygagouchi: and it seems it would require a new interface to work, can you please look at snapd pull requests on github.com/snapcore/snapd, look at the mir interface there and use it as a starting point11:07
gouchizyga: I will try thank you11:07
zygagouchi: then the application using the new interface11:07
zygagouchi: note that application can  gain permissins just by having a slot of a given type, it doesn't have to be connected11:08
zygagouchi: in thise sense, you could use the new interface slot to grant that application to run with direct gpu access11:08
gouchizyga: this one https://github.com/snapcore/snapd/pull/1299 ?11:09
mupPR snapd#1299: create mir interface <Reviewed> <Created by kgunnfront> <https://github.com/snapcore/snapd/pull/1299>11:09
=== chihchun_afk is now known as chihchun
mupPR snapd#1485 closed: snap: add `snap run --shell` <Reviewed> <Created by mvo5> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/1485>11:28
didrockswillcooke: yeah, it's fixed now11:57
willcookedidrocks, thanks!11:57
didrockswillcooke: just rebuild your snap if you didn't since Friday :)11:58
willcookerunning it now11:58
=== hikiko|ln is now known as hikiko
didrockswillcooke: ensure you snapcraft update (I don't know how much it caches)11:59
mupPR snapd#1473 closed: tests, integration-tests: port refresh all test to spread <Created by fgimenez> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/1473>12:06
gouchi_zyga: better http://www.hastebin.com/dubevesula.coffee but I got fatal error12:16
zygagouchi_: is that with mir or with a new interface?12:17
gouchi_zyga: mir interface12:18
gouchi_zyga:   plugs: [network, network-bind, mir, opengl, home]12:18
zygagouchi_: try slots: [mir]12:18
zygagouchi_: mir slot let mir "be mir" and talk to GPUs and such12:18
willcookedidrocks, yay! works.  thank you!!12:19
zygagouchi_: though you will probably need bleeding edge code from that branch12:19
zygagouchi_: do you know how to rebuild snapd?12:19
gouchi_zyga: I followed https://github.com/snapcore/snapd/blob/master/README.md and build it12:19
zygagouchi_: you can use github.com/zyga/devtools to use the new version easily12:20
zygagouchi_: look at the refresh-bits script there12:20
zygagouchi_: I wrote that for working on interfaces12:20
gouchi_zyga: I will try thank you very much12:21
zygagouchi_: refresh-bits snap snapd setup run-snapd restore12:22
zygagouchi_: (read the code to see what happens)12:22
zygagouchi_: it also has useful --help12:22
gouchi_zyga: same error with  slots: [mir]12:26
didrockswillcooke: excellent! yw ;)12:29
mupPR snapd#1458 closed: many: add `snap enable/disable` commands <Created by mvo5> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/1458>12:41
mupPR snapd#1489 opened: snap: ensure unknown arguments to `snap run` are ignored <Created by mvo5> <https://github.com/snapcore/snapd/pull/1489>12:41
sborovkovHi. What is log-observe interface doing? I am using python's systemd API to get journal contents and poll for it's changes but I still get some APPARMOR warnings13:06
sborovkovoperation="open" profile="snap.screenly-client.logger" name="/run/log/journal/a82c9efe141649b3983327aa20a8ed23/system@5b2dfcfdd13348ce82cbcf01123e4caa-000000000000064e-000536e30cc89344.journal"13:06
sborovkovAnd this one: operation="open" profile="snap.screenly-client.logger" name="/run/log/journal/a82c9efe141649b3983327aa20a8ed23/system.journal" pid=1368 comm="python3" requested_mask="r" denied_mask="r" fsuid=0 ouid=013:07
seb128sborovkov, whare snapd version do you use?13:14
sborovkovseb128: snapcraft should be from last week I think. I did build the image before that I think though13:14
seb128sborovkov, snapd, not snapcraft13:15
seb128the snappy system service13:15
seb128not the build tools13:15
sborovkovUnderstood, how do I make it ouput the version?13:15
seb128on ubuntu "dpkg -l | grep snapd"13:16
mvoapt list snapd13:16
mvo(so much nicer IMNSHO)13:17
josephtgood to know13:17
sborovkovwait, I am on RPI13:17
sborovkovno dpkg13:17
seb128well anyway the journal interface got some fixes in 2.0.10 which is less than a week old13:18
seb128make sure you have this version13:18
seb128unsure about your RPI image13:18
seb128maybe mvo can help13:18
sborovkovMy image is more recent, so might be that.13:18
seb128(snapd really needs a -v argument)13:18
mhall119sergiusens: what's the easiest way for me to try your snapcraft pkg-config lookup patches when doing a cleanbuild?13:19
ogra_grmpf .. packaging a java GUI app i can get everything to work fine with the default GTK theme (win95 style) ... but as soon as i ship light-themes and set a theme no icons are found :(13:26
jdstrand_sborovkov: re log-observe and systemd journal> there is a fix in 2.0.10 for that (https://launchpad.net/ubuntu/+source/snapd/2.0.10). If you are using Ubuntu, 2.0.10 hasn't landed yet in 16.04, but is on its way13:28
seb128ogra_, unsure what toolkit they use but you might lack a svg loader13:32
* didrocks wonders why seb128 is talking about svg loader :p13:33
didrockshem hem ;)13:33
seb128lalala ;-)13:33
didrocksogra_: yeah, ensure that you have this svg loader, icons are in svg and we experienced that some theme engine won't even look at them until they have a svg loader/decoder13:34
ogra_seb128, well, i see it in the gdk loaders dir13:34
ogra_and i regenerate the loader cache on app startup13:34
didrocksis it that one doing the decoding?13:34
didrocksdespite using the gtk theme engine in qt, it's qt itself doing the svg decoding13:34
didrocksfor instance13:34
ogra_well, it is a java awt app13:35
didrocksunsure if awt can decode svg13:35
didrockstry shipping a corresponding .png image13:35
seb128well those work on normal desktop I guess13:35
didrocksin the same directory than one icon it's using13:35
seb128that would be a known problem otherwise?13:36
didrocksand see if it can open it13:36
didrocksyeah, what about without devmode?13:36
ogra_i only run without devmode ;)13:36
didrocksand if you run your app directly?13:37
didrockslike without the u-c-l wrapper?13:37
ogra_if i run it as java -jar $path_to_jar i get icons13:38
ogra_so thereis still some env var missing or so13:38
didrocksdid you try with the gtk launcher?13:38
didrocksto see if it's something that you might miss13:39
ogra_way to much overhead13:39
didrockswell, try it at lest13:39
=== jdstrand_ is now known as jdstrand
ogra_but i'll tyr to grab bits from there13:39
didrocksand you know, you can override some stage-packages13:39
didrocksat least, try it and see13:39
didrocksthat will give you some hints13:39
didrocksyou are then welcome bringing a smaller one and contribute :)13:39
didrocksrather than duplicating13:39
* ogra_ is looking for the most minimal setup here 13:40
didrocksthis is what I did with the new desktop launchers13:40
didrocksthe minimal required things for gtk/qt apps13:40
didrocksbut it seems it's been a couple of days you are stuck on this without even trying another one13:40
didrocksjust to ensure that the issue is larger than just missing dependencies13:41
didrocksintellij IDEA is using java awt13:43
didrocksand have working theme here13:43
didrocks(since I switched it to my gtk launcher)13:43
ogra_didrocks, btw, it would be good if you stopped using a bash shebang in these launchers ... on actual snappy images we might not have bash access13:49
ogra_(while /bin/sh is always guaranteed to be there)13:49
didrocksogra_: yeah, patch welcome, otherwise, it's on my list, but for later :)13:50
didrocksogra_: I think it would be better to be compiled anyway, so rewriting in go or such (but then, there is the issue for uncompiled snaps like python and so…)13:50
zygajdstrand: https://bugzilla.suse.com/show_bug.cgi?id=986050#c113:51
zygajdstrand: and https://bugzilla.suse.com/show_bug.cgi?id=986050#c213:51
kyrofaogra_, didrocks doesn't like to be limited by dash13:51
didrocksI'm used to dash, did a big part of porting years ago of our scripts to dash :)13:52
didrocksbut here, I just wanted something which works quickly :p13:52
ogra_you could just put bash into stage packages by default ... whats another 5MB if your hello-world already needs 100MB of deps :P13:53
didrocksogra_: better to do just use dash, it's not user visible anyway13:55
ogra_i was joking ;)13:55
oSoMoNdpm, hey, I’m experimenting with snapcraft and webbrowser-app, and https://github.com/dplanella/qt5conf/blob/master/qt5-launch#L45 appears to be preventing the app from starting, making XDG_DATA_HOME a non-writable location sounds wrong14:15
ogra_oSoMoN, try didrocks' launcher instead14:16
oSoMoNogra_, is that the desktop/qt5 one?14:17
oSoMoNcool, will try that one14:17
sergiusenskyrofa mind quickly fixing https://github.com/snapcore/snapcraft/pull/622 ?14:20
mupPR snapcraft#622: Hard-link local sources instead of symlinking them <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/622>14:20
kyrofasergiusens, yeah I'm working on it as we speak14:20
sergiusenskyrofa oh, nice :-)14:20
mupPR snapcraft#630 closed: Report a nice error when registering too fast <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/630>14:21
kyrofasergiusens, super annoying... can't get it to happen locally :P14:23
gouchi_using x11, opengl, unity interfaces shouldn't make XOpenIM to failed ?14:33
kyrofaelopio, are you around?14:33
kyrofasergiusens, look at this trace: http://pastebin.ubuntu.com/18546556/14:36
kyrofasergiusens, makedirs(..., exist_ok=True) -> "bam, that already exists!" -> death14:36
kyrofasergiusens, this test isn't failing locally and I can't duplicate that behavior. Do you see anything I'm missing? Ensure failure is here: https://travis-ci.org/snapcore/snapcraft/jobs/14246762114:37
sergiusenskyrofa sure, give me a sec14:37
kyrofaIt's a symlink, not a dir, but that still works fine locally. I even tried messing with the permissions in case that threw it off, but it works locally anyway. Do we know which python3 version is on the job machines?14:38
kyrofaOh... wait...14:39
kyrofasergiusens, nevermind, I'm stupid14:39
kyrofasergiusens, that was way too hard14:40
sergiusenskyrofa what was it14:40
* didrocks is interested, I looked at the os code, and I see 2 protections against what you saw :)14:40
elopiokyrofa: hello14:41
elopiohow can I help you?14:41
kyrofasergiusens, didrocks it's a test artifact. The integration test runs on a partially build snap, to test that the change works on an old tree. But the parts/foo/src symlink is absolute, not relative, so it's a broken one in jenkins (doh!)14:42
kyrofaelopio, nevermind, sorry!14:42
gouchi_I will be glad if somebody can test my snap package to see if I am doing something wrong14:42
gouchi_here is the source : https://framadrop.org/r/vEimxEXgr2#SHwKK3R8qJ12soOacXboC+PbzNPvb/xPoijFaq7+e5s=14:42
didrockskyrofa: ahah, that explains! :)14:43
kyrofadidrocks, super confusing :P14:44
gouchi_for now it failed here https://github.com/libretro/RetroArch/blob/master/gfx/common/x11_common.c#L338 using x11, opengl, unity7 interface14:44
zygagouchi_: I can look but not tonight, I have plenty of things to finish14:44
sergiusenskyrofa heh, that's what you get for mocking ;-)14:45
kyrofasergiusens, nothing is mocked!14:45
kyrofasergiusens, it's just a bit of an odd test is all. I made the mistake of committing exactly what snapcraft gave me (an absolute symlink) without thinking about it14:45
gouchi_zyga: thank you no problem I can't find why it stops14:45
sergiusenskyrofa preparing the testbed/fixture would require you to make it absolute again though, right?14:46
kyrofasergiusens, nah, it just copies the entire snap directory that contains the symlink already made14:48
didrockskyrofa: indeed :)14:51
bpeakMan, I really want chromium on snappy14:51
ogra_you can ... but only in devmode14:54
jdstrandroadmr: hey, at some time whenever it is convenient, can you pull r692. this is for puritine click and not urgent (next roll out is fine)14:58
roadmrjdstrand: sure thing, I'll put it in the pipeline!14:59
seb128gouchi_,  ogra_ was having the same XOpenIM issue, unsure if he solved it15:01
ogra_well, i dont remember anymore15:01
jdstrandroadmr: thanks!15:01
ogra_(i remember i had it ...)15:02
ogra_ah, no, i didnt .... that was when i tried to package servo15:02
gouchiogra_: I checked snappy playpen packages and only mpv is using XOpenIM15:08
gouchiogra_: but I doubt it using x11 code as mpv snap package is using only opengl interface15:08
seb128gouchi, does it work in devmode?15:10
kyrofazyga, how do you test snap-confine, since it conflicts with u-c-l?15:11
gouchiseb128: nope15:14
roadmrhey folks, what's a good/recommended way to get or build a snappy-only image for a rpi2? i.e. not xenial-with-snapd but an "all-snap" image?15:14
zygakyrofa: you install ubuntu-core-launcher too :)15:15
zygakyrofa: snap-confine has two binary packages15:16
zygakyrofa: one of them is snap-confine, the other is ubuntu-core-laucher15:16
kyrofazyga, argh, missed that15:16
kyrofaThanks :)15:16
seb128gouchi, I can try having a look if you want15:16
gouchiseb128: I will be glad to, thank you very much15:16
sergiusenselopio the current interface we have for pushing files makes me sad15:22
roadmrzyga: I'm sure you'll know how to get an all-snap image for rpi2 :)15:25
zygaroadmr: yes, get a delorean and come back later ;) in all honesty I didn't work with all snap images for a while, I bet you can still get an image somewhere but I wasn't in the loop around that lately15:26
roadmrzyga: thanks! hehe no problem, I'll walk to the future (also known as : waiting) X-)15:27
zygaroadmr: I think ogra_ might know more15:27
zygaroadmr: note that slangasek works on ubuntu-image so he might also know some useful thigns I dont15:27
roadmrcool! yay, fwiw I'm happy to assemble my own image if pointed to tools/docs15:28
ogra_well, the current images ... even if you buuild them newly .... are months outdated15:28
ogra_i doubt any recent snaps would even work15:28
roadmrogra_: oh bummer, and there's no way to build one with recent components I reckon?15:28
ogra_afaik mvo has a hacked u-d-f that might get you something working that would persist for a few days until you need to break it again15:29
roadmrhaha :)15:29
ogra_roadmr, the recent components are not in the store, ubuntu-core is ages old15:29
roadmrogra_: so for all real purposes I'm better off just using xenial with snapd?15:29
ogra_we would need to update it first, but there are some blockers15:29
ogra_just use a classic install15:30
roadmrok... I can do that, no problem :)15:30
roadmrthanks ogra_ :)15:32
elopiosergiusens: do you mean, the store clients?15:47
sergiusenselopio _upload.py specifically15:51
elopiosergiusens: I agree. It doesn't match the others, and passing the client as an argument is a little spaghetti15:52
elopiosadly, I ended up with no good refactoring in mind for it. Do you have one?15:52
sergiusenselopio I think I am going to spend the rest of the week on a little refactor15:52
ogra_seb128, didrocks, soo ... it seems icons are only broken if i put any icon themes beyond hicolor into stage-packages (i had light-themes in there when it broke) ... ii guess snapcraft omitting all maintainer scripts breaks the icon stuff somehow15:55
seb128that's on possibility15:56
seb128one other is that the other themes use svg files15:56
seb128which you don't have proper support for loading15:56
ogra_i do15:56
didrocksdid you get the same behavior with the desktop launcher?15:56
ogra_i have all pixbuf loaders and the loader cache is updated15:56
seb128is your snap available somewhere so I can have a look?15:56
seb128but java maybe doesn't use gdkpixbuf15:57
seb128like we tried with a qt app and were missing libqsvg or something15:57
ogra_seb128, it works if i launch th jar directly15:57
seb128from outside the snap?15:57
ogra_just using the java  on my desktop i get proper icons15:57
seb128anyway, is your snapcraft.yaml public?15:58
seb128so I can have a look15:58
ogra_not the current version.. but it has only:15:58
ogra_    stage-packages:15:58
ogra_      - libglib2.0-015:58
ogra_      - hicolor-icon-theme15:58
ogra_      - openjdk-8-jdk15:58
ogra_(i dont want *any* integration with the host here .... the app should ship and run everything standalone15:58
ogra_seb128, https://github.com/ogra1/jtiledownloader ... thats the one not usin any themeing16:01
seb128ogra_, k16:01
=== chihchun is now known as chihchun_afk
seb128ogra_, you should try the desktop/gtk3 part launcher, it's a big hammer but if it works it would tell you there is some magic from the wrapper you can borrow16:02
seb128would easily rule out a class of issues16:03
ogra_i borrowed most of it already ... but i'm now pretty sure it is the postinst files breaking the stock icons since as soon as i drop the ubuntu themes and all theme hackery it all works just fine16:03
ogra_also the XDG_DATA_HOME setting the launcher uses breaks awt/swing16:04
MichaelTunnellniemeyer: here is why I've been asked so many random questions lately, https://youtu.be/0ApRUndiXKU16:04
ogra_the app cant start16:04
joc_sergiusens: hi, have you thought any more about the python3 PR? are you still nervous about test coverage?16:08
kyrofasergiusens, I think snapcraft#622 should be good now16:25
mupPR snapcraft#622: Hard-link local sources instead of symlinking them <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/622>16:25
oSoMoNwhen creating a snap of webbrowser-app (which contains oxide), I’m seeing this:16:33
oSoMoNRemoving suid/guid from /build/webbrowser-app/snap/parts/webbrowser-app/install/usr/lib/x86_64-linux-gnu/oxide-qt/chrome-sandbox16:33
oSoMoNthis prevents oxide from functioning correctly16:33
oSoMoNhow can I prevent snapcraft from tempering with the suid/guid of the chrome sandbox?16:33
sergiusensjoc_ I think it is ok16:34
sergiusenselopio mind taking a look at the python --root pr?16:34
elopiosergiusens: I'm out for swimming lessons. I'll do that on the way back.16:35
sergiusenselopio thanks16:36
sergiusensoSoMoN suid and guid would be blocked by apparmor and the review tools16:37
sergiusensjdstrand any alternatives there ^?16:37
oSoMoNsergiusens, why remove the suid and guid then, if apparmor would block them anyway?16:38
sergiusensoSoMoN it was the nice thing to do before --devmode ever existed16:38
oSoMoNsergiusens, ah ok, it predated --devmode, that makes sense16:38
sergiusensoSoMoN by a year almost16:39
oSoMoNsergiusens, shouldn’t it be deprecated now? sounds like --devmode is much better suited16:39
sergiusensoSoMoN we still need to be able to auto remove the suidness somehow for people with confinement strict16:40
sergiusensoSoMoN I suppose we can tie this up to the `confinement` flag; mind logging a bug?16:40
oSoMoNsergiusens, doing right away, against which project?16:40
sergiusensoSoMoN snapcraft16:43
oSoMoNsergiusens, against the project itself, or against the source package? it seems there are existing bug reports targetting both16:44
sergiusensoSoMoN project, we always ask people to log against the project in the release ANN :-)16:44
oSoMoNsergiusens, https://bugs.launchpad.net/snapcraft/+bug/159923416:48
mupBug #1599234: should not remove suid/guid from binaries when confinement is devmode <Snapcraft:New> <https://launchpad.net/bugs/1599234>16:48
* zyga -> supper, then more suse hacking16:49
tsimonq2sergiusens: when's the next planned Snapcraft release?16:49
mupBug #1586581 changed: CAP_SYS_ADMIN interface for unconfined snaps <snapd-interface> <Snappy:Invalid> <https://launchpad.net/bugs/1586581>17:00
sergiusenstsimonq2 today17:12
niemeyerMichaelTunnell: Nice!17:12
tsimonq2sergiusens: oh ok17:13
sergiusenstsimonq2 why?17:13
tsimonq2sergiusens: it really would have been great to land my latest PR before 2.12 but it's no big deal :)17:15
sergiusenstsimonq2 2.12 has been out for a while though ;-)17:15
sergiusenswe are releasing 2.13 ;-)17:15
tsimonq2I meant 2.1317:16
tsimonq2sergiusens: is it already released?17:17
tsimonq2sergiusens: I mean technically, are the git tags set and everything?17:18
tsimonq2sergiusens: if not, any chance you have a minute to help me finish my PR?17:19
tsimonq2PR #61917:24
mupPR snapcraft#619: Add source-checksum option for tar and zip sources (bug 1585913) <Created by tsimonq2> <https://github.com/snapcore/snapcraft/pull/619>17:25
tsimonq2there we are17:25
jdstrandoSoMoN, sergiusens: I commented in the bug17:26
sergiusenstsimonq2 I am in the process of reviewing the last PRs right now though17:27
sergiusenstsimonq2 take your time to finish it, nothing wrong with going into the next release ;-)17:28
sergiusensjdstrand thanks17:28
tsimonq2alright sergiusens, the only things left are some annoying test failures, test failures that could be easily solved today17:29
sergiusenstsimonq2 test failures are the hardest part of the problem; just look at kyrofa, he spent all morning looking into why his PR failed17:29
oSoMoNjdstrand, I’ve seen that, thanks!17:30
kyrofatsimonq2, indeed, look at me17:30
* kyrofa gives his best blue steel look17:30
tsimonq2\o/ kyrofa17:30
tsimonq2sergiusens: how close are you from making 2.13 final? basically, how much time do I have left to fix things if I really want my PR landed in 2.13? :)17:32
kyrofatsimonq2, don't stress about that. If it gets in, it gets in. We try to do weekly releases17:34
tsimonq2kyrofa: alright :)17:35
pachulosergiusens: one question, when do you plan to update the telegram-snap in the store? I've tried to create the snap with telegram-desktop 0.9.56 and it worked ok17:37
kyrofatsimonq2, just focus on quality. Make sure those tests look good17:38
sergiusenspachulo I should just do it I guess, I wanted to fix some input method problems first though17:38
tsimonq2kyrofa: I'm working hard :)17:40
tsimonq2kyrofa: I'm not in a rush, I won't do it halfway, like I said, it would just be *nice* for source-checksum to land in 2.13, so I'm going to work towards that goal :)17:41
kyrofatsimonq2, excellent17:42
pachulosergiusens: i noticed problems when trying to write letters with accents, but thought it was a problem with the application, not the snap versio of it: https://github.com/telegramdesktop/tdesktop/issues/136017:42
sergiusensoh, look at that, niemeyer too ^17:45
sergiusenspachulo thanks, I will see if this is indeed the problem I see17:46
niemeyerI don't have the problem out of a snap, FWIW17:46
niemeyersergiusens: Also, those particular accents work.. probably something else17:46
sergiusensniemeyer yeah, most likely missing something input method related.17:49
mupPR snapcraft#614 closed: python3: use --root instead of --target <Created by jocave> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/614>18:12
pachulomaybe is a silly question, but after checking the documentation could not find an answer: Was trying to create a MAME snap using the "make" plugin, but as it seems that there is no "install" target in the MAME Makefile it fails to build because the snapcraft make plugin tries to install it; how should I proceed?18:20
jrwrenpachulo: thanks for re-asking. I'm watching to see the answer.18:21
kyrofasergiusens, snapcraft's python2 plugin can't seem to install this from pip: https://pypi.python.org/pypi/pymacaroons-pynacl/ . Is it because wheels are disabled? Think the --root PR will fix that?18:26
kyrofasergiusens, oh... that was only python318:30
sergiusenskyrofa it won't18:41
sergiusenskyrofa global-options disable wheeling18:41
kyrofasergiusens, hmm... how might one get around this, then?18:41
kyrofapachulo, do you know off the top of your head what needs to be placed into the snap after the part is made?18:44
kyrofapachulo, i.e. can you safely separate the compiled binaries from the sources?18:45
kyrofapachulo, that's typically what an install rule does-- the upstream project knows what it needs to run and where things need to go18:45
kyrofapachulo, if your upstream project didn't supply that, then you'll need to create a local plugin that simply calls "make" and then copies stuff in explicitly rather than relying on an install rule18:46
sergiusenskyrofa elopio sup18:46
kyrofasergiusens, on my way18:46
pachulokyrofa: not really, but I was planning on looking at the files installed by the repostory package to see what I need to copy18:58
mupBug #1598304 changed: systemd services created by snappy breaks etckeeper <etckeeper:New> <Snappy:Invalid> <https://launchpad.net/bugs/1598304>18:58
pachulokyrofa: I will try the local plugin approach, thanks for your suggestion!18:58
kyrofapachulo, sure!18:59
tsimonq2*sigh* I have been racking my brain on this for too long, it's time to seek help, I've tweaked a lot of different things to no avail...19:02
tsimonq2(in snapcraft#619 )19:02
mupPR snapcraft#619: Add source-checksum option for tar and zip sources (bug 1585913) <Created by tsimonq2> <https://github.com/snapcore/snapcraft/pull/619>19:02
tsimonq2I'll push my changes now, but I'm getting thrown this error in the unit tests:19:02
tsimonq2TypeError: __init__() missing 1 required positional argument: 'source_checksum'19:02
tsimonq2if anybody is around to give me a hand and just point me in the right direction, that would be awesome, I know kyrofa was helping me the other day so I don't know if he wishes to help again :)19:05
kyrofaSure tsimonq2, give me a minute19:06
tsimonq2thanks kyrofa :)19:17
kyrofaAlright tsimonq2 I'm taking a look at the branch now19:37
mupPR snapcraft#631 opened: Fix the store integration tests <Created by elopio> <https://github.com/snapcore/snapcraft/pull/631>19:39
tsimonq2cool thanks kyrofa19:40
netphreakhi, guys!19:44
kyrofatsimonq2, alrighty19:44
kyrofatsimonq2, take a look at the Base class19:44
kyrofatsimonq2, specifically its __init__19:44
kyrofatsimonq2, do you know which parameters are required, and which are optional?19:45
kyrofaHey netphreak :)19:45
netphreakJust uploaded my first snap to the store in the Edge channel, and get the following error from automated review: "package contains external symlinks: usr/lib/jvm/java-8-openjdk-armhf/jre/lib/security/cacerts lint-snap-v2_external_symlinks"19:45
* tsimonq2 pops open the source19:45
netphreakThe snap uses the java/jdk plugin.19:45
tsimonq2kyrofa: in snapcraft/internal/sources.py ?19:45
kyrofatsimonq2, indeed19:45
tsimonq2kyrofa: alright19:46
kyrofanetphreak, can you pastebin the output of ls -l prime/usr/lib/jvm/java-8-openjdk-armhf/jre/lib/security/19:46
tsimonq2kyrofa: what's required is source19:46
tsimonq2the rest are optional depending on what you are trying to do19:47
netphreak: No such file or directory19:47
kyrofatsimonq2, how can you tell? What about source_dir?19:47
kyrofanetphreak, I made the assumption you still had the built tree there. Do you still have your prime directory lying around?19:48
tsimonq2kyrofa: I just guessed from what I know about snapcraft.yaml files, there's not much in the Base class that would tell me a lot about that19:48
tsimonq2except that source_dir kind of stands out19:48
kyrofatsimonq2, sure there is. Note that source and source_dir aren't immediately followed by =19:48
tsimonq2(in the init)19:49
tsimonq2well yes19:49
kyrofatsimonq2, the other parameters all have default values19:49
kyrofatsimonq2, which means they aren't required19:49
tsimonq2kyrofa: well source-checksum isn't required either19:49
kyrofatsimonq2, indeed. And it shouldn't19:49
kyrofatsimonq2, check out Zip's __init__ or Tar's __init__19:49
netphreaki still have the snap build lying around.. but can't see no prime dir19:50
tsimonq2kyrofa: oh, so it seems to be required?19:50
kyrofanetphreak, ah, your snapcraft is a little old then, but no matter: run `ls -l snap/usr/lib/jvm/java-8-openjdk-armhf/jre/lib/security/`19:50
netphreakahh.. sorry.. I do have a prime dir.. moment.19:50
kyrofatsimonq2, indeed19:50
kyrofatsimonq2, but the tests don't pass it in19:50
kyrofatsimonq2, so you get an error saying a param is missing19:50
kyrofatsimonq2, so: is it required or not? If so, the tests need to be updated. If not, the __init__s need to be updated19:51
tsimonq2let me fix and do a test build locally19:51
tsimonq2kyrofa: it's optional, I'll update19:51
kyrofatsimonq2, very good19:51
kyrofanetphreak, yikes, yeah. Hmm... it's too bad I'm so unfamiliar with java, I'm not sure what's causing that19:52
tsimonq2kyrofa: I leanred something new today, I've never worked with *that* complex of Python :)19:52
kyrofatsimonq2, the day is a waste without learning something new!19:53
netphreakWell, i guess everyone using this plugin will have the same problem as i have run into..19:53
kyrofanetphreak, does /etc/ssl/certs/java/cacerts actually exist?19:53
tsimonq2kyrofa: yep! :D19:53
tsimonq2kyrofa: http://paste.ubuntu.com/18577713/ - I'll start looking around for something obvious19:54
tsimonq2oh hm19:54
kyrofatsimonq2, self.source_checksum19:55
netphreaknope.. seems the folder "/etc/ssl/certs/java/" is empty19:55
kyrofatsimonq2, you might consider running ./runtests.sh static19:55
tsimonq2kyrofa: aha, that's right, because it's not called in the constructor19:55
kyrofanetphreak, default-jdk might include that19:56
netphreakHmm.. I think the plugin includes a default java in the final snap...19:57
netphreakwhy this references a file that does not exist is a bit weird..19:58
kyrofanetphreak, because it's pulled in by openjdk-8-jre-headless19:59
kyrofanetphreak, which apparently contains an absolute symlink19:59
tsimonq2kyrofa: I think it would be good to ensure integration test coverage with source-checksum, right?20:00
kyrofaBut these are stage-packages, so they don't go where they think they're going20:00
kyrofatsimonq2, agreed20:00
tsimonq2kyrofa: the unit tests all passed (\o/) so I'll make sure integration tests pass, I'll whip that up, then I should be done20:00
kyrofatsimonq2, nice job :)20:01
tsimonq2kyrofa: \o/ thanks :)20:01
tsimonq2kyrofa: and thank you so much for your help20:02
kyrofatsimonq2, of course20:02
tsimonq2kyrofa: build is failing? huh? https://github.com/snapcore/snapcraft20:12
kyrofatsimonq2, I don't see it failing...20:13
kyrofaelopio, wait... was this actually successful then?
tsimonq2kyrofa: that ^20:16
mupPR snapd#1490 opened: overlord: implement &Retry{After: duration} support for handlers <Created by pedronis> <https://github.com/snapcore/snapd/pull/1490>20:18
netphreakkyrofa: I just created the following bugreport: https://bugs.launchpad.net/ubuntu/+source/snapcraft/+bug/159928120:18
mupBug #1599281: Snap upload automated review error <snapcraft (Ubuntu):New> <https://launchpad.net/bugs/1599281>20:18
kyrofatsimonq2, ah, looks like test ran into a timing issue20:19
kyrofatsimonq2, I requested a new run20:19
tsimonq2kyrofa: oh alright, I just thought that I should mention it20:19
kyrofanetphreak, very good, thank you!20:19
tsimonq2kyrofa: \o/ yes! finally! build passed!20:20
elopiokyrofa: http://paste.ubuntu.com/18579746/ did you touch something on the parser?20:24
kyrofaelopio, no20:24
kyrofaelopio, that's totally external to the snapcraft lifecycle, right?20:25
elopiokyrofa: yes. Maybe a timeout there, but not related to your change, so the execution is ok.20:26
kyrofaelopio, alright. I guess I need to update the branch anyway, so running again! Stop landing stuff, sheesh20:26
tsimonq2kyrofa: you think md5, sha256, and sha512 are good enough to support for checksumming or is there another format that people use that comes to mind?20:36
kyrofatsimonq2, did you see sabdfl's comment on bug #1585913 ?20:37
mupBug #1585913: Snapcraft should allow the user to verify downloaded files with a checksum <Snapcraft:In Progress by tsimonq2> <https://launchpad.net/bugs/1585913>20:37
tsimonq2kyrofa: no, I didn't catch that20:38
tsimonq2kyrofa: hmm, so is he saying to only support SHA3-384?20:39
kyrofatsimonq2, honestly I'm not sure. Most often I see md520:39
kyrofatsimonq2, we probably want to support a range of things there20:39
tsimonq2kyrofa: yeah, which is why I'm asking20:40
kyrofatsimonq2, so make sure 384 is supported20:40
kyrofaBut I'd include the others as well20:40
tsimonq2kyrofa: alright20:40
kyrofatsimonq2, make sure you have tests for each20:40
tsimonq2kyrofa: yeah currently, if my code works, we have md5, sha256, and sha51220:40
kyrofatsimonq2, excellent20:40
tsimonq2kyrofa: I'm running my integration tests for those now, so I'm not entirely sure if it even works yet, lol20:41
tsimonq2kyrofa: would you suggest I respond to Mark in the bug report asking for clarification?20:42
kyrofatsimonq2, sure, he won't bite.20:43
tsimonq2great :)20:43
ogra_tsimonq2, dont wear a tasty tempura crust though ...20:50
tsimonq2ogra_: huh?20:50
ogra_then he might bite ;)20:51
tsimonq2oh hahahaha20:52
Croephaso when I use stage-packages, is it actually downloading each time I run it? it looks like it is, is there a way to tell it to cache the downloads?21:00
kyrofaCroepha, they're pulled along with everything else in the pull step. Once the pull step has completed just don't run it again unless you need to add something to it21:08
kyrofaCroepha, for example, you can clean only back to the build step (snapcraft clean -s build) which will leave pull alone21:09
kyrofaDangit ogra_ I'm hungry21:09
Croephakyrofa: ok, got it, thanks :)21:09
sergiusenselopio so far https://github.com/snapcore/snapcraft/compare/master...sergiusens:refactor/push21:13
sergiusensstill ways to go21:13
sergiusensi will bbl21:13
tsimonq2bug 158591321:19
mupBug #1585913: Snapcraft should allow the user to verify downloaded files with a checksum <Snapcraft:In Progress by tsimonq2> <https://launchpad.net/bugs/1585913>21:19
tsimonq2kyrofa: whoever is hacking on that bot should know that https://pad.lv/1585913 = https://launchpad.net/bugs/158591321:20
robert_ancellev, who's in charge of lp:rnrserver now?21:23
thomisergiusens: did the way we load & run snapcraft plugins change between 2.11 and 2.12? I'm sure I used to be able to set a breakpoint in my plugin and have it work in 2.11. In 2.12 they seem to be ignored somehow?21:28
tsimonq2kyrofa: what does snapcraft/internal/sources.py:141:1: C901 'FileBase.check_checksum' is too complex (10) mean and how do I fix it?21:30
tsimonq2kyrofa: I'm assuming it means I have to dumb down my code?21:31
thomitsimonq2: I believe it's the cyclomatic complexity check built into flake8  - https://en.wikipedia.org/wiki/Cyclomatic_complexity might help21:33
thomitsimonq2: it's not really about 'dumbing down' your code, but rather limiting the complexity of any one function.21:34
tsimonq2thomi: I see21:34
tsimonq2there's a way I can make it simpler21:34
tsimonq2so I'll do that :)21:34
Kamilionyeah, if you're seeing that; you need to reorganize slightly; Ran into that while I was working on Customizer.21:36
Kamiliongood to see you on changelogs though, tsimonq2. :)21:37
tsimonq2hey it's Kamilion! \o/21:37
Kamilionfunnily enough, while I was doing work on checksumming files too.21:37
tsimonq2Kamilion: how'd you find me? lol21:37
tsimonq2hah yeah21:37
Kamilionw'dya mean? I've been here.21:37
tsimonq2Kamilion: I haven't seen you in ages :)21:38
Kamilionno longer in a release crunch21:38
Kamilionwell, 16.04's out now XD21:38
Kamilioni don't have to do another LTS job till 201821:38
Kamilionand most of my papercuts were fixed between 15.04 and 16.04 WRT xen, openvswitch, and ceph.21:39
KamilionStill can't get snapd to work from a livecd though21:39
tsimonq2Kamilion: so you here to submit code to Snapcraft, snap some things, support, what? just curious :)21:39
tsimonq2oh :(21:39
Kamilionas you know, I build appliance images21:40
Kamilionso everything was already running from the iso squashfs21:40
Kamilionnot sure if it's just unhappy about the rootfs being squash too or something else casper is tweaking when it boots the system.21:41
Kamilionthat reminds me, i should say hi to walter when I get a chance21:42
tsimonq2Kamilion: *shrug* I don't know anything about that sort of thing :)21:42
tsimonq2I thought he's online?21:42
Kamilionprobably is; I'm just in like 180 channels21:42
tsimonq2oh lol21:43
Kamilionand busy with several other projects as well21:43
Kamilionbut i saw you talking when I was scrolling through the list and thought I'd say hi21:43
tsimonq2well I hope things are going well :)21:44
Kamilionif they weren't, I wouldn't be around ;)21:47
mwhudsonslangasek: hey, just pushed a patch from zyga to http://anonscm.debian.org/cgit/collab-maint/snap-confine.git/log/?h=debian21:51
mwhudsonslangasek: otherwise i think that package is ready to go, what do you think?21:51
evrobert_ancell: OLS owns rnr as a team - what’s up?22:22
robert_ancellev, was wondering if there's a plan regarding snap reviews. I thought that that would use the click API, but looking into it I'm not sure that's the case22:23
robert_ancellbut 1574586 for reference22:23
robert_ancellbug 157458622:23
mupBug #1574586: Not able to rate/review snaps <Snappy:New> <gnome-software (Ubuntu):Triaged> <https://launchpad.net/bugs/1574586>22:23
evrobert_ancell: I don’t recall snap reviews coming up in vancouver or santa cruz. Has this found its way onto the roadmap?22:24
robert_ancellev, no, I don't think it's on the roadmap either so was wondering if there's been any decisions either way.22:24
robert_ancellor if it should 'just work' with the click API.22:24
evI don’t think it’d just work, but we can do a back of the envelope calculation if you need one22:26
robert_ancellev, that would be handy, thanks22:28
evsure thing22:30

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