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] [PPI] Enable global port late create for SN5600 #15866

Merged
merged 1 commit into from
Aug 28, 2023

Conversation

nazariig
Copy link
Collaborator

@nazariig nazariig commented Jul 17, 2023

Signed-off-by: Nazarii Hnydyn [email protected]

DEPENDS:

  1. [ppi]: Implement port bulk comparison logic sonic-swss#2564

HLD: sonic-net/SONiC#1084

Why I did it

  • Enabled port late create on SN5600 Spectrum-4 switch boots up with no ports
Work item tracking
  • N/A

How I did it

  • Updated SAI xml config file

How to verify it

Run sonic-mgmt tests of fastboot

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

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

Tested branch (Please provide the tested image version)

  • 202211
  • master

Description for the changelog

  • N/A

Link to config_db schema for YANG module changes

  • N/A

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

      .---.        .-----------
     /     \  __  /    ------
    / /     \(  )/    -----
   //////   ' \/ `   ---
  //// / // :    : ---
 // /   /  /`    '--
//          //..\\
       ====UU====UU====
           '//||\\`
             ''``

@xumia
Copy link
Collaborator

xumia commented Jul 18, 2023

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

No commit pushedDate could be found for PR 15866 in repo sonic-net/sonic-buildimage

@nazariig
Copy link
Collaborator Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines 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.

@nazariig
Copy link
Collaborator Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@nazariig nazariig marked this pull request as ready for review July 24, 2023 21:14
@nazariig nazariig changed the title [PPI]: Enable global port late create for SPC-4 [PPI][Mellanox]: Enable global port late create for SPC-4 Aug 14, 2023
@liat-grozovik liat-grozovik changed the title [PPI][Mellanox]: Enable global port late create for SPC-4 [Mellanox] [PPI] Enable global port late create for SN5600 Aug 28, 2023
@liat-grozovik liat-grozovik merged commit 65b0011 into sonic-net:master Aug 28, 2023
19 checks passed
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Sep 3, 2023
…#15866)

- Why I did it
Enabled port late create on SN5600 Spectrum-4 switch boots up with no ports

Work item tracking
N/A

- How I did it
Updated SAI xml config file

- How to verify it
Run sonic-mgmt tests of fastboot

Signed-off-by: Nazarii Hnydyn <[email protected]>
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202305: #16402

mssonicbld pushed a commit that referenced this pull request Sep 3, 2023
- Why I did it
Enabled port late create on SN5600 Spectrum-4 switch boots up with no ports

Work item tracking
N/A

- How I did it
Updated SAI xml config file

- How to verify it
Run sonic-mgmt tests of fastboot

Signed-off-by: Nazarii Hnydyn <[email protected]>
sonic-otn pushed a commit to sonic-otn/sonic-buildimage that referenced this pull request Sep 20, 2023
…#15866)

- Why I did it
Enabled port late create on SN5600 Spectrum-4 switch boots up with no ports

Work item tracking
N/A

- How I did it
Updated SAI xml config file

- How to verify it
Run sonic-mgmt tests of fastboot

Signed-off-by: Nazarii Hnydyn <[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.

5 participants