| kyrofa | elopio, some test failures on snapcraft#1853 | 00:23 |
|---|---|---|
| mup | PR snapcraft#1853: extractors: support appstream icon and desktop <Created by elopio> <https://github.com/snapcore/snapcraft/pull/1853> | 00:23 |
| === AmarokNelg_ is now known as AmarokNelg | ||
| mborzecki | morning | 06:00 |
| zyga-ubuntu | mborzecki: o/ | 06:30 |
| mborzecki | zyga-ubuntu: hey | 06:30 |
| zyga-ubuntu | https://pastebin.ubuntu.com/26370425/ dragonboard 2.30 test results | 06:31 |
| mborzecki | ouch, that took a while | 06:34 |
| kalikiana | morning | 07:14 |
| zyga-ubuntu | o/ | 07:14 |
| zyga-ubuntu | hello mvo | 07:27 |
| zyga-ubuntu | https://pastebin.ubuntu.com/26370425/ dragonboard 2.30 test results | 07:28 |
| mvo | zyga-ubuntu: nice! thanks, I have a look at this one failure. its strange because this test worked on all the other platforms | 07:28 |
| mvo | zyga-ubuntu: we may need to debug this interactively later today (if you have the time for it, if not monday is fine as well) | 07:29 |
| zyga-ubuntu | mvo: it looks like it never passed on DB | 07:30 |
| mborzecki | mvo, kalikiana: morning | 07:31 |
| mup | PR snapd#4479 closed: spread: maybe dnf needs a refresh? <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/4479> | 07:41 |
| mvo | zyga-ubuntu: interessting | 07:41 |
| mvo | mborzecki: good morning | 07:41 |
| mborzecki | mvo: i've restarted the build in #4380 when it's green, once we land it i'd like extend the test a bit adding more directories there | 08:02 |
| mup | PR #4380: tests: add simple snap-mgmt test <Created by mvo5> <https://github.com/snapcore/snapd/pull/4380> | 08:02 |
| mvo | mborzecki: sure, its all yours | 08:03 |
| mborzecki | mvo: zyga-ubuntu: is #4357 good to be merged? | 08:05 |
| mup | PR #4357: wrappers: autogenerate After/Before in systemd's service files for apps <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/4357> | 08:05 |
| mvo | mborzecki: yes | 08:07 |
| mborzecki | mvo: thanks, i'll look into adding some spread tests | 08:08 |
| mup | PR snapd#4357 closed: wrappers: autogenerate After/Before in systemd's service files for apps <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4357> | 08:08 |
| mborzecki | pstolowski: morning | 08:18 |
| mvo | koza: hi - could you do a quick sanity check of https://github.com/snapcore/snapd/pull/4448 - does that make sense in the bluez context (your are our bluez expert, right .) ? | 08:23 |
| mup | PR #4448: Add rules for Media API to the BlueZ D-Bus policy <Created by lhartung> <https://github.com/snapcore/snapd/pull/4448> | 08:23 |
| zyga-ubuntu | mvo: good idea :) | 08:24 |
| mvo | zyga-ubuntu: also added jdstrand_ to pr #4449 but hopefully a no-brainer | 08:24 |
| mup | PR #4449: cmd/libsnap-confine-private, cmd/snap-confine: fix logging of failed [u]mount when run with SNAP_CONFINE_DEBUG=1 <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/4449> | 08:24 |
| pstolowski | good morning! | 08:25 |
| * zyga-ubuntu cannot wake up today | 08:26 | |
| mborzecki | zyga-ubuntu: more coffee | 08:28 |
| * pstolowski rebooting for new kernel update | 08:28 | |
| pstolowski | good, no more panics on shutdown with 4.4.0-109, whatever was broken with -108 apparently got fixed | 08:33 |
| zyga-ubuntu | niemeyer: good morning | 08:56 |
| zyga-ubuntu | niemeyer: I hope you're around today :) | 08:57 |
| niemeyer | zyga-ubuntu: I hope so as well.. still not 100% sure I'm here or if I'm still sleeping | 08:59 |
| zyga-ubuntu | niemeyer: I feel the same thing despite supposedly being awake for some time now | 08:59 |
| mup | PR snapd#4380 closed: tests: add simple snap-mgmt test <Created by mvo5> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/4380> | 08:59 |
| zyga-ubuntu | niemeyer: let's reuse the standup HO, I don't know how to add event-specific hangouts anymore (new calendar UI) | 08:59 |
| zyga-ubuntu | jamesh: good evening | 09:00 |
| niemeyer | zyga-ubuntu: I'm already online on dekstop-snapd | 09:00 |
| jamesh | zyga-ubuntu: hi | 09:00 |
| zyga-ubuntu | jamesh: let's use this link https://hangouts.google.com/hangouts/_/canonical.com/desktop-snapd | 09:00 |
| * Chipaca yawns | 09:05 | |
| mborzecki | tests/main/auto-refresh-private fails on master https://pastebin.canonical.com/207451/ | 09:41 |
| Chipaca | mborzecki: looking | 09:45 |
| mborzecki | looking at snap-mgmt --purge, i suppose we should stop systemd services that came from snaps and clean up the *.service files as well | 09:50 |
| mvo | mborzecki: +1 | 09:51 |
| * Chipaca -> physio | 10:08 | |
| Chipaca | mborzecki: that spread test passed here -- maybe a store glitch? | 10:08 |
| mborzecki | possible, let's see if it comea up again | 10:09 |
| pstolowski | zyga-ubuntu, a few comments to 4471 | 10:27 |
| zyga-ubuntu | thank you | 10:28 |
| pstolowski | mborzecki, and to 4476 | 10:28 |
| mborzecki | pstolowski: thanks | 10:28 |
| justasking | hi everybody, what is the standard procedure for troubleshooting the following situation: | 10:48 |
| justasking | snap installed | 10:48 |
| justasking | snap/programm appears in menu structure | 10:48 |
| justasking | but won't run | 10:49 |
| justasking | program in question is shotcut | 10:49 |
| zyga-ubuntu | justasking: open a termina and "snap run nameofthesnap" | 10:53 |
| sparkiegeek | http://downforeveryoneorjustme.com/forum.snapcraft.io | 11:11 |
| sparkiegeek | did that get covered already? | 11:11 |
| zyga-ubuntu | niemeyer: ^ | 11:12 |
| mvo | sparkiegeek: thanks! we were not aware of this until now | 11:13 |
| mup | PR snapd#4480 opened: tests/main/snap-mgmt: extend the test to cover more directories and files <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/4480> | 11:13 |
| pstolowski | pedronis, hey, do you have a moment for quick HO? | 11:14 |
| mup | PR snapd#4481 opened: image: let consume snapcraft export-login files from tooling <Created by pedronis> <https://github.com/snapcore/snapd/pull/4481> | 11:16 |
| sparkiegeek | ... and it's back? | 11:19 |
| pedronis | pstolowski: not now, after lunch yes | 11:22 |
| pstolowski | pedronis, ok, thanks | 11:22 |
| pedronis | trying to finish something | 11:22 |
| * Chipaca returns from physio, defeated | 11:35 | |
| * zyga-ubuntu hugs Chipaca for fighting | 11:35 | |
| zyga-ubuntu | that was a long but very useful call | 11:36 |
| zyga-ubuntu | ok, 10 minute break and I'll get back to fighting here | 11:36 |
| zyga-ubuntu | mvo: any issues with landings? | 11:36 |
| zyga-ubuntu | mvo: and shall I do something about dragonboard or are you happy with the results so far? | 11:37 |
| Chipaca | zyga-ubuntu: https://www.youtube.com/watch?v=-VsmF9m_Nt8 <- for your break | 11:38 |
| Chipaca | zyga-ubuntu: (just listen; the video is terribe) | 11:38 |
| mvo | zyga-ubuntu: if you could re-run this one failing command with -debug that would be cool but not urgent | 11:41 |
| zyga-ubuntu | mvo: sure thing | 11:48 |
| mup | PR snapd#4482 opened: cmd/libsnap-confine-private: add debug build of libsnap-confine-private.a, link it into snap-confine-debug <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/4482> | 11:55 |
| pedronis | pstolowski: I'm around now but you are probably having lunch yourself | 12:01 |
| pstolowski | pedronis, not yet, let's do a quick HO (standup ho) | 12:03 |
| pedronis | pstolowski: I'm there | 12:04 |
| * zyga-ubuntu enjoys fruit for a quick snack | 12:07 | |
| mup | PR snapd#4483 opened: cmd/libsnap-confine-private: print failed mount/umount regardless of SNAP_CONFINE_DEBUG <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/4483> | 12:12 |
| mup | PR snapd#4449 closed: cmd/libsnap-confine-private, cmd/snap-confine: fix logging of failed [u]mount when run with SNAP_CONFINE_DEBUG=1 <Created by bboozzoo> <Closed by bboozzoo> <https://github.com/snapcore/snapd/pull/4449> | 12:13 |
| * Chipaca starts to rage, then remembers he's probably hangry and goes to fix that | 12:19 | |
| zyga-ubuntu | is hangry like hungry and angry in one word | 12:19 |
| sparkiegeek | yup | 12:19 |
| Chipaca | yes | 12:19 |
| zyga-ubuntu | does it explain hangover? | 12:19 |
| sparkiegeek | https://www.urbandictionary.com/define.php?term=Hangry | 12:19 |
| Chipaca | zyga-ubuntu: it's the anger from being hungry | 12:19 |
| zyga-ubuntu | is the hangover the anger from the hunger and the drinking being over? | 12:20 |
| sparkiegeek | alas, no | 12:20 |
| * kalikiana going for trains/lunch in ~10 | 12:23 | |
| pedronis | pstolowski: btw, don't you a problem running hooks before the snap in in the snaps map in state? | 12:52 |
| pedronis | pstolowski: are you running autoconnect stuff just after setup-profiles? or after link-snap? | 12:53 |
| pstolowski | pedronis, hmm this can be a problem indeed. i didn't hit it yet because autoconnect and hooks branches are not integrated | 12:54 |
| pedronis | I fear so | 12:54 |
| pedronis | we need to think about that | 12:55 |
| pstolowski | good point.. | 12:55 |
| pedronis | which reminds that at some point we had revision in the hook setup struct | 12:55 |
| pedronis | and removed it | 12:55 |
| pedronis | thinking that current would be ok | 12:55 |
| pedronis | mmh | 12:55 |
| pedronis | pstolowski: we need to think, it's tempting to run autoconnect after link-snap bu doesn't sound correct | 12:57 |
| pstolowski | pedronis, it runs after link-snap atm | 12:58 |
| pedronis | it doesn't hit the problem then | 12:58 |
| pedronis | but not sure it's correct | 12:58 |
| pedronis | the snap might not working | 12:58 |
| pstolowski | it would help to land interface-hooks (and unknown-task-handler) first, and then focus on integrating autoconnect | 12:58 |
| pedronis | but is exposed | 12:58 |
| niemeyer | I'll be late for the standup a minute or two but will be there in a moment | 12:59 |
| pedronis | pstolowski: like one of the autoconnect could be for some libraries over content interface | 12:59 |
| pedronis | the snap is there | 12:59 |
| pedronis | but application could crash (no library) | 12:59 |
| pedronis | otoh as I said before the snap is know we cannot run hook atm (afaict) | 13:00 |
| === Trevinho_ is now known as Trevinho | ||
| kalikiana | re | 13:46 |
| Chipaca | mi | 13:53 |
| pedronis | zyga-ubuntu: what's your question? | 13:53 |
| sparkiegeek | fa | 13:53 |
| Chipaca | pedronis: how to get "snap download --revision=xyz foo" to work when one has ownership of foo | 13:54 |
| zyga-ubuntu | pedronis: how to make snap download work with my auth magic so that I can download revisions of snaps I co-own | 13:54 |
| zyga-ubuntu | there we go :) | 13:54 |
| * zyga-ubuntu wonders how will DST affect people working from the moon "call me at noon" | 13:55 | |
| pedronis | zyga-ubuntu: we don't have yet the proper fix (we should be to make just that work) | 13:55 |
| pstolowski | niemeyer, the unknown task handler PR is #4440 | 13:55 |
| mup | PR #4440: state: unknown tasks handler <Created by stolowski> <https://github.com/snapcore/snapd/pull/4440> | 13:55 |
| zyga-ubuntu | ok, it's not urgent (to me) | 13:55 |
| Chipaca | zyga-ubuntu: space is in UTC \o/ | 13:55 |
| * pstolowski lunch | 13:55 | |
| pedronis | zyga-ubuntu: but snapcraft export-login (in beta) plus my last PR (hint hint) should let you do that | 13:55 |
| niemeyer | pstolowski: Thanks | 13:56 |
| pedronis | pstolowski: mvo: I marked it for 2.31 | 13:56 |
| pedronis | #4440 I mean | 13:56 |
| mup | PR #4440: state: unknown tasks handler <Created by stolowski> <https://github.com/snapcore/snapd/pull/4440> | 13:56 |
| mup | PR snapcraft#1866 opened: lxd: setup LXD remote for multipass <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1866> | 13:57 |
| pedronis | zyga-ubuntu: #4481 | 14:00 |
| mup | PR #4481: image: let consume snapcraft export-login files from tooling <Created by pedronis> <https://github.com/snapcore/snapd/pull/4481> | 14:00 |
| pedronis | fedora tests seem to break again indeed :/ | 14:01 |
| zyga-ubuntu | pedronis: I'll disable them | 14:01 |
| mup | PR snapd#4484 opened: spread: switch fedora 26 to manual again <Created by zyga> <https://github.com/snapcore/snapd/pull/4484> | 14:04 |
| mup | PR snapcraft#1867 opened: kernel plugin: remove dependency on magic <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1867> | 14:15 |
| ackk | mvo, hi, I saw that the updated base-18 snap was finally build, any chance it can be pushed to edge? | 14:21 |
| pedronis | acck: it is on edge | 14:26 |
| ackk | pedronis, ah, nice, thanks | 14:27 |
| pedronis | sorry, ackk: see snap info base-18 | 14:27 |
| pedronis | you can always check that way what is available | 14:27 |
| ackk | thanks | 14:29 |
| ackk | pedronis, that's the latest build from LP, right? | 14:29 |
| pedronis | for that you need mvo | 14:29 |
| pedronis | I think he said so but not sure | 14:29 |
| mborzecki | wrapping it up, enjoy your weekend guys | 14:43 |
| * zyga-ubuntu focuses on coding, tweet @zygoon for attention | 14:48 | |
| mvo | ackk: I pushed it to edge, unfortunately it does not contain all the bits I hacked into our custom base-18 snap (sorry for the delay, was in a meeting) | 14:49 |
| elopio | hello! it's friday and we all know it :) | 14:49 |
| sparkiegeek | !infer time in australia | 14:51 |
| ubottu | sparkiegeek: I am only a bot, please don't think I'm intelligent :) | 14:51 |
| mup | sparkiegeek: 1:51:58 am AEDT Saturday, January 13, 2018. | 14:51 |
| sparkiegeek | elopio: not everyone "knows" it's Friday :) | 14:52 |
| sparkiegeek | elopio: PS. Happy Friday! | 14:52 |
| elopio | sparkiegeek: those australians are always thinking ahead :) | 14:52 |
| ackk | mvo, ah I see, ok, I'll stick to that custom built one for now then | 14:53 |
| ackk | mvo, thanks for the update | 14:53 |
| kalikiana | elopio: happy Friday :-D | 14:54 |
| elopio | kalikiana: :D | 14:54 |
| mvo | ackk: I look into what is missing later today or monday | 14:56 |
| ackk | mvo, awesome thanks | 14:56 |
| Chipaca | ok, i'm going offline now | 14:58 |
| Chipaca | have a great weekend, y'all! and niemeyer safe and boring travels | 14:58 |
| * kalikiana going for a break | 14:59 | |
| sergiusens | kalikiana when you get back, want to jump onto call? | 15:17 |
| sergiusens | elopio hey | 15:17 |
| elopio | sergiusens: feeling better? | 15:18 |
| sergiusens | elopio it is a good Friday for those on Friday indeed | 15:18 |
| sergiusens | elopio yes, not 100% but a lot better than before | 15:19 |
| elopio | just in time for the flights | 15:19 |
| sergiusens | elopio I have two requests for you though, I added a comment to #1853 | 15:19 |
| mup | PR #1853: osutil: call sync after cp if requested. overlord/snapstate/backend: switch to use osutil instead of another buggy call to cp <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/1853> | 15:19 |
| sergiusens | elopio the snapcraft one ;-) | 15:20 |
| elopio | looking | 15:20 |
| sergiusens | elopio and also added wanted you to look at 1867 | 15:20 |
| mup | PR snapcraft#1868 opened: tests: call file directly for the HasArchitecture checker <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1868> | 15:21 |
| kalikiana | sergiusens: getting my headphones | 15:22 |
| mup | PR snapd#4484 closed: spread: switch fedora 26 to manual again <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4484> | 15:23 |
| sergiusens | kalikiana when the clock hits 30 | 15:23 |
| elopio | sergiusens: I wanted to preserve the behaviour of returning None, instead of an empty summary when there isn't one. But it seems to me that get_summary is only used for tests, so that would work just the same | 15:23 |
| elopio | kyrofa: do you agree? | 15:23 |
| elopio | sergiusens: 1867 seems to just preserve the behaviour, so ok. Could you please add on the pr why do you want to get rid of magic? | 15:25 |
| kalikiana | sergiusens: I'm ready. Got a URL for me? | 15:31 |
| kalikiana | sergiusens: note that I need to head out in 20min | 15:38 |
| sergiusens | kalikiana the weekly | 15:38 |
| kalikiana | sergiusens: ah. okay, lemme find the url | 15:38 |
| sergiusens | elopio because of popey's bug | 15:38 |
| sergiusens | need to find it again though | 15:39 |
| zyga-ubuntu | pstolowski: hey | 15:40 |
| zyga-ubuntu | pstolowski: still around? | 15:41 |
| zyga-ubuntu | pstolowski: what are the long terms plans for interfaces.Plug, i assume that's going to go away entirely? | 15:41 |
| pstolowski | zyga-ubuntu, yep | 15:41 |
| pstolowski | zyga-ubuntu, yes | 15:41 |
| zyga-ubuntu | pstolowski: thank you for confirming that | 15:41 |
| zyga-ubuntu | pstolowski: is that waiting for something/ | 15:43 |
| zyga-ubuntu | pstolowski: or could I try doing that now? | 15:43 |
| pstolowski | zyga-ubuntu, I planned to do that after #4358, because it still uses Plug/Slot in Autoconnect methods | 15:45 |
| mup | PR #4358: interfaces: interface hooks implementation <Created by stolowski> <https://github.com/snapcore/snapd/pull/4358> | 15:45 |
| pstolowski | zyga-ubuntu, so you won't be able to remove them just yet | 15:45 |
| zyga-ubuntu | pstolowski: understood | 15:50 |
| elopio | sergiusens: what kernel snap are you using to test this? | 15:52 |
| kyrofa | elopio, responded on the PR. I like returning none as well, although I don't think we're using that functionality right now | 15:54 |
| sergiusens | elopio oh, I wanted you to test; but I used pc-kernel | 15:54 |
| kyrofa | The most important thing is that they don't show up in to_dict() | 15:54 |
| elopio | sergiusens: but the pc kernel is using the make plugin, not the kernel plugin. | 15:54 |
| elopio | unless I'm looking at the wrong pc kernel. | 15:55 |
| sergiusens | elopio I only tested the sublogic where things changed not a full kernel with pc-kernel | 15:56 |
| elopio | sergiusens: ok, I'm testing the extraction with the 96boards demo snap. | 15:57 |
| mup | Issue snapcraft#1828 closed: Support desktop and icon in appstream handler <Created by sergiusens> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/issue/1828> | 15:57 |
| mup | PR snapcraft#1853 closed: extractors: support appstream icon and desktop <Created by elopio> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1853> | 15:57 |
| sergiusens | elopio that is good enough; this should all just work though ;-) | 15:59 |
| * kalikiana heading out | 15:59 | |
| zyga-ubuntu | pstolowski: one tiny review please: vvvv | 16:08 |
| pstolowski | sure | 16:09 |
| mup | PR snapd#4485 opened: interfaces/builtin: use snap.{Plug,Slot}Info over interfaces.{Plug,Slot} <Created by zyga> <https://github.com/snapcore/snapd/pull/4485> | 16:09 |
| pstolowski | zyga-ubuntu, hmm will it pass the tests? | 16:11 |
| zyga-ubuntu | pstolowski: it does here | 16:12 |
| zyga-ubuntu | via embedding, I think | 16:12 |
| pstolowski | zyga-ubuntu, ok, second question, will it cause my interface-hooks branch to go all red? ;) | 16:13 |
| zyga-ubuntu | no, I don't think so, note that *none* of the interfaces are touched | 16:13 |
| zyga-ubuntu | this is just one bit of test code | 16:13 |
| zyga-ubuntu | pstolowski: if it does I will unconflict them (warranty ;-) | 16:14 |
| pstolowski | zyga-ubuntu, just checked, no issues after your change, only 2 line fix needed in my branch. thanks! | 16:24 |
| zyga-ubuntu | pstolowski: thank you! | 16:24 |
| pstolowski | zyga-ubuntu, ah sorry, i'm wrong, no fix needed, i missed utils_test.go change that you did | 16:24 |
| pstolowski | so all greeen | 16:24 |
| zyga-ubuntu | woot | 16:24 |
| zyga-ubuntu | I'll merge it when green | 16:24 |
| zyga-ubuntu | thanks, I'm back to my hacking | 16:24 |
| pstolowski | (all green here locally) | 16:25 |
| ogra_ | jdstrand_, is there a reason why the network-setup-control interface gives me ful write access to /etc/netplan but does not allow me to run "netplan generate" and "netplan apply" afterwards so that i need to force a reboot to make a new network config be picked up ? | 16:26 |
| zyga-ubuntu | ogra_: jdstrand_ is on holidays this week | 16:26 |
| ogra_ | zyga-ubuntu, ah, thanks | 16:26 |
| zyga-ubuntu | ogra_: and in cape town next week | 16:27 |
| ogra_ | i'll re-ask next week ... not urgent | 16:27 |
| zyga-ubuntu | ogra_: so should be harder to reach over irc but easier timezone wise | 16:27 |
| ogra_ | yeah | 16:27 |
| zyga-ubuntu | ogra_: and besides, how have you been? | 16:32 |
| zyga-ubuntu | ogra_: I heard that you work with koza now, is that true? | 16:33 |
| ogra_ | yep | 16:33 |
| ogra_ | zyga-ubuntu, (sorry, was busy filling travel forms) ... well, lots of custome HW enablement recently | 16:44 |
| ogra_ | its a lot of fun but shows all our drawbacks and missing features in snapd and UbuntuCore | 16:45 |
| zyga-ubuntu | ogra_: any low hanging fruit? | 16:49 |
| ogra_ | zyga-ubuntu, well, the multi volume thing that michael fixed already was one ... | 16:52 |
| ogra_ | zyga-ubuntu, but there is more to come ... disabling auto-import ... being able to boot without a hardcoded eth0 config on first boot etc | 16:53 |
| mup | PR snapcraft#1869 opened: cli: exported login should only be readable by owner <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1869> | 16:57 |
| * elopio goes to the mechanic and relocates. | 17:14 | |
| mup | PR snapd#4485 closed: interfaces/builtin: use snap.{Plug,Slot}Info over interfaces.{Plug,Slot} <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/4485> | 17:27 |
| sergiusens | kyrofa hey are you up yet? | 17:28 |
| kyrofa | sergiusens, hahaha | 17:28 |
| kyrofa | Of course-- what's up? | 17:28 |
| sergiusens | kyrofa elopio snapcraft#1867 and snapcraft#1868 need reviews, after those are in I can propose the removal of magic completely (from snapcraft.internal.elf) | 17:29 |
| mup | PR snapcraft#1867: kernel plugin: remove dependency on magic <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1867> | 17:29 |
| mup | PR snapcraft#1868: tests: call file directly for the HasArchitecture checker <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1868> | 17:29 |
| kyrofa | sergiusens, taking a look now | 17:30 |
| kyrofa | sergiusens, snapcraft#1869 could use a review as well | 17:30 |
| mup | PR snapcraft#1869: cli: exported login should only be readable by owner <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1869> | 17:30 |
| === CodeMouse92 is now known as CodeMouse92__ | ||
| sergiusens | kyrofa you already have one ;-) | 17:33 |
| matiasb | niemeyer, hola! just a reminder we are waiting for some definition on the appstream field; if you don't get to it before leaving for the sprint, it would be great if you can sync with Sergio and roadmr next week | 18:05 |
| roadmr | hi :) | 18:07 |
| sergiusens | matiasb niemeyer +1 on that, out extract info work for appstream is only missing a definition on that | 18:09 |
| kyrofa | sergiusens, what was your test case for snapcraft#1867 ? | 18:18 |
| mup | PR snapcraft#1867: kernel plugin: remove dependency on magic <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1867> | 18:18 |
| kyrofa | The code looks good, but I don't know of a kernel using that right now that I can use to verify that it works properly | 18:18 |
| sergiusens | kyrofa the manual tests | 18:19 |
| kyrofa | Oh nice | 18:19 |
| sergiusens | kyrofa but elopio was on that, just waiting for his results; given that it is a fallback mechanism I am not scared about the implementation | 18:19 |
| kyrofa | Okay, I'll let him do it. +1 from me on the code, I'll say that in the PR | 18:20 |
| niemeyer | matiasb, sergiusens: Sorry for the delay.. that's still very close to the top of my list, but I need to get the tasks required for me to travel to the sprint out of it first | 18:21 |
| niemeyer | I'll try to have a look today still | 18:21 |
| roadmr | thanks niemeyer :) | 18:23 |
| matiasb | niemeyer, good to know it's in your radar! thanks | 18:24 |
| === Beret- is now known as Beret | ||
| sergiusens | kyrofa replied to both your comments | 18:52 |
| sergiusens | and thanks for the check Gustavo | 18:53 |
| elopio | sergiusens: yep. On it, but it filled my hard disk. I'm moving things around to make space | 18:54 |
| jamesb192 | I have a program I am looking to snapify, however it builds a python module with an .so in the process. I can get the daemon to work but not the python tools. How can I get the python module exposed? | 19:04 |
| mup | PR snapcraft#1868 closed: tests: call file directly for the HasArchitecture checker <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1868> | 19:37 |
| zyga-ubuntu | jamesb192_: hey, it's a late, come back next week and I'll help you out, ok? | 20:04 |
| === lborda_ is now known as lborda_att_poc | ||
| sergiusens | kyrofa I am done with the readelf change, just waiting on elopio to merge the kernel dep PR | 21:06 |
| kyrofa | Awesome! | 21:07 |
| elopio | sergiusens: receiving objects: 21% (1050922/4836407) | 21:10 |
| elopio | I had to do it locally, I couldn't get enough space on my canonistack instance. If you want, I'll hit the merge button once the test finishes. | 21:11 |
| sergiusens | kyrofa any chance you can take a stance at testing that? | 21:15 |
| sergiusens | that is if it is really necessary | 21:15 |
| sergiusens | elopio ^? | 21:15 |
| elopio | sergiusens: well, you can merge now and if something wrong appears, I can yell in panic. | 21:16 |
| kyrofa | Yeah you should probably do a shallow clone | 21:17 |
| kyrofa | sergiusens, happy to do it once I'm done here if necessary | 21:17 |
| kyrofa | Just let me know | 21:18 |
| sergiusens | kyrofa go for it | 21:30 |
| sergiusens | elopio also, kyrofa is right, shallow clone ftw | 21:30 |
| elopio | yes, I stopped, and it's now downloading core -_- | 21:33 |
| elopio | kyrofa: it would be great if you give it a try, just follow the steps from the manual test about 96boards. I will relocate home now and continue from there | 21:34 |
| mup | PR snapcraft#1870 opened: elf: remove dependency on magic <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1870> | 21:34 |
| sergiusens | kyrofa here you go to get started ^ | 21:36 |
| kyrofa | I swear I can never remember my login for this site | 21:39 |
| kyrofa | Running no | 21:42 |
| kyrofa | w | 21:42 |
| sergiusens | stgraber when I run `lxd init` on my system I get "lxd: error while loading shared libraries: liblxc.so.1: cannot open shared object file: No such file or directory" ... is this something common? lxd itself was working fine, I wanted to re-init | 21:52 |
| stgraber | sergiusens: the only cases where I've seen this is when someone installs the lxd snap with --classic by mistake | 21:53 |
| stgraber | sergiusens: though I guess if you have a copy of the lxd binary somewhere (not coming from the snap), that could cause that too | 21:53 |
| sergiusens | stgraber hmm, not installed with classic and I don't even have the deb installed | 21:56 |
| sergiusens | stgraber this profile name looks odd "[ 13.008436] audit: type=1400 audit(1515754069.322:124): apparmor="DENIED" operation="mount" info="failed flags match" error=-13 profile="lxd-packaging_</var/snap/lxd/common/lxd>" name="/var/lib/lxcfs/" pid=4086 comm="(networkd)" flags="ro, nosuid, nodev, remount, bind" | 21:58 |
| * sergiusens tries to install in devmode | 21:59 | |
| sergiusens | nope, doesn't do the trick :/ | 22:00 |
| kyrofa | sergiusens, this demo gives me "make: *** No rule to make target 'Image'. Stop." during the build step | 22:00 |
| kyrofa | Did you need to tweak it? | 22:01 |
| kyrofa | sergiusens, argh, duh, nevermind | 22:03 |
| kyrofa | Muscle memory killing me | 22:03 |
| stgraber | sergiusens: profile name looks fine to me | 22:08 |
| stgraber | sergiusens: what does "which lxd" or "type lxd" get you? | 22:08 |
| sergiusens | stgraber /snap/bin/lxd; I also did a `snap run lxd --shell` and ran through things manually, interestingly liblxc.so.1 is there so I guess it is failing to load for some reason | 22:11 |
| sergiusens | stgraber I was using this system just fine until now | 22:11 |
| stgraber | sergiusens: did the core snap get updated and dropped some bits that liblxc.so.1 relies on somehow maybe? | 22:25 |
| stgraber | though if so, our jenkins tests haven't noticed that yet | 22:26 |
| sergiusens | stgraber don't know, I can show you live on Monday if I don't sort it out by then | 22:33 |
| sergiusens | in the meantime, I have to pack for my early flight tomorrow :-/ | 22:33 |
| sergiusens | I'll switch to edge and see what happens | 22:33 |
| sergiusens | kyrofa how did it go? | 22:33 |
| kyrofa | sergiusens, still building | 22:34 |
| sergiusens | kyrofa ack, in the meantime I hope you are enjoying snapcraft#1870 :-) | 22:34 |
| mup | PR snapcraft#1870: elf: remove dependency on magic <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1870> | 22:34 |
| kyrofa | We're missing some dependencies, I've had to restart it a few times | 22:34 |
| kyrofa | Uh... /tmp/tmp0x9gj2_u: 11: exec: modprobe: not found ? | 22:35 |
| kyrofa | Darn, it's in sbin | 22:35 |
| kyrofa | sergiusens, but that means it got far enough for me to say that it worked | 22:37 |
| kyrofa | (I think) | 22:39 |
| kyrofa | Running again anyway... | 22:39 |
| kyrofa | sergiusens, test failurs on 1870 | 22:40 |
| kyrofa | sergiusens, sure wish we could run arm adt... | 22:42 |
| sergiusens | kyrofa yeah, 1870 will fail tests until that kernel one lands ;-) Focus on the code though :-) | 22:47 |
| kyrofa | Hahaha, this is so gross | 22:49 |
| kyrofa | ly pretty | 22:50 |
| sergiusens | kyrofa if you clone and run all the units they should pass as you have magic installed in your venv; integration should also work | 23:05 |
| sergiusens | kyrofa and the whatever test case you used to verify the glibc from the future feature should also work | 23:05 |
| * sergiusens will bbl | 23:06 | |
| kyrofa | sergiusens, have we talked about looking into pyelftools? | 23:07 |
| kyrofa | We're shelling out so much here, I feel like we could be reading these things directly | 23:08 |
| mup | PR snapcraft#1867 closed: kernel plugin: remove dependency on magic <Created by sergiusens> <Merged by kyrofa> <https://github.com/snapcore/snapcraft/pull/1867> | 23:19 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!