Skip to content

Set the correct admin status when creating kernel and tap ports - #758

Merged
sonikajindal merged 13 commits into
openconfig:mainfrom
gavinz-ship-it:main
Oct 19, 2026
Merged

sonikajindal merged 13 commits into
openconfig:mainfrom
gavinz-ship-it:main

Conversation

@gavinz-ship-it

Copy link
Copy Markdown
Contributor

The original code creates the interfaces with a default admin status. Then send another rpc call to update the status if it is not correct. The new code sets the correct admin status when creating the interfaces.

Add admin_state to KernelPortDesc
Add admin_state to TAPPortDesc
Add code to set the admin status.
Update to set the admin status
Set admin status when creating the ports.
Add imports
@gavinz-ship-it
gavinz-ship-it requested a review from a team as a code owner October 17, 2026 00:44
Remove redundant code
Revert previously changed logic to set the admin status since pcap requires the status to be up.
@sonikajindal

Copy link
Copy Markdown
Contributor

/gcbrun

Added error handling for LinkSet
Fix unit test.
@sonikajindal

Copy link
Copy Markdown
Contributor

/gcbrun

Comment thread dataplane/saiserver/ports.go
Comment thread dataplane/forwarding/fwdport/ports/tap.go
Fix the previously changed oper status for tap ports.
@sonikajindal

Copy link
Copy Markdown
Contributor

/gcbrun

@sonikajindal
sonikajindal merged commit 5f571d0 into openconfig:main Oct 19, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants