| opendevreview | Merged openstack/openstack-ansible-os_octavia stable/xena: Adding octavia_provider_network_mtu-parameter parameter https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/865644 | 00:00 |
|---|---|---|
| noonedeadpunk | o/ mornings | 10:04 |
| damiandabrowski | hi! | 10:04 |
| jrosser | hello | 10:05 |
| opendevreview | Marcus Klein proposed openstack/openstack-ansible-ceph_client master: Ensure role not fail when secret_uuid is not part of ceph_extra_confs https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/866974 | 10:12 |
| noonedeadpunk | sry, got trouble with bouncer | 10:44 |
| noonedeadpunk | so, we've got upgrade jobs unhappy for infra tasks with our neutron upgrade step | 10:47 |
| noonedeadpunk | https://review.opendev.org/c/openstack/openstack-ansible/+/866847 should fix that | 10:47 |
| noonedeadpunk | At last DNM now happy https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/866848 | 10:47 |
| noonedeadpunk | ok, I see rechecks were already issued after yestarday tox thingy | 10:49 |
| *** dviroel|afk is now known as dviroel | 10:51 | |
| noonedeadpunk | Btw, at least 1 distro job is passing now :D https://zuul.opendev.org/t/openstack/build/e3806d1cde0b4c969fff4d5633a519de | 10:54 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Use /healthcheck URI for backends https://review.opendev.org/c/openstack/openstack-ansible/+/864424 | 10:57 |
| *** chandankumar is now known as chkumar|afk | 11:00 | |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Drop CentOS 8 Stream https://review.opendev.org/c/openstack/openstack-ansible/+/863258 | 11:00 |
| noonedeadpunk | So, we've left to land bunch of neutron and ironic patches and maybe some small things in different repos (like lxc/haproxy roles I guess?) | 11:00 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Change defaults for octavia topology and affinity https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/866062 | 11:01 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Remove support for calico ml2 driver. https://review.opendev.org/c/openstack/openstack-ansible/+/866119 | 11:02 |
| noonedeadpunk | jrosser: there was 1 comment on https://review.opendev.org/c/openstack/openstack-ansible/+/860960/2/inventory/group_vars/all/ironic.yml We can leave as is ofc... | 11:05 |
| jrosser | you mean just put the values straight in haproxy service rather than use variables? | 11:06 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/yoga: Define apmhora provider for tempest config https://review.opendev.org/c/openstack/openstack-ansible/+/866898 | 11:06 |
| noonedeadpunk | jrosser: I meant use smth like (ironic_console_type | default('disabled') == nova_console_type) in haproxy | 11:08 |
| noonedeadpunk | though I'm not sure what's better/faster for ansible - process default filter or have many group_vars... | 11:09 |
| jrosser | i did realise that it would do something wrong if they are both 'disabled' | 11:11 |
| noonedeadpunk | ah, ok then | 11:11 |
| jrosser | i mean i think the logic even now in the patch may not be totally correct | 11:12 |
| noonedeadpunk | well, it's tricky indeed but looks like it should work in general | 11:12 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Add coordination to octavia https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/866058 | 11:14 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/xena: Install dynamic-inventory as console_script https://review.opendev.org/c/openstack/openstack-ansible/+/866900 | 11:17 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/yoga: Install dynamic-inventory as console_script https://review.opendev.org/c/openstack/openstack-ansible/+/866902 | 11:17 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Reduce amount of distro jobs https://review.opendev.org/c/openstack/openstack-ansible/+/866982 | 11:27 |
| *** chkumar|afk is now known as chandankumar | 12:03 | |
| mgariepy | good morning | 13:03 |
| noonedeadpunk | o/ | 13:04 |
| mgariepy | how are you doing ? | 13:05 |
| mgariepy | anything needs review before the deadline ? | 13:05 |
| * jrosser needs to attend to ironic patches | 13:13 | |
| opendevreview | Merged openstack/openstack-ansible master: Do not run neutron upgrade when no hosts in group https://review.opendev.org/c/openstack/openstack-ansible/+/866847 | 13:30 |
| opendevreview | Merged openstack/openstack-ansible-os_nova master: Define local facts separately only for distro https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/866697 | 14:44 |
| opendevreview | Merged openstack/openstack-ansible-os_keystone master: Define venv_tag as separate task for distro https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/866684 | 14:47 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Create /var/log/ironic directory https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/867003 | 14:56 |
| opendevreview | Merged openstack/openstack-ansible-os_neutron master: Add lxb jobs https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/865973 | 15:10 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Use /healthcheck URI for backends https://review.opendev.org/c/openstack/openstack-ansible/+/864424 | 15:13 |
| opendevreview | Merged openstack/openstack-ansible master: Simplify glance deployment logic https://review.opendev.org/c/openstack/openstack-ansible/+/866693 | 15:41 |
| opendevreview | Merged openstack/openstack-ansible-os_cinder master: Define local facts separately only for distro https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/866694 | 15:58 |
| *** dviroel is now known as dviroel|lunch | 16:10 | |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_designate stable/yoga: Fix race condition during designate setup https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/867013 | 16:16 |
| opendevreview | Merged openstack/openstack-ansible-os_cinder master: Add coordination support https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/865518 | 16:17 |
| *** dviroel|lunch is now known as dviroel | 17:06 | |
| opendevreview | Merged openstack/openstack-ansible master: Disable sahara tempest tests https://review.opendev.org/c/openstack/openstack-ansible/+/866124 | 17:28 |
| opendevreview | Merged openstack/openstack-ansible master: Drop CentOS 8 Stream https://review.opendev.org/c/openstack/openstack-ansible/+/863258 | 17:58 |
| opendevreview | Merged openstack/openstack-ansible master: Reduce amount of distro jobs https://review.opendev.org/c/openstack/openstack-ansible/+/866982 | 17:58 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Allow extra plugin installation ironic/inspector venvs https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/864511 | 18:00 |
| opendevreview | Merged openstack/openstack-ansible-os_octavia stable/wallaby: Adding octavia_provider_network_mtu-parameter parameter https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/865645 | 18:37 |
| opendevreview | Merged openstack/openstack-ansible-os_ironic master: Refactor ironic deploy image handling. https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/863549 | 19:07 |
| *** dviroel_ is now known as dviroel | 19:17 | |
| opendevreview | Merged openstack/openstack-ansible-os_neutron master: Switch OVN Metadata Agent to journal logging https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/866716 | 19:29 |
| opendevreview | Merged openstack/openstack-ansible-os_neutron master: Do not install neutron venv if not needed. https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/863546 | 19:33 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-zookeeper master: Add envi to the list of allowed commands https://review.opendev.org/c/openstack/ansible-role-zookeeper/+/867049 | 19:34 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Add coordination to octavia https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/866058 | 19:35 |
| noonedeadpunk | hm | 19:38 |
| noonedeadpunk | Looks like we've broken octavia with OVN change | 19:39 |
| noonedeadpunk | https://zuul.opendev.org/t/openstack/build/b2046015ae6b417b8f8ea78233f954bc/log/logs/host/octavia-worker.service.journal-13-08-51.log.txt#4346-4379 | 19:39 |
| noonedeadpunk | jamesdenton: any thoughts ?:) | 19:39 |
| noonedeadpunk | I do have reproduced that in AIO | 19:40 |
| noonedeadpunk | Can give an access to it... | 19:40 |
| noonedeadpunk | DM in case you'll have time for that. It's fine if not - will try to check out tomorrow | 19:45 |
| mgariepy | i do not know exacatly how it works but is it only spawning a vm in a pre-defined project to host the haproxy for the user? | 19:58 |
| jamesdenton | hi | 20:16 |
| jamesdenton | i believe it's a multihomed VM with a leg in the octavia mgmt network and another in the (likely) tenant network | 20:20 |
| *** dviroel is now known as dviroel|afk | 20:20 | |
| jamesdenton | noonedeadpunk yes, it's broken due to provider bridge issues | 20:22 |
| jamesdenton | i'll try and patch it real quick | 20:23 |
| jrosser | would be good to understand how to do this, mix provider vlans into an otherwise OVN overlay setup | 20:36 |
| jamesdenton | Well, in this case, each of these are a "flat" network with a dedicated provider bridge, and are connected to the host via a veth | 20:37 |
| mgariepy | would be also nice to probably use the ovn octavia direcly ? https://docs.openstack.org/ovn-octavia-provider/latest/contributor/loadbalancer.html | 20:37 |
| jamesdenton | agreed. i think there are limitations but would be good to have | 20:38 |
| jamesdenton | git review is being fussy for me | 20:38 |
| mgariepy | i only installed lb service once on kilo or liberty with the haproxy provider. | 20:38 |
| mgariepy | it wasn't very good back then. | 20:39 |
| jamesdenton | ahh, that was pre-amphora i bet | 20:39 |
| jamesdenton | pre-octavia | 20:39 |
| mgariepy | yeah was in neutron directly iirc | 20:39 |
| mgariepy | gave me nightmare up until recently lol | 20:39 |
| jamesdenton | hehe | 20:39 |
| jamesdenton | it's better now | 20:40 |
| mgariepy | i had issue with the DB on an upgrade to U iirc LOL | 20:40 |
| mgariepy | from back then.. | 20:40 |
| opendevreview | Merged openstack/openstack-ansible stable/yoga: Install dynamic-inventory as console_script https://review.opendev.org/c/openstack/openstack-ansible/+/866902 | 20:41 |
| *** anbanerj is now known as frenzy_friday | 20:44 | |
| opendevreview | James Denton proposed openstack/openstack-ansible master: Update AIO to use OVN-style provider network for Octavia https://review.opendev.org/c/openstack/openstack-ansible/+/867052 | 20:46 |
| opendevreview | James Denton proposed openstack/openstack-ansible-os_octavia master: Add coordination to octavia https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/866058 | 20:47 |
| johnsom | mgariepy Be sure you are aware of the limitations of the OVN provider: https://docs.openstack.org/octavia/latest/user/feature-classification/index.html | 20:48 |
| johnsom | No health monitors yet, etc. | 20:48 |
| jamesdenton | do we need this? https://opendev.org/openstack/ovn-octavia-provider | 20:51 |
| johnsom | If you want to offer the OVN provider driver (in addition or in place of the amphora driver). | 20:52 |
| johnsom | It is optional though | 20:52 |
| jamesdenton | thumbsup | 20:52 |
| mgariepy | thanks johnsom | 20:53 |
| johnsom | The amphora provider driver runs fine on the OVN ML2 without that | 20:53 |
| opendevreview | Merged openstack/openstack-ansible-os_ironic master: Allow extra plugin installation ironic/inspector venvs https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/864511 | 21:41 |
| opendevreview | Merged openstack/openstack-ansible-os_ironic master: Add variable for user defined list of deploy images https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/865310 | 21:41 |
| opendevreview | Merged openstack/openstack-ansible-os_ironic master: Update variables for switchport introspection during inspection https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/864437 | 21:45 |
| opendevreview | Merged openstack/openstack-ansible-os_ironic master: Create /var/log/ironic directory https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/867003 | 22:17 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!