| mborzecki | morning | 05:13 |
|---|---|---|
| zyga | Hi | 06:11 |
| zyga | Sooooo sleepy | 06:12 |
| zyga | Hey mvo | 06:21 |
| mvo | hey zyga | 06:22 |
| mvo | zyga: if you adress the feedback from jamie in 6846, could you please force push so that I can cherry pick more easily? | 06:24 |
| mup | PR snapd#6843 closed: tests: avoid removing snaps which are cached to speed up the prepare on boards <Simple 😃> <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6843> | 06:25 |
| mborzecki | zyga: mvo: hey | 06:25 |
| mvo | hey mborzecki | 06:28 |
| mup | PR snapd#6831 closed: tests: retry govendor sync <Simple 😃> <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6831> | 06:32 |
| zyga | low hanging fruit: bump the delta tag please | 06:44 |
| zyga | https://github.com/snapcore/snapd/pull/6855 is a bit big | 06:45 |
| mup | PR #6855: overlord: implement store switch remodeling <Remodel :train:> <Created by pedronis> <https://github.com/snapcore/snapd/pull/6855> | 06:45 |
| mup | PR snapd#6846 closed: cmd/snap-confine: unshare per-user mount ns once <Bug> <Simple 😃> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6846> | 06:45 |
| mvo | zyga: I cherry picked 6846 | 06:47 |
| zyga | Thank you | 06:47 |
| zyga | I'm still working on the main dish | 06:47 |
| === pstolowski|afk is now known as pstolowski | ||
| pstolowski | morning | 07:05 |
| zyga | hey pawel | 07:09 |
| pedronis | mvo: hi, the PR to do store switch remodeling is up, some TODO related to conflicts, but otherwise is all there. Re-reg is a bit more complicated but not complex, needs the dedicated context and a new task kind. | 07:25 |
| pedronis | PRs are a bit big though (espcially tests) might need splitting. | 07:26 |
| pstolowski | mvo: hey, thanks for moving our 1-1! | 07:26 |
| mvo | pstolowski: is that time ok? I'm relatively flexible thanks to mborzecki who also moved the meeting :) | 07:51 |
| pstolowski | mvo: yes, it's perfect! | 07:52 |
| mup | PR snapd#6854 closed: cmd/snap-update-ns: rename applyFstab to executeMountProfileUpdate <Simple 😃> <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6854> | 07:52 |
| mvo | we are below 60 again! | 07:53 |
| mup | PR snapd#6851 closed: cmd/snap: mangle descriptions that have indent > terminal width <Simple 😃> <⚠ Critical> <Created by chipaca> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6851> | 07:53 |
| Chipaca | moin moin | 07:56 |
| pstolowski | hey Chipaca | 07:58 |
| mup | PR snapd#6856 opened: cmd/snap-update-ns: add tests for executeMountProfileUpdate <Created by zyga> <https://github.com/snapcore/snapd/pull/6856> | 07:59 |
| mup | PR snapd#6850 closed: gadget: add volume level update checks <Gadget update> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6850> | 08:02 |
| mborzecki | another one lands :) | 08:02 |
| mvo | !!! | 08:03 |
| mup | PR snapd#6857 opened: gadget: record sector size in positioned volume <Gadget update> <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6857> | 08:08 |
| mborzecki | and one more up /o\ | 08:08 |
| mborzecki | off to school, back in a bit | 08:09 |
| zyga | mborzecki: thank you | 08:14 |
| zyga | man, I needed this coffee | 08:16 |
| Chipaca | pedronis: you around? | 08:39 |
| pedronis | Chipaca: yes | 08:44 |
| Chipaca | pedronis: wrt health-checks, is there anything you'd rather see up in a pr next? | 08:44 |
| pedronis | Chipaca: did you push one already? | 08:45 |
| Chipaca | pedronis: just the draft one | 08:45 |
| Chipaca | pedronis: working on the tests still | 08:45 |
| pedronis | Chipaca: is it too big? | 08:45 |
| Chipaca | pedronis: no i don't think so | 08:45 |
| pedronis | I'm not sure I understand the question | 08:46 |
| Chipaca | pedronis: the pr that is up, that i expect to finish today while you're travelling, covers only a small portion of the feature | 08:48 |
| Chipaca | pedronis: so, what should i work on once that is 'done' (code complete)? | 08:49 |
| pedronis | Chipaca: the PR runs the hook and let the hook set the health, right? | 08:49 |
| Chipaca | pedronis: yes, post-refresh | 08:49 |
| Chipaca | or post-install | 08:50 |
| Chipaca | i mean it doesn't do the pre-refresh nor the on-demand, nor the info, nor the list | 08:50 |
| pedronis | I think the next bit is snap info | 08:50 |
| Chipaca | okie doke | 08:50 |
| pedronis | now that I understand the question | 08:50 |
| Chipaca | :) | 08:50 |
| * Chipaca is always very clear | 08:50 | |
| Chipaca | pedronis: and after that? (being optimistic here) | 08:51 |
| pedronis | regular running of the hook? | 08:52 |
| pedronis | and then the pre refresh one we need to think a bit, we can run it but we wouldn't do much with, would we? | 08:52 |
| pedronis | so it well might be on-demand | 08:53 |
| pedronis | I expect on-demand to be a bit complicated, to do the UX as we discussed it | 08:53 |
| pedronis | because we don't have anything quite like that | 08:53 |
| pedronis | atm | 08:53 |
| zyga | oh | 08:53 |
| zyga | what are you discussing? it sounds curious | 08:54 |
| pedronis | health checks | 08:54 |
| pedronis | or you mean the last bit in particular? | 08:54 |
| pedronis | the last bit is about snap health and returning info to the user for each snap as soon as it is available | 08:54 |
| zyga | about on-demand things, I was wondering if there's some new functionality in hooks that is planned | 08:55 |
| pedronis | don't think is about hooks | 08:55 |
| pedronis | it's more about how we return results | 08:55 |
| pedronis | from API | 08:55 |
| pedronis | but it depends exactly how we implement it | 08:56 |
| pedronis | Chipaca: so, info, regular, on-demand | 08:59 |
| Chipaca | pedronis: regular as in on-a-timer? | 09:00 |
| pedronis | Chipaca: that part of the spec that says we run them (probably triggered from ensure) at given interval, | 09:01 |
| pedronis | more often if unhealthy | 09:01 |
| Chipaca | yep | 09:01 |
| Chipaca | ok | 09:02 |
| pedronis | there are some fun issues there to, but we can have a general sketch of that before digging | 09:03 |
| pedronis | *there too | 09:03 |
| mup | PR snapd#6858 opened: cmd/snap: unit tests for debug timings <Created by stolowski> <https://github.com/snapcore/snapd/pull/6858> | 09:06 |
| zyga | mborzecki: not sure if intersects with your work | 09:13 |
| zyga | running snapd a while prints this a loooot | 09:13 |
| zyga | maj 10 11:00:53 yantra snapd[57785]: kernel_os.go:198: cannot get boot settings: cannot determine bootloader | 09:13 |
| Chipaca | pstolowski: "(and optionally the)" because code is optional! | 09:32 |
| Chipaca | :-p | 09:32 |
| pstolowski | :) | 09:33 |
| * pedronis starts early travel to Lyon | 09:36 | |
| pedronis | have great day and weekend! | 09:36 |
| zyga | pedronis: see you soon :) | 09:36 |
| zyga | pedronis: I'm going through geneva | 09:36 |
| zyga | pedronis: perhaps we might even share a train ride | 09:37 |
| pedronis | maybe, depends on times, but will do geneva -> Lyon on Sunday | 09:37 |
| zyga | same like me | 09:37 |
| pedronis | at some point | 09:37 |
| mup | PR snapd#6751 closed: overlord/snapstate: perform hard refresh check <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6751> | 09:47 |
| Chipaca | what was the right name for 'sideloads'? | 10:10 |
| zyga | Chipaca: none :) | 10:10 |
| zyga | snap install from a file? | 10:10 |
| Chipaca | unsigned? unverified? unpotatoed? | 10:10 |
| zyga | unasserted? | 10:10 |
| Chipaca | unasserted, thanks | 10:10 |
| zyga | note, I just made that up | 10:10 |
| zyga | but it sounds okay :) | 10:10 |
| Chipaca | 'twas that afair | 10:11 |
| Chipaca | AFAIR | 10:11 |
| Chipaca | also | 10:11 |
| mup | PR snapd#6853 closed: interfaces: special-case "snapd" in sanitizeSlotReservedForOS* helper… <Simple 😃> <⚠ Critical> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6853> | 10:20 |
| mvo | second review for 6793 would be nice (maybe mborzecki because he already did a first pass :) ? | 10:22 |
| zyga | mvo: mborzecki will be back in half an hour | 10:25 |
| zyga | mvo: can I help | 10:25 |
| zyga | mvo: I ran into another bug, don't understand it yet, looks related to recent changes | 10:26 |
| zyga | mvo: snap-confine cannot open the base directory in /tmp | 10:26 |
| zyga | debugging now | 10:26 |
| pstolowski | mvo: https://github.com/snapcore/snapd/pull/6823 can land? | 10:32 |
| mup | PR #6823: packaging: build empty package on powerpc <Created by mvo5> <https://github.com/snapcore/snapd/pull/6823> | 10:32 |
| mvo | pstolowski: if it has two reviews I think it can land | 10:32 |
| mvo | zyga: oh, where do you see this bug? | 10:32 |
| zyga | mvo: when developing on opensuse | 10:33 |
| zyga | mvo: just added debugging, hold on | 10:33 |
| zyga | it's odd, I didn't see it before | 10:33 |
| pstolowski | mvo: 2nd review is from vorlonofportland | 10:34 |
| mvo | pstolowski: aha, indeed, then yeah, lets merge it | 10:34 |
| mup | PR snapd#6823 closed: packaging: build empty package on powerpc <Created by mvo5> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6823> | 10:35 |
| mvo | pstolowski: thank you! I also cherry-picked it into 2.39 | 10:37 |
| pstolowski | Chipaca: 6803 has a few conflicts | 10:39 |
| Chipaca | nevah | 10:40 |
| mvo | 6795 also needs a second review, should be an easy win | 10:41 |
| pstolowski | mvo: i've merged #6811 but we need to ping Sergio re your question about if it needs to go to 2.39 | 10:45 |
| mup | PR #6811: tests: make create-user test support managed devices <Simple 😃> <Created by sergiocazzolato> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6811> | 10:45 |
| mvo | pstolowski: indeed | 10:45 |
| mvo | pstolowski: thank you! | 10:45 |
| mup | PR snapd#6811 closed: tests: make create-user test support managed devices <Simple 😃> <Created by sergiocazzolato> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6811> | 10:45 |
| zyga | I need a break | 10:50 |
| zyga | mvo: I think master may be broken | 10:50 |
| zyga | mvo: the /tmp changes affecte stuff that is only picked up by tests running as user | 10:50 |
| zyga | meh | 10:50 |
| zyga | not fun | 10:50 |
| zyga | anyone can run edge and say if stuff is falling apart? | 10:50 |
| zyga | but first | 10:51 |
| zyga | really | 10:51 |
| zyga | break | 10:51 |
| zyga | meh | 10:59 |
| zyga | it's so odd | 11:00 |
| zyga | something must have landed | 11:00 |
| mvo | zyga: I just refreshed my bionic to edge and ran hello-world | 11:00 |
| mvo | zyga: seems to be fine | 11:00 |
| mvo | zyga: hold on, that was core not the snapd snap | 11:01 |
| zyga | perhaps it's something in my setup | 11:02 |
| * zyga explores | 11:02 | |
| zyga | so I get permission denied | 11:02 |
| zyga | but it doesn't seem like apparmor | 11:02 |
| * zyga rechecks | 11:02 | |
| mborzecki | re | 11:03 |
| mvo | zyga: on my system(s) things seem to be ok - at least with test-snapd-tools.echo | 11:03 |
| zyga | thanks for checking | 11:03 |
| zyga | mvo: what are the permissions on /tmp/snap.test-snapd-tools? | 11:03 |
| sparkiegeek | is there a potential for a race when 'snap install thing-with-home-interface', then running thing-with-home-interface and expecting :home to be connected? | 11:06 |
| popey_ | Wimpress: mvo: Chipaca you guys around for catch up at half past? | 11:06 |
| sparkiegeek | Interface Plug Slot Notes | 11:06 |
| sparkiegeek | home documentation-builder:home - - | 11:06 |
| sparkiegeek | trying to work out how that happened, in a fresh LXD container, immediately after 'snap install documentation-builder' | 11:06 |
| zyga | sparkiegeek: odd, is core or snapd installed? | 11:07 |
| mvo | zyga: drwx------ 3 root root 4,0K Mai 10 13:00 /tmp/snap.test-snapd-tools | 11:08 |
| sparkiegeek | zyga: good question, this is repeatedly happening in aforementioned container - in a test run, so can add debugging but loop is longer than I'd like | 11:08 |
| zyga | mvo: thanks | 11:08 |
| mvo | popey_, Wimpress, Chipaca I'm around but I would not mind skipping given that we meet in lyon next week anyway | 11:08 |
| zyga | mvo: for me it is root.users! | 11:08 |
| zyga | mvo: whaaat? | 11:08 |
| * zyga inspects | 11:09 | |
| zyga | mvo: can you remove that | 11:09 |
| zyga | and run it again | 11:09 |
| zyga | and see if that fails? | 11:09 |
| zyga | mvo: I think I know what's wrong | 11:10 |
| zyga | mvo: look at https://github.com/snapcore/snapd/pull/6714/files | 11:10 |
| mup | PR #6714: cmd/snap-confine: reject crafted /tmp/snap.$SNAP_NAME <Bug> <Created by zyga> <https://github.com/snapcore/snapd/pull/6714> | 11:10 |
| zyga | mvo: you don't see it on debian | 11:10 |
| zyga | see that code that changes groups? | 11:10 |
| mvo | zyga: yes | 11:11 |
| zyga | I think we need that now | 11:11 |
| zyga | what's the permission of /usr/lib/snapd/snap-confine? | 11:11 |
| popey_ | mvo: ok, that's cool | 11:11 |
| mvo | zyga: -rwsr-sr-x 1 root root 99K Mär 15 20:54 /usr/lib/snapd/snap-confine | 11:11 |
| zyga | mvo: also in snapd.snap | 11:11 |
| mup | PR snapd#6857 closed: gadget: record sector size in positioned volume <Gadget update> <Simple 😃> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6857> | 11:12 |
| * zyga is confused | 11:12 | |
| zyga | mvo: thank you | 11:12 |
| mvo | popey_: cool, declined it now | 11:13 |
| sparkiegeek | zyga: appears to be neither core nor snapd, but our good friend core18 | 11:13 |
| zyga | sparkiegeek: that's a known bug | 11:13 |
| * zyga looks for the bug URL | 11:13 | |
| popey_ | bug 1819318 | 11:13 |
| popey_ | ? | 11:13 |
| zyga | https://bugs.launchpad.net/snapd/+bug/1819318 | 11:14 |
| zyga | yep | 11:14 |
| mup | Bug #1819318: no interfaces when installing only core18 <snapd:In Progress by zyga> <https://launchpad.net/bugs/1819318> | 11:14 |
| sparkiegeek | zyga: ok, and workaround is to ... manually install core too? | 11:14 |
| zyga | yes | 11:14 |
| mvo | sparkiegeek: yes, this will be fixed in 2.39 (which is in candidate already and should go to stable early next week) | 11:15 |
| mvo | sparkiegeek: sorry for the trouble | 11:15 |
| sparkiegeek | mvo: np. Due to a separate SNAFU that took me out all of yesterday, glad to finally be able to get debugging on this and look under the curtain, happy it's a known issue | 11:16 |
| mup | PR snapd#6795 closed: cmd,sandbox: tweak seccomp version info handling <Created by mvo5> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6795> | 11:24 |
| sparkiegeek | is there a 'grep-friendly' way of determining if I'm hitting ^ that bug. i.e. snap connected documentation-builder | grep -v ':home' || snap install core ? | 11:27 |
| zyga | sparkiegeek: just snap list is sufficient | 11:27 |
| zyga | sparkiegeek: then snap connections will show you there are no slots | 11:27 |
| sparkiegeek | zyga: right, just trying to build a snippet in a Makefile to workaround the bug if I need to, but don't install core if the interface is connected | 11:28 |
| zyga | mborzecki: hey, do you want to repeat our experiment? | 11:54 |
| zyga | mborzecki: I'm stuck, perhaps you will just see what's obviously broken | 11:54 |
| zyga | https://meet.google.com/rha-xhdp-vic?authuser=0 | 11:55 |
| zyga | mvo: or you perhaps? | 11:56 |
| zyga | anyone? | 11:56 |
| mborzecki | zyga: in 5? i'm finishing reading through pstolowski's pr | 12:00 |
| zyga | sure | 12:00 |
| mup | PR snapcraft#2500 closed: cli: add remote build <Created by cmatsuoka> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2500> | 12:08 |
| cmatsuoka | \o/ | 12:08 |
| zyga | Chipaca, pstolowski: saving data of multipass, on removal, is loooong | 12:10 |
| zyga | Chipaca: curious, how do we remove snapshots? | 12:11 |
| Chipaca | zyga: snap forget | 12:11 |
| pstolowski | zyga: uhmmm.. it can be disabled | 12:11 |
| zyga | can I forget all snapshots of a given snap? | 12:11 |
| Chipaca | zyga: yes | 12:11 |
| Saviq | cachio: hey, any chance for a fedora-30 image please? :) | 12:11 |
| Chipaca | zyga: not easily tho :-| | 12:12 |
| Chipaca | zyga: maybe we should add that, although it'd be tricky | 12:12 |
| zyga | Chipaca: I noticed | 12:12 |
| zyga | given that the things are called NNN_$SNAP_NAME_* | 12:12 |
| zyga | is that enough of a hint? | 12:13 |
| zyga | Chipaca: can I just rm them manually? | 12:13 |
| Chipaca | zyga: snap saved thesnap | <some magic> | xargs snap forget {} thesnap \;, or something | 12:13 |
| zyga | or is there state | 12:13 |
| zyga | :/ | 12:13 |
| Chipaca | zyga: there is no state | 12:13 |
| zyga | ah | 12:13 |
| zyga | off they go :) | 12:13 |
| Chipaca | that was part of the design of snapshots | 12:13 |
| zyga | thanks! | 12:13 |
| zyga | though for usability | 12:13 |
| zyga | killing snapshots of a snap | 12:13 |
| zyga | very likely useful | 12:13 |
| Chipaca | zyga: there is state about auto-snapshots, but it shouldn't break anything if it tries to remove them and they're gone | 12:14 |
| Chipaca | zyga: file bugs if it breaks and eats your thesis | 12:14 |
| zyga | who knew | 12:15 |
| zyga | sudo has --background and --command-timeout options | 12:15 |
| Chipaca | should we add a --purge to snap remove to mean "don't auto-snapshot"? | 12:15 |
| zyga | Chipaca: yeah | 12:15 |
| zyga | feels like something I'd do in tests | 12:15 |
| zyga | our tests don't have much data but removing multipass hit my fans running | 12:15 |
| Chipaca | pstolowski: wtyt? | 12:15 |
| zyga | 2GBs | 12:15 |
| Chipaca | wdyt* | 12:16 |
| pstolowski | Chipaca: +1 to 'snap remove --purge', i like that. gives more control than all-or-nothing flag we have now | 12:18 |
| Chipaca | we should either put up a pr, or write something in the forum, before we forget | 12:19 |
| Chipaca | me, i'm going to have lunch | 12:19 |
| * Chipaca prioritises like a baws | 12:19 | |
| === ricab is now known as ricab|lunch | ||
| cmatsuoka | mborzecki: PR #6849 failed some integration tests | 12:25 |
| mup | PR #6849: many: introduce a gadget helper for locating device matching given structure <Gadget update> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6849> | 12:25 |
| Saviq | mborzecki: it's != its ;) | 12:33 |
| Saviq | https://github.com/snapcore/snapd/pull/6849/files#diff-f57f6320470091f8bb514620cc2fbbddR35 | 12:33 |
| mup | PR #6849: many: introduce a gadget helper for locating device matching given structure <Gadget update> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6849> | 12:33 |
| zyga | mvo: mystery solved | 12:38 |
| mborzecki | Saviq: duh, missed that, thanks for spotting | 12:41 |
| mborzecki | pstolowski: May 10 12:01:02 may101137-895818 nsenter[32024]: retry.go:120: DEBUG: Retrying because of temporary net error: &net.OpError{Op:"dial", Net:"tcp", Source:net.Addr(nil), Addr:net.Addr(nil), Err:(*net.DNSError)(0xc00006c5c0)} | 12:44 |
| mborzecki | we're logging the error as #+v ? | 12:45 |
| mborzecki | %+v | 12:45 |
| pstolowski | mborzecki: %#v, yes | 12:47 |
| pstolowski | this is Debug though | 12:48 |
| mborzecki | pstolowski: yup, so probably fine | 12:51 |
| pstolowski | mborzecki: yeah, an it helps understand error conditions better if we need to peel them :} | 12:56 |
| cachio | Saviq, sure, I'll try today | 12:57 |
| mvo | zyga: what is the mystery? | 12:58 |
| pstolowski | cachio: hey, can you take a look if that PR that we landed today needs to go to 2.39? https://github.com/snapcore/snapd/pull/6811 | 13:00 |
| mup | PR #6811: tests: make create-user test support managed devices <Simple 😃> <Created by sergiocazzolato> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6811> | 13:00 |
| zyga | joining | 13:03 |
| mup | PR snapd#6793 closed: cmd/snap: support for --ensure argument for snap debug timings <Created by stolowski> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6793> | 13:04 |
| pstolowski | Chipaca: so, i'll create forum topic re --purge and do impl sometime next week | 13:39 |
| Chipaca | pstolowski: <3 | 13:39 |
| pstolowski | k | 13:44 |
| pstolowski | degville: btw, we need to update documentation with automatic snapshots; is it ok if i send you a short description of it, and you flesh it out/reword and put in a proper context of existing doc? | 13:45 |
| === ricab|lunch is now known as ricab | ||
| degville | pstolowski: absolutely, yes, of course. | 13:49 |
| mvo | Chipaca: I couldn't help it and de-conflicted 6803 | 13:54 |
| mvo | Chipaca: hope you don't mind | 13:54 |
| Chipaca | mvo: HULK SMASH | 13:54 |
| Chipaca | mvo: sorry, i meant "thank you" | 13:55 |
| Chipaca | mvo: typo | 13:55 |
| Chipaca | :-þ | 13:55 |
| mvo | haha | 13:57 |
| * mvo hugs Chipaca | 13:57 | |
| mup | PR core#38 closed: Add another pi-config option <Created by sergey-borovkov> <https://github.com/snapcore/core/pull/38> | 14:17 |
| mup | PR core#83 closed: move most of the ubuntu-core config deb into the snap snap build <Created by mvo5> <https://github.com/snapcore/core/pull/83> | 14:17 |
| mup | PR core#104 closed: snapcraft.yaml: use remote fc-cache-builder <Created by mvo5> <https://github.com/snapcore/core/pull/104> | 14:17 |
| mup | PR core#38 opened: Add another pi-config option <Created by sergey-borovkov> <https://github.com/snapcore/core/pull/38> | 14:18 |
| mup | PR core#83 opened: move most of the ubuntu-core config deb into the snap snap build <Created by mvo5> <https://github.com/snapcore/core/pull/83> | 14:18 |
| mup | PR core#104 opened: snapcraft.yaml: use remote fc-cache-builder <Created by mvo5> <https://github.com/snapcore/core/pull/104> | 14:18 |
| cachio | mborzecki, zyga if you could take another look to #6541 should be great, thanks | 14:43 |
| mup | PR #6541: tests: change how xdg-desktop-portal is prepared and restored for desktop-portal-* tests <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6541> | 14:43 |
| mborzecki | cachio: ok | 14:43 |
| cachio | mborzecki, thanks | 14:43 |
| pstolowski | Chipaca: https://forum.snapcraft.io/t/automatic-snapshots-snap-remove-purge-proposal/11294 | 15:01 |
| * Chipaca reads | 15:01 | |
| Chipaca | pstolowski: thank you | 15:02 |
| pstolowski | cmatsuoka: almost forgot - safe travels! | 16:15 |
| cmatsuoka | thanks! :) | 16:15 |
| === pstolowski is now known as pstolowski|afk | ||
| cachio | pstolowski|afk, hey | 16:31 |
| pstolowski|afk | cachio: hey | 16:32 |
| cachio | I have few issues with core | 16:32 |
| cachio | for example sudo usermod -a -G dialout user1 | 16:32 |
| cachio | does not work | 16:32 |
| cachio | pstolowski|afk, https://paste.ubuntu.com/p/DdZsrRCxnB/ | 16:33 |
| cachio | is it required on core? | 16:33 |
| pstolowski|afk | cachio: dialout is the group owner of serial ports by default; it's needed to actually access serial ports in nested vm (because we log as regular user, not root). you can skip it but the test will need to be much simpler (no tests for actual writing to the port) | 16:39 |
| pstolowski|afk | cachio: but as discussed that's fine, we should start with a much simplified test | 16:39 |
| cachio | pstolowski|afk, ok | 16:41 |
| pstolowski|afk | cachio: don't worry too much about actual test, i can work on tweaking it further. just plugging in the qemu virtual serial adapter + checking if slot was created succesfully will be enough | 16:42 |
| cachio | pstolowski|afk, ok, | 16:43 |
| cachio | and at which point the test should fail? | 16:43 |
| cachio | currently on core16 when I list connections the | 16:43 |
| cachio | qemuusbserial doesn't exist | 16:43 |
| cachio | this is not matching at all --> execute_remote "snap connections system" | MATCH "serial-port .* - .* :$SLOT_NAME" | 16:44 |
| cachio | but /dev/ttyUSB0 exists | 16:45 |
| pstolowski|afk | cachio: it's on official core16? | 16:47 |
| cachio | core 2.38.1 | 16:47 |
| cachio | the stable one | 16:47 |
| cachio | it created an image on nested vm | 16:48 |
| pstolowski|afk | cachio: right. yes, so you won't see the slot there because hotplug changes for serial-port interface didn't land | 16:48 |
| cachio | ah | 16:48 |
| cachio | ok | 16:48 |
| cachio | makes sense | 16:48 |
| pstolowski|afk | cachio: so the test will fail as soon as we check snap intrfaces, expecting the slot to be there | 16:48 |
| pstolowski|afk | cachio: if you see ttyUSB0 then it's good, qemu works as expected with core \o/ | 16:49 |
| cachio | pstolowski|afk, ah, nice | 16:49 |
| cachio | I'll recheck on core18 and I'll create a PR so next week we can test it once the change is landed | 16:49 |
| cachio | pstolowski|afk, thanks | 16:50 |
| pstolowski|afk | cachio: np, ty! i'm signing off for this week, see you! | 16:50 |
| cachio | pstolowski|afk, sure, enjoy your weekend | 16:50 |
| mup | PR snapcraft#2541 closed: Desktop meta: strip ${SNAP} from .desktop icon <Created by diddledan> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2541> | 16:57 |
| dcarmich | When I try to run classically-confined snaps on a Salt-managed Linux system as a user, I get the error: "cannot create user data directory: /Users/username/snap/(name of snap)/version: not a directory". However, when I install them on a stock Ubuntu 18.04 VM, they run just fine as a user. Is there a specific security setting I should look for that is causing this issue? | 17:46 |
| dcarmich | I'm using snap version 2.38+18.04 on Ubuntu 18.04 LTS. | 17:46 |
| cmatsuoka | niemeyer: we managed to remove adapter:full, using adapter:none now | 18:13 |
| mup | PR snapcraft#2562 opened: extensions: add KF5 Neon extension <Created by cmatsuoka> <https://github.com/snapcore/snapcraft/pull/2562> | 18:45 |
| cachio | Saviq, hey, image fedora-30-64 ready | 19:15 |
| cachio | I had to fix some issues with google packages | 19:16 |
| cachio | but right now it is working | 19:16 |
| cachio | just replace tha name you have set for fedora-29 and that should work | 19:16 |
| mup | PR snapd#6859 opened: tests: new hotplug test executed on ubuntu core <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6859> | 21:07 |
| * cachio EOW | 21:15 | |
| mup | PR snapd#6860 opened: tests: running tests on fedora 30 <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6860> | 21:17 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!