[rfe] Add one api support CRUD allowed_address_pairs

Bug #2012332 reported by Liu Xie
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
In Progress
Wishlist
huanghailun

Bug Description

Currently, many customers are using k8s for their VMs, and there is a need to add or remove allowed_address_pairs. However, frequent update neutron port is time-consuming. Can we provide an API only support CRUD allowed_address_pairs?

Tags: rfe-approved
Akihiro Motoki (amotoki)
tags: added: rfe
Changed in neutron:
importance: Undecided → Wishlist
Revision history for this message
Akihiro Motoki (amotoki) wrote :

The request sounds reasonable request to me. The current API requires users to fetch the current set of allowed-address-pairs and then update it. We had the similar issue around router routes and we added "router routes (atomic) extension" [1]. I think we can use the similar approach for allowed-address-pairs too.

I am not sure the current API is "time-consuming" as described in the bug description though.

[1] https://docs.openstack.org/api-ref/network/v2/index.html?expanded=remove-extra-routes-from-router-detail,add-extra-routes-to-router-detail#extra-routes-atomic-extension

Changed in neutron:
status: New → Confirmed
Changed in neutron:
assignee: nobody → Liu Xie (liushy)
Revision history for this message
Rodolfo Alonso (rodolfo-alonso-hernandez) wrote :

This RFE was approved in the Neutron drivers meeting. It was agreed not to request a spec. Any implementation detail will be discussed in the code review.

tags: added: rfe-approved
removed: rfe
huanghailun (lnun)
Changed in neutron:
assignee: Liu Xie (liushy) → huanghailun (lnun)
Changed in neutron:
status: Confirmed → In Progress
Revision history for this message
Rodolfo Alonso (rodolfo-alonso-hernandez) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/880922

Revision history for this message
huanghailun (lnun) wrote (last edit ):
Revision history for this message
huanghailun (lnun) wrote :
Revision history for this message
huanghailun (lnun) wrote :
Revision history for this message
Liu Xie (liushy) wrote :

Waiting for the api-ref is agreed by all people

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron-lib (master)

Reviewed: https://review.opendev.org/c/openstack/neutron-lib/+/880615
Committed: https://opendev.org/openstack/neutron-lib/commit/872807640227193bf3bfb4800b2253ea975538e9
Submitter: "Zuul (22348)"
Branch: master

commit 872807640227193bf3bfb4800b2253ea975538e9
Author: liushy <liuxie_11@163.com>
Date: Mon Apr 17 16:55:07 2023 +0800

    New api-def: allowedaddresspairs-atomic

    Partial-Bug: #2012332
    Change-Id: Ibc713217ffb67ce35e03ae7f341a302a9edef4d1

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (master)

Change abandoned by "Slawek Kaplonski <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/888039
Reason: This review is > 4 weeks without comment, and failed Zuul jobs the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by "Slawek Kaplonski <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/880922
Reason: This review is > 4 weeks without comment, and failed Zuul jobs the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/892116

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (master)

Change abandoned by "Slawek Kaplonski <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/892116
Reason: This review is > 4 weeks without comment, and failed Zuul jobs the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/901895

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (master)

Change abandoned by "Slawek Kaplonski <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/901895
Reason: This review is > 4 weeks without comment, and failed Zuul jobs the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron-lib (master)

Change abandoned by "Rodolfo Alonso <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/neutron-lib/+/887659
Reason: Feel free to restore this patch and address the comments

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.