| === mup_ is now known as mup | ||
| === zigford_ is now known as zigford | ||
| mup | PR snapcraft#2686 closed: New app handler (command-chain and no wrappers if command allows for it) <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2686> | 03:14 |
|---|---|---|
| mup | PR snapcraft#2560 closed: meta: do not wrap commands by default <Created by sergiusens> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/2560> | 03:24 |
| === zigford_ is now known as zigford | ||
| mborzecki | morning | 04:58 |
| mup | PR snapd#7457 closed: tests/main/many: increase kill-timeout to 5m <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7457> | 05:18 |
| mup | PR snapd#7458 closed: interfaces/kubernetes-support: allow systemd-run to ptrace read unconfined <Simple 😃> <Created by jdstrand> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7458> | 05:18 |
| mborzecki | mvo: morning | 05:50 |
| mvo | mborzecki: hey, good morning! | 05:57 |
| zyga | Biking home | 06:01 |
| mvo | hey zyga | 06:09 |
| mvo | zyga: nice, already doing a workout? | 06:09 |
| zyga | yes | 06:21 |
| zyga | every morning | 06:21 |
| zyga | uff :) | 06:21 |
| zyga | this and a standing desk does work wonders | 06:21 |
| zyga | and every time I complain about my back it's really when I was lazy and skipped one or the other | 06:21 |
| zyga | mvo: I'm sorry about not finishing work last night, I went to bed at 9 | 06:22 |
| zyga | last day before paris | 06:22 |
| zyga | let's make it count! | 06:22 |
| mborzecki | brb, new kernel (and gnome) | 06:30 |
| mborzecki | re | 06:32 |
| mvo | mborzecki: you look into 7133 further? should we merge as is and you build on top? or do you want to work in the PR? | 06:57 |
| mborzecki | mvo: left anote and merged it | 06:59 |
| mup | PR snapd#7133 closed: overlord,daemon: adjust startup timeout via EXTEND_TIMEOUT_USEC using an estimate <Created by pedronis> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/7133> | 06:59 |
| mborzecki | mvo: systemd does not update TimeoutStartUSec even though snapd requests more time, perhaps that's how it's suposed to work anyway | 06:59 |
| mborzecki | - google-unstable:ubuntu-19.10-64:tests/main/interfaces-calendar-service | 07:04 |
| mborzecki | - google-unstable:ubuntu-19.10-64:tests/main/interfaces-contacts-service | 07:04 |
| mborzecki | - google-unstable:ubuntu-19.10-64:tests/unit/go | 07:04 |
| mborzecki | these fail on 19.10 | 07:04 |
| mborzecki | unit/go fails bc of gofmt | 07:04 |
| === pstolowski|afk is now known as pstolowski | ||
| pstolowski | morning | 07:06 |
| ackk | zyga, hi, is there a workaround for that limitation about only allowing home directories under /home in snaps? | 07:09 |
| ackk | zyga, I'm trying to use "lxc" from a jenkins job, but the jenkins user has home set to /var/lib/jenkins, so it fails | 07:10 |
| zyga | ackk: only one from the user, that is, to bind-mount the actual home directory to /home/$LOGNAME/ | 07:10 |
| ackk | zyga, I see | 07:10 |
| ackk | thanks | 07:11 |
| mborzecki | pstolowski: hey | 07:12 |
| ackk | zyga, I also need to update the passwd though? | 07:14 |
| zyga | ackk: not really | 07:14 |
| zyga | well | 07:14 |
| zyga | it's up to you | 07:14 |
| ackk | zyga, it's not working here | 07:14 |
| zyga | all that you need to ensure that anywhere your actual $HOME points | 07:14 |
| zyga | there's a bind mount of that to /home/$LOGNAME | 07:14 |
| zyga | hmm | 07:15 |
| zyga | on second though | 07:15 |
| zyga | thought | 07:15 |
| zyga | yes, please update passwd | 07:15 |
| zyga | that's something we don't manage | 07:15 |
| zyga | I'll make a note | 07:15 |
| zyga | we have some ideas how to map that internally but that might be derailed by gecos | 07:15 |
| ackk | zyga, ok, that seems to work, but not it seems snapcraft is looking for build-packages on the host | 07:21 |
| ackk | not in the container | 07:21 |
| ackk | oh, it seems the proxy config is not forwarded to the container :/ | 07:23 |
| ackk | same issue I was hitting with multipass | 07:23 |
| mup | PR snapd#7461 opened: run-checks, tests/main/go: allow gofmt checks to be skipped on 19.10 <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7461> | 07:45 |
| rogpeppe | zyga: FYI that pi is still going down at regular intervals, despite scheduling the updates only to be done once a month :-\ | 07:51 |
| zyga | rogpeppe: that's not good, can you give us the scheduling string please | 07:51 |
| zyga | rogpeppe: as well as "snap changes" | 07:51 |
| rogpeppe | zyga: will do, when it gets restarted again | 07:52 |
| zyga | rogpeppe: I procured another PI and I can now replicate your setup | 07:52 |
| rogpeppe | zyga: have you managed to replicate any of the issues i've been seeing? | 07:52 |
| zyga | rogpeppe: not yet, I needed to get the right hardware first | 07:52 |
| rogpeppe | zyga: ok, cool | 07:52 |
| zyga | rogpeppe: and I was doing other fixes in the meantime | 07:52 |
| zyga | rogpeppe: I will put regular core-18 on the device and then load your uboot.env to see what it claims | 07:53 |
| zyga | then I plan to get the same kernel / base | 07:53 |
| zyga | and same try mode kernel and base | 07:53 |
| zyga | and see if I can reboot correctly | 07:53 |
| zyga | with serial attached and all that | 07:53 |
| rogpeppe | zyga: i'm fascinated to find out if it all works just fine for you :) | 07:55 |
| zyga | rogpeppe: I hope it doesn't | 07:56 |
| zyga | rogpeppe: it's much harder to fix something that works for me | 07:57 |
| zyga | :) | 07:57 |
| pedronis | mvo: I answered to https://github.com/snapcore/snapd/pull/7416#discussion_r324077826 probably degville or Chipaca's input on that is valuable | 08:08 |
| mup | PR #7416: seed/seedwriter: start of Writer and internal policy16/tree16 <Created by pedronis> <https://github.com/snapcore/snapd/pull/7416> | 08:08 |
| degville | pedronis: I'll take a look. | 08:09 |
| zyga | mborzecki: I tweaked https://github.com/snapcore/snapd/pull/7442 based on your idea earlier | 08:11 |
| mup | PR #7442: tests: extend mount-ns test to handle mimics <Created by zyga> <https://github.com/snapcore/snapd/pull/7442> | 08:11 |
| zyga | the diff is short and pretty comprehensible | 08:12 |
| pedronis | degville: thanks | 08:49 |
| degville | pedronis: np. | 08:49 |
| mvo | pedronis: yeah, sorry for the bikesheed there | 08:55 |
| mvo | pedronis: aha, sorry, I missed part of the information. its snaps coming from the new "options" file, I missed that piece. let me medidate a bit but with that my suggestions are indeed not good | 08:56 |
| pedronis | mvo: not from a file, but is snaps coming from ubuntu-image --snaps | 08:57 |
| pedronis | or anything external from the model really | 08:57 |
| mvo | pedronis: yeah, I get it now | 08:57 |
| pedronis | given that is quite generic and ubuntu-image/prepare-image are just the first use case | 08:57 |
| mvo | pedronis: sorry, that was not clear to me, let me comment in the PR but with that my comment does not make much sense anymore | 08:58 |
| mvo | pedronis: +1 | 08:58 |
| pedronis | mvo: to be fair as I said the doc comment is not super clear | 08:58 |
| pedronis | and I should fix it | 08:58 |
| pedronis | also OptionsSnap might be less confusable with optional snap | 08:58 |
| mvo | pedronis: yeah, I like that | 08:59 |
| mvo | pedronis: and +1 on a comment fix, maybe with a small example like "E.g. OptionsSnaps may come via ubuntu-image --snaps ..." | 08:59 |
| pedronis | yea, as I said I might leave a todo for the actual rename to do later in the chain | 09:00 |
| pedronis | because otherwise I fear conflicts | 09:00 |
| pedronis | all over the place | 09:00 |
| pedronis | in it | 09:00 |
| ackk | how do I run a clean build with snapcraft --use-lxd ? | 09:00 |
| zyga | ackk: I'm not sure, sorry, perhaps one of the others knows | 09:04 |
| mvo | pedronis: +1 | 09:10 |
| zyga | hey | 09:16 |
| zyga | a quick question | 09:16 |
| zyga | does anyone know of "git diff" like tool | 09:16 |
| zyga | that shows differences inside one line | 09:16 |
| zyga | like what github can do | 09:17 |
| ackk | emacs :) | 09:17 |
| zyga | any command I can try to see it | 09:19 |
| ackk | zyga, perhaps git diff --word-diff does what you want? https://makandracards.com/makandra/6585-git-diff-changes-in-a-long-line-efficiently | 09:19 |
| zyga | I'm not after a two semester curriculum ;) | 09:19 |
| zyga | oh, that's probably exactly that, thank you so much | 09:19 |
| ackk | np, TIL too | 09:19 |
| * Chipaca goes for coffee | 09:24 | |
| mup | PR snapd#7462 opened: asserts: introduce explicit support for grade for Core 20 models <Created by pedronis> <https://github.com/snapcore/snapd/pull/7462> | 09:27 |
| mup | PR snapd#7463 opened: tests/main/interfaces-{calendar,contacts}-service: disable on 19.10 <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7463> | 09:29 |
| * Chipaca *really* goes for coffee | 09:30 | |
| Chipaca | augh | 09:42 |
| mup | PR snapd#7460 closed: interfaces/kubernetes-support: allow systemd-run to ptrace read unconfined - 2.41 <Simple 😃> <Created by jdstrand> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7460> | 09:48 |
| pstolowski | Chipaca: hey, wdyt about https://bugs.launchpad.net/snapd/+bug/1843725 ? should we simply de-duplicate args or error out right away with "nonsense argument given"? | 09:51 |
| mup | Bug #1843725: snap install with duplicated snap names gives confusing errors <snapd:Confirmed> <https://launchpad.net/bugs/1843725> | 09:51 |
| Chipaca | pstolowski: we need to make electrified keyboards mandatory | 09:59 |
| pstolowski | Chipaca: :) | 09:59 |
| Chipaca | pstolowski: more serious answer involves better checking for this case in both snapstate and cmd/snap | 10:02 |
| Chipaca | pstolowski: -1 to de-duplication | 10:03 |
| pedronis | mvo: applied feedback: https://github.com/snapcore/snapd/pull/7416/commits/fea6a6c7f694b9489f6655c1af66209f76ba576b | 10:03 |
| mup | PR #7416: seed/seedwriter: start of Writer and internal policy16/tree16 <Created by pedronis> <https://github.com/snapcore/snapd/pull/7416> | 10:03 |
| Chipaca | zyga: about in-line diff tool | 10:27 |
| Chipaca | zyga: it's in git itself, but not built | 10:27 |
| Chipaca | zyga: $ git config --get core.pager | 10:28 |
| Chipaca | /home/john/bin/diff-highlight | less | 10:28 |
| Chipaca | zyga: diff-highlight is /usr/share/doc/git/contrib/diff-highlight | 10:28 |
| zyga | thanks, I'll check that out | 10:28 |
| Chipaca | zyga: OTOH, snap install icdiff, git icdiff | 10:29 |
| zyga | it will come in handy for some of the changes I'm doing :) | 10:29 |
| Chipaca | zyga: side-by-side diff with in-line highlighting | 10:29 |
| Chipaca | zyga: something's buggy with the pager detection in icdiff so you want to set icdiff.pager explicitly, I need to dig into that this weekend | 10:31 |
| Chipaca | but it's otherwise good :-) | 10:31 |
| zyga | I'll check it out but I think you're way ahead of me :) | 10:31 |
| pstolowski | Chipaca: thanks for feedback on the bug | 10:37 |
| === ricab is now known as ricab|bbl | ||
| zyga | the diff on https://github.com/snapcore/snapd/pull/7436/files _finally_ makes some sense | 10:51 |
| mup | PR #7436: many: make per-snap mount namespace MS_SHARED <Bug> <Created by zyga> <https://github.com/snapcore/snapd/pull/7436> | 10:51 |
| zyga | I need to double check that it's good | 10:51 |
| zyga | but I feel this branch is close | 10:51 |
| zyga | mborzecki: yeah, as I thought | 10:52 |
| zyga | mborzecki: I think we still got sharing wrong, even with MS_SHARED | 10:52 |
| zyga | the per-user stuff is not a real slave | 10:52 |
| zyga | it's still bi-directional | 10:52 |
| zyga | I need to check that | 10:52 |
| zyga | it seems wrong | 10:52 |
| zyga | and indicates more tests needed | 10:53 |
| zyga | mborzecki: I commented out the MS_SLAVED | MS_REC change for the per-user mount namespace | 11:06 |
| zyga | I wonder if it doesn't work | 11:06 |
| zyga | I want to see the effect it has | 11:06 |
| Chipaca | pedronis: wrt your comment about gadgets having bases, does that mean a gadget without an explicit base needs core? | 11:06 |
| pedronis | Chipaca: yes | 11:06 |
| Chipaca | pedronis: was there a special value of base that meant "no base needed"? | 11:06 |
| pedronis | none | 11:07 |
| mup | PR snapd#6839 closed: devicestate: allow remodel to different kernels <Remodel :train:> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6839> | 11:07 |
| pedronis | but we implemented it but is not used | 11:07 |
| mborzecki | zyga: can you take another look at https://github.com/snapcore/snapd/pull/7451 ? | 11:07 |
| mup | PR #7451: sandbox/cgroup: introduce cgroup wrappers package <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7451> | 11:07 |
| zyga | yes, sorry | 11:07 |
| zyga | looking now | 11:07 |
| Chipaca | pedronis: the string "none"? | 11:07 |
| pedronis | the string "none" yes | 11:08 |
| Chipaca | pedronis: all these changes to canremove, should they go into the same refactor pr? | 11:08 |
| pedronis | Chipaca: as you prefer | 11:08 |
| mup | PR snapd#7464 opened: snapstate: add missing tests for checkGadgetOrKernel <Created by mvo5> <https://github.com/snapcore/snapd/pull/7464> | 11:09 |
| zyga | mborzecki: only small remark about capitalizing sentences and stuff like that | 11:12 |
| mborzecki | zyga: ack, thx | 11:13 |
| zyga | mborzecki: sent now | 11:14 |
| zyga | https://github.com/snapcore/snapd/pull/7451#pullrequestreview-287952596 | 11:14 |
| mup | PR #7451: sandbox/cgroup: introduce cgroup wrappers package <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7451> | 11:14 |
| zyga | I'd like to land https://github.com/snapcore/snapd/pull/7442 -- it will show up more behavior of the sharing bug in the next PR | 11:15 |
| mup | PR #7442: tests: extend mount-ns test to handle mimics <Created by zyga> <https://github.com/snapcore/snapd/pull/7442> | 11:15 |
| mup | PR snapd#7465 opened: snap-confine: fix return value checks for udev functions <Simple 😃> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7465> | 11:16 |
| zyga | oh, interesting mvo | 11:17 |
| zyga | I had a look at this code recently, let me double check if this bug is there too | 11:17 |
| zyga | thank you! | 11:17 |
| zyga | daaaaaaaaaaaaaaaaaaaaaaaaaaaaamn | 11:19 |
| zyga | aww crap :) | 11:19 |
| Chipaca | zyga: ...? | 11:20 |
| zyga | Chipaca: I removed one line from snap-confine | 11:20 |
| zyga | and ... no change to tests | 11:20 |
| zyga | it's a dead line | 11:20 |
| zyga | but one we hoped not to be dead | 11:20 |
| zyga | I'll check out what's going on next | 11:20 |
| mup | PR snapd#7466 opened: cmd/snap: special handling of snap set refresh.hold <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7466> | 11:28 |
| zyga | test rolling now | 11:28 |
| mborzecki | wonder what do you guys thing about 7466 | 11:29 |
| zyga | yeah, another bug | 11:30 |
| zyga | maaaaan | 11:30 |
| zyga | tests | 11:30 |
| Chipaca | mborzecki: +1; 'hold' feels like a duration anyway | 11:30 |
| zyga | thank you | 11:30 |
| Chipaca | zyga: https://www.youtube.com/watch?v=dpWmlRNfLck but with 'tests' | 11:30 |
| zyga | mvo: so, we have another bug, not sure how we never noticed before | 11:33 |
| mborzecki | zyga: where? | 11:34 |
| zyga | https://www.irccloud.com/pastebin/2UL7crS0/ | 11:34 |
| zyga | this has absolutely no impact | 11:34 |
| mborzecki | hah | 11:34 |
| zyga | I was digging at something that showed up in the MS_SHARED test | 11:35 |
| zyga | it was showing something that made no sense | 11:35 |
| zyga | so I kept digging | 11:35 |
| zyga | and now this | 11:35 |
| mborzecki | zyga: mayeb the tests are too limited to catch whetehr this has any impact or not | 11:35 |
| zyga | no | 11:35 |
| zyga | they 100% would catch this | 11:35 |
| Chipaca | zyga: could it be a 1404ism? | 11:35 |
| Chipaca | or something like that? | 11:35 |
| zyga | 1404, no | 11:35 |
| zyga | it's really not working | 11:35 |
| zyga | I'll write a test | 11:36 |
| Chipaca | eh, ok | 11:36 |
| zyga | sucky | 11:36 |
| Chipaca | i'm going to go make lunch and let you have your fun | 11:36 |
| Chipaca | mmmmmmmmmmmmm lunch | 11:36 |
| zyga | but yeah, little by little test show crap | 11:36 |
| * Chipaca needs food | 11:36 | |
| zyga | oh, I'm hungry now | 11:36 |
| zyga | don't tell me about food please | 11:36 |
| * Chipaca was about to, but stops | 11:36 | |
| cachio | sergiusens, hey | 11:37 |
| Chipaca | zyga: I'll tell you that i'm hungry because apparently running 30k makes me hungry for more than 12 hours | 11:37 |
| cachio | sergiusens, about https://travis-ci.org/snapcore/snapcraft/jobs/584308760#L175 error | 11:37 |
| Chipaca | zyga: or maybe it's just my excuse :-) | 11:37 |
| zyga | Chipaca: congratulations, that's nice! | 11:37 |
| * Chipaca goes | 11:37 | |
| sergiusens | hello! | 11:37 |
| cachio | sergiusens, this seems to be missing the key to connect to gce | 11:37 |
| cachio | sergiusens, is this a new travis job? | 11:38 |
| sergiusens | yes and no | 11:38 |
| cachio | sergiusens, what's the new part? | 11:39 |
| sergiusens | cachio: https://travis-ci.org/snapcore/snapcraft/builds/584308757/config I moved the unit tests into spread https://github.com/snapcore/snapcraft/pull/2712/commits/b871fd4777d690a3b51c858084e03d27184f33e8 | 11:39 |
| mup | PR snapcraft#2712: snap: migrate to core18 <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2712> | 11:39 |
| sergiusens | cachio: in the second link | 11:40 |
| sergiusens | cachio: hmm, the only difference is that I am using the snap for spread compared to just downloading it for the spread task that works | 11:41 |
| cachio | so, you didn't change the project | 11:41 |
| cachio | or the google key? | 11:41 |
| cachio | right | 11:41 |
| cachio | # SPREAD_GOOGLE_KEY | 11:42 |
| sergiusens | cachio: do you use the spread snap for your jobs? | 11:43 |
| cachio | sergiusens, I really don't know how spread snap could affect | 11:43 |
| cachio | we dont we it | 11:43 |
| cachio | we download the bin | 11:43 |
| sergiusens | hah, that might explain it | 11:44 |
| sergiusens | is the spread snap known to be working? | 11:44 |
| zyga | sergiusens: I don't know if it is maintained | 11:44 |
| zyga | would be good to check revision history | 11:45 |
| cachio | sergiusens, we don't use it, and we don't maintain this | 11:46 |
| cachio | sergiusens, so, not sure | 11:46 |
| cachio | sergiusens, niemeyer could kno | 11:46 |
| cachio | w | 11:46 |
| zyga | mborzecki: and I know why that is now too | 11:49 |
| pedronis | sergiusens: I do use the snap locally | 11:49 |
| pedronis | it's old though apparently | 11:50 |
| sergiusens | pedronis: and it reads your google key just fine when exported into the environment? | 11:51 |
| pedronis | yes | 11:51 |
| pedronis | I just used it yesterday | 11:51 |
| sergiusens | ok, I will continue checking to see if it is something else, but this is the only diference between a job that works and one that does not on travis | 11:53 |
| mup | PR snapd#7467 opened: seed/seedwriter: resolve channels using channel.Resolve* for snaps <Created by pedronis> <https://github.com/snapcore/snapd/pull/7467> | 11:56 |
| mup | PR snapd#7468 opened: seed/seedwriter: support local snaps <Created by pedronis> <https://github.com/snapcore/snapd/pull/7468> | 11:57 |
| mup | PR snapd#7469 opened: seed/seedwriter,snap/naming: support classic models <Created by pedronis> <https://github.com/snapcore/snapd/pull/7469> | 11:59 |
| Chipaca | pedronis: that's a long chain of PRs :-( | 12:07 |
| mup | PR snapd#7423 closed: overlord/snapstate: config revision code cleanup and extra tests <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/7423> | 12:07 |
| pedronis | Chipaca: it's not even the end of it, but I'll try not to push more up today | 12:09 |
| Chipaca | heh | 12:09 |
| pedronis | Chipaca: it's not visible atm but except the first one that mvo skimmed they are all reasonably sized | 12:10 |
| pedronis | that's why they are many | 12:10 |
| Chipaca | pedronis: cool | 12:10 |
| Chipaca | pedronis: i'm reviewing 7416 fwiw | 12:10 |
| pedronis | Chipaca: indeed that's the start and the biggest one | 12:11 |
| pedronis | thank you | 12:11 |
| pedronis | Chipaca: I pushed some more in case people want to see where things head to review the start | 12:11 |
| pedronis | btw | 12:11 |
| pedronis | in case | 12:11 |
| pedronis | as well | 12:11 |
| Chipaca | very confusing that asserts' core snaps are type "core" | 12:12 |
| Chipaca | (well, that's not the confusing bit, but ykwim) | 12:12 |
| pedronis | Chipaca: we could probably do something in types.go | 12:14 |
| pedronis | anyway there is exactly two snaps with type "os" | 12:14 |
| Chipaca | :) | 12:14 |
| pedronis | (not counting strange test ones) | 12:14 |
| Chipaca | pedronis: wouldn't "AdditionalSnap" have been clearer for OptionsSnap? | 12:18 |
| pedronis | Chipaca: they are not additional | 12:18 |
| pedronis | they can also tweak model ones | 12:18 |
| pedronis | you can see --snap core=edge | 12:19 |
| pedronis | core is not additional there | 12:19 |
| pedronis | s/see/say/ | 12:19 |
| Chipaca | pedronis: unrelatedly, you know how to write graphviz graphs, yes? | 12:24 |
| pedronis | yes | 12:24 |
| Chipaca | pedronis: graph-easy produces ASCII output from 'em | 12:24 |
| Chipaca | pedronis: as in sudo apt install libgraph-easy-perl | 12:24 |
| pedronis | didn't know that | 12:24 |
| Chipaca | then 'graph-easy foo.dot' → ascii graph | 12:25 |
| mup | PR snapd#7408 closed: tests: fix interfaces-timeserver-control on 19.10 <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7408> | 12:25 |
| mup | PR snapd#7126 closed: tests: part3 making tests work on ubuntu-core-18 <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7126> | 12:27 |
| Chipaca | pedronis: looking at that graph, is the line on the left that goes from Start to SnapsToDownload supposed to be there? The 'no local snaps' label seems to be about the one immediately to its right that goes from LocalSnaps to SnapsToDownload | 12:33 |
| === ricab|bbl is now known as ricab | ||
| Chipaca | and the leftmost one just mirrors the rightmost one | 12:33 |
| === pedronis_ is now known as pedronis | ||
| Chipaca | ok my lunch smells ready now | 12:40 |
| * Chipaca grabs a bowl | 12:40 | |
| pedronis | Chipaca: right side jumps: if you didn't call SetOptionsSnaps, you can jump straight from Start to SnapsToDownload, left side jumps: if you called it but none of them where local you can also do that, if you don't have local snaps you can also skip InfoDerived even if you called LocalSnaps | 12:41 |
| pedronis | the right side one is a bit redundant, if you have no option snaps then of course you don't local either | 12:42 |
| mup | PR snapd#7470 opened: DRAFT: core20 snap install <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/7470> | 12:47 |
| mup | PR snapd#7463 closed: tests/main/interfaces-{calendar,contacts}-service: disable on 19.10 <Simple 😃> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/7463> | 12:59 |
| mup | PR snapd#7471 opened: tests: fix newline and wrong test name pointed out in previous PRs <Simple 😃> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7471> | 13:04 |
| === genii_ is now known as genii | ||
| zyga | ijohnson: hey, I know it's not fair because I don't review much lately but https://github.com/snapcore/snapd/pull/7442 is +60,-0 and I've addressed your comments now | 13:29 |
| mup | PR #7442: tests: extend mount-ns test to handle mimics <Created by zyga> <https://github.com/snapcore/snapd/pull/7442> | 13:29 |
| ijohnson | zyga: sure happy to take a look | 13:29 |
| Psil0Cybin | Hey guys can someone guide me through an install of Snappy to install an app for Kali Linux , problem "home is not in /home/user its just in /user" | 14:01 |
| Psil0Cybin | I know snappy doesd not function so, but I found a few work around guides but want perhaps more guidance before i just blindly punch in commands | 14:02 |
| ijohnson | Chipaca: updated #7411 | 14:10 |
| Chipaca | ijohnson: ack | 14:10 |
| mup | PR #7411: cmd/model: output tweaks, add'l tests <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7411> | 14:10 |
| Chipaca | ijohnson: i'll get to a savepoint in #7416 and look at that | 14:10 |
| mup | PR #7416: seed/seedwriter: start of Writer and internal policy16/tree16 <Created by pedronis> <https://github.com/snapcore/snapd/pull/7416> | 14:10 |
| Chipaca | PRs have savepoints, right? | 14:10 |
| ijohnson | sure, no rush | 14:10 |
| ijohnson | save early save often | 14:10 |
| pedronis | Chipaca: btw did you see my answer here about that flow diagram ? | 14:11 |
| Chipaca | pedronis: from before the standup, yes | 14:12 |
| Chipaca | pedronis: if after, no | 14:12 |
| pedronis | yes, from before | 14:12 |
| * Chipaca is a Byzantine General | 14:12 | |
| cachio | mvo, canonistack server where spread-cron runs has been migrated, I'm migrating this as well to the new one | 14:20 |
| cachio | so snapd-vendor-sync won't run until the new instance is ready | 14:21 |
| ijohnson | zyga: #7442 approved | 14:21 |
| mup | PR #7442: tests: extend mount-ns test to handle mimics <Created by zyga> <https://github.com/snapcore/snapd/pull/7442> | 14:21 |
| ijohnson | looks like it's green so you could merge :-) | 14:21 |
| Psil0Cybin | if anyone has time later to plz help me get snap.d working on kali i would really appreciate it just becasue of the home folder situation... | 14:28 |
| Chipaca | Psil0Cybin: in kali, do you run everything as root, or are you your own user? | 14:29 |
| mvo | cachio: nice | 14:30 |
| __chip__ | Psil0Cybin: in kali, do you run everything as root, or are you your own user? | 14:31 |
| __chip__ | (silly train network doing silly things) | 14:31 |
| Psil0Cybin | __chip__, my own user, created my own user for kali hardening | 14:31 |
| Psil0Cybin | but still for some reason my home directory is /newuser | 14:31 |
| __chip__ | Psil0Cybin: where did you put this user's home? | 14:31 |
| Psil0Cybin | instead of /home/newuser. | 14:31 |
| Psil0Cybin | sadly in /newuser | 14:31 |
| Psil0Cybin | why it was default or i did not notice any other settings :( | 14:31 |
| __chip__ | Psil0Cybin: sudo mkdir -p /home/newuser; sudo mount --bind /newuser /home/newuser | 14:32 |
| __chip__ | Psil0Cybin: that should work around the home issue, let you look at the next blocker | 14:32 |
| __chip__ | kali had no shortage of those | 14:32 |
| Psil0Cybin | :O would that work? Can i do this while logged in? | 14:32 |
| __chip__ | like weird network | 14:32 |
| __chip__ | and weird X | 14:32 |
| __chip__ | Psil0Cybin: yes. And yes. | 14:32 |
| Psil0Cybin | yea im down to figure out these blockers and even write a guide since i see non online to avoid similar things happening to users. | 14:33 |
| __chip__ | k | 14:33 |
| __chip__ | good luck | 14:33 |
| Psil0Cybin | perfect first command worked, lets install snapd now :D | 14:33 |
| __chip__ | Psil0Cybin: we're about to disappear for a week as we meet in person to plan the next chunk of work, so use the forum more? or be extra patient :) | 14:33 |
| __chip__ | my train's almost there so ttfn | 14:34 |
| Psil0Cybin | sounds good incase you go missing! Hats off to you gentle man | 14:34 |
| Psil0Cybin | men** | 14:34 |
| sergiusens | pedronis: cachio fyi, by switching to the downloaded spread I could get a machine assigned | 14:57 |
| cachio | sergiusens, nice | 15:06 |
| Chipaca | ijohnson: very quick one: you've double-imported check into asserts_test | 15:09 |
| pedronis | Chipaca: thanks for the review | 15:14 |
| Chipaca | pedronis: ijohnson: was the consensus that 'snap model --verbose' did not print 'brand' at all? | 15:15 |
| Chipaca | just brand-id | 15:16 |
| pedronis | Chipaca: yes | 15:17 |
| jdstrand_ | roadmr: hi! would you mind pulling 20190913-1517UTC? not urgent | 15:18 |
| === jdstrand_ is now known as jdstrand | ||
| roadmr | jdstrand_: sure thing. Probably won't deploy today (Friday the 13th, before a sprint? not a chance ;) but I'll aim to roll out next week | 15:18 |
| jdstrand | roadmr: hehe, sounds perfect | 15:20 |
| pedronis | good, OOPS: 55 passed, 1 FAILED | 15:20 |
| pedronis | that's image_test.go with seedwriter behind the scenes (plugged a bit hackishly atm but working) | 15:20 |
| Chipaca | network hiccup :-| | 15:20 |
| Chipaca | pedronis: ijohnson: was the consensus that 'snap model --verbose' did not print 'brand' at all? just brand-id? | 15:20 |
| pedronis | Chipaca: yes | 15:20 |
| Chipaca | ok | 15:20 |
| ijohnson | Chipaca: whoops sorry | 15:20 |
| Chipaca | ijohnson: a few more coming up | 15:20 |
| ijohnson | ah ok I'll wait for the full review then | 15:21 |
| Chipaca | ijohnson: done | 15:21 |
| mup | PR snapd#7365 closed: tests: spread test for snap refresh/switch channel and risk switching <Created by stolowski> <Closed by stolowski> <https://github.com/snapcore/snapd/pull/7365> | 15:21 |
| Chipaca | ooh, missed one thing | 15:22 |
| Chipaca | ijohnson: you've got an if (something) { continue } else { stuff } | 15:22 |
| Chipaca | ijohnson: that leaves {stuff} more indented than it needs to be | 15:22 |
| Chipaca | ijohnson: silly stylistic, but hey | 15:22 |
| ijohnson | that's what you get paid for right? | 15:22 |
| roadmr | sillistic? | 15:22 |
| Chipaca | ijohnson: no, i get paid for my good looks | 15:23 |
| Chipaca | ijohnson: style is just a bonus | 15:23 |
| ijohnson | of course how could I forget | 15:23 |
| mup | PR snapd#7461 closed: run-checks, tests/main/go: allow gofmt checks to be skipped on 19.10 <Simple 😃> <Created by bboozzoo> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/7461> | 15:23 |
| ijohnson | Chipaca: where's the if thing you're mentioning? | 15:24 |
| Chipaca | ijohnson: cmd_model.go:252 | 15:24 |
| pstolowski | zyga: merged #7461 and didn't notice your small remark there, sorry | 15:24 |
| mup | PR #7461: run-checks, tests/main/go: allow gofmt checks to be skipped on 19.10 <Simple 😃> <Created by bboozzoo> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/7461> | 15:24 |
| ijohnson | got it thanks | 15:25 |
| Chipaca | ijohnson: would be nice to see what the output looks like in this case | 15:25 |
| Chipaca | is there a test for it? | 15:25 |
| Chipaca | i'll look in a bit | 15:25 |
| ijohnson | in which case | 15:25 |
| ijohnson | ? | 15:25 |
| Chipaca | (coming up to a stop) | 15:25 |
| Chipaca | required-snaps | 15:26 |
| ijohnson | yes there's a test with required-snaps in the unit tets | 15:26 |
| ijohnson | tests | 15:26 |
| Chipaca | neat, i'll look | 15:26 |
| Chipaca | thank you | 15:26 |
| Chipaca | ttfn | 15:26 |
| ijohnson | hey jdstrand, pedronis, mvo if I install a snap with system-usernames: snap_daemon in it, then I manually delete the snap_daemon user I can't seem to remove the snap due to seccomp being unable to compile the profile (because we disconnect interfaces first) | 16:27 |
| mup | PR snapd#7442 closed: tests: extend mount-ns test to handle mimics <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/7442> | 16:27 |
| mup | PR snapcraft#2713 closed: tests: completely mock bzr tests <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2713> | 17:04 |
| zyga | re | 17:09 |
| zyga | I was asleep, sorry | 17:09 |
| mup | PR snapcraft#2714 opened: tests: completely mock mercurial tests <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2714> | 17:10 |
| jdstrand | ijohnson: can you file a bug? | 17:16 |
| ijohnson | jdstrand: sure | 17:25 |
| roadmr | why is the snapcraft test suite from source trying to run /snap/bin/snapcraft? | 18:28 |
| roadmr | cachio: ^^ any idea about this? I mean, I can install the snapcraft snap but the point here is to test the source code I cloned | 18:29 |
| cachio | roadmr, I don't know, perhaps sergiusens could help on this | 18:30 |
| roadmr | thanks! | 18:30 |
| roadmr | oh there's a snapcraft channel :) I'll ask there | 18:31 |
| mup | PR snapcraft#2715 opened: Unit 7z <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2715> | 18:52 |
| mup | PR snapd#7471 closed: tests: fix newline and wrong test name pointed out in previous PRs <Simple 😃> <Created by stolowski> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/7471> | 18:54 |
| mup | PR snapd#7429 closed: wrappers/services: add ServicesEnableState + unit tests <Created by anonymouse64> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/7429> | 21:40 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!