[docker-orchagent][201911] Pass ASIC vendor information to swss docker as docker level environment variable#8274
Conversation
…nt variable Motivation: Recently, the reserved buffer of admin-down ports is going to be reclaimed. However, the way to do this differs among vendors. We need to find a way to pass vendor information to swss docker. Implementation: Fetch the ASIC vendor information when the docker is created and pass it to docker as ASIC_VENDOR Signed-off-by: Stephen Sun <[email protected]>
stephenxs
changed the title
|
|
||
| {%- if docker_container_name == "swss" %} | ||
| # Obtain the vendor name | ||
| ASIC_VENDOR=`$SONIC_CFGGEN -y /etc/sonic/sonic_version.yml -v asic_type` |
There was a problem hiding this comment.
Use the 'sonic_asic_platform' var instead of defining "ASIC_VENDOR"
There was a problem hiding this comment.
I agree with @neethajohn 's suggestion and it should go to master, 202012 and later branches. So please fix on master first, and then cherry-pick. If cherry-pick does not work, we can continue review here after master commit merged.
…VENDOR Signed-off-by: Stephen Sun <[email protected]>
Thanks, Qi. |
|
what is the motivation to move expose asic vendor to swss docker? |
Related to the comments in sonic-net/sonic-swss#1837 |
5 participants
Why I did it
Recently, the reserved buffer of admin-down ports is going to be reclaimed.
However, the way to do this differs among vendors.
We need to find a way to pass vendor information to swss docker.
Signed-off-by: Stephen Sun [email protected]
How I did it
Fetch the ASIC vendor information when the docker is created and pass it to the docker as environment variable
ASIC_VENDOR.How to verify it
Which release branch to backport (provide reason below if selected)
Description for the changelog
A picture of a cute animal (not mandatory but encouraged)