| opendevreview | Merged openstack/neutron stable/victoria: Populate self.floating_ips_dict using "ip rule" information https://review.opendev.org/c/openstack/neutron/+/810394 | 02:16 |
|---|---|---|
| ltomasbo | o/ | 06:15 |
| lajoskatona | slaweq, ralonsoh: Good morning, could you please check these 2 backports: https://review.opendev.org/c/openstack/neutron/+/810975 & https://review.opendev.org/c/openstack/neutron/+/810972 ? | 07:06 |
| ralonsoh | lajoskatona, sure, let me check | 07:08 |
| lajoskatona | ralonsoh: thanks | 07:08 |
| slaweq | lajoskatona: sure, I'm in the meeting now | 07:12 |
| slaweq | but will do when I will have few minutes | 07:12 |
| opendevreview | Merged openstack/neutron-dynamic-routing master: Drop dsvm-functional tox env and related files https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/797044 | 08:44 |
| opendevreview | Merged openstack/neutron stable/rocky: Add extra logs to the network update callback in L3 agent https://review.opendev.org/c/openstack/neutron/+/803581 | 09:54 |
| zigo | slaweq: ralonsoh: FYI, neutron-tempest-plugin cannot be built for me with Tempest 27, I had to force Build-Depends-Indep: tempest >= 29.0.0 | 09:58 |
| zigo | Someone may want to adjust this in your upstream code too. | 09:58 |
| ralonsoh | do you have the logs? just to open a bug to justify this change | 10:00 |
| ralonsoh | slaweq, https://review.opendev.org/c/openstack/neutron/+/807692 | 10:59 |
| ralonsoh | if you have time | 10:59 |
| slaweq | ralonsoh: sure | 11:00 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Add a SB Chassis event to track changes in BW config https://review.opendev.org/c/openstack/neutron/+/801712 | 11:08 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add new indexes to RBAC DB models https://review.opendev.org/c/openstack/neutron/+/810072 | 11:53 |
| gibi | neutrinok: Do you have a hint why the tempest.conf [network-feature-enabled]api_extensions config changed from listing the extensions explicitly to simply stating 'all' ? | 11:57 |
| gibi | the nova-grenade-multinode job fails as tempest sees 'all' and therefore try to run the trunk tests, but the trunk service_plugin is not loaded | 11:59 |
| gibi | on stable/wallaby the api_extensions config had an explicit list without the trunk so the test was skipped there | 12:00 |
| lajoskatona | gibi: here we have api_extensions=all as well: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_f91/805849/10/check/neutron-ovs-tempest-slow/f915b04/controller/logs/tempest_conf.txt | 12:06 |
| lajoskatona | which is strange | 12:06 |
| gibi | it seems on master you get 'all' everywhere (I checked nova-next) | 12:06 |
| gibi | on stable/wallaby I get an explicit list of extensions and in case of grenade I get that list without trunk, which then makes tempest skipping trunk testing | 12:07 |
| gibi | fyi, lyarwood trying to unblock our gate by enabling trunk in grenade https://review.opendev.org/c/openstack/grenade/+/811117 | 12:19 |
| *** Guest6 is now known as kmasterson | 12:25 | |
| *** kmasterson is now known as Guest1123 | 12:25 | |
| *** Guest1123 is now known as kmasterson | 12:26 | |
| lajoskatona | gibi: what I see is that we have different tempest.conf for ovn and ovs based runs and for ovn the list is filled for ovs it is all, after wallaby, but no idea what is the source of that | 13:20 |
| lajoskatona | gibi: at least I can't see any changes that can be related to this difference | 13:20 |
| gibi | lajoskatona: I also wasn't able to find what change caused that we now see 'all' on master | 13:21 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Execute the quota reservation removal in an isolated DB txn https://review.opendev.org/c/openstack/neutron/+/811124 | 13:25 |
| ralonsoh | slaweq, ^^ | 13:25 |
| ralonsoh | maybe I could merge https://review.opendev.org/c/openstack/neutron/+/809983 | 13:26 |
| ralonsoh | and then push this patch commenting the differences | 13:26 |
| lajoskatona | slaweq: Could you please check these vpnaas eoling related patches: https://review.opendev.org/q/intopic:%2522%255E.*-eol-remove-periodic-job%2522+(status:open+OR+status:merged) when you have few minutes? | 13:28 |
| slaweq | lajoskatona: sure | 13:29 |
| slaweq | ralonsoh: like cherry-pick but changed, right? | 13:30 |
| ralonsoh | slaweq, yes, the patch for Xena needs some bits missing in the master branch patch | 13:30 |
| ralonsoh | so maybe we could wait until the master branch patch is merged | 13:30 |
| ralonsoh | and then push the xena one | 13:31 |
| ralonsoh | (we'll need, for the master one, a new n-lib release, so I need to wait until releases create the Y branch) | 13:31 |
| ralonsoh | slaweq, do you have time for this? | 13:31 |
| slaweq | ralonsoh: do I have time to wait for the fix? or for what? | 13:32 |
| ralonsoh | for the master patch to be merged (including the new n-lib version) | 13:33 |
| ralonsoh | and then wait for the xena patch based on the master one | 13:33 |
| slaweq | ralonsoh: I'm not sure how critical this issue may be in the xena | 13:35 |
| lajoskatona | ralonsoh, slaweq: sorry for jumping in :P , do we need this for xena rc2 ? | 13:38 |
| ralonsoh | lajoskatona, I think we need. The workaround is failing in out CI | 13:40 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/810873 | 13:40 |
| lajoskatona | ralonsoh: ok, thanks | 13:41 |
| ralonsoh | lajoskatona, I can't share with you the link of our internal CI | 13:42 |
| ralonsoh | but I can share with you the logs | 13:42 |
| ralonsoh | one sec | 13:42 |
| ralonsoh | lajoskatona, https://file.io/5HhGyPgbesS6 | 13:45 |
| lajoskatona | ralonsoh: it's deleted | 14:20 |
| ralonsoh | lajoskatona, what is deleted? | 14:23 |
| ralonsoh | ah sorry | 14:24 |
| lajoskatona | ralonsoh: https://file.io/5HhGyPgbesS6 | 14:24 |
| ralonsoh | I don't know why | 14:24 |
| ralonsoh | one sec | 14:24 |
| ralonsoh | lajoskatona, https://file.io/j3WEVR7PKg4c | 14:24 |
| ralonsoh | try now | 14:24 |
| ralonsoh | (I think they delete the files in just a few minutes) | 14:24 |
| ralonsoh | search for status: 500 | 14:26 |
| opendevreview | Merged openstack/neutron stable/xena: Add retry when executing OF commands if "InvalidDatapath" https://review.opendev.org/c/openstack/neutron/+/810972 | 14:33 |
| opendevreview | Merged openstack/neutron stable/wallaby: [Docs] Ovn backend now supports FIP QoS bandwdith limiting https://review.opendev.org/c/openstack/neutron/+/810902 | 14:33 |
| opendevreview | Merged openstack/neutron stable/rocky: [L3] Use processing queue for network update events https://review.opendev.org/c/openstack/neutron/+/803582 | 14:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Add a SB Chassis event to track changes in BW config https://review.opendev.org/c/openstack/neutron/+/801712 | 14:36 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove "_get_network_lock_id" compatibility method https://review.opendev.org/c/openstack/neutron/+/800967 | 14:45 |
| lajoskatona | bcafarel: Hi, regarding https://review.opendev.org/c/openstack/neutron-vpnaas/+/809183 (and its group on other branches) | 14:54 |
| lajoskatona | bcafarel: I considered again You comment to remove the jobs or jsut making them nonvoting | 14:55 |
| bcafarel | lajoskatona: it was mostly to have a nice and tidy list, but for a pending EOL it does not change a lot | 14:57 |
| bcafarel | lajoskatona: also have you seen last comments on https://review.opendev.org/c/openstack/releases/+/809180 ? If I got it correctly the jobs defined in these repos will be dropped anyway when the branches are deleted | 14:58 |
| lajoskatona | bcafarel: what's in my mind is what will be in the periodic execution, the whole goal is to not execute any of these in periodic runs | 14:58 |
| lajoskatona | bcafarel: thanks, I read it again carefully, and finally got it :-) | 14:59 |
| lajoskatona | bcafarel: ok, so we have to quickly merged these, or abandon to make the deletion of the branch possible | 15:00 |
| bcafarel | lajoskatona: exactly :) | 15:01 |
| *** redrobot is now known as Guest1129 | 15:09 | |
| frickler | slaweq: lajoskatona: I pushed a patch to update the n-d-r grafana panel, please have a look https://review.opendev.org/c/openstack/project-config/+/811182 | 15:14 |
| frickler | this also got me thinking whether we should update the list of periodic jobs and if yes, how https://zuul.opendev.org/t/openstack/builds?project=openstack%2Fneutron-dynamic-routing&pipeline=periodic | 15:15 |
| lajoskatona | frickler: checking | 15:16 |
| lajoskatona | frickler: regarding periodic jobs, perhaps we can check that on tomorrow's team meeting or CI meeting, what do you think? | 15:17 |
| bcafarel | frickler: for neutron I drop the -master jobs (neutron-lib os-ken/ryu) in stable branches as we don't check old branches running against master libs - that would clean a few off that list (and they are in the "periodic:" parts of .zuul.yaml in their respective branches) | 15:18 |
| frickler | bcafarel: ah, so those were just relics from branching, I was wondering what the intention might have been there | 15:19 |
| frickler | lajoskatona: this week I'm not sure I can make it to either meeting, but I'll read up on it if you want to discuss | 15:20 |
| lajoskatona | frickler: ok, I think I will make pariodic as topic for one of the meetings | 15:26 |
| opendevreview | Bernard Cafarelli proposed openstack/neutron stable/xena: [stable/xena] Drop -master jobs https://review.opendev.org/c/openstack/neutron/+/811187 | 15:40 |
| bcafarel | that reminded me to do some xena cleanup ^ | 15:40 |
| opendevreview | Merged openstack/neutron stable/victoria: [L3] Use processing queue for network update events https://review.opendev.org/c/openstack/neutron/+/803574 | 17:03 |
| opendevreview | Merged openstack/neutron stable/queens: [L3] Use processing queue for network update events https://review.opendev.org/c/openstack/neutron/+/803584 | 17:43 |
| erlon | dalvarez: hey Daniel, I ahve have a question a bout this patch[1] of yours. Not sure if this is the best place to ask, but couldn't find you on #openvswitch@libera, so let me know if there's a better place. The question is if you also had in mind fixing that for the case were the arp is not gratuitous. I ask that because I can reproduce the problem in an ovn version with the backported fix (v20.03.2), but only if I ping the | 17:56 |
| erlon | internal port address or arping it. | 17:56 |
| erlon | [1] https://github.com/ovn-org/ovn/commit/578238b36073256c524a4c2b6ed7521f73aa0019 | 17:56 |
| -opendevstatus- NOTICE: Gerrit and Zuul services are being restarted briefly for configuration and code updates but should return to service momentarily | 20:10 | |
| sdanni | Hi all! I'm using a system scoped admin to run 'openstack port create' command and it returns error 'BadRequestException: 400: ... Running without keystone AuthN requires that tenant_id is specified.' I wonder if this is an issue resulting from secure rbac? | 20:24 |
| spatel | I am seeing this error - Launch Instance Fails with "Failed to allocate the network(s), not rescheduling." | 21:05 |
| spatel | what could be wrong? | 21:05 |
| opendevreview | Hang Yang proposed openstack/neutron master: Add shared field to SG API response and filter https://review.opendev.org/c/openstack/neutron/+/811242 | 22:32 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!