opendevreview | Michal Arbet proposed openstack/kolla master: Add mechanism for patching files in containers https://review.opendev.org/c/openstack/kolla/+/829295 | 07:50 |
---|---|---|
opendevreview | Michal Arbet proposed openstack/kolla master: [CI] Test kolla patch https://review.opendev.org/c/openstack/kolla/+/929446 | 07:50 |
opendevreview | Michal Arbet proposed openstack/kolla master: [DNM] [CI] Test kolla patch https://review.opendev.org/c/openstack/kolla/+/929446 | 07:52 |
opendevreview | Michal Nasiadka proposed openstack/kolla stable/2024.2: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/934313 | 09:29 |
opendevreview | Michal Arbet proposed openstack/kolla master: [DNM] [CI] Test kolla patch https://review.opendev.org/c/openstack/kolla/+/929446 | 09:37 |
opendevreview | Michal Nasiadka proposed openstack/kolla stable/2024.1: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/934314 | 09:44 |
opendevreview | Michal Nasiadka proposed openstack/kolla stable/2023.2: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/934315 | 09:44 |
opendevreview | Michal Nasiadka proposed openstack/kolla stable/2023.1: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/934316 | 09:45 |
opendevreview | Michal Arbet proposed openstack/kolla master: Add mechanism for patching files in containers https://review.opendev.org/c/openstack/kolla/+/829295 | 09:53 |
opendevreview | Michal Arbet proposed openstack/kolla master: [DNM] [CI] Test kolla patch https://review.opendev.org/c/openstack/kolla/+/929446 | 09:53 |
opendevreview | yosef proposed openstack/kolla-ansible master: Add election-timer option to ovn-db role https://review.opendev.org/c/openstack/kolla-ansible/+/934106 | 09:55 |
opendevreview | Kien Nguyen proposed openstack/kolla stable/2023.2: Add Consul image https://review.opendev.org/c/openstack/kolla/+/934317 | 10:01 |
opendevreview | Kien Nguyen proposed openstack/kolla master: Add Consul image https://review.opendev.org/c/openstack/kolla/+/934318 | 10:10 |
opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2023.1: Prevent accidental downgrades of RabbitMQ https://review.opendev.org/c/openstack/kolla-ansible/+/933876 | 10:28 |
opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2023.1: Fix: add common options to RabbitMQ version check https://review.opendev.org/c/openstack/kolla-ansible/+/934114 | 10:28 |
SvenKieske | I wish you a good vacation kevko :) don't post too many patches in your free time :) | 10:32 |
kevko | SvenKieske: I am kinda worcoholic 😂 | 10:48 |
SvenKieske | than it's even more important you take a break from time to time :) | 10:50 |
opendevreview | Jake Hutchinson proposed openstack/kayobe master: Add loop control to custom DNF repos https://review.opendev.org/c/openstack/kayobe/+/933761 | 11:06 |
opendevreview | Will Szumski proposed openstack/kayobe master: Add internal VIP address to no_proxy https://review.opendev.org/c/openstack/kayobe/+/922787 | 11:46 |
opendevreview | Merged openstack/kayobe master: Respect comments about inspector_keep_ports https://review.opendev.org/c/openstack/kayobe/+/829743 | 12:33 |
opendevreview | Merged openstack/kayobe master: Bump minimum tox version https://review.opendev.org/c/openstack/kayobe/+/934030 | 12:33 |
opendevreview | Merged openstack/kolla stable/2023.1: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/934316 | 13:19 |
opendevreview | Merged openstack/kolla stable/2024.2: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/934313 | 13:42 |
opendevreview | Merged openstack/kolla stable/2024.1: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/934314 | 13:42 |
kevko | SvenKieske: replying to your comments ..please check | 14:06 |
SvenKieske | kevko: okay, fine with me to do it in a second patchset :) | 14:08 |
SvenKieske | seems CI is not very happy today :( | 14:10 |
kevko | SvenKieske: well, I usually write on IRC when I know the person is there, because I tend to close comments – I’m not sure what your opinion is on that, but I think it’s better this way. When comments aren’t closed, people often end up reading the same comments repeatedly during regular review browsing. That’s precisely why I reach out on | 14:14 |
kevko | IRC. | 14:14 |
SvenKieske | it's okayish, sometimes I'm confused about which patchset you are talking when I reviewed a bunch of them :) | 14:15 |
SvenKieske | also currently in a meeting, so slow to respond | 14:15 |
kevko | SvenKieske: no problem | 14:21 |
kevko | SvenKieske: done, check my comments and let me know | 14:24 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix TLS settings when letsencrypt turned on https://review.opendev.org/c/openstack/kolla-ansible/+/925971 | 14:24 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Don't notify handlers during copy-cert https://review.opendev.org/c/openstack/kolla-ansible/+/924651 | 14:24 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Performance: Don't notify handlers during config https://review.opendev.org/c/openstack/kolla-ansible/+/745164 | 14:24 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Refactor services' check-containers and optimise https://review.opendev.org/c/openstack/kolla-ansible/+/773243 | 14:24 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix unintentional trigger of ansible handlers https://review.opendev.org/c/openstack/kolla-ansible/+/924145 | 14:24 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix proxysql-config's TLS DB configuration https://review.opendev.org/c/openstack/kolla-ansible/+/933922 | 14:24 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add TLS support for MariaDB Connections https://review.opendev.org/c/openstack/kolla-ansible/+/933924 | 14:24 |
SvenKieske | kevko: ok, seems all is resolved for https://review.opendev.org/c/openstack/kolla-ansible/+/925971 but I'm confused if you want to backport this or not? :D I think you wrote you want to backport the fix, but you voted -2 on backportvoting.. | 14:55 |
kevko | SvenKieske: there is an variable change ...so it's better to include in next version and don't backport ..but downstream maintainers can ..of course | 15:55 |
SvenKieske | ok | 15:57 |
opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2023.1: Prevent accidental downgrades of RabbitMQ https://review.opendev.org/c/openstack/kolla-ansible/+/933876 | 16:03 |
opendevreview | Matt Crees proposed openstack/kolla-ansible stable/2023.1: Fix: add common options to RabbitMQ version check https://review.opendev.org/c/openstack/kolla-ansible/+/934114 | 16:03 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2024.1: Fix mariadb backup unable to create history table https://review.opendev.org/c/openstack/kolla-ansible/+/934334 | 16:04 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2023.2: Fix mariadb backup unable to create history table https://review.opendev.org/c/openstack/kolla-ansible/+/934335 | 16:06 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2024.1: Change from PERCONA_SCHEMA to mysql for mariabackup https://review.opendev.org/c/openstack/kolla-ansible/+/934338 | 16:10 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2023.2: Change from PERCONA_SCHEMA to mysql for mariabackup https://review.opendev.org/c/openstack/kolla-ansible/+/934339 | 16:11 |
opendevreview | Will Szumski proposed openstack/kayobe master: Mention change of default in release note https://review.opendev.org/c/openstack/kayobe/+/934344 | 16:26 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2024.1: Add logrotate config for redis https://review.opendev.org/c/openstack/kolla-ansible/+/934345 | 16:29 |
opendevreview | Sven Kieske proposed openstack/kolla stable/2024.1: Add way to backup from MariaDB replica server https://review.opendev.org/c/openstack/kolla/+/934346 | 16:31 |
opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2023.2: [CI] Fix OVN jobs - add required neutron_dns_domain https://review.opendev.org/c/openstack/kolla-ansible/+/934347 | 16:36 |
frickler | SvenKieske: since you wanted to see the deploy-guide link patch: https://review.opendev.org/c/openstack/openstack-manuals/+/934349 (actually will enable other docs links, too) | 16:44 |
SvenKieske | frickler: ah thx! | 16:47 |
SvenKieske | frickler: too bad there is no gerrit search option to search for changesets which have backport voting, but no actual backport yet, the best I came up with so far is: https://review.opendev.org/q/(project:openstack/kolla+or+project:openstack/kolla-ansible)+status:merged+label:Backport-Candidate%3E%3D1+ | 16:48 |
SvenKieske | probably could create a custom label that somehow detects if something has a cherry-pick (the reverse is easily searchable) | 16:49 |
opendevreview | Verification of a change to openstack/kolla stable/2023.2 failed: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/934315 | 17:04 |
yosef | ommmm pipelines are failing? | 19:21 |
opendevreview | Pierre Riteau proposed openstack/kolla-ansible master: [WIP] Work around neutron-l3-agent keepalived issues https://review.opendev.org/c/openstack/kolla-ansible/+/934383 | 21:40 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!