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

Update Nvidia sai.profile SKU files to use common file #25

Closed
wants to merge 2 commits into from

Conversation

noaOrMlnx
Copy link
Owner

Add new common sai.profile,
Update Nvidia/Mellanox sai.profile of all SKUs to remove common parameters.

Why I did it

To have the ability to add common parameters to only 1 file instead of all SKUs.

@noaOrMlnx noaOrMlnx changed the title Update Nvidia sai.profile SKU files to have common file Update Nvidia sai.profile SKU files to use common file Jan 10, 2024
@@ -0,0 +1,3 @@
SAI_DUMP_STORE_PATH=/var/log/mellanox/sdk-dumps
SAI_DUMP_STORE_AMOUNT=10
SAI_DUMP_MFT_CFG_PATH=/etc/mft/fwtrace_cfg

Choose a reason for hiding this comment

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

@noaOrMlnx do we need the same for syncd RPC docker? What about VS docker?

@noaOrMlnx noaOrMlnx force-pushed the sai_profile_enhancements branch 2 times, most recently from b6cea3b to a6806af Compare February 6, 2024 13:22
@noaOrMlnx noaOrMlnx closed this Feb 26, 2024
noaOrMlnx pushed a commit that referenced this pull request Sep 30, 2024
…tically (sonic-net#19872)

#### Why I did it
src/sonic-dash-api
```
* 39e4a63 - (HEAD -> master, origin/master, origin/HEAD) Add privatelink underlay_sip field for route table (#25) (24 hours ago) [Lawrence Lee]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants