| opendevreview | Ashutosh Shukla proposed openstack/ovn-bgp-agent master: Add unit tests for OVN utils module https://review.opendev.org/c/openstack/ovn-bgp-agent/+/984668 | 00:36 |
|---|---|---|
| opendevreview | Rico Lin proposed openstack/neutron master: [OVN] Mark agents down when Chassis is deleted but Chassis_Private remains https://review.opendev.org/c/openstack/neutron/+/984495 | 00:36 |
| opendevreview | Ashutosh Shukla proposed openstack/ovn-bgp-agent master: Narrow broad exception handlers in privileged modules https://review.opendev.org/c/openstack/ovn-bgp-agent/+/984670 | 00:53 |
| opendevreview | Oleksandr Kozachenko proposed openstack/neutron master: Fix OVN network log cross-project drop event attribution https://review.opendev.org/c/openstack/neutron/+/977349 | 01:17 |
| opendevreview | Oleksandr Kozachenko proposed openstack/neutron master: Fix OVN network log cross-project drop event attribution https://review.opendev.org/c/openstack/neutron/+/977349 | 01:18 |
| opendevreview | Dai Dang Van proposed openstack/neutron master: Add dns forwarder l2 extension https://review.opendev.org/c/openstack/neutron/+/951390 | 04:45 |
| opendevreview | Rico Lin proposed openstack/neutron stable/2026.1: Fix AttributeError on DeletedChassis missing other_config https://review.opendev.org/c/openstack/neutron/+/984530 | 04:51 |
| opendevreview | Rico Lin proposed openstack/neutron stable/2025.2: Fix AttributeError on DeletedChassis missing other_config https://review.opendev.org/c/openstack/neutron/+/984531 | 04:51 |
| opendevreview | Rico Lin proposed openstack/neutron stable/2025.1: Fix AttributeError on DeletedChassis missing other_config https://review.opendev.org/c/openstack/neutron/+/984532 | 04:51 |
| opendevreview | Rico Lin proposed openstack/neutron unmaintained/2024.1: Fix AttributeError on DeletedChassis missing other_config https://review.opendev.org/c/openstack/neutron/+/984534 | 04:52 |
| opendevreview | Rico Lin proposed openstack/neutron stable/2024.2: Fix AttributeError on DeletedChassis missing other_config https://review.opendev.org/c/openstack/neutron/+/984533 | 04:54 |
| opendevreview | Rodolfo Alonso proposed openstack/tap-as-a-service master: Change the signature of ``lsp_detach_mirror`` https://review.opendev.org/c/openstack/tap-as-a-service/+/984683 | 06:01 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Change signature of ``lsp_detach_mirror`` method https://review.opendev.org/c/openstack/ovsdbapp/+/984691 | 07:28 |
| *** dmellado47174 is now known as dmellado4717 | 07:31 | |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Fix malformed string formatting in ``row_by_record`` https://review.opendev.org/c/openstack/ovsdbapp/+/984692 | 07:40 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Fix malformed string formatting in ``QoSAddCommand`` https://review.opendev.org/c/openstack/ovsdbapp/+/984693 | 07:43 |
| ralonsoh | hi bcafarel! if you have a minute, please check https://review.opendev.org/q/topic:%22bug/2148211%22 | 07:46 |
| ralonsoh | thanks! | 07:46 |
| opendevreview | Rodolfo Alonso proposed openstack/tap-as-a-service master: Change the signature of ``lsp_detach_mirror`` https://review.opendev.org/c/openstack/tap-as-a-service/+/984683 | 07:46 |
| ralonsoh | hi folks, please check the following n-lib patches: | 07:49 |
| ralonsoh | * https://review.opendev.org/c/openstack/neutron-lib/+/982598 | 07:49 |
| ralonsoh | * https://review.opendev.org/c/openstack/neutron-lib/+/983214 | 07:50 |
| ralonsoh | Once merged, I'll propose a new n-lib release | 07:50 |
| ralonsoh | thanks!! | 07:50 |
| opendevreview | Maor Blaustein proposed openstack/neutron-tempest-plugin master: Migrate QoS with placement tests from tempest https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/984558 | 08:41 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Add neutron-ovn-db-sync-util tests for log and repair modes https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/983699 | 09:12 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Remove trailing space from cls.OC attribute https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/984707 | 09:22 |
| *** dmellado47172 is now known as dmellado4717 | 09:39 | |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Change signature of ``lsp_detach_mirror`` method https://review.opendev.org/c/openstack/ovsdbapp/+/984691 | 09:55 |
| *** ykarel_ is now known as ykarel | 09:57 | |
| opendevreview | Merged openstack/neutron-lib master: Revert "Add VIF_TYPE_VIRTUAL constant to portbindings" https://review.opendev.org/c/openstack/neutron-lib/+/983058 | 10:09 |
| opendevreview | Merged openstack/neutron-lib master: Use project_id in additional places https://review.opendev.org/c/openstack/neutron-lib/+/982598 | 10:55 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Add Neutron functional test to the gate queue https://review.opendev.org/c/openstack/ovsdbapp/+/968031 | 10:58 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Add Neutron functional test to the gate queue https://review.opendev.org/c/openstack/ovsdbapp/+/968031 | 11:03 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Add Neutron functional test to the gate queue https://review.opendev.org/c/openstack/ovsdbapp/+/968031 | 11:05 |
| opendevreview | Renjing Xiao proposed openstack/neutron-tempest-plugin master: Add scenario test for VM connectivity with allowed_address_pairs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/984133 | 11:28 |
| opendevreview | Merged openstack/neutron-lib master: Rehome neutron policy check strings to neutron-lib https://review.opendev.org/c/openstack/neutron-lib/+/983214 | 11:32 |
| ralonsoh | haleyb, hello! please check | 11:34 |
| ralonsoh | * https://review.opendev.org/c/openstack/releases/+/984722 | 11:34 |
| ralonsoh | * https://review.opendev.org/c/openstack/releases/+/984308 | 11:34 |
| ralonsoh | thanks! | 11:34 |
| mblue__ | Hi folks, please check these patches to migrate QoS with placement tests: | 12:02 |
| mblue__ | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/984558 | 12:02 |
| mblue__ | https://review.opendev.org/c/openstack/tempest/+/984559 | 12:02 |
| mblue__ | noticed further CI adjusts needed since skips tests ^ | 12:17 |
| stephenfin | slaweq: you still have two TODOs from ralonsoh here https://review.opendev.org/c/openstack/python-openstackclient/+/981891/8/openstackclient/network/v2/fwaas/rule.py | 12:33 |
| stephenfin | I can approve that once those are fixed | 12:33 |
| ralonsoh | stephenfin, minor nits, thanks for the reminder and approve it in that case | 12:54 |
| ralonsoh | mblue__, thanks! | 12:54 |
| haleyb | lajoskatona, ralonsoh: maybe we can merge https://review.opendev.org/c/openstack/neutron-lib/+/982599 as well? | 13:08 |
| Uggla | haleyb, just to confirm no X session neutron / nova this time ? I have one topic from Layos that we will cover individually. Are we ok ? | 13:50 |
| eolivare | hi, can someone please review the patch that adds a job that tests ovn-bgp? https://review.opendev.org/c/openstack/neutron/+/962188 | 13:52 |
| eolivare | and there's another patch that documents the job: https://review.opendev.org/c/openstack/neutron/+/984318 | 13:53 |
| haleyb | Uggla: the only other topic was around my OSC patch to remove nova-network, but i maybe have to get someone more in ops to chime-in, don't think we need a cross-project. Unless you want me to put it on the agenda to get any feedback | 13:53 |
| opendevreview | Merged openstack/neutron-fwaas master: Commit creation of the default FW group before listing them https://review.opendev.org/c/openstack/neutron-fwaas/+/984584 | 13:55 |
| Uggla | haleyb, no perfectly, no need to have the both teams if it is not required this time. And believe me we have tons of over things to discuss. ;) | 13:56 |
| haleyb | Uggla: ack, sounds good | 13:58 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-fwaas stable/2026.1: Commit creation of the default FW group before listing them https://review.opendev.org/c/openstack/neutron-fwaas/+/984758 | 14:05 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-fwaas stable/2025.2: Commit creation of the default FW group before listing them https://review.opendev.org/c/openstack/neutron-fwaas/+/984759 | 14:06 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-fwaas stable/2025.1: Commit creation of the default FW group before listing them https://review.opendev.org/c/openstack/neutron-fwaas/+/984760 | 14:06 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-fwaas stable/2024.2: Commit creation of the default FW group before listing them https://review.opendev.org/c/openstack/neutron-fwaas/+/984762 | 14:07 |
| opendevreview | Maor Blaustein proposed openstack/neutron-tempest-plugin master: Migrate QoS with placement tests from tempest https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/984558 | 14:21 |
| mblue__ | ralonsoh, np! :) | 14:22 |
| opendevreview | Maor Blaustein proposed openstack/neutron-tempest-plugin master: Migrate QoS with placement tests from tempest https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/984558 | 14:26 |
| opendevreview | Maor Blaustein proposed openstack/neutron-tempest-plugin master: Migrate QoS with placement tests from tempest https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/984558 | 14:42 |
| ralonsoh | haleyb, yes, this could also be added in the new release | 14:52 |
| ralonsoh | I'll ping slaweq: https://review.opendev.org/c/openstack/neutron-lib/+/982599 | 14:52 |
| ralonsoh | please, if you have a few mins, please check ^^^ | 14:52 |
| ralonsoh | haleyb, this is going to be an important release, because we are removing everything related to tenant_id | 14:53 |
| ralonsoh | should we name it as 4.0.0?? | 14:53 |
| ralonsoh | of course, adding this information in the release commit message | 14:53 |
| haleyb | ralonsoh: not everything i think? there is still a lot of code in neutron-lib for tenant_id, i (we) need to figure out when to take the next step | 14:55 |
| haleyb | i'll have to make a list of examples for the ptg i think | 14:57 |
| ralonsoh | haleyb, yeah, right | 14:57 |
| ralonsoh | ok, once we have everything, then we can take this approach | 14:57 |
| haleyb | once we rip out the code that tries to make sure tenant_id and project_id is in every API request we'll see some pain | 14:58 |
| ralonsoh | in any case, we are removing tenant_id from the context | 14:58 |
| haleyb | yes, and i'm fine with a 4.0.0 for that | 14:58 |
| ralonsoh | cool! | 14:58 |
| haleyb | i was going to try and let AI do one of the neutron-* projects like vpnaas or fwaas and see how it does, should just be string replacement | 14:59 |
| ralonsoh | "we should make a skill" heheheeh | 15:00 |
| haleyb | if i can replace myself with AI and still get a paycheck, i can get paid forever! :) | 15:02 |
| opendevreview | Merged openstack/neutron master: Fix AttributeError on DeletedChassis missing other_config https://review.opendev.org/c/openstack/neutron/+/984406 | 15:16 |
| opendevreview | Brian Haley proposed openstack/neutron master: Remove deprecated wsgi config options https://review.opendev.org/c/openstack/neutron/+/984798 | 15:17 |
| winiciusallan[m] | folks, what should be the behavior when creating an "empty" secgroup rule? shoul it be a noop rule or allow-all? | 17:31 |
| winiciusallan[m] | s/shoul/should | 17:31 |
| haleyb | if i say allow-all i'll know you'll say it was the opposite, but without seeing the resultant rule i don't know. i think without adding any arguments it would be wide-open, i.e. default: any (all protocols) | 18:05 |
| haleyb | winiciusallan[m]: ^^ | 18:06 |
| winiciusallan[m] | I was making some tests and it looks like a wide-open rule indeed. I was wondering what the None on port_range_min/max and in the protocol mean | 18:08 |
| winiciusallan[m] | haleyb: thanks! | 18:08 |
| haleyb | https://docs.openstack.org/api-ref/network/v2/index.html#security-group-rules-security-group-rules | 18:10 |
| winiciusallan[m] | I checked the docs, but it wasn't too clear to me. It says "The IP protocol can be represented by a string, an integer, or null." and then "The string any (or integer 0) means all IP protocols." | 18:12 |
| winiciusallan[m] | but not specifying the protocol (null) also looks like "all IP protocols" or am I wrong? | 18:13 |
| haleyb | the default is "all", i'm not sure of the usage of null there, i'm assuming it means 0 | 18:15 |
| haleyb | "the absence of a value" in the generic sense | 18:16 |
| winiciusallan[m] | yeah, it looks like the same behavior of having a 0 there | 18:20 |
| winiciusallan[m] | and for port_range_* equals to null looks like all ports | 18:21 |
| haleyb | a C user probably wrote null, a python one the none | 18:37 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: ovn: Touch only resources managed by Neutron https://review.opendev.org/c/openstack/neutron/+/984412 | 19:17 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: ovn: Touch only resources managed by Neutron https://review.opendev.org/c/openstack/neutron/+/984412 | 20:28 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [OVN] Implement multiple segments per host for routed networks https://review.opendev.org/c/openstack/neutron/+/975111 | 23:21 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [OVN] Agent extension for segment bridges https://review.opendev.org/c/openstack/neutron/+/982018 | 23:25 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!