Skip to content

[Mellanox] Add support for Service Port in SN5600#11609

Merged
liat-grozovik merged 1 commit intosonic-net:masterfrom
DavidZagury:master_service_port
Aug 15, 2022
Merged

[Mellanox] Add support for Service Port in SN5600#11609
liat-grozovik merged 1 commit intosonic-net:masterfrom
DavidZagury:master_service_port

Conversation

@DavidZagury
Copy link
Contributor

@DavidZagury DavidZagury commented Aug 2, 2022

Why I did it

SN5600 has an additional service interface with a different parameters than other interfaces.

How I did it

Added the etp65 interface with the correct parameters.

How to verify it

Run platform test on SN5600 platform.
Check the service port can startup correctly.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205

Description for the changelog

Link to config_db schema for YANG module changes

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

@DavidZagury DavidZagury marked this pull request as draft August 7, 2022 14:58
@DavidZagury DavidZagury marked this pull request as ready for review August 7, 2022 14:59
@liat-grozovik liat-grozovik changed the title [NVIDIA] Add support for Service Port in SN5600 [Mellanox] [Sn5600] Add support for Service Port Aug 8, 2022
@liat-grozovik liat-grozovik changed the title [Mellanox] [Sn5600] Add support for Service Port [Mellanox] [SN5600] Add support for Service Port Aug 8, 2022
@liat-grozovik
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@sonic-net sonic-net deleted a comment from azure-pipelines bot Aug 8, 2022
@liat-grozovik
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik
Copy link
Collaborator

/easycla

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 10, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

@liat-grozovik liat-grozovik merged commit 027bc7f into sonic-net:master Aug 15, 2022
@liat-grozovik liat-grozovik changed the title [Mellanox] [SN5600] Add support for Service Port [Mellanox] Add support for Service Port in SN5600 Aug 15, 2022
yxieca pushed a commit that referenced this pull request Aug 17, 2022
- Why I did it
SN5600 has an additional service interface with a different parameters than other interfaces.

- How I did it
Added the etp65 interface with the correct parameters.

- How to verify it
Run platform test on SN5600 platform.
Check the service port can startup correctly.
@DavidZagury DavidZagury deleted the master_service_port branch August 23, 2022 14:51
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.

5 participants