| opendevreview | Youngjun Yoo proposed openstack/neutron master: refectoring: refectoring docstring https://review.opendev.org/c/openstack/neutron/+/913486 | 02:48 |
|---|---|---|
| *** liuxie is now known as liushy | 07:24 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Open the 2024.2 (Dalmatian) DB branch https://review.opendev.org/c/openstack/neutron/+/912761 | 08:20 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add "socket" NUMA affinity policy https://review.opendev.org/c/openstack/neutron/+/910594 | 08:20 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Create an OVN DB transaction context decorator https://review.opendev.org/c/openstack/neutron/+/910538 | 08:21 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] "Logical_Router" pinned to chassis, OVN L3 scheduler https://review.opendev.org/c/openstack/neutron/+/909194 | 08:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Provide HA functionality to "Logical_Router" chassis pinning https://review.opendev.org/c/openstack/neutron/+/909437 | 08:24 |
| seba | How is the etiquette on resolving comments in gerrit? As a committer, do I mark the comments as resolved once I think it is resolved or do I leave them open for the reviewer to decide this? I'm asking for https://review.opendev.org/c/openstack/neutron/+/912629 but was asking myself this in the past on other reviews. | 11:05 |
| frickler | seba: I don't think there is much consensus on this. it is also a relatively new feature in gerrit, so a lot of reviewers still tend to ignore this completely. IMO it is perfectly fine to mark comments as resolved yourself, people can always respond and re-open if they disagree | 12:09 |
| seba | frickler, so in some cases I feel comfortable that I solved the problem, but in some cases I felt happy with my solution but wanted to leave the reviewer the option to decide for themselves. But as long as this will not stop anyone from reviewing I'll leave it as is for now. :) | 14:09 |
| opendevreview | Seena Fallah proposed openstack/ovsdbapp master: ovn-sb: remove chassis from Chassis_Private on ChassisDelCommand https://review.opendev.org/c/openstack/ovsdbapp/+/817300 | 14:24 |
| opendevreview | liuyulong proposed openstack/neutron master: Add a default goto table=94 for openvswitch fw https://review.opendev.org/c/openstack/neutron/+/907382 | 14:39 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: [OVN] The L3 scheduler does not use all chassis by default https://review.opendev.org/c/openstack/neutron/+/913574 | 14:42 |
| ralonsoh | ykarel_, ^ | 14:43 |
| ykarel_ | ralonsoh, thx | 14:43 |
| haleyb | ralonsoh: do you want to get this n-t-p change in? https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/909860 as i just got the release change to approve | 15:02 |
| ralonsoh | haleyb, yes, that should be running now to check the ovn agent with the extension | 15:02 |
| ralonsoh | haleyb, ah, and https://review.opendev.org/c/openstack/neutron/+/905297\ | 15:03 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/905297 | 15:03 |
| haleyb | ralonsoh: there was another | 15:03 |
| haleyb | https://review.opendev.org/c/openstack/neutron/+/913458 | 15:04 |
| haleyb | if that is required | 15:04 |
| ralonsoh | right, good one yes | 15:04 |
| opendevreview | Merged openstack/neutron stable/2023.2: Fix iptables mapping of 'ipip' protocol https://review.opendev.org/c/openstack/neutron/+/910911 | 15:28 |
| opendevreview | Merged openstack/neutron stable/2023.2: Use the system-dependent string for IP protocol 4 https://review.opendev.org/c/openstack/neutron/+/911604 | 15:28 |
| opendevreview | Merged openstack/neutron master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/neutron/+/913398 | 15:28 |
| opendevreview | Terry Wilson proposed openstack/ovsdbapp master: Don't assume BFD.status doesn't change https://review.opendev.org/c/openstack/ovsdbapp/+/913583 | 16:12 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [OVN] Use the OVN agent in "neutron-tempest-plugin-ovn" https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/909860 | 17:25 |
| opendevreview | Merged openstack/neutron master: [OVN] Add documentation about the OVN agent https://review.opendev.org/c/openstack/neutron/+/905297 | 17:25 |
| opendevreview | Merged openstack/neutron master: Add metadata options to generated ovn_agent.ini https://review.opendev.org/c/openstack/neutron/+/913458 | 17:29 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Add job definitions for 2024.1 (Caracal) branch https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/913450 | 17:49 |
| opendevreview | Merged openstack/ovsdbapp master: ovn-sb: remove chassis from Chassis_Private on ChassisDelCommand https://review.opendev.org/c/openstack/ovsdbapp/+/817300 | 18:41 |
| opendevreview | Lajos Katona proposed openstack/neutron unmaintained/yoga: Set trunk parent port as access port in ovs to avoid loop https://review.opendev.org/c/openstack/neutron/+/913505 | 20:23 |
| opendevreview | Brian Haley proposed openstack/neutron master: Add note on iptables cleanup after OVS firewall migration https://review.opendev.org/c/openstack/neutron/+/911957 | 20:39 |
| opendevreview | Merged openstack/ovsdbapp master: nb: commands: fix passing string to LSP attach/detach mirror API https://review.opendev.org/c/openstack/ovsdbapp/+/910278 | 21:33 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!