Skip to content

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

Merged
qiluo-msft merged 1 commit intosonic-net:masterfrom
isabelmsft:4700_spc3
Aug 30, 2023
Merged

[GCU] RDMA Platform Validator: Add Mellanox-SN4700-O8C48 HwSKU#2951
qiluo-msft merged 1 commit intosonic-net:masterfrom
isabelmsft:4700_spc3

Conversation

@isabelmsft
Copy link
Copy Markdown
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
Comment thread generic_config_updater/gcu_field_operation_validators.conf.json
@qiluo-msft qiluo-msft merged commit d8a04d8 into sonic-net:master Aug 30, 2023
@StormLiangMS
Copy link
Copy Markdown
Contributor

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

@StormLiangMS
Copy link
Copy Markdown
Contributor

@isabelmsft do we have ADO for this?

@isabelmsft
Copy link
Copy Markdown
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