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

modified files relevant to SKU Mellanox-SN3800-D112C8 #4810

Merged
merged 3 commits into from
Jun 21, 2020

Conversation

madhanmellanox
Copy link
Contributor

  • What I did
    Modified all the relevant files to modify SKU Mellanox-SN3800-D112C8

  • How I did it
    Modified / Created all the SKU related files using a XML file provided and a sonic_sku_create.py script to generate the files.

  • How to verify it
    Load the new HWKU related config using follow command and reload the config.
    sudo sonic-cfggen --preset l2 -p -H -k Mellanox-SN3800-D112C8 > /tmp/newconfig
    sudo cp /tmp/newconfig /etc/sonic/config_db.json
    sudo config reload
    show interfaces status

  • Description for the changelog
    Modified SKU related files buffers_defaults_t0.j2, buffers_defaults_t1.j2, buffers.json.j2, pg_profile_lookup.ini, port_config.ini, qos.json.j2, sai_3800.xml and sai.profile.

- A picture of a cute animal (not mandatory but encouraged)

@@ -1 +1 @@
SAI_INIT_CONFIG_FILE=/usr/share/sonic/hwsku/sai_3800_112x50g_8x100g.xml
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kindly keep the name as before as that's a convention followed for other SKU's as well. You can check this .
It can be used to identify the file in syncd docker

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

addressed in the next commit.

@prsunny
Copy link
Contributor

prsunny commented Jun 19, 2020

retest vsimage please

@prsunny
Copy link
Contributor

prsunny commented Jun 19, 2020

@sumukhatv for visibility

@madhanmellanox
Copy link
Contributor Author

retest vsimage please

@prsunny prsunny merged commit b5d0bad into sonic-net:master Jun 21, 2020
abdosi pushed a commit that referenced this pull request Jun 23, 2020
* modified files relevant to SKU Mellanox-SN3800-D112C8

Co-authored-by: Madhan Babu <madhan@arc-build-server.mtr.labs.mlnx>
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