| opendevreview | ZhouHeng proposed openstack/neutron master: "default_availability_zones" need to be considered when validate az https://review.opendev.org/c/openstack/neutron/+/802665 | 00:14 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/neutron master: Add options for the healthcheck middleware https://review.opendev.org/c/openstack/neutron/+/804571 | 00:17 |
| opendevreview | Takashi Kajinami proposed openstack/neutron master: Use importlib instead of imp https://review.opendev.org/c/openstack/neutron/+/804596 | 00:24 |
| *** gibi_pto is now known as gibi | 06:08 | |
| opendevreview | mitya-eremeev-2 proposed openstack/neutron master: Agent RPC step size in config. https://review.opendev.org/c/openstack/neutron/+/802596 | 06:56 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: Add API extenstion for QoS minimum pps rule https://review.opendev.org/c/openstack/neutron/+/803045 | 07:05 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: ovs-agent: Report pkt processing info in heartbeat https://review.opendev.org/c/openstack/neutron/+/800444 | 07:05 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: Report CUSTOM_VNIC_TYPE_ traits on OVS agent RP https://review.opendev.org/c/openstack/neutron/+/800445 | 07:05 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: Bump os-resource-classes lib to 1.1.0 https://review.opendev.org/c/openstack/neutron/+/801066 | 07:06 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: Report pkt processing capacity on OVS agent RP https://review.opendev.org/c/openstack/neutron/+/800446 | 07:08 |
| opendevreview | OpenStack Proposal Bot proposed openstack/neutron-vpnaas-dashboard stable/victoria: Imported Translations from Zanata https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/804607 | 07:47 |
| opendevreview | Oleg Bondarev proposed openstack/neutron-specs master: Add Node-Local Virtual IP Spec https://review.opendev.org/c/openstack/neutron-specs/+/797798 | 08:27 |
| opendevreview | Oleg Bondarev proposed openstack/neutron-lib master: Add Local IP API def https://review.opendev.org/c/openstack/neutron-lib/+/803051 | 08:40 |
| opendevreview | Oleg Bondarev proposed openstack/neutron-lib master: Add Local IP constants and exceptions https://review.opendev.org/c/openstack/neutron-lib/+/804613 | 08:44 |
| opendevreview | Takashi Kajinami proposed openstack/neutron master: Add missing options to neutron.conf auto-generated https://review.opendev.org/c/openstack/neutron/+/804571 | 08:45 |
| opendevreview | Takashi Kajinami proposed openstack/neutron master: Add missing options to generated neutron.conf https://review.opendev.org/c/openstack/neutron/+/804571 | 08:46 |
| opendevreview | Szymon Wróblewski proposed openstack/neutron master: Use payloads for FloatingIP AFTER callbacks https://review.opendev.org/c/openstack/neutron/+/801453 | 08:55 |
| opendevreview | Szymon Wróblewski proposed openstack/neutron master: Improve content of FloatingIP AFTER callbacks https://review.opendev.org/c/openstack/neutron/+/798009 | 08:55 |
| lajoskatona | slaweq: Hi, could You check this one: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/803752 , it fails with regex issue: tempest run --regex '^neutron_tempest_plugin\.tap_as_a_service' | 09:13 |
| lajoskatona | slaweq: perhaps we have to merge 1st the test and than ad the job to execute them? | 09:14 |
| slaweq | lajoskatona: hi, checking | 09:18 |
| slaweq | lajoskatona: I think that we first need to delete it from tap-as-a-service repository: https://opendev.org/openstack/tap-as-a-service/src/branch/master/neutron_taas/tests/tempest_plugin | 09:24 |
| slaweq | because now plugin is installed from there and maybe that's causing problems | 09:25 |
| slaweq | can You try to propose patch which removes it from there and make https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/803752 to be depend on that new one? | 09:25 |
| lajoskatona | slaweq: sure, I check it | 09:36 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Document Network Availability Zones https://review.opendev.org/c/openstack/neutron/+/788040 | 10:16 |
| lucasagomes | slaweq++ thanks for the review, I've added the release note as part of the documentation patch ^ | 10:16 |
| slaweq | lucasagomes++ thx | 10:18 |
| opendevreview | Merged openstack/neutron master: "default_availability_zones" need to be considered when validate az https://review.opendev.org/c/openstack/neutron/+/802665 | 10:19 |
| slaweq | lucasagomes: I have one small comment in that release note | 10:21 |
| lucasagomes | slaweq, will take a look, thanks! | 10:36 |
| lucasagomes | slaweq, apparently there's a merge conflict on the patchs again :-( | 10:37 |
| lucasagomes | I will fix it and fix the release note and other comments in the reviews, thanks! | 10:37 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/wallaby: Add Health Monitor support https://review.opendev.org/c/openstack/ovn-octavia-provider/+/801890 | 10:53 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Make external ports aware of network AZs https://review.opendev.org/c/openstack/neutron/+/762550 | 11:04 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Document Network Availability Zones https://review.opendev.org/c/openstack/neutron/+/788040 | 11:04 |
| lucasagomes | slaweq, ^ sorry had to fix the merge conflict. Also addressed the comment u left in the patch (the if/or) | 11:05 |
| slaweq | lucasagomes: sure, I will take a look | 11:07 |
| lucasagomes | whenever u take time, thanks for reviewing it! | 11:07 |
| lucasagomes | I know it's a long patch :-/ | 11:07 |
| slaweq | lucasagomes: yes, but now I just checked diff from last PS :) | 11:18 |
| slaweq | so it's not that huge | 11:18 |
| lucasagomes | slaweq, cool yeah, it conflict with this change here https://review.opendev.org/c/openstack/neutron/+/802665 that moved a few things around | 11:19 |
| slaweq | lajoskatona: congrats - You now have +2 power in neutron patches for stable branches :) | 11:20 |
| lucasagomes | lajoskatona++ congrats | 11:20 |
| lajoskatona | slaweq, lucasgomes: thanks | 11:22 |
| slaweq | lajoskatona: thank You for Your work with stable releases :) | 11:23 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Tests: remove tempest plugin from taas repository https://review.opendev.org/c/openstack/tap-as-a-service/+/804707 | 11:45 |
| opendevreview | Merged openstack/neutron stable/victoria: Make ARP protection commands compatible with "ebtables-nft" https://review.opendev.org/c/openstack/neutron/+/804056 | 12:12 |
| opendevreview | Merged openstack/neutron stable/ussuri: Make ARP protection commands compatible with "ebtables-nft" https://review.opendev.org/c/openstack/neutron/+/804057 | 12:17 |
| opendevreview | Merged openstack/neutron stable/train: Make ARP protection commands compatible with "ebtables-nft" https://review.opendev.org/c/openstack/neutron/+/804058 | 12:17 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/804510 | 12:17 |
| lajoskatona | slaweq, amotoki: where should we land now new CLI code (like this one: https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/792338 )? | 12:49 |
| lajoskatona | slaweq, amotoki: openstackclient repo should be now the default for new CLI (https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/network/v2 )? | 12:50 |
| slaweq | lajoskatona: in the OSC and/or SDK repos | 12:50 |
| lajoskatona | slaweq: ok so the 2nd option, thanks | 12:50 |
| amotoki | lajoskatona: slaweq: neutronclient OSC plugin hosts network related features in the stadium projects. | 13:07 |
| amotoki | lajoskatona: slaweq: an unclear point is where we should implement python binding support: openstacksdk vs python-neutronclient. | 13:08 |
| lajoskatona | amotoki: like this one for bgpvpn: https://opendev.org/openstack/python-neutronclient/src/branch/master/neutronclient/osc/v2/networking_bgpvpn | 13:09 |
| amotoki | lajoskatona: exactly | 13:09 |
| lajoskatona | amotoki: the best would be to have everything in one place, but that's not cheap :-) | 13:10 |
| amotoki | as of now, neutronclient OSC plugin consumes python bindings in python-neutronclient repo, so adding corresponding stuff to the neutronclient repo would be straight-forward. | 13:10 |
| lajoskatona | and from checking governance perspective as well: python-neutronclient is governed by the team (plus the convenience features like notifications from new patches :P) vs sdk is out of team's hands | 13:12 |
| amotoki | CLI and SDK have a bit different stories. OSC focuses on covering core features. | 13:16 |
| amotoki | on the other hand openstacksdk tries to cover all as much as possible. To make it smooth openstacksdk has many cores from individual project teams. | 13:16 |
| amotoki | as long term, we may need to revisit whether we really need to maintain two python bindings (sdk) but it does not happen soon. | 13:17 |
| amotoki | lajoskatona: so my answer to your question is to implement the support for a new feature of both CLI (OSC plugin) and bindings in dynamic-rotuing in python-neutronclient. | 13:19 |
| opendevreview | Szymon Wróblewski proposed openstack/neutron stable/stein: Fix notify listener syntax for SEGMENT_HOST_MAPPING https://review.opendev.org/c/openstack/neutron/+/803738 | 13:24 |
| opendevreview | Szymon Wróblewski proposed openstack/neutron stable/rocky: Fix notify listener syntax for SEGMENT_HOST_MAPPING https://review.opendev.org/c/openstack/neutron/+/804720 | 13:25 |
| opendevreview | Szymon Wróblewski proposed openstack/neutron stable/queens: Fix notify listener syntax for SEGMENT_HOST_MAPPING https://review.opendev.org/c/openstack/neutron/+/804721 | 13:25 |
| opendevreview | Andrey Bubyr proposed openstack/neutron master: Pass binding:vnic_type in case of bulk port creation https://review.opendev.org/c/openstack/neutron/+/804746 | 13:34 |
| lajoskatona | amotoki: thanks | 13:38 |
| opendevreview | Merged openstack/neutron master: ovn: Don't fail db-sync if port binding changes https://review.opendev.org/c/openstack/neutron/+/804405 | 14:02 |
| opendevreview | Eduardo Olivares proposed openstack/neutron-tempest-plugin master: Checking IP version from extra-dhcp-options https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/804755 | 14:13 |
| opendevreview | Oleg Bondarev proposed openstack/neutron master: [WIP] Add Local IP Extension and DB https://review.opendev.org/c/openstack/neutron/+/804523 | 14:22 |
| opendevreview | mitya-eremeev-2 proposed openstack/neutron master: Agent RPC step size in config. https://review.opendev.org/c/openstack/neutron/+/802596 | 14:25 |
| opendevreview | Merged openstack/neutron-lib master: Allow PUT on direction param in QoS minimum pps rule https://review.opendev.org/c/openstack/neutron-lib/+/803914 | 15:11 |
| opendevreview | Jakub Libosvar proposed openstack/neutron stable/wallaby: ovn: Don't fail db-sync if port binding changes https://review.opendev.org/c/openstack/neutron/+/804723 | 15:51 |
| opendevreview | Merged openstack/neutron master: Remove ``ConfDriver`` code https://review.opendev.org/c/openstack/neutron/+/790999 | 17:36 |
| opendevreview | Terry Wilson proposed openstack/neutron master: Support SB OVSDB connections to non-leader servers https://review.opendev.org/c/openstack/neutron/+/803268 | 22:07 |
| opendevreview | Terry Wilson proposed openstack/neutron master: Support SB OVSDB connections to non-leader servers https://review.opendev.org/c/openstack/neutron/+/803268 | 23:09 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!