| mup | PR snapd#8513 opened: snap-bootstrap: lock access to sealed keys <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/8513> | 00:06 |
|---|---|---|
| zyga | Good morning | 05:19 |
| * zyga is tired after working past midnight | 05:19 | |
| mborzecki | morning | 05:51 |
| zyga | Hey | 05:52 |
| zyga | Made some more progress last night | 05:57 |
| zyga | But tired now | 05:57 |
| zyga | I’ll start at around 9 | 05:57 |
| mborzecki | mvo: zyga: hey | 06:14 |
| mvo | mborzecki: good morning | 06:20 |
| zyga | Hey mvo | 06:24 |
| mvo | hey zyga | 06:25 |
| zyga | How is bit cacher? | 06:25 |
| mvo | zyga: seems to be working, running a real spread run this morning to see if it behaves | 06:25 |
| zyga | Cool | 06:26 |
| zyga | I guess we could then return to phased testing, with one build of snapd deb/snap | 06:26 |
| mvo | zyga: yes, I think those can be distributed via artifcats | 06:27 |
| zyga | That would be superb. | 06:28 |
| mup | PR snapd#8510 closed: boot/bootstate20: small changes to bootloaderKernelState20 <UC20> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8510> | 06:32 |
| mup | PR snapd#8507 closed: packaging: fix build on Centos8 to support BUILDTAGS <Created by mvo5> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/8507> | 06:40 |
| mborzecki | mvo: i'm going to file a bug in rhel about the build tags macro | 06:50 |
| mborzecki | just upadting my rhel8 vm | 06:50 |
| mvo | mborzecki: nice, thank you! fwiw, the git report referenced in the code has what we need | 06:53 |
| pstolowski | morning | 07:01 |
| * zyga starts the day | 07:07 | |
| mup | PR snapcraft#3047 closed: repos: fix returned strings for install_stage_packages() <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3047> | 07:13 |
| mborzecki | pstolowski: hey | 07:20 |
| zyga | my dbus test passes without hanging for the first time | 08:08 |
| zyga | let me clean it up a little and propose :) | 08:08 |
| pstolowski | mvo: can you grant me edit rights to the doc? | 08:28 |
| mvo | pstolowski: please reload, you should have it | 08:28 |
| mup | PR snapd#8514 opened: Extend openvswitch interface to support ovs-appctl <Created by dosaboy> <https://github.com/snapcore/snapd/pull/8514> | 08:31 |
| pstolowski | mvo: ty | 08:32 |
| mvo | mborzecki: hey, just fyi - ondra shared some fun initrd building snippet https://git.launchpad.net/~ondrak/ondras-snaps/+git/linux-kernel/tree/snapcraft.yaml?h=ubuntu-focal-qcomlt-uc20#n50 | 08:35 |
| mborzecki | mvo: mhm, right, that's what we do too | 08:36 |
| mvo | mborzecki: aha, ok | 08:36 |
| mborzecki | otherwise it's lots of pain | 08:37 |
| pstolowski | pedronis: hey, not sure if you saw my yesterday's finding (pastebin from the evening)? mvo figured out the reason (it's due to how writable is handled) and proposed how to handle it; we discussed it in a HO this morning. i'm sharing this notes with you and we can discuss when you have a moment today | 08:39 |
| pedronis | pstolowski: mvo: did you discuss what to do about /etc/cloud, it has the same problem? | 08:40 |
| mvo | pedronis: the soution outlined in the gdoc (number c) should apply to all of them equally | 08:47 |
| pstolowski | pedronis: it's defined as synced per https://github.com/snapcore/core18/blob/master/static/etc/system-image/writable-paths#L54 | 08:47 |
| pedronis | mvo: ok, I'll wait for the doc to be finished | 08:49 |
| mvo | pedronis: it's finished, (d) is really a bit of a distraction | 08:51 |
| mvo | pedronis: it's super interessting but I'm not sure if it's worth it given that we also need to support uc16/uc18 | 08:51 |
| pedronis | mvo: c for 16/18 needs initramfs changes there, right? | 08:52 |
| mvo | pedronis: correct | 08:52 |
| mborzecki | huh, whole session froze | 08:52 |
| pedronis | mvo: maybe if we need to do changes, we should do the same thing as uc20 and call a hook inside the bases? | 08:52 |
| pedronis | but upgrades | 08:53 |
| pedronis | there's probably a way to do that though | 08:53 |
| pedronis | reexec like | 08:54 |
| mvo | pedronis: that's a very interessting idea, let me look at the old initramfs code | 08:54 |
| mvo | pedronis: I like this, should be totally doable (also in a backward compat way). it's a bit risky as it involves initrd changes but would be nice and clean and more uniform | 08:55 |
| pedronis | mvo: notice though that we can't probably remove any synced from core18 writable-paths | 08:56 |
| pedronis | because we don't know what people did with ubuntu-image hooks | 08:57 |
| pedronis | we can/should remove them from core20 though | 08:57 |
| mvo | pedronis: +1 | 08:58 |
| pedronis | mvo: there is probably some other wrinkle as well, related to prepare-image | 08:59 |
| pedronis | I'll write some notes in the doc, when I have a moment. Finishing something else atm | 09:00 |
| mvo | pedronis: thanks! adding some notes myself too | 09:00 |
| pedronis | mvo: ok, yes, one point I wanted to make was indeed iv | 09:04 |
| pedronis | mvo: pstolowski: made a couple of comments | 09:07 |
| mvo | pedronis: thanks for adding the comment about the fallback! | 09:07 |
| pstolowski | thanks | 09:08 |
| mup | PR snapd#8515 opened: testutil: add NewDBusTestConn <Created by zyga> <https://github.com/snapcore/snapd/pull/8515> | 09:16 |
| zyga | I pushed the test helper alone here | 09:21 |
| zyga | and the tests are in https://github.com/snapcore/snapd/pull/7825 | 09:21 |
| mup | PR #7825: many: use transient scope for tracking apps and hooks <Security-High> <Created by zyga> <https://github.com/snapcore/snapd/pull/7825> | 09:21 |
| pstolowski | mborzecki: hey, what's the plan with #7414? | 09:24 |
| mup | PR #7414: tests: keep track of installed packages and restore the state after the test <Test Robustness> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/7414> | 09:24 |
| zyga | I will write a few more tests to exercise all the interactions | 09:24 |
| mborzecki | pstolowski: probably needs a merge from master and making sure it still works | 09:24 |
| zyga | but now I need a break, my back is killing me | 09:24 |
| mborzecki | need to run an errand, back in 1.5-2h, hopefully sooner | 09:26 |
| mup | PR snapcraft#3048 opened: V2 dump plugin <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3048> | 09:38 |
| * zyga is pissed off at go fmt | 10:24 | |
| zyga | can we move to non-ancient go fmt | 10:24 |
| zyga | it doesn't matter for compiling | 10:24 |
| zyga | is anyone in the team still using "go fmt" from 1.6 because they cannot use newer go? | 10:24 |
| zyga | I have to fight my editor to hand-craft restore some alignment | 10:25 |
| pedronis | I made an alias go-fmt that calls the old go fmt from a snap installation fwiw, I'm also happy to discuss how to switch to something newer, but probably after the beta at this point | 10:30 |
| mup | PR snapd#8516 opened: github: format with modern go <Created by zyga> <https://github.com/snapcore/snapd/pull/8516> | 10:37 |
| mup | PR snapd#8516 closed: github: format with modern go <Created by zyga> <Closed by pedronis> <https://github.com/snapcore/snapd/pull/8516> | 10:42 |
| zyga | pedronis: huh | 10:43 |
| pedronis | zyga: what part of until beta you didn't get? | 10:43 |
| zyga | pedronis: I just read you writing that | 10:43 |
| zyga | pedronis: and you could have just marked it as blocked or something | 10:43 |
| zyga | pedronis: not nice | 10:43 |
| pedronis | zyga: if we don't land it, it will need tending, so I don't see how marking it blocked helps | 10:47 |
| pedronis | zyga: I don't know what people workflows are like, I don't think I want to even discuss that before beta | 10:47 |
| pstolowski | it seems that jenkins deb is not authenticated in special-home-can-run-classic-snaps, despite adding the key in the test? i saw this test failing on some PRs | 11:12 |
| mborzecki | re | 11:13 |
| mup | PR core-build#59 opened: initramfs: add new handle_writable_defaults() <Created by mvo5> <https://github.com/snapcore/core-build/pull/59> | 11:19 |
| mvo | pstolowski: ^- that is probably all we need, the interessting part if of course the next step where we test this :) | 11:20 |
| mvo | pstolowski: we can look into this after the standup | 11:20 |
| mvo | zyga: fwiw 8508 works, thanks again for your help with that | 11:21 |
| pstolowski | mvo: oh thank you, i didn't expect you to take this! thanks! | 11:21 |
| mvo | pstolowski: no worries, it's just a tiny step | 11:22 |
| * mvo gets some lunch | 11:22 | |
| mvo | mborzecki: (8505 - your caching idea ftw!) | 11:22 |
| * mvo really gets lunch | 11:22 | |
| mup | PR snapd#8517 opened: tests: temporarily allow unauthenticated jenkins deb <Created by stolowski> <https://github.com/snapcore/snapd/pull/8517> | 11:47 |
| pstolowski | ^ a bit scary, perhaps we should disable the test for now.. | 11:48 |
| pedronis | so much weirdness | 12:00 |
| pedronis | pstolowski: we probably need mvo input on that one | 12:04 |
| mborzecki | cmatsuoka: hi, i seem to be getting a differnt output from lsblk --fs --json than the tests expect | 12:25 |
| cmatsuoka | mborzecki: that sounds bad, what test? | 12:26 |
| mborzecki | cmatsuoka: here's the test data vs what i get from the tool https://paste.ubuntu.com/p/qSCBCC44zF/ | 12:27 |
| mborzecki | cmatsuoka: the parsing only handles blockdevice.[i] elements, does not descend into blockdevice.[i].children[j] | 12:28 |
| cmatsuoka | mborzecki: I think the test output is simulating the output when lsblk is called directly on the partition and not the disk | 12:28 |
| mborzecki | aah w8 | 12:28 |
| mborzecki | right | 12:28 |
| cmatsuoka | does it make sense? | 12:29 |
| mborzecki | yeah, it does now | 12:29 |
| mborzecki | meh, more mocking in tests :/ | 12:29 |
| cmatsuoka | ah ok good | 12:29 |
| ijohnson | morning folks | 12:53 |
| cachio | pstolowski, hei, there is a beg assigned to you, lp:1872115 | 12:54 |
| cachio | pstolowski, did you see that? is there a priority for that? | 12:55 |
| cmatsuoka | hey ijohnson | 12:57 |
| ijohnson | hey cmatsuoka o/ | 12:57 |
| cmatsuoka | cachio: I had problems with the tpm branches yesterday, will try the sb test again after the SU | 12:59 |
| cachio | cmatsuoka, sure | 12:59 |
| pstolowski | cachio: no, i didn't see that, thanks. i guess zyga assigned it to me because of my other rsyslog work, need to check | 12:59 |
| cachio | cmatsuoka, ping me if you need any help | 12:59 |
| ijohnson | ugggh pulseaudio | 13:01 |
| mup | Bug #1873452 opened: 'snap install' | "INFO Waiting for restart..." <Snappy:New> <https://launchpad.net/bugs/1873452> | 13:01 |
| roadmr | hi diddledan are you around? | 13:24 |
| diddledan | ello | 13:24 |
| roadmr | diddledan: I got a request from the mycroft devs to have the snap transferred to them. Are you OK with that? | 13:25 |
| diddledan | yup | 13:25 |
| roadmr | sweet, ok thanks, all I needed was confirmation :) I'll take it from here | 13:25 |
| roadmr | happy Friday | 13:25 |
| diddledan | coolbeans :-) | 13:25 |
| mup | PR snapd#8517 closed: tests: temporarily allow unauthenticated jenkins deb <Created by stolowski> <Closed by stolowski> <https://github.com/snapcore/snapd/pull/8517> | 13:28 |
| mup | PR snapd#8518 opened: tests: disable special-home-can-run-classic-snaps due to jenkins repo issue <Created by stolowski> <https://github.com/snapcore/snapd/pull/8518> | 13:31 |
| mup | PR snapcraft#3046 closed: plugins: introduce v2.GoPlugin <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3046> | 13:41 |
| pedronis | ijohnson: to be clear I'm not sure if there are different scenarios were we need that divergent snap, we don't with the current tests | 14:01 |
| ijohnson | I don't think we will need it, I'm not sure what scenarios we could end up with that, where the kernel the bootloader boots is not the one we want as the "normal" kernel on the bootloader after MarkSuccessful runs | 14:02 |
| pedronis | ijohnson: that sounds right | 14:03 |
| pedronis | we still want to check before and after though | 14:03 |
| pedronis | just for the same value, to be clear | 14:03 |
| ijohnson | what do you mean check before and after? | 14:03 |
| ijohnson | before and after the MarkSuccessful ? | 14:03 |
| pedronis | yes | 14:04 |
| pedronis | I suppose | 14:04 |
| ijohnson | hmm ok I can make that work | 14:04 |
| pedronis | ijohnson: I thougth, it's an easy change, we are probably talk cross-purpose | 14:05 |
| mup | PR snapcraft#3049 opened: build providers: rename default sources <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3049> | 14:11 |
| roadmr | jdstrand: hi, are you around? | 14:26 |
| ackk | hi, I'm having an issue. my lxd snap suddenly broke, looking at /snap/lxd there's no snap dir there, just a broken "current" link | 14:46 |
| jdstrand | roadmr: I'm here today but stepping away for a little bit. please ask and I'll answer when I get back | 14:49 |
| ackk | and no lxd snap file in /var/lib/snapd/snaps | 14:49 |
| roadmr | jdstrand: remember https://dashboard.snapcraft.io/snaps/inkscape/feedback/ timing out? it's because the page lists thousands of items. Are you OK with paginating that by e.g. 100 items at a time? | 14:50 |
| mup | PR snapcraft#3050 opened: meta: split up package repository sanity checks <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3050> | 14:50 |
| jdstrand | roadmr: I can answer that now | 14:51 |
| jdstrand | roadmr: yes. please yes | 14:51 |
| jdstrand | :) | 14:51 |
| roadmr | \o/ hehe ok jdstrand we'll do it that way then | 14:51 |
| jdstrand | zyga: you have mail :) | 14:52 |
| * jdstrand steps away | 14:52 | |
| ackk | is there a way I can make snapd happy and mount the lxd snap again? | 14:54 |
| popey | error: cannot communicate with server: timeout exceeded while waiting for response | 15:00 |
| popey | My machine ^. Can't install or update anything. | 15:01 |
| mup | PR snapcraft#3051 opened: project: introduce 'keys' for project assets <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3051> | 15:12 |
| pstolowski | popey: journalctl -u snapd | 15:15 |
| pstolowski | ? | 15:15 |
| mborzecki | ijohnson: pushed the changes to https://github.com/snapcore/snapd/pull/8487 and marked it as ready for review | 15:16 |
| mup | PR #8487: [RFC] gadget, cmd/snap-bootstrap: hack some things into place to get rpi4 semi-working <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8487> | 15:16 |
| popey | https://paste.ubuntu.com/p/mDrRqBJNrZ/ pstolowski | 15:16 |
| ijohnson | thanks mborzecki | 15:16 |
| ijohnson | I'll give it a try on my pi today | 15:16 |
| ijohnson | popey: can you still run snaps? | 15:17 |
| pstolowski | popey: ok, not much interesting there, anything else in the logs? | 15:18 |
| popey | yes, i can run snaps | 15:18 |
| popey | i just can't refresh, or it does, but i get that error | 15:19 |
| pstolowski | apparently deamon is not running | 15:20 |
| stgraber | mvo, zyga: https://discuss.linuxcontainers.org/t/unable-to-start-snap-lxd-on-debian-10/7456/17?u=stgraber | 15:20 |
| stgraber | this feels like pretty crap experience :) | 15:20 |
| mvo | stgraber: in a meeting right now, but I though we did test lxd on debian :( | 15:20 |
| zyga | mvo: reexec | 15:21 |
| zyga | mvo: backports | 15:21 |
| zyga | stgraber: can you add assumes to lxd? | 15:21 |
| zyga | stgraber: and at least put a FAQ that you need to snap install core | 15:21 |
| zyga | it will break in a nicer way IMO | 15:21 |
| zyga | unless we update backports we have few optiosn | 15:22 |
| zyga | *options | 15:22 |
| zyga | stgraber: I totally agree about the UX | 15:22 |
| pstolowski | popey: 'snap changes' fails the same I presume? | 15:22 |
| mvo | yeah, assumes should fix things | 15:22 |
| stgraber | zyga: where is the doc on assumes? | 15:22 |
| popey | pstolowski snap changes returns immediately | 15:22 |
| zyga | https://snapcraft.io/docs/snapcraft-top-level-metadata#heading--assumes | 15:23 |
| zyga | stgraber: ^ add the assumess snapdX.YY version that meets your expectations | 15:23 |
| stgraber | ah yeah, just found it too | 15:23 |
| stgraber | zyga: so what's the minimum version that's known to behave? | 15:23 |
| zyga | stgraber: perhaps we could patch snapd in debian 10 to say something like | 15:24 |
| mup | PR snapcraft#3049 closed: build providers: rename default sources <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3049> | 15:24 |
| zyga | stgraber: anyone that you pick as right for you | 15:24 |
| zyga | stgraber: it depends on a snap | 15:24 |
| stgraber | zyga: well, in this case the issue is apparently the snapd didn't connect any of the interfaces, so I have no idea when you fixed that :) | 15:24 |
| pstolowski | popey: anything else in the logs? a crash/panic? | 15:24 |
| zyga | stgraber: perhaps we could patch debian 10 snapd to say "you need snapd X.YY, you may try to get it from backports <URL> or by installing core" | 15:24 |
| stgraber | zyga: we purposefuly don't use anything new/fancy in the snap | 15:24 |
| popey | pstolowski not that I have seen :( | 15:24 |
| zyga | stgraber: put a version that you tested as ok and that's enough | 15:25 |
| zyga | stgraber: it's fine to depend on new snapd | 15:25 |
| pstolowski | popey: and what does 'systemctl status snapd.service snapd.socket' say? | 15:25 |
| zyga | pstolowski, popey: this is assertion check being >> than snap client timeout | 15:25 |
| zyga | IIRC we ran into it before and debugged it to that | 15:25 |
| zyga | but I could be wrong | 15:25 |
| stgraber | zyga: looks like 2.39 and higher based on the versions our CI sees on the distros we test | 15:29 |
| popey | pstolowski https://paste.ubuntu.com/p/zPcXzjtpNJ/ | 15:31 |
| pstolowski | zyga: can you elaborate on the assertion check, what do you mean? | 15:34 |
| zyga | pstolowski: that's for stgraber | 15:34 |
| zyga | pstolowski: it's unrelated to the issue you are looking at for popey | 15:34 |
| zyga | I can explain if you want to, just wanted to make this clear | 15:34 |
| pstolowski | ack | 15:35 |
| pstolowski | popey: and 'snap --version' ? | 15:35 |
| popey | https://paste.ubuntu.com/p/zNtPDNF8ns/ | 15:35 |
| zyga | pstolowski: to reproduce install 300 snaps | 15:36 |
| zyga | and refresh | 15:36 |
| popey | I have 113 installed | 15:36 |
| zyga | pstolowski: if I'm correct your work on assertion refreshes will help immensly | 15:36 |
| zyga | pstolowski: but would be good to undestand this in depth, perhaps there are other inefficiences | 15:36 |
| zyga | (each from the store, not local) | 15:37 |
| zyga | hmm, we could actually use | 15:37 |
| pstolowski | zyga: i'm not working on assertion refreshes afaik ;) | 15:37 |
| zyga | test-snapd-manysnap-$(seq...) | 15:37 |
| zyga | in the store | 15:37 |
| zyga | pstolowski: oh, sorry, I though that was in the new store API work | 15:37 |
| zyga | anyway | 15:37 |
| pstolowski | no i was new search v2 api | 15:37 |
| pstolowski | *it* | 15:38 |
| zyga | I see | 15:38 |
| mup | PR snapcraft#3052 opened: build providers: wait for systemd and better nameserver setup on LXD <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3052> | 15:39 |
| ijohnson | zyga: I thought pedronis was working on bulk assertion refreshes | 15:39 |
| pstolowski | yes | 15:40 |
| zyga | ijohnson: yeah, I must have confused the store work with "anything related to store" | 15:40 |
| ijohnson | is this issue related to the internet? must be the store | 15:40 |
| ijohnson | and that folks is how you do investigative work | 15:40 |
| zyga | ijohnson: IIRC just that we cannot grab the lock | 15:41 |
| zyga | ijohnson: needed to produce most "snap foo" responses | 15:41 |
| * ijohnson was joking | 15:41 | |
| zyga | ah :) | 15:41 |
| zyga | sorry | 15:41 |
| ijohnson | I have no idea honestly what this issue is | 15:41 |
| * zyga should really EOD | 15:41 | |
| zyga | I've reviewed half of James' user session services PR | 15:42 |
| zyga | yeah, let's rest | 15:42 |
| mup | PR snapd#8518 closed: tests: disable special-home-can-run-classic-snaps due to jenkins repo issue <Simple 😃> <⚠ Critical> <Created by stolowski> <Merged by anonymouse64> <https://github.com/snapcore/snapd/pull/8518> | 15:42 |
| ijohnson | alright master should be at least not red due to jenkins again | 15:42 |
| ijohnson | pstolowski: you wanna quick open up the PR re-enabling the jenkins test? | 15:43 |
| ijohnson | if you want I can do it too | 15:43 |
| pstolowski | popey: does ps aux|grep apparmor show anything? | 15:43 |
| popey | no | 15:44 |
| pstolowski | ijohnson: please do, thanks, trying to figure out popey's issue | 15:44 |
| ijohnson | ack | 15:44 |
| pstolowski | popey: how about 'sudo snap debug state /var/lib/snapd/state.json' ? | 15:48 |
| popey | (that looks like snap changes)? | 15:48 |
| popey | https://paste.ubuntu.com/p/wHXFrdQhJ2/ | 15:48 |
| pstolowski | popey: yes, but it doesn't talk to the daemon | 15:49 |
| popey | ahh | 15:50 |
| pstolowski | popey: does 'snap list' work? | 15:54 |
| popey | yes | 15:55 |
| pstolowski | popey: ah, so snapd does respond, i got confused by that first communication error. perhaps store is acting up? how about 'snap find ...'? | 15:58 |
| popey | snap find works | 15:58 |
| popey | bah, now I'm not getting the error when I refresh! | 15:58 |
| zyga | pstolowski: my theory is wrong, based on the debug output | 15:58 |
| pstolowski | popey: i bet that was a store problem, maybe of limited scope (i was testing with same snapd as you have without issues) | 16:00 |
| pstolowski | popey: there was no sign of snapd errors/malfunction in the info you got me so far | 16:01 |
| ijohnson | popey: how performant is this machine with 116 snaps? and also what kind of disk is it on? | 16:01 |
| ijohnson | popey: given that it keeps finding bugs, maybe turning on debug logging for snapd would be a good idea, even maybe http debug logging too | 16:02 |
| popey | it's an ThinkPad T450, i7, 32GB RAM | 16:02 |
| popey | brb, meeting | 16:02 |
| ijohnson | pedronis: if you have time yet today, 8511 is updated with your comments | 16:02 |
| ijohnson | ah actually I forgot one thing, let me push that quickly | 16:03 |
| pstolowski | eow, cu all! | 16:10 |
| mup | PR snapd#8409 closed: snap-bootstrap: seal and unseal encryption key using tpm <UC20> <Created by cmatsuoka> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8409> | 16:15 |
| cmatsuoka | \o/ | 16:16 |
| pedronis | ijohnson: yes, I'll try to wrap up a couple of things and some reviews after dinner | 16:17 |
| ijohnson | thanks pedronis | 16:17 |
| ijohnson | congrats cmatsuoka ! | 16:17 |
| cmatsuoka | ijohnson: many fixes still to come | 16:17 |
| pedronis | cmatsuoka: can you merge master into your locking keys PRs now | 16:18 |
| cmatsuoka | pedronis: doing that right now | 16:19 |
| pedronis | thx | 16:19 |
| cmatsuoka | cachio: ok, ready to run the sb/tpm tests | 16:34 |
| cachio | nice | 16:34 |
| cmatsuoka | cachio: I'm using your spread, which back-end should I use? | 16:34 |
| mup | PR snapcraft#3051 closed: project: introduce 'keys' for project assets <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3051> | 16:36 |
| cmatsuoka | cachio: and should I run on master or tests-extend-uc20-nested-suite? | 16:36 |
| cachio | cmatsuoka, you should use the google-tpm I sent yesterday | 16:37 |
| cachio | which test do you want to run? | 16:38 |
| cmatsuoka | uc20-create-partitions-encrypted | 16:38 |
| cachio | you should add this new tpm backend to the spread.yaml | 16:38 |
| cmatsuoka | ah yes, I remember now, let's do this | 16:38 |
| cachio | and run spread -debug google-tpm:ubuntu-20.04-64:tests/main/uc20-create-partitions-encrypted | 16:39 |
| cachio | cmatsuoka, when you run this please tell me so I can monitor the vm | 16:39 |
| mup | PR snapcraft#3050 closed: meta: split up package repository sanity checks <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3050> | 16:48 |
| mup | PR snapcraft#3053 opened: snap_packaging: quote final LD_LIBRARY_PATH <Created by Saviq> <https://github.com/snapcore/snapcraft/pull/3053> | 16:57 |
| mup | PR snapcraft#3054 opened: Add note about docker snap <Created by sixcorners> <https://github.com/snapcore/snapcraft/pull/3054> | 17:03 |
| mup | PR snapcraft#3048 closed: V2 dump plugin <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3048> | 17:06 |
| mup | PR snapcraft#3052 closed: build providers: wait for systemd and better nameserver setup on LXD <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3052> | 17:27 |
| mup | PR snapcraft#3055 opened: repo: add identifiers for gpg keys and sources <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3055> | 17:39 |
| bdx | hello | 18:31 |
| bdx | I'm trying to release a snap that uses classic confinement, and need an approval to release it to the snap store. I have started a discourse post here https://forum.snapcraft.io/t/classic-confinement-request-for-nhc-snap | 18:32 |
| bdx | is there someone from the reviewers team available to possibly give some feedback ^? | 18:33 |
| bdx | thanks! | 18:33 |
| mup | PR snapcraft#3056 opened: remote-build: package up local sources with source-type 'git' <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3056> | 18:39 |
| mup | PR snapd#8325 closed: snap-bootstrap: copy auth data from real ubuntu-data in recovery mode <UC20> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8325> | 18:41 |
| mup | PR snapd#8514 closed: interfaces/openvswitch: support use of ovs-appctl <Created by dosaboy> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8514> | 18:42 |
| mup | PR snapcraft#3057 opened: repo: format $SNAPCRAFT_APT_RELEASE instead of ${release} for suites <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3057> | 18:42 |
| mup | PR snapcraft#3055 closed: repo: add identifiers for gpg keys and sources <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3055> | 19:00 |
| pedronis | ijohnson: I reviewed #8511 | 19:24 |
| mup | PR #8511: tests/boot: refactor to make it easier for new bootloaderKernelState20 impl <Test Robustness> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8511> | 19:24 |
| ijohnson | thanks pedronis | 19:31 |
| ijohnson | Have a good weekend! | 19:31 |
| * ijohnson continues fighting the pi | 19:32 | |
| === Wouter01005 is now known as Wouter0100 | ||
| mborzecki | ijohnson: hey, saw your comment, pushed a patch | 19:48 |
| ijohnson | hey mborzecki thanks let me look now | 19:48 |
| ijohnson | I locally had a simple patch to work around it, but didn't push anything cause I wasn't sure what the right way to do it is | 19:49 |
| mborzecki | ijohnson: missed that ensure layout compatiblity check, but it shoudl be ok now | 19:49 |
| ijohnson | ah yeah I think your check is better, I forgot that the gadgetLayout has Schema on it to check that it's mbr | 19:49 |
| mborzecki | ijohnson: i mean, there's nothign we can check there if there's no way store the name on disk :P so well | 19:50 |
| ijohnson | true | 19:50 |
| mborzecki | as long as we don't erase ubuntu-seed things should be ok xD | 19:51 |
| mborzecki | what reminds me i need to discuss gadget updates in uc20 on monday | 19:51 |
| mborzecki | ijohnson: anyways, thanks for catching that! hope it works now | 19:53 |
| mup | PR snapd#8511 closed: tests/boot: refactor to make it easier for new bootloaderKernelState20 impl <Test Robustness> <UC20> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8511> | 19:54 |
| mup | PR snapcraft#3058 opened: package-repositories: make 'name' optional <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3058> | 19:54 |
| mup | PR snapd#8423 closed: tests: uc20 nested suite part II <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8423> | 20:30 |
| mup | PR pc-amd64-gadget#42 closed: Hide menu by default <Created by xnox> <Merged by xnox> <https://github.com/snapcore/pc-amd64-gadget/pull/42> | 21:29 |
| mup | Issue pc-amd64-gadget#41 closed: Hide the menu by default <Created by xnox> <Closed by xnox> <https://github.com/snapcore/pc-amd64-gadget/issue/41> | 21:38 |
| mup | PR pc-amd64-gadget#37 closed: Drop duplicate grub.cfg-* <Created by xnox> <Merged by xnox> <https://github.com/snapcore/pc-amd64-gadget/pull/37> | 21:38 |
| mup | PR core20#43 opened: extra-packages: add dbus-user-session for user-session dbus <Created by xnox> <https://github.com/snapcore/core20/pull/43> | 21:45 |
| mup | PR snapd#8519 opened: tests/special-home-can-run-classic-snaps: re-enable <Test Robustness> <⚠ Critical> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8519> | 22:09 |
| mup | PR core20#44 opened: travis: use bionic <Created by xnox> <https://github.com/snapcore/core20/pull/44> | 22:27 |
| mup | PR core20#44 closed: travis: use bionic <Created by xnox> <Merged by xnox> <https://github.com/snapcore/core20/pull/44> | 22:31 |
| mup | PR core20#45 opened: Add arches <Created by xnox> <https://github.com/snapcore/core20/pull/45> | 22:33 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!