default_config_db.json for DPB system test #79
Conversation
| @@ -130,6 +130,7 @@ COPY ["files/configdb-load.sh", "/usr/bin/"] | |||
| COPY ["files/arp_update", "/usr/bin/"] | |||
| COPY ["files/buffers_config.j2", "files/qos_config.j2", "/usr/share/sonic/templates/"] | |||
| COPY ["files/sonic_version.yml", "/etc/sonic/"] | |||
| COPY ["default_config_db.json", "/etc/sonic/"] | |||
There was a problem hiding this comment.
@praveen-li @praveen-li Did we agree to change this file name to something more obvious for user? I think it would be better to change the name to avoid confusion.
There was a problem hiding this comment.
what about 'port_breakout_config_db.json'. I will change in Dev Code as well.
There was a problem hiding this comment.
+1 to "port_breakout_config_db.json"
There was a problem hiding this comment.
good, go ahead with this new name.
There was a problem hiding this comment.
VLAN_INTERFACE table and VLAN table does not have a port-related config. We can avoid them.
Keeping them is harmless. But it is better to keep only relevant config in this file. So that we can show this as an example to NetOps.
If possible, please address this.
| "scope": "global" | ||
| } | ||
| }, | ||
| "VLAN_INTERFACE": { |
There was a problem hiding this comment.
VLAN_INTERFACE table and VLAN table does not have a port-related config. We can avoid them.
Keeping them is harmless. But it is better to keep only relevant config in this file. So that we can show this as an example to NetOps.
…nic-net#5103) * src/sonic-platform-common be1cc24...f3f3573 (3): > Remove sonic-config-engine dependency from setup.py (sonic-net#109) > Migrate from sonic-daemon-base package to sonic-py-common package (sonic-net#103) > [Cables] Add support for 'Extended Specification Compliance' for QSFP cables (sonic-net#108) > [sfputilbase.py] Add application_advertisement NA field to transceiver_info_dict for platform api 1.0 (sonic-net#104) * src/sonic-platform-daemons 49d145c...8e0704e (1): > Proper fix for thermalctld using sonic-py-common (zhenggen-xu#80) > Fix thermalctld tests which were broken by the transition to sonic-py-common (zhenggen-xu#79) > Migrate from sonic-daemon-base package to sonic-py-common package (zhenggen-xu#74)
[sonic-linkmgrd][master] submodule updates e7e00f2 Jing Zhang Tue Jun 14 17:27:22 2022 -0700 Fix IP header checksum in handleSendSwitchCommand (zhenggen-xu#88) 127fd3d Jing Zhang Tue Jun 7 09:33:03 2022 -0700 Check self's mux mode before switching peer to standby & add support for `detach` mode (zhenggen-xu#79) sign-off: Jing Zhang zhangjing@microsoft.com
- What I did
Added default_config_db.json file and updated Docker file Jinja template to to copy the file to /etc/sonic directory of vs container
- How I did it
Manually created the default config file.
- How to verify it
By creating VS container and ensuring that default config file is copied to the correct directory in container.
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)