all PR comment changes

Author: rh-jugraham

virttest/utils_test/__init__.py

@@ -254,28 +254,28 @@ def update_boot_option(
 def update_vm_default_kernel(
     vm,
     kernel_version,
-    reboot=True,
+    reboot,
+    guest_arch_name,
     timeout="",
-    guest_arch_name="x86_64",
     serial_login=False,
 ):
     """
     Update guest default kernel.
 
     :param vm: The VM object.
     :param kernel_version: The full kernel version that should be set as the new default.
+                           Can use the entire kernel path or just be the full kernel name.
     :param reboot: Whether to reboot the VM and update the kernel used or not.
-    :param timeout: Timeout for login and reboot.
     :param guest_arch_name: Guest architecture, e.g. x86_64, s390x, aarch64.
+    :param timeout: Timeout for login and reboot.
     :param serial_login: Login guest via serial session.
+    :raise NotImplementedError: Raised if Windows guest, not Linux.
     :raise exceptions.TestError: Raised if failed to update the guest kernel.
     """
-    session = None
     if vm.params.get("os_type") == "windows":
-        msg = "update_vm_default_kernel() is supported only for Linux guest"
-        LOG.warning(msg)
-        return
-    timeout = int(timeout) or int(vm.params.get("login_timeout"))
+        raise NotImplementedError("update_vm_default_kernel() is supported only for Linux guest")
+    
+    timeout = int(timeout) if timeout else int(vm.params.get("login_timeout"))
     session = vm.wait_for_login(
         timeout=timeout, serial=serial_login, restart_network=True
     )