/srv/irclogs.ubuntu.com/2017/09/22/#snappy.txt

mupPR snapcraft#1558 closed: tests: add fake pip fixture <Created by kyrofa> <Closed by kyrofa> <https://github.com/snapcore/snapcraft/pull/1558>00:17
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
=== JoshStrobl|zzz is now known as JoshStrobl
=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
mupPR snapd#3947 closed: cmd/snap-repair: fix tests when running as root <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3947>06:18
* zyga-ubuntu has the worst friday possible06:22
zyga-ubuntuI removed my snapd tree, including all of git history last night06:22
zyga-ubuntuso plan for today, is to ensure this doesn't happen again06:25
mupPR snapcraft#1566 opened: recording: record build-snaps installed during the pull <Created by elopio> <https://github.com/snapcore/snapcraft/pull/1566>06:27
zyga-ubuntukyrofa: hey06:28
zyga-ubuntukyrofa: did you use that pastebin I gave you last night?06:28
cjwatsondaily automated backups of everything - it's the only way06:35
zyga-ubuntucjwatson: deja dup backups got corrupted06:39
zyga-ubuntucjwatson: I'll use something else as this is not the first time backup was not working, just this time was more damaging06:40
cjwatsonYeah, I used deja-dup for a little while and then decided it was far too complex06:40
cjwatsonI use rsbackup now which is basically just glorified rsync --link-dest; much less to go wrong06:41
zyga-ubuntuthank you, I'll check that out06:42
mvozyga-ubuntu: could oyu please check 3953?06:49
zyga-ubuntumvo: sure06:49
zyga-ubuntumvo: why not go all the way and just do this unconditionally?06:50
mvozyga-ubuntu: not sure, to minimize risk maybe?06:54
mvozyga-ubuntu: I'm fine with all-the-way for 2.2906:54
zyga-ubuntumvo: sounds good to me06:54
mvothanks06:55
zyga-ubuntudone06:56
mupPR snapd#3953 closed: snap-confine: fix base snaps on core <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3953>07:00
mvothanks zyga-ubuntu07:00
zyga-ubuntuthank you for doing this :)07:01
=== ikey is now known as ikey|zzz
mvoPharaoh_Atem: is 2.28~rc3 doing ok on fedora or anything needs fixing before 2.28-final?07:04
kalikianazyga-ubuntu: backups of git? in my word, that's what you get by pushing ;-)07:09
kalikianagood morning, snappy world07:10
* kalikiana no more headache for now, just caughing, may the health be with me today07:12
oSoMoNMy latest upload to the store of chromium failed the automated review because the chrome sandbox executable has the setuid bit set, but that used to be allowed previously (not sure if it was an exception for that snap only, or something else). Can anyone from the store/review team help?07:12
oSoMoN(oh, and good morning everyone, by the way!)07:12
mvozyga-ubuntu: http://paste.ubuntu.com/25590963/ <- snapd needs to create that dir in the re-exec case or fail gracefully, right now re-exec with edge is briken07:24
mvobroken07:24
zyga-ubuntumvo: loking07:25
zyga-ubuntuoh07:26
zyga-ubuntugood point07:26
zyga-ubuntumvo: how did tests not catch that?07:26
zyga-ubuntumvo: ah, I think I know07:26
zyga-ubuntumvo: ok I have a patch07:33
mupPR snapd#3955 opened: dirs,interfaces: create snap-confine.d on demand when re-executing <Created by zyga> <https://github.com/snapcore/snapd/pull/3955>07:39
zyga-ubuntumvo: https://github.com/snapcore/snapd/pull/395507:39
mupPR #3955: dirs,interfaces: create snap-confine.d on demand when re-executing <Created by zyga> <https://github.com/snapcore/snapd/pull/3955>07:39
zyga-ubuntuChipaca: can you please look at https://github.com/snapcore/snapd/pull/3955/files08:03
mupPR #3955: dirs,interfaces: create snap-confine.d on demand when re-executing <Created by zyga> <https://github.com/snapcore/snapd/pull/3955>08:03
Chipacaaye aye cap'n08:04
zyga-ubuntuthank you08:04
zyga-ubuntuChipaca: I lost my snapd tree with 2 weeks of code in it08:04
zyga-ubuntunot a great day08:04
Chipacazyga-ubuntu: did you look under the bed08:04
zyga-ubuntuChipaca: I wanted to stay there today actually08:05
Chipacazyga-ubuntu: under the bed?08:07
zyga-ubuntuin it08:08
Chipacazyga-ubuntu: my dog hides under the bed during thunderstorms08:08
Chipacashe's brave like that08:09
zyga-ubuntuCI is slow today?08:33
* Chipaca quotes poetry at niemeyer 08:35
sparkiegeekChipaca: Vogon poetry?08:45
Chipacasparkiegeek: unix poetry08:45
Chipacasparkiegeek: tomeito/tomahto08:45
mupPR snapcraft#1567 opened: recording: record the snaps installed in the machine <Created by elopio> <https://github.com/snapcore/snapcraft/pull/1567>08:48
mupPR snapd#3956 opened: snap-confine: bind mount /usr/lib/snapd relative to snap-confine <Created by mvo5> <https://github.com/snapcore/snapd/pull/3956>08:52
mvozyga-ubuntu: ^- something for you :) should be easy but was a bit tricky to figure out08:53
zyga-ubuntulooking08:59
zyga-ubuntumvo: interesting,09:03
zyga-ubuntumvo: I suspect we can do away with _some_ of those snprintfs but it looks good09:03
mupPR snapd#3955 closed: dirs,interfaces: create snap-confine.d on demand when re-executing <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3955>09:04
zyga-ubuntumvo: done09:10
pedronismvo: zyga-ubuntu: should 't we change the name of sc_mount_config.on_classic_distro to something else , now that is true on core also with base snaps09:28
zyga-ubuntupedronis: yes, I think it will become "use_classic_confinement" or similar09:34
zyga-ubuntupedronis: as everything else will be identical09:34
zyga-ubuntuwe may not need that variable afte rall09:34
pedronisclassic_confinement is a bit strange given that is not confiment: classic09:38
zyga-ubuntuin which way?09:39
zyga-ubuntuI mean that the only remaning boolean variable is if we need a mount namespace or not09:40
pedronisI'm saying that using classic in the name of a bool that we use both on classic and core is strange, that classic confiment means too many things already09:43
zyga-ubuntupedronis: right, I think we are in agreement, it is not done yet but eventually there will be just one flag "mount namespace needed", we need to finish the transition and rename things09:44
ackkniemeyer, hi, left a question on https://github.com/snapcore/snapd/pull/3916 about validateion10:35
mupPR #3916: snap,wrappers: add support for socket activation <Created by albertodonato> <https://github.com/snapcore/snapd/pull/3916>10:35
ackk*validation10:35
dokosnapcraft autopkg test failure: https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-artful/artful/amd64/s/snapcraft/20170922_010609_57ea7@/log.gz10:44
dokooutput: {{{b'/snap/godd/x1/bin/godd: relocation error: /snap/godd/x1/lib/x86_64-linux-gnu/libpthread.so.0: symbol __mmap, version GLIBC_PRIVATE not defined in file libc.so.6 with link time reference\n'}}}10:45
dokobut it's built with glibc-2.26 ...10:46
ogra_godd is also a properly confined snap (i have only seen that with --classic snaps on artful yet) which makes it even weirder ...10:48
ogra_mvo, ^^^10:48
ogra_*classic snaps that get built *on* and *against* artful10:48
mvoogra_: looking10:49
mupPR snapd#3934 closed: snap-repair: implement `snap-repair {list,show}` <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3934>11:35
mupPR snapd#3928 closed: interfaces/system-observe: allow clients to enumerate DBus connection names <Created by jdstrand> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3928>11:36
ackkogra_, mvo does snapd already have code to validate URIs (hostname:port, ip:port, ipv6:port, port and so on)?11:41
* ogra_ doesnt know ... i think we used to have code to check for occupied ports when we used to support the "ports:" keyword in snap.yaml during 15.04, but i dont know if that code survived11:49
ackkogra_, now that I think about it, for the listen-stream case I think I could just validate that it's a port, there's no real point in specifying an address11:52
ackkwell, maybe localhost11:52
ogra_ackk, well, what happens if i install a snap using port 80 that already has an apache deb installed ?11:52
ackkogra_, that's a separate issue though11:53
ogra_right, but there needs to be some check so my snap doesnt madly try to respawn11:53
kyrofazyga-ubuntu, I did, thank you! It was tremendously helpful. I tried to thank you last night, but you were gone :(12:07
sergiusenskyrofa up so early?12:08
kyrofasergiusens, I've been a little more popular than I was anticipating... haven't had time to put that lightning talk together! This is my solution :P12:08
sergiusenselopio mind looking at the artful issues with snapcraft (mentioned on ubuntu-release and here by doko)? I'll be running errands all day today12:08
zyga-ubuntukyrofa: I was upset because while doing that I removed my whole snapd tree with unmerged code12:18
zyga-ubuntukyrofa: I had a symlink12:18
kyrofazyga-ubuntu, I'm sorry about that! Were you able to recover from backup?12:19
zyga-ubuntukyrofa: and I wasn't operating on a copy :(12:19
zyga-ubuntukyrofa: no, my backup was broken (deja-***-dup)12:19
kyrofa:(12:19
zyga-ubuntuanyway12:19
zyga-ubuntuI kind of remember what I wrote12:19
zyga-ubuntuguess what I'm doing today12:19
mvoackk: hi, re url parsing> is https://golang.org/pkg/net/url/#Parse that you need?12:21
ChipacaI wish there was an Equals interface in go12:22
ackkmvo, not really. listen-stream for TCP/UDP is basically in the form of  [address:]port12:25
mvoackk: hm, I see12:26
* Chipaca takes a break12:27
ackkmvo, I mean, I can implement the validation, just wondering if there's something like that already12:28
mvoackk: not sure what exactly listen-stream looks like, but https://play.golang.org/p/Fi-0n-cRNF is using url.Parse() even without schema12:29
ackkmvo, also, it may be that we just care about the port (see my comment https://forum.snapcraft.io/t/socket-activation-support/2050/28?u=ack)12:29
mvoackk: I have a meeting now, but can check after the meeting for listen-stream12:29
ackkmvo, it doesn't really work https://play.golang.org/p/QPUxhKV_Dv :)12:34
niemeyerMoin moin12:49
niemeyerackk: Thanks, responded12:55
mvoniemeyer: please start without me, I'm in a different meeting13:00
=== ShalokShalom_ is now known as ShalokShalom
zyga-ubuntuoh13:02
zyga-ubuntustandup13:02
zyga-ubuntujoining13:02
phil_mzyga-ubuntu: hi, how are you today?13:06
zyga-ubuntuphil_m: hi13:07
zyga-ubuntuphil_m: I'm around13:07
phil_mgood.13:09
phil_mjust wanted to ask you about the current progress you made with Manjaro.13:09
phil_many news. some I should test on my end?13:09
zyga-ubuntuphil_m: I didn't have time to try it on real hardware but I have a spare HDD so I plan to install it over weekend; apart from that it is ready and I just want to test it13:12
zyga-ubuntuphil_m: (on hardware, works in VM)13:12
zyga-ubuntuphil_m: I had a pretty bad day last nigth though because my I managed to rm -rf my work directory on my main desktop13:13
phil_mzyga-ubuntu: ouch.13:14
phil_mthat is really sad to hear.13:14
ackkniemeyer, thanks. basically I think I only need a final decision about what is acceptable for listen-stream13:14
ogra_niemeyer, how do i add an "ogra" tag in the forum (does that need you with admin rights ?)13:18
niemeyerogra_: Yeah, not just me, but someone with permissions13:18
zyga-ubuntuphil_m: but no worries, all my patches are still in that VM so I didn't completely blow my data13:18
niemeyerogra_: Only thing I ask is to please follow the conventions in that topic (can find the link if you want) about tagging with upcoming and backlog as well, so the username tag should not be alone13:19
phil_mzyga-ubuntu: ok. makes sense13:19
ogra_niemeyer, it is for https://forum.snapcraft.io/t/split-initrd-implementation/2224/113:19
ogra_(allready has upcoming and will likelyy get more names on it)13:20
ogra_niemeyer, thanks!13:20
niemeyerogra_: Done, tag created13:20
ogra_oh, gone again13:20
niemeyerogra_: You should be able to use it now13:20
ogra_k13:20
ogra_yup, works13:21
mupPR snapcraft#1568 opened: lxd: don't re-inject the same snaps <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1568>13:21
mvojdstrand: when you have a moment (not urgent) a review of 3956 would be appreciated13:22
* kalikiana feeling dizzy and hungry, taking a break and looking for some food13:23
mupBug #1718942 opened: running favorited snap shows two icons in Ubuntu dock <Snappy:New> <https://launchpad.net/bugs/1718942>13:39
phil_mzyga-ubuntu: I just created a new package of snapd with version 2.28dev+g072349313:52
phil_mzyga-ubuntu: still face the error 'cannot locate the base snap: ubuntu-core: No such file or directory' on my end.13:53
ogra_phil_m, weird, does it really still say ubuntu-core ?13:54
ogra_phil_m, also ... did you see https://forum.snapcraft.io/t/snap-on-arch-liri-os/2223/113:54
phil_myep.13:54
phil_mzyga-ubuntu: used files can be found here: https://github.com/manjaro/packages-community/tree/master/snapd-git13:55
phil_mand the standard stable release at: https://github.com/manjaro/packages-community/tree/master/snapd13:55
ogra_(funny that is seems to be kind of working with a few minor changes on liriOS but not on manjaro ... )13:56
phil_mmaybe I should change PATH=$PATH:/var/lib/snapd/snap/bin:/snap/bin to just PATH=$PATH:/snap/bin13:57
phil_morga_: we already use: https://github.com/snapcore/snapd/commit/413a2a099dc57e3fb7c7e82a6c7250cf4c2f74fd14:00
ogra_yeah14:01
zyga-ubuntuphil_m: hmmmmm14:05
zyga-ubuntuphil_m: we had a patch for that error message14:05
zyga-ubuntuphil_m: one sec14:05
kyrofaHey kalikiana I've got an odd bug. As you've been touching the repo code most recently, any chance you've come across it? https://bugs.launchpad.net/snapcraft/+bug/168648114:06
mupBug #1686481: stage-packages doesn't respect architectures <kubernetes> <Snapcraft:Confirmed> <https://launchpad.net/bugs/1686481>14:06
kyrofaOr otherwise have any insight14:07
zyga-ubuntuphil_m: do you have 8c821c13bb62703c5119a1b0b1edbd53ce7f48aa in your tree?14:08
phil_mlet me check14:09
phil_mno, will try some now.14:10
mupPR snapd#3957 opened: cmd,dirs: treat "liri" the same way as "arch" <Created by zyga> <https://github.com/snapcore/snapd/pull/3957>14:13
phil_mhmm, the error is gone, but now it simply hangs a while14:14
phil_mchromium now started with a delay14:14
ogra_well14:14
phil_msecond start was faster.14:14
ogra_snaps typically coopy some stuff arund on first start14:14
ogra_s thats kind of expected14:14
phil_mseems it is now kinda working14:14
ogra_\o/14:15
zyga-ubuntuphil_m: is snapd running?14:15
phil_mseems so14:15
ogra_if the chroium snap runs it better should :P14:16
phil_mnow I only don't know if it is in /snap or in the /var/lib/snapd/snap14:16
zyga-ubuntuphil_m: what did you rebuild?14:17
zyga-ubuntuphil_m: can you please look at https://forum.snapcraft.io/t/snap-on-arch-liri-os/2223/414:17
zyga-ubuntuphil_m: using that source package (just with .6 and manjaro patch instead of liro patch) you should be mostly fine14:17
phil_mwill upload the changes soon14:17
=== cachio_ is now known as cachio
jdstrandmvo: done14:38
flexiondotorgHey zyga-ubuntu14:40
flexiondotorgSo, you're still seeking an Arch TU to collaborate with?14:41
zyga-ubuntuflexiondotorg: hey14:46
zyga-ubuntuflexiondotorg: yes, very much so14:46
zyga-ubuntuflexiondotorg: especially now with arch derivatives suffering from old package ther14:46
zyga-ubuntuflexiondotorg: do you have someone that could help?14:48
flexiondotorgWell, I've been reaching out to the main derivatives Manjaro and Antergos.14:49
flexiondotorgI'll see if I still have any credit in the Arch community. I'll see if I can find a sponsor to reprise my role as Arch Linux TU14:50
zyga-ubuntuflexiondotorg: thank you!14:50
flexiondotorgIf I can get a sponsor, it will still be down to a vote and that takes a month14:53
zyga-ubuntuyes, I read the protocol14:55
zyga-ubuntuthe best we can do though14:55
zyga-ubuntuI really wish timothy didn't abandon the package14:55
zyga-ubuntubut it seems he has14:55
zyga-ubuntu(also ignoring email)14:56
flexiondotorgYep, I've tried contacting him too14:56
zyga-ubuntumaybe we could give him a ring via RH14:57
flexiondotorgIndeed😉14:57
zyga-ubuntu"step down considerably" is not a virtue it seems14:57
zyga-ubuntue14:57
zyga-ubuntuer14:57
zyga-ubuntu"resposibly" was it?14:57
zyga-ubuntuanyway14:57
Pharaoh_Atemmvo: let me give it a whirl15:01
Pharaoh_Atemzyga-ubuntu: I'll see if I can get a hold of tradelli15:02
mvoPharaoh_Atem: thanks15:02
Pharaoh_Atemzyga-ubuntu: he's in the Fedora community somewhat since he now works at Red Hat15:03
zyga-ubuntuPharaoh_Atem: thank you!15:03
zyga-ubuntuPharaoh_Atem: note that we made multiple attempts to contact him (irc, twitter, email, linkedin) and got no replies15:04
zyga-ubuntu(over extended amount of time)15:04
Pharaoh_Atemzyga-ubuntu: I've talked to him a couple of times recently15:04
Pharaoh_Atemhis dead is down in the weeds in DPDK and kernel networking stuff15:04
Pharaoh_Atems/dead/head/15:04
mvojdstrand: thanks a bunch! I addressed your comments15:04
zyga-ubuntuPharaoh_Atem: haha15:05
Pharaoh_Atemhe took over from Panu Matilainen when he returned to the helm of rpm development/maintenence15:05
zyga-ubuntuPharaoh_Atem: DPDK?15:05
Pharaoh_AtemData Plane Developer Kit15:05
Pharaoh_Atemfrom Intel, now a Linux Foundation project15:05
Pharaoh_Atemdoes weird things to network subsystem to make things go fasterer15:05
Pharaoh_Atemhttps://en.wikipedia.org/wiki/Data_Plane_Development_Kit15:06
* ogra_ points to apt-cache show dpdk15:06
zyga-ubuntumagic15:07
Pharaoh_Atembut basically, DPDK is terrible and sucks the life out of you15:07
zyga-ubuntuPharaoh_Atem: it must pay well then15:08
Pharaoh_Atemmvo: err, why is the spec in 2.28 branch telling me it's 2.27.5?15:09
phil_mzyga-ubuntu: Manjaro now supports fully snaps15:11
Pharaoh_Atemmvo: ah, you haven't been bumping the versions like you did for 2.27 during the rc development15:12
phil_mI've tested the curren stable v2.17 and current dev series v2.2815:12
zyga-ubuntuphil_m: \o/ !!15:18
zyga-ubuntuphil_m: excellent15:18
zyga-ubuntuphil_m: what kind of GPU do you have?15:18
zyga-ubuntuphil_m: and can you tell me more about your kernel15:18
zyga-ubuntuphil_m: (and can we add a logo with installation instructions to snapcraft.io?)15:18
=== ikey|zzz is now known as ikey
phil_myou can add a logo and the instruction # pacman -Sy snapd when we released it also to our stable branches15:22
phil_mI've an Nvidia gpu15:23
zyga-ubuntuphil_m: excellent, I'll make that happen! Thank you! :)15:23
phil_mand the kernel is more or less vanilla with some patches added.15:23
zyga-ubuntuphil_m: do you have a preferred logo (quality, etc) I should use?15:23
davidcallezyga-ubuntu: phil_m: can you open a bug with the logo?15:23
phil_mhttps://github.com/manjaro/packages-core/tree/master/linux41315:23
phil_mis the current stable kernel15:24
phil_mdavidcalle: in which tracker should I open the bug?15:24
davidcallehttps://github.com/canonical-websites/snapcraft.io-static-pages15:25
davidcalleThanks phil_m :)15:25
zyga-ubuntuthank you both!15:25
kalikianakyrofa: Sorry for the late reply, this stupid cold forced me to rest... regarding bug 1686481 I'll add a comment in a moment15:29
mupBug #1686481: stage-packages doesn't respect architectures <kubernetes> <Snapcraft:Confirmed> <https://launchpad.net/bugs/1686481>15:29
jdstrandmvo: hey, did you see my followup comment on PATH_MAX?15:31
jdstrandmvo: also, think you need a go fmt15:31
phil_mdavidcalle: https://github.com/canonical-websites/snapcraft.io-static-pages/issues/38715:32
zyga-ubuntujdstrand: thank you for the instructive reviews!15:33
zyga-ubuntumvo: can I merge https://github.com/snapcore/snapd/pull/395215:33
mupPR #3952: cmd: update "make hack" <Created by zyga> <https://github.com/snapcore/snapd/pull/3952>15:33
zyga-ubuntuone more test...15:40
kalikianakyrofa: Good job spotting that old bug :-D I posted a test snippet that seems to work for me. Although this is almost untested so not sure if it does what they need - but at least I've got some tests for this in my queue15:49
kyrofakalikiana, excellent, thank you very much :)15:53
ogra_ARGH !16:05
ogra_my rocket snap just made my system get stuck16:05
ogra_niemeyer, is it wanted that mup just sent ~1000 PR messages to #snapcraft on rocket ?16:06
niemeyerogra_: I hope you know the answer to that :)16:06
ogra_heh16:06
ogra_well, something is weird16:07
niemeyerogra_: IIRC mup was actually disabled there16:07
ogra_yeah16:07
niemeyerogra_: Someone is probably fiddling with settings16:07
ogra_thats what i meant with weird :)16:07
niemeyerogra_: They broke the plugin API for the Nth time a couple of months ago and I disabled it.. need to look into it to see if it's fixed16:08
ogra_seems to have stopped now16:08
sergiusenskalikiana elopio how about a look on snapcraft#1565 ?16:08
mupPR snapcraft#1565: cli: add the assemble command <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1565>16:08
niemeyerogra_: Probably someone just flipped the switch enabling it16:08
ogra_yep, for a minute or two and it replayed some caxche i guess16:08
kalikianasergiusens: You got some comments16:21
ikeylooks like manjaro landed snap support16:25
ikeythe commune groweth16:25
zyga-ubuntuikey: indeed :)16:31
ikeylooks like everyone is getting on the bandwagon now16:33
* ikey likes cascading self ordering chaos16:33
niemeyerikey, zyga-ubuntu: Nice!16:49
=== alan_g is now known as alan_g_
ogra_nacc, i guess kyrofa would know more about that ;)16:52
ogra_(and i'm saying this here because he isnt in #ubuntu-release=16:52
ogra_)16:52
ogra_:)16:53
naccogra_: ack :)16:55
mvozyga-ubuntu: I think so16:57
mvojdstrand: yeah, I fixed that, thanks and sorry :)16:58
mvoPharaoh_Atem: I can push with 2.28~rc4 if you want16:58
mupPR snapd#3952 closed: cmd: update "make hack" <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/3952>16:58
jdstrandmvo: thanks!17:01
zyga-ubuntumvo: thank you17:07
Pharaoh_Atemmvo: I'm testing to see if the new cheggaaa-pb package works without the hack atm17:15
mvoPharaoh_Atem: thanks17:15
Pharaoh_Atemif it does, I'm going to send a PR to revert that commit and then the corresponding revert can be done in 2.2817:15
* zyga-ubuntu -> supper17:17
kyrofaNooo, I got up at 5am to prep a lightning talk only to arrive and learn they switched it to a raffle instead of first-come-first-served... and didn't get in18:03
kyrofanacc, ogra_ I seem to be missing the scrollback that contains context. What would I know more about?18:06
nacckyrofa: it's mostly in #ubuntu-release18:06
nacckyrofa: let me get the linnk18:07
kyrofaHaha, well that would explain the missing context18:07
nacchttps://irclogs.ubuntu.com/2017/09/22/%23ubuntu-release.html18:07
nacckyrofa: exec. summary: 1) snapcraft dep8 tests are broken on artful 2) artful builds of snaps are broken18:07
naccaiui, the solutions are multiple: 1) snapcraft dep8 should use cleanbuild18:08
zyga-ubuntuman, travis and github don't love me today18:08
nacc2) there needs to be blacklist libs file added for all non-16.04 releases (elopio had a test for this for 17.10 specifically)18:08
zyga-ubuntunetwork isuses18:08
kyrofaYes indeedy, that's not surprising at all18:08
nacc3) the fallback for not havinng a filelist should be an error (IMO), not a fallback. As it just doesn't work. Or the fallback list should be everything the libc deb provides18:09
naccnow that won't fix artful builds of snaps, as they still have a xenial core snap and there be dragons18:09
naccbut it means the libc issue that's currently busted would work, i think18:10
naccs/would work/would not manifest itself/18:10
kyrofaSorry guys, I didn't realize that was a room in which I should idle18:11
kyrofaRectified now18:11
nacckyrofa: we probably should have brought the convo here18:11
kyrofanacc, so yeah, snapcraft crawls prime and essentially sucks over any libs it discovers via ldd on every elf18:13
kyrofanacc, it ships the libraries/16.04 file as a blacklist of stuff in the core snap to NOT suck over18:13
kyrofaBut of course, it checks the OS before deciding whether or not to use libraries/16.04, or libraries/16.10, etc18:14
kyrofaThe latter of which doesn't exist18:14
kyrofaThus nothing gets excluded, and it sucks everything off the host18:14
nacckyrofa: yep that was my analysis from the source as well18:14
nacckyrofa: since snapcraft's dep8 is runnig natively on artful, it didn't find any blacklist file18:15
kyrofaThis is a problem not only per release, but per arch, as different arch core snaps contain different things, and thus need separate blacklists18:15
nacckyrofa: ouch :/18:15
kyrofaTo compound matters, we've decided the library sucking feature sucks (heh) because it's magical and unreliable, as you've discovered, so we haven't really invested in making it better18:15
naccwell, as of right now there are two relatively high-profile (to me) issues then: snapcraft dep8 is failing and any snap built on artful right now is going to be broken18:16
kyrofaYes and yes. Agreed18:16
naccI happened to get built by the latter before the former was seen, I guess :)18:16
asafniv1[m]any good snap apps?18:16
kyrofaI really don't think we should just remove the feature though, or current snaps will stop working18:17
kyrofaSo we probably need to invest some time in improving it18:17
asafniv1[m]we should make snaps use system fonts and theme18:17
nacckyrofa: yeah, i'm not sure the best way forward -- i think elopio's file list is a good start (i'm not sure how many non-amd64 non-classic snaps are being built on artful, e.g.)18:17
nacckyrofa: but like you said, that's insufficient on its own18:17
nacci'm also switching my snap to build on xenial (presuming the tests pass :)18:18
kyrofaogra_, if I install a snap on artful, what core snap am I getting? The 16 one, right?18:20
kyrofaWhich means even if we DO blacklist the stuff in the core snap, if the libc in artful changes, it'll still be pulled in18:21
zyga-ubuntukyrofa: yes18:21
kyrofaEven if the library sucking feature was gone, the snap would be built against a different libc version than that with which it will be run18:22
kyrofaThere's no solution to this problem!18:22
kyrofa:P18:22
nacckyrofa: yeah, this is all pretty buggy (building on artful) until we get base snaps18:22
kyrofaYep18:22
naccthat's why, honestly, it should be disabled (but there's no way to communicate that effectively it feels like)18:22
naccit's just guaranteed to have weird failures eventually :)18:22
kyrofanacc, well, like I said, even if it was disabled, building on artful won't result in a snap that runs18:23
nacckyrofa: no, i mean, don't allow buildig on artful :)18:23
naccat least not via LP18:23
kyrofaAhhh18:23
asafniv1[m]Don't use ubuntu18:23
kyrofaAgreed. It should only be xenial18:23
naccit's just goign to lead to bad publishes to the store right now18:23
nacckyrofa: yep18:23
kyrofaIndeed18:23
kyrofaAnd the autopkgtests should be cleanbuild18:24
nacckyrofa: yeah, that will at least get them passing, regardless of aythign18:24
kyrofaAlthough that means less of snapcraft is tested, which isn't great18:24
asafniv1[m]what if someone makes a snap virus18:25
zyga-ubuntuasafniv1[m]: what then?18:25
asafniv1[m]would canonical remove it?18:25
asafniv1[m]btw do they test snaps?18:25
asafniv1[m]or scan?18:26
zyga-ubuntuasafniv1[m]: we don't test snaps18:27
zyga-ubuntuasafniv1[m]: we scan them for certain things18:27
zyga-ubuntuasafniv1[m]: and check which interfaces they want to use, snaps get manual review if they request privileged interface for the first time18:27
asafniv1[m]ok18:28
nacckyrofa: yeah, the coverage would go down, i suppose18:28
zyga-ubuntuasafniv1[m]: the security model depends on a combination of sandbox, distinction between privileged and common interfaces and packages that are signed by the snap store18:29
zyga-ubuntuasafniv1[m]: you can upload any code you like, it will not be allowed to do anything from the privileged set unless there is a store review and we essentially trust you18:29
zyga-ubuntuniemeyer: hey19:20
zyga-ubuntuah, found it :)19:21
niemeyerzyga-ubuntu: Heya :)19:21
zyga-ubuntuniemeyer: I flagged some spam on the forum19:22
zyga-ubuntuniemeyer: wasn't sure if you are still around19:22
niemeyerzyga-ubuntu: Done19:28
zyga-ubuntuthank you :)19:36
niemeyerzyga-ubuntu: np, thanks for flagging!19:39
naccwe've got a jenkins job that is trying to test MPs for git-ubuntu before we land them by building a snap on xenial (using cleanbuild) in a xenial VM and then testing that locally built snap. Anyone know why we're gettinng the following error related to the core snap?https://paste.ubuntu.com/25594413/19:51
sergiusenskyrofa nacc we shouldn't improve it, we should remove it as we discussed months ago, breakage here is a simple fix20:29
sergiusensand there probably won't be base snaps for non LTS releases20:30
sergiusensbut we haven't gotten into the details around that, probably a discussion for next week with niemeyer20:30
naccsergiusens: yeah, i'm +1 onn removing it if there really can't be support for it20:31
naccsergiusens: i'm not going to be at the rally, but i thikn i'll be getting hte info from my team20:31
mupBug #1690880 changed: test_snappy_version fails on artful <cloud-init:Fix Released> <Snappy:Fix Released> <https://launchpad.net/bugs/1690880>20:54
kyrofasergiusens, I don't think that's a good idea-- lots of snaps will break. But it's your call21:00
sergiusenskyrofa they will have a simple fix21:02
kyrofasergiusens, it's still breakage that requires them to fix it21:03
naccsergiusens: build from source on xenial?21:03
kyrofaThink CI, daily builds21:03
sergiusenskyrofa if CI never breaks, why do you run it?21:04
sergiusensI know it will break, people running CI will detect it fast and adapt21:04
kyrofaCI is for catching MY problems :P21:04
sergiusenssame thing for new features and deprecations in snapd (e.g.; aliases)21:04
kyrofaBut yeah, your call21:05
naccsergiusens: is what i said the fix?21:06
naccis this expected?21:40
naccls -ahl /snap/core/current/lib64/ld-linux-x86-64.so.221:40
nacclrwxrwxrwx 1 root root 32 Sep 11 23:36 /snap/core/current/lib64/ld-linux-x86-64.so.2 -> /lib/x86_64-linux-gnu/ld-2.23.so21:40
nacc(which does't exist on artful)21:40
naccogra_: --^ i switched my snnap to build with cleanbuild and now i get the relocation error on artful :)21:40
ogra_heh21:41
naccstill waiting for jenkins to finish to see if it works on 16.04 now, at least21:41
naccspecifically /usr/bin/python3: /snap/core/current/lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.25' not found (required by /usr/bin/python3)21:43
naccnot sure why it' susing the python3 from the host21:43
ogra_is it using the python plugin ?21:46
naccogra_: no, my app just uses the dump plugin21:47
naccand every command is pointed at a wrapper script21:47
naccthat invokes python3 *in* my snap21:47
ogra_kyrofa, sorry, i didnt mean you should permanently idle in #ubuntu-release ... that was more a ping for if you were around to participate in the discussion :)21:47
=== JoshStrobl is now known as JoshStrobl|zzz
ogra_nacc, right ... the pytjon plugin makes sure you get the interpreter inside iirc21:48
ogra_*python21:48
naccogra_: hrm, by doing what? i mean does it tell snapd something?21:48
ogra_so you should perhaps add another part using the python plugin and not doing anything else21:49
ogra_it ships it in $SNAPp21:49
naccoh i do have one for python221:49
ogra_it ships it in $SNAP/bin21:49
nacchrm21:49
naccok21:49
naccogra_: i have a python3 in my snap21:49
nacci'm wondering if env is messed up now21:49
* ogra_ gets dinner and needs to go afk ... 21:49
naccogra_: np, thanks for all the help today!21:50
ogra_:)21:50
naccok relocation error was a bug in how i was calling the python3 argcomplete21:54
kwmonroeahoy snapaholics - in the content interface doc (https://forum.snapcraft.io/t/the-content-interface/1074) it mentions writable data should "share all of $SNAP_DATA or $SNAP_COMMON for now."  is that still a thing?  i'm able to share writable subdirs in core-16-2.27.6, but dunno what nasties may come.21:57
naccogra_: lol, i see it now and am not sure how to fix it22:01
nacc#!/usr/bin/env python3 in my pseudo-confined snap will segfault on artful22:01
naccbecause i've modified LD_LIBRARY_PATH to point to the core snap22:02
naccbut /usr/bin/env is from artful22:02
naccogra_: so that was definitely one issue (which is hard to debug, as the segfault happens in the python script wrapped)23:42
naccogra_: and i foudn the other one, i had forgotten to drop the stage-packages i was no building from src, so they were overriding my built binaries and leading to segfaults ;)23:42
nacci'm rebuilding now to test23:42
mupPR snapcraft#1569 opened: tests: refactor the fake snapd to not hardcode values <Created by elopio> <https://github.com/snapcore/snapcraft/pull/1569>23:47
nacckyrofa: sergiusens: do you want me to file a bug or something for this?23:50
naccogra_: --^23:50

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