| opendevreview | Oleksandr Kozachenko proposed openstack/neutron master: Fix a race condition in OVN metadata port update https://review.opendev.org/c/openstack/neutron/+/962463 | 00:08 |
|---|---|---|
| opendevreview | Oleksandr Kozachenko proposed openstack/neutron master: Fix a race condition in OVN metadata port update https://review.opendev.org/c/openstack/neutron/+/962463 | 00:10 |
| *** dmellado471725 is now known as dmellado47172 | 00:17 | |
| opendevreview | Brian Haley proposed openstack/neutron-lib master: WIP: Tenant_id has left the building https://review.opendev.org/c/openstack/neutron-lib/+/986960 | 05:03 |
| opendevreview | Brian Haley proposed openstack/neutron-lib master: WIP: Tenant_id has left the building https://review.opendev.org/c/openstack/neutron-lib/+/986960 | 05:39 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Add subscript access support to RowView https://review.opendev.org/c/openstack/ovsdbapp/+/987452 | 07:27 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Add subscript access support to RowView https://review.opendev.org/c/openstack/ovsdbapp/+/987452 | 07:27 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add missing address_group CRUD and action policy rules https://review.opendev.org/c/openstack/neutron/+/986814 | 08:18 |
| opendevreview | Merged openstack/neutron-specs master: Add spec for security groups default statefulness API https://review.opendev.org/c/openstack/neutron-specs/+/984350 | 08:36 |
| opendevreview | Lajos Katona proposed openstack/networking-sfc master: Use 'flow_classifier' and 'sfc' API ext definition from neutron_lib https://review.opendev.org/c/openstack/networking-sfc/+/553819 | 09:07 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Remove Dalmatian (2024.2) jobs (part 1) https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/986964 | 09:18 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Add subscript access support to RowView https://review.opendev.org/c/openstack/ovsdbapp/+/987452 | 10:11 |
| opendevreview | Dai Dang Van proposed openstack/neutron master: Add dns forwarder l2 extension https://review.opendev.org/c/openstack/neutron/+/951390 | 10:11 |
| ralonsoh | lajoskatona, hello! Sorry, I was offline yesterday and I saw the issue with SFC | 10:22 |
| ralonsoh | there is another option: to disable `_filter_validation_support` in the plugin | 10:22 |
| ralonsoh | I'm proposing, just for testing, a patch with this variable =False | 10:22 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-sfc master: Disable filter-validation for ``SfcPlugin`` https://review.opendev.org/c/openstack/networking-sfc/+/987469 | 10:24 |
| ralonsoh | lajoskatona, ^^ | 10:24 |
| ralonsoh | that should disable this extension and should fix the current problems | 10:24 |
| ralonsoh | of course, enabling the extension and fixing the APIs is more elegant | 10:24 |
| opendevreview | Ivan Anfimov proposed openstack/neutron-vpnaas-dashboard master: Update Babel configuration https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/987472 | 10:30 |
| opendevreview | Ivan Anfimov proposed openstack/neutron-vpnaas-dashboard master: Update Babel configuration https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/987472 | 10:31 |
| lajoskatona | ralonsoh: cool, indeed that is also a good idea | 10:33 |
| lajoskatona | I was just strugling with unit tests, as locally all passes | 10:33 |
| lajoskatona | ralonsoh: we can go with disabling it for quick fix and push later the API extension fix | 10:34 |
| ralonsoh | lajoskatona, yes, that could be an option: first disable, change the API and enable again | 10:34 |
| ralonsoh | if the upper patch works, it would be better to first disable the extension | 10:34 |
| ralonsoh | btw, luckily the 3.35.0 n-lib version is in 2026.2 only | 10:35 |
| ralonsoh | so we don't need to backport anything | 10:35 |
| ralonsoh | sorry, 3.25.0. That includes https://review.opendev.org/c/openstack/neutron-lib/+/978554 | 10:35 |
| ralonsoh | so we are good for stable versions | 10:35 |
| lajoskatona | ack, thanks | 10:37 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-sfc master: Disable filter-validation for ``SfcPlugin`` and ``FlowClassifierDbPlugin`` https://review.opendev.org/c/openstack/networking-sfc/+/987469 | 11:09 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-sfc master: Disable filter-validation for SfcPlugin and FlowClassifierDbPlugin https://review.opendev.org/c/openstack/networking-sfc/+/987469 | 11:09 |
| opendevreview | Ivan Anfimov proposed openstack/neutron-fwaas-dashboard master: Update Babel configuration https://review.opendev.org/c/openstack/neutron-fwaas-dashboard/+/987487 | 11:09 |
| opendevreview | Ivan Anfimov proposed openstack/neutron-fwaas-dashboard master: Update Babel configuration https://review.opendev.org/c/openstack/neutron-fwaas-dashboard/+/987487 | 11:10 |
| opendevreview | Ivan Anfimov proposed openstack/neutron-fwaas-dashboard master: Update Babel configuration https://review.opendev.org/c/openstack/neutron-fwaas-dashboard/+/987487 | 11:10 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add compute1 node to BGP multinode tempest job https://review.opendev.org/c/openstack/neutron/+/986075 | 11:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use ``query.first() is not None`` to check resources https://review.opendev.org/c/openstack/neutron/+/986513 | 11:23 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use ``query.first() is not None`` to check resources https://review.opendev.org/c/openstack/neutron/+/986513 | 11:28 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Drop unnecessary override of ovn-metadata-agent https://review.opendev.org/c/openstack/ovn-octavia-provider/+/987045 | 11:57 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Revert "[eventlet-removal] Remove "logger" mechanism from ML2/OVN CI jobs" https://review.opendev.org/c/openstack/ovn-octavia-provider/+/986745 | 11:58 |
| ralonsoh | haleyb, hello! I've talked to Lajos about https://bugs.launchpad.net/neutron/+bug/2150864 | 12:52 |
| ralonsoh | I've proposed https://review.opendev.org/c/openstack/networking-sfc/+/987469 to disable the extension in both SFC plugins | 12:52 |
| ralonsoh | this is affected by n-lib 3.25.0, only present in Hibiscus, not Gazpacho | 12:53 |
| ralonsoh | after this ^^ patch, we can fix the SFC and n-lib APIs | 12:53 |
| ralonsoh | and then, revert it | 12:53 |
| ralonsoh | with https://review.opendev.org/c/openstack/networking-sfc/+/987469 we have time to push the n-lib and SFC patches with no rush | 12:54 |
| haleyb | ralonsoh: ack, what about some of the extensions, i'll find the patch | 13:01 |
| haleyb | https://review.opendev.org/c/openstack/neutron-lib/+/987378 | 13:05 |
| ralonsoh | haleyb, yes, that makes sense. Because we are modifying existing APIs, we shoudl at least provide a reno with a "fixes" section | 13:06 |
| ralonsoh | in order to justify why we are changing existing APIs | 13:06 |
| ralonsoh | haleyb, there are legit comments from stephenfin, using SDK as reference | 13:07 |
| ralonsoh | I agree with these comments | 13:07 |
| ralonsoh | pffff these APIs are a bit messy | 13:08 |
| haleyb | ralonsoh: and as commented there are probably more things we can filter on, don't know how far to go | 13:08 |
| ralonsoh | right... we can maybe raise all these things using the SDK functional tests | 13:10 |
| opendevreview | Stephen Finucane proposed openstack/neutron master: Handle unset socket https://review.opendev.org/c/openstack/neutron/+/987498 | 13:12 |
| opendevreview | Stephen Finucane proposed openstack/neutron master: trivial: Convert add_keepalives to function https://review.opendev.org/c/openstack/neutron/+/987499 | 13:12 |
| opendevreview | Stephen Finucane proposed openstack/os-vif master: Drop support for Python 3.10 https://review.opendev.org/c/openstack/os-vif/+/987500 | 13:17 |
| opendevreview | Brian Haley proposed openstack/neutron master: Change ovn_router_indirect_snat config option to True https://review.opendev.org/c/openstack/neutron/+/987308 | 13:21 |
| opendevreview | Merged openstack/neutron-lib master: remove reference to non-existent method https://review.opendev.org/c/openstack/neutron-lib/+/987437 | 13:25 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: [OVN] Only set NAT gateway_port when distributed FIP is enabled https://review.opendev.org/c/openstack/neutron/+/987187 | 13:25 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: [OVN] Only set NAT gateway_port when distributed FIP is enabled https://review.opendev.org/c/openstack/neutron/+/987187 | 13:25 |
| opendevreview | Stephen Finucane proposed openstack/os-vif master: Remove linux bridge plugin https://review.opendev.org/c/openstack/os-vif/+/941586 | 13:28 |
| opendevreview | Stephen Finucane proposed openstack/os-vif master: Remove linux bridge plugin https://review.opendev.org/c/openstack/os-vif/+/941586 | 13:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Split L3RpcNotifierMixin to avoid RPC callbacks for OVN https://review.opendev.org/c/openstack/neutron/+/987502 | 13:33 |
| opendevreview | Merged openstack/neutron master: Sorting, limiting and pagination support for agent list https://review.opendev.org/c/openstack/neutron/+/983337 | 13:47 |
| ralonsoh | ykarel_, just a heads-up (no action): https://bugs.launchpad.net/neutron/+bug/2151304 | 13:50 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Split L3RpcNotifierMixin to avoid RPC callbacks for OVN https://review.opendev.org/c/openstack/neutron/+/987502 | 13:54 |
| haleyb | ralonsoh: i wanted to get your take on this recent bug - https://bugs.launchpad.net/neutron/+bug/2151245 - there is a fix mentioned at the end i could get started on, seemed legit to me but i know you worked on this stuff | 13:57 |
| haleyb | sorry, i know you're catching-up from some time off too | 13:57 |
| ralonsoh | haleyb, uffff another problem related to segment range initialization??? | 13:59 |
| ralonsoh | I'll check it now | 13:59 |
| ykarel_ | ralonsoh, ack | 13:59 |
| ralonsoh | haleyb, hmmmm ok, now I understand the problem | 14:00 |
| haleyb | unfortunately. the submittor had pinged me internally about some wsgi things and now it makes sense | 14:00 |
| ralonsoh | the question here is: do we support anything but wsgi?? | 14:00 |
| haleyb | yes, that is the question, the docs i thought said uwsgi only | 14:01 |
| ralonsoh | because how is now implemented (let me find the patch) relays on the wsgi library and that we must always have worker 1 | 14:01 |
| ralonsoh | the problem with the proposal ` if worker_id not in (None, wsgi_utils.FIRST_WORKER_ID):` is that this will be executed by any worker | 14:01 |
| ralonsoh | and that's wrong or at least dangerous | 14:01 |
| haleyb | the doc does say "There are two ways to deploy using WSGI: ``uwsgi`` and Apache ``mod_wsgi``" | 14:03 |
| ralonsoh | yeah... ok, I need to start the API using apache | 14:04 |
| ralonsoh | I usually use uwsgi, that is the default one provided by devstack | 14:04 |
| haleyb | if there is anything you need from Myles let me know he's in my timezone, i could have him join the channel too | 14:06 |
| ralonsoh | thanks | 14:06 |
| ralonsoh | haleyb, I've commented in the bug. I must be missing something in my Apache configuration | 14:17 |
| ralonsoh | I would appreciate some help deploying the Apache Neutron API server | 14:17 |
| haleyb | ralonsoh: ack, i'll ping him now | 14:17 |
| ralonsoh | ah, we still see random pep8 issues since the -j0 patch | 14:20 |
| opendevreview | Merged openstack/neutron master: Replace OVN Metadata agent with OVN agent https://review.opendev.org/c/openstack/neutron/+/987047 | 14:20 |
| ralonsoh | I'm going to open a LP bug to group all possible patches we submit to fix them | 14:20 |
| haleyb | ralonsoh: sigh, that was all the speed-up my local pep8 run, if we have to revert to make things stable i'll just get a faster system | 14:22 |
| ralonsoh | haleyb, we can have this LP bug and push patches while we catch them | 14:22 |
| ralonsoh | for example, the last ones: | 14:22 |
| ralonsoh | https://e59199e17fab3c042e89-bd1b156529b2e46ea276b201241da879.ssl.cf1.rackcdn.com/openstack/bfd3ff9704714242aad83a32ae4c1e14/job-output.txt | 14:22 |
| ralonsoh | related to https://review.opendev.org/c/openstack/neutron/+/987502, that is not touching these files | 14:22 |
| haleyb | i have seen the #noqa warnings which aren't fatal but not the others | 14:24 |
| haleyb | ralonsoh: but isn't the import order one in that patch? l3_db should be earlier | 14:26 |
| haleyb | ralonsoh: btw i pinged Myles and he's in the channel now as mylesjp, he'll drop a comment in the bug regarding mod_wsgi | 14:28 |
| ralonsoh | thanks | 14:32 |
| opendevreview | Merged openstack/networking-sfc master: Disable filter-validation for SfcPlugin and FlowClassifierDbPlugin https://review.opendev.org/c/openstack/networking-sfc/+/987469 | 15:08 |
| opendevreview | Lajos Katona proposed openstack/neutron-lib master: sfc: Add is_filter and is_sort_key to SFC API https://review.opendev.org/c/openstack/neutron-lib/+/987372 | 15:15 |
| opendevreview | Brian Haley proposed openstack/neutron master: Stop using neutron-lib project info code https://review.opendev.org/c/openstack/neutron/+/986961 | 19:15 |
| opendevreview | Helen Chen proposed openstack/neutron master: Created OVN Agent EVPN extension & netlink monitor https://review.opendev.org/c/openstack/neutron/+/984409 | 19:44 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Add PVLAN service plugin https://review.opendev.org/c/openstack/neutron/+/978798 | 20:23 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Add PVLAN OVN Driver https://review.opendev.org/c/openstack/neutron/+/985150 | 20:23 |
| opendevreview | Merged openstack/neutron-lib master: sfc: Add is_filter and is_sort_key to SFC API https://review.opendev.org/c/openstack/neutron-lib/+/987372 | 20:27 |
| opendevreview | Jakub Libosvar proposed openstack/neutron-lib master: evpn: Do not set evpn_vni to None by default https://review.opendev.org/c/openstack/neutron-lib/+/987354 | 20:38 |
| opendevreview | Brian Haley proposed openstack/neutron master: Stop using neutron-lib project info code https://review.opendev.org/c/openstack/neutron/+/986961 | 20:56 |
| opendevreview | Brian Haley proposed openstack/neutron master: WIP: Test neutron-lib attribute rename change https://review.opendev.org/c/openstack/neutron/+/986962 | 21:41 |
| opendevreview | Brian Haley proposed openstack/neutron master: WIP: Test neutron-lib attribute rename change https://review.opendev.org/c/openstack/neutron/+/986962 | 22:57 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [OVN] Agent extension for segment bridges https://review.opendev.org/c/openstack/neutron/+/982018 | 23:04 |
| opendevreview | JayGue Lee proposed openstack/ovn-octavia-provider master: [OVN] Protect LB VIP ports with device_id and device_owner https://review.opendev.org/c/openstack/ovn-octavia-provider/+/986730 | 23:41 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!