| opendevreview | liuyulong proposed openstack/neutron master: Add meter bandwidth limit support https://review.opendev.org/c/openstack/neutron/+/860767 | 01:03 |
|---|---|---|
| opendevreview | Miguel Lavalle proposed openstack/neutron master: Router flavors and service type for OVN https://review.opendev.org/c/openstack/neutron/+/883988 | 02:37 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: [WIP] DHCP agent get ovn dhcp port from server https://review.opendev.org/c/openstack/neutron/+/903572 | 02:50 |
| opendevreview | Takashi Kajinami proposed openstack/neutron master: Remove unused metadata options from ovs agent https://review.opendev.org/c/openstack/neutron/+/902014 | 05:52 |
| opendevreview | Merged openstack/neutron stable/2023.2: [OVN] Add a log message after the "post_fork_initialize" method https://review.opendev.org/c/openstack/neutron/+/903332 | 06:42 |
| opendevreview | Merged openstack/neutron stable/2023.1: [OVN] Add a log message after the "post_fork_initialize" method https://review.opendev.org/c/openstack/neutron/+/903333 | 06:43 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3 scheduler: calculate load of chassis per priority https://review.opendev.org/c/openstack/neutron/+/893653 | 06:59 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: reschedule lower priorities https://review.opendev.org/c/openstack/neutron/+/893654 | 06:59 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: reschedule priorities on new chassis https://review.opendev.org/c/openstack/neutron/+/893655 | 06:59 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3 router scheduler: reproduce scheduling issue https://review.opendev.org/c/openstack/neutron/+/893656 | 06:59 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: implement caching for Scheduler https://review.opendev.org/c/openstack/neutron/+/893657 | 06:59 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: try to keep routers at current chassis https://review.opendev.org/c/openstack/neutron/+/893658 | 06:59 |
| opendevreview | Felix Huettner proposed openstack/neutron master: ovn-l3: value the higher priorities when scheduling https://review.opendev.org/c/openstack/neutron/+/893659 | 06:59 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Ensure NDB Proxy gets added for provider IPs too https://review.opendev.org/c/openstack/ovn-bgp-agent/+/903402 | 07:25 |
| opendevreview | Merged openstack/neutron master: Remove vnic_type_blacklist deprecated name https://review.opendev.org/c/openstack/neutron/+/901478 | 07:48 |
| opendevreview | Merged openstack/ovn-bgp-agent master: Python 3.12: use importlib instead of imp https://review.opendev.org/c/openstack/ovn-bgp-agent/+/903505 | 07:51 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][FT] Make explicit the "publish" call check in "test_port_forwarding" https://review.opendev.org/c/openstack/neutron/+/903579 | 08:11 |
| *** jph5 is now known as jph1 | 08:19 | |
| *** jph1 is now known as jph | 08:20 | |
| slaweq | ralonsoh lajoskatona ykarel hi, can You check once again https://review.opendev.org/c/openstack/neutron/+/892542 ? It had 2x+2 already but I had to rebase it to resolve some conflicts | 08:39 |
| ralonsoh | sure, let me check | 08:43 |
| *** elodilles_pto is now known as elodilles | 09:06 | |
| opendevreview | huanghailun proposed openstack/neutron master: Add allowed_address_pairs add/remove atomic operations https://review.opendev.org/c/openstack/neutron/+/901895 | 09:09 |
| ykarel | slaweq, ack | 09:27 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: Fix IGMP inconsistency across drivers https://review.opendev.org/c/openstack/neutron/+/901753 | 10:00 |
| opendevreview | huanghailun proposed openstack/neutron master: Add allowed_address_pairs add/remove atomic operations https://review.opendev.org/c/openstack/neutron/+/901895 | 10:01 |
| opendevreview | Merged openstack/ovn-bgp-agent master: Ensure NDB Proxy gets added for provider IPs too https://review.opendev.org/c/openstack/ovn-bgp-agent/+/903402 | 10:06 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent stable/2023.2: Ensure NDB Proxy gets added for provider IPs too https://review.opendev.org/c/openstack/ovn-bgp-agent/+/903546 | 10:12 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/2023.1: [OVN] Enhanced external port scheduling https://review.opendev.org/c/openstack/neutron/+/903510 | 10:13 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Fix UT assertion https://review.opendev.org/c/openstack/neutron/+/903584 | 10:22 |
| opendevreview | Merged openstack/ovn-bgp-agent master: [UT] Fix the ``TestLinuxNet`` wrong assert calls https://review.opendev.org/c/openstack/ovn-bgp-agent/+/903511 | 10:39 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent stable/2023.2: [UT] Fix the ``TestLinuxNet`` wrong assert calls https://review.opendev.org/c/openstack/ovn-bgp-agent/+/903548 | 10:49 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Add the bridge name to the port VIF details https://review.opendev.org/c/openstack/neutron/+/903494 | 10:59 |
| ralonsoh | sean-k-mooney, hello! ^^ please check that (in particular the release note --> we can't populate these parameters in a OVS->OVN migration) | 11:00 |
| ralonsoh | solutions? set br-int/system by default? | 11:00 |
| ralonsoh | feedback is welcome | 11:00 |
| ralonsoh | lucasagomes, https://review.opendev.org/c/openstack/neutron/+/901753/comment/94153ba6_a0346a4f/. Just a comment, to avoid possible upgrade problems in those users with "igmp_snooping_enable=True" that didn't upgrade the config files | 11:32 |
| lucasagomes | ralonsoh, will take a look | 11:34 |
| lucasagomes | ralonsoh, by upgrade check you mean improving the release note ? Or some code somewhere ? | 11:37 |
| ralonsoh | lucasagomes, code in the upgrade_checks.checks.CoreChecks | 11:43 |
| ralonsoh | but this is just a suggestion, to avoid problems in future releases | 11:44 |
| lucasagomes | ralonsoh, I see, it's a good thing. I will take a look at it | 11:44 |
| lucasagomes | Was unfamiliar with such checks | 11:45 |
| ralonsoh | thanks! | 11:45 |
| sean-k-mooney | ralonsoh: if datapath is not set then the migration will break for ovn with dpdk or windows. but ya the solution would be to default to br-int/system and perhaps enhance the migration script to takek the datapath as a paramater | 11:47 |
| ralonsoh | sean-k-mooney, that datapath should be something global for all ports | 11:47 |
| sean-k-mooney | not all ports | 11:48 |
| ralonsoh | for bound ports | 11:48 |
| sean-k-mooney | but all ports on a given hosts | 11:48 |
| sean-k-mooney | you might have diffent datapaths on diffent hosts | 11:48 |
| ralonsoh | I was going to ask this | 11:48 |
| ralonsoh | so the problem is how to provide this info per host | 11:49 |
| sean-k-mooney | you can have hardware offloaed ovs, kernel ovs and dpdk all in the same cloud | 11:49 |
| ralonsoh | yeah | 11:49 |
| sean-k-mooney | dont we have this avaiabel in ovn? | 11:49 |
| ralonsoh | no, this is a local OVS configuration | 11:49 |
| sean-k-mooney | i see | 11:49 |
| sean-k-mooney | that proably is an over sight but that does not help here. the issue is the only thing that knows this normally is neutrion via the l2 agent config option | 11:50 |
| sean-k-mooney | https://docs.openstack.org/neutron/latest/configuration/openvswitch-agent.html#ovs.datapath_type | 11:51 |
| ralonsoh | I need to check if at this point (when executing the "neutron-ovn-db-sync-util" script, the compute nodes have been migrated | 11:51 |
| sean-k-mooney | that obviously is not used with ovn although you could set that i guess in the ovs db liek the other options like bridge mappings | 11:52 |
| ralonsoh | that means we have the metadata agent running. If that is the case (and with the code added in this patch), we have the DP type | 11:52 |
| sean-k-mooney | oh ok | 11:53 |
| sean-k-mooney | i havent really looked at the ovn metadta agent but i guess that ueses the same config values as the l2 agent did? | 11:53 |
| sean-k-mooney | if so then ya you can jsut use this | 11:53 |
| ralonsoh | yes, it reads the br-int bridge datapath_type | 11:54 |
| ralonsoh | and writes this info the corresponding chassis_private.external_ids["neutron:datapath_type"] | 11:55 |
| ralonsoh | **this info in the | 11:55 |
| sean-k-mooney | ok then for migration you just need to start the metadta agent first before migrating any ports | 11:56 |
| sean-k-mooney | we can also have a fallback to br-int/system by default in the code if that info is not avaiable | 11:56 |
| sean-k-mooney | so i think that will work in general | 11:57 |
| ralonsoh | yes but the metadata in the compute nodes | 11:57 |
| ralonsoh | and I think at this point I don't know if we have the compute nodes migrated | 11:57 |
| ralonsoh | I need to check that with jlibosva | 11:57 |
| sean-k-mooney | i see. so the migration scrip is direclly modifying the db right | 11:57 |
| sean-k-mooney | its not using the normal port bining logic | 11:58 |
| ralonsoh | yes, the Neutron one | 11:58 |
| ralonsoh | nope | 11:58 |
| sean-k-mooney | ya ok i was going to say there is an agent health check to confrim its up in the normal flow | 11:58 |
| sean-k-mooney | that woudl at least need the chassi to be registered in ovn at that point | 11:58 |
| sean-k-mooney | but since your modifying the db that check wont exist | 11:58 |
| sean-k-mooney | so it might now be installed yet | 11:59 |
| opendevreview | Merged openstack/neutron-lib stable/2023.1: Add the "cancellable" flag to the ``CallbacksManager`` events https://review.opendev.org/c/openstack/neutron-lib/+/903335 | 12:30 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: [DNM] Testing error in "neutron-functional-with-uwsgi-fips" stable/yoga https://review.opendev.org/c/openstack/neutron/+/903592 | 13:05 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2023.1: Change device_owner used for OVN LB HM port https://review.opendev.org/c/openstack/ovn-octavia-provider/+/901180 | 13:16 |
| opendevreview | Christian Berendt proposed openstack/neutron-lib stable/2023.1: vpnaas: add support for more ciphers (auth, encryption, pfs modes) https://review.opendev.org/c/openstack/neutron-lib/+/903551 | 14:43 |
| opendevreview | Christian Berendt proposed openstack/neutron-lib stable/2023.2: vpnaas: add support for more ciphers (auth, encryption, pfs modes) https://review.opendev.org/c/openstack/neutron-lib/+/903552 | 14:44 |
| SvenKieske | would be nice if we could speed up above backports, if these are acceptable as backports, that is. | 14:57 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: [WIP] DHCP agent get ovn dhcp port from server https://review.opendev.org/c/openstack/neutron/+/903572 | 15:15 |
| opendevreview | Merged openstack/neutron stable/2023.2: get_hosts_mapped_with_segments add filter agt_type https://review.opendev.org/c/openstack/neutron/+/901788 | 15:30 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Add the bridge name and datapath type to the port VIF details https://review.opendev.org/c/openstack/neutron/+/903494 | 16:19 |
| ralonsoh | SvenKieske, this is marked as a feature in the release note. We should not backport features | 16:21 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: [DNM] Testing error in "neutron-functional-with-uwsgi-fips" stable/yoga https://review.opendev.org/c/openstack/neutron/+/903592 | 16:31 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2023.1: Add maintenance task to update entities on component start https://review.opendev.org/c/openstack/ovn-octavia-provider/+/901181 | 17:27 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!