Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PATCH l2vpn should accept single fields in request body #348

Open
usmanazFIU opened this issue Oct 30, 2024 · 1 comment
Open

PATCH l2vpn should accept single fields in request body #348

usmanazFIU opened this issue Oct 30, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@usmanazFIU
Copy link

PATCH l2vpn should accept one field or all fields from the request body that needs to be edited. right now it expects endpoints and everything else because it is first deleting the connection and then creating a connection with the same service id. it should not delete and only edit as a PATCH would work

@YufengXin YufengXin added the enhancement New feature or request label Oct 30, 2024
@YufengXin
Copy link
Collaborator

YufengXin commented Oct 30, 2024

We'll deal with PATCH as one task after SC because it needs several issues corresponding to each field to be changed.

For example, if the patch changes the VLAN id, or BW constraints, the connection has to be deleted, and then re-provisioned. Another example is if it's allowed to change the endpoints?

This is the same as Sharon's ticket.

I tag this as "Enhancement" - a feature request to be discussed and dealt with later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants