Skip to content
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

Render SONiC config_db.json directly without using sonic-cfggen #211

Merged
merged 18 commits into from
Sep 22, 2023

Conversation

robertvolkmann
Copy link
Contributor

@robertvolkmann robertvolkmann commented Sep 5, 2023

Previously, our method involved persisting the entirety of the running configuration into /etc/sonic/config_db.json by running config save in a task. This approach led to undesired configurations being retained, especially noticeable after a switch reboot. A key example of such undesired configurations arises from metal-core.

To refine this, we have now transitioned to a selective persistence strategy. Instead of saving the entire running configuration, only crucial segments are extracted and stored. Specifically, these segments relate to the breakout and port configuration. This approach ensures a cleaner and more focused /etc/sonic/config_db.json, mitigating the presence of unintended configurations.

Optimize persistence strategy by selectively storing essential segments, such as breakout and port configurations, instead of the entire running configuration. This mitigates undesired configurations, particularly those arising from metal-core.
@robertvolkmann robertvolkmann force-pushed the refine-sonic-config-persistence branch from 4058429 to e233af9 Compare September 8, 2023 11:59
@Gerrit91 Gerrit91 changed the title Optimize /etc/sonic/config_db.json Persistence by Selectively Storing Essential Configurations Render SONiC config_db.json directly without using sonic-cfggen Sep 22, 2023
@Gerrit91 Gerrit91 merged commit 7abfe01 into master Sep 22, 2023
1 check passed
@Gerrit91 Gerrit91 deleted the refine-sonic-config-persistence branch September 22, 2023 09:26
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.

2 participants