Fixed: Kdump Remote Patch

scripts/sonic-kdump-config

@@ -433,7 +433,7 @@ def write_num_dumps(num_dumps):
 def write_kdump_remote():
     # Assuming there's a function that retrieves the remote value from the config database
     remote = get_kdump_remote()  # This function needs to be implemented
-    
+
     kdump_cfg = '/etc/default/kdump-tools'
 
     if remote:
@@ -641,7 +641,7 @@ def cmd_kdump_config_next(verbose):
 
 
 def kdump_disable(verbose, image, booter_config_file_path):
-    """Unloads Kdump kernel and remove parameter `crashkernel=*` from configuration file of 
+    """Unloads Kdump kernel and remove parameter `crashkernel=*` from configuration file of
     kernel boot loader.
 
     Args:
@@ -826,6 +826,9 @@ def main():
     parser.add_argument('--num_dumps', nargs='?', type=int, action='store', default=False,
         help='Maximum number of kernel dump files stored')
 
+    parser.add_argument('--remote', nargs='?', type=bool, action='store', default=False,
+        help='remote ssh enable disable')
+
     parser.add_argument('--ssh_string', nargs='?', type=str, action='store', default=False,
             help='ssh_string for remote kdump')
 

tests/sonic_kdump_config_test.py

@@ -4,6 +4,7 @@
 import unittest
 from unittest.mock import patch, mock_open, Mock
 from utilities_common.general import load_module_from_source
+from /home/ali/Desktop/sonic-kdump-8th-aprail-2025/sonic-utilities/scripts/sonic-kdump-config import main
 
 from sonic_installer.common import IMAGE_PREFIX
 
@@ -328,6 +329,24 @@ def test_get_image(self, mock_get_bootloader):
             return_result = sonic_kdump_config.get_next_image()
         self.assertEqual(sys_exit.exception.code, 1)
 
+    # @patch('os.geteuid')
+    # @patch('sonic_kdump_config.cmd_kdump_remote')
+    # def test_cmd_remote(self, mock_cmd_kdump_remote, mock_geteuid):
+    #     """Tests the function `cmd_kdump_remote(...)` in script `sonic-kdump-config`.
+    #     """
+    #     # Mock the output of cmd_kdump_remote
+    #     mock_cmd_kdump_remote.return_value = None  # Assuming the function prints directly
+
+    #     # Mock os.geteuid to return 0 (root user)
+    #     mock_geteuid.return_value = 0
+
+    #     # Call the function with the --remote argument
+    #     with patch('sys.argv', ['sonic-kdump-config', '--remote']):
+    #         main()
+
+    #     # Verify that cmd_kdump_remote was called
+    #     mock_cmd_kdump_remote.assert_called_once()
+
     @patch("sonic_kdump_config.run_command")
     @patch("sonic_kdump_config.get_kdump_remote")
     def test_write_kdump_remote_true(self, mock_read_remote, mock_run_command):