| mup | PR snapcraft#2568 closed: docs: consolidate on a simple HACKING.md <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2568> | 00:07 |
|---|---|---|
| === jamesh_ is now known as jamesh | ||
| mborzecki | morning | 05:06 |
| zyga | Hey mborzecki | 06:00 |
| mborzecki | zyga: hey | 06:00 |
| mborzecki | zyga: how's your day off coming? :) | 06:00 |
| zyga | Haha | 06:00 |
| zyga | I was just wondering the same thing | 06:00 |
| zyga | Should I take today off for real | 06:00 |
| zyga | Or work some more to get stuff done | 06:00 |
| zyga | So many things to work on | 06:01 |
| zyga | mborzecki: I sent a small patch for spread-shellcheck | 06:05 |
| zyga | And had an idea on how to fix not MATCH | 06:05 |
| zyga | We can save MATCH to lib/bin on startup | 06:07 |
| zyga | Like we do now to match.sh include file | 06:07 |
| zyga | Since it is on path it will be more flexible | 06:07 |
| zyga | Then MATCH is no longer a function | 06:08 |
| zyga | Then we can put “not” on path and fix all tests | 06:08 |
| mborzecki | zyga: in other news, got an image built by snap-image (strawman) to boot up to a point where i got console conf :P | 06:15 |
| zyga | Wooooot | 06:16 |
| zyga | That is great | 06:16 |
| zyga | It still feels that landing your changes will take a month | 06:17 |
| mborzecki | yeah, and tests and all that | 06:17 |
| mborzecki | zyga: if you guys have a system-rescue role that you'd like me to add or something, this coudl be done really easily | 06:18 |
| zyga | Role as in partition type? | 06:19 |
| zyga | I think we should talk about how the recovery system is designed, so far, to keep you in the loop | 06:19 |
| mborzecki | zyga: role as role in gadget.yaml terms, and whatever that means practically, eg. like system-boot is a bit special | 06:19 |
| mborzecki | zyga: and so is system-data | 06:19 |
| zyga | I think some things there will change | 06:20 |
| zyga | Though I think that is best for next week | 06:20 |
| zyga | re, back in the office | 06:37 |
| zyga | mborzecki: can you do a quick review of https://github.com/snapcore/snapd/pull/6906 | 06:38 |
| zyga | just a variable rename | 06:38 |
| mup | PR #6906: cmd/snap-update-ns: rename ctx to upCtx <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6906> | 06:38 |
| zyga | though more importantly I need a review for https://github.com/snapcore/snapd/pull/6903 to get 2.39.1 fix in place | 06:38 |
| mup | PR #6903: spread-shellcheck: add support for variants and environment <Created by zyga> <https://github.com/snapcore/snapd/pull/6903> | 06:38 |
| zyga | the shellcheck change is not perfect | 06:38 |
| zyga | but it's also not totally incorrect | 06:38 |
| zyga | and allows my tests to not get stuck on that | 06:38 |
| * zyga quick coffee | 07:01 | |
| zyga | mborzecki: if you can review the spread-shellcheck change I will be able to propose the propagation fixes | 07:01 |
| zyga | I will do some branch gardening soon | 07:03 |
| zyga | but first coffee | 07:03 |
| pstolowski|afk | morning | 07:04 |
| === pstolowski|afk is now known as pstolowski | ||
| zyga | Hey :-) | 07:09 |
| zyga | Interesting https://forum.snapcraft.io/t/snapshots-can-expose-sensitive-data/11493 | 07:19 |
| pstolowski | zyga: hmm interesting indeed | 07:27 |
| mup | PR snapd#6906 closed: cmd/snap-update-ns: rename ctx to upCtx <Simple 😃> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6906> | 07:27 |
| zyga | makes me feel that snapshots should be saved back to $HOME | 07:27 |
| zyga | as anything else is not correct wrt NFS and encryption | 07:27 |
| zyga | pstolowski: quick trivial https://github.com/snapcore/snapd/pull/6907 | 07:28 |
| mup | PR #6907: cmd/snap-update-ns: add several TODO comments <Created by zyga> <https://github.com/snapcore/snapd/pull/6907> | 07:28 |
| zyga | just comments | 07:28 |
| zyga | but that's *all* of the refactoring :D | 07:28 |
| zyga | wooooot | 07:28 |
| mup | PR snapd#6907 opened: cmd/snap-update-ns: add several TODO comments <Created by zyga> <https://github.com/snapcore/snapd/pull/6907> | 07:28 |
| pstolowski | +1§ | 07:29 |
| mborzecki | pstolowski: hey | 07:30 |
| pstolowski | o/ | 07:30 |
| mup | PR snapd#6894 closed: tests: add "not" command as replacement for "!" in tests <Created by zyga> <Closed by zyga> <https://github.com/snapcore/snapd/pull/6894> | 07:40 |
| mup | PR snapd#6908 opened: tests: add "not" command <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6908> | 07:44 |
| zyga | mborzecki, pstolowski: how do you feel about ^ not | 08:05 |
| zyga | I want to build this on top https://github.com/snapcore/snapd/pull/6909 | 08:05 |
| mup | PR #6909: spread.yaml,tests: change MATCH and REBOOT to cmds <Created by zyga> <https://github.com/snapcore/snapd/pull/6909> | 08:05 |
| zyga | and then fix our scripts to use not rather than ! | 08:05 |
| mup | PR snapd#6909 opened: spread.yaml,tests: change MATCH and REBOOT to cmds <Created by zyga> <https://github.com/snapcore/snapd/pull/6909> | 08:06 |
| pstolowski | zyga: interesting; don't we still need to make it a compound expressionm e.g. { ! "$@" } ? | 08:11 |
| zyga | no, why? | 08:11 |
| zyga | ! functions correctly | 08:12 |
| zyga | note that this executable is not sourced, it's a program that runs without set -e | 08:12 |
| pstolowski | zyga: ok, so -e is the key here. i was reading also https://stackoverflow.com/questions/39581150/why-do-i-need-parenthesis-in-bash-set-e-and-negated-return-code/39582012 | 08:12 |
| === icey_ is now known as icey | ||
| zyga | I wonder if snapd is going to get any sponsors: https://techcrunch.com/2019/05/23/github-launches-sponsors-lets-you-pay-your-favorite-open-source-contributors/ | 08:22 |
| zyga | mborzecki: https://github.com/google/pytype | 08:40 |
| zyga | mborzecki: from that video you linkedto | 08:40 |
| zyga | mborzecki: updated https://github.com/snapcore/snapd/pull/6903 | 08:51 |
| mup | PR #6903: spread-shellcheck: add support for variants and environment <Created by zyga> <https://github.com/snapcore/snapd/pull/6903> | 08:51 |
| mborzecki | zyga: nhm, lgtm | 08:55 |
| zyga | nhm? | 08:56 |
| zyga | mborzecki: something for you :) https://github.com/snapcore/snapd/pull/6910 | 09:00 |
| mup | PR #6910: spread.yaml: use "snap connections" in debug <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6910> | 09:00 |
| mup | PR snapd#6910 opened: spread.yaml: use "snap connections" in debug <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6910> | 09:00 |
| * zyga really gets that coffee now | 09:03 | |
| zyga | after the break I will try to land "not" and "MATCH" as commands and see if I can fix ! tree-wide | 09:03 |
| zyga | and revert shellcheck patch so that we track edge again | 09:03 |
| zyga | back now :) | 09:17 |
| zyga | aaaand, "not" is red | 09:17 |
| mup | PR snapd#6907 closed: cmd/snap-update-ns: add several TODO comments <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6907> | 09:20 |
| pstolowski | and i just had google:ubuntu-14.04-64:tests/main/parallel-install-aliases failure on one of my PRs | 09:35 |
| pstolowski | never seen it failing | 09:35 |
| zyga | pstolowski: how did it fail? | 09:38 |
| pstolowski | zyga: it was expecting the change to fail with 'cannot enable alias "alias1" for "aliases_foo", already enabled for "aliases"', but instead got 'Done today at 15:27 UTC today at 15:27 UTC Setup manual alias "alias1" => "cmd1" for snap "aliases"' | 09:40 |
| pstolowski | zyga: i wonder it snap change --last=alias check there isn't racy | 09:40 |
| zyga | huh, I saw that fail once as well | 09:41 |
| zyga | yesterday | 09:41 |
| pstolowski | because just above we test happy execution | 09:41 |
| pstolowski | zyga: i'll kick it again on travis and in the meantime execute and investigate it locally | 09:41 |
| zyga | k | 09:41 |
| pstolowski | and if that's it, will fix it | 09:42 |
| zyga | pstolowski: can you look at https://github.com/snapcore/snapd/pull/6903 | 09:42 |
| mup | PR #6903: spread-shellcheck: add support for variants and environment <Created by zyga> <https://github.com/snapcore/snapd/pull/6903> | 09:42 |
| pstolowski | k | 09:42 |
| zyga | I need a 2nd review to open the fix for the mount propagation PR | 09:42 |
| zyga | mborzecki: curious, permissions package seems to be updated with some snapd bits | 09:52 |
| zyga | as a simple build in obs failed on that | 09:52 |
| * zyga looks | 09:52 | |
| zyga | settle not converging | 09:53 |
| * zyga loves when unit tests fail | 09:53 | |
| zyga | pstolowski: does this sound familiar? | 09:54 |
| zyga | https://www.irccloud.com/pastebin/02IEvvjO/ | 09:54 |
| zyga | ah, this is device registration, not hotplug | 09:55 |
| zyga | sorry | 09:55 |
| zyga | bah | 09:55 |
| zyga | this happens each time | 09:56 |
| * zyga debugs | 09:56 | |
| mup | PR snapd#6893 closed: gadget: helper for shifting structure start position <Gadget update> <Simple 😃> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6893> | 10:01 |
| zyga | hrm | 10:01 |
| zyga | go test passes on my tree with 2.39 | 10:01 |
| zyga | fails in the tarball | 10:01 |
| zyga | whhyyyyy | 10:01 |
| zyga | hmmm | 10:05 |
| zyga | I disabled unit tests and installed the resulting package | 10:14 |
| zyga | let's see how it operates | 10:14 |
| zyga | then I can see why they fail again | 10:14 |
| zyga | eh | 10:20 |
| zyga | in x11 mode I don't have /snap/bin on path | 10:20 |
| zyga | in wayland mode some apps just crash | 10:20 |
| zyga | lovely | 10:20 |
| zyga | after reboot things seem to be have | 10:23 |
| zyga | behave even | 10:23 |
| pstolowski | zyga: hmm somewhat, seen failures like this from time to time while working on these tests, usually was a race. you're saying it fails each time like this? | 10:35 |
| zyga | do you remember https://github.com/snapcore/snapd/pull/6360 <- I just merged master into it :) | 10:35 |
| mup | PR #6360: cmd/snap-update-ns: refactor of profile application <Per-user mount ns 🐎> <Created by zyga> <https://github.com/snapcore/snapd/pull/6360> | 10:35 |
| zyga | pstolowski: yeah | 10:35 |
| zyga | but only in packaging build | 10:35 |
| zyga | not in a checkout | 10:35 |
| zyga | I didn't look deeper yet | 10:35 |
| zyga | "not" failed again | 10:36 |
| pstolowski | zyga: on which test? | 10:37 |
| zyga | service watchdog | 10:37 |
| pstolowski | zyga: +1 on 6903 with a question | 10:51 |
| zyga | looking | 10:51 |
| pstolowski | of course parallel-install-aliases is passing when run locally... will keep it running during lunch | 10:55 |
| zyga | pstolowski: replied there | 10:56 |
| pstolowski | ty | 11:00 |
| * pstolowski lunch | 11:00 | |
| mup | PR snapd#6903 closed: spread-shellcheck: add support for variants and environment <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6903> | 11:03 |
| zyga | I've opened https://github.com/snapcore/snapd/pull/6891 for real now | 11:04 |
| mup | PR #6891: many: make per-snap mount namespace MS_SHARED <Created by zyga> <https://github.com/snapcore/snapd/pull/6891> | 11:04 |
| zyga | tests *should* pass now | 11:04 |
| zyga | so fingers crossed | 11:04 |
| zyga | I think I need a break | 11:05 |
| zyga | but it's moving along | 11:05 |
| zyga | what do you guys think about https://github.com/snapcore/snapd/pull/6909 ? | 11:05 |
| mup | PR #6909: spread.yaml,tests: change MATCH and REBOOT to cmds <Created by zyga> <https://github.com/snapcore/snapd/pull/6909> | 11:05 |
| zyga | I will work on mountinfo-tool to support earlier versions of python so that we can use it in 16.04 | 11:05 |
| mup | PR snapd#6911 opened: cmd/snap-update-ns: rename leftover ctx to upCtx <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6911> | 11:09 |
| * zyga breaks | 11:11 | |
| mup | PR snapd#6912 opened: gadget: keep track of the index where structure content was defined <Gadget update> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6912> | 11:14 |
| mup | PR snapd#6883 closed: tests: fix how strings are matched on auto-refresh-retry test <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6883> | 11:15 |
| mup | PR snapd#6910 closed: spread.yaml: use "snap connections" in debug <Simple 😃> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6910> | 11:15 |
| zyga | mborzecki: is https://github.com/snapcore/snapd/pull/6890 really for review? | 11:17 |
| mup | PR #6890: gadget: mounted filesystem writer & updater <Gadget update> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6890> | 11:17 |
| mborzecki | zyga: heh, yes | 11:17 |
| zyga | *gulp* | 11:17 |
| mborzecki | zyga: want to dive in? :) | 11:17 |
| zyga | they told me to split a 1K diff | 11:18 |
| zyga | took several moths to review | 11:18 |
| zyga | *months | 11:18 |
| zyga | (i'd be faster with moths :D | 11:18 |
| mborzecki | zyga: 1.6k is tests :P | 11:18 |
| zyga | for me 80% was tests | 11:18 |
| zyga | good luck :-) | 11:18 |
| zyga | no way to chunk that? | 11:18 |
| mborzecki | idk, maybe i could sample, method by method :/ | 11:19 |
| zyga | at the end of the day I can review and approve but you need extra reviews from samuele | 11:19 |
| mborzecki | hm maybe i could pull the writer thing apart, that's ~10% of the thing | 11:19 |
| zyga | ok, really time to break now | 11:20 |
| zyga | I'll merge "not" when green | 11:20 |
| zyga | and if you guys can review https://github.com/snapcore/snapd/pull/6909 we could fix all of ! foo bugs | 11:20 |
| mup | PR #6909: spread.yaml,tests: change MATCH and REBOOT to cmds <Created by zyga> <https://github.com/snapcore/snapd/pull/6909> | 11:20 |
| mup | PR snapd#6913 opened: tests: force snap-reexec test to fail to reproduce the error on ubuntu 14.04 <⛔ Blocked> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6913> | 11:20 |
| mborzecki | zyga: should we have NOT instead of lowercase not to match MATCH and REBOOT? | 11:29 |
| popey_ | chipaca not working today? | 11:36 |
| zyga | mborzecki: MATCH and REBOOT come from spread | 11:46 |
| zyga | popey_: I think he's off today | 11:46 |
| zyga | mborzecki: I think lowercase "not" looks nicer | 11:46 |
| popey_ | ta | 11:46 |
| zyga | mborzecki: but to answer your question, I kept REBOOT and MATCH uppercase only to match spread names and avoid code changes | 11:47 |
| zyga | not failed again :D | 11:47 |
| zyga | heh | 11:47 |
| zyga | landing a trivial unused file | 11:47 |
| mborzecki | pstolowski: a look at https://github.com/snapcore/snapd/pull/6912 ? | 11:47 |
| mup | PR #6912: gadget: keep track of the index where structure content was defined <Gadget update> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6912> | 11:47 |
| zyga | this time on refresh-delta-from-core | 11:48 |
| zyga | meh | 11:48 |
| zyga | I'll grab some food | 11:48 |
| zyga | ttyl | 11:48 |
| === ricab is now known as ricab|lunch | ||
| === alan_g is now known as alan_g|lunch | ||
| zyga | tests still churning ... | 12:19 |
| mborzecki | zyga: can you do another pass of #6750? you did one there already, and I pushed some changes in the meantime | 12:35 |
| mup | PR #6750: overlord/devicestate: update-gadget task handler with stubbed gadget callbacks <Gadget update> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6750> | 12:35 |
| zyga | yep | 12:35 |
| mup | PR snapd#6908 closed: tests: add "not" command <Simple 😃> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6908> | 12:35 |
| zyga | nice! | 12:35 |
| zyga | mborzecki: can you discuss that portal test change PR with cachio please? | 12:36 |
| zyga | mborzecki: https://github.com/snapcore/snapd/pull/6912 needs formatting fixes | 12:37 |
| mup | PR #6912: gadget: keep track of the index where structure content was defined <Gadget update> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6912> | 12:37 |
| cachio | zyga, sure | 12:37 |
| zyga | probably need to reformat with ancient go | 12:37 |
| zyga | (yuck) | 12:37 |
| zyga | settle not converging (unit tests) https://www.irccloud.com/pastebin/KgUHJ07T/ | 12:38 |
| cachio | mborzecki, did you see the comments in the PR? | 12:38 |
| zyga | loads of those | 12:38 |
| mborzecki | omg, i hate they made the formatting change in 1.11 | 12:42 |
| mborzecki | super annoying | 12:42 |
| jdstrand | zyga: fyi, I added a sprinking of emojis, a comment on your response to /etc and then also https://github.com/snapcore/snapd/pull/6891#issuecomment-495609846 | 12:51 |
| mup | PR #6891: many: make per-snap mount namespace MS_SHARED <Created by zyga> <https://github.com/snapcore/snapd/pull/6891> | 12:51 |
| zyga | :-) | 12:51 |
| zyga | thanks! | 12:51 |
| zyga | I noticed it failed for real in one specific test | 12:51 |
| zyga | looking now | 12:51 |
| zyga | pstolowski: standup | 13:01 |
| pstolowski | yep | 13:01 |
| Wimpress | Snapcraft Live is just starting - https://www.youtube.com/watch?v=yAnRM4m63MY | 13:01 |
| zyga | Wimpress: nice! | 13:02 |
| mborzecki | Wimpress: nice, shirt, tie and shorts right? :) | 13:02 |
| zyga | mborzecki: *hopefully* there are shorts | 13:03 |
| mborzecki | the upsides of working remotely :) | 13:03 |
| mup | PR snapd#6911 closed: cmd/snap-update-ns: rename leftover ctx to upCtx <Simple 😃> <Created by zyga> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6911> | 13:09 |
| === alan_g|lunch is now known as alan_g | ||
| === ricab|lunch is now known as ricab | ||
| zyga | jdstrand: I consistently get a denial for a permission I hold | 13:39 |
| zyga | perhaps something quirky with mount options and 14.04 (4.4) kernel | 13:39 |
| jdstrand | zyga: does it go away with a bare mount rule? | 13:39 |
| zyga | jdstrand: just "mount," ? | 13:39 |
| jdstrand | yes | 13:40 |
| zyga | jdstrand: curiously, it does not | 13:40 |
| zyga | ahhh | 13:40 |
| zyga | I'm a tool | 13:40 |
| zyga | sorryt | 13:41 |
| zyga | reexec bit me | 13:41 |
| jdstrand | heh. it always does at least once ;) | 13:41 |
| zyga | jdstrand: bare mount rule is ok | 13:42 |
| zyga | let's see if without it it also works :D | 13:42 |
| zyga | so it works but I don't know why ... let's restart and see | 13:43 |
| zyga | jdstrand: I think something is still fishy but I cannot point to anything apart from the failure in spread | 13:44 |
| zyga | jdstrand: I tweaked things a little os that they read: | 13:46 |
| zyga | relevant fragment of apparmor permissions https://www.irccloud.com/pastebin/v267dsbt/ | 13:46 |
| pstolowski | zyga: do you have anything needing 2nd review that i can take a look at? | 13:46 |
| zyga | the denial I got is associated with permission on line 6 | 13:46 |
| zyga | pstolowski: https://github.com/snapcore/snapd/pull/6891 | 13:46 |
| zyga | it's not 2nd but it needs review | 13:47 |
| mup | PR #6891: many: make per-snap mount namespace MS_SHARED <Created by zyga> <https://github.com/snapcore/snapd/pull/6891> | 13:47 |
| zyga | that's the 2.39.1 patch | 13:47 |
| jdstrand | zyga: so, you are saying there is no apparmor denial, but an eperm, but with a bare mount rule, it all works? | 13:48 |
| zyga | there's a denial | 13:48 |
| zyga | bare mount fixes it | 13:48 |
| jdstrand | what is the denial? | 13:48 |
| zyga | the denial is | 13:49 |
| zyga | May 24 13:41:11 may241325-532686 kernel: audit: type=1400 audit(1558705271.095:74): apparmor="DENIED" operation="mount" info="failed flags match" error=-13 profile="/snap/snapd/x1/usr/lib/snapd/snap-confine" name="/tmp/snap.rootfs_dPNxms/" pid=18844 comm="snap-confine" flags="rw, rshared" | 13:49 |
| jdstrand | you're missing rw | 13:49 |
| zyga | but wait, rw I had | 13:49 |
| zyga | I removed it | 13:49 |
| zyga | (also rw makes no sense for propagation changes) | 13:50 |
| jdstrand | I can only comment on what you have shown me | 13:50 |
| jdstrand | :) | 13:50 |
| zyga | yeah I know :) | 13:50 |
| jdstrand | zyga: I'm not sure why rw would pop out. that may be a bug or it may be some subtlety in the kernel that needs it cause it is shared | 13:51 |
| jdstrand | seeing the strace of that might help... | 13:51 |
| zyga | yeah | 13:51 |
| roadmr | jdstrand: hey! tools r... wait... checking the actual version... "20190522-1231UTC click-reviewers-tools version " is now in production \o/ I think you can now get rid of your bzr branch | 13:53 |
| jdstrand | zyga: I mean, looking at the strace I have here, the rbinds don't specify rw, but there it is on line 4 of your paste | 13:53 |
| jdstrand | roadmr: yay!! | 13:53 |
| jdstrand | \o\ | 13:54 |
| jdstrand | /o/ | 13:54 |
| jdstrand | \o\ | 13:54 |
| jdstrand | \o/ | 13:54 |
| * jdstrand hugs roadmr :) | 13:54 | |
| roadmr | 🙌 hehe | 13:54 |
| jdstrand | zyga: I suspect if you add the rw and account for reexec, you will be on your way | 13:55 |
| zyga | there's rw there already | 13:55 |
| zyga | it's not that | 13:55 |
| zyga | I'm debugging | 13:55 |
| * jdstrand is confused | 13:56 | |
| zyga | jdstrand: there *was* rw, I removed it thinking it is the issue | 13:57 |
| jdstrand | zyga: but reexec? | 13:59 |
| zyga | I changed the source and re-ran spread | 13:59 |
| zyga | gah, I'm tired | 14:00 |
| zyga | jdstrand: I merged master into the ancient huge PRs | 14:00 |
| zyga | jdstrand: https://github.com/snapcore/snapd/pull/6341 is interesting (diff wise small) | 14:01 |
| mup | PR #6341: WIP: persist per-user mount namespace profile <Created by zyga> <https://github.com/snapcore/snapd/pull/6341> | 14:01 |
| mup | Bug #1830382 opened: Have a progress indicator for snap pack (like with snapcraft pack) <Snappy:New> <https://launchpad.net/bugs/1830382> | 14:02 |
| zyga | jdstrand: with that branch I should be able to answer your questions about the set of required capabilities | 14:03 |
| jdstrand | these PRs are like rabbits | 14:05 |
| mup | Bug #1830382 changed: Have a progress indicator for snap pack (like with snapcraft pack) <snapd:Triaged> <https://launchpad.net/bugs/1830382> | 14:05 |
| jdstrand | I review a bunch and then end up with more than I started with | 14:05 |
| zyga | yeah | 14:05 |
| sparkiegeek | mvo, zyga: btw, shouldn't https://bugs.launchpad.net/snapd/+bug/1819318 be at least Fix Committed? and arguably Fix Released? | 14:07 |
| mup | Bug #1819318: no interfaces when installing only core18 <snapd:In Progress> <https://launchpad.net/bugs/1819318> | 14:07 |
| zyga | sparkiegeek: in a call | 14:08 |
| zyga | mvo worked on that | 14:08 |
| zyga | jdstrand: https://github.com/snapcore/snapd/pull/6891#issuecomment-495643768 I'm giving up now | 14:15 |
| mup | PR #6891: many: make per-snap mount namespace MS_SHARED <Created by zyga> <https://github.com/snapcore/snapd/pull/6891> | 14:15 |
| zyga | need to take a walk and think | 14:15 |
| pstolowski | a few hours later and... parallel-install-aliases doesn't want to fail | 14:22 |
| mup | PR snapd#6912 closed: gadget: keep track of the index where structure content was defined <Gadget update> <Simple 😃> <Created by bboozzoo> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6912> | 14:22 |
| mup | PR snapd#6914 opened: tests: change strace parameters on snap-run test to avoid the test gets stuck <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6914> | 15:34 |
| * cachio lunch | 15:57 | |
| === pstolowski is now known as pstolowski|Afk | ||
| * zyga feels sick somehow, EODing now | 16:55 | |
| mup | Bug #1830412 opened: QT5 classic snaps segmentation fault <Snappy:New> <https://launchpad.net/bugs/1830412> | 17:10 |
| * cachio afk | 18:59 | |
| mup | PR pc-amd64-gadget#17 closed: add .travis.yml to enable basic smoke testing (20) <Created by mvo5> <Merged by vorlonofportland> <https://github.com/snapcore/pc-amd64-gadget/pull/17> | 20:53 |
| mup | PR pc-amd64-gadget#16 closed: add .travis.yml to enable basic smoke testing (18) <Created by mvo5> <Merged by vorlonofportland> <https://github.com/snapcore/pc-amd64-gadget/pull/16> | 20:55 |
| mup | PR pc-amd64-gadget#15 closed: add .travis.yml to enable basic smoke testing <Created by mvo5> <Merged by vorlonofportland> <https://github.com/snapcore/pc-amd64-gadget/pull/15> | 21:00 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!