search

sonic-net / sonic-sairedis

SAI object interface to Redis database, as used in the SONiC project
Other
57 stars 272 forks source link

Use temp view also in fast-reboot #1401

Open arfeigin opened 4 months ago

arfeigin commented 4 months ago

syncd_init_common.sh is checking fast-reboot by reading /proc/cmdline. However, /proc/cmdline will not change after config reload. So, consider a case like fast-reboot -> config reload. In the config reload process, syncd_init_common.sh will also treat it as fast-reboot and probably enter wrong logic.

arfeigin commented 4 months ago

@kcudnik Can you please help reviewing this PR?

arfeigin commented 4 months ago

@vaibhavhd Can you please reviewing this PR?

Junchao-Mellanox commented 4 months ago

Hi @vaibhavhd , could you please help review this?

stepanblyschak commented 1 month ago

@vaibhavhd The change is in our internal 202405 branch and running in regression for few month now with no issues related to fast-reboot

bingwang-ms commented 1 month ago

@arfeigin Can you please sync with master branch? The PR can't be merged because it's out-of-date with master branch

dprital commented 1 day ago

@bingwang-ms , Can you please merge and cherry pick to 202405 ?