| opendevreview | liuyulong proposed openstack/neutron master: Add meter bandwidth bandwidth support https://review.opendev.org/c/openstack/neutron/+/860767 | 01:31 |
|---|---|---|
| opendevreview | Merged openstack/neutron master: Update tap ip in metadata agent when metadata port ip updated https://review.opendev.org/c/openstack/neutron/+/864715 | 06:31 |
| slaweq | ralonsoh lajoskatona ykarel hi, can You check https://review.opendev.org/c/openstack/neutron/+/865040 when You will have few minutes? I think it's ready to review now | 08:15 |
| tobias-urdin | frickler: ack | 08:19 |
| tobias-urdin | nice work on ndr ovn support frickler and ralonsoh | 08:19 |
| lajoskatona | slaweq: ack | 08:20 |
| elvira | slaweq: ralonsoh: I answered your comments in https://review.opendev.org/c/openstack/neutron/+/864152 . Whenever you have time, no rush :) | 08:22 |
| ralonsoh | slaweq, let me check | 08:33 |
| ralonsoh | elvira, suer | 08:33 |
| ralonsoh | sure | 08:33 |
| ralonsoh | elvira, https://review.opendev.org/c/openstack/neutron/+/864152/3..4/neutron/services/logapi/drivers/ovn/driver.py#329 | 08:42 |
| ralonsoh | slaweq and I did the same comment | 08:42 |
| ykarel | slaweq, ack | 08:42 |
| ralonsoh | you can check that before the loop | 08:42 |
| elvira | ralonsoh: but in the comment I explain why I make the check there. Because I remove entries so the list can become empty (and therefore return false) during the execution of the loop | 08:48 |
| ralonsoh | elvira, yeah, now I see it | 08:48 |
| ralonsoh | I'm writing a comment | 08:48 |
| ralonsoh | now I understand | 08:49 |
| elvira | thanks a lot! I'm totally open to any idea that makes this more readable | 08:50 |
| lajoskatona | slaweq: I have also some CI related patches: https://review.opendev.org/c/openstack/networking-bgpvpn/+/861960 & https://review.opendev.org/c/openstack/tap-as-a-service/+/862109 & https://review.opendev.org/c/openstack/networking-sfc/+/862102 & https://review.opendev.org/c/openstack/neutron-vpnaas/+/862108 & https://review.opendev.org/c/openstack/networking-bgpvpn/+/864397 | 08:54 |
| lajoskatona | slaweq: these are easy just adding py39 to periodic mostly :-) | 08:55 |
| lajoskatona | slaweq: perhaps you know who can vote on this one: https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/862286 ? this is to add py310 templates with n-lib and oslo-master | 08:56 |
| 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 | 09:17 |
| 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 | 09:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: "convert_to_sanitized_mac_address" accepts netaddr.EUI type values https://review.opendev.org/c/openstack/neutron-lib/+/865517 | 09:42 |
| opendevreview | Merged openstack/neutron-vpnaas master: CI: Add openstack-tox-py39-with-oslo-master to periodic weekly queue https://review.opendev.org/c/openstack/neutron-vpnaas/+/862108 | 09:42 |
| opendevreview | Felix Huettner proposed openstack/neutron stable/yoga: Since OVN 20.06, config is stored in "Chassis.other_config" https://review.opendev.org/c/openstack/neutron/+/865135 | 09:47 |
| opendevreview | Luca Czesla proposed openstack/neutron master: Add address scope to the OVN LSP port registers https://review.opendev.org/c/openstack/neutron/+/861719 | 10:04 |
| opendevreview | Merged openstack/neutron master: db: add segment_index to the unique constraint https://review.opendev.org/c/openstack/neutron/+/841459 | 10:23 |
| opendevreview | Maximilian Stinsky proposed openstack/neutron master: Fix duplicated routes exceptions https://review.opendev.org/c/openstack/neutron/+/865525 | 10:49 |
| opendevreview | Maximilian Stinsky proposed openstack/neutron master: Add function to refresh accept_ra on router state change https://review.opendev.org/c/openstack/neutron/+/824947 | 11:16 |
| opendevreview | Maximilian Stinsky proposed openstack/neutron master: Fix duplicated routes exceptions https://review.opendev.org/c/openstack/neutron/+/865525 | 11:44 |
| opendevreview | Maximilian Stinsky proposed openstack/neutron master: Fix duplicated routes exceptions https://review.opendev.org/c/openstack/neutron/+/865525 | 12:00 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Fix behaviour of enable/disable in OVN network log https://review.opendev.org/c/openstack/neutron/+/864152 | 12:06 |
| *** tbachman_ is now known as tbachman | 12:11 | |
| opendevreview | Merged openstack/neutron stable/xena: Avoid register config options on imports https://review.opendev.org/c/openstack/neutron/+/865341 | 12:17 |
| opendevreview | Felix Huettner proposed openstack/neutron stable/yoga: Since OVN 20.06, config is stored in "Chassis.other_config" https://review.opendev.org/c/openstack/neutron/+/865135 | 12:18 |
| opendevreview | Maximilian Stinsky proposed openstack/neutron master: Fix duplicated routes exceptions https://review.opendev.org/c/openstack/neutron/+/865525 | 12:21 |
| lajoskatona | sahid: Hi, pushed tempest test for the new segment thing: https://review.opendev.org/c/openstack/tempest/+/665155 | 12:24 |
| lajoskatona | sahid: I have to reach out for QA team as I have no idea in which job these should run, and how to enable all things to be set in local.conf for it | 12:25 |
| lajoskatona | sahid: anoter thing which is more or less independent of your current work is that it seems that the resource allocation is not working (like I use IP from the subnet but it is not decreased in placement db) | 12:26 |
| lajoskatona | sahid: if you have time to check what I misunderstood there would be really good | 12:27 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Configure placement credentials for segments plugin https://review.opendev.org/c/openstack/neutron/+/815093 | 13:09 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Routed provider nets: Fix subnet allocation logic https://review.opendev.org/c/openstack/neutron/+/777443 | 13:09 |
| sahid | lajoskatona: oh great! | 13:40 |
| sahid | lajoskatona: hum that is interesting... this behavior happens on both subnets? | 13:41 |
| ralonsoh | hello all, please propose your topics for the Neutron drivers meeting | 14:26 |
| ralonsoh | agenda: https://wiki.openstack.org/wiki/Meetings/NeutronDrivers | 14:26 |
| ralonsoh | the meeting is tomorrow at 14UTC | 14:26 |
| *** ykarel is now known as ykarel|away | 15:23 | |
| lajoskatona | sahid: sorry, meetings.... | 15:27 |
| lajoskatona | sahid: yeah I think on all subnets, but as I see my old patch it happened before your patches also: https://review.opendev.org/c/openstack/neutron/+/777443 | 15:28 |
| lajoskatona | sahid; I have to check what I did when I proposed originally this patch, as it was so long time ago :-) | 15:29 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Avoid race condition when deleting trunk bridges https://review.opendev.org/c/openstack/neutron/+/837780 | 15:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DNM] Testing patch https://review.opendev.org/c/openstack/neutron/+/865039 | 15:33 |
| opendevreview | Bence Romsics proposed openstack/neutron master: Bump pecan to fix #1829042 https://review.opendev.org/c/openstack/neutron/+/865575 | 15:35 |
| sahid | lajoskatona: no worries I was on meeting as-well :-) | 15:38 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Test sha1 pylint error https://review.opendev.org/c/openstack/neutron-lib/+/865578 | 15:52 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Use a secure hash method: replace "sha1" with "sha256" https://review.opendev.org/c/openstack/neutron-lib/+/865578 | 16:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: "convert_to_sanitized_mac_address" accepts netaddr.EUI type values https://review.opendev.org/c/openstack/neutron-lib/+/865517 | 16:02 |
| ralonsoh | slaweq, we could have another fire in the CI | 16:17 |
| ralonsoh | https://3e7cdff9e56ac3733874-d434493bb46e5d37fd5df3f066caeb22.ssl.cf5.rackcdn.com/837780/16/check/tempest-integrated-networking/2acc1d8/controller/logs/screen-q-svc.txt | 16:17 |
| ralonsoh | check this https://zuul.opendev.org/t/openstack/status#837780 | 16:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove deprecation warning of "ovs-ofctl" usage https://review.opendev.org/c/openstack/neutron/+/865588 | 17:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use the neutron-lib constant for PACKET_RATE_LIMIT table number https://review.opendev.org/c/openstack/neutron/+/865590 | 17:26 |
| 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 | 17:38 |
| opendevreview | Merged openstack/neutron-lib stable/xena: Added "qos" plugin to "EXT_PARENT_RESOURCE_MAPPING" https://review.opendev.org/c/openstack/neutron-lib/+/864738 | 18:24 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [WIP][DNM][OVN] Change the default firewall policy https://review.opendev.org/c/openstack/neutron/+/839066 | 19:16 |
| opendevreview | Merged openstack/neutron master: [S-RBAC] Allow admin user to do all API requests by default https://review.opendev.org/c/openstack/neutron/+/865040 | 20:05 |
| *** prometheanfire is now known as Guest0 | 20:53 | |
| opendevreview | Maximilian Stinsky proposed openstack/neutron master: Fix duplicated routes exceptions https://review.opendev.org/c/openstack/neutron/+/865525 | 22:13 |
| *** Guest0 is now known as prometheanfire | 22:50 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!