| opendevreview | Merged openstack/neutron stable/xena: ovn: first tear down old metadata namespaces, then deploy new https://review.opendev.org/c/openstack/neutron/+/864979 | 00:06 |
|---|---|---|
| opendevreview | Merged openstack/neutron stable/victoria: ovn: first tear down old metadata namespaces, then deploy new https://review.opendev.org/c/openstack/neutron/+/865064 | 01:04 |
| opendevreview | Merged openstack/neutron stable/xena: Check subnet overlapping after add router interface https://review.opendev.org/c/openstack/neutron/+/864297 | 02:27 |
| *** ministry is now known as __ministry | 03:31 | |
| opendevreview | Merged openstack/neutron master: Add tempest job which will run on the oldest Ubuntu release https://review.opendev.org/c/openstack/neutron/+/862492 | 04:44 |
| opendevreview | Dr. Jens Harbott proposed openstack/networking-midonet stable/train: Replace "x/tap-as-a-service" with "openstack/tap-as-a-service" https://review.opendev.org/c/openstack/networking-midonet/+/864861 | 06:10 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: db: add segment_index to the unique constraint https://review.opendev.org/c/openstack/neutron/+/841459 | 07:43 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: segment: enable multisegments support for host https://review.opendev.org/c/openstack/neutron/+/841838 | 07:43 |
| opendevreview | Merged openstack/neutron master: Always create a "router_extra_attributes" register per router https://review.opendev.org/c/openstack/neutron/+/864051 | 08:05 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: [S-RBAC] Allow admin user to do all API requests by default https://review.opendev.org/c/openstack/neutron/+/865040 | 08:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Avoid race condition when deleting trunk bridges https://review.opendev.org/c/openstack/neutron/+/837780 | 09:02 |
| ralonsoh | hi all ^ this patch should fix the issue with trunk ports and live migration | 09:03 |
| ralonsoh | I'll push another one on top of it to check several times the tempest multinode job | 09:03 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DNM] Testing patch https://review.opendev.org/c/openstack/neutron/+/865039 | 09:05 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-midonet stable/victoria: Replace "x/tap-as-a-service" with "openstack/tap-as-a-service" https://review.opendev.org/c/openstack/networking-midonet/+/864867 | 09:24 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Add pylint to pep8 and adopt code to the results https://review.opendev.org/c/openstack/tap-as-a-service/+/865187 | 09:28 |
| ralonsoh | slaweq, https://review.opendev.org/c/openstack/neutron/+/862492 | 09:45 |
| ralonsoh | good stuff | 09:45 |
| opendevreview | Sam Morrison proposed openstack/neutron master: [OVN] Add new ML2 extension "dhcp_agent_scheduler" https://review.opendev.org/c/openstack/neutron/+/865081 | 09:53 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: Always create a "router_extra_attributes" register per router https://review.opendev.org/c/openstack/neutron/+/865380 | 09:56 |
| ralonsoh | lajoskatona, https://review.opendev.org/q/project:openstack%252Fnetworking-midonet+status:open if you have some mins | 09:58 |
| ralonsoh | I've also sent a mail to ask for mantainers | 09:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: Always create a "router_extra_attributes" register per router https://review.opendev.org/c/openstack/neutron/+/865387 | 10:36 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Always create a "router_extra_attributes" register per router https://review.opendev.org/c/openstack/neutron/+/865389 | 10:40 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: Always create a "router_extra_attributes" register per router https://review.opendev.org/c/openstack/neutron/+/865412 | 10:47 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Avoid race condition when deleting trunk bridges https://review.opendev.org/c/openstack/neutron/+/837780 | 10:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DNM] Testing patch https://review.opendev.org/c/openstack/neutron/+/865039 | 10:55 |
| ralonsoh | slaweq, lajoskatona hi! are you working on the MacvtapAgentTestCase.test_get_all_devices issue? | 10:56 |
| ralonsoh | https://bugs.launchpad.net/neutron/+bug/1997341 | 10:56 |
| mnasiadka | hello | 11:21 |
| mnasiadka | ralonsoh: unfortunately I'm coming back with https://bugs.launchpad.net/neutron/+bug/1995078/comments/16 ;-) | 11:21 |
| ralonsoh | mnasiadka, what is the traffic path? if I'm correct, what you are testing is those baremetal servers using the router and routing this trafifc though the GW | 11:25 |
| mnasiadka | ralonsoh: yes, two baremetal servers in different VLANs routed through GW | 11:25 |
| mnasiadka | ralonsoh: if the traffic is between bare metal and VM in different VLANs/Geneve networks - then the traffic is ,,capped'' in one direction | 11:26 |
| ralonsoh | what GW chassis is using the baremetal network? | 11:27 |
| ralonsoh | is the same as the other VLAN network? | 11:27 |
| ralonsoh | did you try to set only one HA node? I mean, defining one single chassis as GW | 11:29 |
| ralonsoh | to avoid what you are explaining in the BZ | 11:29 |
| ralonsoh | that the external port HA chassis group is different from the baremetal VLAN network | 11:29 |
| mnasiadka | ralonsoh: now it's the same, because we changed all ha_chassis_groups priorities to match gateway_chassis - so without setting ovn-chassis-mac-mappings on the network nodes it works | 11:31 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM test "test_get_all_devices" concurrency 1 https://review.opendev.org/c/openstack/neutron/+/865414 | 11:32 |
| mnasiadka | ralonsoh: but this is not how Neutron is setting those up | 11:32 |
| ralonsoh | ok, maybe we need to sync both GW schedulers | 11:33 |
| ralonsoh | mnasiadka, but that's ok, you don't want to make the internal VLAN network distributed | 11:37 |
| ralonsoh | you want all traffic of the VLAN network, that has the baremetal ports, to be centralized | 11:38 |
| ralonsoh | this is why you should not define "ovn-chassis-mac-mappings" | 11:38 |
| ralonsoh | just to be sure | 11:38 |
| ralonsoh | you have (1) an external network (for internet) and (2) a private VLAN network for baremetal ports | 11:38 |
| ralonsoh | in this deployment, you don't want distributed traffic, all baremetal ports will route though the GW chassis | 11:39 |
| ralonsoh | that is not distributed | 11:39 |
| ralonsoh | mnasiadka, hi? | 11:50 |
| mnasiadka | ralonsoh: right, I don't need it distributed - I think the ha_chassis_group priorities are need to be lined up with gateway_chassis priorities for HA to work (if one network node dies - to be failed over to a second one) | 12:03 |
| ralonsoh | mnasiadka, ok, I'll ask OVN folks to check this and see if we can align both scheduling methods | 12:04 |
| mnasiadka | ralonsoh: I think numans replied to my mail on ovs-discuss that they should be aligned, but he doesn't remember why ;-) | 12:04 |
| mnasiadka | ralonsoh: https://mail.openvswitch.org/pipermail/ovs-discuss/2022-October/052068.html | 12:06 |
| ralonsoh | I'll check it too | 12:07 |
| mnasiadka | Thanks | 12:14 |
| opendevreview | Merged openstack/os-ken master: Enforce new msgpack format (use_bin_type=True) https://review.opendev.org/c/openstack/os-ken/+/864868 | 12:15 |
| opendevreview | Merged openstack/os-ken master: Add msgpack 1.0 support https://review.opendev.org/c/openstack/os-ken/+/864869 | 12:16 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Disable "test_live_migration_with_trunk" execution from dvr job https://review.opendev.org/c/openstack/neutron/+/865424 | 12:17 |
| slaweq | ralonsoh lajoskatona hi ^^ quick follow up after ralonsoh's patch :) | 12:17 |
| slaweq | please review when You will have some time | 12:17 |
| opendevreview | Merged openstack/neutron-lib stable/zed: Added "qos" plugin to "EXT_PARENT_RESOURCE_MAPPING" https://review.opendev.org/c/openstack/neutron-lib/+/864735 | 12:18 |
| lajoskatona | slaweq: ack | 12:18 |
| ralonsoh | slaweq, please check https://review.opendev.org/c/openstack/neutron/+/837780/14 first | 12:18 |
| ralonsoh | this is solving the issue | 12:18 |
| ralonsoh | and reverting the patch I sent yesterday | 12:18 |
| slaweq | ralonsoh ok, great :) | 12:18 |
| ralonsoh | and DNM patch testing it: https://review.opendev.org/c/openstack/neutron/+/865039/ | 12:18 |
| slaweq | I will review it today | 12:18 |
| ralonsoh | CI is happy so far | 12:18 |
| ralonsoh | hmmm no, it is still failing in OVN, I need to check what is happening there | 12:21 |
| opendevreview | Merged openstack/os-ken master: Update nx_actions.py, fixing a typo on example https://review.opendev.org/c/openstack/os-ken/+/864870 | 12:24 |
| lajoskatona | sahid: Hi, I saw "Mapping VLAN for network <net-UUID> with segmentation id <all> not found." exceptions in ovs-agent logs | 12:46 |
| lajoskatona | sahid: sorry for long link: https://opensearch.logs.openstack.org/_dashboards/app/discover/?security_tenant=global#/?_g=(filters:!(),refreshInterval:(pause:!t,value:0),time:(from:now-30d,to:now))&_a=(columns:!(build_status),filters:!(),index:'94869730-aea8-11ec-9e6a-83741af3fdcd',interval:auto,query:(language:kuery,query:'message:%22Mapping%20VLAN%20for%20network%20%22'),sort:!()) | 12:46 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Execute functional test "test_get_all_devices" alone https://review.opendev.org/c/openstack/neutron/+/865414 | 12:53 |
| opendevreview | Merged openstack/networking-midonet stable/victoria: Replace "x/tap-as-a-service" with "openstack/tap-as-a-service" https://review.opendev.org/c/openstack/networking-midonet/+/864867 | 13:38 |
| opendevreview | Merged openstack/networking-midonet stable/ussuri: Replace "x/tap-as-a-service" with "openstack/tap-as-a-service" https://review.opendev.org/c/openstack/networking-midonet/+/864866 | 13:44 |
| opendevreview | Merged openstack/networking-midonet stable/train: Replace "x/tap-as-a-service" with "openstack/tap-as-a-service" https://review.opendev.org/c/openstack/networking-midonet/+/864861 | 13:45 |
| *** dasm|off is now known as dasm | 13:59 | |
| ralonsoh | folks, if you have 1 min: https://review.opendev.org/c/openstack/neutron/+/865414 | 14:05 |
| ralonsoh | CI FT job is passing | 14:05 |
| lajoskatona | ralonsoh: cool, so this is again some concurrency issue.... | 14:26 |
| ralonsoh | I think so, yes | 14:26 |
| lajoskatona | as I wrote to the LP, I was not able to reproduce locally, but my env where I run functional is not that big (I even played with it on jammy vs focal) | 14:27 |
| opendevreview | Merged openstack/neutron stable/zed: ovn: first tear down old metadata namespaces, then deploy new https://review.opendev.org/c/openstack/neutron/+/864977 | 16:05 |
| opendevreview | Merged openstack/neutron stable/yoga: Check subnet overlapping after add router interface https://review.opendev.org/c/openstack/neutron/+/863275 | 16:19 |
| gmann | lajoskatona: slaweq: on RBAC fix, it seems we need to skip trunk live migration test in neutron-ovs-tempest-dvr-ha-multinode-full job also | 16:57 |
| gmann | https://review.opendev.org/c/openstack/neutron/+/865040/ is failing with that | 16:58 |
| gmann | lajoskatona: slaweq : never mind, slaweq is always ahead on things :) https://review.opendev.org/c/openstack/neutron/+/865424 | 17:00 |
| lajoskatona | gmann: thanks for reminding, I forgot to vote on that | 17:01 |
| gmann | thanks | 17:02 |
| opendevreview | Anton Vazhnetsov proposed openstack/ovsdbapp master: ovs_idl: provide 'update_timeout' https://review.opendev.org/c/openstack/ovsdbapp/+/865454 | 17:35 |
| opendevreview | Anton Vazhnetsov proposed openstack/ovsdbapp master: ovs_idl: provide 'update_timeout' https://review.opendev.org/c/openstack/ovsdbapp/+/865454 | 17:44 |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron master: Exclude neutron-dynamic-routing from abandon script https://review.opendev.org/c/openstack/neutron/+/865462 | 19:09 |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron master: neutron-dynamic-routing now works with OVN https://review.opendev.org/c/openstack/neutron/+/738641 | 19:26 |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron-tempest-plugin master: Test neutron-dynamic-routing on OVN https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/865464 | 19:49 |
| frickler | slaweq: tobias-urdin: bcafarel: ^^ couple of things for you to look at. I don't think we need to keep running a second for for ndr on OVS, do you agree? | 19:50 |
| frickler | also regarding midonet, I wonder if https://opendev.org/openstack/neutron-tempest-plugin/src/branch/master/neutron_tempest_plugin/neutron_dynamic_routing/scenario/test_simple_bgp.py can be dropped, I don't think we have any CI coverage for that | 19:51 |
| opendevreview | Merged openstack/neutron-lib stable/yoga: Added "qos" plugin to "EXT_PARENT_RESOURCE_MAPPING" https://review.opendev.org/c/openstack/neutron-lib/+/864736 | 19:57 |
| opendevreview | Merged openstack/neutron stable/wallaby: Check subnet overlapping after add router interface https://review.opendev.org/c/openstack/neutron/+/863852 | 19:59 |
| opendevreview | Merged openstack/neutron stable/wallaby: ovn: first tear down old metadata namespaces, then deploy new https://review.opendev.org/c/openstack/neutron/+/864980 | 19:59 |
| opendevreview | Merged openstack/neutron master: Execute functional test "test_get_all_devices" alone https://review.opendev.org/c/openstack/neutron/+/865414 | 19:59 |
| opendevreview | Merged openstack/neutron master: ovn: inline ensure_all_networks_provisioned method https://review.opendev.org/c/openstack/neutron/+/865183 | 19:59 |
| opendevreview | Merged openstack/neutron master: ovn: use dicts instead of sets of tuples https://review.opendev.org/c/openstack/neutron/+/865184 | 19:59 |
| opendevreview | Merged openstack/neutron master: Disable "test_live_migration_with_trunk" execution from dvr job https://review.opendev.org/c/openstack/neutron/+/865424 | 19:59 |
| bcafarel | frickler: nice to see these! and also +1 on just keeping OVN (default mech driver) for testing | 20:28 |
| opendevreview | Merged openstack/neutron master: Make MechanismDriverContext plugin_context public https://review.opendev.org/c/openstack/neutron/+/864855 | 20:33 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Drop unused branch_override https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/864995 | 20:33 |
| opendevreview | Merged openstack/neutron master: Code move for metadata haproxy https://review.opendev.org/c/openstack/neutron/+/864648 | 20:33 |
| opendevreview | Merged openstack/neutron stable/yoga: Avoid register config options on imports https://review.opendev.org/c/openstack/neutron/+/865333 | 20:33 |
| opendevreview | Merged openstack/neutron stable/yoga: ovn: Use ovsdb-client to create neutron_pg_drop https://review.opendev.org/c/openstack/neutron/+/865159 | 20:33 |
| opendevreview | Anton Kurbatov proposed openstack/neutron master: Add a DHCP provisioning block for the default device_owner https://review.opendev.org/c/openstack/neutron/+/865470 | 20:34 |
| opendevreview | Merged openstack/neutron-lib stable/wallaby: Added "qos" plugin to "EXT_PARENT_RESOURCE_MAPPING" https://review.opendev.org/c/openstack/neutron-lib/+/864739 | 20:51 |
| opendevreview | Merged openstack/neutron stable/wallaby: Avoid register config options on imports https://review.opendev.org/c/openstack/neutron/+/865344 | 21:31 |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron master: Exclude neutron-dynamic-routing from abandon script https://review.opendev.org/c/openstack/neutron/+/865462 | 21:40 |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron-dynamic-routing master: Add oslo.log to config generator https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/865473 | 21:45 |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron master: neutron-dynamic-routing now works with OVN https://review.opendev.org/c/openstack/neutron/+/738641 | 21:57 |
| opendevreview | Merged openstack/neutron stable/zed: Always create a "router_extra_attributes" register per router https://review.opendev.org/c/openstack/neutron/+/865380 | 22:09 |
| *** dasm is now known as dasm|off | 23:49 | |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [WIP][DNM][OVN] Change the default firewall policy https://review.opendev.org/c/openstack/neutron/+/839066 | 23:56 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!