| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Require `is_modified` for "root" objects only https://review.opendev.org/c/openstack/neutron/+/918490 | 00:57 |
|---|---|---|
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Require `is_modified` for "root" objects only https://review.opendev.org/c/openstack/neutron/+/918490 | 01:13 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Don't update revision number if object was not modified https://review.opendev.org/c/openstack/neutron/+/918492 | 01:30 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: tests: Don't use neutron object where db model is expected https://review.opendev.org/c/openstack/neutron/+/918493 | 01:30 |
| opendevreview | Merged openstack/python-neutronclient master: BGPVPN: Fix resource comparison https://review.opendev.org/c/openstack/python-neutronclient/+/917835 | 02:08 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: Don't update revision number if object was not modified https://review.opendev.org/c/openstack/neutron/+/918492 | 02:20 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: tests: Don't use neutron object where db model is expected https://review.opendev.org/c/openstack/neutron/+/918493 | 02:20 |
| *** logan_ is now known as Guest4824 | 05:45 | |
| tkajinam | o/ I wonder if https://review.opendev.org/c/openstack/neutron-vpnaas/+/915838 can get some attention. this may be needed before sqlalchemy in global requirements is bumped | 06:36 |
| lajoskatona | amotoki: Hi, Hope you are well :-) just a highlight that next week you are the bug deputy for Neutron, if it is not fit for you just ping mlavalle, me (lajoskatona) or slaweq, and we arrange it for you :-) | 07:08 |
| slaweq | lajoskatona ykarel hi, can one of You check https://review.opendev.org/c/openstack/neutron/+/918492 - we internally seems that we need it quickly so I would be very grateful for help with that one :) | 09:13 |
| opendevreview | Gaudenz Steinlin proposed openstack/neutron master: Add L3 HA fullstack failover test https://review.opendev.org/c/openstack/neutron/+/917429 | 10:02 |
| opendevreview | Gaudenz Steinlin proposed openstack/neutron master: Add conntrackd support to HA routers in L3 agent https://review.opendev.org/c/openstack/neutron/+/917430 | 10:02 |
| ykarel | slaweq, ack | 10:32 |
| *** iurygregory_ is now known as iurygregory | 11:21 | |
| opendevreview | Ihtisham ul Haq proposed openstack/neutron master: Optimize deletion of static routes https://review.opendev.org/c/openstack/neutron/+/914900 | 11:26 |
| *** liuxie is now known as liushy | 11:55 | |
| opendevreview | Elod Illes proposed openstack/networking-bgpvpn unmaintained/zed: [CI] Remove tripleo-ci-centos-8-scenario004-standalone job https://review.opendev.org/c/openstack/networking-bgpvpn/+/918632 | 12:48 |
| opendevreview | Merged openstack/neutron master: Don't update revision number if object was not modified https://review.opendev.org/c/openstack/neutron/+/918492 | 13:08 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron stable/2024.1: Don't update revision number if object was not modified https://review.opendev.org/c/openstack/neutron/+/918641 | 13:14 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron stable/2023.2: Don't update revision number if object was not modified https://review.opendev.org/c/openstack/neutron/+/918642 | 13:15 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron stable/2023.1: Don't update revision number if object was not modified https://review.opendev.org/c/openstack/neutron/+/918643 | 13:15 |
| lowercase | I'm working toward looking through and understanding the neutron code base. It was advised here: https://wiki.openstack.org/wiki/NeutronDevstack#Basic_Setup to setup a devstack. It appears at some point, devstack switched to a OVN backend, leaving the devstack documentation possibly out of date. Does anyone here use devstack, if not what do you use? If you do use devstack, could you share you local.conf | 13:28 |
| slaweq | hi lowercase, I think that most of us are using devstack | 13:35 |
| slaweq | I have some own ansible roles which makes local.conf file for me. It can be found in https://github.com/slawqo/ansible/blob/master/roles/devstack/templates/local.conf.j2 and https://github.com/slawqo/ansible/blob/master/roles/devstack/templates/local.conf.ovn.j2 | 13:36 |
| slaweq | and I am using them with vagrant https://github.com/slawqo/vagrant/tree/master/devstack/Ubuntu/ovn | 13:36 |
| slaweq | you can also check local.conf used in the ci jobs, like e.g. https://zuul.opendev.org/t/openstack/build/2ed234a874014922a0431076f994df23/logs - local.conf file is there in controller/logs directory there | 13:38 |
| ykarel | i used to use https://github.com/openstack/neutron/blob/master/devstack/ovn-local.conf.sample and works fine | 13:42 |
| lowercase | slaweq ykarel thank you for your replies. I'm going through the code now. | 13:48 |
| seba | If I have a self-written ml2_api.MechanismDriver and I want that driver to start its own RPC backend, how would I do that? We've split neutron-server+uwsgi and neutron-rpc-server, but I don't see a way that start_rpc_listeners() is called anywhere on that driver (as they are no service plugins). The Ml2Plugin itself gets a start_rpc_listeners() call, but doesn't ask its drivers for endpoints. Any way around that? | 13:50 |
| seba | This also kinda feels like something that could solve https://bugs.launchpad.net/neutron/+bug/1912359 | 13:50 |
| *** haleyb|out is now known as haleyb | 14:40 | |
| opendevreview | Elod Illes proposed openstack/networking-bgpvpn unmaintained/yoga: [CI] Remove tripleo-ci-centos-8-scenario004-standalone job https://review.opendev.org/c/openstack/networking-bgpvpn/+/918666 | 15:04 |
| seba | I think it would make sense if Ml2Plugin.start_rpc_listeners() would call MechanismManager.start_driver_rpc_listeners() which then call start_rpc_listeners() on each driver. I've just implemented that in my qa environment and it seems to work (for the limited set of tests I have readily available). If done properly this would also require a change in neutron-lib for the MechanismDriver baseclass | 15:07 |
| seba | also it looks like nobody is starting workers in uwsgi / rpc split mode, is that right? I found a patch in our infra explicitly changing start_rpc_workers() to start_all_workers(), which also starts plugin workers | 15:10 |
| seba | I can open bug reports for both of these things, I just wanted to feel the waters beforehand :) | 15:11 |
| seba | okay, it looks like the workers are started alongside the api... I had expected them to run inside the neutron-rpc-server | 15:24 |
| opendevreview | Merged openstack/neutron-vpnaas master: SQLAlchemy 2.0: Drop use of removed autoload https://review.opendev.org/c/openstack/neutron-vpnaas/+/915838 | 15:42 |
| opendevreview | Brian Haley proposed openstack/neutron master: Put monitors dictionary back in MetadataDriverBase class https://review.opendev.org/c/openstack/neutron/+/918674 | 15:49 |
| opendevreview | Merged openstack/networking-bgpvpn unmaintained/zed: [CI] Remove tripleo-ci-centos-8-scenario004-standalone job https://review.opendev.org/c/openstack/networking-bgpvpn/+/918632 | 16:06 |
| opendevreview | Merged openstack/neutron master: tests: Don't use neutron object where db model is expected https://review.opendev.org/c/openstack/neutron/+/918493 | 18:59 |
| opendevreview | Merged openstack/neutron stable/2023.2: [stable/2023.2 only] Update grenade_from_branch to unmaintained/zed https://review.opendev.org/c/openstack/neutron/+/918378 | 18:59 |
| opendevreview | Brian Haley proposed openstack/neutron master: Put monitors dictionary back in MetadataDriverBase class https://review.opendev.org/c/openstack/neutron/+/918674 | 21:32 |
| opendevreview | Merged openstack/neutron stable/2023.2: Don't update revision number if object was not modified https://review.opendev.org/c/openstack/neutron/+/918642 | 21:51 |
| opendevreview | Merged openstack/neutron stable/2023.1: Don't update revision number if object was not modified https://review.opendev.org/c/openstack/neutron/+/918643 | 23:05 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!