| opendevreview | Michal Arbet proposed openstack/kolla master: Add podman for image building https://review.opendev.org/c/openstack/kolla/+/825791 | 07:15 |
|---|---|---|
| opendevreview | Michal Arbet proposed openstack/kolla master: Add podman for image building https://review.opendev.org/c/openstack/kolla/+/825791 | 07:17 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add podman for image building https://review.opendev.org/c/openstack/kolla/+/825791 | 07:34 |
| kevko | mnasiadka what about Podman kolla image build ? some reviews :) ? | 07:37 |
| mnasiadka | not today, busy with other things | 07:38 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add podman for image building https://review.opendev.org/c/openstack/kolla/+/825791 | 07:42 |
| kevko | mnasiadka: maybe someone else from StackHPC ? | 07:43 |
| kevko | frickler: or frickler maybe ? :) ^^ | 07:43 |
| frickler | kevko: ack, thx for your efforts on this | 07:55 |
| kevko | frickler: no problem, i think i can approve it ...as author is different and i just rebased and made some fixes and debug | 07:55 |
| kevko | frickler: there is probably import error (as I added similar fix for podman error logging )... let me fix .. | 08:01 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add podman for image building https://review.opendev.org/c/openstack/kolla/+/825791 | 08:05 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add podman for image building https://review.opendev.org/c/openstack/kolla/+/825791 | 08:21 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.1: Configure Nova libvirt.num_pcie_ports to 16 by default https://review.opendev.org/c/openstack/kolla-ansible/+/895371 | 08:36 |
| opendevreview | Merged openstack/kolla-ansible stable/zed: Configure Nova libvirt.num_pcie_ports to 16 by default https://review.opendev.org/c/openstack/kolla-ansible/+/895372 | 09:05 |
| opendevreview | Merged openstack/kolla-ansible stable/yoga: Configure Nova libvirt.num_pcie_ports to 16 by default https://review.opendev.org/c/openstack/kolla-ansible/+/895373 | 09:05 |
| opendevreview | Célestin Matte proposed openstack/kolla-ansible master: kolla_address: check correct error return value https://review.opendev.org/c/openstack/kolla-ansible/+/895736 | 09:32 |
| opendevreview | Rafal Lewandowski proposed openstack/kolla-ansible master: Added ML2/OVN and ML2/OVS setting checks for neutron https://review.opendev.org/c/openstack/kolla-ansible/+/895143 | 09:49 |
| opendevreview | Merged openstack/kolla stable/2023.1: Fix stderr logging when image build fail https://review.opendev.org/c/openstack/kolla/+/895030 | 10:59 |
| opendevreview | Merged openstack/kolla stable/zed: Add support for running Nova upgrade checks https://review.opendev.org/c/openstack/kolla/+/881328 | 10:59 |
| *** ralonsoh_ is now known as ralonsoh | 11:07 | |
| opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/2023.1: README: link to the meetings page directly https://review.opendev.org/c/openstack/kolla-ansible/+/895787 | 13:42 |
| opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/zed: README: link to the meetings page directly https://review.opendev.org/c/openstack/kolla-ansible/+/895788 | 13:42 |
| opendevreview | Sven Kieske proposed openstack/kolla-ansible stable/yoga: README: link to the meetings page directly https://review.opendev.org/c/openstack/kolla-ansible/+/895789 | 13:42 |
| SvenKieske | how far should we backport https://review.opendev.org/c/openstack/kolla/+/895157 ? it's currently only targeted for 2023.1 and merged in master. | 13:51 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add podman for image building https://review.opendev.org/c/openstack/kolla/+/825791 | 14:08 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add podman for image building https://review.opendev.org/c/openstack/kolla/+/825791 | 14:10 |
| kevko | SvenKieske: can u verify in which version error logging is not rendered in case of build errror ? | 14:20 |
| kevko | SvenKieske: i mean, for which branches the patch is actual | 14:20 |
| opendevreview | Michal Arbet proposed openstack/kolla master: [CI] Add podman build https://review.opendev.org/c/openstack/kolla/+/895817 | 14:36 |
| SvenKieske | kevko: I think I don't have the time currently, I hoped someone would already know it :D | 14:43 |
| mnasiadka | I was wondering if we could write a unit test that would catch this? | 14:54 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add podman for image building https://review.opendev.org/c/openstack/kolla/+/825791 | 15:13 |
| opendevreview | Michal Arbet proposed openstack/kolla master: [CI] Add podman build https://review.opendev.org/c/openstack/kolla/+/895817 | 15:13 |
| kevko | ^^ let's merge podman build as it's requirement to merge kolla-ansible podman which i have work-in-progress ..but can't use cross dependency for now :( | 15:16 |
| kevko | until not merged | 15:16 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: drop temporary vars for Bookworm switch https://review.opendev.org/c/openstack/kolla/+/895822 | 15:16 |
| kevko | (i want to verify that all tests will pass ..also aarch64) | 15:17 |
| opendevreview | Will Szumski proposed openstack/kolla-ansible master: Support exposing prometheus_server externally https://review.opendev.org/c/openstack/kolla-ansible/+/828695 | 16:10 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!