Skip to content

[config reload]Config Reload Enhancement#45

Merged
qiluo-msft merged 12 commits intosonic-net:masterfrom
dgsudharsan:config_rel_en
Apr 11, 2023
Merged

[config reload]Config Reload Enhancement#45
qiluo-msft merged 12 commits intosonic-net:masterfrom
dgsudharsan:config_rel_en

Conversation

@dgsudharsan
Copy link
Contributor

Code changes for config reload Enhancement sonic-net/SONiC#1203.
Enhancing config reload to sequence the services and faster system initialization.

Immediately restart the critical services during config reload.
The non critical should be started only after all the ports are initialized.
Services can be configured to be started immediately or delayed. This can be using a field in FEATURE table.
The existing timers should be removed by this event driven approach.
This flow is applicable in case of all reboots (warm/fast/cold) as well as config reload.

@xumia
Copy link
Contributor

xumia commented Feb 24, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dgsudharsan dgsudharsan requested a review from qiluo-msft March 8, 2023 23:41
@dgsudharsan
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link

/AzurePipelines run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 45 in repo sonic-net/sonic-host-services

@dgsudharsan
Copy link
Contributor Author

/azpw run sonic-net.sonic-host-services

@mssonicbld
Copy link

/AzurePipelines run sonic-net.sonic-host-services

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 45 in repo sonic-net/sonic-host-services

@dgsudharsan
Copy link
Contributor Author

@xumia @qiluo-msft Can you please restart the pipeline here?

@xumia
Copy link
Contributor

xumia commented Mar 11, 2023

/azp run

1 similar comment
@xumia
Copy link
Contributor

xumia commented Mar 11, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xumia
Copy link
Contributor

xumia commented Mar 11, 2023

@dgsudharsan, the azp has an issue relative to the GitHub connection, fixed.

@xumia
Copy link
Contributor

xumia commented Mar 11, 2023

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Mar 18, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

@dgsudharsan dgsudharsan requested a review from qiluo-msft March 18, 2023 00:45
@dgsudharsan dgsudharsan force-pushed the config_rel_en branch 2 times, most recently from 7f848a9 to 4bcc9dc Compare March 18, 2023 02:57
@dgsudharsan
Copy link
Contributor Author

/EasyCLA

@liat-grozovik
Copy link
Collaborator

@qiluo-msft could you please help to review?

@qiluo-msft qiluo-msft merged commit b08e940 into sonic-net:master Apr 11, 2023
dgsudharsan added a commit to dgsudharsan/sonic-host-services that referenced this pull request May 26, 2023
Code changes for config reload Enhancement sonic-net/SONiC#1203.
Enhancing config reload to sequence the services and faster system initialization.

Immediately restart the critical services during config reload.
The non critical should be started only after all the ports are initialized.
Services can be configured to be started immediately or delayed. This can be using a field in FEATURE table.
The existing timers should be removed by this event driven approach.
This flow is applicable in case of all reboots (warm/fast/cold) as well as config reload.
StormLiangMS pushed a commit that referenced this pull request Jun 10, 2023
Backporting #45
Code changes for config reload Enhancement sonic-net/SONiC#1203.
Enhancing config reload to sequence the services and faster system initialization.

Immediately restart the critical services during config reload.
The non critical should be started only after all the ports are initialized.
Services can be configured to be started immediately or delayed. This can be using a field in FEATURE table.
The existing timers should be removed by this event driven approach.
This flow is applicable in case of all reboots (warm/fast/cold) as well as config reload.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants