| opendevreview | Miro Tomaska proposed openstack/neutron master: [WIP] Notify mech drivers with original and modified network https://review.opendev.org/c/openstack/neutron/+/840146 | 02:18 |
|---|---|---|
| *** tbachman_ is now known as tbachman | 04:30 | |
| *** marlinc- is now known as marlinc | 07:26 | |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: CI: add required projects for unit test jobs https://review.opendev.org/c/openstack/networking-bagpipe/+/840166 | 09:32 |
| frickler | lajoskatona: if this patch were in n-d-r I would single-approve it, maybe you could give it a +2 at least? https://review.opendev.org/c/openstack/neutron/+/839792 | 09:49 |
| lajoskatona | frickler: sure, I just checked if we have more people to check it and have some knowledge of BGP | 09:52 |
| frickler | lajoskatona: right, maybe bcafarel or tobias-urdin want to have a look ^^ | 10:15 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Handle properly ObjectNotFound while deleting network from DHCP agent https://review.opendev.org/c/openstack/neutron/+/839779 | 10:27 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Bump oslo.policy to 3.12.0 https://review.opendev.org/c/openstack/neutron/+/840171 | 10:48 |
| opendevreview | Merged openstack/neutron master: [Doc] packet rate limit user workflow and notes https://review.opendev.org/c/openstack/neutron/+/828673 | 11:12 |
| *** whoami-rajat__ is now known as whoami-rajat | 11:15 | |
| lajoskatona | slaweq: Hi, When you have some time could you please check these patches for ODL and Bagpipe: https://review.opendev.org/c/openstack/networking-odl/+/838451 & https://review.opendev.org/c/openstack/networking-bagpipe/+/840166 to open the gate of these | 11:16 |
| tobias-urdin | frickler: ack | 11:17 |
| tobias-urdin | lgtm | 11:18 |
| frickler | tobias-urdin: thx | 11:20 |
| opendevreview | Merged openstack/neutron master: Imported Translations from Zanata https://review.opendev.org/c/openstack/neutron/+/838078 | 11:51 |
| opendevreview | Vishal Manchanda proposed openstack/networking-bgpvpn master: Address RemovedInDjango40Warning https://review.opendev.org/c/openstack/networking-bgpvpn/+/839980 | 12:10 |
| opendevreview | Merged openstack/python-neutronclient master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/python-neutronclient/+/840092 | 13:10 |
| slaweq | lajoskatona: hi, can You maybe approve https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/838460 ? | 13:12 |
| slaweq | or do You think something more would be needed there? | 13:12 |
| lajoskatona | slaweq: sure, I checked, and waited for experimental, but now see your comment | 13:18 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Consolidate neutron-tempest-plugin-api with scenario jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/836912 | 13:19 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Make test test_remove_associated_ports to be compatible with L3HA https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/838460 | 13:19 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Consolidate neutron-tempest-plugin-api with scenario jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/836912 | 13:20 |
| slaweq | lajoskatona: I just rebased itso if You can give +W again, that would be great | 13:20 |
| slaweq | and thx for help with it | 13:20 |
| opendevreview | Merged openstack/neutron stable/train: Docs build: Add requirements.txt to deps https://review.opendev.org/c/openstack/neutron/+/839915 | 13:20 |
| slaweq | lajoskatona: and the result of that patch should be visible in | 13:20 |
| slaweq | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/836912 | 13:20 |
| slaweq | :) | 13:20 |
| opendevreview | Merged openstack/neutron-lib master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/neutron-lib/+/840099 | 13:27 |
| lajoskatona | slaweq: When you have some time could you please check these patches for ODL and Bagpipe: https://review.opendev.org/c/openstack/networking-odl/+/838451 & https://review.opendev.org/c/openstack/networking-bagpipe/+/840166 to open the gate of these projects | 13:31 |
| opendevreview | Merged openstack/neutron stable/yoga: [API] Return 403 for POST requests when user is not authorized https://review.opendev.org/c/openstack/neutron/+/837487 | 13:52 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [bgpvpn] Remove _create_network & _log_console_output https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/837495 | 13:52 |
| slaweq | lajoskatona: +2 on both | 14:02 |
| mlavalle | lajoskatona: tomorrow I might not make it to the weekly Neutron meeting, depending on whether an appointment gets delayed or not | 14:10 |
| lajoskatona | mlavalle: ok, thanks | 14:11 |
| lajoskatona | mlavalle: slaweq will be off also, perhaps we can skip tomorrow's meeting | 14:12 |
| mlavalle | lajoskatona: chances are I'll be on time to the meeting. But with medical appointments you never know, at least in this country. You can't be late but they can | 14:13 |
| lajoskatona | mlavalle: ok, (I sadly hear your sentence about medical appointments, as it seems these things are global ;)) | 14:16 |
| slaweq | lajoskatona: +1 for skipping it tomorrow | 14:23 |
| mlavalle | slaweq: ci meeting tomorrow? | 14:23 |
| slaweq | mlavalle: I'm just checking now how it is going this week | 14:23 |
| slaweq | and I will send email about it | 14:23 |
| mlavalle | slaweq: thanks | 14:24 |
| lajoskatona | mlavalle, slaweq: last week we said that skip CI meeting | 14:24 |
| slaweq | lajoskatona: we said that I will check it on Monday | 14:24 |
| slaweq | and if there will be nothing critical we will skip it | 14:24 |
| mlavalle | yeap, we said to check today before making a decision | 14:24 |
| slaweq | if there will be something, ralonsoh will chair the meeting :) | 14:24 |
| slaweq | and I'm just checking it now | 14:25 |
| lajoskatona | slaweq, mlavalle: then et's skip team meeting also, I added some topic for jemmy jellyfish which would be good to discuss with the whole team | 14:25 |
| slaweq | lajoskatona: mlavalle if You have few minutes, please check https://review.opendev.org/c/openstack/neutron/+/840171 | 14:38 |
| slaweq | it's pretty small patch :) | 14:38 |
| slaweq | thx in advance | 14:38 |
| mlavalle | slaweq: approved | 14:41 |
| slaweq | mlavalle: lajoskatona thx, that was fast :) | 14:44 |
| slaweq | mlavalle: lajoskatona ralonsoh ykarel I just sent email to cancel our tomorrow's ci meeting | 14:45 |
| ykarel | ack | 14:46 |
| slaweq | I don't see anything urgent/critical to discuss this week | 14:46 |
| mlavalle | slaweq: ack. thanks! | 14:46 |
| ykarel | we also have public holiday tomorrow | 14:46 |
| lajoskatona | ykarel, slaweq, mlavalle: ok, I send out also the mail about canceling the meeting | 14:56 |
| slaweq | lajoskatona++ thx | 14:56 |
| mlavalle | lajoskatona: ack, thanks! | 15:57 |
| opendevreview | Merged openstack/neutron master: Bump oslo.policy to 3.12.0 https://review.opendev.org/c/openstack/neutron/+/840171 | 16:18 |
| opendevreview | Merged openstack/neutron master: doc: update peering network BGP guide https://review.opendev.org/c/openstack/neutron/+/839792 | 16:18 |
| lajoskatona | haleyb: Hi, shall I have an IPv6 question? | 16:38 |
| haleyb | lajoskatona: sure, what's the question? | 16:39 |
| lajoskatona | haleyb: our downstream team found again an old bug: https://bugs.launchpad.net/neutron/+bug/1719806 | 16:40 |
| lajoskatona | haleyb: the story is that if you create a net with IPv6 subnet boot a VM on it, and after you decide to add an IPv4 net to it the port is not updated automatically with it (it works by manually updating the port) | 16:41 |
| lajoskatona | haleyb: do you remember by chance if there is something behind it our just a bug in the code? | 16:41 |
| lajoskatona | haleyb: In the code I would add another condition here to hanlde the port update for IPv4 subnet alaso: https://opendev.org/openstack/neutron/src/branch/master/neutron/db/db_base_plugin_v2.py#L771 but I fear that there is a story behind it perhaps :-) | 16:43 |
| haleyb | lajoskatona: that is an old bug :) I'm not sure it's a bug as comment 1 mentions - in the case an IPv4 subnet is added first, then an IPv6 stateful (dhcp) subnet is added, ports aren't automatically update either | 16:43 |
| lajoskatona | haleyb: that's why add_auto_addrs_on_network_ports() is called to select only the ones which needs update really, am I right? | 16:44 |
| haleyb | IPv6 stateless is a little different as the Router Advertisement will start to be sent and VMs will just configure the address | 16:44 |
| * haleyb looks at code link | 16:44 | |
| lajoskatona | haleyb: but if IPv4 is added later that and dhcp is enabled that is something to start port update as I see | 16:46 |
| haleyb | So the comment above that code seems to agree - if it's "auto" addressing update the ports else don't | 16:47 |
| haleyb | lajoskatona: so if dhcp is disabled then enabled after subnet add things work? | 16:48 |
| lajoskatona | haleyb: you mean add the 2nd subnet than disable / re-enable dhcp? that I have't tried | 16:49 |
| haleyb | lajoskatona: maybe i didn't understand your last comment about dhcp being enabled | 16:50 |
| haleyb | " but if IPv4 is added later that and dhcp is enabled that is something to start port update as I see" | 16:50 |
| lajoskatona | haleyb: I mean I create an IPv4 subnet with dhcp enabled, and add that to the network | 16:51 |
| lajoskatona | haleyb: that should trigger the port update, no? | 16:51 |
| haleyb | lajoskatona: in that case does an existing VM have it's port updated? i didn't think it did | 16:54 |
| haleyb | i guess there'd have to be a subnet to boot a VM on the network though... | 16:58 |
| lajoskatona | haleyb: yeah that is the IPv6 subnet on which the VM is booted, and the IPv4 subnet added later | 17:06 |
| haleyb | lajoskatona: i'm trying to look at this from the other perspective - what are the downsides to auto updating all the ports? there is probably more than one reason not to explicitly do it | 17:13 |
| haleyb | for example, if i have a network with a single IPv4 subnet, VMs booted, and add an other since it's addresses are exhausted, automatically updating ports will exhaust that one too. | 17:15 |
| haleyb | theoretically an IPv6 subnet can support more ports than an IPv4 one, so updating all the ports could fail | 17:15 |
| *** dasm|ruck|off is now known as dasm|ruck | 17:16 | |
| lajoskatona | haleyb: hmmm, sounds reasonable | 17:32 |
| lajoskatona | haleyb: I update the bug with these, could you check it tomorrow if I havent written dumb things, and I will update it to be invalid with explanation | 17:34 |
| lajoskatona | haleyb: perhaps we can add also a line / warning to doc also, what do you think? | 17:34 |
| haleyb | lajoskatona: that's only my opinion of course, I've been known to be wrong. And making sure the docs have a reason for this would be a good thing, it might have only been documented in a spec or something when IPv6 was added, don't know | 17:35 |
| lajoskatona | haleyb: sounds like a plan then, and during the review perhaps others appear to add their opinion also | 17:45 |
| opendevreview | Ghanshyam proposed openstack/neutron master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/neutron/+/840087 | 18:02 |
| opendevreview | Ghanshyam proposed openstack/networking-bagpipe master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/networking-bagpipe/+/840098 | 18:10 |
| opendevreview | Ghanshyam proposed openstack/networking-odl master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/networking-odl/+/840097 | 18:16 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: Avoid register config options on imports https://review.opendev.org/c/openstack/neutron/+/839783 | 21:34 |
| *** dasm|ruck is now known as dasm|ruck|bbl | 22:23 | |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [fwaas] Remove create_server & _create_network https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/837520 | 23:20 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [fwaas] Remove _log_console_output & ping_ip_address https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/837521 | 23:20 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: Avoid race condition when deleting trunk bridges https://review.opendev.org/c/openstack/neutron/+/837780 | 23:28 |
| opendevreview | Merged openstack/neutron master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/neutron/+/840087 | 23:58 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [fwaas] Remove _get_server_port_id_and_ip4 & create_floating_ip https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/837545 | 23:58 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [fwaas] Remove get_remote_client & check_vm_connectivity https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/837692 | 23:58 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [fwaas] Remove _create_subnet & setup_clients https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/837700 | 23:58 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Make test test_remove_associated_ports to be compatible with L3HA https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/838460 | 23:59 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [bgpvpn] Remove _get_server_port_id_and_ip4 & create_floating_ip https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/837508 | 23:59 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!