mwhudsonbah snapcraft just said this to me:00:15
mwhudsonSorry, an error occurred in Snapcraft:00:15
mwhudson'ReviewInProgress' is not one of ['Published', 'Unpublished', 'ManualReviewPending', 'NeedsInformation', 'AutomaticallyRejected', 'Rejected']00:15
mwhudsondoing list-revisions00:15
zygagood morning, hey mborzecki07:03
mborzeckizyga: hey07:03
mborzeckiquick errand, back in 3007:53
zygahey pstolowski08:03
zygahey mvo, good morning08:08
mvogood morning zyga !08:17
mvozyga: quick question - you had a pi and an sd card that was affected by this vfat write bug iirc. we have a fix in the pi gadget in edge, would be be easy for you to test? if not that would be lovely. otherwise I will test but I need to find an sd card first that I can reproudce the original issue with :(08:18
zygaxnox pinged me on Friday08:19
zygayeah, I'm going to be working on a pi today08:19
zygado you think I can test this on core or do I need cassic08:19
mborzeckipstolowski: mvo: hey08:24
mborzeckimvo: can you take a look at https://github.com/snapcore/snapd/pull/9751 ?08:25
mupPR #9751: cmd/snap-update-ns: fix sorting of overname mount entries wrt other entries <Squash-merge> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9751>08:25
mvozyga: core is fine (sorry for the delay)08:38
mvomborzecki: sure08:39
zygamvo, thanks!08:41
mupPR snapd#9754 closed: o/devicestate: save model with serial in the device save db (2.48) <Created by pedronis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9754>08:50
mupPR snapd#9751 closed: cmd/snap-update-ns: fix sorting of overname mount entries wrt other entries <Squash-merge> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9751>08:55
mupPR snapd#9750 closed: tests: fix the scenario when the "$SRC".orig file does not exist <Simple 😃> <Created by sergiocazzolato> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/9750>09:10
mupPR snapd#9749 closed: tests: fix lp-1899664 test when snapd_x1 is not installed in the system <Simple 😃> <Created by sergiocazzolato> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/9749>09:15
zygaok, morning meetings over, I'll work on some packaging and then switch gears to pi09:32
xnoxzyga:  we have new uboot in both core (edge) and in classic (enable & upgrade to proposed) so whichever.09:38
zygaxnox yeah, that's best, I have a core installation and that will be easier for me to work with09:39
xnoxzyga:  we don't bump the edition in the gadget.... so you might need to copy things out by hand from the new gadget =) or like copy things from the edge channel build of the image.09:39
zygathat's okay09:40
zygaxnox do you plan to bump it?09:40
xnoxnot until we have dtbs sorted.09:41
zygaxnox that's smart, I was about to ask09:41
zygaright now old installs are hosed by both bugs09:41
* mvo hugs pstolowski for his review of 973809:57
pstolowskimvo: yw, i hope it makes sense09:59
mvopstolowski: I think so, thanks!09:59
mupPR snapd#9738 closed: secboot: use `fde-reveal-key` if available to unseal key <Squash-merge> <UC20> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9738>10:15
mupPR snapd#9757 opened: tests: move snapstate config defaults tests to a separate file <Created by stolowski> <https://github.com/snapcore/snapd/pull/9757>10:15
mupPR snapd#9758 opened: secboot: add new LockSealedKeys() that uses either TPM/fde-reveal-key <Squash-merge> <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9758>10:20
mupPR snapd#9759 opened: cmd/snap-update-ns: add better unit test for overname sorting <Simple 😃> <Skip spread> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/9759>10:35
ograerror: error running snapctl: snap "scummvm" has "install-snap" change in progress11:08
ograis that known ? (the install hook runs "snapctl", it works fine on a 20.10 install but not on a PiOS install (same snapd version)11:09
ogra(see https://forum.snapcraft.io/t/scummvm-snap-failing-to-install-on-rpi-4/21394 for details)11:09
ograalan_g, ^^^11:10
mupPR snapd#9759 closed: cmd/snap-update-ns: add better unit test for overname sorting <Simple 😃> <Skip spread> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/9759>11:16
mupPR snapd#9748 closed: gadget: start separating rule/convention validation from basic soundness  <Cleanup :broom:> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9748>11:21
pedronispstolowski: seems preseed reset needs tweaks maybe, it failed here: https://github.com/snapcore/snapd/pull/9755/checks?check_run_id=151003148711:21
mupPR #9755: daemon: split aliases support to its own api_*.go files <Created by pedronis> <https://github.com/snapcore/snapd/pull/9755>11:22
pstolowskipedronis: looking11:22
pedronispstolowski: might also be not preseed that needs tweaks but the inhbit lock logic vs preseed itself11:23
pedronis*not preseed reset11:23
* pstolowski lunch11:47
=== ddstreet_away is now known as ddstreet
pedronispstolowski: I did a pass on #9732 as well12:57
mupPR #9732: asserts: snapasserts method to validate installed snaps against validation sets <validation-sets :white_check_mark:> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9732>12:57
=== hggdh_ is now known as hggdh
mupPR snapd#9760 opened: cmd/snap-preseed: reset run inhibit locks on --reset <Created by stolowski> <https://github.com/snapcore/snapd/pull/9760>14:01
pedronispstolowski: btw I left some comments in https://github.com/snapcore/snapd/pull/9513 let me know if you have questions14:31
mupPR #9513: snapshotstate: detect duplicated snapshot imports <Needs Samuele review> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9513>14:31
mupPR snapd#9755 closed: daemon: split aliases support to its own api_*.go files <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9755>15:27
pstolowskipedronis: re your validation sets comment, for snaps at wrong revisions the error would show the required revision coming from assertion, wdyt?15:41
pedronispstolowski: that seems a good step15:41
pedronispstolowski: to be clear if the validation sets itself is not in conflict, there should be exactly one value15:42
pstolowskipedronis: good point15:43
* zyga-mbp fiddles with pi which no longer prints anything on serial15:54
pedronispstolowski: https://github.com/snapcore/snapd/pull/9761 is the PR I mentioned in the standup15:54
mupPR #9761: daemon: actually move APIBaseSuite to daemon_test.apiBaseSuite <Created by pedronis> <https://github.com/snapcore/snapd/pull/9761>15:55
mupPR snapd#9761 opened: daemon: actually move APIBaseSuite to daemon_test.apiBaseSuite <Created by pedronis> <https://github.com/snapcore/snapd/pull/9761>15:57
zyga-mbptook of the hat, fixed the problem15:59
pstolowskipedronis: ack, ty16:00
pstolowskipedronis: i updated validation-sets PR16:06
pedronismvo: I made a high-level comment here https://github.com/snapcore/snapd/pull/9149/files#r537628269 let me know if you want to chat about it16:09
mupPR #9149: gadget: provide new gadget.ResolveContentPaths() (2/N) <Needs Samuele review> <Squash-merge> <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9149>16:09
pedronispstolowski: thx16:10
ijohnsonmvo: pedronis: should I include mvo in our sync tomorrow to talk about next things for me? I think so, but the calendar is rather tight tomorrow for both of you together16:14
pedronisijohnson: I suppose after standup should work? usually standup is shortish these days16:15
ijohnsonok that sounds good then16:15
ijohnsonsent for half past the hour16:16
zyga-mbpxnox verifying now16:23
zyga-mbpxnox I even found the card16:23
zyga-mbpmvo if you are interested it is a 8GB samsung evo16:24
zyga-mbpprobably bottom barrel card16:24
mvozyga-mbp: great, thank you!16:24
zyga-mbpgrabbing new pi snap16:26
pedronispstolowski: I did a pass on #9429 too16:26
mupPR #9429: o/daemon: validation sets api and basic spread test <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <https://github.com/snapcore/snapd/pull/9429>16:27
pstolowskipedronis: ty. i reviewed 976116:29
zyga-mbpwhat a day16:41
zyga-mbpsna zyga@pi3-2:~$ snap16:41
zyga-mbp[  162.936252] SQUASHFS error: xz decompression failed, data probably corrupt16:41
zyga-mbp[  162.943214] SQUASHFS error: squashfs_read_data failed to read block 0x576e0716:41
zyga-mbp[  162.967676] SQUASHFS error: xz decompression failed, data probably corrupt16:41
zyga-mbp[  162.974650] SQUASHFS error: squashfs_read_data failed to read block 0x576e0716:41
zyga-mbp[  162.998901] SQUASHFS error: xz decompression failed, data probably corrupt16:41
zyga-mbp[  163.005899] SQUASHFS error: squashfs_read_data failed to read block 0x576e0716:41
zyga-mbpfatal error: unexpected signal during runtime execution16:41
zyga-mbp[signal SIGBUS: bus error code=0x2 addr=0xa5396c pc=0x5d4dc]16:41
pstolowskioh sdcard failed?16:42
zyga-mbppstolowski not sure yet, maybe16:43
mvopedronis: I updated 9741, hope I did not miss anything :)16:47
pedronispstolowski: I re-reviewed 9732, some ideas there16:48
xnoxzyga-mbp:  is that mbp M1 ?!16:48
pstolowskixnox: not *yet* i think ;)16:50
zyga-mbpxnox nah, my "old" 9th gen 16"16:50
zyga-mbpvintage intel ;)16:50
zyga-mbpxnox if you want m1 status ask kissiel16:55
zyga-mbpxnox m1 is oOOh fast in his words, x2 speed of 4800u ryzen thinkpad t1416:55
mupPR snapd#9752 closed: gadget,o/devicestate: set implicit values for schema and role directly instead of relying on Effective* accessors <Cleanup :broom:> <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9752>17:12
=== ijohnson is now known as ijohnson|lunch
mupPR snapd#9762 opened: gadget: prepare gadget kernel refs (0/N) <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/9762>19:18
=== ijohnson|lunch is now known as ijohnson
stgraberroot@snap-ubuntu-core-20:~# snap refresh20:25
stgrabererror: cannot perform the following tasks:20:25
stgraber- Download snap "pc" (115) from channel "20/stable" (cannot read device key pair: cannot find key pair)20:25
stgraberhow do I fix that? :)20:26
stgraberit's breaking LXD's CI on Ubuntu Core 2020:26
stgraberijohnson: looks like you saw a similar report on the forum a few weeks back?20:27
ijohnsonstgraber: this is from old uc20 images20:27
ijohnsonyou need to reinstall that uc20 image with newer snaps in the image20:27
ijohnsons/uc20 image/uc20 system/20:27
stgraberwell, that's gonna be fun20:28
ijohnsonit's a consequence of the changes we needed to make for uc20 fde support, we needed to move where keys are located and this breaks old systems unfortunately20:28
ijohnsonI thought that images should have been respun already if you pull from cdimage automatically or something20:28
ijohnsonbut I haven't checked cdimages in a while20:29
stgraberit's a persistent test system20:29
stgraberhence the "fun"20:29
stgrabersince I need to figure out how it was built and then go to create it, update DNS records, re-setup snap proxy and the like20:29
stgraberanyway, it as pre-release so I can't complain too much other than make a mental note not to setup CI for Core 22 until it's fully released ;)20:30
ijohnsonyeah it's definitely unfortunate we didn't intend to break unencrypted systems like this but we just didn't have the time to try and implement support for both key locations20:32
stgraberijohnson: the images your doc points to date from May, so presumably will hit the exact same issue?20:36
ijohnsonstgraber: good point we need to update that , but try http://cdimage.ubuntu.com/ubuntu-core/20/candidate/pending/ instead20:36
ijohnsoncandidate > beta :-)20:36
mupPR snapd#9761 closed: daemon: actually move APIBaseSuite to daemon_test.apiBaseSuite <Created by pedronis> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/9761>22:04
mupPR snapd#9763 opened: daemon: split find support to its own api_*.go files and move some helpers <Cleanup :broom:> <Created by pedronis> <https://github.com/snapcore/snapd/pull/9763>22:09
mupPR snapd#9764 opened: interfaces/builtin/docker-support: allow /run/containerd/s/ <Needs security review> <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/9764>22:39

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