| *** tbachman_ is now known as tbachman | 02:01 | |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [Secure RBAC] Add scope enforcement enabled job for master branch https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/867518 | 02:10 |
|---|---|---|
| *** tbachman_ is now known as tbachman | 02:20 | |
| *** tbachman_ is now known as tbachman | 05:02 | |
| ralonsoh | lajoskatona, gmann I don't think this friday is a good day, we all have PTO in Red Hat | 07:30 |
| ralonsoh | but let me check that first | 07:30 |
| ralonsoh | in any case, I've talked to slaweq and he is in favor of backporting all these patches | 07:30 |
| ralonsoh | including the router:external one | 07:31 |
| ralonsoh | if the plan is to support sRBAC in Zed to make the transition to A with full support, I'm in favor of this | 07:32 |
| lajoskatona | ralonsoh: ack, anyway, let's discuss this to have a common understanding, I think next week is good also | 07:59 |
| lajoskatona | ralonsoh: I am fine with it, my concern is only that yesterday we agreed the other way and perhaps there are more background which we forgot yesterday and considering those the team will agree to do the backport | 08:00 |
| ralonsoh | lajoskatona, sure, let's discuss first during the next team meeting | 08:13 |
| slaweq | Hi, yesterday I proposed to have this "new-policies" job also for stable/zed | 08:19 |
| slaweq | I will update the router:external patch today to not remove old rule but maybe just change the new one somehow | 08:19 |
| slaweq | but still I think that if there is no rule at all, it will work as "RULE_ANY" so default behaviour of this get network:router:external shouldn't change IMO | 08:20 |
| ralonsoh | I think we can keep the patch as is now | 08:20 |
| ralonsoh | about the RBAC backport, we'll have a new discussing next tuesday | 08:21 |
| slaweq | ++ | 08:21 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: [Secure RBAC] Add scope enforcement enabled job for Zed branch https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/874709 | 08:24 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/zed: Remove policy rule for get_network:router:external https://review.opendev.org/c/openstack/neutron/+/874398 | 08:24 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Set DVR qr-xyz interfaces DOWN on backup node https://review.opendev.org/c/openstack/neutron/+/869741 | 08:25 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Set DVR qr-xyz interfaces DOWN on backup node https://review.opendev.org/c/openstack/neutron/+/869741 | 08:26 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Set DVR qr-xyz interfaces DOWN on backup node https://review.opendev.org/c/openstack/neutron/+/869741 | 08:26 |
| slaweq | ralonsoh lajoskatona patch https://review.opendev.org/c/openstack/neutron/+/874536 seems to be fine now and I just proposed https://review.opendev.org/c/openstack/neutron/+/869741 on top of this mine patch to check if this dvr job will now be fine really | 08:26 |
| ralonsoh | cool, let me check your patch | 08:27 |
| slaweq | thx | 08:28 |
| lajoskatona | +1 | 08:38 |
| slaweq | #endmeeting | 08:48 |
| opendevmeet | Meeting ended Wed Feb 22 08:48:19 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 08:48 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-02-21-15.00.html | 08:48 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-02-21-15.00.txt | 08:48 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2023/neutron_ci.2023-02-21-15.00.log.html | 08:48 |
| slaweq | ups | 08:48 |
| slaweq | :) | 08:48 |
| ralonsoh | hahahaha | 08:48 |
| slaweq | sorry | 08:48 |
| slaweq | it was looooong ci meeting this week :P | 08:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove the ``OVNSqlFixture`` class workaround https://review.opendev.org/c/openstack/neutron/+/874669 | 09:17 |
| ralonsoh | folks: https://review.opendev.org/c/openstack/releases/+/874754 | 09:43 |
| ralonsoh | ping bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, sahid, slawek, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu | 09:43 |
| ralonsoh | sorry for the broad ping | 09:43 |
| ralonsoh | please check the upper patch, Neutron Antelope highlights | 09:43 |
| ralonsoh | if you find something relevant to add, please comment on the patch | 09:43 |
| ralonsoh | thanks in advance! | 09:43 |
| * bcafarel reads | 09:44 | |
| obondarev | ack | 09:44 |
| elvira | thanks ralonsoh ! Checking | 09:47 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Use ovsdbapp commands to add/del backends to ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/874384 | 09:53 |
| lajoskatona | ralonsoh: thanks, checking | 09:57 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: Allow Multiple External Gateways https://review.opendev.org/c/openstack/neutron/+/873593 | 10:14 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Implement support for external-gateway-multihoming extension https://review.opendev.org/c/openstack/neutron/+/874199 | 10:53 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add helper for retrieving LR associated with LRP https://review.opendev.org/c/openstack/neutron/+/873698 | 10:53 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Apply soft anti-affinity for LRs with multiple LRPs when scheduling https://review.opendev.org/c/openstack/neutron/+/873699 | 10:53 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Allow L3 scheduler to be aware of current transaction https://review.opendev.org/c/openstack/neutron/+/874760 | 10:53 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Reset member provisioning status to NO_MONITOR when a HM is deleted https://review.opendev.org/c/openstack/ovn-octavia-provider/+/874609 | 11:31 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Reduce number of routers created in ``test_gateway_chassis_rebalance`` https://review.opendev.org/c/openstack/neutron/+/874765 | 11:37 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/zed: Reduce coverage threshold on stable branches https://review.opendev.org/c/openstack/ovn-octavia-provider/+/874426 | 11:43 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/zed: Avoid use of ovn metadata port IP for HM checks https://review.opendev.org/c/openstack/ovn-octavia-provider/+/874270 | 11:44 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add full support for OVN NB "Gateway_Chassis" table https://review.opendev.org/c/openstack/neutron/+/874767 | 12:14 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: [Stateless SG] Add IPv6 and DHCP related tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/873380 | 13:31 |
| ozzzo_work | ralonsoh: I had some trouble with argparse so I read up on it and it looks like I need to use add_argument so I tried this but it's still throwing errors:https://paste.openstack.org/show/b5HxP4xn7QTTT65Podem/ | 13:56 |
| ralonsoh | ozzzo_work, you don't need to add this argument | 13:57 |
| ralonsoh | you need to pass it when calling the script | 13:57 |
| ozzzo_work | I'm setting options at line 172 and then trying to open the connection at 57 | 13:58 |
| ralonsoh | remove L713 | 13:58 |
| ralonsoh | l173 | 13:58 |
| ozzzo_work | I tried just passing without setting but then the timeout value is ignored | 14:00 |
| ralonsoh | print options in https://github.com/openstack/openstacksdk/blob/master/openstack/config/loader.py#L770 | 14:00 |
| ralonsoh | in any case, that falls under the SDK domain | 14:01 |
| ralonsoh | you can raise this question in openstack-sdks | 14:01 |
| *** haleyb_out is now known as haleyb | 14:01 | |
| ozzzo_work | ok I'll try there, ty! | 14:02 |
| opendevreview | Arnau Verdaguer proposed openstack/neutron stable/yoga: ovn migration: Turn validations off by default https://review.opendev.org/c/openstack/neutron/+/874774 | 14:10 |
| opendevreview | Arnau Verdaguer proposed openstack/neutron stable/xena: ovn migration: Turn validations off by default https://review.opendev.org/c/openstack/neutron/+/874775 | 14:12 |
| opendevreview | Arnau Verdaguer proposed openstack/neutron stable/wallaby: ovn migration: Turn validations off by default https://review.opendev.org/c/openstack/neutron/+/874776 | 14:13 |
| jamesdenton | lucasagomes do you have some time to discuss https://bugs.launchpad.net/neutron/+bug/2007167? | 14:21 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [sqlalchemy-20] Enable SQLAlchemy 2.0 deprecation warnings https://review.opendev.org/c/openstack/neutron/+/874778 | 14:40 |
| lucasagomes | jamesdenton, hi james, sure... I was reading about ur findings in the LP a few days ago | 14:45 |
| jamesdenton | thanks | 14:46 |
| jamesdenton | I am testing a CX6 now to see if it behaves differently | 14:46 |
| lucasagomes | right, yeah I remember u said that option 66 was being ignored by the client in use of the 64 (server-id) one right | 14:46 |
| lucasagomes | 54* | 14:47 |
| jamesdenton | Well, it seems like 66 may be ignored due to the (lack of) null termination and is falling back to server-id, i can't tell | 14:47 |
| lucasagomes | right | 14:48 |
| jamesdenton | the server-id being the subnet gateway IP is problematic, also, since that IP doesn't respond to ARP unless I've explicitly attached that subnet to a neutron router | 14:49 |
| jamesdenton | not sure if that's intended or not | 14:49 |
| lucasagomes | jamesdenton, I need to dig to see who's setting it tho | 14:50 |
| lucasagomes | cause I don't think Ironic does set it AFAICT | 14:50 |
| jamesdenton | what's that, the null char? | 14:50 |
| lucasagomes | I mean the server-id DHCP option | 14:52 |
| lucasagomes | the null char should be the DHCP server itself AFAIUI | 14:52 |
| lucasagomes | which OVN does not | 14:52 |
| jamesdenton | Oh, well as far as server-id goes, OVN is replying to DHCP requests as the gateway IP address - and isn't OVN constructing the DHCP reply? | 14:54 |
| lucasagomes | yes, but ot | 14:55 |
| lucasagomes | but it's also a DHCP option I think you can set it* | 14:55 |
| lucasagomes | jamesdenton, judging by https://docs.openstack.org/neutron/latest/ovn/dhcp_opts.html | 14:55 |
| jamesdenton | i gotcha | 14:56 |
| jamesdenton | I could try that. I did try setting option 66 that way, but it didn't like my string | 14:57 |
| lucasagomes | yeah I saw you tried to add the ^@ | 14:58 |
| lucasagomes | perhaps if u escape as \^@ | 14:58 |
| lucasagomes | idk | 14:58 |
| jamesdenton | tried that, too. | 14:58 |
| jamesdenton | Do you recall which NIC you used in testing? | 14:58 |
| lucasagomes | jamesdenton, don't remember, I believe it was some DRAC (from DELL) machines | 14:59 |
| lucasagomes | but the nic itself I don't recall, it's been awhile and the machine was giving to me by some folks working on openshift | 14:59 |
| jamesdenton | and what are your thoughts on OVN using that gateway IP, rather than a reserved DHCP IP? especially if that network/subnet aren't attached to a router? | 15:00 |
| jamesdenton | understood, thanks | 15:00 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Allow L3 scheduler to be aware of current transaction https://review.opendev.org/c/openstack/neutron/+/874760 | 15:01 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Implement support for external-gateway-multihoming extension https://review.opendev.org/c/openstack/neutron/+/874199 | 15:01 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add helper for retrieving LR associated with LRP https://review.opendev.org/c/openstack/neutron/+/873698 | 15:01 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Apply soft anti-affinity for LRs with multiple LRPs when scheduling https://review.opendev.org/c/openstack/neutron/+/873699 | 15:01 |
| lucasagomes | jamesdenton, it doesn't feel right, as you said it doesn't belong to neutron as a port | 15:01 |
| lucasagomes | jamesdenton, I sometimes think that for baremetal we should always use a fully fledged DHCP server | 15:03 |
| lucasagomes | which OVN DHCP's is not | 15:03 |
| jamesdenton | cool, just checking. i'll continue to mess with this and try different hardware in the meantime | 15:03 |
| lucasagomes | there's always something trick with the different NICs and vendors | 15:03 |
| lucasagomes | tricky | 15:03 |
| jamesdenton | yeah, that seems like a game you don't want to have to play | 15:03 |
| lucasagomes | yeah | 15:03 |
| lucasagomes | cause OVN's DHCP is made "dumb" for a reason | 15:04 |
| lucasagomes | just keep it simple | 15:04 |
| jamesdenton | yep | 15:04 |
| gmann | slaweq: +1 on rbac job on zed. I will add it on tempest side also | 15:34 |
| gmann | slaweq: on ' get_network:router:external ' we need to keep old rule on master also as old policy are still supported there even they are not default | 15:34 |
| slaweq | gmann yes, I will update that get_network:router:external thing | 15:56 |
| gmann | slaweq: thanks | 15:56 |
| *** dkehn_ is now known as dkehn | 15:57 | |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: Add extra router attributes for ECMP and BFD https://review.opendev.org/c/openstack/neutron/+/874797 | 18:13 |
| opendevreview | Brian Haley proposed openstack/neutron master: Change DHCP agent setup code to deal with small MTUs https://review.opendev.org/c/openstack/neutron/+/874167 | 22:49 |
| opendevreview | Brian Haley proposed openstack/neutron master: Add warning about setting network MTU < 1280 https://review.opendev.org/c/openstack/neutron/+/874036 | 23:41 |
| *** tbachman_ is now known as tbachman | 23:47 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!