| opendevreview | Brian Haley proposed openstack/neutron master: DNM: Test removal of enforce_scope https://review.opendev.org/c/openstack/neutron/+/986729 | 00:18 |
|---|---|---|
| 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 | 00:28 |
| opendevreview | Brian Haley proposed openstack/neutron master: DNM: Test removal of enforce_scope https://review.opendev.org/c/openstack/neutron/+/986729 | 01:16 |
| Mike-- | haleyb: I used openstack loadbalancer * which works to my knowledge | 05:32 |
| Mike-- | I was hoping for 'something' but nothing is an answer to (can stop looking) | 05:33 |
| opendevreview | Merged openstack/neutron-dynamic-routing master: Migrate setup configuration to pyproject.toml https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/961685 | 05:47 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Fix pylint invalid-sequence-index false positive for internal_ports https://review.opendev.org/c/openstack/neutron/+/986735 | 06:23 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: DNM - test pylint/pep8 https://review.opendev.org/c/openstack/neutron/+/986736 | 06:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Remove logging-resource API definition and documentation https://review.opendev.org/c/openstack/neutron-lib/+/986739 | 07:07 |
| tafkamax | Hi neutrinos :) | 07:22 |
| tafkamax | Any thoughts on this patch? https://review.opendev.org/c/openstack/neutron/+/986349 | 07:22 |
| open10k8s | hi folks, can i request review on these patches? https://review.opendev.org/c/openstack/neutron/+/977349 https://review.opendev.org/c/openstack/neutron/+/962463 | 07:31 |
| ralonsoh | tafkamax, let me check | 07:32 |
| ralonsoh | open10k8s, https://review.opendev.org/c/openstack/neutron/+/977349 looks fine | 07:32 |
| ralonsoh | the other one is problematic, I would need time to manually check it | 07:32 |
| open10k8s | ralonsoh: thanks. let me know if need any other hands from our side. :) | 07:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Remove fip64 API definition and documentation https://review.opendev.org/c/openstack/neutron-lib/+/986742 | 07:39 |
| tafkamax | ralonsoh: thx! | 07:50 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Remove router-interface-fip API definition and documentation https://review.opendev.org/c/openstack/neutron-lib/+/986743 | 07:51 |
| ralonsoh | bcafarel, hello! Please check https://review.opendev.org/q/Idc7d789ffe0289bf28fb289456196c307dd4a706. In particular the 2024.2 patch. If this patch is merged today, I would update the EOL 2024.1 patch | 07:53 |
| ralonsoh | --> https://review.opendev.org/c/openstack/releases/+/984977 | 07:53 |
| ralonsoh | sorry, not 2024.1 EOL but 2024.2 | 07:53 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Add PVLAN service plugin https://review.opendev.org/c/openstack/neutron/+/978798 | 08:07 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: PVLAN OVN Driver https://review.opendev.org/c/openstack/neutron/+/985150 | 08:07 |
| opendevreview | Takashi Kajinami proposed openstack/neutron stable/2026.1: Revert "[eventlet-removal] Remove logger mechanism from functional tests" https://review.opendev.org/c/openstack/neutron/+/986744 | 08:08 |
| opendevreview | Takashi Kajinami proposed 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 | 08:10 |
| opendevreview | Takashi Kajinami proposed openstack/neutron master: Revert "[eventlet-removal] Remove logger mechanism from functional tests" https://review.opendev.org/c/openstack/neutron/+/986746 | 08:11 |
| elodilles | hi neutron team, quick double check, i'm about to merge the 2024.2-eol tagging patch for neutron, but there seem to be some open patch (even about to merge? O.o). so the question, do you really want to merge them? https://review.opendev.org/q/project:openstack/neutron+branch:stable/2024.2+is:open | 08:26 |
| ralonsoh | elodilles, yeah, I'm checking that now | 08:27 |
| ralonsoh | elodilles, how much time do we have to merge these patches? | 08:27 |
| elodilles | note that the planned EOL date was yesterday, so requirements is already tagged with 2024.2-eol and the branch should be deleted soon | 08:27 |
| elodilles | ralonsoh: a couple of hours is okay o:) | 08:28 |
| ralonsoh | ok, cool | 08:28 |
| elodilles | ralonsoh: we were not that strict in the past but release team don't want to delay the transition for weeks, so best would be if i could finish the EOL'ing today | 08:29 |
| ralonsoh | it will be today. If the patches are not merged, I'll ping you to go ahead | 08:30 |
| elodilles | ralonsoh: ACK, i've commented the eol patch for now: https://review.opendev.org/c/openstack/releases/+/984977 | 08:31 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add compute1 node to BGP multinode tempest job https://review.opendev.org/c/openstack/neutron/+/986075 | 08:35 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Restore txn for lrouter creation https://review.opendev.org/c/openstack/neutron/+/986757 | 09:23 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == DNM == OVN L3 scheduler test (3) https://review.opendev.org/c/openstack/neutron/+/981212 | 09:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove fip64 extension support from floating IP pools https://review.opendev.org/c/openstack/neutron/+/986763 | 09:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Remove fip64 API definition and documentation https://review.opendev.org/c/openstack/neutron-lib/+/986742 | 09:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use Neutron FIP UUID as OVN ``NAT`` row UUID https://review.opendev.org/c/openstack/neutron/+/984408 | 10:19 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: bgp: Handle chassis scale in/out https://review.opendev.org/c/openstack/neutron/+/985730 | 10:40 |
| opendevreview | Merged openstack/neutron stable/2024.2: Fix return value of empty-MAC guard in metadata agent https://review.opendev.org/c/openstack/neutron/+/986614 | 11:03 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Populate OVN AgentCache in the db-sync tool https://review.opendev.org/c/openstack/neutron/+/985953 | 11:49 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Remove router-interface-fip API definition and documentation https://review.opendev.org/c/openstack/neutron-lib/+/986743 | 11:51 |
| opendevreview | Merged openstack/neutron master: Fix pylint invalid-sequence-index false positive for internal_ports https://review.opendev.org/c/openstack/neutron/+/986735 | 12:25 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Add PVLAN service plugin https://review.opendev.org/c/openstack/neutron/+/978798 | 12:31 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: PVLAN OVN Driver https://review.opendev.org/c/openstack/neutron/+/985150 | 12:31 |
| elodilles | ralonsoh: i saw you've updated the hash on the 2024.2-eol patch. is that ready now to merge? | 12:49 |
| opendevreview | Merged openstack/os-vif master: Remove url tags from README https://review.opendev.org/c/openstack/os-vif/+/976271 | 12:56 |
| haleyb | elodilles: it is good to go, the other patch is unmergeable at the moment, and it's not worth it to spend cycles on it imo | 13:09 |
| elodilles | haleyb: ACK, thanks for confirming o/ | 13:10 |
| ralonsoh | elodilles, yes | 13:14 |
| ralonsoh | hi folks, an easy patch that needs to be backported to 2026.1 too: https://review.opendev.org/c/openstack/neutron/+/985953 | 13:20 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fix FIP NAT external_mac race when LSP is already UP https://review.opendev.org/c/openstack/neutron/+/986805 | 13:49 |
| opendevreview | Jakub Libosvar proposed openstack/neutron-lib master: evpn: Add EVPN API definitions https://review.opendev.org/c/openstack/neutron-lib/+/986579 | 13:54 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove fip64 extension support from floating IP pools https://review.opendev.org/c/openstack/neutron/+/986763 | 13:57 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Add compute1 node to BGP multinode tempest job https://review.opendev.org/c/openstack/neutron/+/986075 | 14:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fix FIP NAT external_mac race when LSP is already UP https://review.opendev.org/c/openstack/neutron/+/986805 | 14:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add missing address_group CRUD and action policy rules https://review.opendev.org/c/openstack/neutron/+/986814 | 14:31 |
| nathharp | hi Neutronians - I've just raised a bug here: https://bugs.launchpad.net/neutron/+bug/2150754. Does this seem like a valid bug, or have I missed something? | 14:32 |
| ralonsoh | nathharp, the current resource_request mechanism (the placement API info) is coupled to the QoS service plugin. If you don't define a physnet BWs (ingres, egress) and at the same time, request a BW for a port, resource_request will be empty | 14:35 |
| nathharp | ralonsoh, is that the intended behaviour though? | 14:36 |
| ralonsoh | yes, we only provide information to the scheduler when a QoS policy is attached to this port | 14:36 |
| ralonsoh | I'm not sure what you are trying there, but I think you are expecting to schedule the port in a specific physnet | 14:37 |
| nathharp | trying to schedule Ironic nodes (via Nova compute) only to ironic nodes that are actually connected to that physnet. We have two rows of identical systems, each with their own physnet. Provider networks then created for each row, specifying the physnet for the right row. We then get build failures on the Ironic nodes because Placement has no information to go on for which nodes to schedule. If resource_request | 14:43 |
| nathharp | included required': ['CUSTOM_PHYSNET'] then placement could schedule onto the right machines | 14:43 |
| ralonsoh | nathharp, this is a RFE, not a bug. We don't have a way, right now, to tint the port information (resource_request) with that info | 14:45 |
| ralonsoh | haleyb, hi! I think https://review.opendev.org/c/openstack/neutron/+/981695/1/tools/coding-checks.sh is making the pylint check a nightmare | 14:46 |
| ralonsoh | or, from another point of view, that surfaced some errors we have in the code | 14:47 |
| ralonsoh | for example: patch https://review.opendev.org/c/openstack/neutron/+/986763 | 14:47 |
| ralonsoh | pep8 execution: https://38bd62aca65991894919-e48e76b6f53fcff59de7a7c1c3da6c62.ssl.cf5.rackcdn.com/openstack/98b9ac5d24e24833945d03a1245f54bc/job-output.txt | 14:47 |
| ralonsoh | totally unrelated | 14:47 |
| nathharp | ralonsoh, ok - I was keen to validate if it was currently working as expected, but it looks like it is. Thanks for the response | 14:47 |
| ralonsoh | haleyb, we tried this morning this: https://review.opendev.org/c/openstack/neutron/+/986735/. But we still have the same error | 14:48 |
| ralonsoh | nathharp, yw | 14:48 |
| ralonsoh | I'm going to propose a more direct approach, that is to mute these two errors explicitly | 14:49 |
| haleyb | ralonsoh: i did see those random pep8 failures, if you just want to revert that's fine, or we can fix the bugs | 14:50 |
| ralonsoh | haleyb, no, I want to push an update over this patch | 14:53 |
| haleyb | ralonsoh: oh, that E1126 one, i hadn't clicked the link. Fell free to push something, i'll take a look. | 14:56 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Fix pylint invalid-sequence-index false positive for internal_ports (2) https://review.opendev.org/c/openstack/neutron/+/986828 | 15:17 |
| opendevreview | Jakub Libosvar proposed openstack/neutron-lib master: evpn: Add EVPN API definitions https://review.opendev.org/c/openstack/neutron-lib/+/986579 | 15:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use Neutron FIP UUID as OVN ``NAT`` row UUID https://review.opendev.org/c/openstack/neutron/+/984408 | 15:46 |
| opendevreview | Helen Chen proposed openstack/neutron master: Created OVN Agent EVPN extension & netlink monitor https://review.opendev.org/c/openstack/neutron/+/984409 | 16:11 |
| opendevreview | Helen Chen proposed openstack/neutron master: EVPN extension finite state machine event handler https://review.opendev.org/c/openstack/neutron/+/986832 | 16:11 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add missing address_group CRUD and action policy rules https://review.opendev.org/c/openstack/neutron/+/986814 | 16:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Fix pylint invalid-sequence-index false positive for internal_ports (2) https://review.opendev.org/c/openstack/neutron/+/986828 | 16:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fix pylint invalid-sequence-index false positive for internal_ports (2) https://review.opendev.org/c/openstack/neutron/+/986828 | 17:00 |
| ralonsoh | haleyb, I've checked twice this patch (with 10 jobs) ^^ and is working now | 17:00 |
| haleyb | ralonsoh: ack. just curiuos, will adding typing annotations help with something like this? | 17:27 |
| opendevreview | Brian Haley proposed openstack/neutron master: Remove setting of enforce_scope in unit tests https://review.opendev.org/c/openstack/neutron/+/986729 | 18:09 |
| opendevreview | Jakub Libosvar proposed openstack/neutron-lib master: evpn: Add EVPN API definitions https://review.opendev.org/c/openstack/neutron-lib/+/986579 | 18:57 |
| opendevreview | Merged openstack/neutron master: bgp: Handle chassis scale in/out https://review.opendev.org/c/openstack/neutron/+/985730 | 19:30 |
| opendevreview | Doug Goldstein proposed openstack/neutron master: allow physical_network to be set for provider networks on VXLAN https://review.opendev.org/c/openstack/neutron/+/985837 | 22:53 |
| cardoe | nathharp: the issue you're seeing about the scheduling to specific nodes like that is something similar to what I'm working on | 23:41 |
| cardoe | nathharp: We've got the following change in Ironic https://opendev.org/openstack/ironic/commit/e721c56e681a2f20a0f7edd2605339c3e6b275f3 which will pass along the physical_network in the binding_profile | 23:43 |
| cardoe | It's ultimately related to https://bugs.launchpad.net/neutron/+bug/2105855 for me | 23:44 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!