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

[Mellanox] Update C256X1 SKU to be C256S1 in gcu_field.json (#3611) #3615

Closed

Conversation

noaOrMlnx
Copy link
Collaborator

Update Mellanox-SN5600-C256X1 to be Mellanox-SN5600-C256S1 in gcu_field_operation_validators.conf.json

What I did

cherry pick to #3611
Update Mellanox-SN5600-C256X1 to be Mellanox-SN5600-C256S1 in gcu_field_operation_validators.conf.json

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)

…t#3611)

Update Mellanox-SN5600-C256X1 to be Mellanox-SN5600-C256S1 in gcu_field_operation_validators.conf.json
@liat-grozovik
Copy link
Collaborator

@bingwang-ms please help to merge

@bingwang-ms
Copy link
Contributor

@noaOrMlnx Please handle conflict if the code is needed for 202405.

@noaOrMlnx
Copy link
Collaborator Author

@bingwang-ms fixed

@bingwang-ms
Copy link
Contributor

No file is changed. Probably the change has been merged by another PR. Please double check

@noaOrMlnx
Copy link
Collaborator Author

@bingwang-ms that's because the first PR you merged contained the fix already - #3616

then this PR can be closed

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.

3 participants