| opendevreview | Miguel Lavalle proposed openstack/neutron master: [WIP][DNM][OVN] Change the default firewall policy https://review.opendev.org/c/openstack/neutron/+/839066 | 00:33 |
|---|---|---|
| *** dmitriis is now known as Guest22 | 00:56 | |
| opendevreview | Merged openstack/neutron stable/ussuri: ovn: first tear down old metadata namespaces, then deploy new https://review.opendev.org/c/openstack/neutron/+/864981 | 03:00 |
| opendevreview | Luca Czesla proposed openstack/neutron master: Add address scope to the OVN LSP port registers https://review.opendev.org/c/openstack/neutron/+/861719 | 07:48 |
| bcafarel | slaweq: good morning can you check https://review.opendev.org/c/openstack/neutron/+/865163/ ? a backport in a chain that missed for second +2 (all others are W+1) | 08:00 |
| ralonsoh | I can check it | 08:02 |
| slaweq | thx ralonsoh - You were faster than me :) | 08:09 |
| bcafarel | ralonsoh: thanks! | 08:09 |
| slaweq | frickler hi, please check https://review.opendev.org/c/openstack/devstack/+/865014/ when You will have some time | 08:15 |
| slaweq | it's removal of the lib/neutron (finally) :) | 08:15 |
| slaweq | thx in advance | 08:15 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Use a secure hash method: replace "sha1" with "sha256" https://review.opendev.org/c/openstack/neutron-lib/+/865578 | 08:18 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: "convert_to_sanitized_mac_address" accepts netaddr.EUI type values https://review.opendev.org/c/openstack/neutron-lib/+/865517 | 08:18 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove deprecation warning of "ovs-ofctl" usage https://review.opendev.org/c/openstack/neutron/+/865588 | 08:19 |
| ralonsoh | folks, we need https://review.opendev.org/c/openstack/neutron/+/865590 in order to bump n-lib | 08:20 |
| ralonsoh | patch for requirements blocked: https://review.opendev.org/c/openstack/requirements/+/864899 | 08:20 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Fix bulk create without mac https://review.opendev.org/c/openstack/neutron/+/863881 | 08:45 |
| frickler | ralonsoh: may I ask how that change is handled during upgrades? | 08:47 |
| ralonsoh | frickler, what change? | 08:48 |
| frickler | ralonsoh: the table number change in n-lib that requires https://review.opendev.org/c/openstack/neutron/+/865590 | 08:49 |
| ralonsoh | frickler, first of all, I was against a change in the FW table numbers. In any case, this number is used only by the OVS agent locally. When the agent is restarted with the new n-lib, the OF table is rebuilt | 08:51 |
| frickler | does that happen on any agent restart? that would mean it would always disrupt traffic iiuc | 08:52 |
| ralonsoh | frickler, that could affect to the conntrack entries, but should be rebuilt inmediately | 08:53 |
| frickler | "should" is a pretty soft term, but likely that's a more general issue, thx anyway | 08:57 |
| ralonsoh | frickler, this is what is happening right now, even without this change | 08:57 |
| ralonsoh | the datapath will change, but that means a packet will cross now table 58 instead of 59 | 08:58 |
| ralonsoh | but this change is done in the OF table in one shot | 08:58 |
| opendevreview | Maximilian Stinsky proposed openstack/neutron master: Fix duplicated routes exceptions https://review.opendev.org/c/openstack/neutron/+/865525 | 08:58 |
| frickler | ralonsoh: yes, not a new issue, I agree | 09:07 |
| opendevreview | Merged openstack/neutron master: Fix missing [designate] options https://review.opendev.org/c/openstack/neutron/+/865220 | 09:07 |
| frickler | slaweq: looking pretty good already, just two minor comments | 09:07 |
| opendevreview | Merged openstack/neutron stable/wallaby: Delete MAC binding for LRP when the port is deleted https://review.opendev.org/c/openstack/neutron/+/865163 | 09:07 |
| opendevreview | Merged openstack/neutron stable/wallaby: Ensure only one worker creates neturon_pg_drop https://review.opendev.org/c/openstack/neutron/+/865167 | 09:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add an env variable "PROCESS_TAG" in ``ProcessManager`` https://review.opendev.org/c/openstack/neutron/+/865018 | 09:14 |
| sahid | lajoskatona: I commented on your patches, I hope that will make sense, thanks ! | 09:30 |
| lajoskatona | sahid: thanks I will check it | 09:31 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-octavia-provider master: Make clear distinction between health check and health monitor https://review.opendev.org/c/openstack/ovn-octavia-provider/+/864817 | 09:36 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-octavia-provider master: Make clear distinction between health check and health monitor https://review.opendev.org/c/openstack/ovn-octavia-provider/+/864817 | 10:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Avoid race condition when deleting trunk bridges https://review.opendev.org/c/openstack/neutron/+/837780 | 10:20 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DNM] Testing patch https://review.opendev.org/c/openstack/neutron/+/865039 | 10:23 |
| opendevreview | Merged openstack/neutron master: Configure placement credentials for segments plugin https://review.opendev.org/c/openstack/neutron/+/815093 | 11:07 |
| opendevreview | Merged openstack/neutron stable/xena: Delete MAC binding for LRP when the port is deleted https://review.opendev.org/c/openstack/neutron/+/865162 | 11:07 |
| stephenfin | ralonsoh: Afternoon o/ Does this error message look familiar to you? https://storage.googleapis.com/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_cluster-api-provider-openstack/1396/pull-cluster-api-provider-openstack-e2e-test/1595934082498301952/artifacts/devstack/controller-devstack.log | 11:22 |
| ralonsoh | stephenfin, yes but not there | 11:22 |
| ralonsoh | stephenfin, let me check that | 11:22 |
| ralonsoh | is the first time you see it? | 11:23 |
| stephenfin | Yeah, I see a bug from Thomas Goirand but that's only in the unit tests https://bugs.launchpad.net/neutron/+bug/1993502 | 11:23 |
| ralonsoh | stephenfin, this is only for UTs | 11:23 |
| stephenfin | Yup. There's a bug in k8s cluster-api-provider-openstack https://github.com/kubernetes-sigs/cluster-api-provider-openstack/issues/1397 | 11:23 |
| ralonsoh | stephenfin, is that using neutron master? | 11:24 |
| stephenfin | I don't think so. I think we use stable devstack | 11:24 |
| stephenfin | stable/yoga iirc, but I'm checking | 11:25 |
| stephenfin | yes yoga https://github.com/kubernetes-sigs/cluster-api-provider-openstack/blob/main/hack/ci/create_devstack.sh#L34 | 11:27 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Support for minimum bandwidth rules in tunnelled networks https://review.opendev.org/c/openstack/neutron/+/860639 | 11:27 |
| ralonsoh | stephenfin, ok, I'll check it now | 11:27 |
| stephenfin | the bug mentions this commit which was merged three days ago https://github.com/openstack/neutron/commit/44ec1f150338ec69a811e1b2cfa4214c07a9af83 | 11:28 |
| stephenfin | that would coincide with this | 11:28 |
| opendevreview | Merged openstack/neutron stable/ussuri: OVN: Add support for DHCP option "domain-search" for IPv4 https://review.opendev.org/c/openstack/neutron/+/865011 | 11:29 |
| stephenfin | FYI here's our local.conf https://github.com/kubernetes-sigs/cluster-api-provider-openstack/blob/main/hack/ci/cloud-init/controller.yaml.tpl | 11:32 |
| stephenfin | My initial guess is that we're enabling some combination of services that isn't tested upstream on stable/yoga, but I'm checking myself too | 11:32 |
| ralonsoh | stephenfin, are there more logs? are those logs from the neutron-server? | 11:36 |
| ralonsoh | well, they are | 11:36 |
| ralonsoh | but what I mean is if you have all the neutron-server logs | 11:36 |
| stephenfin | Let me check | 11:36 |
| stephenfin | okay, here's a recent prow build for CAPO https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_cluster-api-provider-openstack/1393/pull-cluster-api-provider-openstack-e2e-test/1596076314777882624 | 11:37 |
| stephenfin | no more logs | 11:37 |
| stephenfin | https://gcsweb.k8s.io/gcs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_cluster-api-provider-openstack/1393/pull-cluster-api-provider-openstack-e2e-test/1596076314777882624/artifacts/devstack/ | 11:37 |
| stephenfin | ralonsoh: I'll modify the job and get you some. Will be a few hours | 11:38 |
| ralonsoh | stephenfin, thanks a lot. | 11:38 |
| ralonsoh | please ping me | 11:38 |
| labedz | hi guys, I am thinking about bumping python ovs libraries: ovs and ovsdbapp like: https://review.opendev.org/c/openstack/requirements/+/865672 | 11:51 |
| labedz | but I am supprised that 'proper' version is since Yoga | 11:51 |
| labedz | am I missing anything? | 11:51 |
| ralonsoh | labedz, sorry, that does it means? | 11:52 |
| labedz | ok, let's rewind :) | 11:53 |
| labedz | there is a bug: https://bugs.launchpad.net/os-vif/+bug/1929446 that impact me | 11:53 |
| labedz | fix appear with new libraries ovs 2.16 and ovsdbapp 1.14 | 11:53 |
| mnasiadka | ovs is tricky, because it should match your installed openvswitch version | 11:53 |
| labedz | but until Yoga we are still using versions bellow | 11:53 |
| mnasiadka | (or at least not be older) - from what we've noticed over time ;) | 11:54 |
| labedz | which means that without 'admin tuning' we are affected | 11:54 |
| ralonsoh | I would need to check but I don't know if we are using ovs 2.16 in the CI | 11:54 |
| labedz | mnasiadka: true | 11:54 |
| ralonsoh | labedz, now we are using Ubuntu 22.04, we can consider using newer OVS versions | 11:56 |
| ralonsoh | I'll check if we can bump to this pyhton-ovs version, along with the ovsdbapp library | 11:56 |
| labedz | but if you are unlucky this issue can break your system totally (like not processing 50% of OVN notification) | 11:57 |
| ralonsoh | labedz, https://review.opendev.org/c/openstack/requirements/+/865672 won't be accepted | 12:00 |
| ralonsoh | CI in ussuri is not running ubuntu 20.04 | 12:00 |
| ralonsoh | this OS is installing an older version of OVS | 12:00 |
| labedz | sure - just an example | 12:00 |
| ralonsoh | actually I though I was looking at a master branch patch | 12:01 |
| ralonsoh | this is already solved in master | 12:01 |
| ralonsoh | I mean, the versions are bumped | 12:01 |
| labedz | yes - ut's since Yoga | 12:01 |
| ralonsoh | in your env (Ussuri) you'll need to use a never OVS version with newer python-ovs and ovsdbapp | 12:01 |
| opendevreview | Felix Huettner proposed openstack/neutron stable/yoga: Since OVN 20.06, config is stored in "Chassis.other_config" https://review.opendev.org/c/openstack/neutron/+/863281 | 12:02 |
| labedz | now I know :) | 12:02 |
| labedz | but would be nice to have some sollution to mittigate this issue | 12:02 |
| labedz | as we have still some users <Yoga :) | 12:02 |
| ralonsoh | labedz, requirements is for testing, that doesn't prevent you from installing newer versions in your env | 12:03 |
| labedz | understand | 12:04 |
| labedz | still: tests are also impacted (there is some probability that issue will hit) and operators has to be at least aware of the issue somehow | 12:05 |
| labedz | anyway smth to consider | 12:05 |
| ralonsoh | sahid, hey, please check https://bugs.launchpad.net/neutron/+bug/1997964 | 12:23 |
| stephenfin | Quick q: Can an extension _remove_ API routes or response fields? Can it modify response fields (i.e. switch from string to list)? | 12:38 |
| ralonsoh | stephenfin, when we build the resources, we create the maps. By default we use the n-lib API definitions. There are some of them that are "built" in Neutron, but that's all | 12:40 |
| ralonsoh | are you talking aboyt something specific? | 12:41 |
| stephenfin | I'm trying to describe in a sentence or two how neutron does API versioning (for OSC docs) | 12:41 |
| stephenfin | so far I have this | 12:41 |
| stephenfin | .. rubric:: Extensions | 12:41 |
| ralonsoh | stephenfin, we don't have API versioning | 12:41 |
| stephenfin | This is used by the Networking services (neutron). An extension can add | 12:41 |
| ralonsoh | we have the extensions | 12:41 |
| stephenfin | features and vendor-specific functionality to the API. This can include new API | 12:41 |
| stephenfin | resources and well as new fields in API requests and responses. If you want to | 12:42 |
| stephenfin | depend on a feature added by an extension, you can check if the extension is | 12:42 |
| stephenfin | present. | 12:42 |
| ralonsoh | stephenfin, yes, that could describe how is done in Neutron | 12:42 |
| ralonsoh | and yes, we don't have a version number but you can check if the extension is enabled | 12:43 |
| stephenfin | Cool. So my question is does the n-lib API definitions ever remove stuff or is purely additive? | 12:43 |
| ralonsoh | we can add, remove or modify | 12:43 |
| stephenfin | i.e. if extension foo was present, could that mean /bar disappears | 12:43 |
| stephenfin | okay, sweet | 12:43 |
| stephenfin | so instead of | 12:44 |
| stephenfin | "An extension can add features and vendor-specific functionality to the API" | 12:44 |
| stephenfin | it would be better to say | 12:44 |
| stephenfin | "An extension can add, remove or features and vendor-specific functionality to the API" | 12:44 |
| stephenfin | *add, remove or modify | 12:45 |
| ralonsoh | right | 12:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Avoid race condition when deleting trunk bridges https://review.opendev.org/c/openstack/neutron/+/837780 | 12:49 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DNM] Testing patch https://review.opendev.org/c/openstack/neutron/+/865039 | 12:49 |
| opendevreview | Felix Huettner proposed openstack/neutron stable/yoga: Since OVN 20.06, config is stored in "Chassis.other_config" https://review.opendev.org/c/openstack/neutron/+/863281 | 12:51 |
| opendevreview | Merged openstack/neutron stable/xena: ovn: Use ovsdb-client to create neutron_pg_drop https://review.opendev.org/c/openstack/neutron/+/865161 | 13:30 |
| slaweq | ralonsoh hi, I just wrote a question in https://review.opendev.org/c/openstack/neutron-lib/+/865578 - if this will not be a problem, I'm happy to approve it :) | 14:04 |
| slaweq | but please check and reply to my question first | 14:04 |
| ralonsoh | slaweq, ufff hold on, I didn't think about this... to be honest, how we build the interface name is trivial, but we should be consistent | 14:06 |
| ralonsoh | I'll change it, skipping the pylint error | 14:07 |
| opendevreview | Elvira García Ruiz proposed openstack/python-neutronclient master: Fix help sentence in network log create --enable https://review.opendev.org/c/openstack/python-neutronclient/+/865694 | 14:09 |
| opendevreview | Elvira García Ruiz proposed openstack/python-neutronclient master: Fix help sentence in network log create --enable https://review.opendev.org/c/openstack/python-neutronclient/+/865694 | 14:10 |
| slaweq | I think You can add parameter "usedforsecurity=False" or something like that | 14:10 |
| slaweq | and it will not complain anymore | 14:11 |
| slaweq | IIRC | 14:11 |
| ralonsoh | I prefer to skip just this warning there | 14:11 |
| ralonsoh | ah no, I know what you mean now | 14:12 |
| slaweq | :) | 14:15 |
| *** tbachman_ is now known as tbachman | 14:18 | |
| sahid | o/ Neutron steam, how do you feel about to ebable the feature multisegments? https://review.opendev.org/c/openstack/neutron/+/841838 Even if ofcourse I will be glad to ensure that it is merged for this release :-) I think it's a good point to have it merged sooner in the release so we can address and fix any bugs that may exist :D | 14:19 |
| sahid | s/steam/team | 14:19 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: User "sha1" with "usedforsecurity=False" https://review.opendev.org/c/openstack/neutron-lib/+/865578 | 14:20 |
| ralonsoh | sahid, hey, please check https://bugs.launchpad.net/neutron/+bug/1997964 | 14:21 |
| sahid | ralonsoh: oh! let me see what's going on and share a fix | 14:24 |
| sahid | i mean, are you woirking on it? | 14:24 |
| ralonsoh | not today | 14:24 |
| sahid | ralonsoh: you see... having the whole thing merged soon as possible is better ;-) | 14:24 |
| sahid | let me assign myself on it | 14:24 |
| opendevreview | Elvira García Ruiz proposed openstack/python-neutronclient master: Fix help sentence in network log create --enable https://review.opendev.org/c/openstack/python-neutronclient/+/865694 | 14:33 |
| sahid | lajoskatona: just to be sure, I have assigned myself on https://bugs.launchpad.net/neutron/+bug/1997964, you are not on it right? | 14:47 |
| sahid | the whole point is to understand when we can be in a sitatuation of calling this method with network == None, if it is ligit, then we could just return I guess since there are nothing we can do | 14:50 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: Skip "test_port_resource_request_no_provider_net_conflict" https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/863880 | 14:56 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Support for minimum bandwidth rules in tunnelled networks https://review.opendev.org/c/openstack/neutron/+/860639 | 14:56 |
| lajoskatona | sahid: no actually I just checked when it was introduced, so don't worry I am not on it :P | 14:58 |
| opendevreview | Maximilian Stinsky proposed openstack/neutron master: Fix duplicated routes exceptions https://review.opendev.org/c/openstack/neutron/+/865525 | 15:15 |
| opendevreview | Merged openstack/neutron master: Use the neutron-lib constant for PACKET_RATE_LIMIT table number https://review.opendev.org/c/openstack/neutron/+/865590 | 15:29 |
| sahid | lajoskatona: ok cool, don't revert anything I will fix it soon :-) I noticed you have marked it as Critical, It may be a bit too much I think :-) | 15:41 |
| ralonsoh | this is the default priority if a bug is affecting the CI | 15:42 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Fix behaviour of enable/disable in OVN network log https://review.opendev.org/c/openstack/neutron/+/864152 | 15:42 |
| opendevreview | Felix Huettner proposed openstack/neutron master: Fix handling the restart of ovn-controllers https://review.opendev.org/c/openstack/neutron/+/865697 | 15:43 |
| *** elvira1 is now known as elvira | 16:18 | |
| opendevreview | Merged openstack/ovn-octavia-provider master: Ensure HM updates work as expected https://review.opendev.org/c/openstack/ovn-octavia-provider/+/864821 | 17:05 |
| sahid | ralonsoh: ack | 17:27 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-octavia-provider stable/zed: Ensure HM updates work as expected https://review.opendev.org/c/openstack/ovn-octavia-provider/+/865710 | 17:30 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-octavia-provider stable/yoga: Ensure HM updates work as expected https://review.opendev.org/c/openstack/ovn-octavia-provider/+/865711 | 17:31 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-octavia-provider stable/xena: Ensure HM updates work as expected https://review.opendev.org/c/openstack/ovn-octavia-provider/+/865712 | 17:31 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-octavia-provider stable/wallaby: Ensure HM updates work as expected https://review.opendev.org/c/openstack/ovn-octavia-provider/+/865713 | 17:32 |
| opendevreview | Merged openstack/python-neutronclient master: Fix help sentence in network log create --enable https://review.opendev.org/c/openstack/python-neutronclient/+/865694 | 18:06 |
| opendevreview | Maximilian Stinsky proposed openstack/neutron master: Fix duplicated routes exceptions https://review.opendev.org/c/openstack/neutron/+/865525 | 18:40 |
| opendevreview | Maximilian Stinsky proposed openstack/neutron master: Add function to refresh accept_ra on router state change https://review.opendev.org/c/openstack/neutron/+/824947 | 23:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!