sonic-swss-common
sonic-swss-common copied to clipboard
Add ProfileProvider class to support read profile config from PROFILE_DB.
Why I did it
Add default value and profile support to swss-swss-common.
How I did it
Add ProfileProvider class to support read profile config from PROFILE_DB.
How to verify it
Add new UT. Pass all existing UT and E2E test.
Which release branch to backport (provide reason below if selected)
- [ ] 201811
- [ ] 201911
- [ ] 202006
- [ ] 202012
- [ ] 202106
- [ ] 202111
Description for the changelog
Add ProfileProvider class to support read profile config from PROFILE_DB.
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)
This PR depends on another PR to pass UT: https://github.com/sonic-net/sonic-swss-common/pull/681 The reason is UT need modify redis config to create a new UT, which is part of azure pipeline change.
"LOGLEVEL_DB" : {
There is a ongoing PR to remove this DB. Possible to use it?
In reply to: 1260486336
In reply to: 1260486336
Refers to: tests/redis_multi_db_ut_config/database_config1.json:25 in 9b60524. [](commit_id = 9b60524cc312df68a6feed4fb3e0fec60d63d1fb, deletion_comment = False)
"LOGLEVEL_DB" : {
There is a ongoing PR to remove this DB. Possible to use it?
Refers to: tests/redis_multi_db_ut_config/database_config1.json:25 in 9b60524. [](commit_id = 9b60524, deletion_comment = False)
I check the loglevel PR, seems that PR only move logger table from LOGLEVEL_DB, seems still something left in that DB. So, before LOGLEVEL_DB been complete clean-up in database_config.json and schema.h and table.cpp, we can't use ID for that DB.
/azp run
Azure Pipelines successfully started running 1 pipeline(s).
Will merge after this validation pass: https://github.com/sonic-net/sonic-buildimage/pull/10575
Will merge this PR after the validation PR passed: https://github.com/sonic-net/sonic-buildimage/pull/10575
Could you check build failures?
Could you check build failures? The build failure caused by code change in sairedis, following PR will fix it: https://github.com/sonic-net/sonic-swss-common/pull/747
/azp run
Azure Pipelines successfully started running 1 pipeline(s).