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

Kdump enhancement for remote ssh, cli subcommands added. #3317

Open
wants to merge 49 commits into
base: master
Choose a base branch
from

Conversation

bilal-ismail
Copy link

What I did

Worked on the enhancement of kdump utility for sonic. Added two new subcommands along with two options i.e
1- sudo config kdump remote ssh --conn-str ___ --key-path ___
2- sudo config kdump remote disable

How I did it

Made changes to 3 files in total,
1- config/kdump (added two new subcommands and options for modentry to config_db)
2- show/kdump (cli command to print running config)
3- scripts/sonic-kdump-config (scripts or ready methods for usage upon load and unload of kdump module)

How to verify it

upon loading this utilities' build to the running sonic image we can try and test these commands for verifying it's behavior as per the requirements.

bilal-ismail and others added 30 commits May 6, 2024 10:27
An update for the current fork to keep working on latest codebase for the use of kdump remote usecase.
…ies-forked into kdump_cli_updates

updateing the branch with master
…ies-forked into kdump_cli_updates

updating the branch with latest comits to the repo
…ate>=0.8.2,<=0.9.0". For resolving conflict with sonic-yang-mgmt 1.0 dependency of tabulate.
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