/srv/irclogs.ubuntu.com/2021/09/06/#snappy.txt

mborzeckimorning05:59
mardymborzecki: 'morning!06:01
mborzeckimardy: heya06:02
pstolowskimorning06:27
mborzeckipstolowski: hey06:27
mardypstolowski: hi!06:27
mborzeckipstolowski: another school day and I already hate it ;)06:28
pstolowskimborzecki: haha06:33
* mardy going to be afk for 45 mins or so06:41
zyga-mbpgood morning guys!06:43
zyga-mbpmborzecki oh, what's going on?06:43
mupPR snapd#10730 closed: strutil: add Intersection() <Simple 😃> <UC20> <Skip spread> <Created by anonymouse64> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/10730>06:49
mupPR snapd#10722 closed: tests/nested/manual/refresh-revert-fundamentals: fix variable use <Simple 😃> <Run nested> <Created by bboozzoo> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10722>07:14
mupPR snapd#10243 closed: tests: failure of prereqs on content interface doesn't prevent install <Skip spread> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10243>07:44
sil2100mvo, pstolowski: hey guys! Is there any progress in the snapd riscv64 build failure on focal?07:47
pstolowskisil2100: hi, there was one bump-timeout tweak (i think you saw that PR last week). the 2nd issue of test exceeding 10m (causing panic from go test) is unclear, is that one reproducible?07:50
mvosil2100, pstolowski we could as a temporary workaround simply disable unit tests on riscv for this one upload08:06
sil2100mvo, pstolowski: yeah, or like skipping the ones that time-out/flaky-fail for one upload - since I just wouldn't want to suddenly drop this arch08:10
mborzeckihm snap keys suite keeps failing on obs due to gpg, we also see it occasionally on github08:15
pstolowskisil2100: i can prepare a PR to skip them, let's hope it doesn't fail on another test after that ;)08:16
pstolowskisil2100, mvo : https://github.com/snapcore/snapd/pull/10736 should do it08:37
mupPR #10736: tests: skip overlord tests on riscv64 due to timeouts <Created by stolowski> <https://github.com/snapcore/snapd/pull/10736>08:37
mvopstolowski: nice08:38
mupPR snapd#10736 opened: tests: skip overlord tests on riscv64 due to timeouts <Created by stolowski> <https://github.com/snapcore/snapd/pull/10736>08:39
mborzeckizyga-mbp: trivial update https://build.opensuse.org/request/show/91701208:52
mborzeckimvo: hi, do you think you could take a look at https://github.com/snapcore/snapd/pull/10661 ?09:03
mupPR #10661: cmd/libsnap-confine-private: device cgroup v2 support <Complex> <Security-High> <Needs security review> <cgroupv2> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10661>09:03
zyga-mbpmborzecki looking09:03
zyga-mbpmborzecki done09:04
zyga-mbpmborzecki I failed to do the review over weekend, a bit too much stuff with the kitchen09:04
zyga-mbpmborzecki I can look in the evening today09:04
mborzeckizyga-mbp: that's fine, youre under no obligation to look at it 😉 it's already very courteous of you to take a look at some of the PRs anyway09:07
zyga-mbpmborzecki that's one very close to heart and I really want to dig into iot09:07
zyga-mbpinto it*09:07
mborzeckizyga-mbp: i guess iot is close to your heart too ;)09:11
zyga-mbphehe09:11
zyga-mbpyes09:11
mupPR snapd#10737 opened: o/snapstate: validation sets enforcing on update <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10737>10:14
mborzeckicrappy fault injection in snapd: https://paste.ubuntu.com/p/QBw4wzY9KS/10:57
mborzeckia fun feature, one can inject faults per specific snap name10:58
pstolowskimborzecki: nice! how does it work?11:04
mborzeckipstolowski: you set SNAPPY_TESTING=1 (as done in the tests) and then SNAPD_FAULT_INJECT=<tag>:<kind>[,<tag>:<kind>]11:05
mborzeckikind is 'panic' (which calls panic()) or 'reboot' which does echo b > /proc/sysrq-trigger11:05
zyga-mbpmborzecki nice11:06
zyga-mbpmborzecki how about error?11:06
mborzeckitag is free form, in the code you only place a call osutil.MaybeInjectFault("link-snap")11:06
zyga-mbpthat might help with testing undo11:06
mborzeckihm worth trying, tbh only have panic and reboot right now, that's for the extreme scenarios11:06
zyga-mbpit might replace the special hacks done in unit tests as well11:07
mborzeckiand right before a fault is injected, thre's a stamp file left under /var/lib/snapd/fault/<tag>:<kind>11:07
zyga-mbpwith snap names that encode failure11:07
mborzeckizyga-mbp: yeah, the tag is free form text so you can add osutil.MaybeInjectFault(fmt.Sprintf("link-snap@%s", snapName)) and then SNAPD_FAULT_INJECT=link-snap@hello-world:reboot :P11:09
pstolowskimborzecki: wdyt about https://github.com/snapcore/snapd/pull/10736#discussion_r702724230 ? i'm mostly thinking about making it as easy as possible for the builders right now, so sil2100 doesn't have to introduce flags elsewhere12:17
mupPR #10736: tests: skip overlord tests on riscv64 due to timeouts <Simple 😃> <Skip spread> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10736>12:17
mupPR snapd#10738 opened: [RFC] o/hookstate/ctlcmd: Implement snapctl refresh --show-inhibit-lock command <Needs Samuele review> <Refresh control> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10738>12:20
mupPR snapd#10739 opened: mount-control: step 2 <Created by mardy> <https://github.com/snapcore/snapd/pull/10739>12:25
mupPR snapd#10740 opened: osutil: helper for injecting run time faults in snapd  <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10740>12:35
mvomborzecki: I need to look at your cgroups v2 yes, will see if I can do it this afternoon, I spend a bit of time this morning to get netplan ready :/12:50
miguelpiresmvo, can you merge https://github.com/snapcore/snapd/pull/10701 please?13:49
mupPR #10701: many: remove unused parameters <Needs Samuele review> <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10701>13:49
mvomiguelpires: sure13:55
mupPR snapd#10701 closed: many: remove unused parameters <Needs Samuele review> <Created by MiguelPires> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10701>14:00
mupPR snapd#10741 opened: interfaces/block-devices: support to access the state of block devices <Created by woodrow-shen> <https://github.com/snapcore/snapd/pull/10741>16:15
mupPR snapd#10728 closed: ifacestate: undo repository connection if doConnect fails <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10728>18:36
mupPR snapd#10742 opened: tests: update nested tool - part1 <Run nested> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10742>19:26
mupPR snapd#10743 opened: tests: fix fakedevicesvc service already exists <Run nested> <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10743>21:46
hiparcokdenlive snap is unusable on my machine once I can't really render any video22:38
hiparcorunning with snap run kdenlive, it states a missing library22:40

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