| mwhudson | kyrofa: it's not that bad http://paste.ubuntu.com/24284541/ | 00:08 |
|---|---|---|
| slangasek | zyga, elopio: LP: #1673568 and LP: #1626359 are linked from the changelog of the SRU for snapd 2.23.6, but neither bug has an SRU template. Will someone be adding test cases to these bugs, or should the source be reuploaded without the bug refs? | 00:13 |
| mup | Bug #1673568: snapd 2.23.6 SRU tracking bug <snapd:New> <https://launchpad.net/bugs/1673568> | 00:13 |
| mup | Bug #1626359: Cannot authorise quotactl syscall for Q_GETQUOTA <snapd-interface> <verification-done> <Snappy:In Progress by jdstrand> <snapd (Ubuntu):Triaged> <snapd (Ubuntu Trusty):Triaged> <snapd (Ubuntu Xenial):Triaged> <snapd (Ubuntu Yakkety):Triaged> <https://launchpad.net/bugs/1626359> | 00:13 |
| slangasek | zyga, elopio: sorry, LP: #1673568 is clearly an SRU tracking bug from the title - so only LP: #1626359 is at issue | 00:14 |
| === chihchun_afk is now known as chihchun | ||
| morphis | Son_Goku: hah, https://koji.fedoraproject.org/koji/buildinfo?buildID=874052 ; that is awesome! great work! | 04:59 |
| morphis | zyga: hey! can you give https://github.com/snapcore/snapd/pull/3096 a review today? | 06:17 |
| mup | PR snapd#3096: many: abstract path to /bin/{true,false} <Created by morphis> <https://github.com/snapcore/snapd/pull/3096> | 06:17 |
| zyga | morphis: hey | 06:19 |
| zyga | morphis: looking | 06:19 |
| morphis | zyga: thanks | 06:21 |
| morphis | zyga: and not sure if you saw https://koji.fedoraproject.org/koji/buildinfo?buildID=874052 already :-) | 06:23 |
| zyga | morphis: done | 06:25 |
| zyga | very nice! | 06:25 |
| zyga | morphis: what was the problem with ppc64? I see it's green now | 06:26 |
| zyga | morphis: btw, why two binary packages? | 06:26 |
| zyga | morphis: snap-confine / snapd | 06:26 |
| morphis | zyga: good question for Son_Goku | 06:26 |
| morphis | zyga: ppc64le was green before too | 06:26 |
| Son_Goku | ppc64le is green | 06:27 |
| morphis | ppc64 was failing and still is | 06:27 |
| morphis | so the be variant | 06:27 |
| zyga | aah | 06:27 |
| zyga | ok | 06:27 |
| zyga | Son_Goku: I'd rather have one binary package (snapd/snap-confine) | 06:27 |
| morphis | Son_Goku: any reason why I don't see snapd yet with dnf on my rawhide vm? still need to wait for the sync? | 06:27 |
| zyga | Son_Goku: it will always be tightly coupled anyway so why splitting it | 06:27 |
| zyga | thanks for working on this guys! awesome work | 06:28 |
| zyga | I need to help with kids to get them out to school | 06:28 |
| zyga | ttyl | 06:28 |
| Son_Goku | zyga: because I needed the old RPM to go away in the F26 and Rawhide trees | 06:34 |
| Son_Goku | I'll merge snap-confine into snapd for 2.24, probably | 06:34 |
| morphis | Son_Goku: so why do I not see snapd in rawhide yet? is there any special trick like adding kojii repos? | 06:43 |
| Son_Goku | morphis: it depends on your local mirror | 06:43 |
| Son_Goku | it is definitely in the Koji internal repos | 06:43 |
| morphis | Son_Goku: according to https://apps.fedoraproject.org/packages/snapd it should be there | 06:43 |
| Son_Goku | it may not have been mashed out to the mirrors yet | 06:43 |
| morphis | Son_Goku: where do I check which mirror I am using? | 06:43 |
| Son_Goku | dnf will tell you when it starts doing stuff | 06:44 |
| Son_Goku | but you can also check with #fedora-releng about these things | 06:44 |
| Son_Goku | morphis: anyway, I just updated snapd-glib as well | 06:44 |
| Son_Goku | we need snapd-xdg-open for desktop apps to work | 06:44 |
| morphis | Son_Goku: is there a spec file for it already? | 06:45 |
| Son_Goku | morphis, zyga had a review that he never responded to... https://bugzilla.redhat.com/show_bug.cgi?id=1369560 | 06:46 |
| morphis | hah | 06:47 |
| morphis | Son_Goku: then let me put that on my list | 06:47 |
| Son_Goku | I'd like to have snapd, snapd-glib, and snapd-xdg-open all pushed in the same update | 06:47 |
| morphis | sounds good | 06:48 |
| morphis | Son_Goku: will clean the spec file today and integrate your review | 06:49 |
| morphis | looks like we don't do proper releases for snapd-xdg-open | 06:49 |
| Son_Goku | yeah | 06:51 |
| Son_Goku | anyway, must sleeps | 06:51 |
| Son_Goku | I've been up this whole time... | 06:51 |
| zyga | Son_Goku: makes sense | 07:21 |
| Son_Goku | ? | 07:22 |
| zyga | Son_Goku: I replied to your earlier comment about snap-confine | 07:22 |
| zyga | Son_Goku: sorry, daughter had damp moo | 07:23 |
| zyga | Son_Goku: didn't want to go to school; didn't want to stay at home | 07:23 |
| zyga | Son_Goku: decided to actually get dressed and eat breakfast as we were leaving | 07:23 |
| zyga | hey pstolowski | 07:29 |
| zyga | let me know if you have something to review | 07:29 |
| mup | PR snapd#3117 opened: tests: parameterize gadget snap channel <Created by fgimenez> <https://github.com/snapcore/snapd/pull/3117> | 07:29 |
| pstolowski | zyga, hi! | 07:29 |
| pstolowski | zyga, yeah, i'll have 3070 in a moment, something landed and it has new conflicts | 07:30 |
| zyga | pstolowski: sounds good | 07:31 |
| pedronis | pstolowski: sorry, my fault, I landed the locking changes we were discussing yesterday, https://forum.snapcraft.io/t/transactionality-locking-and-other-concurrency-coordination/50/8 | 07:36 |
| pstolowski | pedronis, no worries, nice change btw and conflicts were trivial | 07:37 |
| pstolowski | zyga, pedronis #3070 ready for re-review | 07:38 |
| mup | PR snapd#3118 opened: cmd: import snapd-xdg-open and integrate with our infrastructure <Created by morphis> <https://github.com/snapcore/snapd/pull/3118> | 07:46 |
| morphis | Son_Goku: didn't you say you need some sleep? :-) | 07:48 |
| zyga | pstolowski: looking now | 08:11 |
| pstolowski | thx | 08:13 |
| fgimenez | hey zyga a new kernel snap was promoted to beta yesterday, we had this new test error https://travis-ci.org/snapcore/spread-cron/builds/216891204#L392 could you please take a look? | 08:15 |
| zyga | fgimenez: hey | 08:16 |
| zyga | fgimenez: looking | 08:16 |
| zyga | fgimenez: can you in turn tell me what you think about this | 08:16 |
| zyga | 2017/03/30 19:17:29 Discarding autopkgtest:ubuntu-17.04-amd64, cannot connect: cannot connect to autopkgtest:ubuntu-17.04-amd64: ssh: must specify HostKeyCallback | 08:16 |
| zyga | fgimenez: I got this on https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-zesty-snappy-dev-image/zesty/amd64/s/snapd/20170330_191740_5aeb7@/log.gz | 08:16 |
| zyga | fgimenez: aha, let me prepare a quick PR, the test needs to be disabled; it will not work without another kernel change | 08:17 |
| fgimenez | zyga: sure looking | 08:17 |
| fgimenez | zyga: great thank you | 08:17 |
| mup | PR snapd#3119 opened: interfaces: API additions for interface hooks <Created by stolowski> <https://github.com/snapcore/snapd/pull/3119> | 08:19 |
| fgimenez | zyga: i've never seen before the autopkgtest error, it seems to happen at a very early stage, maybe something changed in zesty recently that makes the ssh connection fail, let me try to reproduce locally | 08:20 |
| zyga | fgimenez: I just pushed a change to https://github.com/snapcore/snapd/pull/3076, once it passes we should merge this | 08:22 |
| mup | PR snapd#3076: cmd: disable the re-associate fix as requested by jdstrand <Created by zyga> <https://github.com/snapcore/snapd/pull/3076> | 08:22 |
| mup | PR snapd#3120 opened: [WIP] interfaces: expose attrs to the interface API, snapctl enhancements (step #4) <Created by stolowski> <https://github.com/snapcore/snapd/pull/3120> | 08:22 |
| fgimenez | zyga: if you could execute the failing test locally using the beta kernel snap that would be great to be extra sure, "export SPREAD_KERNEL_CHANNEL=beta" and then run spread with the failing test filter | 08:28 |
| zyga | fgimenez: you mean 1644439 or the earlier one? | 08:34 |
| zyga | fgimenez: I mean that test is definitely off for the next few weeks, we can only release this after the _next_ kernel gets released | 08:34 |
| pstolowski | only for the brave: #3119, #3120 | 08:37 |
| zyga | haha, is that the hook code? | 08:39 |
| zyga | the attribute things | 08:39 |
| pstolowski | yes | 08:39 |
| pstolowski | the 2nd PR isnt really as big as it may seem (it includes the 1st one) | 08:40 |
| fgimenez | zyga: aha if it's going to be disabled it's ok then, thanks | 08:40 |
| === carlolo_ is now known as carlolo | ||
| morphis | zyga: you had a chance to look at https://build.opensuse.org/request/show/483839 already? | 08:50 |
| zyga | morphis: no, still on my queue | 08:52 |
| zyga | pstolowski: I just reviewed 3070 | 08:53 |
| zyga | pstolowski: have a look, I think you want to clarify locking and maybe think how this will scale | 08:53 |
| zyga | pstolowski: we won't have *that* many revisions but we can have any number of snaps | 08:53 |
| zyga | morphis: looking now | 08:53 |
| zyga | morphis: nice, too easy :) | 08:54 |
| zyga | morphis: thank you! | 08:54 |
| morphis | :-) | 08:54 |
| zyga | morphis: can you look at https://github.com/snapcore/snapd/wiki/Distributions#support-matrix | 08:54 |
| zyga | morphis: I think it needs a small lifting :) | 08:54 |
| pstolowski | zyga, thanks, looking. yes, that's a valid point although this is a general problem of state now, i think we need to solve this globally. with config snapshots we will have ~3 configs per snaps | 08:55 |
| zyga | pstolowski: yes, though I wonder how this works now, does golang json marshaller support partial encoding/decoding or is it all just smoke and mirros? | 08:56 |
| zyga | pstolowski: there's this json.RawMessage thing but I didn't look at the implementation to see what happens there | 08:56 |
| pstolowski | zyga, dunno, but I'd be surprised if it was that smart | 08:59 |
| zyga | pstolowski: and locking? | 09:00 |
| pstolowski | zyga, locking is fine, done by the caller, i'm adding a comment to these methods | 09:04 |
| zyga | pstolowski: I assumed as much because Get/Set check locking AFAIR | 09:05 |
| zyga | pstolowski: but I wanted to be sure | 09:05 |
| pstolowski | yes | 09:05 |
| zyga | pstolowski: can you look at https://github.com/snapcore/snapd/pull/3103 | 09:05 |
| mup | PR snapd#3103: interfaces/mount: add function for parsing fstab-like file <Created by zyga> <https://github.com/snapcore/snapd/pull/3103> | 09:05 |
| zyga | pstolowski: it's super short, 2nd review | 09:05 |
| pstolowski | zyga, sure | 09:06 |
| zyga | fgimenez: I think the issue affects everything now `2017/03/31 08:43:13 Discarding autopkgtest:ubuntu-16.04-amd64, cannot connect: cannot connect to autopkgtest:ubuntu-16.04-amd64: ssh: must specify HostKeyCallback` | 09:11 |
| * ogra curses ... so why does core fail to build today | 09:12 | |
| zyga | ogra: must be Friday | 09:13 |
| ogra | heh, yeah | 09:13 |
| fgimenez | zyga: running now, let's see.. | 09:16 |
| zyga | fgimenez: maybe ssh changed recently? | 09:16 |
| fgimenez | pedronis: hi, i've uploaded the expect snap to staging, these are the results on the ubuntu-core system http://paste.ubuntu.com/24286896/ could you please take a look when you have a moment? i think you mentioned we needed to run security-devpts in there, and it is failing | 09:18 |
| mup | PR snapd#3076 closed: cmd: disable the re-associate fix as requested by jdstrand <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3076> | 09:20 |
| zyga | ok, I've merged the patch that disabled 1644439 regression test | 09:20 |
| zyga | pstolowski: quick shot: https://github.com/snapcore/snapd/pull/3092 | 09:21 |
| mup | PR snapd#3092: interfaces: capitalize Udev... as UDev <Created by zyga> <https://github.com/snapcore/snapd/pull/3092> | 09:21 |
| zyga | pstolowski: let's land it | 09:21 |
| zyga | morphis: 2nd review ^: this is a 1 char rename across the tree | 09:22 |
| zyga | (capitalization change) | 09:22 |
| morphis | zyga: aye | 09:22 |
| zyga | niemeyer: I'd like to land https://github.com/snapcore/snapd/pull/3039 | 09:24 |
| mup | PR snapd#3039: many: add support for partially static builds <Created by zyga> <https://github.com/snapcore/snapd/pull/3039> | 09:24 |
| pedronis | fgimenez: mvo: hi, I was getting this on all my PRs from autopkgtests: Discarding autopkgtest:ubuntu-16.04-i386, cannot connect: cannot connect to autopkgtest:ubuntu-16.04-i386: ssh: must specify HostKeyCallback do you know what it is about ? | 09:24 |
| zyga | niemeyer: I think all the feedback is addressed now, please have a 2nd look | 09:24 |
| zyga | pedronis: I don't see mvo here today | 09:25 |
| zyga | pedronis: is he off/ | 09:25 |
| morphis | zyga: btw. is there a way to invoke snap-confine manually? | 09:25 |
| zyga | morphis: sure | 09:25 |
| zyga | morphis: `make hack` in the tree | 09:25 |
| zyga | morphis: then you can use it | 09:25 |
| zyga | morphis: alternatively just run it, you need two things: | 09:25 |
| zyga | morphis: export SNAP_NAME=... | 09:25 |
| fgimenez | pedronis: looking into it right now, zyga pointed it out too | 09:25 |
| zyga | morphis: and call it with a security tag (snap.$SNAP_NAME.$APP_NAME) | 09:25 |
| morphis | ah | 09:25 |
| zyga | morphis: make hack is nice as it just works | 09:25 |
| zyga | morphis: replaces system snap-confine | 09:25 |
| zyga | morphis: very useful for iteration | 09:25 |
| morphis | zyga: yeah, but SNAP_NAME=.. is enough for now | 09:26 |
| zyga | morphis: sure but you need to call it with the mandatory argument ^_^ | 09:28 |
| morphis | yeah | 09:28 |
| morphis | works already :-) | 09:28 |
| ogra | slangasek, looks like your live-build change scrwed up our core snap builds (it tries to switch the image over to upstart) | 09:28 |
| zyga | upstart ? | 09:28 |
| ogra | yes | 09:28 |
| zyga | wow, blast from the past :) | 09:28 |
| zyga | retro base :) | 09:28 |
| ogra | http://launchpadlibrarian.net/309249343/live-build_3.0~a57-1ubuntu25.1_3.0~a57-1ubuntu25.2.diff.gz | 09:29 |
| ogra | ++# remove all installed packages that are Prio: required in the release | 09:29 |
| ogra | ++# pocket but something else in the latest version | 09:29 |
| zyga | so we remove systemd and get upstart to satisfy some dependencies? | 09:29 |
| ogra | in the logs i see the builds remove systemd-sysv and try to install all upstart packages | 09:29 |
| zyga | right | 09:29 |
| ogra | https://launchpadlibrarian.net/313704330/buildlog_snap_ubuntu_xenial_amd64_core_BUILDING.txt.gz | 09:29 |
| ogra | zyga, and it seems you were right ... its friday (that looks like something to fill the day :/ ) | 09:31 |
| zyga | I could also use a review for https://github.com/snapcore/snapd/pull/3114 | 09:36 |
| mup | PR snapd#3114: interfaces/mount: add function for saving fstab-like file <Created by zyga> <https://github.com/snapcore/snapd/pull/3114> | 09:36 |
| zyga | it's even smaller and would allow me to start doing real stuff (tm) in update-ns | 09:36 |
| zyga | pstolowski: 3119 reviewed | 09:41 |
| zyga | pstolowski: have a look | 09:41 |
| zyga | pstolowski: get a 2nd review and lets land it | 09:41 |
| pstolowski | zyga, thanks, looking | 09:43 |
| zyga | pstolowski: I'll review the 2nd one after this lands, I need to focus on a branch I want to land today | 09:45 |
| mup | PR snapd#3092 closed: interfaces: capitalize Udev... as UDev <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3092> | 09:57 |
| zyga | morphis: test seem to fail on https://travis-ci.org/snapcore/snapd/builds/217086502 | 09:58 |
| zyga | aha, timeout :/ | 09:58 |
| zyga | some trouble allocating VMs | 09:58 |
| zyga | morphis: maybe restart it when more tests are quiet | 09:58 |
| morphis | zyga: yeah let me do a force push to trigger another run .. | 09:59 |
| mup | PR snapd#3097 closed: interfaces/seccomp: add bind as part of the default seccomp policy (backport) <Created by zyga> <Closed by zyga> <https://github.com/snapcore/snapd/pull/3097> | 09:59 |
| zyga | morphis: maybe merge master | 10:01 |
| zyga | morphis: that should cure some failures | 10:01 |
| zyga | morphis: I'm going for a small walk now | 10:01 |
| zyga | morphis: I'll check back in half an hour :) | 10:02 |
| zyga | morphis: please pleaes please review the two fstab PRs, they are tiny and I need them today | 10:02 |
| zyga | pstolowski: tell me if you want me to the Validate helpers that cuts boilerplate | 10:02 |
| zyga | pstolowski: and get a 2nd review and land your PR | 10:02 |
| zyga | pstolowski: I'll do it on otp | 10:02 |
| zyga | top | 10:02 |
| zyga | ttyl | 10:03 |
| mup | PR snapd#3121 opened: WIP: cmd/snap-confine: always pass xauth file through regardless where it is located <Created by morphis> <https://github.com/snapcore/snapd/pull/3121> | 10:13 |
| ogra | Bug #1678042 | 10:22 |
| mup | Bug #1678042: removal of priority changed packages breaks builds of the core snap <live-build (Ubuntu):New> <https://launchpad.net/bugs/1678042> | 10:22 |
| ogra | sigh ... | 10:22 |
| fgimenez | pedronis: not sure what's going on with the autopkgtest failure, using locally a prebuilt xenial image, without updating spread and running autopkgtest with snapd from the archive i still get the same error, checking changes in the dependencies now.. | 10:34 |
| Chipaca | ok, back to work for me | 10:40 |
| Chipaca | not 100% yet but getting there | 10:40 |
| Chipaca | anything on fire? :-) | 10:40 |
| zyga | Chipaca: hey :) | 10:43 |
| zyga | Chipaca: autopkgtests are all broken, something changed in ssh | 10:43 |
| zyga | Chipaca: but nothing on fire, just faint smoke | 10:43 |
| zyga | Chipaca: how are you? | 10:43 |
| morphis | Son_Goku, Pharaoh_Atem, King_InuYasha: when you're back, https://github.com/snapcore/snapd/pull/3118 is something we should consider instead of creating another package we will drop real soon | 10:46 |
| mup | PR snapd#3118: cmd: import snapd-xdg-open and integrate with our infrastructure <Created by morphis> <https://github.com/snapcore/snapd/pull/3118> | 10:46 |
| Chipaca | zyga— headache died down, stomach settling, bored out of my mind but until recently couldn't turn my head fast | 10:48 |
| zyga | morphis: reviewed 3121 | 10:48 |
| zyga | Chipaca: ouch :/ | 10:48 |
| zyga | Chipaca: I had a rough evening, that youghurt *was* expired but ... well | 10:48 |
| zyga | Chipaca: it was still tasty though ;-) | 10:49 |
| Chipaca | heh | 10:49 |
| zyga | Chipaca: I could use some reviews on the fstab PRs | 10:49 |
| Chipaca | me it's like a hangover, but without the fun | 10:49 |
| zyga | Chipaca: and later on if you have a sec, I could use some hints on how to test some tricky code | 10:49 |
| Chipaca | zyga— https://thisisagoodsign.files.wordpress.com/2014/08/do_not_feed_hallucinogens_to_the_alligators.jpg | 10:53 |
| * Chipaca looks at PRs | 10:53 | |
| zyga | Chipaca: poor crocks :) | 10:54 |
| Chipaca | well, at least it was mushrooms and not nutmeg | 10:54 |
| Chipaca | zyga— what's the expected use case for `SaveFSTab`? | 10:59 |
| zyga | Chipaca: snap-update-ns will write the things it did to /run | 11:04 |
| zyga | Chipaca: it will be a file in /run/snapd/ns/$SNAP_NAME.fstab | 11:04 |
| Chipaca | so, /run doesn't need atomicity in the sense we mean there | 11:04 |
| zyga | Chipaca: it should match /var/lib/snapd/mount/ | 11:04 |
| Chipaca | but /var does | 11:04 |
| zyga | Chipaca: but if it crashes the code will pick up from the start | 11:05 |
| zyga | Chipaca: and see if it needs to mount anything | 11:05 |
| Chipaca | ah, if snapd crashes? yeah | 11:05 |
| zyga | Chipaca: not snapd, this will be in a separate process | 11:05 |
| Chipaca | k | 11:05 |
| Chipaca | i was thinking of the machine crashing, not the program crashing | 11:05 |
| zyga | Chipaca: if machine crashes it's all good | 11:05 |
| zyga | Chipaca: this is purely ephemeral | 11:06 |
| Chipaca | yes | 11:06 |
| Chipaca | zyga— in what sense should things in run match things in var? | 11:06 |
| zyga | if all is good yes | 11:06 |
| zyga | some changes can be rejected | 11:06 |
| zyga | so they will not be mounted | 11:06 |
| zyga | and such entries won't show up | 11:06 |
| zyga | some entries can fail for various reasons | 11:07 |
| zyga | this is just a helper for the algorithm we merged earlier | 11:07 |
| zyga | that descries 'current' state | 11:07 |
| zyga | the desired state is always in /var/ | 11:07 |
| zyga | written by snapd | 11:07 |
| zyga | using all the nice atomic helpers | 11:07 |
| zyga | and the current file will be only written by this thing | 11:07 |
| zyga | and maybe by snap-confine but this is really optional since it will cross-check with mountinfo to see what is really mounted | 11:08 |
| Chipaca | zyga— ok. Can you add a note about this to the method? | 11:08 |
| zyga | Chipaca: sure | 11:08 |
| Chipaca | zyga, thanks | 11:08 |
| pstolowski | oh my how quickly my branches fall into conflicts recently ;) | 11:12 |
| pstolowski | I guess this is what you get if you touch 10*n files (where n>1) | 11:13 |
| zyga | Chipaca: added | 11:14 |
| zyga | pstolowski: try touching n**10 files ;) | 11:14 |
| pstolowski | ;) | 11:14 |
| zyga | woow | 11:16 |
| zyga | snow in the pyreneeyes | 11:17 |
| Chipaca | zyga— +1 | 11:18 |
| Chipaca | now i need to person up and go get lunch going before the boys get home ravenous | 11:18 |
| zyga | so much snow :) | 11:18 |
| mup | PR snapd#3103 closed: interfaces/mount: add function for parsing fstab-like file <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3103> | 11:18 |
| Chipaca | (term's out) | 11:18 |
| Son_Goku | morphis: I'm not a fan of this whole merging everything into snapd | 11:21 |
| zyga | Son_Goku: why not? | 11:25 |
| zyga | Son_Goku: it really belongs together | 11:25 |
| * zyga small break | 11:27 | |
| morphis | Son_Goku: that is the future and just follows what we're already doing | 11:30 |
| morphis | Son_Goku: so what do you think about integrating that as a patch now into the 2.23 package instead of doing the migration with 2.24 of a then to be dropped snapd-xdg-open package? | 11:37 |
| mup | PR snapd#3122 opened: packaging: do not compile spread for autopkgtests <Created by fgimenez> <https://github.com/snapcore/snapd/pull/3122> | 11:38 |
| mup | Bug #1678076 opened: console-conf crashes with eth0 and wlan0 on Pi 3 <Snappy:New> <https://launchpad.net/bugs/1678076> | 11:43 |
| === pstolowski is now known as pstolowski|schoo | ||
| === pstolowski|schoo is now known as pstolowski-schoo | ||
| zyga | Chipaca: hmm, how do I pass a []byte to a C function that gets a char *? | 12:26 |
| zyga | Chipaca: I see C.CBytes | 12:27 |
| zyga | is that it? | 12:27 |
| zyga | NOTE: I need an out argument (I want the C function to populate the buffer) | 12:27 |
| CoderEurope | watched the video - but didn't feel it completed well : https://youtu.be/GiRbILdIVnA?t=15m58s | 12:36 |
| Son_Goku | morphis: no | 12:37 |
| zyga | garh | 12:37 |
| zyga | I don't know how to get this to work | 12:38 |
| Chipaca | zyga— ? | 12:39 |
| ogra | yay ... core builds fixed again ... phew | 12:39 |
| zyga | Chipaca: having C function; ssize_t read_cmdline(char* buf, size_t buf_size) | 12:39 |
| sborovkov | Hi, I am getting this apparmor error using dbus in strict: [ 816.152055] audit: type=1400 audit(1490957861.012:208): apparmor="DENIED" operation="connect" profile="snap.screenly-client.websocket" name="/run/dbus/system_bus_socket" pid=3893 comm="python3" requested_mask="wr" denied_mask="wr" fsuid=0 ouid=0 | 12:40 |
| sborovkov | any idea what can it be caused by? I connect to dbus plug. So it works in devmode at least. | 12:40 |
| Chipaca | zyga— unsafe.Pointer(&buf[0]) ? | 12:41 |
| zyga | Chipaca: I tried that, cgo says no | 12:42 |
| zyga | ./bootstrap.go:64: cannot use unsafe.Pointer(&buf[0]) (type unsafe.Pointer) as type *C.char in argument to _Cfunc_read_cmdline | 12:42 |
| Chipaca | zyga— show me da code? | 12:42 |
| zyga | Chipaca: pushed as 'WIP' to setns-on-startup branch | 12:42 |
| zyga | Chipaca: have a look | 12:42 |
| zyga | Chipaca: note that place is also a bit bogus as len != cap | 12:44 |
| zyga | Chipaca: and even if I get this to work via the pointer somehow it will not change len | 12:44 |
| zyga | sborovkov: hmmm | 12:45 |
| ogra | sborovkov, sounds like a question for jdstrand | 12:45 |
| zyga | sborovkov: yes (who is off today) | 12:45 |
| zyga | let me look | 12:45 |
| sborovkov | so I am running a daemon. And this happens when connecting to system bus | 12:46 |
| Chipaca | zyga— ah, convert them to the right type and you're set | 12:46 |
| Chipaca | zyga— https://play.golang.org/p/BLn0jYkDmX | 12:47 |
| zyga | sborovkov: that seems to be covered by dbus-strict abstraction | 12:47 |
| zyga | sborovkov: so no idea, please report a bug | 12:47 |
| zyga | d'oh :)) | 12:48 |
| sborovkov | zyga: is it supposed to be used manually somehow in the snapcraft.yaml's code? just for reference that's what I have https://hastebin.com/apaxuseyob.lua | 12:49 |
| Chipaca | zyga— type safety goes out the window, very obviously | 12:49 |
| zyga | Chipaca: who cares about safety ;-) | 12:49 |
| zyga | Chipaca: this is just for unit testing | 12:49 |
| Chipaca | o/ | 12:49 |
| ogra | sborovkov, you want "daemon: dbus" i belive | 12:49 |
| ogra | looking at https://github.com/snapcore/snapd/blob/4ef0a46ea8ee10f45e73a91925cbfa5fdfbbe2e5/snap/validate.go#L137 | 12:49 |
| zyga | Chipaca: trying, thanks | 12:50 |
| sborovkov | ogra: ahh, alright I will try converting all my services that use dbus to that type from simple then. Thanks. | 12:50 |
| zyga | wooot | 12:51 |
| zyga | Chipaca: thanks!! | 12:51 |
| zyga | Chipaca: two things | 12:51 |
| zyga | Chipaca: casting "backwards" (C habbit) | 12:51 |
| ogra | sborovkov, well, perhaps i'm wrong, try with one first :) | 12:51 |
| zyga | Chipaca: and mandatory () around *C.char | 12:51 |
| zyga | and all backwards type syntax ;) | 12:51 |
| Chipaca | zyga— it's not casting, it's converting | 12:51 |
| Chipaca | there's a copy involved | 12:51 |
| Chipaca | but yes, it gets awkward quickly | 12:51 |
| zyga | Chipaca: copy of a pointer is OK | 12:52 |
| zyga | Chipaca: let me see if this works | 12:52 |
| zyga | yes!! | 12:52 |
| Chipaca | zyga— but then you get a copy of a copy and it starts to degrade :-p | 12:52 |
| Chipaca | suddenly all the halftones are gone | 12:52 |
| Chipaca | i swear it was just ibuprophene | 12:53 |
| Chipaca | ibuprofen* | 12:53 |
| CoderEurope | Chipaca: meds ? What are you telling the group ? | 12:54 |
| Chipaca | CoderEurope— ? | 12:54 |
| Chipaca | telling which group? | 12:54 |
| Chipaca | use small words | 12:55 |
| CoderEurope | Chipaca: you mentioned ibuprofen on a packaging channel ? | 12:55 |
| Chipaca | CoderEurope— I am not well, today | 12:56 |
| Chipaca | CoderEurope— so I've got an excuse for my usual non-sense | 12:57 |
| CoderEurope | Chipaca: have you tried the 4-7-8 rule ? | 12:57 |
| Chipaca | CoderEurope— that's for sleeping? | 12:58 |
| CoderEurope | yeah, --okay well try this sometime, bye-ya https://redd.it/5zyykt | 12:58 |
| === pstolowski-schoo is now known as pstolowski | ||
| sborovkov | ogra: hmm, no that does not seem to work. Issues while validating snapcraft.yaml: The 'apps/websocket/daemon' property does not match the required schema: 'dbus' is not one of ['simple', 'forking', 'oneshot', 'notify'] Or it's in new snapcraft.. I updated it like last week | 13:05 |
| zyga | sborovkov: that is not supported I think | 13:06 |
| sborovkov | zyga: alright I will try asking jdstrand when he is around, and if it's not something I am doing wrong then I will file a bug. | 13:07 |
| zyga | sborovkov: I think we had a PR to support dbus deamones | 13:07 |
| sborovkov | ah, ok | 13:07 |
| ogra | zyga, https://github.com/snapcore/snapd/blob/4ef0a46ea8ee10f45e73a91925cbfa5fdfbbe2e5/snap/validate.go#L137 | 13:09 |
| ogra | zyga, is that not released yet ? | 13:09 |
| ogra | (that is what i just pointed sborovkov to) | 13:09 |
| zyga | niemeyer: I think I'm a bit too much out in the nature today | 13:10 |
| zyga | niemeyer: my connection is not sufficient for the hangout because of latency | 13:10 |
| zyga | niemeyer: I can do my update here | 13:10 |
| niemeyer | zyga: Remember that topic in the cafe? :) | 13:11 |
| zyga | niemeyer: yes, yes | 13:12 |
| zyga | ogra: what is the name of the amd64 kernel snap? | 13:23 |
| zyga | ogra: run "snapcraft history $that_snap_name" | 13:23 |
| ogra | pc-kernel | 13:23 |
| ogra | edge is at v55 currently | 13:24 |
| zyga | ogra: I cannot query forthat apparently | 13:24 |
| ogra | whats the query you use, i should be able to | 13:24 |
| zyga | ogra: that what I said | 13:24 |
| zyga | ogra: snapcraft history $SNAP_NAME | 13:24 |
| zyga | ogra: try that | 13:24 |
| ogra | damn .. that only works with a logged in machine | 13:25 |
| ogra | fgimenez, that was only candidate you wanted rolled back ? | 13:26 |
| fgimenez | ogra: yep thank you, that would make the old validation process work again, although i think things have moved because of this problem to prevent situations like this in the future | 13:32 |
| ogra | fgimenez, candidate is back to rev57 for amd64 now, should be good again | 13:39 |
| fgimenez | ogra: thanks! :) | 13:39 |
| === chihchun is now known as chihchun_afk | ||
| fgimenez | pedronis: about https://github.com/snapcore/snapd/pull/3117#issuecomment-290711403 what would happen when the CHANNEL env vars are different? for instance, a gadget snap is promoted to beta | 14:16 |
| mup | PR snapd#3117: tests: parameterize gadget snap channel <Created by fgimenez> <https://github.com/snapcore/snapd/pull/3117> | 14:16 |
| mup | PR snapd#3123 opened: osutil: introducing GetenvInt64, like GetenvBool but Int64er <Created by chipaca> <https://github.com/snapcore/snapd/pull/3123> | 14:21 |
| Chipaca | #3123 is a super easy review if you're bored | 14:24 |
| Chipaca | (used by the next commit on #2895) | 14:24 |
| pedronis | fgimenez: you do what you wanted to do on the branch | 14:27 |
| pedronis | fgimenez: well there are various options | 14:27 |
| pedronis | fgimenez: sorry, I'm probably not understanding the question | 14:28 |
| fgimenez | pedronis: you suggested that, when the env vars are the same, we shouldn't download each snap and add them as --extra-snaps, with your previous comment all is clear now i think, when they are different we should download and as extra snaps right? | 14:30 |
| fgimenez | well except core, it should be added always as extra-snap because it's modified | 14:30 |
| pedronis | fgimenez: yes | 14:30 |
| fgimenez | pedronis: ok thank you, i'll push the changes shortly | 14:30 |
| pedronis | fgimenez: also you always get kernel from the store I suppose if we want | 14:31 |
| pedronis | whatever is easier really | 14:31 |
| fgimenez | pedronis: ok when KERNEL_CHANNEL is edge makes sense of course | 14:32 |
| kyrofa | mwhudson, thanks for sharing the LP build snippet, that could prove handy down the road | 14:33 |
| pedronis | fgimenez: sorry, what I mean is that probably we can always avoid using "snap download" for all 3 | 14:42 |
| pedronis | fgimenez: because we can always pass the channel of one to ubuntu-image if the other two have the same channel or are sideloaded anyway | 14:43 |
| === nacc_ is now known as nacc | ||
| elopio | slangasek: fgimenez takes care of the snapd bugs, I do the snapcraft ones. | 14:48 |
| fgimenez | pedronis: great thanks, on it | 14:56 |
| kyleN | jdstrand, hi we have a blocking issue just reported to snapcraft list subject: "Issues using dbus in strict". Can you please take a look? | 14:58 |
| morphis | Son_Goku, Pharaoh_Atem: so what is your call on the snapd-xdg-open integration? | 14:59 |
| Son_Goku | morphis: for now, actually package it | 14:59 |
| Son_Goku | it's got a review request, just finish that | 14:59 |
| morphis | even if we have to drop that package soon after it? | 14:59 |
| davidcalle | @niemeyer hey, I haven't received the confirmation email for discourse, can you help? | 15:01 |
| nothal | davidcalle: No such command! | 15:01 |
| Son_Goku | morphis: you really want to merge more things into snapd? | 15:01 |
| morphis | Son_Goku: why not? | 15:02 |
| davidcalle | niemeyer: ^ | 15:02 |
| morphis | naturally that is where it belongs to | 15:02 |
| Son_Goku | not really | 15:02 |
| Son_Goku | it's not like snap-confine where it's actually tightly coupled with it | 15:02 |
| morphis | so it needs to follow the same release process, needs the same test setup etc. | 15:02 |
| morphis | only makes sense to share that with the existing snapd infrastructure | 15:02 |
| Son_Goku | fine, whatever | 15:03 |
| Son_Goku | fuck it all, but I wouldn't add it until you've already merged snapd-xdg-open into the master | 15:03 |
| Son_Goku | it's not worth arguing over this... | 15:04 |
| morphis | sure, I am just trying to get this whole thing into a good shape and polished | 15:04 |
| Son_Goku | morphis: propose the patch merge and get niemeyer or mvo to merge it in today, then *after*, I will backport it | 15:05 |
| morphis | Son_Goku: the PR is already there, but we wont get it in by today | 15:05 |
| morphis | Son_Goku: if you say its fine to take a package in and pull it out afterwards from fedora that is ok for me | 15:05 |
| morphis | just trying to safe us necessary work | 15:06 |
| Son_Goku | if you're absolutely certain that this will be merged in this way, I'll do it this way | 15:06 |
| morphis | let me see if I get a final vote from niemeyer today | 15:06 |
| morphis | niemeyer: ping | 15:06 |
| mup | PR snapd#3118 closed: cmd: import snapd-xdg-open and integrate with our infrastructure <Created by morphis> <Closed by niemeyer> <https://github.com/snapcore/snapd/pull/3118> | 15:07 |
| niemeyer | morphis, Son_Goku: I need to have lunch now or will miss the family, but I think we need to fix the situation of xdg-open in a proper way | 15:07 |
| niemeyer | I'll follow through in the forum after lunch | 15:07 |
| ogra | ppisati, will we get an explanation on bug 1674509 why it is invalid for the kernel ? | 15:08 |
| mup | Bug #1674509: Unable to find bluetooth device on RPi3 running Ubuntu Core 16 <Snappy:Confirmed> <linux-raspi2 (Ubuntu):Invalid by p-pisati> <https://launchpad.net/bugs/1674509> | 15:08 |
| morphis | niemeyer: thanks! | 15:08 |
| ogra | ppisati, ah, sorry for being impatient :) | 15:09 |
| morphis | zyga: hey! what is your final call of integrating snapd-xdg-open into snapd? | 15:09 |
| morphis | zyga: were just talking with Son_Goku about it and if we should push a separate snapd-xdg-open package into fedora if we're going to change this with a next release anyway | 15:10 |
| morphis | Pharaoh_Atem, King_InuYasha: btw. feel free to put your points to the discussion at https://forum.snapcraft.io/t/integrate-snapd-xdg-open-into-snapd-repository/100/ | 15:11 |
| slangasek | elopio: ah, thanks for the info. In that case, fgimenez: LP: #1626359 is linked from the changelog of the SRU for snapd 2.23.6, but has no SRU template. Will someone be adding test cases to these bugs, or should the source be reuploaded without the bug refs? | 15:11 |
| mup | Bug #1626359: Cannot authorise quotactl syscall for Q_GETQUOTA <snapd-interface> <verification-done> <Snappy:In Progress by jdstrand> <snapd (Ubuntu):Triaged> <snapd (Ubuntu Trusty):Triaged> <snapd (Ubuntu Xenial):Triaged> <snapd (Ubuntu Yakkety):Triaged> <https://launchpad.net/bugs/1626359> | 15:11 |
| fgimenez | hey slangasek, i think jibel is the right person to take a decision about this, he and his team are now working on snapd's SRUs | 15:17 |
| slangasek | jibel: ping ;) | 15:18 |
| fgimenez | jibel: could you please take a look to bug #1626359 when you have a moment? | 15:20 |
| mup | Bug #1626359: Cannot authorise quotactl syscall for Q_GETQUOTA <snapd-interface> <verification-done> <Snappy:In Progress by jdstrand> <snapd (Ubuntu):Triaged> <snapd (Ubuntu Trusty):Triaged> <snapd (Ubuntu Xenial):Triaged> <snapd (Ubuntu Yakkety):Triaged> <https://launchpad.net/bugs/1626359> | 15:20 |
| fgimenez | slangasek: sorry for the annoyance :) | 15:20 |
| jibel | fgimenez, slangasek how can I help? | 15:24 |
| jibel | fgimenez, last time mvo reuploaded snapd without bug references | 15:24 |
| slangasek | jibel: I can handle the reupload to remove the bug ref, I just need a decision that this is what is wanted | 15:25 |
| jibel | slangasek, yes, that's what mvo did for a previous release | 15:25 |
| slangasek | I think this one was missed by whatever scripts he uses to clean the changelog because the 'LP' and bug # were split across lines | 15:29 |
| jibel | slangasek, so go ahead, and reupload without the bug ref. It's the only bug mentioned in the changelog | 15:29 |
| slangasek | jibel: already done | 15:32 |
| jibel | thanks | 15:32 |
| === ycheng-afk is now known as ycheng | ||
| lutostag | hey all, curious if this is intended... paste (http://pastebin.ubuntu.com/24288480/) tracking is set to stable even if I specify a revision to pin to | 15:38 |
| lutostag | (so that when I snap refresh it gets stable instead of specifically rev'd version I installed) | 15:39 |
| mup | PR snapcraft#1226 opened: Better check of the error code in StatusTracker <Created by facundobatista> <https://github.com/snapcore/snapcraft/pull/1226> | 15:39 |
| mup | PR snapcraft#1193 closed: repo: track per-part build-packages <Created by josepht> <Merged by kyrofa> <https://github.com/snapcore/snapcraft/pull/1193> | 15:42 |
| mup | PR snapcraft#1212 closed: store: better retry strategy for GETs <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1212> | 15:54 |
| Pharaoh_Atem | morphis: you're not going to like what I have to say about this | 15:58 |
| Chipaca | in fixing this error message, i've hit an issue, maybe: if you say “snap install foo” is the message “error: "foo": access denied (try with sudo)” better, or worse, for having “"foo"” in there? | 16:07 |
| qengho | Chipaca: If the "foo" part is not the cause of the problem, then it's worse. If changing "foo" to something else would succeed, or you need to distinguish between "foo" an a "bar" in the same place, then it'OCs better to keep. | 16:10 |
| Chipaca | yeah, that was my reading of it too | 16:11 |
| Chipaca | it make syou think the fact that it's "foo" is the cause of your disgrace | 16:11 |
| Chipaca | “maybe if I try "bar" it'll install” | 16:11 |
| Chipaca | on the *other* hand, if you say “snap install foo bar” and you get back “error: "bar": configure hook failed”, that's super clear | 16:12 |
| Chipaca | and, alas, our errors aren't contextual enough to tell the difference between "this failed because you tried to do something unpossible" and "this failed because this step of this snap failed" | 16:13 |
| morphis | Pharaoh_Atem: :-) | 16:23 |
| morphis | Pharaoh_Atem: I am prepping the spec file nevertheless | 16:24 |
| mup | PR snapd#3124 opened: interfaces/mount: small tweaks <Created by zyga> <https://github.com/snapcore/snapd/pull/3124> | 16:35 |
| morphis | zyga: still there? | 16:39 |
| zyga | morphis: yep, I need to figoure out some better IRC story | 16:46 |
| zyga | IRC is so unreliable | 16:46 |
| zyga | and this client (irssi) is a compromise between crappiness and more crappiness | 16:47 |
| morphis | zyga: it is .. | 16:47 |
| morphis | zyga: thanks for merging! | 16:47 |
| CoderEurope | zyga, It's on the wishlist :D #WeeChat https://redd.it/62148w | 16:50 |
| Pharaoh_Atem | morphis, zyga: I've decided to be less scathing in my response: https://forum.snapcraft.io/t/integrate-snapd-xdg-open-into-snapd-repository/100/16?u=conan_kudo | 17:53 |
| Pharaoh_Atem | taking an hour away to eat lunch has made me feel less awful about this | 17:54 |
| zyga | ha | 18:00 |
| zyga | 19:58 < zyga> Pharaoh_Atem, morphis: how would you guys feel if we move to rocket | 18:00 |
| zyga | 19:58 < zyga> IRC is constantly disconnecting me (on 3/4G) | 18:00 |
| zyga | 19:59 < zyga> and I don't see messages or have any confidence my messages arrived | 18:00 |
| ogra | zyga should stop using 3/4G ... and use a full 1G network at least :) | 18:11 |
| Pharaoh_Atem | It's so exhausting dealing with so many different chat systems | 18:13 |
| Pharaoh_Atem | between Slack, Rocket, Gitter, IRC, Hangouts, and so many others I had to juggle... | 18:13 |
| ogra | well, it is always a good excuse to get another monitor for your desktop machine ... | 18:14 |
| ogra | need more space for $new_chat_tool | 18:14 |
| Pharaoh_Atem | I have three already | 18:14 |
| ogra | same here ... | 18:14 |
| Pharaoh_Atem | I don't think I can handle more | 18:14 |
| ogra | make it five and rotate them all ;) | 18:14 |
| ogra | gives you a bigger screen in the end | 18:14 |
| Pharaoh_Atem | if I were going to do that, I might as well get three 4K screens like some other people | 18:15 |
| mup | PR snapd#3124 closed: interfaces/mount: small tweaks <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/3124> | 19:09 |
| CoderEurope | https://www.youtube.com/watch?v=g4-tu-td5xU Is this the right youtube & is #snappy the right channel for discussion | in 10minutes - it starts. | 19:52 |
| kyrofa | CoderEurope, no, in rocket | 19:53 |
| CoderEurope | ah okay | 19:53 |
| kyrofa | https://rocket.ubuntu.com/channel/snapcraft | 19:53 |
| brunch875 | damn, I better haste with dinner | 19:54 |
| brunch875 | is popey going to host it? | 19:54 |
| kyrofa | I believe so | 19:59 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
