[gnoi_shutdown_daemon] Log gNOI Reboot failure details

[chartsai-nvidia]

Why I did it

When the gNOI Reboot (HALT) call to a DPU failed during graceful shutdown, the daemon only logged
"Reboot command failed" and hid stderr. That made failures hard to debug.

Work item tracking

• Microsoft ADO (number only): N/A

How I did it

• Stopped hiding stderr on the gNOI Reboot call.
• Added the return code, target IP:port, and the real error text to the log message.

How to verify it

• Run a graceful shutdown where the gNOI Reboot call fails (for example, an unreachable DPU).
• Check syslog and confirm the log now shows rc, target IP:port, and the gNOI error text.

Description for the changelog

Log full gNOI Reboot failure details (rc, target, stderr) in gnoi_shutdown_daemon.

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[rameshraghupathy]

@chartsai-nvidia Please enhance test_send_reboot_command_failure to verify the behavior introduced by this PR. Besides this it LGTM.

[mssonicbld]

Hi @sonic-net/sonic-host-services-maintainer, this approved PR has workflow run(s) waiting for approval. Please help review. Thanks!

_{---Powered by SONiC BuildBot}

[hdwhdw]

LGTM. But beware of our work in sonic-net/sonic-buildimage#28341. When this is merged we should migrate the entire framework to using python native gnoi client, so you won't have this problem.

[vvolam]

LGTM

[rookie-who]

This PR has the "Request for 202605 Branch" label but is missing 202605-specific test results. Please:

1. Tick the 202605 checkbox in the "Tested branch" section of the PR description
2. Provide 202605 branch test evidence in the "Test result" section
3. Add the label "Tested for 202605 branch" once done

Master branch test results alone are not sufficient for 202605 cherry-pick approval.

Comment on behalf of @vaibhavhd
Review by @vaibhavhd via automated tooling

[mssonicbld]

This PR has backport request label(s) for branch(es): 202605, but is missing required test information. Please make sure you tick the tested branch(es) in the Tested branch section and provide test evidence (e.g., 202605: <test result>) in the Test result section as well in your PR description.

_{---Powered by SONiC BuildBot}

[vaibhavhd]

202605 tests still not updated