| mup | PR snapcraft#1783 closed: tests: run codespell as part of static tests <Created by kyrofa> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1783> | 00:07 |
|---|---|---|
| mcphail | In my snapcraft.yaml, I have a part which depends on a library and header files I build in a previous part. How do I reference those files in the build? Passing "-I/usr/include" as a CFLAG doesn't automagically resolve, so what path should I pass instead? | 00:08 |
| mcphail | http://termbin.com/icbu is the work in progress | 00:10 |
| diddledan | mcphail: you need to point to their location using $SNAPCRAFT_STAGE | 00:12 |
| mcphail | diddledan: aah. cool. I'll experiment with that. Cheers | 00:12 |
| diddledan | so if they're installing into `/usr/include` then you'd use `-I$SNAPCRAFT_STAGE/usr/include` - but you don't seem to be telling the autotools build where you want the files installed | 00:13 |
| diddledan | which usually means autotools will use `/usr/local` pathjs | 00:13 |
| diddledan | paths* | 00:13 |
| mcphail | yeah - i'll need to work out how to pass a --prefix as well | 00:14 |
| mcphail | Ta. I'll get back to work! :) | 00:16 |
| diddledan | you can pass arguments to ./configure command with `configflags:` in your snapcraft.yaml | 00:18 |
| mcphail | Ta. I think it sets a blank --prefix by deafult, doesn't it? | 00:18 |
| diddledan | `configflags` expects a yaml list, so either new lines with `- ` indicating each item, or `[--flag1, --flag2, ...]` | 00:19 |
| diddledan | difficult to do multiline pastes in IRC :-p | 00:19 |
| mcphail | :) - that's brilliant. Cheers diddledan | 00:20 |
| diddledan | here's an example of the two types of list https://www.irccloud.com/pastebin/8HsmUyMD/snapcraft.yaml | 00:22 |
| mcphail | ta | 00:23 |
| diddledan | I had a hard time grokking yaml when I first came across it because the rules weren't clear to me, but I'm learning | 00:23 |
| mcphail | yay - builds! | 00:25 |
| diddledan | \o/ | 00:25 |
| mcphail | That's super. Thanks for your help | 00:25 |
| diddledan | no probs :-) | 00:28 |
| pattern | hey anyone have a second to help me with ubuntu core for raspberry pi? | 02:45 |
| mup | PR snapcraft#1785 opened: Add type hinting to file_utils.py <Created by daniellim2000> <https://github.com/snapcore/snapcraft/pull/1785> | 04:04 |
| === kennyloggins is now known as bujeremy | ||
| Pharaoh_Atem | I'm very annoyed | 05:35 |
| Pharaoh_Atem | I think I'm going to have to go back to vendoring snap-mgmt in Fedora Dist-Git if it breaks like this again | 05:37 |
| Pharaoh_Atem | I'm incredibly disappointed that my feedback was not solicited as the author of snap-mgmt before merging the PR | 05:38 |
| Pharaoh_Atem | whatever, good night everyone | 05:38 |
| bujeremy | would bytemark be an SSL or an email-provider with LibreSSL ? | 05:53 |
| mborzecki | morning | 06:15 |
| === cpaelzer_ is now known as cpaelzer | ||
| === cpaelzer is now known as cpaelzer_ | ||
| === cpaelzer_ is now known as cpaelzer | ||
| === cpaelzer is now known as cpaelzer_ | ||
| === cpaelzer_ is now known as cpaelzer | ||
| === jamesh_ is now known as jamesh | ||
| zyga-ubuntu | good morning! | 07:13 |
| mborzecki | hey | 07:17 |
| mvo | hey zyga-ubuntu and mborzecki | 07:22 |
| zyga-ubuntu | mvo: hey :) | 07:30 |
| zyga-ubuntu | mvo: how was last friday? | 07:32 |
| zyga-ubuntu | mvo: any fires? | 07:32 |
| mvo | zyga-ubuntu: all pretty quiet, no worries | 07:33 |
| mborzecki | zyga-ubuntu: quick question, Son_Goku raised an issue https://github.com/snapcore/snapd/pull/4316#discussion_r154561886 and I don't really have enough info to act on it, I see that this is an extra profile that's set up for snap-confine in the core snap, any idea where's the code that installs it? | 07:33 |
| mup | PR #4316: cmd/snap-mgmt: introduce snap-mgmt tool <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4316> | 07:33 |
| zyga-ubuntu | mborzecki: the interfaces/apparmor backend does this | 07:37 |
| zyga-ubuntu | mborzecki: setupSnapConfineReexec | 07:37 |
| mborzecki | looking | 07:38 |
| mborzecki | zyga-ubuntu: so the 'extra' profile is always '${snap-mount-dir}/core/<rev>/usr/lib/snapd/snap-confine'? | 07:41 |
| zyga-ubuntu | mborzecki: yes | 07:41 |
| mborzecki | zyga-ubuntu: hmm, do you understand what Son_Goku was talking about here then? https://github.com/snapcore/snapd/pull/4316#discussion_r154561886 | 07:43 |
| mup | PR #4316: cmd/snap-mgmt: introduce snap-mgmt tool <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4316> | 07:43 |
| zyga-ubuntu | I think he was just confused about this aspect | 07:43 |
| zyga-ubuntu | base snaps, even once we have them, can have any layout but this particular one is specific to the ubuntu-based core snap | 07:44 |
| mborzecki | that's my understading as well, just wanted to make sure i didn't miss anything | 07:44 |
| zyga-ubuntu | as long as snap-confine is in the core snap this code is correct | 07:44 |
| mborzecki | zyga-ubuntu: another quick question, any idea why it's sleeping here? https://github.com/snapcore/snapd/blob/master/packaging/ubuntu-16.04/snapd.postrm#L7 i assumed that it's a woraround for some weirdness in older systemd, but then tracked it back to https://github.com/snapcore/snapd/commit/67902f1ef14dd29ff809fabfcb9f7b2cbcd5fb7f and there does not seem to be any reason given for adding this | 07:48 |
| zyga-ubuntu | mborzecki: looking | 07:51 |
| zyga-ubuntu | mborzecki: not 100% sure but isn't it the case that "stop" can return before the thing is really stopped? | 07:52 |
| * zyga-ubuntu observes fresh snow falling outside | 07:52 | |
| mborzecki | yeah, this would happen if you hit a timeout in systemctl stop | 07:55 |
| mborzecki | hmm the opensuse package has not been updated for quite some time, and the tumbleweed build fails https://build.opensuse.org/package/show/system:snappy/snapd | 07:59 |
| zyga-ubuntu | ah | 08:01 |
| zyga-ubuntu | mborzecki: those failures are from new golang | 08:01 |
| zyga-ubuntu | mborzecki: they are fixed in master, we should just find some time to update the package | 08:01 |
| mborzecki | iirc there was some magic that you could do with obs to have it look/update package automatically | 08:05 |
| zyga-ubuntu | mborzecki: yes but I'm afraid packaging needs real hand care each time | 08:08 |
| zyga-ubuntu | mborzecki: and we need to test it before unleashing it onto our users | 08:08 |
| zyga-ubuntu | mborzecki: I'd like to see mvo's face if we had automatic releases to xenial, that just get made without testing :) | 08:10 |
| mborzecki | well, ideally all the testing you need should be part of the CI right? | 08:11 |
| zyga-ubuntu | mborzecki: just like we do in ubuntu | 08:11 |
| zyga-ubuntu | mborzecki: I think we're not ready for that, there's a class of tests disabled for opensuse | 08:11 |
| zyga-ubuntu | mborzecki: and nobody on the team is using it 24/7 | 08:12 |
| mvo | zyga-ubuntu: my face is nothing compared to the users faces ;) | 08:12 |
| ogra_ | it might make us find regressions in no time though :P | 08:16 |
| mborzecki | crowsouring QA | 08:17 |
| mborzecki | s/crowsouring/crowdsourcing/ | 08:18 |
| zyga-ubuntu | mborzecki: s/crowdsourcing/crowdangering/ | 08:18 |
| mup | PR snapd#4344 opened: cmd/snap-mgmt: fixes <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/4344> | 08:29 |
| mup | PR snapd#4345 opened: packaging/opensuse-42.2: package and use snap-mgmt <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/4345> | 08:30 |
| mvo | zyga-ubuntu: 4325 looks ready, if you could re-review we have one less PR :) | 08:34 |
| zyga-ubuntu | mvo: looking | 08:39 |
| kalikiana | o/ | 08:40 |
| zyga-ubuntu | done | 08:41 |
| zyga-ubuntu | hey kalikiana | 08:41 |
| mup | PR snapd#4325 closed: Adding test for netlink-connector interface <Created by sergiocazzolato> <Merged by zyga> <https://github.com/snapcore/snapd/pull/4325> | 08:42 |
| mup | PR snapd#4346 opened: Make snapd.autoimport configurable <Created by ogra1> <https://github.com/snapcore/snapd/pull/4346> | 09:07 |
| ogra_ | mvo, ^^^ | 09:07 |
| mvo | ogra_: thanks, looking | 09:15 |
| mvo | ogra_: heh, thats a small patch | 09:15 |
| ogra_ | yeah, thanks to having a function like that :) | 09:15 |
| mvo | ogra_: yeah, I was thinking that too: "looks like this was done right" | 09:16 |
| ogra_ | we should look at the other bits too one day, i'm sure there is more that could be generalized in such a way | 09:17 |
| mvo | ogra_: how urgent is it? it looks trivial enough so that we call pull it into .30 even | 09:17 |
| ogra_ | not super urgent | 09:17 |
| mvo | ok, thanks | 09:17 |
| ogra_ | (like ... could wait for a 30.1 if one happens anyway or even for .31) | 09:18 |
| zyga-ubuntu | 2.30.1.1 | 09:18 |
| mvo | ogra_: ok, I tagged it 2.30, there will be at least one more 2.30~rc2 | 09:18 |
| mvo | ogra_: so we can pull it in there | 09:18 |
| ogra_ | cool | 09:18 |
| m4sk1n_ | 21:08:33 <m4sk1n_> I'm trying to add i18n support for snapcraft | 09:45 |
| m4sk1n_ | 21:18:48 <m4sk1n_> marked strings, added build_i18n to setup.py, but can't get to make it using .mo files | 09:45 |
| mup | PR snapd#4347 opened: tests: increase warn-timeout to 15m <Created by mvo5> <https://github.com/snapcore/snapd/pull/4347> | 09:49 |
| mup | PR snapd#4348 opened: tests: increase workers for ubuntu-{14,16}.04-64 by one <Created by mvo5> <https://github.com/snapcore/snapd/pull/4348> | 10:02 |
| zyga-ubuntu | hmm | 10:13 |
| zyga-ubuntu | so | 10:13 |
| zyga-ubuntu | on core the / inside the core snap doesn't match /snap/core/current | 10:13 |
| zyga-ubuntu | s/inside the core snap/inside the executiong environment/ | 10:13 |
| mvo | mborzecki: I'm looking at 4340 right now, expect a review in some minutes | 10:14 |
| mborzecki | mvo: thanks | 10:15 |
| zyga-ubuntu | hmmmm | 10:21 |
| kalikiana | zyga-ubuntu: that's surely expected? | 10:23 |
| * zyga-ubuntu thinks about http://pastebin.ubuntu.com/26111465/ | 10:23 | |
| zyga-ubuntu | kalikiana: no, it's not | 10:23 |
| zyga-ubuntu | kalikiana: I think this is a bug | 10:23 |
| kalikiana | zyga-ubuntu: what do you see there that's wrong? | 10:24 |
| kalikiana | I'd expect mounts to only show up on / | 10:24 |
| zyga-ubuntu | kalikiana: well, for once, / should really be the core snap (outside the execution environment) | 10:25 |
| zyga-ubuntu | kalikiana: then looking at that list I find it ... too long | 10:25 |
| zyga-ubuntu | anyway, I just need to focus and analyze that | 10:26 |
| * kalikiana needs coffee | 10:28 | |
| * ikey needs an army of testers -_- | 10:33 | |
| mborzecki | mvo: thanks for suggestions, it looks quite nice | 10:33 |
| mvo | mborzecki: great, happy that you like them | 10:34 |
| ikey | 27 downloads and 8 of them are me lol | 10:34 |
| mvo | mborzecki: one thing I forgot, I wonder if we should also ensure that "daemon" is != "" when before/after is used | 10:34 |
| mvo | ikey: for what snap? | 10:35 |
| ikey | the lsi snap | 10:35 |
| ikey | once snapd 2.30 drops ima go enlist an army of testers :p | 10:36 |
| zyga-ubuntu | hey ikey | 10:37 |
| ikey | rawr | 10:37 |
| mvo | ikey: heh, sounds great. I am a bit scared of installing it, once I have steam I will not stop playing it ;) | 10:37 |
| mborzecki | mvo: i think after/before is applied regardless of what's in [Service], eg you can have a oneshot service that starts after some target | 10:37 |
| ikey | mvo, well just install broken games and you'll be saved :p | 10:37 |
| mvo | haha | 10:37 |
| zyga-ubuntu | mvo: make sure to play the games you like the least | 10:38 |
| zyga-ubuntu | ;-) | 10:38 |
| mborzecki | mvo: otoh, we cannot really express such relation in snap yaml atm | 10:38 |
| mvo | mborzecki: yeah, sorry, I was not precise. an app can be any sort of daemon or a standalone binary. so we might want to check to ensure it is really != a standalone applicaton | 10:38 |
| * ikey goes off in search of copious amounts of caffeine | 10:40 | |
| mborzecki | mvo: ok, what you suggest is to raise an error when !app.IsService() ? | 10:40 |
| ikey | an IV drip will do.. | 10:40 |
| zyga-ubuntu | ikey: try geen tea | 10:40 |
| ikey | oh you mean monster? ok | 10:40 |
| ikey | >_> | 10:41 |
| mvo | mborzecki: yeah, I think so | 10:41 |
| mborzecki | ok | 10:41 |
| mvo | mborzecki: or do you think thats inappropriate? | 10:41 |
| mvo | zyga-ubuntu, ikey: green tea++ | 10:41 |
| mborzecki | i think it's ok, once we'll probably learn more about use cases once this is introduced into the wild | 10:42 |
| mvo | mborzecki: *nod* | 10:42 |
| mborzecki | mvo: btw, https://gist.github.com/mvo5/6e0ac963aacce9d0c78f1c7cf829f9c4#file-validate-go-L15 this will exhaust stack calling a.String() in a loop :) | 10:49 |
| mvo | mborzecki: oh, silly me | 10:50 |
| mvo | mborzecki: indeed, tests ftw :) | 10:51 |
| zyga-ubuntu | aha, I think I found the bug now | 10:52 |
| zyga-ubuntu | I think I need that coffee and then I can fix it :) | 10:53 |
| mup | PR snapd#4349 opened: Make snapd.autoimport configurable <Created by ogra1> <https://github.com/snapcore/snapd/pull/4349> | 10:54 |
| ikey | so i dont know how much use this is to anyone but i ported the remainder of the 4.13 apparmor stuff into the solus 4.14 kernel last night | 10:55 |
| ikey | and its working fine for snapd | 10:55 |
| zyga-ubuntu | ikey: thank you for doing that! | 11:00 |
| zyga-ubuntu | ikey: is there a lot of delta left? (vanilla vs apparmor) | 11:00 |
| ikey | 19 files changed, 1882 insertions(+), 28 deletions(-) | 11:00 |
| ikey | https://dev.solus-project.com/source/linux-current/browse/master/files/security/0001-apparmor-Merge-items-still-missing-from-upstream.patch | 11:00 |
| zyga-ubuntu | interesting, thank you | 11:01 |
| ikey | majority of it is because of af_unix/dbus | 11:01 |
| ikey | well lets generalise that as "socket" | 11:01 |
| ikey | because net.c is a big component of it too | 11:01 |
| mup | PR snapd#4346 closed: Make snapd.autoimport configurable <Created by ogra1> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/4346> | 11:02 |
| ikey | honestly i think some of that stuff could probably belong to securityfs and lsm APIs by now | 11:04 |
| ikey | reduce duplication, same as for the secureexec bits | 11:04 |
| pedronis | mvo: I added Gustavo to ogra PR, not sure we want to surface in the config the implementation detail that this is a separate service | 11:08 |
| pedronis | I'm also a b it worried that people will not understand the implications of turning it off | 11:09 |
| ogra_ | it isnt like it is publically exposed anywhere or so | 11:10 |
| mvo | pedronis: ok | 11:11 |
| pedronis | mvo: alsos why does it make boot slow? do we wait on it? | 11:12 |
| mup | PR snapd#4345 closed: packaging/opensuse-42.2: package and use snap-mgmt <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4345> | 11:12 |
| ogra_ | pedronis, the particular customer has a bunch of other disk devices on the HW (that are used for other stuff but some contain vfat, are attached via the usb bus and are slow in responding) | 11:14 |
| mvo | pedronis: we don't wait for it afaict | 11:14 |
| ogra_ | (and they will never use the feature) | 11:15 |
| pedronis | an option to list devices it should use might make more sense | 11:15 |
| mvo | review for 4338 looks like an easy win | 11:15 |
| pedronis | I don't know | 11:15 |
| pedronis | either way I suspect Gustavo might have opinions how this is spelled | 11:15 |
| ogra_ | well, if we cant use the services function it will become a lot more complex i suspect | 11:16 |
| ogra_ | but yeah, understood | 11:16 |
| mvo | mborzecki: does 4308 needs a master merge? | 11:16 |
| mup | PR snapd#4304 closed: debian: demote gnupg from dependencies to recommends <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/4304> | 11:18 |
| mborzecki | mvo: i'd like to have #4344 resolved first | 11:20 |
| mup | PR #4344: cmd/snap-mgmt: fixes <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/4344> | 11:20 |
| mborzecki | mvo: shall i add blocked tag 4308? | 11:20 |
| mvo | mborzecki: sounds good | 11:21 |
| mvo | mborzecki: and a note ideally | 11:21 |
| mvo | mborzecki: why its blocked | 11:21 |
| mborzecki | Son_Goku: thanks for looking at 4344 | 11:22 |
| * Son_Goku mumbles something about reviewed... | 11:23 | |
| mup | PR snapcraft#1784 closed: cli: show helpful message for 'snapcraft list-keys' with no keys <Created by BayMinimum> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1784> | 11:27 |
| mup | PR snapd#4350 opened: debian: make "gnupg" a recommends <Created by mvo5> <https://github.com/snapcore/snapd/pull/4350> | 11:30 |
| mup | PR snapd#4344 closed: cmd/snap-mgmt: fixes <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/4344> | 11:33 |
| zyga-ubuntu | mvo: do you have a second? | 11:40 |
| sergiusens | hey mpt, can you please take a look at the UX side of snapcraft#1780 ? I've added some comments you can agree or disagree on and pasted some output for it to be easier to review | 11:45 |
| mup | PR snapcraft#1780: cli: add export-login command <Created by kyrofa> <https://github.com/snapcore/snapcraft/pull/1780> | 11:45 |
| cachio | mvo, I'll be 5 minutes late today | 11:48 |
| cachio | mvo, still 2 weeks of kinesiology missing | 11:49 |
| mup | PR snapd#4341 closed: tests: new test to validate location control interface <Created by sergiocazzolato> <Closed by sergiocazzolato> <https://github.com/snapcore/snapd/pull/4341> | 11:51 |
| mup | PR snapd#4351 opened: tests: new test to validate location control interface <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/4351> | 11:56 |
| sborovkov | Hi! | 11:57 |
| sborovkov | How can I find what those changes are "error: cannot refresh "core": snap "core" has changes in progress"? | 11:57 |
| sborovkov | I get the same when trying to update my own snap | 11:57 |
| sborovkov | And it does not seem to really have any changes | 11:57 |
| mpt | looking | 12:02 |
| jdstrand | ikey: hey, cool. fyi, you might be interested in https://gitlab.com/apparmor/apparmor-kernel. I don't know how jjohansen is going to have separate branches for different kernels, but you might want to talk to him about that | 12:32 |
| zyga-ubuntu | jdstrand: hey | 12:33 |
| zyga-ubuntu | jdstrand: I'm still on that stale core issue, it misbehaves on core systems; I'm looking at it | 12:33 |
| jdstrand | zyga-ubuntu: is the misbehavior the needing of the additional rule? | 12:34 |
| jdstrand | '/'? | 12:34 |
| zyga-ubuntu | jdstrand: no, it's a bug where the namespace is always stale | 12:34 |
| jdstrand | oh, ok | 12:34 |
| zyga-ubuntu | jdstrand: trying to check if this is bad logic or bad test setup | 12:34 |
| jdstrand | zyga-ubuntu: and hi :) | 12:34 |
| jdstrand | zyga-ubuntu: fyi, mvo said that this is not a 2.30 item, but I have a number of 2.30 items to work on this week, so I plan to focus on those and be responsive to your layouts work as have time and hopefully swing back into it next week | 12:36 |
| zyga-ubuntu | jdstrand: sure | 12:36 |
| zyga-ubuntu | jdstrand: I think we are close, I need one review on https://github.com/snapcore/snapd/pull/4315 -- I plan to remove the panic but other than that I'd like your eyes whenver you can | 12:37 |
| mup | PR #4315: cmd/snap-update-ns: add execWritableMimic <Created by zyga> <https://github.com/snapcore/snapd/pull/4315> | 12:37 |
| zyga-ubuntu | jdstrand: I'll focus on bugfixes and finishing this this week | 12:38 |
| jdstrand | zyga-ubuntu: cool. 4315 is already on the list for background this and planned focus next week | 12:44 |
| zyga-ubuntu | jdstrand: thank you! good hunting :) | 12:44 |
| zyga-ubuntu | mvo: hey | 12:47 |
| zyga-ubuntu | mvo: do you have a moment? | 12:47 |
| mup | PR snapd#4324 closed: cmd/snap-confine: discard stale mount namespaces <Created by zyga> <Closed by zyga> <https://github.com/snapcore/snapd/pull/4324> | 12:53 |
| mvo | zyga-ubuntu: sure | 12:58 |
| zyga-ubuntu | mvo: hey, let's talk in the standup | 12:58 |
| mvo | cachio: 5min late> no problme | 12:59 |
| mvo | zyga-ubuntu: ok | 12:59 |
| mup | PR snapcraft#1786 opened: lxd: always install squashfuse <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1786> | 13:09 |
| mup | PR snapd#4078 closed: tests: new test to check interfaces after reboot the system <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/4078> | 13:19 |
| mup | PR snapcraft#1787 opened: tests: move refresh to unit folder <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1787> | 13:21 |
| mborzecki | mvo: https://travis-ci.org/snapcore/snapd/builds/311190657?utm_source=github_status&utm_medium=notification this is a build related to arch CI that timed out | 13:22 |
| kalikiana | sergiusens: Hey! I'd like to highlight snapcraft#1787 - the refresh test isn't currently run at all and at least one PR erroneously passed because of it, although master seems fine | 13:23 |
| mup | PR snapcraft#1787: tests: move refresh to unit folder <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1787> | 13:24 |
| kalikiana | Also elopio, when you get online, you may want to have a look. I'm wondering how this happened, and if maybe we can have some checks to catch "lost" test cases ^^ | 13:24 |
| om26er | elopio: Hello, are you back from your holidays ? | 13:28 |
| mvo | mborzecki: thank you! | 13:31 |
| * pstolowski heading to the dentist, bb in ~1,5h | 13:43 | |
| pedronis | mvo: btw in 2.30 on edge there's no nice transition with tasks from configure-snapd to back to configure, afaict the refresh/task stays stuck | 13:48 |
| mup | PR snapd#4347 closed: tests: increase warn-timeout to 15m <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/4347> | 13:49 |
| mup | PR snapd#4348 closed: tests: increase workers for ubuntu-{14,16}.04-64 by one <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/4348> | 13:51 |
| cachio | niemeyer, sorry, I lost internet connection | 13:55 |
| niemeyer | cachio: It's okay, we're done.. if issue is entroy we agreed to create /dev/random with the same device as /dev/urandom | 13:55 |
| niemeyer | cachio: We really don't care about having proper entropy for tests | 13:56 |
| cachio | niemeyer, sure | 13:56 |
| niemeyer | cachio: This should solve the issue for all cases | 13:56 |
| cachio | niemeyer, perfect | 13:56 |
| cachio | niemeyer, is it going to be part of that PR? | 13:58 |
| kalikiana | sergiusens: when you have some time, can you have a look at snapcraft#1746 ? to see what's the right way of dealing with the mypy error | 14:08 |
| mup | PR snapcraft#1746: cli: add version command <Created by gsilvapt> <https://github.com/snapcore/snapcraft/pull/1746> | 14:08 |
| sergiusens | kalikiana ok, but have you noticed you have an assigned milestone task https://github.com/snapcore/snapcraft/milestone/12 ? | 14:09 |
| mup | PR snapcraft#1785 closed: Add type hinting to file_utils.py <Created by daniellim2000> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1785> | 14:10 |
| zyga-ubuntu | I'll get something to eat | 14:10 |
| kalikiana | sergiusens: Yeah, I'm just going through the queue atm | 14:15 |
| sergiusens | kalikiana well that issue is due for tomorrow, so I suggest you give it priority | 14:19 |
| sergiusens | kyrofa this is something for you https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-xenial/xenial/armhf/s/snapcraft/20171204_024823_8b570@/log.gz (blocking 2.35 from entering xenial) | 14:19 |
| kalikiana | Aye | 14:20 |
| === cachio_ is now known as cachio | ||
| cachio | zyga-ubuntu, when you have some time could you take a look to #4218 | 14:31 |
| mup | PR #4218: tests: adding tests for time*-control interfaces <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/4218> | 14:31 |
| elopio | Good morning! | 14:31 |
| zyga-ubuntu | cachio: sure | 14:31 |
| popey | sergiusens: do you think you or your team might come up with a simple dotnet app I can use for docs? | 14:31 |
| popey | morning elopio | 14:31 |
| sergiusens | popey I can, I have been playing with it | 14:32 |
| popey | ooh | 14:32 |
| popey | I tried building something I randomly found - openra, and it all fall apart looking for some .net components | 14:32 |
| popey | so looking for a simpler example | 14:32 |
| sergiusens | elopio also note you have tasks due tomorrow https://github.com/snapcore/snapcraft/milestone/12 ... ideally merged today to catch issues for tomorrow | 14:33 |
| elopio | On it... | 14:33 |
| sergiusens | popey I have a simple forum parser; before New York I had been automating our reports | 14:33 |
| sergiusens | I can polish that up a bit and share | 14:34 |
| sergiusens | or write something similar to wethr | 14:34 |
| * sergiusens commutes back home | 14:34 | |
| popey | sergiusens: something small and simple would be fine | 14:37 |
| mvo | mborzecki: I added some more code for the log analyzing and ran it through your timed out PR: http://paste.ubuntu.com/26112553/ - so it looks like we were short on machines when this PR ran and core had just 2 instead of 4 workers allocated (cc niemeyer). lets keep an eye on PRs that time out and see if we find a pattern | 14:42 |
| mborzecki | mvo: thanks for investigating this | 14:42 |
| mvo | cachio, zyga-ubuntu, pedronis if you ran accross prs that time out, please let me know, I want to get to the bottom of those :) | 14:42 |
| mvo | mborzecki: sure, its a bit annoying so we need to fix it | 14:42 |
| niemeyer | mvo: We'll indeed need to increase a bit the number of machines to accommodate the extra distributions being tested | 14:42 |
| cachio | mvo, https://travis-ci.org/snapcore/snapd/builds/311266636?utm_source=github_status&utm_medium=notification | 14:43 |
| mvo | thanks cachio | 14:43 |
| mvo | niemeyer: thanks | 14:43 |
| cachio | mvo, this is being caused by Cannot allocate linode:debian-9-64: no powered off servers in Linode account exceed halt-timeout | 14:43 |
| niemeyer | mvo: Getting to the bottom of it sounds quite correct here :P | 14:43 |
| mvo | niemeyer: lol | 14:43 |
| cachio | mvo, I think most of the issues that we see is because lack of linode servers | 14:43 |
| cachio | mvo, we are in the limit | 14:43 |
| niemeyer | cachio, mborzecki: either way works for me.. either mborzecki tries to fix his PR, and then we generalize it.. or mborzecki already generalizes upfront.. whatever sounds easier | 14:44 |
| mvo | cachio: yeah, looks like your PR was particularly "bad": http://paste.ubuntu.com/26112570/ | 14:44 |
| cachio | mvo, do you have other travis logs to take a look? | 14:45 |
| mvo | cachio: I was looking at the one from mborzecki for arch but that was about it | 14:45 |
| mborzecki | hmm set up a bot, monitor travis results for jobs started on snapd repo, send email to mvo's inbox :) | 14:46 |
| mvo | cachio: I wrote two scripts, one looks for allocation times (which are around 1min-2min) for spread machines, except when it can't find any | 14:46 |
| mvo | cachio: and the other that looks at the individual tests to see how long those take, to see if we waste time somewhere | 14:46 |
| mvo | mborzecki: heh :) | 14:47 |
| cachio | mvo, I saw many problems to allocate linode servers the last weeks | 14:47 |
| mvo | cachio: but it looks like the alloc script is silly, spread already tells us when it cannot allocate things, I was wondering if sometimes linode is slow providing machines but that seems to be not the case | 14:47 |
| mvo | cachio: yeah, exactly | 14:47 |
| mup | PR snapd#4218 closed: tests: adding tests for time*-control interfaces <Created by sergiocazzolato> <Merged by zyga> <https://github.com/snapcore/snapd/pull/4218> | 14:56 |
| mpt | sergiusens, done | 15:01 |
| sergiusens | mpt thanks! | 15:02 |
| mup | PR snapd#4352 opened: tests: increase amount of workers for ubuntu-16.04-64 by one <Created by mvo5> <https://github.com/snapcore/snapd/pull/4352> | 15:03 |
| pedronis | mvo: did you see my question about 2.30 ? | 15:25 |
| kyrofa | sergiusens, yikes, this is xenial armhf? That means packages have different contents on different archs, that's sad | 15:38 |
| kyrofa | Although usr/share/doc/python-minimal/changelog.Debian.gz really shouldn't conflict unless a new version was released | 15:42 |
| kyrofa | That seems odd | 15:43 |
| kyrofa | sergiusens, that failure is consistent? | 15:44 |
| sergiusens | kyrofa you can check all the runs here: http://autopkgtest.ubuntu.com/packages/s/snapcraft/xenial/armhf | 16:04 |
| sergiusens | kyrofa look for triggers "snapcraft/2.35" | 16:04 |
| sergiusens | kyrofa would appreciate a follow up on snapcraft#1781 | 16:05 |
| mup | PR snapcraft#1781: many: set rpath for elf files for classic <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1781> | 16:05 |
| mvo | pedronis: sorry, I have not seen your question | 16:06 |
| mvo | pedronis: here? | 16:06 |
| pedronis | mvo: yes, what should we do/tell people that had 2.30 edge with configure-snapd, our change/fix doesn't unstuck them afaict (I'm in that situation) | 16:09 |
| mvo | pedronis: snap abort chg-id should get them out of misery | 16:13 |
| mvo | pedronis: if too many people are affected we may need to do something automatically | 16:13 |
| pedronis | mvo: it doesn't , the next try will still make a change with configure-snapd | 16:13 |
| pedronis | and then we don't understand it anymore, so we get stuck again | 16:14 |
| mvo | pedronis: hm, ok. thats annoying, so we need a dummy configure-snapd task for edge/2.30 ? | 16:17 |
| pedronis | I fear so, dummy or does the same as configure | 16:17 |
| mvo | pedronis: ok, I start a PR, thanks for the heads up | 16:18 |
| pedronis | given that it affected onyl edge for a little bit | 16:18 |
| pedronis | we can probably drop it after a while | 16:18 |
| mvo | +1 | 16:19 |
| pedronis | or we wait to have epochs and do it with the big cleanup those should allow us | 16:20 |
| mvo | pedronis: yeah, do you have a recommendation for a place to put it? I would say configstate but that has no runner anymore :) | 16:21 |
| pstolowski | can we use repair instead? | 16:21 |
| pedronis | let me look | 16:21 |
| pedronis | mvo: I would put it in snapmgr with a comment | 16:22 |
| pedronis | or hookstate | 16:22 |
| mvo | pedronis: yeah, was thinking hookstate too, but no strong opinion | 16:22 |
| mup | PR snapcraft#1787 closed: tests: move refresh to unit folder <Created by kalikiana> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/1787> | 16:22 |
| mvo | pedronis: thanks, I am on it, not sure if I can finish before hockey but I will try :) | 16:26 |
| mvo | cachio: looks like CE has finished automatic validation, good to promote to candidate | 16:31 |
| cachio | mvo, I just received an email | 16:31 |
| cachio | :) | 16:31 |
| mvo | cachio: yay | 16:31 |
| cachio | mvo, promotin | 16:32 |
| cachio | g | 16:32 |
| cachio | mvo, promotion done | 16:33 |
| mvo | cachio: thank you! | 16:36 |
| cachio | niemeyer, I can't use fedora-rawhide-64 image | 16:39 |
| niemeyer | Hm? | 16:39 |
| cachio | niemeyer, I am getting "cannot connect: cannot connect to linode:fedora-rawhide-64 (Spread-72): dial tcp 74.207.250.224:22: i/o timeout | 16:40 |
| cachio | " | 16:40 |
| niemeyer | cachio: Ok, that probably means the image is boosted somehow | 16:40 |
| niemeyer | busted even | 16:41 |
| niemeyer | Would be nice for it to be boosted, though | 16:41 |
| niemeyer | cachio: I don't have any better clues on my end | 16:41 |
| cachio | I am using kernel: GRUB 2 | 16:42 |
| cachio | niemeyer, could be that affecting? | 16:42 |
| niemeyer | cachio: If you didn't set up grub and a kernel on the image, definitely | 16:42 |
| cachio | niemeyer, I already set up grub2 in that image | 16:43 |
| cachio | and the kernel | 16:43 |
| niemeyer | cachio: Sorry, I really have absolutely no clue.. I've never looked into that image at all | 16:43 |
| cachio | niemeyer, in that case I'll try with the linode kernel to see if the grub setup has a problem | 16:44 |
| niemeyer | Thanks | 16:45 |
| mup | PR snapd#4353 opened: hookstate: add compat "configure-snapd" task <Created by mvo5> <https://github.com/snapcore/snapd/pull/4353> | 16:51 |
| cachio | niemeyer, could you please create fedora-rawhide-64 image based on spread-32? | 17:18 |
| cachio | niemeyer, I updated the kernel and installed the las rc | 17:18 |
| niemeyer | cachio: Creating image | 17:19 |
| cachio | niemeyer, tx | 17:19 |
| niemeyer | cachio: Wait | 17:22 |
| niemeyer | cachio: Damn | 17:23 |
| niemeyer | cachio: I'm sorry, I've made a mistake.. forgot to take it out of the pool and was overrun by spread itself :( | 17:24 |
| cachio | niemeyer, :), np | 17:24 |
| cachio | niemeyer, I'll do t again | 17:24 |
| niemeyer | cachio: Sorry | 17:24 |
| cachio | no problem | 17:25 |
| mup | PR snapcraft#1788 opened: repo: error for packages with broken dependencies <Created by kalikiana> <https://github.com/snapcore/snapcraft/pull/1788> | 17:31 |
| sergiusens | kyrofa https://www.amazon.com/photos/share/7kcMwZif1Ap8HlPrWRwH7vaUz3mubXEwbzBVlmCxWyD | 17:36 |
| kyrofa | sergiusens, oh crap | 17:37 |
| kyrofa | What is that nested list? | 17:37 |
| sergiusens | kyrofa you were there! | 17:38 |
| kyrofa | sergiusens, are you saying you remember? ;) | 17:38 |
| sergiusens | kyrofa yes | 17:38 |
| kyrofa | Then why didn't you correct the proposal I made without the picture to reference? | 17:38 |
| sergiusens | kyrofa elopio ad-hoc meeting, now-ish(?) | 17:38 |
| sergiusens | kyrofa which one where? | 17:39 |
| kyrofa | In the breakdown doc | 17:39 |
| kyrofa | I'm good with a meeting | 17:39 |
| sergiusens | kyrofa oh, during the meeting I said it was incomplete, don't you recall? | 17:39 |
| sergiusens | tbh, I don't like what was decided but you guys were totally onboard with it during the rally | 17:40 |
| kyrofa | No, I don't recall. I recall you agreeing and then you used my time estimate for the Project, so... | 17:40 |
| kyrofa | Regardless, let's chat about what this is | 17:41 |
| sergiusens | kyrofa oh, the estimate is fine | 17:41 |
| sergiusens | sure | 17:41 |
| kyrofa | No it's not, I already spent a day on a format that doesn't support that picture | 17:41 |
| cachio | niemeyer, spread-14 | 17:41 |
| cachio | fedora-rawhide-64 | 17:42 |
| cjwatson | Hmm. I originally installed a snap (which I'm helping to develop) from the store. Then I installed a local version with --dangerous to try to debug something. Now I'd quite like to get back to the store version without losing the existing data. Is there any way to do that? "snap refresh --edge" won't let me | 17:43 |
| niemeyer | cachio: On it | 17:43 |
| cjwatson | ('error: snap "foo" is local') | 17:43 |
| cachio | niemeyer, tx | 17:43 |
| elopio | om26er: hello. I'm back | 17:44 |
| niemeyer | cjwatson: Does "install" work? We've discussed this a few times, and I remember we agreed we'd like to fix that, but I don't recall where we stand to be honest | 17:44 |
| cjwatson | niemeyer: It doesn't, and tells me to use refresh - but ah, I just found https://forum.snapcraft.io/t/cannot-refresh-from-store-over-local-install/2542 | 17:44 |
| cjwatson | (which is further amusing because "snap download" doesn't support private snaps, I think, but I can work around that) | 17:45 |
| niemeyer | cjwatson: Ah, it's the opposite.. install allows the going out of store step | 17:45 |
| elopio | sergiusens: meeting where? | 17:45 |
| cjwatson | Yeah - it let me go out of store but didn't tell me that it would be hard to go back. I probably should've known :) | 17:46 |
| sergiusens | kyrofa elopio our weekly hngout | 17:46 |
| kyrofa | Alright, on my way | 17:46 |
| elopio | omw | 17:46 |
| cachio | niemeyer, I'll be afk, I'll try the new image once I am back | 17:47 |
| niemeyer | cjwatson: I think we should allow the transition towards the store more smoothly.. we may need that flag as we're going from something completely unsigned and unknown to something from the store, but other than than don't see many issues | 17:47 |
| niemeyer | cachio: Image is back at 2GB | 17:48 |
| cjwatson | niemeyer: OK, and yeah, I don't either - mind if I quote you on that forum thread? | 17:48 |
| niemeyer | cjwatson: Let me write something there | 17:48 |
| cjwatson | Cheers, appreciate it | 17:48 |
| cachio | niemeyer, could yoy plarse start the machine, I'll take a look | 17:48 |
| niemeyer | cachio: Coming up.. still out of the pool | 17:49 |
| niemeyer | cjwatson: np | 17:49 |
| cachio | niemeyer, tx | 17:49 |
| elopio | sergiusens: waiting for you | 17:50 |
| m4sk1n_ | can anyone help me with making snapcraft i18n-able? | 18:02 |
| niemeyer | cjwatson: Please have a look | 18:06 |
| niemeyer | m4sk1n_: Many of us can, but this is really a topic better held in the forum | 18:07 |
| niemeyer | m4sk1n_: as there are many angles to it, and feature design involved | 18:07 |
| m4sk1n_ | niemeyer: ok, I have problems with applying .mo files, I don't know how to use gettext with python in this case | 18:10 |
| m4sk1n_ | I guess that python-related channel is better place for this, but I thought that people here would like to help me in this case | 18:14 |
| cjwatson | niemeyer: yep, LGTM, thanks - replied | 18:16 |
| mup | PR snapcraft#1782 closed: project_loader: support grammar on architectures <Created by kyrofa> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/1782> | 18:20 |
| jjohansen | ikey: the plan is to move the backport kernels to gitlab, but I just haven't had time to do the work yet | 18:22 |
| jjohansen | I am planning on doing some backport kernel work this week, so hopefully it will happen | 18:22 |
| jjohansen | submissions to upstream will still go through the kernel.org tree, but I will be relying on kernel.ubuntu.com a lot less | 18:23 |
| jjohansen | it will be for only ubuntu specific patches, and hopefully that diff will disappear soon | 18:23 |
| sergiusens | m4sk1n_ is there a task to add translations to snapcraft itself? | 19:08 |
| sergiusens | elopio ? ^ | 19:08 |
| m4sk1n_ | no | 19:08 |
| m4sk1n_ | in fact I completed task to fix snapcraft bug | 19:08 |
| elopio | sergiusens: adding translations to the snapcraft cli? That's a huge task. | 19:11 |
| mup | PR snapcraft#1776 closed: ci: name the travis jobs <Created by sergiusens> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/1776> | 19:17 |
| m4sk1n_ | I don't know how to make it use .mo files | 19:19 |
| m4sk1n_ | I have snapcraft repo in ~/snapcraft/ | 19:21 |
| m4sk1n_ | here I have po/ folder with .pot template, pl.po and pl.mo (incomplete, for testing purposes) | 19:22 |
| m4sk1n_ | all strings are marked as translatable (these which should be translatable) and I generated .pot without any issues | 19:22 |
| jdstrand | popey: hey, fyi, 3 more revisions of vlc came in with the same desktop file issue | 19:32 |
| mup | PR snapcraft#1789 opened: cli: improve the help command <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/1789> | 20:20 |
| sergiusens | kyrofa elopio ^ | 20:21 |
| cachio | niemeyer, spread-14 ready again | 20:22 |
| cachio | niemeyer, I was uninstalling more stuff to reduce the size | 20:23 |
| sergiusens | elopio kyrofa the rpath one is updated, please review | 20:32 |
| om26er | elopio: who shall I ping to promote snapcraft 2.35 from xenial-proposed to -updates ? | 20:45 |
| om26er | ...or do we have a SRU bug for it ? | 20:47 |
| om26er | at least 2 Apps that I snapped depend on a fix in snapcraft 2.35. AndroidStudio and SublimeText. | 20:48 |
| om26er | popey: is there a way you can grant me permissions to close issues on https://github.com/snapcrafters/android-studio (Not asking for write access to repository). | 20:51 |
| mup | PR snapcraft#1790 opened: cli: include consistent commands to fix error conditions <Created by elopio> <https://github.com/snapcore/snapcraft/pull/1790> | 20:53 |
| elopio | sergiusens: before leaving, I +1'ed the SRU. Is there something else you want to verify there, or should I update the tags to get it released? | 20:55 |
| om26er | please do it ;-) | 20:58 |
| kyrofa | elopio, om26er we have that arm test failing that I'm investigating right now | 21:04 |
| kyrofa | Building a ROS snap on an rpi is not blazingly fast I'm afraid | 21:05 |
| elopio | om26er: for the record, this is the bug: https://bugs.launchpad.net/ubuntu/+source/snapcraft/+bug/1729417 | 21:05 |
| mup | Bug #1729417: [SRU] New stable micro release 2.35 <verification-done-xenial> <verification-done-zesty> <verification-needed> <verification-needed-artful> <snapcraft (Ubuntu):Fix Committed> <snapcraft (Ubuntu Xenial):Fix Committed> <snapcraft (Ubuntu Zesty):Fix Committed> <snapcraft (Ubuntu | 21:05 |
| mup | Artful):Fix Committed> <snapcraft (Ubuntu Bionic):Fix Committed> <https://launchpad.net/bugs/1729417> | 21:05 |
| elopio | sergiusens: kyrofa: okay, my 4 PRs ready for review/landing. I'm going for lunch now and get back to review rpath. | 21:07 |
| om26er | elopio: thanks, due to some reason I was looking for the SRU bug on github. | 21:07 |
| kyrofa | noise][, nessita the dashboard is giving me a 504 | 21:08 |
| noise][ | kyrofa: looks ok for me - any particular page? | 21:10 |
| kyrofa | noise][, just https://dashboard.snapcraft.io/dev/snaps/ . Came up now though | 21:11 |
| kyrofa | Just seemed to be weird for about two minutes or so | 21:11 |
| kyrofa | Oh for heaven's sake. The core snap just updated and rebooted the pi while it was running my test | 21:19 |
| om26er | don't give up, we are counting on you :P | 21:22 |
| mup | PR snapcraft#1758 closed: tests: move the ppa test trigger to lxd <Created by elopio> <Merged by kyrofa> <https://github.com/snapcore/snapcraft/pull/1758> | 21:26 |
| kyrofa | Haha, it's going, just had to start over | 21:32 |
| niemeyer | cachio: Image baked | 21:58 |
| kyrofa | sergiusens, elopio I'm afraid I can't duplicate these results on my rpi2: https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-xenial/xenial/armhf/s/snapcraft/20171204_024823_8b570@/log.gz everything works fine | 22:01 |
| kyrofa | The changelog has the same md5, etc. | 22:01 |
| * kyrofa goes to shoot some footage, back soon | 22:13 | |
| sergiusens | kyrofa ok, going to retrigger then | 22:29 |
| sergiusens | elopio kyrofa but don't forget to review! I would like to fix and have you guys ack the fixes before EOD so we have a smooth release day tomorrow | 22:30 |
| diddledan | RELEASE DAY TOMORROW?! \o/ YEY \o/ | 22:43 |
| diddledan | what we unleashing, the dogs of war? | 22:43 |
| diddledan | cry havoc and let slip the dogs of war! | 22:43 |
| lundmar | I'm looking for a +1 on this https://forum.snapcraft.io/t/request-autoconnect-avahi-observe-plug-for-lxi-tools/2976 :) | 22:44 |
| cachio | niemeyer, thanks | 23:05 |
| cachio | niemeyer, fedora-rawhide-64 not working with grub in linode | 23:52 |
| cachio | niemeyer, not sure why | 23:52 |
| cachio | if yuou could take a look to the console perhaps there is more info | 23:52 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!