[hostcfgd] [202012] Fix the delay type to 'boot' delay instead of a unit activation delay #8896
Conversation
Signed-off-by: Shlomi Bitton <shlomibi@nvidia.com>
|
@dgsudharsan can you please review? |
|
@lguohan Can you please review and signoff? |
|
Should this go master first, and then backport to 202012? |
no, there is a PR against master also : #8895 |
|
In master branch PR, there is change in file src/sonic-host-services-data/debian/sonic-host-services-data.hostcfgd.timer. Why not here? |
@qiluo-msft there is a change here for 'sonic-host-services-data.hostcfgd.timer', I think you are referring to 'sonic-host-services-data.aaastatsd.timer' and we don't have it on 202012 branch, this is why we don't have it on this PR. |
|
@renukamanavalan @qiluo-msft Can you please sign off? |
Signed-off-by: Shlomi Bitton shlomibi@nvidia.com
Why I did it
With current code the delay will take place even if simple 'config reload' command executed and this is not desired.
This delay should be used only when fast-rebooting.
How I did it
Change the type of delay to OnBootSec instead of OnActiveSec.
How to verify it
Fast-reboot with this PR and observe the delay.
Run 'config-reload' command and observe no delay is running.
Which release branch to backport (provide reason below if selected)
Description for the changelog
A picture of a cute animal (not mandatory but encouraged)