Hua Liu
Hua Liu
> hi can you explain a bit more about the motivation of this PR? thanks This still a POC PR, not ready for review, description updated.
Compare of current code and original code: Current code: if args.namespace is not None: SonicDBConfig.initializeGlobalConfig() else: SonicDBConfig.initialize() Original code: if is_multi_asic(): if not swsscommon.SonicDBConfig.isGlobalInit(): SonicDBConfig.initializeGlobalConfig()
Created fix PR: https://github.com/sonic-net/sonic-utilities/pull/3257
@mlok-nokia , according to Judy's comments in this PR: https://github.com/sonic-net/sonic-utilities/pull/3257 https://github.com/sonic-net/sonic-utilities/pull/3257#discussion_r1550740054: I feel it is more like a timing issue as you pointed out, Can we call db_migrator, after /var/run/redis0/sonic-db/database-config.json...
@nmoray , please fix coverage issue by add new test case: @azure-pipelines-wrapper coverage.Azure.sonic-utilities.Python3 — Pull Request Coverage Total: 44 lines **Missing: 39 lines** Coverage: 11% Threshold: 80%
> > @nmoray , please fix coverage issue by add new test case: > > @azure-pipelines-wrapper coverage.Azure.sonic-utilities.Python3 — Pull Request Coverage > > Total: 44 lines **Missing: 39 lines** Coverage:...
This PR is for manually cherry-pick https://github.com/sonic-net/sonic-swss-common/pull/544
> I was just wondering whether it's possible to introduce a way to "shadow" the fields that existed in `STATIC_CONFIG_DB` but are removed from `CONFIG_DB` by the user. Maybe we...