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 fix, initializing network interface on crash kernel boot over 2023_11 release #18882

Open
wants to merge 7 commits into
base: 202311
Choose a base branch
from

Conversation

bilal-ismail
Copy link

Why I did it

For pre-boot initialization of network interfaces.
For my case, I had to test and implement the remote kdump file transfer using SSH and the crash kernel did not load configs from /etc/network/interfaces config file.

How I did it

New file created for a custom script, "network-interface-state-init.sh"
Modified the "build_debian.sh" file for adding this to the /usr/sbin/network-interface-state-init.sh.

How to verify it

After this change the remote SSH kdump file transfer would be possible because network not reachable issue arose as the default crash kernel image did not change state of the interfaces to "UP" upon load. Neither did it load configs from /etc/network/interfaces config file.

@bilal-ismail bilal-ismail requested a review from lguohan as a code owner May 6, 2024 14:06
… been incorporated to call our shell script to initialize the interfaces on crashkernel load.
…mage-forked into custom_kdump_fix_2023_11

updated the current branch with latest 202311
…tools installed and have a script file at location "/usr/sbin/kdump-config".
…uild_debian.sh to sonic_debian_extension.j2

branch back to fea28ec commit
@bilal-ismail
Copy link
Author

This fix had build issues in both "marvell_armhf" and "marvell_arm64" artifacts, therefore because of a conditional statement these changes have been skipped in build of those artifacts where kdump-tools packages is not installed on default location/path i.e for the script, "/usr/sbin/kdump-config"

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.

1 participant