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

[sonic-cfggen] Loading the configuration from init_cfg.json and then from config_db.json #4148

Merged
merged 4 commits into from
Mar 5, 2020
Merged

[sonic-cfggen] Loading the configuration from init_cfg.json and then from config_db.json #4148

merged 4 commits into from
Mar 5, 2020

Conversation

yozhao101
Copy link
Contributor

- What I did
I changed the loading orders of sonic-cfggen such that it will first read configuration
from init_cfg.json which is by default configuration file and then config_db.json into Config_DB.

- How I did it

- How to verify it

loading the init_cfg.json and then loading config_db.json.

Signed-off-by: Yong Zhao <yozhao@microsoft.com>
/etc/sonic/init_cfg.json.

Signed-off-by: Yong Zhao <yozhao@microsoft.com>
files/build_templates/docker_image_ctl.j2 Outdated Show resolved Hide resolved
files/scripts/configdb-load.sh Outdated Show resolved Hide resolved
and init_cfg.json are existing. Otherwise sonic-cfggen will only load
one of them if it does exist.

Signed-off-by: Yong Zhao <yozhao@microsoft.com>
files/build_templates/docker_image_ctl.j2 Outdated Show resolved Hide resolved
files/scripts/configdb-load.sh Outdated Show resolved Hide resolved
…ig_db.json exit, we

will load them. Otherwise we only load config_db.json.

Signed-off-by: Yong Zhao <yozhao@microsoft.com>
@jleveque
Copy link
Contributor

jleveque commented Mar 4, 2020

Retest vsimage please

@jleveque jleveque merged commit 7c8da20 into sonic-net:master Mar 5, 2020
abdosi pushed a commit that referenced this pull request Mar 15, 2020
tiantianlv pushed a commit to SONIC-DEV/sonic-buildimage that referenced this pull request Apr 24, 2020
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.

3 participants