| opendevreview | Ihtisham ul Haq proposed openstack/neutron stable/yoga: Use explicit inner join for networks in port query https://review.opendev.org/c/openstack/neutron/+/883712 | 06:36 |
|---|---|---|
| opendevreview | Ihtisham ul Haq proposed openstack/neutron stable/zed: Use explicit inner join for networks in port query https://review.opendev.org/c/openstack/neutron/+/883713 | 06:37 |
| opendevreview | Ihtisham ul Haq proposed openstack/neutron stable/2023.1: Use explicit inner join for networks in port query https://review.opendev.org/c/openstack/neutron/+/883714 | 06:39 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: [ovn] Avoid unwanted ACL_NOT_FOUND error when deleting log objects https://review.opendev.org/c/openstack/neutron/+/883693 | 07:26 |
| opendevreview | Felix Huettner proposed openstack/neutron-dynamic-routing master: Ignore disabled routers for advertising https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/882560 | 08:12 |
| skraynev | Hi folks, could someone else please review backport patche for neutron: https://review.opendev.org/c/openstack/neutron/+/883422 (and backports to other branches) | 08:38 |
| opendevreview | Merged openstack/neutron master: Update pylint version https://review.opendev.org/c/openstack/neutron/+/883464 | 08:41 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Revert "Delete sg rule which remote is the deleted sg" https://review.opendev.org/c/openstack/neutron/+/883582 | 08:57 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-specs master: Default SG rules - update fields in the API examples https://review.opendev.org/c/openstack/neutron-specs/+/883481 | 09:40 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Add retry on case of sqlite3.InterfaceError on FT https://review.opendev.org/c/openstack/ovn-octavia-provider/+/883662 | 09:43 |
| opendevreview | Maximilian Sesterhenn proposed openstack/networking-bgpvpn master: WIP: Add OVN-based Neutron BGPVPN driver https://review.opendev.org/c/openstack/networking-bgpvpn/+/883060 | 09:50 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: Fix issues due to rcent RBAC changes and removal of neutron.debug https://review.opendev.org/c/openstack/networking-bagpipe/+/883064 | 10:36 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2023.1: Add retry on case of sqlite3.InterfaceError on FT https://review.opendev.org/c/openstack/ovn-octavia-provider/+/883801 | 10:46 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: rpc/handlers/dhcp: fix get_network_info when no local subnets https://review.opendev.org/c/openstack/neutron/+/880131 | 10:48 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/zed: Add retry on case of sqlite3.InterfaceError on FT https://review.opendev.org/c/openstack/ovn-octavia-provider/+/883802 | 10:57 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/yoga: Add retry on case of sqlite3.InterfaceError on FT https://review.opendev.org/c/openstack/ovn-octavia-provider/+/883803 | 10:57 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/xena: Add retry on case of sqlite3.InterfaceError on FT https://review.opendev.org/c/openstack/ovn-octavia-provider/+/883804 | 10:57 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/wallaby: Add retry on case of sqlite3.InterfaceError on FT https://review.opendev.org/c/openstack/ovn-octavia-provider/+/883805 | 10:57 |
| opendevreview | Merged openstack/os-vif stable/wallaby: Use TCP keepalives for ovsdb connections https://review.opendev.org/c/openstack/os-vif/+/841773 | 11:07 |
| opendevreview | Merged openstack/os-vif stable/wallaby: only register tables used by os-vif https://review.opendev.org/c/openstack/os-vif/+/841774 | 11:07 |
| ykarel | tobias-urdin, frickler when you get a chance can you please check what can cause those bgp peer disconnections https://bugs.launchpad.net/neutron/+bug/2020349 | 11:52 |
| opendevreview | Merged openstack/neutron master: [OVN] Use the API context in ``OVNClient._add_router_ext_gw`` method https://review.opendev.org/c/openstack/neutron/+/882857 | 12:23 |
| opendevreview | Merged openstack/neutron master: port-hint-ovs-tx-steering: agent side https://review.opendev.org/c/openstack/neutron/+/872905 | 12:23 |
| opendevreview | Merged openstack/neutron master: port-hint-ovs-tx-steering: shim extension https://review.opendev.org/c/openstack/neutron/+/873113 | 12:23 |
| opendevreview | Merged openstack/neutron stable/2023.1: Fix not working use_random_fully config option https://review.opendev.org/c/openstack/neutron/+/883422 | 12:26 |
| opendevreview | Merged openstack/neutron stable/zed: Fix not working use_random_fully config option https://review.opendev.org/c/openstack/neutron/+/883423 | 12:26 |
| opendevreview | Merged openstack/neutron stable/yoga: Fix not working use_random_fully config option https://review.opendev.org/c/openstack/neutron/+/883425 | 12:27 |
| opendevreview | Merged openstack/neutron stable/xena: Fix not working use_random_fully config option https://review.opendev.org/c/openstack/neutron/+/883424 | 12:27 |
| opendevreview | Merged openstack/neutron stable/wallaby: Fix not working use_random_fully config option https://review.opendev.org/c/openstack/neutron/+/883426 | 12:27 |
| opendevreview | Merged openstack/neutron stable/victoria: Fix not working use_random_fully config option https://review.opendev.org/c/openstack/neutron/+/883427 | 12:27 |
| *** dmellado90 is now known as dmellado | 12:48 | |
| skraynev | bcafarel: hi. thank you for review backports. as I see CI failed on train and ussuri https://review.opendev.org/c/openstack/neutron/+/883428 , will it be enough to write "recheck functional" for both? | 12:51 |
| opendevreview | Merged openstack/neutron stable/xena: [stable-only] Don't use "sqlalchemy.select" with SQLAlchemy<1.4 https://review.opendev.org/c/openstack/neutron/+/883288 | 12:51 |
| bcafarel | skraynev: for ussuri yes, we had Verified+1 before so recheck is OK - train it looks like the devstack POST_FAILURE is reproducible and needs to be fixed before (I will check logs later to confirm and open launchpad if that's the case) | 12:59 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: Fix issues due to rcent RBAC changes and removal of neutron.debug https://review.opendev.org/c/openstack/networking-bagpipe/+/883064 | 13:00 |
| skraynev | bcafarel: awesome! thank you | 13:01 |
| skraynev | bcafarel: will it trigger recheck, if I leave a comment: recheck functional , or it works only for core reviewers? | 13:02 |
| skraynev | regarding ussuri | 13:02 |
| opendevreview | Lajos Katona proposed openstack/neutron-tempest-plugin master: Adopt bgpvpn scenarios to recent sRBAC changes https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/883838 | 13:06 |
| opendevreview | Lajos Katona proposed openstack/networking-bgpvpn master: Fix issues due to rcent RBAC changes https://review.opendev.org/c/openstack/networking-bgpvpn/+/883057 | 13:07 |
| bcafarel | skraynev: from anyone so you can comment - recheck itself is the trigger keyword, after that we add a free-form reason text so not to have "bare rechecks" (people blindly rechecking while there may be a valid failure) | 13:07 |
| skraynev | ok | 13:09 |
| opendevreview | Lajos Katona proposed openstack/neutron-tempest-plugin master: Adopt bgpvpn scenarios to recent sRBAC changes https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/883838 | 13:10 |
| bcafarel | skraynev: and confirmed for train, the failing job looks broken - I filled https://bugs.launchpad.net/neutron/+bug/2020363 (once fixed the train backport will be able to go in) | 13:21 |
| opendevreview | Lajos Katona proposed openstack/python-neutronclient master: OSC: Remove BGPVPN calls to neutronclient https://review.opendev.org/c/openstack/python-neutronclient/+/875728 | 13:22 |
| skraynev | bcafarel: thank you ! | 13:27 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Bump neutron-lib to 3.6.1 https://review.opendev.org/c/openstack/neutron/+/883483 | 13:29 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Revert "Use a writer context for the online alembic migrations" https://review.opendev.org/c/openstack/neutron/+/883120 | 13:29 |
| ralonsoh | slaweq, please, check https://review.opendev.org/c/openstack/neutron/+/883483 when you have time | 13:30 |
| ralonsoh | thanks! | 13:30 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Update api extension for default sg rules API https://review.opendev.org/c/openstack/neutron/+/883269 | 13:44 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Add description field to the security_group_default_rules resource https://review.opendev.org/c/openstack/neutron/+/883386 | 13:44 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Default SG api rules template - DB and OVO models https://review.opendev.org/c/openstack/neutron/+/883246 | 13:44 |
| noonedeadpunk | hey folks! What's the process of upgrading OVN majorly? As I have weird issue of `OVN Metadata agent` and `OVN Controller Gateway agent` appearing as DOWN after upgrade. | 14:16 |
| noonedeadpunk | And to solve that it needs to restart neutron-server right afterwards of ovn-host restart or smth like that. And it seems there's some timeout involved as well, as in case of some delay or "wrong" ordering doesn't help to get them back up | 14:17 |
| noonedeadpunk | even though neutron-server is running on completely different environment then ovn parts | 14:18 |
| noonedeadpunk | and obviously nothing obvious in logs | 14:22 |
| haleyb | lajoskatona: hi Lajos, wondering if you saw https://review.opendev.org/c/openstack/neutron/+/883582 ? | 14:24 |
| noonedeadpunk | Hm, seems that it's due to ovn-controller not being restarted but stopped during the upgrade process.... | 14:29 |
| lajoskatona | haleyb: yes, I updated the commit msg to have reference for the old and recent bugs | 14:32 |
| haleyb | lajoskatona: thanks, i hadn't seen the bug miguel had filed | 14:35 |
| haleyb | so i don't know whether you want to revert, or fix? | 14:35 |
| haleyb | it back backported pretty far :( | 14:36 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: New basic API tests for the default SG rules templates CRUDs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/883553 | 14:36 |
| lajoskatona | yeah, last week we shortly discussed, but I was on a conference Thursday/Friday and perhaps he also had other things to work on | 14:36 |
| lajoskatona | haleyb: yes, the "big" mistake was to change OVN to behave exaclty as OVS works, to make it simpler from developer view :-( | 14:38 |
| lajoskatona | haleyb: so I think it is better if it is reverted and go for a clean solution which fixes only OVS, and let OVN as it is, as it works now just different from how OVS can do this "clean rules from backend thing" | 14:39 |
| ralonsoh | agree with this | 14:39 |
| haleyb | lajoskatona: ack | 14:39 |
| ralonsoh | lajoskatona, and thanks for addressing it | 14:40 |
| opendevreview | Merged openstack/neutron master: [ovn] Avoid unwanted ACL_NOT_FOUND error when deleting log objects https://review.opendev.org/c/openstack/neutron/+/883693 | 14:50 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron stable/2023.1: [ovn] Avoid unwanted ACL_NOT_FOUND error when deleting log objects https://review.opendev.org/c/openstack/neutron/+/883717 | 15:01 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron stable/zed: [ovn] Avoid unwanted ACL_NOT_FOUND error when deleting log objects https://review.opendev.org/c/openstack/neutron/+/883718 | 15:01 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron stable/yoga: [ovn] Avoid unwanted ACL_NOT_FOUND error when deleting log objects https://review.opendev.org/c/openstack/neutron/+/883719 | 15:02 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron stable/xena: [ovn] Avoid unwanted ACL_NOT_FOUND error when deleting log objects https://review.opendev.org/c/openstack/neutron/+/883720 | 15:02 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron stable/wallaby: [ovn] Avoid unwanted ACL_NOT_FOUND error when deleting log objects https://review.opendev.org/c/openstack/neutron/+/883721 | 15:02 |
| opendevreview | Miro Tomaska proposed openstack/neutron stable/wallaby: [OVN][Migration] Enable settings backup subnet for NFS clients https://review.opendev.org/c/openstack/neutron/+/883722 | 15:03 |
| sahid | o/ haleyb, regarding your comments https://review.opendev.org/c/openstack/neutron/+/880131/11/neutron/tests/unit/api/rpc/handlers/test_dhcp_rpc.py | 15:04 |
| sahid | I'm not sure to get the whole point, could you give some some details of what you are looking? | 15:04 |
| sahid | very sorry for that | 15:05 |
| sahid | (well... when you have a moment :-) | 15:05 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Improve the ``PortBindingUpdateVirtualPortsEvent`` match filter https://review.opendev.org/c/openstack/neutron/+/883681 | 15:15 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: gate: bump ovn to the latest LTS release (22.03) https://review.opendev.org/c/openstack/neutron/+/880890 | 15:15 |
| haleyb | sahid: maybe i'm mis-understanding the test, but should ret['segments'] be set at all if network.segments is empty? | 15:21 |
| sahid | haleyb: yes the test is a bit diffucult to understand for me aswell | 15:31 |
| sahid | since we have segment plugin activated (in context of the test) | 15:31 |
| sahid | and non local subnet I guess we have all the valid condition to say ret['segments'] not empty | 15:32 |
| sahid | non local subnets means from my understanding subnet attached to a segment | 15:33 |
| * haleyb is in another meeting but will respond in a minute | 15:34 | |
| sahid | that is said I should have perhaps used if router_network as a valid condition... less confusing | 15:40 |
| opendevreview | Lajos Katona proposed openstack/python-neutronclient master: OSC: Remove FWAAS V2 calls to neutronclient https://review.opendev.org/c/openstack/python-neutronclient/+/880629 | 15:58 |
| haleyb | sahid: in the end the code seems to do the right thing, it is just different enough I start to question things. For example, using the variable network_segments in the rpc code but it's slightly different in the test. So I guess the code is fine | 15:58 |
| sahid | ack haleyb thanks for your time on it | 16:12 |
| opendevreview | Lajos Katona proposed openstack/neutron-tempest-plugin master: Adopt bgpvpn scenarios to recent sRBAC changes https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/883838 | 17:27 |
| opendevreview | Merged openstack/neutron master: Bump neutron-lib to 3.6.1 https://review.opendev.org/c/openstack/neutron/+/883483 | 19:24 |
| opendevreview | Merged openstack/neutron master: Revert "Use a writer context for the online alembic migrations" https://review.opendev.org/c/openstack/neutron/+/883120 | 19:49 |
| opendevreview | Ghanshyam proposed openstack/neutron master: DNM testing cirros 0.6.1 https://review.opendev.org/c/openstack/neutron/+/883874 | 20:15 |
| opendevreview | Merged openstack/neutron stable/ussuri: Fix not working use_random_fully config option https://review.opendev.org/c/openstack/neutron/+/883428 | 20:22 |
| opendevreview | Brian Haley proposed openstack/neutron master: Start metadata proxy even if IPv6 DAD fails https://review.opendev.org/c/openstack/neutron/+/883193 | 21:39 |
| *** gregoryo is now known as clayjar | 23:51 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!