Thursday, 2022-04-14

« Wednesday, 2022-04-13 Index Friday, 2022-04-15 »
yoctozeptohrw: who knows07:00
yoctozepto:D07:00
mnasiadkamath is hard07:03
mnasiadkakayobe rc1 out07:03
opendevreviewOpenStack Release Bot proposed openstack/kayobe stable/yoga: Update .gitreview for stable/yoga  https://review.opendev.org/c/openstack/kayobe/+/83782707:04
opendevreviewOpenStack Release Bot proposed openstack/kayobe master: Update master for stable/yoga  https://review.opendev.org/c/openstack/kayobe/+/83782807:05
opendevreviewOpenStack Release Bot proposed openstack/kayobe master: Add Python3 zed unit tests  https://review.opendev.org/c/openstack/kayobe/+/83782907:05
opendevreviewOpenStack Release Bot proposed openstack/kayobe-config stable/yoga: Update .gitreview for stable/yoga  https://review.opendev.org/c/openstack/kayobe-config/+/83783007:05
opendevreviewOpenStack Release Bot proposed openstack/kayobe-config stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga  https://review.opendev.org/c/openstack/kayobe-config/+/83783107:05
opendevreviewOpenStack Release Bot proposed openstack/kayobe-config-dev stable/yoga: Update .gitreview for stable/yoga  https://review.opendev.org/c/openstack/kayobe-config-dev/+/83783207:05
opendevreviewOpenStack Release Bot proposed openstack/kayobe-config-dev stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga  https://review.opendev.org/c/openstack/kayobe-config-dev/+/83783307:05
opendevreviewMichal Nasiadka proposed openstack/kolla stable/yoga: [release] centos: switch to CentOS cloud SIG packages  https://review.opendev.org/c/openstack/kolla/+/83730007:07
mnasiadkaso, what needs to be backported to stable/yoga - and do we have any breakages?07:09
opendevreviewMark Goddard proposed openstack/kayobe stable/yoga: ironic: default to ipxe booting  https://review.opendev.org/c/openstack/kayobe/+/83779307:49
hrwgit issue in ubuntu07:51
opendevreviewMerged openstack/kolla-ansible master: ovs-dpdk: add ovs-dpdkctl.sh to the role itself  https://review.opendev.org/c/openstack/kolla-ansible/+/83770308:02
opendevreviewMerged openstack/kolla-ansible master: libvirt: Fix nova-libvirt-cleanup command  https://review.opendev.org/c/openstack/kolla-ansible/+/83582008:03
opendevreviewMark Goddard proposed openstack/kayobe master: CI: Add some more irrelevant files to zuul jobs  https://review.opendev.org/c/openstack/kayobe/+/83783608:04
opendevreviewMerged openstack/kolla-ansible stable/wallaby: [CI] Use Tenks in Ironic job  https://review.opendev.org/c/openstack/kolla-ansible/+/83282308:05
opendevreviewMerged openstack/kayobe master: Update master for stable/yoga  https://review.opendev.org/c/openstack/kayobe/+/83782808:07
opendevreviewMerged openstack/kayobe master: Add Python3 zed unit tests  https://review.opendev.org/c/openstack/kayobe/+/83782908:07
opendevreviewMerged openstack/kayobe-config stable/yoga: Update .gitreview for stable/yoga  https://review.opendev.org/c/openstack/kayobe-config/+/83783008:07
opendevreviewMark Goddard proposed openstack/kayobe master: Revert "[release] Use OpenStack Yoga release"  https://review.opendev.org/c/openstack/kayobe/+/83779408:09
opendevreviewMerged openstack/kayobe-config stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga  https://review.opendev.org/c/openstack/kayobe-config/+/83783108:10
yoctozeptomnasiadka: https://review.opendev.org/c/openstack/kolla-ansible/+/83451208:10
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/yoga: libvirt: Fix nova-libvirt-cleanup command  https://review.opendev.org/c/openstack/kolla-ansible/+/83779508:20
hrwyoctozepto: +2+W08:22
hrwso what we do with https://review.opendev.org/c/openstack/kolla/+/837710 one? (git fix in gnocchi)08:22
yoctozeptohrw: thx08:22
hrwyoctozepto: I waited for our cloud team08:22
yoctozeptohrw: regarding gnocchi - I think we are forced to do it now for all places unless git reverts this wondrous fix08:23
yoctozeptootherwise git references will be broken08:24
hrwyoctozepto: filename=nova-25.0.0.0rc2.dev2-py3-none-any.whl so it looks like it grabs versions properly for tarballed ones08:43
hrwso only git ones are problem where (in default setup) it is only gnocchi and even that one we can fetch from pypi08:44
hrwok team - what do we do then? Cause for now our CI is RED and will stay that way. We just get more red entries as distros update their git packages.09:12
opendevreviewMerged openstack/kolla-ansible master: CI: cephadm - stop copying keyrings  https://review.opendev.org/c/openstack/kolla-ansible/+/83043409:15
opendevreviewMerged openstack/kolla-ansible master: Deprecate enable_ironic_pxe_uefi  https://review.opendev.org/c/openstack/kolla-ansible/+/83451209:17
opendevreviewChristian Berendt proposed openstack/kolla-ansible stable/xena: ovs-dpdk: add ovs-dpdkctl.sh to the role itself  https://review.opendev.org/c/openstack/kolla-ansible/+/83779809:31
opendevreviewChristian Berendt proposed openstack/kolla-ansible stable/yoga: ovs-dpdk: add ovs-dpdkctl.sh to the role itself  https://review.opendev.org/c/openstack/kolla-ansible/+/83779909:31
mgoddardyoctozepto, hrw: needs a bug report, since it's user facing09:45
mgoddardI don't understand the ownership issue. Isn't it run as root, and owned by root, same as parent directory?09:45
hrwmgoddard: it is run by root, owned by host user09:49
mgoddardhrw: oh09:49
hrwyep09:49
mgoddardso the ownership depends on who builds the image?09:49
hrwI added 'ls -l' into image and got 1003:100609:49
mgoddardok09:49
hrwmgoddard: once it is installed we do not have to care as pip writes files as root09:50
mgoddardThen I think we need a macro that does ADD --chown root:root09:51
hrwADD --chown is bullshit09:51
hrwgot 1003:1006 with "ADD --chown=0:0"09:51
hrwmgoddard: https://review.opendev.org/c/openstack/kolla/+/83771009:52
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/yoga: Deprecate enable_ironic_pxe_uefi  https://review.opendev.org/c/openstack/kolla-ansible/+/83780009:53
mgoddardhrw: https://paste.opendev.org/show/bxoFA86ov87jBPaRZiy9/09:54
mgoddardhrw: I get root:root always...09:55
hrw20:45 < hrw> INFO:kolla.common.utils.gnocchi-base:-rw-rw-r--. 1 1003 1006  72 Apr 13 18:45 deprecate-moving-average-a7596a0009be5b12.yaml09:55
hrw20:45 < hrw> INFO:kolla.common.utils.gnocchi-base:-rw-rw-r--. 1 1003 1006 117 Apr 13 18:45 deprecate-noauth-01b7e961d9a17e9e.yaml09:55
yoctozeptomgoddard: try with directories09:55
mgoddardk09:55
hrwdocker--09:56
yoctozeptoyup09:56
mgoddardyoctozepto: same result09:57
yoctozeptosame good or same wrong?09:57
mgoddardDocker version 19.03.9, build 9d988398e709:57
yoctozeptohmm09:57
yoctozeptomaybe they broke later09:57
mgoddardcopied directories and files owned by root09:57
yoctozeptowell, we have 20.10 something in here09:58
yoctozeptoeh docker09:58
mgoddardhttps://paste.opendev.org/show/bLzrQx1XA7ChBQhcv4xm/09:58
mgoddardmkdir foo; echo bar > foo/bar; docker build .09:59
mgoddardsee if you can reproduce09:59
hrwthis one has root:root10:00
yoctozeptoI confirm, your reproducer is also root:root10:00
yoctozeptohmm, whatever happens in kolla then10:01
yoctozeptoVersion:          20.10.5+dfsg110:01
yoctozeptobtw10:01
hrw12:01 (s) marcin@puchatek:~$ docker --version10:01
hrwDocker version 20.10.14, build a22408610:01
mgoddard"All new files and directories are created with a UID and GID of 0, unless the optional --chown flag specifies a given username, groupname, or UID/GID combination to request specific ownership of the content added"10:01
mgoddardhttps://docs.docker.com/engine/reference/builder/#add10:01
hrwyeah, read that already10:02
yoctozeptook, so what kolla does differently now10:02
mgoddardtarball10:02
yoctozeptowell, it's from git, the tarballs are happy10:02
mgoddardI think we clone and build a tarball?10:03
yoctozeptoah, the way *we* build a tarball is wrong then10:03
hrwhttps://paste.centos.org/view/7e52b408 - apply and check output?10:03
hrwwe clone. we do not tarball10:03
yoctozeptoyeah, we clone and archive it10:04
mgoddardreproduced the issue with a tarball10:04
yoctozeptoyeah, and usual tarballs simply work because they include the metadata already, with no git to break on10:05
mgoddardhttps://paste.opendev.org/show/bgaTXG0RA9P9L0xQ3rJ0/10:05
yoctozeptowhat we could do is to produce an sdist from the git repo10:05
yoctozeptoand use that sdist instead of our archive10:05
yoctozeptothat would include metadata and avoid any issues10:05
yoctozeptowdyt?10:06
yoctozeptothis way we also don't need git in the images10:06
hrwok, I have other idea. checking...10:08
mgoddardhttps://docs.python.org/3/library/tarfile.html10:08
mgoddard"How to create an archive and reset the user information using the filter parameter in TarFile.add():"10:08
mgoddardlemme try10:08
hrwhttps://paste.centos.org/view/cc5c8c86 solved problem10:09
hrwmgoddard: that's what I did10:09
yoctozeptoyay10:09
yoctozepto++ you both10:09
mgoddardgood job hrw10:09
yoctozeptoteam work, dream work10:09
mgoddardyou want to propose hrw?10:10
hrwyes10:10
mgoddardgreat10:10
mgoddardbug report & reno would be nice too :)10:10
yoctozeptoawesome10:10
hrwmgoddard: go with bug while I am finishing commit and reno10:12
mgoddardugh words10:13
hrwmgoddard: https://bugs.launchpad.net/pbr/+bug/1968877 is where you want to add Kolla ;D10:14
mgoddardI think we'll be crowding that one10:15
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: Fix git fetches (due recent git security fix)  https://review.opendev.org/c/openstack/kolla/+/83771010:17
hrw3rd revision, 3rd solution10:17
hrwreno may need work10:18
hrwI wanted to write that it needs polishing but we already have too many Polish people so it could be too Polish English at the end ;d10:19
mgoddardhrw: https://bugs.launchpad.net/kolla/+bug/196909610:21
mgoddardhrw: it needs unpolishing :)10:21
hrwmgoddard: as British English speaker you are perfect person to amend reno. Can add bug number at same time.10:22
mgoddardI'll polish it :p10:22
hrwbtw - English grammar joke from my daughter: What gift you give to English person when you want to give something great? Present Perfect!10:23
hrw(there is similar one about easy gifts)10:24
opendevreviewMark Goddard proposed openstack/kolla master: Fix image builds with sources using a type=git  https://review.opendev.org/c/openstack/kolla/+/83771010:33
mgoddardhrw: most native English speakers would not even understand it :)10:33
hrwmgoddard: I know. worked with natives in past10:39
hrwmy English grammar sucks but was good enough to teach them grammar10:39
hrw"no, 'a/an/the' are not to be used randomly"10:39
mgoddardwhom10:40
hrwmgoddard: nice reno/commitmsg10:41
mgoddardhrw: thanks10:41
hrwnice that final fix turned out to be so simple10:41
yoctozeptomgoddard: I'm more inclined to say hrw taught his children rather than natives10:47
yoctozeptobut who knows :-)10:47
hrwor that my translation of joke lost too much10:48
hrwyoctozepto: s/children/child/ 10:49
yoctozeptohrw: a girl if I recall properly10:50
hrwyes10:50
yoctozepto:-)10:50
opendevreviewKyle Dean proposed openstack/kolla-ansible master: Switch prometheus to active/passive mode  https://review.opendev.org/c/openstack/kolla-ansible/+/79093611:15
opendevreviewMerged openstack/kolla-ansible stable/yoga: libvirt: Fix nova-libvirt-cleanup command  https://review.opendev.org/c/openstack/kolla-ansible/+/83779512:27
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Multiple DHCP ranges for Ironic Inspector  https://review.opendev.org/c/openstack/kayobe/+/82765312:42
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Multiple DHCP ranges for Ironic Inspector  https://review.opendev.org/c/openstack/kayobe/+/82765312:47
hrwI got something to improve zuul on kolla jobs13:07
hrwbbl13:07
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: enable logging to file for quiet mode  https://review.opendev.org/c/openstack/kolla/+/83787213:36
opendevreviewMerged openstack/kayobe stable/yoga: ironic: default to ipxe booting  https://review.opendev.org/c/openstack/kayobe/+/83779313:43
opendevreviewMerged openstack/kayobe master: Update documentation link for NCLU  https://review.opendev.org/c/openstack/kayobe/+/83733513:43
opendevreviewMerged openstack/kayobe master: docs: Fix custom LVM example  https://review.opendev.org/c/openstack/kayobe/+/83692213:43
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: enable logging to file for quiet mode  https://review.opendev.org/c/openstack/kolla/+/83787213:46
hrwhttps://zuul.openstack.org/stream/ddd18979b0144b95b49be11a711c99d9?logfile=console.log - zuul in quiet mode13:54
yoctozeptomnasiadka: https://review.opendev.org/c/openstack/kolla/+/837710 gate fix14:01
hrwmnasiadka: we need 837710 in master and everything under. 837872 would be nice to have in supported branches too14:05
hrwonce it shows proper results14:05
mnasiadkaoh boy ;)14:12
hrwand I hope it works as this is something I want for my local builds ;D14:12
hrwmnasiadka: ?14:12
mnasiadkajust laughing, that every day something needs to break14:12
hrwwelcome to CI world ;D14:13
yoctozeptowe are getting better and better14:18
yoctozeptothe fact is these are usually "hidden issues waiting to be discovered"14:18
hrw;D14:18
hrwI learnt new tricks with logging module14:19
opendevreviewKyle Dean proposed openstack/kolla-ansible master: Switch prometheus to active/passive mode  https://review.opendev.org/c/openstack/kolla-ansible/+/79093614:20
hrwI wonder how much usable will be adding (x/y) to quiet output14:21
hrwwhere both numbers may change14:21
opendevreviewKyle Dean proposed openstack/kolla-ansible master: Switch prometheus to active/passive mode  https://review.opendev.org/c/openstack/kolla-ansible/+/79093614:21
hrw"2022-04-14 14:20:52.230256 | primary | {"built": [{"name": "aodh-api"}, " - is anyone using that json drop?14:24
hrwhttps://zuul.opendev.org/t/openstack/build/15f8b2bf461b4703bc63497e683c3a85 - how fast it is?14:30
hrwjob-output.json is 5MB isntead of 29M14:31
opendevreviewMerged openstack/kayobe master: Revert "[release] Use OpenStack Yoga release"  https://review.opendev.org/c/openstack/kayobe/+/83779414:33
opendevreviewKyle Dean proposed openstack/kolla-ansible master: Switch prometheus to active/passive mode  https://review.opendev.org/c/openstack/kolla-ansible/+/79093614:44
opendevreviewMerged openstack/kolla master: Fix image builds with sources using a type=git  https://review.opendev.org/c/openstack/kolla/+/83771015:43
opendevreviewRadosław Piliszek proposed openstack/kolla stable/yoga: Fix image builds with sources using a type=git  https://review.opendev.org/c/openstack/kolla/+/83780615:44
opendevreviewRadosław Piliszek proposed openstack/kolla stable/xena: Fix image builds with sources using a type=git  https://review.opendev.org/c/openstack/kolla/+/83780715:44
opendevreviewRadosław Piliszek proposed openstack/kolla stable/wallaby: Fix image builds with sources using a type=git  https://review.opendev.org/c/openstack/kolla/+/83780915:45
opendevreviewRadosław Piliszek proposed openstack/kolla stable/victoria: Fix image builds with sources using a type=git  https://review.opendev.org/c/openstack/kolla/+/83791015:45
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: enable logging to file for quiet mode  https://review.opendev.org/c/openstack/kolla/+/83787216:25
opendevreviewMark Goddard proposed openstack/kayobe stable/yoga: Update documentation link for NCLU  https://review.opendev.org/c/openstack/kayobe/+/83791116:29
opendevreviewMark Goddard proposed openstack/kayobe stable/xena: Update documentation link for NCLU  https://review.opendev.org/c/openstack/kayobe/+/83791216:29
opendevreviewMark Goddard proposed openstack/kayobe stable/wallaby: Update documentation link for NCLU  https://review.opendev.org/c/openstack/kayobe/+/83791316:29
opendevreviewMark Goddard proposed openstack/kayobe stable/yoga: docs: Fix custom LVM example  https://review.opendev.org/c/openstack/kayobe/+/83791416:29
hrwok, so what opinion do you have on https://review.opendev.org/c/openstack/kolla/+/837872 patch? and it's change to zuul16:29
opendevreviewMark Goddard proposed openstack/kayobe stable/xena: docs: Fix custom LVM example  https://review.opendev.org/c/openstack/kayobe/+/83791516:29
opendevreviewMark Goddard proposed openstack/kayobe stable/wallaby: docs: Fix custom LVM example  https://review.opendev.org/c/openstack/kayobe/+/83791616:30
opendevreviewMark Goddard proposed openstack/kayobe stable/victoria: docs: Fix custom LVM example  https://review.opendev.org/c/openstack/kayobe/+/83791716:30
opendevreviewMerged openstack/kolla-ansible stable/yoga: Deprecate enable_ironic_pxe_uefi  https://review.opendev.org/c/openstack/kolla-ansible/+/83780017:38
opendevreviewMerged openstack/kolla stable/wallaby: Fix image builds with sources using a type=git  https://review.opendev.org/c/openstack/kolla/+/83780918:26
opendevreviewMerged openstack/kolla stable/xena: Fix image builds with sources using a type=git  https://review.opendev.org/c/openstack/kolla/+/83780718:44
opendevreviewMerged openstack/kolla stable/yoga: Fix image builds with sources using a type=git  https://review.opendev.org/c/openstack/kolla/+/83780618:55
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: drop install_type almost everywhere  https://review.opendev.org/c/openstack/kolla/+/83728719:08
opendevreviewMerged openstack/kayobe stable/yoga: Update .gitreview for stable/yoga  https://review.opendev.org/c/openstack/kayobe/+/83782719:57
« Wednesday, 2022-04-13 Index Friday, 2022-04-15 »

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