mupPR snapcraft#3271 opened: build providers: use the releases endpoint for LXD <enhancement> <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3271>00:22
mupPR snapcraft#3272 opened: storeapi: improve to channel map docstrings <bug> <documentation> <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3272>03:38
clmsyHello everyone! any chance we have a date in mind for this milestone :) https://launchpad.net/snapd/+milestone/2.46 ?06:31
mvogood morning pstolowski07:07
mupPR snapd#9251 opened: o/snapstate, features: add feature flag for disk space check on remove <Disk space awareness> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9251>07:25
mupPR core#117 closed: extrafiles/writable-paths: make /etc/default/crda writable <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/core/pull/117>07:27
mupPR snapd#9138 closed: many: refactor tpm seal parameter setting <UC20> <Created by cmatsuoka> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9138>07:35
zygaI need a review for a test helper https://github.com/snapcore/snapd/pull/924308:57
mupPR #9243: testutil: add checkers for symbolic link target <Created by zyga> <https://github.com/snapcore/snapd/pull/9243>08:57
zygathis will unblock my other patches as they all rely on it08:58
zygatests/main/interfaces-cups-control  is failing in a weird way on 20.1009:00
zygalpr: Error - no default destination available.09:00
zygait looks like really wants to print09:00
zygaok, I'll make coffee and will return shortly09:16
mupPR snapd#8843 closed: [RFC] many: export tools from core/snapd to mount namespaces <Needs Samuele review> <Created by zyga> <Closed by zyga> <https://github.com/snapcore/snapd/pull/8843>09:16
zygaback with coffee09:33
zygamvo good morning09:33
zygahow was your luck yesterday?09:33
* zyga spends a moment to move hexchat config to a different host09:35
zyganow to prepare a ZFS test environment09:43
zygaI have a pair of old 10K RPM drives09:45
zygaperfect for scratch disks09:45
zyganow to install them09:46
zygaI live very far away from train  tracks but once a day a huge cargo train supplies the power plan and uses the horn/whistle while crossing roads09:48
zygathat sound travels!09:48
zygadrives installed, let's do ZFS10:21
mvozyga: hey, sorry in various meetings this morning10:22
zygahmm, I think I didn't plug power10:22
zygamvo no worries, how are you doing?10:22
zygathank you for the review on the test helper10:22
mvozyga: doing fine, wish I had more time for $things but that's it10:22
zygamvo I'm sorry, maybe it will be better with time10:24
* zyga checks cables as disk are not detected in bios or anywhere10:26
zygagot it, the power cable was unplugged :D10:29
zyga-x240ok, disks are working10:32
zygaall this rebooting10:40
zygathe drives have an MD array and I want to take a peek before wiping them10:41
mupPR snapd#9241 closed: tests: do not set rsyslog.disable in core18 config defaults test <Test Robustness> <⛔ Blocked> <Created by stolowski> <Closed by stolowski> <https://github.com/snapcore/snapd/pull/9241>10:51
zyga-x240pstolowski: would you mind doing a 2nd review on a small test helper11:06
mupPR #9243: testutil: add checkers for symbolic link target <Created by zyga> <https://github.com/snapcore/snapd/pull/9243>11:06
=== zyga-x240 is now known as zyga-kaveri
pedronismvo: I re-reviewed #921011:16
mupPR #9210: daemon: add /v2/systems "reboot" action API <Created by mvo5> <https://github.com/snapcore/snapd/pull/9210>11:16
mvopedronis: \o/ thank you11:19
zyga-kaveriI've installed ZFS and am familiarizing myself with the CLI11:21
pstolowskizyga-kaveri: sure11:31
zyga-kaveripstolowski: thank you, it's simple and unlocks interesting code11:32
zyga-kaveripstolowski: thank you11:49
mupPR snapd#9243 closed: testutil: add checkers for symbolic link target <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/9243>11:51
zyga-kaveriplaying with zfs12:18
zyga-kaveriI used zfs on freebsd a long while ago12:18
zyga-kaverimost things are similar but I don't remember the details12:18
zyga-kaverifirst odd thing is that creating a pool called "test" mounted it at /test12:19
mupPR snapd#9252 opened: osutil: add ExchangeFiles <Created by zyga> <https://github.com/snapcore/snapd/pull/9252>12:37
zyga-kaveriso I played with zfs, renameat2() fails with EINVAL, we just need to code the flock-based replacement for that12:41
zyga-kaveriI'll do that now12:41
mupPR snapd#9253 opened: sysconfig/cloudinit.go: add AllowCloudInit and use GadgetDir for cloud.conf <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9253>12:57
zyga-kaverimvo: I figured out why my sound broke13:32
zyga-kaverimvo: I booted a VM, this re-configures bluetooth and stuff falls a apart, known issue in vmware13:32
zyga-kaverimvo: I routinely boot infrequently used VMs to upgrade them during standup calls13:32
zyga-kaveriso now I know ;013:33
mvozyga-kaveri: ok13:34
ijohnsonzyga-kaveri: why the new IRC nick ?13:35
zyga-kaveriijohnson: I don't use irc cloud for a while13:35
zyga-kaveriijohnson: so each of my systems uses a distinct one to avoid clashes13:35
ijohnsonzyga-kaveri: right but what does kaveri mean13:35
ijohnsonah I see, that's one of your machines13:35
zyga-kaveriijohnson: this is my old kaveri FM2 system13:35
zyga-kaveriijohnson: it's a AMD APU fro 2016 IIRC13:35
ijohnsonah it's an amd processor line13:36
zyga-kaveriijohnson: it's native too :)13:36
zyga-kaveriijohnson: I have zyga-matisse as well but that's not booted now :) (upgrades)13:36
zyga-kaveribut I probably won't use it for daily work as this box is good enough for typing13:36
ijohnsonzyga-kaveri: I see makes sense13:37
ijohnsonafter my irccloud subscription runs out, I'm going to try and migrate IRC to mattermost with the bridge and see how that fares13:38
ijohnsonI've heard mixed reviews of using the bridge like that13:38
ijohnsonit would be nice to have all chats in one app though13:38
cmatsuokapedronis: I have some questions related to bootloader boot chains, please let me know when you have some time to discuss13:39
zyga-kaveriijohnson: interesting, let me know how that works13:39
ijohnsonyeah I'm hoping it works out fine, remains to be seen how resilient / robust my local network connection actually is to get 24 hr connections, I'm sure it will cut out and I will lose/reconnect the bridge every so often13:41
zyga-kaveriah, so the bridge is local?13:41
mvopstolowski: I merged 9251 it will need a dedicated PR for 2.46. cherry pick has conflicts unfortunately13:51
mupPR snapd#9251 closed: o/snapstate, features: add feature flag for disk space check on remove <Disk space awareness> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9251>13:52
pedroniscmatsuoka: we can chat before the 2nd Town Hall or after it? do you have a preference?13:52
cmatsuokaI have both slots open, either will do13:52
pedroniscmatsuoka: how much time do you think we need?13:53
cmatsuokapedronis: ten minutes or so, it should be a quick alignment13:53
pedroniscmatsuoka: done13:56
cmatsuokapedronis: thanks13:56
pstolowskimvo: yes, i was expecting it due to the safeMargin helper13:56
pstolowskimvo: i'll prepare it13:57
mvopstolowski: ta13:58
mupPR snapcraft#3273 opened: spread tests: lock down setuptools for plainbox <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3273>14:04
pedronisijohnson: I forgot about also this option: https://github.com/snapcore/snapd/pull/9237#discussion_r48116958214:14
mupPR #9237: [RFC] many: enable cloud-init on uc20 for grade signed and secured <Complex> <Run nested> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9237>14:14
ijohnsonsorry was afk a bit, silly plumbing problems with our new faucet14:31
ijohnsonpedronis: looking14:31
ijohnsonpedronis: right yes that's true in general, but numbering the files we copy from seed is an orthogonal concern to the current PR14:33
ijohnsonor maybe it's not, just that this PR doesn't change how files from seed are handled14:33
pedronisijohnson: yes, but changes the reasoning around picking a number there14:48
ijohnsonpedronis: how does the numbering scheme for files from seed affect the numebring scheme for the gadget ?14:48
pedronisijohnson: because the question we discussed the last time14:48
ijohnsonpedronis: I have not yet talked with the maas/curtin team about what numbering scheme they use and what we should fit into with that14:48
pedronisijohnson: (in a differe meeting I will get back to you in a bit)14:48
pedronisijohnson: my point is that we know we have the option to renumber, we don't have to pick a number based on what we expect the numbering used by curtin, we can decide when we renumber whether to renumber before or after our chosen number14:54
pedronisijohnson: we can pick 70- and make what comes from the seed either 69- or 71-14:56
pedronisfor example14:56
pedronisdepending the sematics we want14:56
ijohnsonok, so does that mean we want the gadget cloud.conf to be 70 ?14:56
pedronisijohnson: I would pick "80_" as prefix15:01
ijohnsonpedronis: ok15:11
pedronisijohnson: I proposed something, but not 100% happy with it yet15:12
ijohnsonah I forgot to fix that PR I just opened with the filename15:20
* ijohnson has too many open branches / prs15:20
cachiozyga-kaveri, bad news, my zfs instance is not booting anymore locally, need to see which dependency caused that15:30
cachiozyga-kaveri, I have a snapshot to restore15:32
* cachio lunch15:58
pstolowskimvo: opened https://github.com/snapcore/snapd/pull/9255 ; it's slightly annoying, it's a single cherry pick but i guess it will conflict when marging back (not sure)16:05
mupPR #9255: o/snapstate, features: add feature flag for disk space check on remove (2.46) <Created by stolowski> <https://github.com/snapcore/snapd/pull/9255>16:05
zyga-kavericachio: ok, keep me poosted please16:05
pstolowskimvo: basically it depended on the new helper and new error type name from another PR, but it's impossible to cherry pick those as they already depended on install-many changes16:06
mupPR snapd#9254 opened: sysconfig/cloudinit.go: add DisableNoCloud to CloudInitRestrictOptions <Simple 😃> <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9254>16:07
mupPR snapd#9255 opened: o/snapstate, features: add feature flag for disk space check on remove (2.46) <Created by stolowski> <https://github.com/snapcore/snapd/pull/9255>16:07
mvopstolowski: thanks, that's ok - unavoidable sometimes16:07
zyga-kaverimvo: thank you for the review16:08
pedronis#9249 needs a 2nd review16:23
mupPR #9249: boot,bootloader,gadget: apply new bootloader.Options.Role <Run nested> <UC20> <Created by pedronis> <https://github.com/snapcore/snapd/pull/9249>16:23
ijohnsonpedronis: I reviewed 924917:48
mupPR core20#83 opened: static/writable-paths: make /etc/default/crda writable <Created by anonymouse64> <https://github.com/snapcore/core20/pull/83>17:54
mupPR core18#167 opened: static/writable-paths: make /etc/default/crda writable <Created by anonymouse64> <https://github.com/snapcore/core18/pull/167>17:54
cachiozyga-kaveri, this is better18:03
cachioI'll try that18:03
cachiozyga-kaveri, it is very hard to create a cloud image based on a desktop because I need to upload 4 GB to gcloud and I tried but at some point my internet breanks and the upload too18:04
pedronisijohnson: thanks I'll merge it and consider your comments for a follow, so cmatsuoka can possibly start using Role and its constants18:07
ijohnsonpedronis: sure that's fine18:07
cmatsuokapedronis: thanks, I'll use it in BootFile18:09
mupPR snapd#9249 closed: boot,bootloader,gadget: apply new bootloader.Options.Role <Run nested> <UC20> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9249>18:13
pedronisijohnson: created https://github.com/snapcore/snapd/pull/925618:54
mupPR #9256: boot,bootloader: clarifications after the changes to introduce bootloader.Options.Role <Created by pedronis> <https://github.com/snapcore/snapd/pull/9256>18:54
ijohnsoncool I'll have a look18:55
mupPR snapd#9256 opened: boot,bootloader: clarifications after the changes to introduce bootloader.Options.Role <Created by pedronis> <https://github.com/snapcore/snapd/pull/9256>18:58
ijohnsonwhy is our assertion database so complicated :-(19:38
pedronisijohnson: anything in particular?19:41
ijohnsonpedronis: I am still fighting getting uc20 with test keys to finish seeding properly, I thought I had it working with just seeding the fake snapd snap, but then realized it was failing to seed actually because snap-bootstrap needs to be built with test keys too, so I need to seed both the fake snapd snap and the fake kernel snap and now my change I made to prepare-image to pick up the local assert files isn't working19:43
ijohnsonfor multiple fake snaps19:43
ijohnsonwhat's confusing right now is that I add these assertions to the db, then immediately after seedwriter goes to find those assertions and doesn't find them somehow19:43
pedronisyou made a change to prepare-image ?19:43
ijohnsonpedronis: yes I have to change prepare-image, as it doesn't work with the test keys as I mentioned in the SU doc19:44
ijohnsonI changed it to (try to) import/use assert files matching the --snap file arguments19:44
ijohnsonotherwise, prepare-image thinks all --snap arguments are dangerous unasserted snaps19:45
pedronisthis is going to be very hard to land19:45
ijohnsonI mean it's getting very hard just to get working19:46
ijohnsonunless I'm missing something very obvious19:46
pedronisijohnson: my idea was to point prepare-image to the fakestore19:47
pedronisand have the snaps there19:47
ijohnsonpedronis: yeah I guess I haven't tried that yet, perhaps I should try that instead of pointing prepare-image at everything locally19:47
pedronisI thought at least one of the tests I mentioned does that19:47
ijohnsonyes there is an example of that there, it just seemed like more work than using the local files/assertions but actually in hindsight it is probably less work if it just works19:48
ijohnsonit just _feels_ like providing local assertions and files to prepare-image should be simpler19:48
pedronisijohnson: the problem with what you are trying is that there's no design for it19:48
ijohnsonthat's also what I'm confused about, because ubuntu-image specifically works like this, and I thought it worked like this because of prepare-image, but perhaps ubuntu-image has it's own magic19:49
ijohnsonbecause I can do `ubuntu-image --snap=my-local-snap.snap` and if there's an my-local-snap.assert file, then the snap is not treated as unasserted19:49
ijohnsonbut prepare-image doesn't work that way it seems19:49
pedronisijohnson: yes, but you still need the fake store19:49
ijohnsondoes ubuntu-image use the fake store ?19:50
pedronisijohnson: what happens in that case is that the file is locally but the assertions come from the store19:50
ijohnsonpedronis: ahhhhhhhhhh so it's cheating really19:50
ijohnsonwell that's just silly but ok19:50
ijohnsonalright I will switch to trying to use fakestore as a proxy19:50
pedronisis not silly19:50
pedronisbut is not very intuitive19:51
ijohnson... but what if you wanted to build an image without a connection to the store ? you have the assert files and the snap files, but you still need to talk to the store because ... ?19:51
ijohnsonanyways it's not material at this point I suppose19:51
pedronisthat's not the use case that was added for19:51
pedronisthe use case is I have kept old snaps and I want to rebuild an image excactly with those19:52
pedronisit also shows up in some usage pattern when building classic images19:53
ijohnsonsorry I guess as a user it just feels very weird to say "here take these snaps and assertions and build me an image with them" and snapd then proceeds to ignore the assertions provided and goes and gets it's own assertions from the storyt19:54
pedronisthere's no way to say take this assertion atm though19:55
ijohnsonhaha yes that is quite clear to me now19:55
mupPR snapcraft#3273 closed: spread tests: lock down setuptools for plainbox <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3273>19:55
ijohnsonsorry anyways I am probably getting us both off track now19:55
pedronisijohnson: so the easiest is probably to use the fakestore only for assertions20:00
pedronisstill pass your snaps with --snap20:00
pedronisand then maybe it should work20:00
ijohnsonmmm I don't think I can do that tho20:00
ijohnsonwhenever I try to provide --snap ... to prepare-image it says local snaps are not allowed with dangerous20:01
ijohnsonerr not allowed with signed20:01
pedronisthat sounds strange but possible20:01
ijohnsonI mean unless ubuntu-image has some magic that prepare-image doesn't20:01
pedronisijohnson: afaict it should give that error if we could not fetch assertions for the snap20:06
pedronisthe check seems late enough20:06
pedronisat least20:06
ijohnsonmmm I guess I didn't have the SAS mocking setup when I tried it, perhaps now with the right mocking of the fakestore it will work20:07
pedronisijohnson: anyway I don't know if I pointed to it, but tests/main/classic-prepare-image-no-core does what I described20:08
pedronisin terms of using the fakestore only for some assertions20:08
ijohnsonpedronis: alright well that was actually much easier20:18
* ijohnson feels rather foolish20:18
ijohnsonnah it's my fault for being stubborn and trying to make it work the way I thought it should work20:20
pedronisanyway afaict using --snap with signed (as long as we can find assertions) works, but is not tested, I should add a test20:22
ijohnsonyes it does work with mocking SAS20:23
pedronisthere's a test about this in seedwriter but is using dangerous anyway because is doing also some other things that aren't allowed20:23
pedronisI should have a test with also just the allowed subset20:24
pedronisand signed20:24
ijohnsoncan I just say how thankful I am that we don't also depend on signed assets from the core20 snap20:24
ijohnsonif we did, then literally every snap would need to be resigned to do local builds like this20:24
ijohnsonI'm already at 3/4 snaps20:24
ijohnsonyay we at least made it out of the initramfs20:30
* ijohnson waits anxiously to see if we make it out of install mode20:30
ijohnsonmmm seems not :-/20:31
mupPR snapd#9256 closed: boot,bootloader: clarifications after the changes to introduce bootloader.Options.Role <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9256>20:43
cachiozyga, around?21:03
* pedronis calling it a day21:11
ijohnsoneyyyyy got it working21:16
* ijohnson soft EODs21:16
mupPR snapcraft#3271 closed: build providers: use the releases endpoint for LXD <enhancement> <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3271>22:00
mupPR snapcraft#3272 closed: storeapi: improve to channel map docstrings <bug> <documentation> <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3272>22:00
cachioijohnson, hey, about the test in 822:33
cachiowe do have all the failover tests as part of the core suite22:33
cachiois it different to have it in the nested one?22:34
ijohnsoncachio: well it's fine to include for the regular core suite, but I also want that test to run with secure boot and fde, even though it's not specifically fde related, it's boot related23:05
mupPR snapcraft#3274 opened: schema: rename package-repository's "deb-types" to "format" <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3274>23:15
mupPR snapcraft#3275 opened: cli: ignore sudo warning when using multipass <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3275>23:35
mupPR snapcraft#3276 opened: meta: add suite/component validation checks for package-repositories <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3276>23:40

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!