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

[GCU] RDMA Platform Validator: Add Mellanox-SN4700-O8C48 HwSKU #2951

Merged
merged 1 commit into from
Aug 30, 2023

Conversation

isabelmsft
Copy link
Contributor

What I did

Add Mellanox-SN4700-O8C48 HwSKU to GCU RDMA platform validator conf file

How I did it

How to verify it

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

@isabelmsft isabelmsft marked this pull request as ready for review August 22, 2023 20:49
@qiluo-msft qiluo-msft merged commit d8a04d8 into sonic-net:master Aug 30, 2023
4 checks passed
@StormLiangMS
Copy link
Contributor

@neethajohn do we have ADO to support cherry-pick?

@StormLiangMS
Copy link
Contributor

@isabelmsft do we have ADO for this?

@isabelmsft
Copy link
Contributor Author

@isabelmsft do we have ADO for this?

ADO 25281868

StormLiangMS pushed a commit that referenced this pull request Oct 7, 2023
#### What I did
Add Mellanox-SN4700-O8C48 HwSKU to GCU RDMA platform validator conf file
isabelmsft added a commit to isabelmsft/sonic-utilities that referenced this pull request Nov 22, 2023
#### What I did
Add Mellanox-SN4700-O8C48 HwSKU to GCU RDMA platform validator conf file
yxieca pushed a commit that referenced this pull request Nov 29, 2023
Cherry-pick PR #2692 (set up test infrastructure used in later PR 2857), #2857 (platform validator PR), #2913 (edge case fix), #2951, #3018 (add support for new Mellanox HwSKU in conf file)
This change stops GCU from modifying a protected RDMA field.

Signed-off-by: Stephen Sun <[email protected]>
Co-authored-by: Stephen Sun <[email protected]>
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.

4 participants