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.

Sign up for GitHub

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

Jump to bottom

Add unixctl library to interface with OVS #1

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Add unixctl library to interface with OVS #1

wants to merge 6 commits into from

Conversation

amorenoz
Copy link
Contributor

@amorenoz amorenoz commented Nov 5, 2024 •
edited
Loading

This is the first PR of this project. It adds a very basic library functionality that can serve as basis for more OVS appctl commands.

Includes:

  • a jsonrpc library that can work with any transport (as in theory unixctl can be accessed via tcp). Note OVS does not implement any standard jsonrpc version
  • a unix socket transport for the jsonrpc library
  • a ovs-unixctl wrapper that adds some well known commands and parses the responses
  • a bit of CI

@amorenoz amorenoz requested a review from vlrpl November 5, 2024 23:17
@amorenoz amorenoz force-pushed the first branch 2 times, most recently from fc2ca88 to cfbd31e Compare November 7, 2024 21:28
@amorenoz
add a tiny JSON-RPC client over UNIX socket
2bce20a 
OVS' JSON-RPC interface is not a standard one so work around all the
quirks, e.g: params being mandatory, errors being strings, etc.

Signed-off-by: Adrian Moreno <[email protected]>
@amorenoz
ovs: introduce OvsUnixCtl struct
ec94fa5 
For now, only constructors are added.

Signed-off-by: Adrian Moreno <[email protected]>
@amorenoz
ovs: support listing commands
1a5aca1 
Add a test that spawns OVS and sends the command against it.

Signed-off-by: Adrian Moreno <[email protected]>
@amorenoz
ovs: add "version" command
bf52eeb 
Refactor tests to make sure cleanup does happen without relying on c-p.

Signed-off-by: Adrian Moreno <[email protected]>
@amorenoz
ovs: allow arbitrary command runner
a7c4362 
Test it using a vlog/set and vlog/list.

Signed-off-by: Adrian Moreno <[email protected]>
@amorenoz
add basic CI
61acbc5 
Signed-off-by: Adrian Moreno <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vlrpl vlrpl Awaiting requested review from vlrpl

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@amorenoz