Skip to content

Conversation

@jkmar
Copy link
Contributor

@jkmar jkmar commented Oct 6, 2025

What I did

Cherry pick #3872 to 202505 branch

How I did it

How to verify it

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

…c-net#3872)

* Fix warm-reboot script so it can be run via reboot DBus service.
Use whoami if logname failed.

* Add UT

* Fix review comments

* Update scripts/fast-reboot

---------
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Verma-Anukul
Copy link
Contributor

@vvolam

Please merge this PR

cherry-picked from #3872

gnmi/test_gnoi_system_reboot.py test fails sometimes, this patch is required for fixing the same.

@Verma-Anukul
Copy link
Contributor

@ganglyu @saiarcot895 @qiluo-msft @yaqiangz @prgeor @xincunli-sonic

Please help in merging this PR

@qiluo-msft
Copy link
Contributor

@yejianquan could you check 202505 PR?

@Verma-Anukul
Copy link
Contributor

@yejianquan

Can you please review and approve this PR

@Verma-Anukul
Copy link
Contributor

@yejianquan

Can you please review and approve this PR

@Verma-Anukul
Copy link
Contributor

@xincunli-sonic @prgeor @qiluo-msft @liat-grozovik @yaqiangz @vvolam

Can you please help in merging this cherry-picked change from master

@qiluo-msft qiluo-msft requested a review from yejianquan October 29, 2025 18:07
@Verma-Anukul
Copy link
Contributor

@xincunli-sonic @prgeor @qiluo-msft @liat-grozovik @yaqiangz @vvolam

Can you please help in merging this cherry-picked change from master

@Verma-Anukul
Copy link
Contributor

@qiluo-msft @yejianquan

Please check

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.

4 participants