| opendevreview | Ghanshyam proposed openstack/neutron-tempest-plugin master: Run neutron-tempest-plugin sceanrio slow tests serially https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/807800 | 00:01 |
|---|---|---|
| opendevreview | Ghanshyam proposed openstack/neutron-tempest-plugin master: Run neutron-tempest-plugin sceanrio slow tests serially https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/807800 | 00:03 |
| opendevreview | Ghanshyam proposed openstack/neutron-tempest-plugin master: Run neutron-tempest-plugin sceanrio slow tests serially https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/807800 | 00:10 |
| opendevreview | Ghanshyam proposed openstack/neutron-tempest-plugin master: Run neutron-tempest-plugin scenario slow tests serially https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/807800 | 00:15 |
| opendevreview | Ghanshyam proposed openstack/neutron-tempest-plugin master: Run neutron-tempest-plugin scenario slow tests serially https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/807800 | 00:22 |
| opendevreview | Ghanshyam proposed openstack/neutron-tempest-plugin master: Run neutron-tempest-plugin scenario slow tests serially https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/807800 | 00:26 |
| opendevreview | Ghanshyam proposed openstack/neutron-tempest-plugin master: Run neutron-tempest-plugin scenario slow tests serially https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/807800 | 00:28 |
| opendevreview | Ghanshyam proposed openstack/neutron-tempest-plugin master: Run neutron-tempest-plugin scenario slow tests serially https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/807800 | 01:08 |
| opendevreview | Merged openstack/neutron stable/stein: Don't use singleton in routes.middleware.RoutesMiddleware https://review.opendev.org/c/openstack/neutron/+/807636 | 01:11 |
| opendevreview | Kevin Li proposed openstack/neutron master: update subport status when trunk/subport create/delete is triggerred https://review.opendev.org/c/openstack/neutron/+/807076 | 01:38 |
| opendevreview | Ghanshyam proposed openstack/neutron-tempest-plugin master: DNM: testing run-tempest new var https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/807809 | 02:01 |
| opendevreview | Manu B proposed openstack/neutron-dynamic-routing master: Push router,peer events to BGP l3 agent extension https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/802445 | 05:08 |
| opendevreview | Manu B proposed openstack/neutron-dynamic-routing master: Support peer association to BGP speaker https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/800882 | 05:08 |
| opendevreview | Manu B proposed openstack/neutron-dynamic-routing master: Push router,peer events to BGP l3 agent extension https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/802445 | 05:08 |
| opendevreview | Arkady Shtempler proposed openstack/neutron-tempest-plugin master: Refactoring - Use existing Tempest APIs in "QoS bandwidth limit rule" tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/806977 | 05:45 |
| opendevreview | Manu B proposed openstack/neutron-dynamic-routing master: Support router association to BGP speaker https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/792338 | 06:34 |
| opendevreview | Manu B proposed openstack/neutron-dynamic-routing master: Support peer association to BGP speaker https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/800882 | 06:34 |
| opendevreview | Manu B proposed openstack/neutron-dynamic-routing master: Push router,peer events to BGP l3 agent extension https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/802445 | 06:34 |
| opendevreview | Manu B proposed openstack/neutron-dynamic-routing master: Push router,peer events to BGP l3 agent extension https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/802445 | 06:36 |
| *** slaweq_ is now known as slaweq | 07:06 | |
| *** pjakuszew7 is now known as pjakuszew | 07:09 | |
| opendevreview | Bernard Cafarelli proposed openstack/neutron-vpnaas stable/train: Pin isort to 4.3.21 https://review.opendev.org/c/openstack/neutron-vpnaas/+/805969 | 07:40 |
| bcafarel | ralonsoh: morning a small review if I can add your list https://review.opendev.org/c/openstack/networking-sfc/+/796788 | 07:51 |
| ralonsoh | sure | 07:51 |
| *** elodilles_pto is now known as elodilles | 08:44 | |
| opendevreview | Manu B proposed openstack/neutron-dynamic-routing master: Push router,peer events to BGP l3 agent extension https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/802445 | 08:53 |
| opendevreview | Arkady Shtempler proposed openstack/neutron-tempest-plugin master: Refactoring - Use existing Tempest APIs in "QoS bandwidth limit rule" tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/806977 | 08:54 |
| opendevreview | Elod Illes proposed openstack/os-vif stable/train: [stable-only] Fix lower-constraints https://review.opendev.org/c/openstack/os-vif/+/807839 | 08:55 |
| opendevreview | Oleg Bondarev proposed openstack/neutron master: Add Local IP Extension and DB https://review.opendev.org/c/openstack/neutron/+/804523 | 09:25 |
| opendevreview | Oleg Bondarev proposed openstack/neutron master: Local IP RPC server-agent interface https://review.opendev.org/c/openstack/neutron/+/807116 | 09:25 |
| opendevreview | Merged openstack/networking-sfc master: Enable tls-proxy support for test jobs https://review.opendev.org/c/openstack/networking-sfc/+/796788 | 09:30 |
| opendevreview | Oleg Bondarev proposed openstack/neutron master: Add Local IP Extension and DB https://review.opendev.org/c/openstack/neutron/+/804523 | 09:38 |
| opendevreview | Oleg Bondarev proposed openstack/neutron master: Local IP RPC server-agent interface https://review.opendev.org/c/openstack/neutron/+/807116 | 09:38 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Deprecate 'allow_overlapping_ips' config option https://review.opendev.org/c/openstack/neutron/+/807848 | 09:46 |
| opendevreview | Bernard Cafarelli proposed openstack/neutron-vpnaas stable/train: Pin isort to 4.3.21 https://review.opendev.org/c/openstack/neutron-vpnaas/+/805969 | 10:18 |
| opendevreview | Manu B proposed openstack/neutron-dynamic-routing master: Push router,peer events to BGP l3 agent extension https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/802445 | 10:21 |
| opendevreview | Manu B proposed openstack/neutron-dynamic-routing master: Support router association to BGP speaker https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/792338 | 11:10 |
| mnasiadka | good afternoon | 11:10 |
| mnasiadka | obondarev: do you plan to backport https://review.opendev.org/c/openstack/neutron/+/807243 all the way to stable/train? | 11:10 |
| opendevreview | Manu B proposed openstack/neutron-dynamic-routing master: Support peer association to BGP speaker https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/800882 | 11:11 |
| opendevreview | Manu B proposed openstack/neutron-dynamic-routing master: Push router,peer events to BGP l3 agent extension https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/802445 | 11:11 |
| obondarev | mnasiadka: hi, yes, will do | 11:11 |
| mnasiadka | obondarev: networking-generic-switch is kind of broken :-) | 11:11 |
| obondarev | mnasiadka, my bad, sorry | 11:12 |
| mnasiadka | obondarev: well, if the networking-generic-switch job would be included in Neutron's CI - maybe it would be easier to spot the error :) | 11:12 |
| obondarev | mnasiadka, is networking-generic-switch tested against neutron master regularly? | 11:15 |
| mnasiadka | there's a tempest job running there | 11:16 |
| mnasiadka | https://zuul.openstack.org/job/networking-generic-switch-tempest-dlm | 11:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Update the DHCP options when the metadata port is modified https://review.opendev.org/c/openstack/neutron/+/807692 | 11:23 |
| opendevreview | Manu B proposed openstack/neutron-dynamic-routing master: Push router,peer events to BGP l3 agent extension https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/802445 | 11:25 |
| ralonsoh | bcafarel, https://review.opendev.org/c/openstack/neutron-vpnaas/+/805969/9/.zuul.yaml#32 | 11:29 |
| ralonsoh | I think you need to keep the \. | 11:29 |
| ralonsoh | tempest_test_regex: ^neutron_vpnaas\. | 11:29 |
| ralonsoh | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_4e0/805969/9/check/neutron-vpnaas-tempest/4e0a88d/job-output.txt | 11:29 |
| fungi | slaweq: ralonsoh: lajoskatona: quick feedback on the proposed impact description on public vulnerability report https://launchpad.net/bugs/1942179 would be appreciated so i can move forward with a cve assignment request | 11:32 |
| ralonsoh | fungi, did you send a patch? | 11:33 |
| fungi | ralonsoh: a comment in the bug contains a proposed impact description, what we use to provide information to mitre when requesting a cve assignment, and what will eventually become the text of the advisory we publish | 11:34 |
| ralonsoh | fungi, ok perfect | 11:35 |
| opendevreview | Oleg Bondarev proposed openstack/neutron stable/wallaby: Ensure net dict has provider info on precommit delete https://review.opendev.org/c/openstack/neutron/+/807763 | 11:35 |
| fungi | ralonsoh: comment #24, it's a bit lost in all the backport merge notifications so easy to miss | 11:36 |
| opendevreview | Oleg Bondarev proposed openstack/neutron stable/wallaby: Ensure net dict has provider info on precommit delete https://review.opendev.org/c/openstack/neutron/+/807763 | 11:36 |
| opendevreview | Oleg Bondarev proposed openstack/neutron stable/victoria: Ensure net dict has provider info on precommit delete https://review.opendev.org/c/openstack/neutron/+/807764 | 11:37 |
| ralonsoh | fungi, I think this description is accurate. We can also say this was happening in combination with evenlet (that is used by default) | 11:37 |
| fungi | ralonsoh: are there non-eventlet deployments? in that case I can say something like "neutron deployments using eventlet are affected" instead | 11:38 |
| opendevreview | Oleg Bondarev proposed openstack/neutron stable/ussuri: Ensure net dict has provider info on precommit delete https://review.opendev.org/c/openstack/neutron/+/807765 | 11:39 |
| ralonsoh | fungi, no, that's the point, Neutron is programmed for evenlet, without it does not work | 11:39 |
| opendevreview | Oleg Bondarev proposed openstack/neutron stable/train: Ensure net dict has provider info on precommit delete https://review.opendev.org/c/openstack/neutron/+/807766 | 11:39 |
| ralonsoh | so maybe is not relevant | 11:39 |
| fungi | ralonsoh: in that case yeah, keeping it short is best. the idea is to convey enough detail to differentiate this from any potential future api worker memory leaks, with sufficient detail for operators to know what deployments need to be patched, but the less extraneous info we include the better (they can always read the bug or patches if they want to dig deeper) | 11:41 |
| ralonsoh | agree | 11:41 |
| obondarev | mnasiadka, backports created, thanks for raising! Regarding CI: adding all 3rd party CIs to neutron does not scale well ;) regular testing against neutron master should do the job and help identify regressions early :) | 11:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Metadata ports device_owner is "network:distributed" only https://review.opendev.org/c/openstack/neutron/+/807707 | 11:44 |
| mnasiadka | obondarev: thanks :) | 11:44 |
| slaweq | fungi: ralonsoh: sorry, I was on lunch - for me this description proposed by You in comment sounds good and accurate | 11:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][FT] Check UNIX socket file before using it https://review.opendev.org/c/openstack/neutron/+/807862 | 11:49 |
| bcafarel | ralonsoh: ack thanks, that was kind of blind fix attempt - it seems not to find the tempest tests (though install went fine) | 11:49 |
| opendevreview | Bernard Cafarelli proposed openstack/neutron-vpnaas stable/train: Pin isort to 4.3.21 https://review.opendev.org/c/openstack/neutron-vpnaas/+/805969 | 11:49 |
| ralonsoh | bcafarel, let's see this one | 11:49 |
| fungi | thanks ralonsoh and slaweq! i'll get the cve assignment request submitted shortly in that case | 11:49 |
| bcafarel | ralonsoh: crossing fingers :) | 11:50 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron-lib master: (wip) Remove registry notify https://review.opendev.org/c/openstack/neutron-lib/+/807869 | 13:04 |
| opendevreview | Lajos Katona proposed openstack/neutron-tempest-plugin master: WIP: Add tap-as-a-service API tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/803752 | 13:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Set "cache_ok=True" in all "TypeDecorator" derived classes https://review.opendev.org/c/openstack/neutron-lib/+/807360 | 13:29 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Add API shim extension "quota-check-limit" https://review.opendev.org/c/openstack/neutron-lib/+/807876 | 13:43 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron-lib master: Remove depricated method registry.notify https://review.opendev.org/c/openstack/neutron-lib/+/807869 | 13:55 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: [Docs] Add info about how to use shared SG with VMs https://review.opendev.org/c/openstack/neutron/+/807878 | 13:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Check quota limits https://review.opendev.org/c/openstack/neutron/+/801470 | 14:18 |
| bcafarel | ralonsoh: https://zuul.opendev.org/t/openstack/build/ca5da25baf9e47239b8fa92bc87486d1 similar failure :( strange as I see earlier in log "local plugins=,neutron-tempest-plugin,neutron-vpnaas" so neutron_vpnaas should match something? | 14:21 |
| ralonsoh | bcafarel, I really don't understand this.. let me check what we are installing here | 14:23 |
| bcafarel | ralonsoh: thanks, nothing urgent I guess it just needs some zuul voodoo change | 14:24 |
| opendevreview | Oleg Bondarev proposed openstack/neutron stable/wallaby: Ensure net dict has provider info on precommit delete https://review.opendev.org/c/openstack/neutron/+/807763 | 14:25 |
| opendevreview | Oleg Bondarev proposed openstack/neutron stable/victoria: Ensure net dict has provider info on precommit delete https://review.opendev.org/c/openstack/neutron/+/807764 | 14:26 |
| opendevreview | Oleg Bondarev proposed openstack/neutron stable/ussuri: Ensure net dict has provider info on precommit delete https://review.opendev.org/c/openstack/neutron/+/807765 | 14:26 |
| opendevreview | Oleg Bondarev proposed openstack/neutron stable/train: Ensure net dict has provider info on precommit delete https://review.opendev.org/c/openstack/neutron/+/807766 | 14:26 |
| ralonsoh | bcafarel, my question there is what tempest test do we want to execute? | 14:44 |
| ralonsoh | because there are no neutron_vpnaas tempest tests | 14:44 |
| ralonsoh | shouldn't this be tempest_test_regex: ^neutron_tempest_plugin\.vpnaas | 14:46 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-vpnaas stable/train: Pin isort to 4.3.21 https://review.opendev.org/c/openstack/neutron-vpnaas/+/805969 | 14:47 |
| bcafarel | good question, checking the branches | 15:01 |
| opendevreview | Arkady Shtempler proposed openstack/neutron-tempest-plugin master: Refactoring - Use existing Tempest APIs in "QoS bandwidth limit rule" tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/806977 | 15:05 |
| bcafarel | ralonsoh: train still had neutron_vpnaas/tests/tempest/ though the tests in neutron-tempest-plugin look identical (not many updates) - that could work :) | 15:11 |
| ralonsoh | bcafarel, https://zuul.opendev.org/t/openstack/stream/1ab93ec1dd1340b1ad4ff29312c4b6cd?logfile=console.log | 15:12 |
| bcafarel | moment of truth is not far away | 15:13 |
| ralonsoh | bcafarel, pfff | 15:15 |
| bcafarel | hmm it does not seem to find many tests at all :( | 15:15 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Update the DHCP options when the metadata port is modified https://review.opendev.org/c/openstack/neutron/+/807692 | 15:38 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Add API shim extension "quota-check-limit" https://review.opendev.org/c/openstack/neutron-lib/+/807876 | 15:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Replace "tenant_id" with "project_id" in Quota engine https://review.opendev.org/c/openstack/neutron/+/805849 | 15:44 |
| ade_lee_ | bcafarel, hey -- on the fips job, I don't see an openvswitch package in brew for rhel 8/centos 8 -- I do see builds for rhel 7 -- so what's the equivalent in rhel 8? | 16:01 |
| ade_lee_ | actually I guess if I scroll through enough pages then I do .. | 16:02 |
| bcafarel | ade_lee_: I hope we have (or will get soon at least) ovs there :) | 16:03 |
| ade_lee_ | bcafarel, so yeah -- I'm trying to understand what the fix needed here is -- are we expecting to see an openvswitch package in centos-8-stream repos? | 16:07 |
| bcafarel | ade_lee_: not tested but it sounds like we need to add centos-release-nfv-openvswitch repo maybe somewhere around https://github.com/openstack/devstack/blob/ebc7b0ed255bc879078ec9cf54f9d6f179cec72c/stack.sh#L290 (or in neutron specific bits) | 16:14 |
| bcafarel | ade_lee_: oh and probably need to specify the openvswitch version too :/ https://wiki.centos.org/SpecialInterestGroup/NFV | 16:15 |
| ade_lee_ | bcafarel, ok - that makes sense -- we can look at where openvswitch id currently being installed -- thats probably in neutron specific bits | 16:19 |
| ade_lee_ | which version do we need? | 16:19 |
| bcafarel | the newest in repo, we just use master in devstack | 16:28 |
| bcafarel | hmm | 16:28 |
| bcafarel | ade_lee_: https://review.opendev.org/c/openstack/devstack/+/799621 may be another way too (we can build ovs/ovn from source) | 16:28 |
| ade_lee_ | bcafarel, it seems slaweq was already thinking about this to solve the fips job problem | 16:30 |
| ade_lee_ | slaweq, if that patch is ready enough, we can add it as a dependency on the fips job to see if it will allow those jobs to run | 16:32 |
| ade_lee_ | bcafarel, ^^ | 16:33 |
| bcafarel | sounds like a good idea (at least to get some results) - and set Q_BUILD_OVS_FROM_GIT to use that in the fips job | 16:36 |
| bcafarel | I will defer to slaweq he will know better :) | 16:37 |
| ade_lee_ | bcafarel, is that set in the devstack_localrc? | 16:38 |
| ade_lee_ | or maybe just as a var actually .. | 16:39 |
| ade_lee_ | I see the same for OVN_BRANCH etc . | 16:40 |
| bcafarel | devstack_localrc should work, though I do not see many jobs using it (one grenade example https://github.com/openstack/neutron/blob/804cc660e7de1d88ee0ac3d1aab96384442e77c4/zuul.d/grenade.yaml#L196 ) | 16:42 |
| opendevreview | Ade Lee proposed openstack/neutron master: WIP/DNM - Add FIPS enabled jobs https://review.opendev.org/c/openstack/neutron/+/797537 | 16:47 |
| ade_lee_ | bcafarel, slaweq ^^ lets see what happens! | 16:47 |
| bcafarel | :) | 17:05 |
| ade_lee_ | bcafarel, slaweq interesting -- results are back -- seems like something else broke .. | 18:09 |
| ade_lee_ | although ovs seemed to pass this time .. | 18:10 |
| *** elodilles is now known as elodilles_pto | 20:12 | |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org is going offline momentarily for a host migration and zuul upgrade, downtime should be only a few minutes. | 21:06 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!