Thursday, 2024-08-22

opendevreviewVerification of a change to openstack/kolla stable/2024.1 failed: Configure Apache services to use correct locale  https://review.opendev.org/c/openstack/kolla/+/92679501:53
opendevreviewMichal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS  https://review.opendev.org/c/openstack/kolla/+/90758905:46
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: systemd: Add kolla.target  https://review.opendev.org/c/openstack/kolla-ansible/+/92242005:48
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: systemd: Add kolla.target  https://review.opendev.org/c/openstack/kolla-ansible/+/92242005:48
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: systemd: Add kolla.target  https://review.opendev.org/c/openstack/kolla-ansible/+/92242005:49
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: systemd: Add kolla.target  https://review.opendev.org/c/openstack/kolla-ansible/+/92242005:50
mnasiadkamorning06:17
mnasiadkafrickler: any idea what would need to be change in kolla gerrit acls so we could remove other people RP+1 when they don't respond to reviews?06:19
opendevreviewRoman Krček proposed openstack/kolla master: Fix dev mode  https://review.opendev.org/c/openstack/kolla/+/92571206:28
fricklermnasiadka: I'm not sure whether that would be possible at all without being gerrit admin. maybe one would need to override with voting RP-1 and adopt dashboard filters accordingly06:35
mnasiadkafrickler: yeah, seems you need to be a project owner or theres RemoveReviewer option, but then you can remove any vote06:40
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: add missing license for hvac library  https://review.opendev.org/c/openstack/kolla-ansible/+/92261806:42
opendevreviewMerged openstack/kolla stable/2023.1: Configure Apache services to use correct locale  https://review.opendev.org/c/openstack/kolla/+/92679706:50
mnasiadkafrickler, bbezak: I think it's time to review & merge https://review.opendev.org/c/openstack/kolla-ansible/+/90918806:54
mnasiadkaMatus has done a lot of work, and it's only the first patch in the series06:55
opendevreviewIvan Halomi proposed openstack/kolla-ansible master: Refactor of docker worker  https://review.opendev.org/c/openstack/kolla-ansible/+/90829506:56
mnasiadkaI've groomed a bit our RP+1 list06:57
mnasiadkaSvenKieske: https://review.opendev.org/c/openstack/kolla-ansible/+/898227 I believe we can abandon that?07:11
SvenKieskeyes, will do07:13
mnasiadkaand if you could update https://review.opendev.org/c/openstack/kolla-ansible/+/915403 - I think we have a consensus that we should not support tls 1.0/1.1 ;-)07:14
opendevreviewSven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs  https://review.opendev.org/c/openstack/kolla-ansible/+/92620907:15
SvenKieskeok07:15
SvenKieskemhm wait07:17
fricklergate is broken for the reno job07:18
fricklerfix will be to pin it to jammy, but needs to happen globally, as all of openstack is affected07:18
SvenKieskemnasiadka: are you sure that this doesn't already forbids TLS 1.1/0?07:20
mnasiadkayeah, true07:20
mnasiadkabut still, let's enforce people to use less then 10 years old browsers?07:20
SvenKieskeok, so switch to "modern"? :)07:21
mnasiadkabbezak: modern or as is now?07:27
SvenKieskemodern basically is "the same" but blocks tls1.2 as well and gets rid of ssl-default-server-ciphers (you only specify ciphersuites) as well as the default ciphers. works since:07:31
SvenKieske Supports Firefox 63, Android 10.0, Chrome 70, Edge 75, Java 11, OpenSSL 1.1.1, Opera 57, and Safari 12.1 07:31
opendevreviewVerification of a change to openstack/kolla-ansible master failed: Fix redundant extra config files in grafana role  https://review.opendev.org/c/openstack/kolla-ansible/+/92109607:32
SvenKieskeso tls1.3 mind sound scary, but as long as you don't have ancient android phones you're golden :)07:32
opendevreviewMichal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS  https://review.opendev.org/c/openstack/kolla/+/90758907:39
mnasiadkafrickler: well then all gates are broken :)07:42
opendevreviewRoman Krček proposed openstack/kolla-ansible master: Rewrite kolla-ansible CLI to python  https://review.opendev.org/c/openstack/kolla-ansible/+/92311007:48
opendevreviewVerification of a change to openstack/kolla-ansible master failed: external/internal vip: use proper ip comparison  https://review.opendev.org/c/openstack/kolla-ansible/+/92617508:18
opendevreviewMichal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS  https://review.opendev.org/c/openstack/kolla/+/90758908:27
SvenKieskegate is shaky somehow?08:29
SvenKieskebuild-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/0d2c47fe759f45c092b5e0722e2beee9 : RETRY_LIMIT in 5m 04s08:29
opendevreviewRoman Krček proposed openstack/kolla master: Change copy-cacerts behaviour  https://review.opendev.org/c/openstack/kolla/+/92688208:31
opendevreviewRoman Krček proposed openstack/kolla-ansible master: Don't notify handlers during copy-cert  https://review.opendev.org/c/openstack/kolla-ansible/+/92465108:34
opendevreviewVerification of a change to openstack/kolla-ansible master failed: Add NVMe-TCP Cinder driver support for Pure Storage  https://review.opendev.org/c/openstack/kolla-ansible/+/91997708:43
opendevreviewRafal Lewandowski proposed openstack/kolla master: Add missing iso packages to ironic-conductor  https://review.opendev.org/c/openstack/kolla/+/92688408:56
fricklerSvenKieske: that's what I mentioned earlier, should be fixed by running on jammy now, feel free to recheck08:58
opendevreviewRoman Krček proposed openstack/kolla-ansible master: Fix dev mode  https://review.opendev.org/c/openstack/kolla-ansible/+/92571409:01
opendevreviewVerification of a change to openstack/kolla-ansible master failed: Add REQUESTS_CA_BUNDLE to kolla-toolbox container  https://review.opendev.org/c/openstack/kolla-ansible/+/92310509:04
opendevreviewRoman Krček proposed openstack/kolla master: Fix dev mode  https://review.opendev.org/c/openstack/kolla/+/92571209:06
opendevreviewRoman Krček proposed openstack/kolla master: Fix dev mode  https://review.opendev.org/c/openstack/kolla/+/92571209:09
opendevreviewMagnus Lööf proposed openstack/kolla-ansible master: Enable TLS backend for designate  https://review.opendev.org/c/openstack/kolla-ansible/+/86652410:01
opendevreviewMerged openstack/kolla-ansible master: add missing license for hvac library  https://review.opendev.org/c/openstack/kolla-ansible/+/92261810:24
opendevreviewMerged openstack/kolla-ansible master: Avoid unnecessary secondary fact gathering failure  https://review.opendev.org/c/openstack/kolla-ansible/+/91226310:24
opendevreviewIvan Vnučko proposed openstack/kolla-ansible master: Fix HAProxy TLS config when only backend TLS enabled  https://review.opendev.org/c/openstack/kolla-ansible/+/92689410:40
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/2024.1: Avoid unnecessary secondary fact gathering failure  https://review.opendev.org/c/openstack/kolla-ansible/+/92689511:08
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/2023.2: Avoid unnecessary secondary fact gathering failure  https://review.opendev.org/c/openstack/kolla-ansible/+/92689611:09
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/2023.1: Avoid unnecessary secondary fact gathering failure  https://review.opendev.org/c/openstack/kolla-ansible/+/92689711:10
opendevreviewSven Kieske proposed openstack/kolla master: replace sources.py with sources.json  https://review.opendev.org/c/openstack/kolla/+/92690111:17
SvenKieskethis might be a bit controversial^^ but it's really hard to work with sources.py :P11:18
SvenKieskewould need to delete sources.py still though.11:18
opendevreviewSven Kieske proposed openstack/kolla master: replace sources.py with sources.json  https://review.opendev.org/c/openstack/kolla/+/92690111:19
opendevreviewRafal Lewandowski proposed openstack/kolla master: Add missing iso packages to ironic-conductor  https://review.opendev.org/c/openstack/kolla/+/92688411:19
opendevreviewMichal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS  https://review.opendev.org/c/openstack/kolla/+/90758911:22
kevkoSvenKieske: commented 11:29
opendevreviewMerged openstack/kolla-ansible master: Fix redundant extra config files in grafana role  https://review.opendev.org/c/openstack/kolla-ansible/+/92109611:36
opendevreviewSven Kieske proposed openstack/kolla master: replace sources.py with sources.json  https://review.opendev.org/c/openstack/kolla/+/92690111:49
SvenKieskethx kevko :)11:49
SvenKieskereplied/updated; wrt to json vs yaml: I didn't want to rewrite the whole thing :P :D11:51
kevkoSvenKieske: replied :P11:56
opendevreviewSven Kieske proposed openstack/kolla master: replace sources.py with sources.json  https://review.opendev.org/c/openstack/kolla/+/92690111:59
opendevreviewSven Kieske proposed openstack/kolla master: replace sources.py with sources.json  https://review.opendev.org/c/openstack/kolla/+/92690112:26
mnasiadkaSvenKieske: we've used https://opendev.org/openstack/kolla/src/branch/master/tools/version-check.py in the past, json is too controversial ;-)12:28
SvenKieskemnasiadka: yes I've seen it, but it's very openstack specific, so can't really handle anything not hosted on opendev.org, but I could extend it, then again, the code being there is of almost no use for external codebases imho12:31
opendevreviewMichal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS  https://review.opendev.org/c/openstack/kolla/+/90758912:34
SvenKieskeand "json is controversial" doesn't make any sense, for external dependencies we use json _now_ and in the past already. of course I could do cat json | yq -P to change it to yaml..12:35
mnasiadkawell, it has been yaml, you would do more good making management of that easier12:36
mnasiadkathan by just moving that to json and claiming that it's easier now12:36
SvenKieskeno, I didn't move it to json, look at sources.py, it's a json disguised as a python dict :P but I can convert back to yaml (afaik it was yaml some releases ago?)12:39
SvenKieskeI'll do it, just need to rewrite the dump logic I guess12:39
mnasiadkawell, it was always embedded inside a python code12:40
mnasiadkaI can understand the rationale of doing it in a way a user can supply a .yaml file12:41
mnasiadka(similar to repos.yaml)12:41
SvenKieskewell making it a plain yaml or json file is imho way better than embedding :)12:42
SvenKieskeI'll make a yaml via ruamel.yaml I guess, we already have that in k-a12:42
SvenKieskethx for the feedback12:42
mnasiadkaand then ideally if we could have a script that checks for newer versions of those packages that we don't install from tarballs.openstack.org - then it would be really helpful12:44
kevko+1 13:00
kevko(hate json )13:00
kevko:D13:00
opendevreviewVerification of a change to openstack/kolla stable/2024.1 failed: Configure Apache services to use correct locale  https://review.opendev.org/c/openstack/kolla/+/92679513:13
opendevreviewMerged openstack/kayobe master: Remove unnecessary dependency  https://review.opendev.org/c/openstack/kayobe/+/92666413:15
opendevreviewSven Kieske proposed openstack/kolla master: replace sources.py with sources.yaml  https://review.opendev.org/c/openstack/kolla/+/92690113:43
opendevreviewSven Kieske proposed openstack/kolla master: replace sources.py with sources.yaml  https://review.opendev.org/c/openstack/kolla/+/92690113:45
opendevreviewSven Kieske proposed openstack/kolla master: replace sources.py with sources.yaml  https://review.opendev.org/c/openstack/kolla/+/92690114:19
SvenKieskenow we have yaml and I hate ruamel already :D14:21
opendevreviewMerged openstack/kolla-ansible master: external/internal vip: use proper ip comparison  https://review.opendev.org/c/openstack/kolla-ansible/+/92617514:26
opendevreviewMatt Crees proposed openstack/kayobe master: Support setting Apt preferences  https://review.opendev.org/c/openstack/kayobe/+/92383614:43
opendevreviewMichal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS  https://review.opendev.org/c/openstack/kolla/+/90758914:56
opendevreviewVerification of a change to openstack/kolla stable/2024.1 failed: Configure Apache services to use correct locale  https://review.opendev.org/c/openstack/kolla/+/92679515:36
opendevreviewPierre Riteau proposed openstack/kayobe master: seed-vm/infra-vms: Add support for EFI and q35  https://review.opendev.org/c/openstack/kayobe/+/92166015:36
opendevreviewMerged openstack/kolla-ansible master: Add NVMe-TCP Cinder driver support for Pure Storage  https://review.opendev.org/c/openstack/kolla-ansible/+/91997715:44
opendevreviewMichal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS  https://review.opendev.org/c/openstack/kolla/+/90758916:14
opendevreviewVerification of a change to openstack/kolla-ansible master failed: Add command to upgrade to a target version of RMQ  https://review.opendev.org/c/openstack/kolla-ansible/+/92612416:40
opendevreviewDawud proposed openstack/kolla-ansible master: Add size limits to Fluentd buffers  https://review.opendev.org/c/openstack/kolla-ansible/+/92435916:45
opendevreviewMerged openstack/kolla stable/2024.1: Configure Apache services to use correct locale  https://review.opendev.org/c/openstack/kolla/+/92679518:44
arcayneGot some questions about VF-LAG w/ Kayobe, anyone around? :)19:06
opendevreviewMichal Nasiadka proposed openstack/kolla master: bifrost: bump Ansible version to 8/9 (rpm) and 9/10 (deb)  https://review.opendev.org/c/openstack/kolla/+/92424620:24
opendevreviewMichal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS  https://review.opendev.org/c/openstack/kolla/+/90758920:24
opendevreviewIvan Vnučko proposed openstack/kolla-ansible master: Fix HAProxy TLS config when only backend TLS enabled  https://review.opendev.org/c/openstack/kolla-ansible/+/92689420:53
opendevreviewMerged openstack/kolla-ansible master: Add command to upgrade to a target version of RMQ  https://review.opendev.org/c/openstack/kolla-ansible/+/92612421:15

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