| mup | PR snapcraft#937 opened: Incorporate all part properties into state tracking <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/937> | 00:53 |
|---|---|---|
| josharenson | When I run refresh-bits ala git@github.com:zyga/devtools, I get an error about not having systemd installed? (running zesty) | 01:18 |
| mup | Bug #1646333 opened: bind mounts related to content interface plugs remain stale on snap disconnect/connect or snap updates <Snappy:New> <https://launchpad.net/bugs/1646333> | 03:46 |
| mup | PR snapd#2391 opened: Discard mount namespace on a content i/f plug connect/disconnect <Created by albaguirre> <https://github.com/snapcore/snapd/pull/2391> | 03:53 |
| === chihchun_afk is now known as chihchun | ||
| === Ian|zh_CN is now known as IanLi-AFK | ||
| foxmask | bonjello | 07:38 |
| mup | PR snapd#2390 closed: tests: do not use external snaps <Created by fgimenez> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/2390> | 07:57 |
| dholbach | hey hey | 07:57 |
| zyga | o/ | 07:57 |
| seb128 | hey dholbach zyga, how are you? | 07:58 |
| dholbach | salut seb128 | 07:59 |
| dholbach | doing all right - how about you? | 07:59 |
| seb128 | I'm good thanks :-) | 07:59 |
| zyga | seb128: hey, I'm good :-) how are you? | 08:49 |
| seb128 | zyga, I'm good thanks ;-) | 09:07 |
| tsdgeos | guys, snapcraft should never use one of my libs from /usr to end up in the snap but instead use the libs from the .deb files it downloads, no? | 09:32 |
| kalikiana | I would intuitively expect the .deb files to take priority | 09:35 |
| tsdgeos | because i think i found a case in which is not happening | 09:36 |
| tsdgeos | which i have to say was unexpected | 09:36 |
| Chipaca | tsdgeos: that's a bug | 09:38 |
| Chipaca | tsdgeos: without knowing the details I can't say whether it's a bug in snapcraft, or in your yaml | 09:38 |
| tsdgeos | Chipaca: how would the yaml make that happen? | 09:41 |
| Chipaca | tsdgeos: if the lib needed isn't in the deb you download (or maybe even if it's in the deb but needs the library path tweaked by postinst) | 09:42 |
| tsdgeos | ok | 09:44 |
| mup | Bug #1646415 opened: cannot run configure hook <Snappy:New> <https://launchpad.net/bugs/1646415> | 09:45 |
| mup | Bug #1646415 changed: cannot run configure hook <Snappy:Invalid> <https://launchpad.net/bugs/1646415> | 09:48 |
| mup | PR snapd#2392 opened: partition: add support for native grubenv read/write and use it <Created by mvo5> <https://github.com/snapcore/snapd/pull/2392> | 10:43 |
| caio1982 | ng | 11:02 |
| caio1982 | good morning | 11:02 |
| zyga | hey caio1982 | 11:02 |
| grigio | hi, I get ❯ env LANG=C ./electron-quick-start ./electron-quick-start: line 6: cd: /lib/node_modules/electron-quick-start: No such file or directory ./electron-quick-start: line 7: /bin/npm: No such file or directory | 11:11 |
| grigio | I tried this example http://bazaar.launchpad.net/~3v1n0/+junk/electron-quick-start-snap/files | 11:12 |
| renato__ | mvo, hi, could you approve this MR? https://myapps.developer.ubuntu.com/dev/click-apps/6062/rev/21/ | 11:19 |
| mvo | renato__: sure | 11:26 |
| renato__ | mvo, thanks | 11:27 |
| mup | PR snapd#2393 opened: interfaces/apparmor: use distinct apparmor template for classic <Created by zyga> <https://github.com/snapcore/snapd/pull/2393> | 11:45 |
| mup | PR snapd#2372 closed: interfaces/seccomp: add support for classic confinement <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/2372> | 11:52 |
| === __alecu__ is now known as alecu | ||
| renato__ | mvo, could you approve two more packages: https://myapps.developer.ubuntu.com/dev/click-apps/6019 rev 7 and 8 | 13:00 |
| renato__ | mvo, same for address-book-app: https://myapps.developer.ubuntu.com/dev/click-apps/6010 rev 19 and 20 | 13:12 |
| ondra | ogra_ ping | 13:21 |
| ogra_ | hey ondra | 13:21 |
| ondra | ogra_ hi, this would be simple one, in shell script, what is best reliable way to detect app is running as snap? | 13:22 |
| ogra_ | hmm ... check the path of the binary ? | 13:23 |
| ogra_ | something like that | 13:23 |
| ogra_ | there is a bunch of $SNAP_ variables you could look for | 13:23 |
| ondra | how about check is SNAP_COMMON exist? | 13:23 |
| ogra_ | SNAP_NAME or SNAP_VERSION .... | 13:23 |
| ondra | ogra_ is it something we will keep for foreseeable future? | 13:24 |
| ogra_ | i guess so | 13:24 |
| ondra | ogra_ OK, then I will stick to those :) | 13:24 |
| ondra | ogra_ cheers! | 13:24 |
| ogra_ | install hello-world, run hello-world.sh ... | 13:24 |
| ogra_ | then check the env | 13:24 |
| ogra_ | (or there is even a hello-world.env i think) | 13:25 |
| ondra | ogra_ yeah there is | 13:25 |
| mvo | jdstrand, renato__: I approved https://myapps.developer.ubuntu.com/dev/click-apps/6010/ - what is needed to make this automatic? | 13:31 |
| renato__ | mvo, I know that jdstrand is working on it. I am not sure about the status | 13:32 |
| mvo | renato__: aha, sure, thats fine then | 13:32 |
| renato__ | mvo, could you approve 6019 too? | 13:33 |
| mvo | renato__: I did, didn't i? | 13:34 |
| renato__ | mvo, no still blocked | 13:34 |
| mvo | renato__: oh, you are right | 13:36 |
| renato__ | mvo, thanks | 13:36 |
| icey | should I push a BS commit to trigger the autopgktests on this again, or maybe merge in master for the commit to get the fix from master? https://github.com/snapcore/snapcraft/pull/908 | 13:49 |
| mup | PR snapcraft#908: Let Rust plugin fetch dependencies in pull <Created by ChrisMacNaughton> <https://github.com/snapcore/snapcraft/pull/908> | 13:49 |
| renato__ | mvo, sorry to keep disturbing you, but 6019 rev 9 still blocked :( | 13:49 |
| jdstrand | renato__ (cc mvo): you need to request a manual review | 13:51 |
| jdstrand | renato__: it doesn't show up in the list unless you do that | 13:51 |
| renato__ | jdstrand, humm sorry I miss this one | 13:52 |
| jdstrand | renato__: and the person who pushes that button can't then approve it | 13:52 |
| jdstrand | (so it can't be us) | 13:52 |
| jdstrand | mvo: I handled it | 13:52 |
| renato__ | thanks | 13:52 |
| jdstrand | renato__: done | 13:52 |
| mvo | jdstrand: thanks, was in a meeting | 14:00 |
| === mup_ is now known as mup | ||
| flexiondotorg | sergiusens, kyrofa I've discovered a bug with source-type: deb | 14:12 |
| flexiondotorg | Want to find out if it is known. | 14:12 |
| flexiondotorg | If you use a deb as source and that deb has sym-links in it, the link names are created as directories in the snap. | 14:13 |
| flexiondotorg | In my use case the binary is trying to load a lib which is effectively doesn't exist. | 14:14 |
| === chihchun is now known as chihchun_afk | ||
| mup | PR snapd#2394 opened: snap: show last update time <Created by mvo5> <https://github.com/snapcore/snapd/pull/2394> | 14:23 |
| mup | PR snapcraft#938 opened: store: return specific error when already owned <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/938> | 14:26 |
| === jamespag` is now known as jamespage | ||
| alex-abreu | mvo, could you comment on https://github.com/snapcore/snapweb/pull/101 ? | 14:32 |
| mup | PR snapweb#101: replace find vs findone <Created by AlexandreAbreu> <https://github.com/snapcore/snapweb/pull/101> | 14:32 |
| mup | PR snapd#2395 opened: make state and squashes immutable when appropriate <Created by chipaca> <https://github.com/snapcore/snapd/pull/2395> | 14:36 |
| mvo | alex-abreu: let me have a look | 14:39 |
| timp | jdstrand: hello, I have a question about this bug https://bugs.launchpad.net/ubuntu-ui-toolkit-examples/+bug/1645377 | 14:47 |
| mup | Bug #1645377: AppArmor policy error for networking at initialization, even with the correct network plug. <snapd-interface> <Snappy:Invalid> <ubuntu-ui-toolkit-examples:New> <https://launchpad.net/bugs/1645377> | 14:47 |
| timp | jdstrand: that app just wants to download something from internet, for that it should not need to use the network-manager interface right? | 14:48 |
| timp | any ideas why it currently needs that? | 14:48 |
| renato__ | popey, hey, could you review this? https://code.launchpad.net/~renatofilho/ubuntu-calculator-app/unity8-snap/+merge/312260 | 14:54 |
| timp | renato__: why do you need unity7 and unity8? | 14:56 |
| renato__ | timp, the unity8 session that tedg is working needs unity8 interface. And unity7 is to keep it compatible with unity7 | 14:57 |
| timp | renato__: ah, ok. Will we need to add unity8 to all GUI apps? | 14:59 |
| renato__ | timp, I think so | 14:59 |
| timp | renato__: hmm.. and all GUI apps also need opengl I guess. So maybe we should find out if it is possible for [platform] to also imply [opengl, unity7, unity8] | 15:00 |
| timp | I don't know if it would work if we simply add those plugs to the ubuntu-app-platform snap. | 15:00 |
| timp | sergiusens: ^how would we do that? | 15:02 |
| renato__ | timp, I am not sure about that. I think that is ok to explicitly say which interface you app request. S | 15:02 |
| renato__ | timp, and some apps can be ready only for unity7 | 15:02 |
| timp | renato__: but if you use the platform snap, you'll need all those plugs | 15:03 |
| renato__ | timp, some apss can uses plataform and does not have any ui | 15:03 |
| timp | renato__: ah, that's true. | 15:03 |
| timp | hmm | 15:03 |
| timp | well the core of the platform (at least the dependencies that we put in there first) is the UITK | 15:03 |
| renato__ | timp, i can have a app that only uses qtcore or network, etc.. | 15:04 |
| timp | hmm | 15:04 |
| timp | yeah | 15:04 |
| timp | but would you use the platform snap then? Platform has a lot more than just qt. | 15:04 |
| timp | but if you don't want to compile qt, then currently the platform snap is what you need to use. | 15:04 |
| renato__ | timp, if the plataform is part of the system already. I do not see why not use that | 15:04 |
| timp | That really depends on the use case. | 15:07 |
| renato__ | if I have a gadget that only runs my app problably I can pack qt on it. | 15:07 |
| timp | yes | 15:07 |
| timp | renato__: ok. We can leave it as it is for now then :) | 15:07 |
| renato__ | popey, one more: https://code.launchpad.net/~renatofilho/ubuntu-clock-app/unity8-snap/+merge/312263 | 15:09 |
| jdstrand | timp: you are right, it shouldn't need network-manager. It is (almost certainly) only accessing network-manager to see if the network is available. this is a very longstanding conversation regarding Qt and network-manager and is precisely why connectivity-api was developed. see bug #1341548 | 15:13 |
| mup | Bug #1341548: Online detection does not work with confined apps on Nexus 4 <rtm14> <touch-2014-08-21> <Dekko:Fix Released by dpniel> <Network Menu:Fix Released by kaijanmaki> <apparmor-easyprof-ubuntu (Ubuntu):Fix Released by jdstrand> <connectivity-api (Ubuntu):Fix Released by kaijanmaki> | 15:13 |
| mup | <indicator-network (Ubuntu):Fix Released by kaijanmaki> <connectivity-api (Ubuntu RTM):Fix Released by kaijanmaki> <indicator-network (Ubuntu RTM):Fix Released by kaijanmaki> <https://launchpad.net/bugs/1341548> | 15:13 |
| timp | jdstrand: so I will be required to use connectivity-api whenever I access the network, even when I use the [network] plug already? | 15:17 |
| timp | in which channel should I ask about snap store reviews and automatic publishing of LP projects to the store? | 15:24 |
| timp | maybe here? | 15:24 |
| ogra_ | here :) | 15:25 |
| timp | Ok. :) I configured https://launchpad.net/~ubuntu-sdk-team/+snap/ubuntu-ui-toolkit-examples to automatically upload a snap on the edge channel in the store, but on https://myapps.developer.ubuntu.com/dev/click-apps/6427/rev/3/ it appears in the Release channel | 15:25 |
| timp | ^is that a bug? | 15:25 |
| zyga | renato__: no idea, sorry | 15:25 |
| zyga | renato__: did you trace the calls from that failure back to base policy? | 15:25 |
| timp | ignore my question. It was already answered somewhere else <noise> timp: that's not a channel name, that's the action button to release it into channels | 15:26 |
| ogra_ | heh | 15:26 |
| ogra_ | i was about to ask what you mean with release channel | 15:26 |
| zyga | ogra_: new channels, there's one every weekend ;) | 15:27 |
| zyga | matrix keeps changing | 15:27 |
| ogra_ | heh | 15:27 |
| timp | ogra_: right :) | 15:27 |
| timp | who can I kick to review the ubuntu-ui-toolkit-examples snap in the store? | 15:28 |
| sborovkov | Hello. snap store seems to be very slow. I get constant timeouts when clicking on revision? Any idea what's up with it? | 15:28 |
| jdstrand | timp: if your app is trying to check if the network is available, it will need to plugs something else, yes. if it is using network-manager, it needs to plugs network-manager (that will require a snap interface manual connect), if it is using connectivity-api, it needs to plugs it (that will be auto-connected) | 15:30 |
| jdstrand | timp: note that connectivity-api interface isn't available yet. it is part of the Ubuntu Personal work. I don't know the status | 15:30 |
| timp | jdstrand: so if you use network you will always need network-manager or connectivity-api? | 15:31 |
| jdstrand | zyga: fyi, you mentioned not looking for other occurences of the i2c bug. I just did. i2c is the only thing not using utils.go and utils.go is fine | 15:31 |
| timp | I assume when you use the network, it will check whether that's available. | 15:31 |
| jdstrand | timp: plenty of apps need only 'plugs: [network]'. your app is trying to be smart and as a result it needs to plugs something else | 15:32 |
| jdstrand | (in addition to network) | 15:32 |
| zyga | jdstrand: good to know, thank you | 15:33 |
| jdstrand | timp: in Ubuntu Touch, we put connectivity-api in the network policy group. the core snap does not have the connectivity-api daemon in it, so it will be provided via a snap. as such, you will need to 'plugs: [network, connectivity-api]' on snappy (whenever connectivity-api is available) | 15:33 |
| zyga | jdstrand: I'm iching to rewrite interfaces internally as we discussed, maybe over xmas :) | 15:33 |
| zyga | itching* | 15:33 |
| jdstrand | zyga: heh, please make sure that the policy remains easily auditable within the codebase. that's all tyhicks and I are asking :) | 15:34 |
| timp | jdstrand: This is the full app http://bazaar.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk/view/head:/examples/jokes/jokes.qml | 15:35 |
| zyga | jdstrand: as agreed :) | 15:35 |
| timp | It does not seem to do anything smart, I guess the XMLHttpRequest(); does that then. | 15:35 |
| zyga | jdstrand: I'd definitely do that to ensure my sanity :) | 15:35 |
| timp | I'm just thinking how to make it easy for developers to use that. | 15:35 |
| timp | I don't know the details of what the XMLHttpRequest object does, just trying to snap the UITK examples. | 15:36 |
| jdstrand | timp: if this is for qtubuntu apps, seems reasonable to default the snapcraft.yaml to use 'plugs: [network, connectivity-api]' just like for click it use '"policy_groups": ["network"]' | 15:37 |
| jdstrand | timp: in the sdk | 15:37 |
| timp | jdstrand: yes, it is a qtubuntu app (using the ubuntu-app-platform snap) with "network" int he policy groups. | 15:40 |
| timp | jdstrand: I'll add a note in my snapcraft.yaml to add that when the connectivity-api is done. Let's see if I can find the bug for that. | 15:41 |
| Odd_Bloke | ogra_: Still looking at this week for that Azure bug? | 15:47 |
| ogra_ | Odd_Bloke, bug 1639878 ? | 15:48 |
| mup | Bug #1639878: pc-kernel.snap missing drivers necessary for Hyper-v <Snappy:Fix Committed by ogra> <https://launchpad.net/bugs/1639878> | 15:48 |
| Odd_Bloke | ogra_: Yep. | 15:48 |
| ogra_ | (new kernel snap is in edge with the modules included) | 15:48 |
| Odd_Bloke | Oh, I must not be subscribed, I hadn't seen updates. | 15:48 |
| Odd_Bloke | Fail. | 15:48 |
| ogra_ | not sure when the next release is planned though ... mvo might know | 15:49 |
| ogra_ | then it should hit stable | 15:49 |
| mvo | ogra_: next release is next thursday | 15:51 |
| mvo | Odd_Bloke: -^ | 15:51 |
| ogra_ | :) | 15:51 |
| Odd_Bloke | Cool, thank you both! | 15:52 |
| mvo | ogra_: I think we can bump the kernel to beta now (if it booted) and then I can ask qa to test it | 15:52 |
| ogra_ | mvo, yeah, though we are missing an updated rpi2 one it seems | 15:53 |
| ogra_ | dragon and pc broth had updates since my vacation | 15:53 |
| ogra_ | pi not | 15:53 |
| mvo | ok | 15:54 |
| ogra_ | not sure what keeps it in proposed ... ppisati ? | 15:54 |
| ogra_ | but we can indeed push the others up to beta | 15:54 |
| mvo | ogra_: thank you! | 15:57 |
| alex-abreu | mvo, btw this wont be added back to 2.18 right ? https://github.com/snapcore/snapd/pull/2394 | 15:58 |
| mup | PR snapd#2394: snap: show last update time <Created by mvo5> <https://github.com/snapcore/snapd/pull/2394> | 15:58 |
| mvo | alex-abreu: correct, this is targeted for 2.19 | 15:59 |
| alex-abreu | mvo, ok thx | 15:59 |
| mvo | (which is just one week away :) | 15:59 |
| alex-abreu | mvo, even better thx :) | 15:59 |
| mup | PR snapd#2396 opened: tests: fix incorrect restore of the current symlink <Created by mvo5> <https://github.com/snapcore/snapd/pull/2396> | 16:31 |
| zyga | jdstrand: can you join a call about classic confinement and the store now? | 16:32 |
| kyrofa | flexiondotorg, just ANY symlink? | 16:34 |
| kyrofa | flexiondotorg, not known by me anyway | 16:34 |
| kyrofa | flexiondotorg, ah, wait, LP: #1634813 | 16:35 |
| mup | Bug #1634813: Symbolic links inside .deb pulled as directories <sources> <Snapcraft:Confirmed> <https://launchpad.net/bugs/1634813> | 16:35 |
| flexiondotorg | kyrofa, Yep, that is the issue. | 16:35 |
| flexiondotorg | I'll subscribe to that. | 16:35 |
| flexiondotorg | I've got a fix in progress. | 16:36 |
| kyrofa | flexiondotorg, thanks for the poke. Ah! You know what the issue is? | 16:36 |
| flexiondotorg | kyrofa, The issue is apt_install.extractall() doesn't do the right thing. | 16:36 |
| flexiondotorg | The Tar() class in internal/sources.py does the right thing, so can be the basis of a fix. | 16:37 |
| kyrofa | Good deal. Happy to investigate if you're short on time, let me know? | 16:37 |
| flexiondotorg | Well, I'll be leaving the office in a bit. | 16:38 |
| kyrofa | flexiondotorg, alright I'll take a look once I'm done with my current PR | 16:39 |
| flexiondotorg | kyrofa, Thanks. | 16:39 |
| flexiondotorg | I'm just finishing something else too. | 16:39 |
| kyrofa | flexiondotorg, thanks for the pointers :) | 16:40 |
| flexiondotorg | In the Tar() class you'll see the _extract() method. That is what I was going to reimplement for the Deb() class. | 16:40 |
| bitpushr | Howdy all - setting up snappy, however the key on my ubuntu one account was dsa - and I can't SSH in with it because it is a rejected protocol. Anyway to trigger the snappy device to redownload my ssh public keys? | 16:44 |
| bitpushr | (without logging in or running a fresh install) | 16:45 |
| kyrofa | ogra_, do you know the answer to that question? ^^ | 16:50 |
| ogra_ | nope | 16:57 |
| ogra_ | i saw the question before ... have never used a dsa key | 16:57 |
| ogra_ | (i think someone asked the same at the beginning of the week) | 16:57 |
| ogra_ | kyrofa, sounds more liek a question to some store person :) | 16:59 |
| ogra_ | *like | 16:59 |
| kyrofa | ogra_, indeed, thank you. nessita, are you around? | 17:00 |
| kyrofa | mvo, you might be able to help with that as well. Is there any way to trigger Ubuntu Core to re-download SSH keys? 1) as a logged in user, and 2) without being able to login? | 17:01 |
| nessita | kyrofa, around otp, could you repeat the question? | 17:08 |
| jdstrand | niemeyer: can you meet now for snap decl talk>? | 17:10 |
| niemeyer | jdstrand: Yep, is your mic working agian? | 17:10 |
| niemeyer | https://hangouts.google.com/hangouts/_/canonical.com/base-declaration | 17:10 |
| jdstrand | niemeyer: I'll get it to work. can you start a new hangout and I'll join? | 17:10 |
| jdstrand | ok | 17:10 |
| jdstrand | niemeyer: it says I am the first to join | 17:12 |
| kyrofa | nessita, sure thing-- bitpushr had a dsa key on his SSO, so when Ubuntu Core fetched the keys that's all it got, and it doesn't work. If he's since uploaded an rsa key, do you know if there's a way to re-trigger the key download? | 17:12 |
| nessita | kyrofa, hum, I have no idea, the store is not involved in that part | 17:22 |
| nessita | other than providing an API to get all the public sshs for an emai; | 17:22 |
| nessita | email* | 17:22 |
| kyrofa | nessita, yeah, me neither :( . Alright, thanks for your time! | 17:23 |
| nessita | kyrofa, :-) | 17:33 |
| bitpushr | kyrofa: you have same issue? | 17:33 |
| pedronis | kyrofa: refreshing keys is still a TODO afaik | 17:33 |
| kyrofa | bitpushr, I'm afraid not, I use rsa keys | 17:33 |
| bitpushr | It would be a good idea to refresh them on boot at least | 17:34 |
| bitpushr | I thought it might but it does not | 17:34 |
| kyrofa | pedronis, so what is the path forward. A reinstall? | 17:34 |
| pedronis | I don't know | 17:34 |
| bitpushr | No worries, but I would like to put that in as a feature request. Would also help handle expired keys, updated keys, etc | 17:35 |
| bitpushr | This was an old key I had | 17:35 |
| sergiusens | flexiondotorg will look into it | 17:35 |
| sergiusens | flexiondotorg but we are using the python deb extraction implementation | 17:36 |
| renato__ | jdstrand, could you approve messaging-app package in the store: https://myapps.developer.ubuntu.com/dev/click-apps/6192 rev 4 e 5 | 17:41 |
| pedronis | bitpushr: I created this https://bugs.launchpad.net/snappy/+bug/1646559 | 17:45 |
| mup | Bug #1646559: should periodically refresh ssh keys that were obtained from SSO/store for local users <Snappy:New> <https://launchpad.net/bugs/1646559> | 17:45 |
| mup | Bug #1646559 opened: should periodically refresh ssh keys that were obtained from SSO/store for local users <Snappy:New> <https://launchpad.net/bugs/1646559> | 17:45 |
| bitpushr | Thank you pedronis | 18:00 |
| jdstrand | renato__: looking now. sorry was in meetings | 18:07 |
| renato__ | jdstrand, np. thanks | 18:12 |
| zyga | tyhicks, jdstrand: https://github.com/snapcore/snap-confine/pull/199 | 18:40 |
| mup | PR snap-confine#199: Add error support code <Created by zyga> <https://github.com/snapcore/snap-confine/pull/199> | 18:40 |
| zyga | this is a prelude to https://github.com/snapcore/snap-confine/commit/f7ee6df44916109731ce8a611ff57d9a70954b56 | 18:41 |
| zyga | and on top a small patch that adds --classic support | 18:42 |
| mup | PR snapcraft#900 closed: ftp source support <Created by 3v1n0> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/900> | 18:44 |
| zyga | which is https://github.com/snapcore/snap-confine/commit/6f289caa3a7adcfcedef25c13378fb9bd5de2f95 | 18:47 |
| mup | PR snapcraft#939 opened: Replace coveralls with codecov <Created by elopio> <https://github.com/snapcore/snapcraft/pull/939> | 18:47 |
| zyga | I got a +0.5 from mvo on telegram, I'd love a review from you | 18:47 |
| zyga | one more branch and I'll switch to that for parsing command line | 18:47 |
| zyga | and then (no-op) classic can be dput to a image ppa | 18:48 |
| mup | PR snapd#2387 closed: asserts: introduce auto-aliases header in snap-declaration <Critical> <Created by pedronis> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/2387> | 18:48 |
| mup | PR snapd#2396 closed: tests: fix incorrect restore of the current symlink <Created by mvo5> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/2396> | 18:51 |
| mup | PR snapd#2369 closed: snap: disable support for socket activation <Created by stolowski> <Merged by niemeyer> <https://github.com/snapcore/snapd/pull/2369> | 18:52 |
| mup | PR snapcraft#940 opened: Implement delta uploads in push <Created by squidsoup> <https://github.com/snapcore/snapcraft/pull/940> | 19:26 |
| alex-abreu | niemeyer, do you have an update on the snapd-control & install/connect issue for local snaps? | 19:58 |
| jdstrand | alex-abreu (cc niemeyer): we had a meeting today. we are getting close to having an answer | 20:02 |
| alex-abreu | jdstrand, ok it will be backported to 2.18 right ? | 20:03 |
| jdstrand | alex-abreu: that is the intent, yes | 20:03 |
| alex-abreu | jdstrand, thank you, ... any eta? | 20:04 |
| jdstrand | alex-abreu: well, no since we don't have the answer yet | 20:04 |
| jdstrand | but getting there. once we've worked through the answer, someone will send something to the list | 20:05 |
| alex-abreu | jdstrand, ok, ... thank you | 20:05 |
| icey | is there anny chance of (non-root) installable snaps? | 20:05 |
| kyrofa | icey, probably not, at least not in the near future | 20:09 |
| tedg | jdstrand: Are the snap confine tools closing all the file descriptors? | 21:30 |
| tedg | jdstrand: Trying to pass one through and it's not making it... | 21:30 |
| zyga | jdstrand: hey | 21:37 |
| jdstrand | tedg: not that I can see | 21:37 |
| zyga | jdstrand: wow, you are here :) | 21:37 |
| tedg | jdstrand: Hmm, okay, must be messing something else up :-) | 21:38 |
| zyga | tedg: no, but remember there are snap-run snap-exec there as well | 21:38 |
| zyga | tedg: being in go they may do that in the runtime | 21:38 |
| zyga | tedg: can you show me a reprouction test case? | 21:38 |
| tedg | Uhg, yeah. | 21:38 |
| tedg | Not really right now, I have things heavily modified to recreate it. | 21:38 |
| zyga | jdstrand: hey, is there a chance for you to look at https://github.com/snapcore/snap-confine/pull/199 today | 21:38 |
| mup | PR snap-confine#199: Add error support code <Created by zyga> <https://github.com/snapcore/snap-confine/pull/199> | 21:38 |
| zyga | tedg: even a small test case I can run locally would be great | 21:39 |
| zyga | tedg: e..g something hacked in shell | 21:39 |
| tedg | zyga: Let me chase down a couple other threads first, might be me screwing things up as well. | 21:40 |
| mup | Bug #1646625 opened: on first boot rpi2 cannot configure (core16) <Snappy:New> <https://launchpad.net/bugs/1646625> | 21:41 |
| zyga | tedg: that's ok, it's pretty late for me anyway, if you share something I can tinker with I'll try to help you out tomorrow | 21:41 |
| jdstrand | zyga: looking at that PR I would want to spend some time with it (more than I have this afternoon). I can look at it first thing tomorrow | 21:42 |
| jdstrand | zyga: if you need it sooner, perhaps tyhicks could look at it, but I don't know what he has going on | 21:42 |
| zyga | jdstrand: thanks, if you want more context look at https://github.com/snapcore/snap-confine/branches (specifically https://github.com/snapcore/snap-confine/tree/arguments ) | 21:42 |
| zyga | https://github.com/snapcore/snap-confine/commit/a84be35d70be79b6f92dd192d40a82f6f0717856 | 21:43 |
| zyga | this patch | 21:43 |
| jdstrand | zyga: what is the motivation of this patch? | 21:46 |
| tyhicks | zyga: hey - those look a little more like code churn than a PR that we should drop other stuff and review today | 21:46 |
| tyhicks | maybe I'm missing something | 21:47 |
| tyhicks | (it is really nice to have non-fatal error message handling) | 21:48 |
| zyga | tyhicks: it's okay, you don't have to review it today | 21:48 |
| jdstrand | tyhicks: well, fatal message handling has the nice benefit that you know you fail closed | 21:48 |
| zyga | jdstrand: argument parsing is going to grow a little and I didn't want to add more hackery to main() | 21:48 |
| zyga | jdstrand: and I wanted to write code that can follow one style of error handling and die()/death in proper places | 21:49 |
| tyhicks | jdstrand: that's true | 21:49 |
| zyga | jdstrand: and lastly I wanted something that can be unit tested a little bit better | 21:49 |
| jdstrand | zyga: what arguments are being added? | 21:49 |
| zyga | jdstrand: --classic | 21:49 |
| zyga | jdstrand: but I be we'll need more soon | 21:49 |
| zyga | jdstrand: e.g. --base to switch base snaps | 21:49 |
| zyga | jdstrand: perhaps --revision to not care about current symlinks | 21:50 |
| * zyga EODs | 22:09 | |
| zyga | good night everyone | 22:09 |
| mup | PR snapcraft#937 closed: Incorporate all part properties into state tracking <Created by kyrofa> <Closed by kyrofa> <https://github.com/snapcore/snapcraft/pull/937> | 22:14 |
| mup | PR snapcraft#937 opened: Incorporate all part properties into state tracking <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/937> | 23:08 |
| mup | Bug #1545871 opened: Be able to query with multiple terms <Snappy:New for chipaca> <https://launchpad.net/bugs/1545871> | 23:39 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!