Tuesday, 2022-04-19

@fzzfh:matrix.org * hi,when I use nodepool DIB build ubuntu-focal image occur error.https://bugs.launchpad.net/diskimage-builder/+bug/196909301:44
```
2022-04-13 08:42:56.123 | INFO diskimage_builder.block_device.utils [-] Calling [sudo umount /var/cache/nodepool/dib_tmp/dib_build.3DMuQbbI/mnt/]
2022-04-13 08:42:56.675 | INFO diskimage_builder.block_device.utils [-] Calling [sudo kpartx -d /dev/loop0]
2022-04-13 08:42:56.715 | DEBUG diskimage_builder.block_device.utils [-] exec_sudo: device-mapper: remove ioctl on loop0p1 failed: Device or resource busy exec_sudo /usr/lib/python3.6/site-packages/diskimage_builder/block_device/utils.py:135
```
I found after build. df -h still have /dev/mapper/loop0p1 like follow. if I run unmount /run/media/root/cloudimg-rootfs,then `kpartx -d /dev/loop0` would be success. Does this need to be unmount. but I don't found unmount /root/media/root/clouding-rootfs on elements script.
```
[root@managesf.sftests.com /]# df -h
Filesystem Size Used Avail Use% Mounted on
devtmpfs 16G 0 16G 0% /dev
tmpfs 16G 20K 16G 1% /dev/shm
tmpfs 16G 19M 16G 1% /run
tmpfs 16G 0 16G 0% /sys/fs/cgroup
/dev/mapper/centos-root 863G 33G 831G 4% /
/dev/sda2 1014M 154M 861M 16% /boot
/dev/sda1 200M 12M 189M 6% /boot/efi
/dev/mapper/centos-home 50G 58M 50G 1% /home
tmpfs 3.1G 52K 3.1G 1% /run/user/0
/dev/mapper/loop0p1 3.8G 2.0G 1.6G 57% /run/media/root/cloudimg-rootfs
```
-@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 837506: Don't overwrite result data in compat mode https://review.opendev.org/c/zuul/zuul/+/83750606:17
-@gerrit:opendev.org- Alfredo Moralejo proposed: [zuul/zuul-jobs] 838450: Update gpg key file for extras-common in CS9 https://review.opendev.org/c/zuul/zuul-jobs/+/83845008:19
-@gerrit:opendev.org- Zuul merged on behalf of Albin Vass: [zuul/nodepool] 823978: Add option of configuring imagePullSecrets for openshift drivers https://review.opendev.org/c/zuul/nodepool/+/82397808:59
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 837766: Fix git error in stream jobs https://review.opendev.org/c/zuul/zuul/+/83776617:07
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 838561: Make nodepool hostvars available on unreachable hosts https://review.opendev.org/c/zuul/zuul/+/83856117:19
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 837757: Add IBM and Azure deps to the default ansible config https://review.opendev.org/c/zuul/zuul/+/83775717:22
@clarkb:matrix.orgcorvus: in https://review.opendev.org/c/zuul/zuul/+/837770/2/zuul/merger/merger.py you rmtree the repo, but will the next operation implicitly reclone?21:20
@clarkb:matrix.orgor will we still fail the operation because the next operation won't have a git repo?21:21
@jim:acmegating.comClark: that build will be toast, but the next one will reclone21:28
@jim:acmegating.com(that's generally the pattern we have for the "OMG git repo makes no sense" errors)21:28
@clarkb:matrix.orgok that is sort of what I assumed. Should we reclone there and attempt to not fail?21:29
@jim:acmegating.comthese are super rare, so i'd rather keep it simple+safe21:29
@clarkb:matrix.orgwfm21:29
@jim:acmegating.com(i mean, i'm not necessarily opposed to that, but at least at the moment, i think the trade-off isn't worth it)21:30
@clarkb:matrix.orgcorvus: left a comment on https://review.opendev.org/c/zuul/zuul/+/83743621:33
@clarkb:matrix.orgmostly worried we're ripping out too much of the testing framework when testing very similar things related to bwrap instead of untrusted vs trusted paths would be useful21:34
@jim:acmegating.comClark: if we want to do that, i think that would be a new test.  i don't think the "remote" tests are the right place for that, since that's testing the action module behavior on remote nodes.  i can add that as a followup commit if we think it's important.21:40
@clarkb:matrix.orgoh ya I don't think we need remote nodes to check that. I do think it would be worth testing just to make sure our assumptions about bwrap behavior hold true (and continue to do so over time)21:41
@iwienand:matrix.orghttps://review.opendev.org/c/zuul/zuul/+/837459 is a hopefully easy one that uncaps sphinx.  it would be helpful as the current capped version doesn't work with 3.10.  it required a cap because it didn't work with python 3.8, but fixes are released for that now21:52
@iwienand:matrix.orgs/with 3.10/with python 3.10/21:52
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 838602: Add a bubblewrap ro mount test https://review.opendev.org/c/zuul/zuul/+/83860222:08
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 838603: Remove unused bwrap-mounts test fixtures https://review.opendev.org/c/zuul/zuul/+/83860322:09
@jim:acmegating.comClark: ^ and also v22:09
@clarkb:matrix.orgthanks I'll try to catch up on reviews after this short indoor bike ride22:17
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 837770: Delete repo if unable to reset https://review.opendev.org/c/zuul/zuul/+/83777022:45
-@gerrit:opendev.org- Zuul merged on behalf of Ian Wienand: [zuul/zuul] 837459: doc: uncap sphinx https://review.opendev.org/c/zuul/zuul/+/83745923:08

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!