mborzecki | morning | 05:33 |
---|---|---|
mborzecki | mvo: hey | 06:50 |
mvo | hey mborzecki | 06:51 |
mborzecki | mvo: an interesting bug https://bugs.launchpad.net/snapd/+bug/1890046 | 06:52 |
mup | Bug #1890046: Handle SOURCE_DATE_EPOCH for SquashFS <Snapcraft:Triaged> <snapd:Triaged> <https://launchpad.net/bugs/1890046> | 06:52 |
mvo | mborzecki: uh, looking | 06:55 |
mvo | mborzecki: hm, doing this for the snapd snap should be easy, core is a bit harder. i wonder if we should just sru squashfs-tools | 06:56 |
pstolowski | morning | 07:01 |
mborzecki | pstolowski: hey | 07:02 |
mvo | good morning pstolowski | 07:03 |
mborzecki | mvo: there's probably some concerns about the review-tools/store side of things? | 07:03 |
mvo | mborzecki: yeah, probably something to talk to jamie about | 07:05 |
mvo | mborzecki: hm, the bug report talks about reproducible builds, this is exactly what we want too I guess | 07:09 |
mborzecki | mvo: right, common goals may make sru'ing squashfs-tools a bit easier ;) | 07:11 |
mvo | :) | 07:11 |
mborzecki | need to go and drop some paperwokr at my accountant | 07:12 |
mborzecki | re | 08:17 |
pedronis | mvo: pstolowski: hi, have we prepared backports for snap debug seeding stuff ? | 08:39 |
pstolowski | pedronis: afaiu mvo was going to look at the commits yesterday so i haven't do anything about that. i'm happy to take this over if not started yet | 08:44 |
pedronis | pstolowski: it seems like we need to add preseed code to all interfaces backends no? | 08:45 |
mvo | pstolowski: I have not started yet, if you have capacity for this, that would be great | 08:45 |
pstolowski | pedronis: nb, we may have an issue with udev though, see mattermost chat | 08:45 |
pstolowski | pedronis: yep, udev backend | 08:46 |
pedronis | pstolowski: yes, that's why my comment about interfaces backends | 08:46 |
pedronis | pstolowski: well, we need to look at all backends and see which one can trigger external programs/services | 08:46 |
pstolowski | pedronis: yes | 08:46 |
pedronis | and see which one don't work in a chroot | 08:46 |
pedronis | and see what to do | 08:47 |
pstolowski | mvo, pedronis i've doctor appointment in 15, shouldn't take long, will be back on this afterwards | 08:47 |
pedronis | ok | 08:47 |
mvo | pstolowski: not super urgent, no worries. see you | 08:48 |
* pstolowski bbiab | 08:52 | |
zyga | good morning | 08:59 |
* zyga came to check if 2.45.3.1 built fine for suse | 08:59 | |
mborzecki | zyga: hey, how are you feeling today? | 09:05 |
zyga | mborzecki: much better | 09:15 |
zyga | stronger than before | 09:15 |
mborzecki | zyga: yay, that's great | 09:15 |
mborzecki | zyga: koza sends regards ;) | 09:15 |
zyga | oh cool | 09:15 |
zyga | I wonder how he is doing | 09:15 |
pstolowski | back | 09:21 |
pstolowski | hey zyga! | 09:21 |
zyga | hey Pawel! | 09:22 |
zyga | last day off | 09:23 |
zyga | I can walk more each day | 09:23 |
zyga | I think tomorrow I could do two hour standing sessions interspaced with one hour in-bed sessions | 09:23 |
zyga | and intermixed with prescribed exercise | 09:23 |
mborzecki | mvo: can you take a look at https://github.com/snapcore/snapd/pull/9078 ? | 09:56 |
mup | PR #9078: [RFC] boot: fancy marshaller for modeenv values <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9078> | 09:56 |
mborzecki | mvo: and i'll poke ijohnson too when he's online, it'd be nice to land it soonish | 09:57 |
mvo | mborzecki: yes | 10:01 |
mborzecki | mvo: thanks! | 10:01 |
zyga | hey mvo | 10:21 |
zyga | I was just chatting with Pawel | 10:22 |
zyga | I will take a break now, I'll come back later | 10:22 |
pstolowski | pedronis: do you have a moment for #9088 ? | 10:34 |
mup | PR #9088: cmd/snap-preseed: use snapd from the deb if newer than from seeds <Preseeding 🍞> <Run nested> <Squash-merge> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9088> | 10:34 |
pedronis | pstolowski: do you have a question? or do you need a review? | 10:51 |
pstolowski | pedronis: review | 10:52 |
pedronis | ah, no, I confused it with a different PR | 10:52 |
pedronis | I put it in my queue (having lunch break atm) | 10:52 |
pstolowski | thanks, enjoy | 10:52 |
mup | PR snapd#9090 closed: cmd/snap/debug/seeding: use unicode for proper yaml <Preseeding 🍞> <Simple 😃> <⚠ Critical> <Created by anonymouse64> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9090> | 11:18 |
pedronis | mvo: ^ I forced merged that one, we have a problem with debian-sid atm, it will need a backport | 11:18 |
mvo | pedronis: ok | 11:43 |
pedronis | pstolowski: reviewed, made some maybe slightly annoying comments, and also a very different suggestion in: https://github.com/snapcore/snapd/pull/9088#discussion_r465666390 | 11:43 |
mup | PR #9088: cmd/snap-preseed: use snapd from the deb if newer than from seeds <Preseeding 🍞> <Run nested> <Squash-merge> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9088> | 11:43 |
=== benfrancis0 is now known as benfrancis | ||
pstolowski | pedronis: thanks, will take a look | 11:54 |
pedronis | pstolowski: I think the suggestion is simpler than either current or what I suggested, if a bit less clean, but probably clean enough | 11:55 |
mborzecki | moving back home | 12:03 |
ijohnson | morning folks | 12:15 |
pedronis | mvo: I did a pass on #8982, looks good, left some comments. It needs new reviews though because it has changed quite a bit | 12:16 |
mup | PR #8982: snapshots: export of snapshots <Needs Samuele review> <Created by slimjim777> <https://github.com/snapcore/snapd/pull/8982> | 12:16 |
mvo | pedronis: thank you so much | 12:17 |
mup | PR snapd#9092 opened: interfaces/udev: Do not reload udevadm rules when preseeding <Bug> <Preseeding 🍞> <Squash-merge> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9092> | 12:18 |
pstolowski | ^ i need to check other backends, will address them separately if needed | 12:18 |
pstolowski | ouch https://www.engadget.com/honda-recalls-608000-minivans-and-suvs-with-fault-software-issues-121511778.html | 12:27 |
pstolowski | shoud have auto-refresh enabled ;) | 12:27 |
pedronis | pstolowski: I reviewed that | 12:30 |
pstolowski | ty | 12:32 |
ijohnson | what's the situation with the udev backend and preseeding ? | 12:32 |
mvo | pedronis: 8946 got a review from Graham, he +1 but also mentioend to add a period at the end of each sentence. I guess I will just add the "." and push and once tests are good merge ? or do you think the extra spread run is not needed? | 12:33 |
pstolowski | ijohnson: I got past the mount issue on xenial, but then it failed as apparently udevadm was triggered by some interface in the seeded snaps (see my standup notes) | 12:35 |
* ijohnson looks | 12:35 | |
pedronis | mvo: not, needed but the tool needs changes | 12:36 |
pedronis | as well | 12:36 |
pedronis | mvo: I can also look into that today | 12:37 |
pedronis | mvo: doing cherry-picks for 2.45 is probably more urgent | 12:37 |
ijohnson | pstolowski: interesting, perhaps we should try seeding other snaps which have other interfaces | 12:38 |
ijohnson | err preseeding | 12:38 |
pedronis | ijohnson: I asked to look at all backends | 12:38 |
pstolowski | ijohnson: i'm going to check other backends | 12:38 |
ijohnson | sorry I just meant as a spread test | 12:38 |
ijohnson | since we may change things over time in the backends setup | 12:38 |
pedronis | yes | 12:38 |
pedronis | but a bit unclear how to do that, we mostly need to cover backends more than interfaces, but backends might vary. We have a snap with all possible interfaces | 12:39 |
pedronis | but not sure it would play well with preseeding | 12:39 |
ijohnson | also the issue is that snap with all possible interfaces doesn't have store decls, does it ? | 12:40 |
pedronis | exactly | 12:40 |
ijohnson | we would need store assertions for the backends to actually run when interfaces are connected | 12:40 |
pedronis | also connecting them doesn't always make sense | 12:40 |
ijohnson | mmm | 12:40 |
pedronis | I mean, with things like some of the support interfaces it gets messy | 12:41 |
pedronis | I mean I don't think is sensible to have such set of store declarations | 12:41 |
pedronis | I think we need just enough to cover our bases | 12:41 |
ijohnson | agreed | 12:41 |
pedronis | sensibly | 12:41 |
pedronis | maybe there's a why to write backends that would make this kind of issue easier to track | 12:42 |
pedronis | mvo: I'll take care of 8946 today | 12:43 |
mvo | pedronis: thank you | 12:46 |
pedronis | mvo: pushed, please double check my last changes when you have a moment | 12:52 |
pstolowski | kmod backend needs the same treatment | 12:53 |
pedronis | pstolowski: systemd as well I think | 12:55 |
pstolowski | pedronis: just looking at it.. yes | 12:55 |
pstolowski | i forgot about all these corners | 12:58 |
mborzecki | ijohnson: can you take a look at https://github.com/snapcore/snapd/pull/9078 later today? | 13:40 |
mup | PR #9078: [RFC] boot: fancy marshaller for modeenv values <UC20> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9078> | 13:40 |
ijohnson | sure | 13:41 |
mborzecki | ijohnson: thanks, it'd be nice to land it to unblock the horrors in modeenv | 13:41 |
* ijohnson read it as "unlock the horrors" and I imagine it's not actually too far off | 13:41 | |
mborzecki | hm damn build tags | 13:58 |
mup | PR snapd#9093 opened: interfaces/kmod: don't load kernel modules in kmod backend when preseeding <Bug> <Preseeding 🍞> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9093> | 14:23 |
pstolowski | should #9041 be tagged for 2.45 as well? | 14:27 |
mup | PR #9041: osutil/group.go: treat all non-nil errs from user.Lookup{Group,} as Unknown* <Bug> <Needs security review> <Preseeding 🍞> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9041> | 14:27 |
ijohnson | pstolowski: no not necessary for 2.45, as it's blocked on security | 14:28 |
pstolowski | ack | 14:29 |
* cachio lunch | 14:46 | |
pedronis | pstolowski: I just landed #9091 | 15:07 |
mup | PR #9091: cmd/snap: display the error in snap debug seeding if seeding is in error <Preseeding 🍞> <Squash-merge> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9091> | 15:07 |
pstolowski | ack, ty | 15:08 |
mup | PR snapd#9091 closed: cmd/snap: display the error in snap debug seeding if seeding is in error <Preseeding 🍞> <Squash-merge> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9091> | 15:09 |
pedronis | pstolowski: so I think with that all the debug seeding PR are on master | 15:09 |
pstolowski | pedronis: yes; i'll look at backporting now | 15:12 |
pedronis | thank you | 15:12 |
mup | PR snapd#8946 closed: client: move all error kinds into errors.go and add doc strings <Needs Samuele review> <Created by mvo5> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/8946> | 15:24 |
pedronis | degville: mvo: I did the merge: https://github.com/snapcore/snapd/commit/a777978c1102ea95f0a820e174ee066e6efcbcb4 | 15:25 |
mvo | pedronis: thank you! | 15:25 |
degville | pedronis: thanks! I'll update the API page accordingly. | 15:27 |
pedronis | thank you | 15:27 |
mup | PR snapd#9094 opened: corecfg: add "system.hostname" setting to the system settings <Created by mvo5> <https://github.com/snapcore/snapd/pull/9094> | 15:39 |
pstolowski | so everything re seeding debug backported cleanly, except for spread tests & their helper scripts :}, a bit of a mess there | 16:01 |
mup | PR snapcraft#3197 closed: experimental extension support <enhancement> <specification> <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3197> | 16:22 |
mup | PR snapd#9083 closed: tests: new parameters for nested execution <Run nested> <Simple 😃> <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/9083> | 16:44 |
mup | PR snapcraft#3239 opened: lxd: update connectivity check url <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3239> | 17:12 |
cachio | ijohnson, cmatsuoka hi, someone could please take a quick look to this one? #8942 | 17:26 |
mup | PR #8942: tests: support different images on nested execution <Run nested> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8942> | 17:26 |
cachio | it needs a second review | 17:27 |
cachio | I need that merged to start pushing a new set of improvements for nested | 17:27 |
cachio | thanks | 17:27 |
ijohnson | cachio: sure I can take a look | 17:38 |
cachio | ijohnson, tx+ | 17:46 |
ijohnson | cachio: did you try that branch with the cloud-init nested spread tests ? | 18:19 |
cachio | all the tests are executed with cloud init | 18:20 |
ijohnson | cachio: no I mean did you run the specific tests/nested/manual/cloud-init-* spread tests with this branch ? | 18:20 |
cachio | ijohnson, not sure if I am understanding whe question | 18:20 |
ijohnson | I imagine those tests would fail with your change | 18:20 |
ijohnson | because now when we call start_nested_vm_unit the image file modifications previously done in those tests/nested/manual/cloud-init-* tests would be undone and thus cause those tests to break | 18:21 |
cachio | well, they are going to be executed as part of the github actions tests | 18:22 |
cachio | let me check the results | 18:22 |
ijohnson | ok, but I still think that bit of changes would break those tests | 18:23 |
cachio | I'll make a review now to see | 18:23 |
ijohnson | if those tests still pass then perhaps the tests are written poorly | 18:23 |
ijohnson | thank you | 18:23 |
cachio | ij | 18:23 |
cachio | ijohnson, it should work | 18:24 |
cachio | because in this pr after the manual test we clean the images | 18:24 |
cachio | so it is not a problem | 18:24 |
cachio | your concern if fixed in the following one, because the manual tests haev a specific name in the images | 18:24 |
cachio | and the images are not cleaned anymore | 18:25 |
cachio | so they can be reused | 18:25 |
cachio | does it make sense? | 18:25 |
ijohnson | cachio: no what I mean is that in the tests/nested/manual/cloud-init-nocloud-not-vuln test, we manually call `start_nested_core_vm_unit` | 18:25 |
cachio | yes, all the manual tests do that | 18:26 |
ijohnson | so now when that is called on line 150 of that task.yaml, your code will overwrite the image file that was used earlier in the test when we called start_nested_core_vm on line 40 | 18:26 |
ijohnson | cachio: sorry I think the point I'm trying to make is that we have tests where start_nestec_core_vm_unit is called multiple times during one test | 18:26 |
cachio | ah, ok, got it, let me see | 18:26 |
cachio | ijohnson, so the problem is that we do | 18:29 |
cachio | cp "$IMAGE_DIR/$IMAGE_NAME" "$CURRENT_IMAGE" | 18:29 |
ijohnson | yes that will overwrite the CURRENT_IMAGE used earlier in the test | 18:29 |
cachio | when we start ther image and this overwrite the image right? | 18:30 |
ijohnson | yes | 18:30 |
cachio | ijohnson, ok, good point, I can fix that | 18:30 |
ijohnson | thanks | 18:30 |
cachio | thakns for the review!!! | 18:31 |
ijohnson | np | 18:32 |
cachio | ijohnson, I think it should be fised | 18:37 |
cachio | fixed | 18:37 |
cachio | let see the test results | 18:40 |
cachio | ijohnson, I am going to the kinesiologist now | 18:40 |
cachio | I'll be back in 1 hour | 18:40 |
cachio | thanks for the review | 18:40 |
ijohnson | cachio: ack I'll take a look at the test results when it's done | 18:40 |
cachio | I had to leave the copy in the function start_nested_core_vm_unit because now the same of the image changes | 18:45 |
* cachio afk -> kinesiologist | 18:49 | |
mup | PR snapcraft#3239 closed: lxd: update connectivity check url <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3239> | 20:37 |
mup | PR snapd#9085 closed: daemon,many: switch to use client.ErrorKind and drop the local errorKind <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9085> | 21:35 |
mup | PR snapd#9095 opened: boot/bootstate20: unify commit method impls, rm bootState20MarkSuccessful <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9095> | 21:45 |
mup | PR snapd#9096 opened: strutil: add ListsSame helper <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9096> | 21:45 |
mup | PR snapd#9097 opened: boot/modeenv: add m.deepEquals, m.Origin helpers to simplify bootstate20 refactor <Simple 😃> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9097> | 21:45 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!