| mborzecki | morning | 06:03 |
|---|---|---|
| zyga | o/ | 06:09 |
| mup | PR snapd#8809 opened: tests: fix and trim debug section in xdg-open-portal <Simple 😃> <Test Robustness> <Created by zyga> <https://github.com/snapcore/snapd/pull/8809> | 06:11 |
| zyga | some weird failures related to snap-mgmt script | 06:18 |
| zyga | chasing one now | 06:18 |
| mup | PR snapd#8799 closed: interfaces/system-packages-doc: fix typo in variable names <Simple 😃> <Created by jdstrand> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8799> | 06:21 |
| mup | PR snapd#8805 closed: tests: port interfaces-calendar-service to tests.session <Test Robustness> <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8805> | 06:21 |
| mup | PR snapd#8806 closed: tests: install/run the lzo test snap too <Simple 😃> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8806> | 06:21 |
| zyga | hey mvo | 06:22 |
| zyga | mvo: note: please don't merge the retry PR as it needs updates now | 06:22 |
| mborzecki | zyga: mvo: hey | 06:24 |
| mvo | zyga: hey | 06:25 |
| mvo | zyga: can you mark it blocked please? | 06:26 |
| mvo | mborzecki: good morning | 06:26 |
| zyga | ok | 06:26 |
| mup | PR snapd#8792 closed: interfaces: miscellanious policy updates xlv <Created by jdstrand> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8792> | 06:26 |
| mup | PR snapd#8793 closed: interfaces: miscellanious policy updates xlv - 2.45 <Created by jdstrand> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8793> | 06:26 |
| mborzecki | wow, 74 PRs, it was lower 60s yesterday | 06:26 |
| mup | PR snapd#8788 closed: cmd/snap-confine: add support for libc6-lse <Bug> <Needs security review> <Squash-merge> <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8788> | 06:31 |
| mup | PR snapd#8801 closed: vendor: update to latest github.com/snapcore/bolt for riscv64 <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8801> | 06:31 |
| zyga | + snap install --channel=edge core20 | 06:33 |
| zyga | error: too early for operation, device not yet seeded or device model not acknowledged | 06:33 |
| zyga | this is from https://github.com/snapcore/snapd/pull/8798/checks?check_run_id=734681152 | 06:33 |
| mup | PR #8798: data/selinux: allow checking /var/cache/app-info <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8798> | 06:33 |
| zyga | missing wait in our code somewhere? | 06:33 |
| mvo | mborzecki: 8798 has some failures in centos, looks like related to the diff? | 06:33 |
| mborzecki | zyga: yes, apaprently the centos 7 policy is quite old | 06:33 |
| mborzecki | well entirely unexpected, but it's missing the interface, hope it has the right types at least | 06:34 |
| mup | PR snapd#8778 closed: tests: modernize and use snapd.tool <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8778> | 07:12 |
| mborzecki | zyga: heh, so there's something about optional_policy() i don't understand, maybe #fedora-selinux folks can help | 07:19 |
| pedronis | zyga: hi, you could also add a retry-tool symlink to 8796 with no spread and land it, and do another one to fix the other tests and remove the symlink later | 07:20 |
| zyga | pedronis: good idea, I'll do that | 07:20 |
| ogra | hmm, it is really hard to get the available disk space from a snap on core ... seems /writable is nowhere to be seen from inside the snap | 07:24 |
| ogra | and the size of /var/lib/snapd/hostfs is the size of / ... which is the core snap | 07:25 |
| zyga | ogra: I replied to something similar from a customer before, one simple way is to check the size of $SNAP_DATA and $SNAP_USER_DATA | 07:25 |
| zyga | as in statfs | 07:25 |
| ogra | oh, i actually remember that discussion, hah | 07:25 |
| ogra | thanks !! | 07:25 |
| ogra | root@pi4😕home/ogra# df -h $SNAP_DATA | 07:26 |
| ogra | Filesystem Size Used Avail Use% Mounted on | 07:26 |
| ogra | /dev/sda1 458G 68G 367G 16% /var/snap | 07:26 |
| ogra | yeah, that works fine | 07:26 |
| ogra | (looks like hexchats emoji pligun needs fixing too ...) | 07:28 |
| ogra | *plugin | 07:28 |
| zyga | hmmm | 07:35 |
| zyga | curious failures | 07:35 |
| zyga | https://www.irccloud.com/pastebin/WxFz0F8w/ | 07:37 |
| zyga | I've seen this many times today | 07:37 |
| zyga | must be something recently introduced | 07:37 |
| zyga | hmm hmm h | 07:38 |
| pedronis | would be good to know what is in there that makes it non-empty | 07:43 |
| pedronis | it would give us a clue | 07:43 |
| pedronis | what system is that on? | 07:44 |
| mborzecki | didn't we have a find/ls -l /var/lib/snapd/ in the spread.yaml level debug section? | 07:51 |
| pedronis | not atm afaict | 07:53 |
| zyga | pedronis: it seems it was one of the first things this test did | 07:57 |
| zyga | it was still preparing the suite | 07:57 |
| zyga | I'll add a debug section to this | 07:57 |
| zyga | back from random power failure on x240 :( | 07:57 |
| zyga | I tried reproducing it with seed but failed twice | 07:58 |
| zyga | so it may not be test related but another kind of race that is just inside the system | 07:58 |
| zyga | .... and I know that holding my x240 close to the right side of the hinge shuts it down instantly | 08:00 |
| zyga | some cable is getting pinched? | 08:01 |
| zyga | not the best moment for this | 08:01 |
| mup | PR snapd#8809 closed: tests: fix and trim debug section in xdg-open-portal <Simple 😃> <Test Robustness> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8809> | 08:12 |
| mup | PR snapd#8810 opened: spread.yaml: show /var/lib/snapd in debug <Test Robustness> <Created by zyga> <https://github.com/snapcore/snapd/pull/8810> | 08:37 |
| zyga | no luck reproducing locally, I opened a PR to see if that has more luck | 08:38 |
| zyga | huh | 08:46 |
| zyga | we're not lucky today | 08:46 |
| zyga | check out this *unit test* failure | 08:46 |
| zyga | FAIL: cmd_export_key_test.go:60: SnapKeysSuite.TestExportKeyAccount | 08:46 |
| zyga | cmd_export_key_test.go:65: | 08:46 |
| zyga | c.Assert(err, IsNil) | 08:46 |
| zyga | ... value *errors.errorString = &errors.errorString{s:"/usr/bin/gpg2 --batch --list-secret-keys --fingerprint --with-colons --fixed-list-mode failed: exit status 2 (\"gpg: starting migration from earlier GnuPG versions\\ngpg: can't connect to the agent: IPC connect call failed\\ngpg: error: GnuPG agent unusable. Please check that a GnuPG agent can be started.\\ngpg: migration aborted\\n\")"} ("/usr/bin/gpg2 --batch | 08:46 |
| zyga | --list-secret-keys --fingerprint --with-colons --fixed-list-mode failed: exit status 2 (\"gpg: starting migration from earlier GnuPG versions\\ngpg: can't connect to the agent: IPC connect call failed\\ngpg: error: GnuPG agent unusable. Please check that a GnuPG agent can be started.\\ngpg: migration aborted\\n\")") | 08:46 |
| zyga | perhaps we should do something about ~/.gnupg in github actions startup | 08:47 |
| zyga | or perhaps we should mock gpg entirely and only test it in spread tests | 08:48 |
| zyga | mvo, pedronis: ^ any preference? | 08:48 |
| mvo | zyga: in a meeting | 08:50 |
| mup | PR snapd#8796 closed: tests: modernize retry tool <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8796> | 08:57 |
| zyga | hmmm | 08:59 |
| zyga | "too early for operation" *after* waiting for seeding https://www.irccloud.com/pastebin/faUH8YSh/ | 08:59 |
| pedronis | zyga: don't we have code to make sure we shutdown the agent? | 09:01 |
| zyga | pedronis: this is during the go test ./... phase in github action itself, not in spread | 09:02 |
| zyga | pedronis: it seems the image we are running on top of, needs to perform the migration | 09:02 |
| zyga | (2nd topic: this failure occurred in prepare.sh:664 | 09:03 |
| zyga | which is weird, because there's clearly a "snap wait system seed.loaded" above | 09:03 |
| pedronis | zyga: I see, sounds like we need to wait for something gpg related then in the action. I think we have code in the spread stuff related to that | 09:07 |
| zyga | pedronis: I'll look around | 09:07 |
| zyga | pedronis: the wait thing is more mysterious, it suggests there's a bug in snapd | 09:07 |
| pedronis | pstolowski: seems there's a real unit test error on 14.04 in core 20 defaults PR | 09:28 |
| pstolowski | pedronis: looking | 09:29 |
| pstolowski | hmm interesting | 09:31 |
| pstolowski | will investigate in a sec | 09:31 |
| mup | PR snapd#8795 closed: cmd/snap-bootstrap/initramfs-mounts: also copy systemd clock + netplan files <UC20> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8795> | 09:32 |
| mup | PR snapd#8797 closed: snap/naming: add helpers to parse app and hook security tags <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8797> | 09:32 |
| mup | PR snapd#8807 closed: Revert "Enable riscv64 builds in the edge PPA without PIE" <Created by xnox> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8807> | 09:32 |
| zyga | thanks mvo! | 09:32 |
| zyga | I will have more cgroup patches shortly | 09:34 |
| === joc_ is now known as joc | ||
| zyga | pstolowski: the preseed-lxd test has a small bug, I'll send a patch shortly | 10:12 |
| mborzecki | ehh, selinux is so arcane | 10:13 |
| pstolowski | zyga: thanks | 10:13 |
| mborzecki | 2h+ of fighting with selinux policy, optional_policy(), ifndef(), m4 and make | 10:13 |
| mborzecki | #8798 is a trivial fix but making it work on all distros we built it on, with the limitations of how kernel policy files are compiles is a pita | 10:15 |
| mup | PR #8798: data/selinux: allow checking /var/cache/app-info <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8798> | 10:15 |
| zyga | pstolowski: https://github.com/snapcore/snapd/pull/8811 | 10:18 |
| mup | PR #8811: tests: autoremove after removing lxd in preseed-lxd test <Test Robustness> <Created by zyga> <https://github.com/snapcore/snapd/pull/8811> | 10:18 |
| * zyga break, need to try to move a little | 10:18 | |
| pstolowski | zyga: thank you! i think i saw this once when working on this test and running entire testsuite, then it couldn't reproduce | 10:20 |
| mup | PR snapd#8811 opened: tests: autoremove after removing lxd in preseed-lxd test <Test Robustness> <Created by zyga> <https://github.com/snapcore/snapd/pull/8811> | 10:22 |
| pstolowski | pedronis: pushed a fix to #8567, it was missing mocking for systemctl | 10:26 |
| mup | PR #8567: o/devicestate: core20 early config from gadget defaults <UC20> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8567> | 10:26 |
| mborzecki | so what's the policy of adding snaps under tests/lib/snaps vs under the test directory? | 10:30 |
| mborzecki | there's test-snapd-sh in tests/lib/snaps and another one under tests/main/interfaces-appstream-metadata | 10:30 |
| mborzecki | oh and there's one in the store too | 10:32 |
| zyga | mborzecki: IIRC the current preference is to not share snaps if they are really specific to a test | 10:33 |
| zyga | you can create a shared snap if you think it makes sense to do so | 10:33 |
| zyga | sharing snaps is problematic because we had a pattern of sharing and then changing the snap | 10:33 |
| zyga | that had unexpected consequences | 10:33 |
| zyga | I have a feeling we could unshare an number of snaps | 10:34 |
| zyga | and then be left with a small pool of really shared snaps | 10:34 |
| zyga | that have well defined semantics | 10:34 |
| pedronis | likely yes, but probably not a good time for that change, we don't have even a good story how we mantain those snaps | 10:37 |
| zyga | interesting, there's also a subset that is in the store | 10:40 |
| zyga | that are there for assertions | 10:40 |
| zyga | yeah, it's a bit messy, my recommendation is not to make it worse :) | 10:40 |
| zyga | mvo: question on https://github.com/snapcore/snapd/pull/8804#discussion_r435155159 | 10:42 |
| mup | PR #8804: tests: port xdg-settings test to tests.session <Test Robustness> <Created by zyga> <https://github.com/snapcore/snapd/pull/8804> | 10:42 |
| mvo | zyga: replied | 10:45 |
| mup | PR snapd#8785 closed: sandbox/cgroup: move FreezerCgroupDir from dirs.go <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8785> | 10:53 |
| mup | PR snapd#8790 closed: tests: update the file used to detect the boot path on uc20 <Created by sergiocazzolato> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8790> | 10:53 |
| mup | PR snapd#8810 closed: spread.yaml: show /var/lib/snapd in debug <Test Robustness> <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8810> | 10:53 |
| zyga | thanks! | 10:53 |
| zyga | please alert me if tests fail on dpkg | 10:53 |
| zyga | we might find out what was there now | 10:53 |
| mborzecki | heh, shellcheck complains about tests/main/lxd https://paste.ubuntu.com/p/6pzGfwKC2n/ | 11:01 |
| mborzecki | mvo: zyga: i've updated https://github.com/snapcore/snapd/pull/8798 since you reviewed it before, please take another look | 11:08 |
| mup | PR #8798: data/selinux: allow checking /var/cache/app-info <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8798> | 11:08 |
| zyga | sure | 11:08 |
| mvo | ta | 11:09 |
| mborzecki | it'd still be nice to get it into 2.45.1 | 11:09 |
| mborzecki | ok, back to review | 11:10 |
| zyga | mborzecki: do you need the optional_policy now that you have the ifdef? | 11:10 |
| mborzecki | zyga: in theory it's nth | 11:10 |
| zyga | ? | 11:11 |
| mborzecki | nice to have, i should probably take a look at other interface uses and wrap them as well | 11:11 |
| zyga | mborzecki: what is the rename? github truncates things | 11:12 |
| zyga | ah, I see it in a tooltip | 11:12 |
| mborzecki | test-snapd-sh -> test-snapd-appstream-metadata | 11:13 |
| zyga | Could you update the snap to have less apps and the "sh" app actually holds the interface you want | 11:14 |
| zyga | otherwise it's a bit weird | 11:14 |
| zyga | or is it the only app there? | 11:14 |
| zyga | it's kind of verbose for no good reason saying the same thing twice | 11:14 |
| mborzecki | like test-snapd-apptream-metadata.sh? | 11:15 |
| zyga | yeah | 11:15 |
| zyga | personal preference, if you like it | 11:15 |
| zyga | ok, I need to take a break | 11:17 |
| zyga | try to move around a little | 11:17 |
| mborzecki | Eighth_Doctor: about the label https://paste.ubuntu.com/p/ZyzpPrZDHx/ unless it's owned by multiple packages, but not sure to make rpm query to show that | 11:17 |
| zyga | next up more cgroup branches | 11:17 |
| zyga | please remember to merge master and report test failures | 11:17 |
| Eighth_Doctor | mborzecki: rpm -qf /var/cache/app-info | 11:17 |
| zyga | if claudio asks: I did *not* deploy spread upgrade yet, because we had a backlog of tests to run through and I wanted to avoid interruptions, since everything is back to normal now I will do it tonight | 11:18 |
| Eighth_Doctor | offhand, appstream and PackageKit own that too | 11:18 |
| mborzecki | Eighth_Doctor: it shown only packagekit, | 11:18 |
| Eighth_Doctor | and actually... fwupd does not own that directory | 11:18 |
| Eighth_Doctor | ugh | 11:18 |
| Eighth_Doctor | I love SELinux, but god damn it | 11:18 |
| mborzecki | hahah | 11:18 |
| zyga | heheh | 11:18 |
| zyga | some humor in grim days | 11:19 |
| * zyga afk | 11:19 | |
| mborzecki | Eighth_Doctor: feels like there should be a separate label, appstream_cache_t or somesuch | 11:20 |
| Eighth_Doctor | yes | 11:20 |
| mborzecki | Eighth_Doctor: also, the file contexts are defined in fwupd policy module :P | 11:20 |
| Eighth_Doctor | GAAH | 11:20 |
| Eighth_Doctor | that's definitely a bug | 11:21 |
| Eighth_Doctor | the label is wrong, and the ownership is broken | 11:21 |
| mborzecki | Eighth_Doctor: filed https://bugzilla.redhat.com/show_bug.cgi?id=1843881 | 11:29 |
| mborzecki | Eighth_Doctor: notice how /var/cache/fwupd inherits var_t :P | 11:30 |
| Eighth_Doctor | 🤦♂️ | 11:33 |
| mup | PR snapd#8591 closed: secboot,cmd/snap-bootstrap: add tpm sealing support to secboot <Needs Samuele review> <UC20> <Created by cmatsuoka> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/8591> | 11:33 |
| mup | PR snapd#8812 opened: o/snapstate: service-control task handler (4/N) <Needs Samuele review> <Services ⚙️> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8812> | 11:33 |
| mborzecki | yay sealing landed | 11:33 |
| mborzecki | wonder it'd be possible to chop #8340 into smaller bits | 11:34 |
| mup | PR #8340: boot, snap-bootstrap: move initramfs-mounts logic to boot pkg <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8340> | 11:34 |
| mup | PR snapd#8813 opened: gadget,cmd/snap-bootstrap: move partitioning to gadget <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/8813> | 11:48 |
| mup | PR snapd#8811 closed: tests: autoremove after removing lxd in preseed-lxd test <Test Robustness> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/8811> | 12:23 |
| === Ps1-Jack is now known as Psi-Jack | ||
| zyga | I'm seeing some network woes in gce | 12:50 |
| zyga | both on snap install/download and apt install | 12:50 |
| zyga | - Download snap "test-snapd-dbus-provider" (6) from channel "beta" (Get https://canonical-bos01.cdn.snapcraft.io/download-origin/canonical-lgw01/vnDT8UYR44P8qyBwRSiXHHCoaoq9pz9z_6.snap?token=1591286400_c8896515d76222f533efc97384292fa6e4826cc6: dial tcp 91.189.91.42:443: connect: connection timed out) | 12:53 |
| zyga | reported internally | 12:55 |
| zyga | interactively I cannot install any snap from the store, from GCE | 12:55 |
| zyga | https://github.com/snapcore/snapd/pull/7825/files is now at sub 2K additions, | 13:12 |
| mup | PR #7825: many: use transient scope for tracking apps and hooks <Needs security review> <Security-High> <Created by zyga> <https://github.com/snapcore/snapd/pull/7825> | 13:12 |
| zyga | I will trim it some more soon | 13:12 |
| clmsy | hi everyone | 13:21 |
| clmsy | i just build the core16 image use to flash some devices for test purposes | 13:21 |
| clmsy | but something is wrong | 13:21 |
| clmsy | snap list returns an empty list | 13:21 |
| clmsy | under /var/lib/snapd/seed/snaps/ all snaps are here but they are not installed | 13:22 |
| clmsy | snap warnings say that seeding failed with assertion is signed with expired public key | 13:22 |
| clmsy | it says go here check it out https://forum.snapcraft.io/t/incorrect-seed-yaml-for-some-system/16341 | 13:22 |
| clmsy | the mv command tells me to move the whole folder that includes snaps as well | 13:23 |
| clmsy | this does not make that much sense to me | 13:23 |
| clmsy | does the topic owner meant move the seed.yaml file or something | 13:23 |
| zyga | clmsy: that topic was about a bug in some seeds that were produced in development releases | 13:25 |
| zyga | how are you building your uc16 image? | 13:25 |
| zyga | the instructions there are correct | 13:25 |
| zyga | I think we need to understand what is wrong in your case | 13:25 |
| clmsy | i have a kernel snap and a gadget snap based on core16 and i bundle them together with ubuntu-image tool | 13:27 |
| clmsy | i can confirm this has worked multiple times before | 13:27 |
| clmsy | but today i get this message: | 13:28 |
| clmsy | "no matching public key "BWDE***********redacted" for signature by "canonical" | 13:28 |
| zyga | hmmm | 13:28 |
| zyga | is the key in the image? | 13:28 |
| zyga | not sure, maybe some new bug | 13:28 |
| clmsy | if i try to do snap install something it says "too early for operation, device not yet seeded or device model not acknowledged" | 13:28 |
| clmsy | hm | 13:29 |
| ogra | clock off ? | 13:29 |
| ogra | (does the device have an RTC ? does that have the correct time ) | 13:29 |
| clmsy | maybe its time related let me double check that | 13:30 |
| ogra | expired key is pretty typical if your clock is completely off | 13:30 |
| zyga | heh | 13:31 |
| zyga | it's ironic that the message says | 13:31 |
| zyga | "too early for operation" | 13:31 |
| zyga | :D | 13:31 |
| * zyga small break | 13:33 | |
| ogra | hmpf ... that store outage isnt nice ... (my PRs fail with download errors in travis) | 13:35 |
| * ogra considers a break too | 13:35 | |
| clmsy | haha | 13:39 |
| clmsy | anyway yes you are correct | 13:39 |
| clmsy | it was clock related issue | 13:39 |
| clmsy | was a very new device i did not expect the date to be in 2016 | 13:39 |
| clmsy | apologies | 13:39 |
| clmsy | .. | 13:39 |
| ogra | heh, no problem 🙂 | 13:40 |
| ogra | (i have run into that 1000s of times already ... though usually only on RTC-less devices) | 13:41 |
| clmsy | same to be honest, sometimes we forget the "time" :) | 13:47 |
| cachio | mvo_, hey, in 2.45.1 is included the support for core.experimental.user-daemons right? | 13:48 |
| cachio | because it is affecting the tests for uc20 | 13:48 |
| zyga | ogra, clmsy: https://github.com/snapcore/snapd/pull/8814 | 14:01 |
| mup | PR #8814: sanity: check for unsynchronized real time clock <Created by zyga> <https://github.com/snapcore/snapd/pull/8814> | 14:01 |
| clmsy | <3 | 14:02 |
| ijohnson | hey zyga and mborzecki do you have any suggestions on how to get the machine arch on non-debian machines i.e. fedora/arch and map those to our snap arch values ? | 14:03 |
| ijohnson | I was using dpkg --print-architecture, but obvs that won't work on non-debian distros | 14:03 |
| mup | PR snapd#8814 opened: sanity: check for unsynchronized real time clock <⛔ Blocked> <Created by zyga> <https://github.com/snapcore/snapd/pull/8814> | 14:03 |
| zyga | ijohnson: fedora uses kernel names IIRC | 14:05 |
| zyga | ijohnson: uname -m | 14:05 |
| ijohnson | zyga: you mean just `uname -m` ? | 14:05 |
| ijohnson | right | 14:05 |
| zyga | ijohnson: would use that | 14:05 |
| zyga | you can map from that to debian arch names relatively easily | 14:05 |
| zyga | at least, it's a bound problem | 14:05 |
| ijohnson | zyga: ok that's what I was thinking of doing | 14:06 |
| ijohnson | zyga: do you have or know where I could find such a mapping? | 14:06 |
| zyga | I *think* we have a few implementations of that in the gree | 14:06 |
| zyga | yes :) | 14:06 |
| zyga | one sec | 14:06 |
| ijohnson | thank you | 14:06 |
| zyga | it's a cool find from last few weeks | 14:06 |
| zyga | it's right on your system in... | 14:06 |
| zyga | two files: /usr/share/perl5/Dpkg/Arch.pm and /usr/share/dpkg/cputable | 14:08 |
| zyga | the former has some more logic | 14:08 |
| zyga | the latter is a map that will have most instant answers | 14:08 |
| zyga | good luck :) | 14:08 |
| === facundo__ is now known as facubatista | ||
| mborzecki | hm aren't we mapping that already somewhere? | 14:12 |
| zyga | pedronis: I think it would help if a blocked label had an accompanying comment | 14:12 |
| mborzecki | ijohnson: some comments in https://github.com/snapcore/snapd/pull/8340 i think we need a diagram or something to capture the whole of bootloader/initramfs/userspace snapd interaction | 14:16 |
| mup | PR #8340: boot, snap-bootstrap: move initramfs-mounts logic to boot pkg <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8340> | 14:16 |
| ijohnson | mborzecki: sure I can try to break it up and clarify the interaction between the interactions | 14:16 |
| pedronis | zyga: my hunch is that is not great idea as is, but I don't have time to think it through or formulate it right now, I don't want it to land it why I'm not paying attention | 14:16 |
| pedronis | s/why/while/ | 14:16 |
| mborzecki | ijohnson: i mean not for this PR, but in general, maybe when we're done with the refactor | 14:17 |
| zyga | pedronis: sure, that's fine, making such comment on the page can help when others come and look | 14:17 |
| mup | PR snapd#8567 closed: o/devicestate: core20 early config from gadget defaults <UC20> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/8567> | 14:18 |
| pedronis | pstolowski: ^ great! | 14:21 |
| pstolowski | yes.. | 14:21 |
| pstolowski | ijohnson: hey, do you have a moment for another look at #8780? | 14:22 |
| mup | PR #8780: tests: core20 early defaults spread test <Created by stolowski> <https://github.com/snapcore/snapd/pull/8780> | 14:22 |
| ijohnson | pstolowski: I will try to look at it in my PM today | 14:22 |
| pstolowski | thanks! | 14:26 |
| zyga | mvo_: bump about https://github.com/snapcore/snapd/pull/8352 -- should I close it or do you see it as something useful and worth rebasing? | 15:09 |
| mup | PR #8352: wrappers: generate service files with EnsureDirState [WIP] <Created by zyga> <https://github.com/snapcore/snapd/pull/8352> | 15:09 |
| * zyga EODs and goes to the doctor | 15:15 | |
| mup | PR snapd#8352 closed: wrappers: generate service files with EnsureDirState [WIP] <Created by zyga> <Closed by pedronis> <https://github.com/snapcore/snapd/pull/8352> | 15:24 |
| === ErichEickmeyer is now known as Eickmeyer | ||
| * cachio lunch | 15:52 | |
| === verterok` is now known as verterok | ||
| === AdmWiggin is now known as tianon | ||
| zyga | back from doc | 16:45 |
| zyga | but afk due to pain | 16:45 |
| ogra | un 04 17:00:34 pi4 kernel: audit: type=1400 audit(1591290034.897:477😞 apparmor="DENIED" operation="capable" profile="/snap/snapd/7779/usr/lib/snapd/snap-confine" pid=4626 comm="snap-confine" capability=4 capname="fsetid" | 17:02 |
| ogra | hmm | 17:02 |
| ogra | thats new on my pi4 ... (freshly booted) | 17:02 |
| jdstrand | ogra: it isn't a new thing. it was reintroduced with snap-confine refactor | 17:23 |
| jdstrand | ogra: it is harmless but on my list to investigate | 17:23 |
| ogra | ok, i just had never noticed it ... and it shows up along with only two (expected) app denials ... that got my attention | 17:24 |
| jdstrand | well, it might be new in 2.45, but I've been seeing it for a while (we used to have it and I fixed it, but then some snap-confine changes (ie, the removal of setgid stuff) added it back | 17:24 |
| jdstrand | ogra: yes, thanks for bringing it up :) | 17:24 |
| ogra | the magic word was "harmless" 😉 | 17:24 |
| ogra | i'll appily ignore it | 17:25 |
| diddledan | "mostly harmless" | 17:30 |
| mup | PR snapd#8815 opened: tests: port snap-handle-link test to tests.session <Test Robustness> <Created by zyga> <https://github.com/snapcore/snapd/pull/8815> | 18:49 |
| === Ringtailed-Fox is now known as RingtailedFox | ||
| mup | PR snapd#8816 opened: tests: port 2 uc20 part1 <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8816> | 20:35 |
| === KindTwo is now known as KindOne | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!