| opendevreview | Brian Haley proposed openstack/neutron master: Do not allow a tenant to create a default SG for another one https://review.opendev.org/c/openstack/neutron/+/855580 | 01:18 |
|---|---|---|
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/yoga: Fix healthMonitor events affecting to unrelated LB https://review.opendev.org/c/openstack/ovn-octavia-provider/+/855614 | 07:33 |
| sahid | Morning guys, if I can request final approval for those 4 patches: https://review.opendev.org/c/openstack/neutron/+/839479/29, https://review.opendev.org/c/openstack/neutron/+/840415/29, https://review.opendev.org/c/openstack/neutron/+/840416/30, https://review.opendev.org/c/openstack/neutron/+/840416/30 | 07:36 |
| slaweq | sahid: hi, looking right now | 07:37 |
| slaweq | sahid: done | 07:39 |
| slaweq | all 4 should be in the gate now | 07:40 |
| sahid | thanks a lot slaweq i'm working to address other comments in patches left | 07:43 |
| slaweq | sahid++ thx | 07:44 |
| opendevreview | Gregory Thiemonge proposed openstack/ovn-octavia-provider master: Fix create_vip_port prototype based on octavia-lib https://review.opendev.org/c/openstack/ovn-octavia-provider/+/854764 | 07:51 |
| opendevreview | Szymon Wróblewski proposed openstack/neutron master: Retry connections to Nova https://review.opendev.org/c/openstack/neutron/+/855625 | 09:27 |
| *** sean-k-mooney1 is now known as sean-k-mooney | 09:31 | |
| opendevreview | Merged openstack/neutron master: ovs: make vlanmanager to handle more vlan mapping per network https://review.opendev.org/c/openstack/neutron/+/839479 | 09:37 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/xena: Fix healthMonitor events affecting to unrelated LB https://review.opendev.org/c/openstack/ovn-octavia-provider/+/855633 | 09:42 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/wallaby: Fix healthMonitor events affecting to unrelated LB https://review.opendev.org/c/openstack/ovn-octavia-provider/+/855634 | 09:42 |
| opendevreview | Merged openstack/neutron master: ovs: use a local vlan per network/segmentation https://review.opendev.org/c/openstack/neutron/+/840415 | 09:49 |
| opendevreview | Merged openstack/neutron master: dhcp/rpc: retrieve network details with segments https://review.opendev.org/c/openstack/neutron/+/840416 | 09:49 |
| opendevreview | Merged openstack/neutron master: utils: add option to generate dhcp device id per segmentation https://review.opendev.org/c/openstack/neutron/+/840417 | 09:49 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ml2/mech: bind port on segment that subnet belongs https://review.opendev.org/c/openstack/neutron/+/840418 | 10:09 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: make device manager to clean only not used devices https://review.opendev.org/c/openstack/neutron/+/840419 | 10:09 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: add/use cleanup stale devices API https://review.opendev.org/c/openstack/neutron/+/840420 | 10:09 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: support multiple segmentations per network https://review.opendev.org/c/openstack/neutron/+/840421 | 10:09 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: db: add segment_index to the unique constraint https://review.opendev.org/c/openstack/neutron/+/841459 | 10:09 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: segment: enable multisegments support for host https://review.opendev.org/c/openstack/neutron/+/841838 | 10:09 |
| opendevreview | Gregory Thiemonge proposed openstack/ovn-octavia-provider master: Fix create_vip_port prototype based on octavia-lib https://review.opendev.org/c/openstack/ovn-octavia-provider/+/854764 | 10:09 |
| opendevreview | Arnau Verdaguer proposed openstack/neutron master: [Trunk] Update the trunk status with the parent status https://review.opendev.org/c/openstack/neutron/+/853779 | 10:48 |
| *** carloss is now known as carloss|afk | 11:28 | |
| opendevreview | liuyulong proposed openstack/neutron stable/stein: Fix migration failed due TypeError of new_vlan_tag https://review.opendev.org/c/openstack/neutron/+/853752 | 12:33 |
| opendevreview | liuyulong proposed openstack/neutron stable/stein: Clean up db residual record from dvr port https://review.opendev.org/c/openstack/neutron/+/852752 | 12:34 |
| *** dasm|off is now known as dasm | 13:00 | |
| *** carloss|afk is now known as carloss | 13:03 | |
| sahid | obondarev: o/ quick question, what should I do regarding CI, this patch https://review.opendev.org/c/openstack/neutron/+/840418 was passing CI prior a rebase, then it fails `neutron-ovs-tempest-multinode-full`, next patch to that one is passing `neutron-ovs-tempest-multinode-full` but fails `neutron-functional-with-uwsgi`, finally next patch is passing all the jobs but zuul has put -1 | 13:35 |
| sahid | https://review.opendev.org/c/openstack/neutron/+/840420/32 | 13:35 |
| sahid | :-) | 13:35 |
| sahid | oh wait, sorry there is a POST_FAILURE on neutron-functional-with-uwsgi | 13:36 |
| obondarev | sahid: hey, you have to check what caused CI to fail - whether it's a known issue or not. Then just recheck the patch with "recheck bug <id>", or "recheck POST_FAILURE", or "recheck unrelated functional test <test name> failure", etc. | 13:38 |
| sahid | obondarev: perfect, thanks a lot ! | 13:49 |
| obondarev | sahid: np | 13:51 |
| sahid | btw mlavalle, nothing hurry, just that i replied to you https://review.opendev.org/c/openstack/neutron/+/840418/comments/adb33cf0_d720078a I'm not quite sure of what should I do | 13:51 |
| mlavalle | sahid: ack. I'll look at it later today | 14:09 |
| lajoskatona | sahid: I checked and as I see the multinode failure and py310 are unrelated | 14:31 |
| lajoskatona | sahid, mlavalle: I just tested the situation with mixed subnets (to related to segment and to not related to segment) and I got exception (BadRequestException: 400: Client Error for url: http://100.109.0.113:9696/networking/v2.0/subnets, All of the subnets on network '7f8d886f-cd13-450e-b1ca-ad065580c0d9' must either all be associated with segments or all not associated with any segment. ) | 14:32 |
| lajoskatona | sahid, mlavalle: please check my comment on the patch (https://review.opendev.org/c/openstack/neutron/+/840418 ), perhaps I missed something | 14:34 |
| lajoskatona | I haven't checked situations when I update the subnet to remove segment or add it later | 14:34 |
| opendevreview | Jake Yip proposed openstack/neutron master: Enable qos extension_driver in ovn_db_sync https://review.opendev.org/c/openstack/neutron/+/855668 | 14:41 |
| opendevreview | Brian Haley proposed openstack/neutron master: Do not allow a tenant to create a default SG for another one https://review.opendev.org/c/openstack/neutron/+/855580 | 14:59 |
| * haleyb wonders why there are always more bugs on his deputy week | 15:03 | |
| sahid_ | lajoskatona: oh ! thanks a lot for this check | 15:26 |
| sahid_ | looks like I was wrong in the orders of the commands, so it seems that we have to pass the segment to the subnet during creation which is even better :-) | 15:27 |
| sahid_ | i will do a double check and comment on the review | 15:27 |
| sahid_ | againm thanks a lot for that | 15:27 |
| opendevreview | Terry Wilson proposed openstack/ovsdbapp master: Fix TRY_AGAIN handling https://review.opendev.org/c/openstack/ovsdbapp/+/855531 | 15:37 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: Add multiple ip addresses in one priv call https://review.opendev.org/c/openstack/neutron/+/855677 | 15:43 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: Add multiple ip addresses in one priv call https://review.opendev.org/c/openstack/neutron/+/855677 | 15:53 |
| mlavalle | lajoskatona: so what you are saying is that you created a network (multisegment1) which has one subnet associated with one segment. Then you try to add the subnet multisegment1-no-segment, which, as its name indicates it's not associated to a segment | 16:02 |
| mlavalle | lajoskatona: and you get the exception, right? | 16:02 |
| mlavalle | and the exception comes from the ipam mixin | 16:03 |
| mlavalle | lajoskatona: and you did all that with sahid's patch, right? | 16:13 |
| opendevreview | Efim Verzakov proposed openstack/neutron master: Add validation to ip_version field https://review.opendev.org/c/openstack/neutron/+/855456 | 16:29 |
| *** dasm is now known as dasm|off | 21:32 | |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: Fix openstack-tox-py39-with-oslo-master job https://review.opendev.org/c/openstack/neutron/+/855703 | 22:16 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!