| opendevreview | liuyulong proposed openstack/neutron master: [Doc] packet rete limit user workflow and notes https://review.opendev.org/c/openstack/neutron/+/828673 | 02:55 |
|---|---|---|
| opendevreview | liuyulong proposed openstack/neutron master: Add policy for packet rate limit rules https://review.opendev.org/c/openstack/neutron/+/829161 | 02:55 |
| opendevreview | liuyulong proposed openstack/neutron master: Meter flows and ovsdb action for ovs bridge https://review.opendev.org/c/openstack/neutron/+/816800 | 02:55 |
| opendevreview | liuyulong proposed openstack/neutron master: Support pps limitation for openvswitch agent https://review.opendev.org/c/openstack/neutron/+/816802 | 02:55 |
| opendevreview | liuyulong proposed openstack/neutron master: Fullstack tests of packet rate limit for ovs qos driver https://review.opendev.org/c/openstack/neutron/+/819418 | 02:55 |
| opendevreview | ZhouHeng proposed openstack/neutron-fwaas master: Revert "Retire neutron-fwaas project" https://review.opendev.org/c/openstack/neutron-fwaas/+/828149 | 05:39 |
| *** ykarel_ is now known as ykarel | 06:23 | |
| zhouhenglc | lajoskatona, slaweq https://review.opendev.org/c/openstack/neutron-fwaas/+/828149 functional test and tempest test have no problem, is there anything to be changed? | 06:27 |
| ykarel | ralonsoh, hi | 08:10 |
| ykarel | can you revisit https://review.opendev.org/c/openstack/neutron/+/830938 | 08:10 |
| ykarel | i tried to answer there | 08:10 |
| ykarel | thx in advance | 08:11 |
| ralonsoh | ok | 08:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use the "connectivity" property of "MechanismDriver" https://review.opendev.org/c/openstack/neutron/+/826502 | 08:29 |
| ykarel | ralonsoh, thx for the response, seems i am not able to explain enough the benefits, all the patches link i shared are related in terms of outcome efficiency | 08:48 |
| ykarel | in terms of both ci time and person time | 08:48 |
| ralonsoh | how? | 08:48 |
| ykarel | so if you imagine ci jobs running with and without that patch | 08:49 |
| ykarel | there are unnecessary jobs running without this patch | 08:49 |
| ykarel | and if u have failures there for the unnecessary jobs, u have to check logs, recheck | 08:50 |
| ralonsoh | how are you improving that with https://review.opendev.org/c/openstack/neutron/+/830938? | 08:51 |
| ykarel | ralonsoh, https://review.opendev.org/c/openstack/neutron/+/827302 for example | 08:52 |
| ykarel | ran with and without that patch in ps 1 and 2 | 08:52 |
| ralonsoh | this is another patch | 08:52 |
| ralonsoh | that can be implemented without https://review.opendev.org/c/openstack/neutron/+/830938 | 08:52 |
| ykarel | ralonsoh, yes i am ok if that can be done without ^ + without leaving coverage? | 08:53 |
| ralonsoh | ? | 08:53 |
| ralonsoh | leaving coverage? | 08:53 |
| ykarel | yes i want to run the jobs to run when we update projects.yaml | 08:53 |
| ykarel | periodic, experimental and tox jobs are special case so that's why i switched those | 08:54 |
| ralonsoh | not tox | 08:54 |
| ykarel | periodic/experimental need different triggers then check/gate | 08:54 |
| ykarel | and tox one for irrelevant-files overide | 08:55 |
| ralonsoh | do you have an example of this? | 08:55 |
| ralonsoh | where is this going to be needed | 08:55 |
| ykarel | sorry needed what? | 08:56 |
| ralonsoh | where will you need those templates | 08:56 |
| ralonsoh | an example | 08:56 |
| ralonsoh | a patch needing those templates | 08:56 |
| ykarel | in neutron project.yaml | 08:56 |
| ralonsoh | no | 08:56 |
| ralonsoh | where will you need those templates | 08:56 |
| ralonsoh | moving the defintions to another file is not the reason | 08:57 |
| ykarel | ralonsoh, you seen https://review.opendev.org/c/openstack/neutron/+/827302 ? | 08:58 |
| ykarel | jobs ran in PS1 and PS2? | 08:58 |
| ralonsoh | again no | 08:58 |
| ralonsoh | you can implement this patch directly on job templates | 08:58 |
| ralonsoh | sorry | 08:58 |
| ralonsoh | zuul.d/projects | 08:58 |
| ykarel | ralonsoh, but it will trigger unnecessary jobs that i want to avoid | 08:59 |
| ralonsoh | what jobs? when? how? | 08:59 |
| ykarel | https://review.opendev.org/c/openstack/neutron/+/827302 PS1 and PS2 have example for that, similar in other linked patches | 09:00 |
| ralonsoh | so you are saying that, only when you need to change the job definitions and because you don't want to trigger other tests | 09:01 |
| ralonsoh | just on those kind of patches | 09:01 |
| ralonsoh | you are relocating those lists? | 09:02 |
| ralonsoh | again | 09:02 |
| ralonsoh | I prefer to have in one file | 09:02 |
| ykarel | for periodic/experimental yes ^ right as they have special triggers | 09:02 |
| ralonsoh | again no | 09:02 |
| ralonsoh | when experimental or periodic jobs are executed, you have your job lists | 09:02 |
| ralonsoh | just because in a couple of patches you don't want to run some tests | 09:03 |
| ralonsoh | I'm won't +1 this patch | 09:03 |
| ralonsoh | again | 09:03 |
| ralonsoh | I want to have the job definition list in one file | 09:03 |
| ralonsoh | that is, for now, short enough | 09:03 |
| ykarel | periodic runs as per scheduled-cron, while experimental need check experimental | 09:03 |
| ralonsoh | sorry, I need to continue working | 09:03 |
| ralonsoh | I won't +1 this patch | 09:04 |
| ykarel | ralonsoh, sorry for taking too much of your time, but trust me it will have benefits at scale | 09:04 |
| ralonsoh | I don't trust | 09:04 |
| ykarel | will look for others perspective too | 09:04 |
| ralonsoh | I didn't see any improvement there | 09:04 |
| ralonsoh | so I don't need to trust | 09:04 |
| ralonsoh | I need to see that | 09:04 |
| ykarel | ralonsoh, so you seen job runs in PS1 and PS2? | 09:05 |
| ykarel | last question | 09:05 |
| ykarel | and u didn't see the improvement in terms of ci time and person time there? | 09:05 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/ussuri: Fix race condition retrieving logical router rows https://review.opendev.org/c/openstack/ovn-octavia-provider/+/827077 | 10:09 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Clean duplicated QoS bandwidth related methods in ovs_lib module https://review.opendev.org/c/openstack/neutron/+/827112 | 10:10 |
| opendevreview | Merged openstack/neutron master: Add new class for Logging API methods https://review.opendev.org/c/openstack/neutron/+/829522 | 10:25 |
| opendevreview | Marios Andreou proposed openstack/neutron stable/wallaby: Add centos-9 jobs for wallaby zuul layout https://review.opendev.org/c/openstack/neutron/+/831192 | 10:37 |
| opendevreview | Marios Andreou proposed openstack/neutron stable/wallaby: Add centos-9 jobs for wallaby zuul layout https://review.opendev.org/c/openstack/neutron/+/831192 | 10:39 |
| ykarel | ralonsoh, https://review.opendev.org/c/openstack/neutron/+/830563 failed CI :( | 10:53 |
| ykarel | devstack one merged, so functional jobs will remain impacted until ^ merges | 10:54 |
| ralonsoh | yes | 10:54 |
| ralonsoh | what is the problem? | 10:54 |
| ykarel | just we need to get that merged, until that master functional job should be broken | 10:56 |
| ralonsoh | that's something know, this is why I pushed the patch in Neutron depending on devstack one | 10:56 |
| ralonsoh | that will be just a matter of hours | 10:56 |
| ykarel | yes should be ok, ^ just for visibility | 10:57 |
| opendevreview | ZhouHeng proposed openstack/neutron master: [ovn]Refusing to bind port to dead agent https://review.opendev.org/c/openstack/neutron/+/825428 | 11:00 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/ussuri: Fix race condition retrieving logical router rows https://review.opendev.org/c/openstack/ovn-octavia-provider/+/827077 | 11:03 |
| opendevreview | yatin proposed openstack/neutron master: Remove _standard_attr_segment_lib and use definition from neutron-lib https://review.opendev.org/c/openstack/neutron/+/794957 | 12:44 |
| opendevreview | Marios Andreou proposed openstack/neutron stable/wallaby: Add centos-9 jobs for wallaby zuul layout https://review.opendev.org/c/openstack/neutron/+/831192 | 13:33 |
| *** dasm|off is now known as dasm|rover | 13:33 | |
| opendevreview | Marios Andreou proposed openstack/neutron stable/wallaby: Add centos-9 jobs for wallaby zuul layout https://review.opendev.org/c/openstack/neutron/+/831192 | 13:35 |
| opendevreview | Arnau Verdaguer proposed openstack/neutron stable/ussuri: [OVN] Handle RouterNotFound exception in set_gateway_mtu https://review.opendev.org/c/openstack/neutron/+/830867 | 13:53 |
| opendevreview | Arnau Verdaguer proposed openstack/neutron stable/victoria: [OVN] Handle RouterNotFound exception in set_gateway_mtu https://review.opendev.org/c/openstack/neutron/+/830866 | 14:06 |
| opendevreview | yatin proposed openstack/neutron master: Ensure gateway is set for prefix delegated subnets https://review.opendev.org/c/openstack/neutron/+/831214 | 14:18 |
| opendevreview | Merged openstack/neutron master: Change OVS and OVN installation directories https://review.opendev.org/c/openstack/neutron/+/830563 | 14:26 |
| opendevreview | yatin proposed openstack/neutron master: Ensure gateway is set for prefix delegated subnets https://review.opendev.org/c/openstack/neutron/+/831214 | 14:30 |
| opendevreview | Merged openstack/networking-ovn stable/train: Allow to create ovn loadbalancer on dual-stack provider networks https://review.opendev.org/c/openstack/networking-ovn/+/830152 | 15:00 |
| opendevreview | Arnau Verdaguer proposed openstack/networking-ovn stable/train: [OVN] Handle RouterNotFound exception in set_gateway_mtu https://review.opendev.org/c/openstack/networking-ovn/+/831218 | 15:07 |
| opendevreview | Merged openstack/neutron master: Add port IDs in "RouterInUse" exception during router deletion https://review.opendev.org/c/openstack/neutron/+/830813 | 15:09 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check ovn ovs experimental https://review.opendev.org/c/openstack/neutron/+/831220 | 15:10 |
| opendevreview | Merged openstack/neutron master: Use the "connectivity" property of "MechanismDriver" https://review.opendev.org/c/openstack/neutron/+/826502 | 15:12 |
| opendevreview | Eduardo Olivares proposed openstack/neutron-tempest-plugin master: Adapt StatefulConnection class to advance image https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/829104 | 15:16 |
| opendevreview | Arnau Verdaguer proposed openstack/networking-ovn stable/train: [OVN] Handle RouterNotFound exception in set_gateway_mtu https://review.opendev.org/c/openstack/networking-ovn/+/831218 | 15:41 |
| opendevreview | Pedro Henrique Pereira Martins proposed openstack/neutron master: Extend database to support portforwardings with port range https://review.opendev.org/c/openstack/neutron/+/798961 | 16:12 |
| opendevreview | Merged openstack/neutron-lib master: Rehome exception ``L3ExtensionException`` https://review.opendev.org/c/openstack/neutron-lib/+/831002 | 17:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP][DNM] [L3][QoS] L3 agent QoS extension to handle duplicated FIPs https://review.opendev.org/c/openstack/neutron/+/831238 | 18:00 |
| opendevreview | Eduardo Olivares proposed openstack/neutron-tempest-plugin master: Adapt StatefulConnection class to advance image https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/829104 | 18:10 |
| opendevreview | Merged openstack/neutron stable/wallaby: Remove tox_install_siblings=False from the functional job's definition https://review.opendev.org/c/openstack/neutron/+/830718 | 18:15 |
| opendevreview | Merged openstack/neutron stable/wallaby: Add functional and fullstack jobs with FIPS enabled https://review.opendev.org/c/openstack/neutron/+/829527 | 18:18 |
| opendevreview | Merged openstack/neutron stable/queens: Wait longer before deleting DPDK vhu trunk bridges https://review.opendev.org/c/openstack/neutron/+/829046 | 19:13 |
| opendevreview | Merged openstack/neutron stable/wallaby: [OVN] Handle RouterNotFound exception in set_gateway_mtu https://review.opendev.org/c/openstack/neutron/+/830865 | 19:32 |
| opendevreview | Merged openstack/neutron stable/xena: [OVN] Handle RouterNotFound exception in set_gateway_mtu https://review.opendev.org/c/openstack/neutron/+/830864 | 19:32 |
| *** dkehn_ is now known as dkehn | 20:36 | |
| opendevreview | Candido Campos Rivas proposed openstack/neutron-tempest-plugin master: Recover ssh interface config and interface reload https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/831321 | 23:36 |
| opendevreview | Candido Campos Rivas proposed openstack/neutron-tempest-plugin master: Recover ssh interface config and interface reload https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/831321 | 23:41 |
| opendevreview | Krzysztof Tomaszewski proposed openstack/neutron master: Don't set HA ports down while L2 agent restart. https://review.opendev.org/c/openstack/neutron/+/826545 | 23:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!