-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Temporary WA for the issue that asic_table.json can not be rendered #13888
Conversation
We suspect the issue is caused by redis server temporarily unavailable so we do not use -d in sonic-cfggen for now Signed-off-by: Stephen Sun <stephens@nvidia.com>
@stephenxs was this approach should be taken in other places in the code? |
Hi @liat-grozovik |
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
@qiluo-msft could you please review/approve? |
/azpw run azure.sonic-buildimage |
/AzurePipelines run azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
…onic-net#13888) - Why I did it We suspect the issue sonic-net#13791 is caused by redis server being temporarily unavailable during system initialization so we do not use -d in sonic-cfggen, for now, to avoid accessing redis server - How I did it Provide a string containing required json data when calling sonic-cfggen - How to verify it Manually test it Signed-off-by: Stephen Sun <stephens@nvidia.com>
Cherry-pick PR to 202211: #14857 |
Why I did it
We suspect the issue #13791 is caused by redis server being temporarily unavailable during system initialization so we do not use -d in sonic-cfggen, for now, to avoid accessing redis server
Signed-off-by: Stephen Sun stephens@nvidia.com
How I did it
Provide a string containing required json data when calling
sonic-cfggen
How to verify it
Manually test it
Which release branch to backport (provide reason below if selected)
Description for the changelog
Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)