Skip to content

convert vppdev.sh to Go binary #727

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

aritrbas
Copy link
Collaborator

@aritrbas aritrbas commented Jun 11, 2025

[CALICOVPP-9] This converts vppdev.sh to a Go binary to leverage Golang APIs.

Placed it under an "util" folder.
"make util" creates the "calivppctl" Go executable under util/bin/ directory.

Usage:

  • calivppctl vppctl [NODENAME] - Get a vppctl shell on a specific node
  • calivppctl log [-f] [-vpp|-agent|-felix] [NODENAME] - Get the logs of vpp (dataplane) or agent (controlplane) or felix daemon
  • calivppctl clear - Clear vpp internal stats
  • calivppctl export - Create an archive with vpp & k8 system state for debugging.
  • calivppctl exportnode [NODENAME] - Create an archive with vpp & k8 system state for a specific node.
  • calivppctl gdb - Attach a gdb to the running vpp on the current machine
  • calivppctl sh [vpp|agent] [NODENAME] - Get a shell in vpp (dataplane) or agent (controlplane) container

Copy link
Collaborator

@sknat sknat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot @aritrbas ! That's neat !
Added a few comments inline, some are more debatable (e.g. I don't think we need to remove every single CLI commands for the V0) but I think we'll eventually want to converge there.

Happy to discuss this with you

@aritrbas aritrbas force-pushed the abasu-gocalivppctl branch from e55cde0 to 41aa962 Compare June 24, 2025 23:29
Signed-off-by: Aritra Basu <[email protected]>
@aritrbas aritrbas force-pushed the abasu-gocalivppctl branch from 41aa962 to 50a4793 Compare July 23, 2025 19:32
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