Update review comments

Author: adityagesh

lisa/node.py

@@ -768,11 +768,7 @@ def _collect_logs_using_non_ssh_executor(self) -> None:
 
         if self.features.is_supported(NonSshExecutor):
             non_ssh_executor = self.features[NonSshExecutor]
-            outputs = non_ssh_executor.execute(commands=commands)
-            collected_info = "\n\n".join(outputs)
-            self.log.info(
-                f"Collected information using NonSshExecutor:\n{collected_info}"
-            )
+            non_ssh_executor.execute(commands=commands)
         else:
             self.log.debug(
                 f"NonSshExecutor is not supported on {self.name}, "

lisa/sut_orchestrator/azure/features.py

@@ -3784,20 +3784,15 @@ def execute(self, commands: List[str]) -> str:
             parameters=run_command_input,
         )
         result = wait_operation(operation=operation, failure_identity="run command")
-        try:
-            # Since wait_operation returns a dict (result.as_dict()), access as dict
-            value = result.get("value")
-            if value and len(value) > 0 and value[0].get("message"):
-                message = value[0]["message"]
-            else:
-                raise LisaException(
-                    "RunCommand did not run successfully. "
-                    f"Got response: '{value}'. Expected response to contain `value[0]['message']`"
-                )
+        value = result.get("value")
+        if value and value[0].get("message"):
+            message = value[0]["message"]
+        else:
+            raise LisaException(
+                "RunCommand did not run successfully. "
+                f"Got response: '{value}'. Expected response to contain `value[0]['message']`"
+            )
 
-        except Exception:
-            self._log.info("RunCommand failed to return expected result.")
-            raise
         return message
 
     def _add_echo_before_command(self, commands: List[str]) -> List[str]: