Skip to content

gNOI Cold Reboot - Integrated Rebootbackend changes#20710

Open
rkavitha-hcl wants to merge 2 commits intosonic-net:masterfrom
rkavitha-hcl:gnoi_cr_PR2
Open

gNOI Cold Reboot - Integrated Rebootbackend changes#20710
rkavitha-hcl wants to merge 2 commits intosonic-net:masterfrom
rkavitha-hcl:gnoi_cr_PR2

Conversation

@rkavitha-hcl
Copy link
Contributor

Why I did it

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

  1. This PR has the code for reboot backend processing to handle Cold Reboot feature.
  2. Framework docker starts the rebootbackend process which receives notification from Redis when there is any gNOI request for Cold Reboot.
  3. It processes the request and updates the Reboot status flags and then notifies sonic-host-services to trigger the reboot thro' dbus interface.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@rkavitha-hcl rkavitha-hcl requested a review from lguohan as a code owner November 6, 2024 07:31
@qiluo-msft qiluo-msft requested review from hdwhdw and vvolam November 7, 2024 17:29
@rkavitha-hcl rkavitha-hcl force-pushed the gnoi_cr_PR2 branch 2 times, most recently from 5ac7797 to c4b69a6 Compare November 18, 2024 09:17
@hdwhdw hdwhdw self-requested a review November 20, 2024 16:21
Copy link
Contributor

@hdwhdw hdwhdw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally OK. Just some nits.

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@bibhuprasad-hcl
Copy link
Contributor

@kishanps, could you please review

@bibhuprasad-hcl
Copy link
Contributor

@vvolam , could you please review

@bibhuprasad-hcl
Copy link
Contributor

@hdwhdw , could you please review

hdwhdw
hdwhdw previously approved these changes Feb 20, 2025
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

5 participants