| gmann | slaweq: lajoskatona not sure if you have noticed it before, tempest-integrated-compute-centos-8-stream job is failing consistently . https://169dddc67bd535a0361f-0632fd6194b48b475d9eb0d8f7720c6c.ssl.cf2.rackcdn.com/824478/5/check/tempest-integrated-compute-centos-8-stream/e0db6a9/testr_results.html | 02:10 |
|---|---|---|
| gmann | seeing this traceback in log https://zuul.opendev.org/t/openstack/build/e0db6a9a7ba04e66b0781ba7d259357d/log/controller/logs/screen-q-svc.txt#32875 | 02:10 |
| *** EugenMayer3 is now known as EugenMayer | 04:05 | |
| frickler | gmann: slaweq: isn't that possibly just the "unpriviledged ping fails" issue discussed in #opendev? | 05:32 |
| lajoskatona | frickler, gmann: thanks for highlight | 07:36 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron-tempest-plugin master: Add local ip scenario tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/823007 | 07:58 |
| ralonsoh | bcafarel, slaweq https://review.opendev.org/q/topic:%22bug%252F1956476%22+(status:open%20OR%20status:merged) | 08:17 |
| ralonsoh | good morning! | 08:17 |
| ralonsoh | if you have some time, thanks in advance | 08:17 |
| bcafarel | ralonsoh: sure, and good morning :) | 08:17 |
| slaweq | ralonsoh: hi, sure, give me 10 minutes and I will check it :) | 08:19 |
| bcafarel | ralonsoh: https://review.opendev.org/c/openstack/neutron/+/824348/1/neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_impl_idl.py#89 did not work in train backport? | 08:33 |
| ralonsoh | bcafarel, yes but in networking-ovn | 08:34 |
| ralonsoh | https://review.opendev.org/c/openstack/networking-ovn/+/824359 | 08:34 |
| bcafarel | ah yes I was asking about test_multiple_physnets_in_one_bridge test | 08:36 |
| bcafarel | it is not in https://review.opendev.org/c/openstack/networking-ovn/+/824359 | 08:37 |
| opendevreview | liuyulong proposed openstack/neutron master: Add missing DHCP opts for genconfig https://review.opendev.org/c/openstack/neutron/+/824679 | 08:37 |
| ralonsoh | bcafarel, ahhhh sorry | 08:37 |
| ralonsoh | this is because I did a manual cherry-pick | 08:37 |
| ralonsoh | bcafarel, I'll push a new patch | 08:38 |
| bcafarel | ralonsoh: ack thanks :) | 08:38 |
| opendevreview | Rodolfo Alonso proposed openstack/networking-ovn stable/train: [OVN] Allow only one physical network per bridge https://review.opendev.org/c/openstack/networking-ovn/+/824359 | 08:39 |
| slaweq | ralonsoh: bcafarel lajoskatona just FYI, we have some issue with functional tests job: https://bugs.launchpad.net/neutron/+bug/1957887 | 08:40 |
| slaweq | I will work on it right now | 08:41 |
| slaweq | obondarev: also FYI ^^ | 08:41 |
| lajoskatona | slaweq: thanks | 08:41 |
| ralonsoh | thanks, I'll take a look too | 08:42 |
| ralonsoh | slaweq, hehehe this in configure_for_testing.sh | 08:46 |
| ralonsoh | compile_ovn False /usr /var | 08:46 |
| ralonsoh | should be | 08:46 |
| ralonsoh | compile_ovn /usr /var | 08:46 |
| ralonsoh | you removed the first input parameter | 08:47 |
| ralonsoh | https://review.opendev.org/c/openstack/devstack/+/822717 | 08:47 |
| ralonsoh | https://codesearch.openstack.org/?q=compile_ovn&i=nope&literal=nope&files=&excludeFiles=&repos= | 08:48 |
| ralonsoh | seems that this is the only place, outside devstack using it | 08:48 |
| obondarev | slaweq: Hi! Sorry, was offline, could you repeat what was FYI? | 08:48 |
| ralonsoh | obondarev, ^^ what I have described in previous lines | 08:49 |
| slaweq | ralonsoh: true, and there is patch for that already: https://review.opendev.org/c/openstack/neutron/+/822719 | 08:50 |
| slaweq | I will rebase it right now | 08:50 |
| slaweq | I forgot about it :D | 08:50 |
| ralonsoh | hahahaha | 08:50 |
| slaweq | stupid me :P | 08:50 |
| lajoskatona | We are close the weekend it seems | 08:52 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Fix call to the compile_ovn function from Devstack https://review.opendev.org/c/openstack/neutron/+/822719 | 08:52 |
| ralonsoh | done | 08:53 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Fix call to the compile_ovn function from Devstack https://review.opendev.org/c/openstack/neutron/+/822719 | 08:53 |
| slaweq | obondarev: ralonsoh lajoskatona ^^ patch rebased | 08:54 |
| slaweq | please check it ASAP to unblock our gate :) | 08:54 |
| slaweq | thx in advance | 08:54 |
| lajoskatona | slaweq: done | 08:54 |
| obondarev | slaweq: ack :) | 08:54 |
| slaweq | lajoskatona: thx | 08:54 |
| bcafarel | fixes already pending in gerrit, that is nice :) | 08:57 |
| labedz | hi folks, you may take a look in your spear time: https://bugs.launchpad.net/neutron/+bug/1957794, thanks | 09:36 |
| opendevreview | Merged openstack/os-ken stable/xena: Avoid missing key 'password' for neighbor_add https://review.opendev.org/c/openstack/os-ken/+/824546 | 09:42 |
| opendevreview | Merged openstack/os-ken stable/wallaby: Avoid missing key 'password' for neighbor_add https://review.opendev.org/c/openstack/os-ken/+/824547 | 09:48 |
| opendevreview | Merged openstack/os-ken stable/victoria: Avoid missing key 'password' for neighbor_add https://review.opendev.org/c/openstack/os-ken/+/824548 | 09:48 |
| opendevreview | Merged openstack/os-ken stable/ussuri: Avoid missing key 'password' for neighbor_add https://review.opendev.org/c/openstack/os-ken/+/824549 | 09:48 |
| opendevreview | Merged openstack/os-ken stable/train: Avoid missing key 'password' for neighbor_add https://review.opendev.org/c/openstack/os-ken/+/824610 | 09:48 |
| opendevreview | Rodolfo Alonso proposed openstack/ovsdbapp master: Update QoS register in "QoSAddCommand" if exists https://review.opendev.org/c/openstack/ovsdbapp/+/822138 | 09:51 |
| opendevreview | Merged openstack/neutron stable/wallaby: [OVN] Allow only one physical network per bridge https://review.opendev.org/c/openstack/neutron/+/824346 | 10:09 |
| opendevreview | Merged openstack/neutron stable/victoria: [OVN] Allow only one physical network per bridge https://review.opendev.org/c/openstack/neutron/+/824347 | 10:22 |
| opendevreview | Merged openstack/neutron stable/ussuri: [OVN] Allow only one physical network per bridge https://review.opendev.org/c/openstack/neutron/+/824348 | 10:23 |
| opendevreview | Merged openstack/neutron master: Fix call to the compile_ovn function from Devstack https://review.opendev.org/c/openstack/neutron/+/822719 | 10:23 |
| slaweq | gmann: frickler I don't know really why it's failing like that, please open bug for it and I will check it | 10:44 |
| opendevreview | Oleg Bondarev proposed openstack/neutron master: Add Local IP fullstack test cases https://review.opendev.org/c/openstack/neutron/+/816327 | 11:02 |
| opendevreview | Oleg Bondarev proposed openstack/neutron master: Local IP internal documentation and release note https://review.opendev.org/c/openstack/neutron/+/820031 | 11:03 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: "L3AgentExtensionsManager" check loaded extensions https://review.opendev.org/c/openstack/neutron/+/818540 | 11:06 |
| opendevreview | Merged openstack/neutron master: Make configure_for_func_testing compatible with e.g. Centos https://review.opendev.org/c/openstack/neutron/+/799625 | 11:17 |
| ralonsoh | slaweq, https://review.opendev.org/c/openstack/requirements/+/824699 | 11:46 |
| slaweq | ralonsoh: thx, it makes sense for me | 11:51 |
| opendevreview | Oleg Bondarev proposed openstack/neutron master: Add Local IP fullstack test cases https://review.opendev.org/c/openstack/neutron/+/816327 | 12:41 |
| *** dasm|off is now known as dasm | 13:32 | |
| frickler | slaweq: this seems to be a good description of the situation https://lists.openstack.org/pipermail/openstack-discuss/2022-January/026682.html | 13:37 |
| lajoskatona | #startmeeting neutron_drivers | 14:01 |
| opendevmeet | Meeting started Fri Jan 14 14:01:49 2022 UTC and is due to finish in 60 minutes. The chair is lajoskatona. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| opendevmeet | The meeting name has been set to 'neutron_drivers' | 14:01 |
| slaweq | thx frickler | 14:01 |
| mlavalle | o/ | 14:01 |
| slaweq | o/ | 14:01 |
| ralonsoh | hi | 14:01 |
| lajoskatona | o/ | 14:01 |
| amotoki | hi | 14:02 |
| obondarev | hi | 14:03 |
| lajoskatona | Ok so our topic for today is: Can neutron-fwaas project be revived? | 14:03 |
| lajoskatona | see the thread: #link http://lists.openstack.org/pipermail/openstack-discuss/2021-December/026413.html | 14:03 |
| lajoskatona | it was Inspur who would like to use and maintain fwaas | 14:03 |
| lajoskatona | I hope they can join | 14:04 |
| haleyb | hi | 14:05 |
| lajoskatona | Actually I am fine with opening fwaas again and have it as Neutron stadium | 14:05 |
| ralonsoh | do you know what functionality from fwaas was needed? | 14:05 |
| ralonsoh | just asking | 14:05 |
| lajoskatona | what I miss in context that what fwaas can offer that is not covered with security-groups? | 14:05 |
| ralonsoh | L3 FW | 14:06 |
| lajoskatona | ralonsoh: thanks | 14:06 |
| ralonsoh | on router ports, for example | 14:06 |
| mlavalle | +1 | 14:06 |
| slaweq | I'm personally not against reviving it but I would vote for doing it in the x/ namespace, at least for now | 14:06 |
| ralonsoh | right, ok with this | 14:07 |
| lajoskatona | slaweq: ok | 14:07 |
| slaweq | if problem will be really maintained, we can thing about moving it to the stadium some day, but not now | 14:07 |
| slaweq | that's my opinion about it | 14:07 |
| zhouhenglc | We have many customers who want to set simple rules through the security group. Layer 3 sets complex rules through the firewall | 14:07 |
| lajoskatona | zhouhenglc: hi, so you need L3 firewall ? | 14:08 |
| zhouhenglc | We have implemented firewall based on ovn | 14:08 |
| zhouhenglc | Logical_Router_Policy | 14:08 |
| zhouhenglc | lajoskatona: yes | 14:08 |
| slaweq | zhouhenglc: so You have Your own implementation of the fwaas, right? | 14:09 |
| slaweq | do You want to maintain only that new, ovn based implementation or the original one which was in fwaas too? | 14:09 |
| zhouhenglc | We are based on the original neutral-fwaas implementation | 14:09 |
| mlavalle | so you would manintain both, then, correct? | 14:10 |
| mlavalle | the entire thing I mean | 14:10 |
| zhouhenglc | yes both | 14:10 |
| lajoskatona | if we vote to have it now under x/ you have to start to renaming process | 14:11 |
| lajoskatona | I can help in that, I have to fetch my old memories | 14:12 |
| amotoki | lajoskatona: renaming does not happen. a new repo named as x/neutron-fwaas should be created. | 14:12 |
| lajoskatona | amotoki: yes that is true | 14:13 |
| amotoki | it can be created from openstack/neutron-fwaas, so the new x/ repo can have the whole history. | 14:13 |
| lajoskatona | yes it happened with networking-l2gw | 14:13 |
| opendevreview | jpic proposed openstack/neutron stable/xena: WIP: allow multiple segments per host https://review.opendev.org/c/openstack/neutron/+/824718 | 14:14 |
| lajoskatona | as I remember to x/ direction is simpler don't need to wait for gerrit restart, but that is not that iteresting now, sorry,.... | 14:14 |
| lajoskatona | zhouhenglc: what do you say, is that ok for you to have fwaas revived under x/ namespace, and we can later see to move it back to openstack/neutron-fwaas? | 14:15 |
| zhouhenglc | lajoskatona: I think it's good, too. | 14:16 |
| lajoskatona | zhouhenglc: ok, cool | 14:16 |
| lajoskatona | shall we vote on then to revive neutron-fwaas under x/ namespace? | 14:17 |
| mlavalle | yeap | 14:17 |
| lajoskatona | +1 from me | 14:17 |
| mlavalle | +1 from me | 14:17 |
| ralonsoh | +1 | 14:17 |
| amotoki | +1 | 14:17 |
| slaweq | +1 from me | 14:17 |
| haleyb | +1 | 14:17 |
| lajoskatona | ok, we have the result, thanks :-) | 14:18 |
| zhouhenglc | I only contributed to the neutron project before. What do I need to do about neutron-fwass now | 14:18 |
| ralonsoh | nothing else, same as for Neutron | 14:19 |
| lajoskatona | zhouhenglc: to start the moving/renaming process a patch is necessary and TC will dicsuss it on next TC meeting | 14:19 |
| lajoskatona | I have to check the name of the repo and the format | 14:20 |
| mlavalle | zhouhenglc: to be clear... in the ml thread I suggested you and your team start creating a document similar to https://review.opendev.org/c/openstack/neutron-specs/+/506012. If you are happy with the x/ namespace and plan to remain there, that document is not necessary. It would be necessary only if in the future you want to bring fwaas to the Neutron stadium | 14:20 |
| mlavalle | I don't want you to spend cycles on something that might not be necessary | 14:20 |
| lajoskatona | mlavalle: thanks | 14:21 |
| amotoki | lajoskatona: to be clear I think no TC discussion is required as x/ namespace is not controlled by TC. | 14:21 |
| lajoskatona | The repo is openstack/governance | 14:21 |
| lajoskatona | ok perhaps it is only necessary for the other direction... | 14:21 |
| amotoki | it is just required when opentack/neutron-fwaas repo will be retired completely including the stable branches. | 14:22 |
| lajoskatona | ok, I think we can close the "official" part of the meeting, and I can ask infra what we need to revive fwaas under x/ | 14:24 |
| amotoki | the next step is to prepare a new repo x/neutron-fwaas. | 14:24 |
| amotoki | we can follow https://docs.opendev.org/opendev/infra-manual/latest/creators.html | 14:24 |
| lajoskatona | amotoki: thanks, I was looking for this :-) | 14:25 |
| zhouhenglc | amotoki: "prepare a new repo x/neutron-fwaas" need me to do something? | 14:26 |
| amotoki | zhouhenglc: you can. we can support you. | 14:26 |
| amotoki | zhouhenglc: the above URL of the infra manual explains what you need to do. | 14:27 |
| zhouhenglc | amotoki: thanks, I learn the above | 14:28 |
| zhouhenglc | URL | 14:28 |
| lajoskatona | ok, I think we can close than the meeting, thanks for the participation | 14:29 |
| bpetermann | If I may, I'd like to bring up https://bugs.launchpad.net/neutron/+bug/1905391 (RFE VPNaaS for OVN) | 14:29 |
| bpetermann | the work there is actually finished, I'd appreciate a review. If there's anything missing, please let me know | 14:30 |
| lajoskatona | You mean this patch: https://review.opendev.org/c/openstack/neutron-vpnaas/+/765353 ? | 14:31 |
| bpetermann | yes | 14:31 |
| lajoskatona | thanks for highlighting it, | 14:31 |
| lajoskatona | ok, if there is no more things to discuss... | 14:32 |
| lajoskatona | #endmeeting | 14:33 |
| opendevmeet | Meeting ended Fri Jan 14 14:33:21 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:33 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-01-14-14.01.html | 14:33 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-01-14-14.01.txt | 14:33 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-01-14-14.01.log.html | 14:33 |
| lajoskatona | o/ | 14:33 |
| mlavalle | o/ | 14:33 |
| ralonsoh | bye | 14:33 |
| amotoki | o/ | 14:33 |
| slaweq | o/ | 14:34 |
| bcafarel | slaweq: o/ Friday cleanup can you recheck https://review.opendev.org/c/openstack/os-ken/+/823759 ? small and passing CI now | 14:48 |
| slaweq | bcafarel: done | 14:51 |
| bcafarel | thanks! | 14:53 |
| opendevreview | Merged openstack/os-ken stable/train: Fix sockaddr lib with python 3 https://review.opendev.org/c/openstack/os-ken/+/823759 | 15:03 |
| *** tbachman_ is now known as tbachman | 15:09 | |
| opendevreview | jpic proposed openstack/neutron master: Fix OVS OVSNeutronAgent.reclaim_local_vlan() https://review.opendev.org/c/openstack/neutron/+/824727 | 15:33 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Add 'port-device-profile' to the API extensions list https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/812945 | 15:43 |
| ralonsoh | ok I think functional gate CI is broken right now | 16:28 |
| ralonsoh | ok, not only the gate one but check too | 16:29 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Revert "Make configure_for_func_testing compatible with e.g. Centos" https://review.opendev.org/c/openstack/neutron/+/824750 | 16:38 |
| ralonsoh | lajoskatona, slaweq ^^ sorry for this revert (I don't like to push a revert) | 16:39 |
| lajoskatona | ralonsoh: thanks | 16:40 |
| lajoskatona | ralonsoh: yeah it seems to be red by last few results: https://zuul.openstack.org/builds?job_name=neutron-functional-with-uwsgi | 16:40 |
| ralonsoh | I'm filling a launchpad bug | 16:40 |
| ralonsoh | I'll amend the ID in the commit message | 16:40 |
| lajoskatona | https://4d3e42afa4b7ce910413-97904cf796f863517063072efe36b962.ssl.cf2.rackcdn.com/824727/1/check/neutron-functional-with-uwsgi/66d5147/testr_results.html | 16:40 |
| opendevreview | Merged openstack/neutron stable/victoria: Add wait event for metadataagent sb_idl https://review.opendev.org/c/openstack/neutron/+/823850 | 16:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Revert "Make configure_for_func_testing compatible with e.g. Centos" https://review.opendev.org/c/openstack/neutron/+/824750 | 16:42 |
| lajoskatona | ralonsoh: perhaps it is enough to use Q_BUILD_OVS_FROM_GIT here as well: https://opendev.org/openstack/neutron/src/branch/master/roles/configure_functional_tests/tasks/main.yaml#L18 | 16:43 |
| ralonsoh | Q_BUILD_OVS_FROM_GIT are internal variables, inside the scripts | 16:43 |
| ralonsoh | shouldn't be used as input parameters | 16:43 |
| ralonsoh | Q_* I mean | 16:44 |
| lajoskatona | ralonsoh: ack | 16:49 |
| gmann | slaweq: lajoskatona frickler not sure its same ping issue or not but it blocking nova and tempest gate (might be other too) | 16:55 |
| ralonsoh | gmann, do you have a link? | 16:56 |
| ralonsoh | just to check | 16:56 |
| gmann | I am going to make jobs n-v to unblock gate until we figure out the issue | 16:56 |
| gmann | ralonsoh: https://zuul.opendev.org/t/openstack/build/e0db6a9a7ba04e66b0781ba7d259357d/logs | 16:56 |
| ralonsoh | gmann, no, this is not the same problem | 16:56 |
| gmann | ohk | 16:56 |
| gmann | let me open bug and n-v the job then | 16:57 |
| jpic | hi all! I have a working but dirty/brute-forced patch here to support multiple segments per host: https://review.opendev.org/c/openstack/neutron/+/824718 as you can see, it required a major change in the dnsmasq code so that it uses one interface per subnet instead, would you like to see that in a separate RFE or can it stay with the "support multiple segments per host" RFE | 16:58 |
| opendevreview | Balazs Gibizer proposed openstack/neutron master: DNM: trigger tempest https://review.opendev.org/c/openstack/neutron/+/824605 | 16:58 |
| gmann | slaweq: lajoskatona ralonsoh frickler filed the bug https://bugs.launchpad.net/neutron/+bug/1957941 | 17:11 |
| ralonsoh | thanks | 17:11 |
| lajoskatona | jpic: you mean you changed dnsmasq code or just Neutron which I see in https://review.opendev.org/c/openstack/neutron/+/824718 ? | 17:16 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Update PortTestCasesResourceRequest for new Neutron API ext https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/817357 | 17:17 |
| lajoskatona | gmann: yeah it seems a different failure, but isn't it possible that there's some common centos issue? | 17:17 |
| gmann | may be | 17:18 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: When creating a VXLAN interface, a device is mandatory https://review.opendev.org/c/openstack/neutron/+/824752 | 17:20 |
| evrardjp | hello folks | 17:29 |
| lajoskatona | ralonsoh: Hi, regarding https://review.opendev.org/c/openstack/neutron/+/824750 , no functional or fullstack test is running for tools/configure_for_func_testing.sh changes | 18:04 |
| lajoskatona | ralonsoh: if you have no time, I can change irrelevant-files: for the jobs | 18:05 |
| opendevreview | Merged openstack/ovsdbapp master: Use "api.lookup" to search for a "Logical_Switch" https://review.opendev.org/c/openstack/ovsdbapp/+/822909 | 18:08 |
| *** tbachman_ is now known as tbachman | 18:43 | |
| opendevreview | Lajos Katona proposed openstack/neutron master: [Functional/fullstack] Remove tools from irrelevant list https://review.opendev.org/c/openstack/neutron/+/824773 | 18:47 |
| *** tbachman_ is now known as tbachman | 19:50 | |
| *** tbachman_ is now known as tbachman | 20:32 | |
| *** tbachman_ is now known as tbachman | 21:22 | |
| *** dasm is now known as dasm|off | 21:46 | |
| *** tbachman_ is now known as tbachman | 22:24 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!