| opendevreview | Yuki Hirano proposed openstack/openstack-ansible-os_octavia master: Add additional security group rules support https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/927353 | 07:44 |
|---|---|---|
| noonedeadpunk | good morning | 07:56 |
| jrosser | o/ hello | 08:05 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2024.1: Do not install Skyline with os-infra_hosts https://review.opendev.org/c/openstack/openstack-ansible/+/927363 | 09:03 |
| opendevreview | Yuki Hirano proposed openstack/openstack-ansible-os_octavia master: Add additional security group rules support https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/927353 | 09:10 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-rabbitmq_server master: Manage apt repositores and keys using deb822_repository module https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/907833 | 09:25 |
| jrosser | ^ there is an attempt to check that old-style repo definitions are not in use here | 09:26 |
| noonedeadpunk | jrosser: I think I'd assert based on `repo` rather then `filename` as latter is optional from what I see in code | 09:54 |
| jrosser | ok i can do that | 09:56 |
| noonedeadpunk | but otherwise it looks really good | 09:56 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-rabbitmq_server master: Manage apt repositores and keys using deb822_repository module https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/907833 | 10:00 |
| noonedeadpunk | a "typo" in selectattr :) | 10:03 |
| noonedeadpunk | I've suggested edit just in case in gerrit | 10:04 |
| opendevreview | Merged openstack/openstack-ansible master: Simplify handling of 'telemetry' scenario in CI tests https://review.opendev.org/c/openstack/openstack-ansible/+/926206 | 10:12 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-rabbitmq_server master: Manage apt repositores and keys using deb822_repository module https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/907833 | 10:43 |
| jrosser | doh | 10:43 |
| noonedeadpunk | jrosser: hm, somehow that failed... but repo seems in place at least: https://zuul.opendev.org/t/openstack/build/c898e5d421eb48fe91ea8326132bc7f5/log/logs/etc/openstack/aio1-rabbit-mq-container-2fa65270/apt/sources.list.d/rabbitmq.sources.txt | 14:23 |
| noonedeadpunk | but installed version is obviously wrong: https://zuul.opendev.org/t/openstack/build/c898e5d421eb48fe91ea8326132bc7f5/log/logs/openstack/aio1-rabbit-mq-container-2fa65270/apt/history.log.txt#14 | 14:23 |
| noonedeadpunk | and it's coming from the native repo | 14:24 |
| jrosser | hmm i'll try that locally | 15:58 |
| jrosser | itss been a very long time since i looked at this | 15:58 |
| jrosser | noonedeadpunk: so the rabbitmq being from the wrong repo is becasue there is not equivalent of `apt update` happening | 16:48 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-rabbitmq_server master: Manage apt repositores and keys using deb822_repository module https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/907833 | 17:32 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!