opendevreview | Verification of a change to openstack/kolla stable/2024.1 failed: Configure Apache services to use correct locale https://review.opendev.org/c/openstack/kolla/+/926795 | 01:53 |
---|---|---|
opendevreview | Michal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS https://review.opendev.org/c/openstack/kolla/+/907589 | 05:46 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: Add kolla.target https://review.opendev.org/c/openstack/kolla-ansible/+/922420 | 05:48 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: Add kolla.target https://review.opendev.org/c/openstack/kolla-ansible/+/922420 | 05:48 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: Add kolla.target https://review.opendev.org/c/openstack/kolla-ansible/+/922420 | 05:49 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: systemd: Add kolla.target https://review.opendev.org/c/openstack/kolla-ansible/+/922420 | 05:50 |
mnasiadka | morning | 06:17 |
mnasiadka | frickler: 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 |
opendevreview | Roman Krček proposed openstack/kolla master: Fix dev mode https://review.opendev.org/c/openstack/kolla/+/925712 | 06:28 |
frickler | mnasiadka: 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 accordingly | 06:35 |
mnasiadka | frickler: yeah, seems you need to be a project owner or theres RemoveReviewer option, but then you can remove any vote | 06:40 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: add missing license for hvac library https://review.opendev.org/c/openstack/kolla-ansible/+/922618 | 06:42 |
opendevreview | Merged openstack/kolla stable/2023.1: Configure Apache services to use correct locale https://review.opendev.org/c/openstack/kolla/+/926797 | 06:50 |
mnasiadka | frickler, bbezak: I think it's time to review & merge https://review.opendev.org/c/openstack/kolla-ansible/+/909188 | 06:54 |
mnasiadka | Matus has done a lot of work, and it's only the first patch in the series | 06:55 |
opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Refactor of docker worker https://review.opendev.org/c/openstack/kolla-ansible/+/908295 | 06:56 |
mnasiadka | I've groomed a bit our RP+1 list | 06:57 |
mnasiadka | SvenKieske: https://review.opendev.org/c/openstack/kolla-ansible/+/898227 I believe we can abandon that? | 07:11 |
SvenKieske | yes, will do | 07:13 |
mnasiadka | and 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 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible master: CI: validate prometheus configs https://review.opendev.org/c/openstack/kolla-ansible/+/926209 | 07:15 |
SvenKieske | ok | 07:15 |
SvenKieske | mhm wait | 07:17 |
frickler | gate is broken for the reno job | 07:18 |
frickler | fix will be to pin it to jammy, but needs to happen globally, as all of openstack is affected | 07:18 |
SvenKieske | mnasiadka: are you sure that this doesn't already forbids TLS 1.1/0? | 07:20 |
mnasiadka | yeah, true | 07:20 |
mnasiadka | but still, let's enforce people to use less then 10 years old browsers? | 07:20 |
SvenKieske | ok, so switch to "modern"? :) | 07:21 |
mnasiadka | bbezak: modern or as is now? | 07:27 |
SvenKieske | modern 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 |
opendevreview | Verification 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/+/921096 | 07:32 |
SvenKieske | so tls1.3 mind sound scary, but as long as you don't have ancient android phones you're golden :) | 07:32 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS https://review.opendev.org/c/openstack/kolla/+/907589 | 07:39 |
mnasiadka | frickler: well then all gates are broken :) | 07:42 |
opendevreview | Roman Krček proposed openstack/kolla-ansible master: Rewrite kolla-ansible CLI to python https://review.opendev.org/c/openstack/kolla-ansible/+/923110 | 07:48 |
opendevreview | Verification of a change to openstack/kolla-ansible master failed: external/internal vip: use proper ip comparison https://review.opendev.org/c/openstack/kolla-ansible/+/926175 | 08:18 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS https://review.opendev.org/c/openstack/kolla/+/907589 | 08:27 |
SvenKieske | gate is shaky somehow? | 08:29 |
SvenKieske | build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/0d2c47fe759f45c092b5e0722e2beee9 : RETRY_LIMIT in 5m 04s | 08:29 |
opendevreview | Roman Krček proposed openstack/kolla master: Change copy-cacerts behaviour https://review.opendev.org/c/openstack/kolla/+/926882 | 08:31 |
opendevreview | Roman Krček proposed openstack/kolla-ansible master: Don't notify handlers during copy-cert https://review.opendev.org/c/openstack/kolla-ansible/+/924651 | 08:34 |
opendevreview | Verification 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/+/919977 | 08:43 |
opendevreview | Rafal Lewandowski proposed openstack/kolla master: Add missing iso packages to ironic-conductor https://review.opendev.org/c/openstack/kolla/+/926884 | 08:56 |
frickler | SvenKieske: that's what I mentioned earlier, should be fixed by running on jammy now, feel free to recheck | 08:58 |
opendevreview | Roman Krček proposed openstack/kolla-ansible master: Fix dev mode https://review.opendev.org/c/openstack/kolla-ansible/+/925714 | 09:01 |
opendevreview | Verification 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/+/923105 | 09:04 |
opendevreview | Roman Krček proposed openstack/kolla master: Fix dev mode https://review.opendev.org/c/openstack/kolla/+/925712 | 09:06 |
opendevreview | Roman Krček proposed openstack/kolla master: Fix dev mode https://review.opendev.org/c/openstack/kolla/+/925712 | 09:09 |
opendevreview | Magnus Lööf proposed openstack/kolla-ansible master: Enable TLS backend for designate https://review.opendev.org/c/openstack/kolla-ansible/+/866524 | 10:01 |
opendevreview | Merged openstack/kolla-ansible master: add missing license for hvac library https://review.opendev.org/c/openstack/kolla-ansible/+/922618 | 10:24 |
opendevreview | Merged openstack/kolla-ansible master: Avoid unnecessary secondary fact gathering failure https://review.opendev.org/c/openstack/kolla-ansible/+/912263 | 10:24 |
opendevreview | Ivan Vnučko proposed openstack/kolla-ansible master: Fix HAProxy TLS config when only backend TLS enabled https://review.opendev.org/c/openstack/kolla-ansible/+/926894 | 10:40 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/2024.1: Avoid unnecessary secondary fact gathering failure https://review.opendev.org/c/openstack/kolla-ansible/+/926895 | 11:08 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/2023.2: Avoid unnecessary secondary fact gathering failure https://review.opendev.org/c/openstack/kolla-ansible/+/926896 | 11:09 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/2023.1: Avoid unnecessary secondary fact gathering failure https://review.opendev.org/c/openstack/kolla-ansible/+/926897 | 11:10 |
opendevreview | Sven Kieske proposed openstack/kolla master: replace sources.py with sources.json https://review.opendev.org/c/openstack/kolla/+/926901 | 11:17 |
SvenKieske | this might be a bit controversial^^ but it's really hard to work with sources.py :P | 11:18 |
SvenKieske | would need to delete sources.py still though. | 11:18 |
opendevreview | Sven Kieske proposed openstack/kolla master: replace sources.py with sources.json https://review.opendev.org/c/openstack/kolla/+/926901 | 11:19 |
opendevreview | Rafal Lewandowski proposed openstack/kolla master: Add missing iso packages to ironic-conductor https://review.opendev.org/c/openstack/kolla/+/926884 | 11:19 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS https://review.opendev.org/c/openstack/kolla/+/907589 | 11:22 |
kevko | SvenKieske: commented | 11:29 |
opendevreview | Merged openstack/kolla-ansible master: Fix redundant extra config files in grafana role https://review.opendev.org/c/openstack/kolla-ansible/+/921096 | 11:36 |
opendevreview | Sven Kieske proposed openstack/kolla master: replace sources.py with sources.json https://review.opendev.org/c/openstack/kolla/+/926901 | 11:49 |
SvenKieske | thx kevko :) | 11:49 |
SvenKieske | replied/updated; wrt to json vs yaml: I didn't want to rewrite the whole thing :P :D | 11:51 |
kevko | SvenKieske: replied :P | 11:56 |
opendevreview | Sven Kieske proposed openstack/kolla master: replace sources.py with sources.json https://review.opendev.org/c/openstack/kolla/+/926901 | 11:59 |
opendevreview | Sven Kieske proposed openstack/kolla master: replace sources.py with sources.json https://review.opendev.org/c/openstack/kolla/+/926901 | 12:26 |
mnasiadka | SvenKieske: we've used https://opendev.org/openstack/kolla/src/branch/master/tools/version-check.py in the past, json is too controversial ;-) | 12:28 |
SvenKieske | mnasiadka: 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 imho | 12:31 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS https://review.opendev.org/c/openstack/kolla/+/907589 | 12:34 |
SvenKieske | and "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 |
mnasiadka | well, it has been yaml, you would do more good making management of that easier | 12:36 |
mnasiadka | than by just moving that to json and claiming that it's easier now | 12:36 |
SvenKieske | no, 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 |
SvenKieske | I'll do it, just need to rewrite the dump logic I guess | 12:39 |
mnasiadka | well, it was always embedded inside a python code | 12:40 |
mnasiadka | I can understand the rationale of doing it in a way a user can supply a .yaml file | 12:41 |
mnasiadka | (similar to repos.yaml) | 12:41 |
SvenKieske | well making it a plain yaml or json file is imho way better than embedding :) | 12:42 |
SvenKieske | I'll make a yaml via ruamel.yaml I guess, we already have that in k-a | 12:42 |
SvenKieske | thx for the feedback | 12:42 |
mnasiadka | and 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 helpful | 12:44 |
kevko | +1 | 13:00 |
kevko | (hate json ) | 13:00 |
kevko | :D | 13:00 |
opendevreview | Verification of a change to openstack/kolla stable/2024.1 failed: Configure Apache services to use correct locale https://review.opendev.org/c/openstack/kolla/+/926795 | 13:13 |
opendevreview | Merged openstack/kayobe master: Remove unnecessary dependency https://review.opendev.org/c/openstack/kayobe/+/926664 | 13:15 |
opendevreview | Sven Kieske proposed openstack/kolla master: replace sources.py with sources.yaml https://review.opendev.org/c/openstack/kolla/+/926901 | 13:43 |
opendevreview | Sven Kieske proposed openstack/kolla master: replace sources.py with sources.yaml https://review.opendev.org/c/openstack/kolla/+/926901 | 13:45 |
opendevreview | Sven Kieske proposed openstack/kolla master: replace sources.py with sources.yaml https://review.opendev.org/c/openstack/kolla/+/926901 | 14:19 |
SvenKieske | now we have yaml and I hate ruamel already :D | 14:21 |
opendevreview | Merged openstack/kolla-ansible master: external/internal vip: use proper ip comparison https://review.opendev.org/c/openstack/kolla-ansible/+/926175 | 14:26 |
opendevreview | Matt Crees proposed openstack/kayobe master: Support setting Apt preferences https://review.opendev.org/c/openstack/kayobe/+/923836 | 14:43 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS https://review.opendev.org/c/openstack/kolla/+/907589 | 14:56 |
opendevreview | Verification of a change to openstack/kolla stable/2024.1 failed: Configure Apache services to use correct locale https://review.opendev.org/c/openstack/kolla/+/926795 | 15:36 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: seed-vm/infra-vms: Add support for EFI and q35 https://review.opendev.org/c/openstack/kayobe/+/921660 | 15:36 |
opendevreview | Merged openstack/kolla-ansible master: Add NVMe-TCP Cinder driver support for Pure Storage https://review.opendev.org/c/openstack/kolla-ansible/+/919977 | 15:44 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS https://review.opendev.org/c/openstack/kolla/+/907589 | 16:14 |
opendevreview | Verification 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/+/926124 | 16:40 |
opendevreview | Dawud proposed openstack/kolla-ansible master: Add size limits to Fluentd buffers https://review.opendev.org/c/openstack/kolla-ansible/+/924359 | 16:45 |
opendevreview | Merged openstack/kolla stable/2024.1: Configure Apache services to use correct locale https://review.opendev.org/c/openstack/kolla/+/926795 | 18:44 |
arcayne | Got some questions about VF-LAG w/ Kayobe, anyone around? :) | 19:06 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: bifrost: bump Ansible version to 8/9 (rpm) and 9/10 (deb) https://review.opendev.org/c/openstack/kolla/+/924246 | 20:24 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: Switch to Ubuntu 24.04 LTS https://review.opendev.org/c/openstack/kolla/+/907589 | 20:24 |
opendevreview | Ivan Vnučko proposed openstack/kolla-ansible master: Fix HAProxy TLS config when only backend TLS enabled https://review.opendev.org/c/openstack/kolla-ansible/+/926894 | 20:53 |
opendevreview | Merged openstack/kolla-ansible master: Add command to upgrade to a target version of RMQ https://review.opendev.org/c/openstack/kolla-ansible/+/926124 | 21:15 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!