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

platform/marvell renaming to platform/marvell-prestera #20860

Merged
merged 1 commit into from
Mar 12, 2025

Conversation

krismarvell
Copy link
Contributor

@krismarvell krismarvell commented Nov 19, 2024

Why I did it

Renaming of platforms and asic type for marvell
This PR has dependencies on the below submodule PRs

Repo PR title State
sonic-swss platform/marvell renaming to platform/marvell-prestera Merged
sonic-utilities platform/marvell renaming to platform/marvell-prestera Merged
sonic-sairedis platform/marvell renaming to platform/marvell-prestera Merged
sonic-linux-kernel Rename marvell to marvell-prestera Merged
sonic-mgmt Changes to support asic name change from marvell to marvell-prestera Merged
Work item tracking
  • Microsoft ADO (number only):

How I did it

renamed platform folder names and asic types as below
platform/marvell renamed to platform/marvell-prestera

How to verify it

Verify marvell-prestera builds going through, and verify the images in prestera switches and verified linkup of interfaces
verified below architecture of switches and build for marvell-prestera
arm64
armhf
amd64

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)

Description for the changelog

Link to config_db schema for YANG module changes

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

@lguohan
Copy link
Collaborator

lguohan commented Dec 16, 2024

@krismarvell , can you help to resolve the conflict?

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@krismarvell
Copy link
Contributor Author

/azpw ms_conflict

1 similar comment
@krismarvell
Copy link
Contributor Author

/azpw ms_conflict

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@krismarvell
Copy link
Contributor Author

@lguohan I have rebased to the latest after fixing the conflicts. Looks like the Elastictest failures are seen. Can you pls let me know if anything can be done from my side to fix these failures ?

@krismarvell
Copy link
Contributor Author

/azpw ms_conflict

2 similar comments
@saiarcot895
Copy link
Contributor

/azpw ms_conflict

@lguohan
Copy link
Collaborator

lguohan commented Dec 19, 2024

/azpw ms_conflict

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@paulmenzel
Copy link
Contributor

Why is the renaming done?

@bluecmd
Copy link
Contributor

bluecmd commented Jan 31, 2025

Why is the renaming done?

Likely because the Innovium Teralynx was purchased and subsequently renamed Marvell Teralynx. Thus it would be confusing to have one marvell and one marvell-teralynx. But I'm just guessing :-)

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@krismarvell
Copy link
Contributor Author

/azp run Azure.sonic-buildimage

Copy link

Commenter does not have sufficient privileges for PR 20860 in repo sonic-net/sonic-buildimage

@krismarvell
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca yxieca merged commit d93686e into sonic-net:master Mar 12, 2025
4 checks passed
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.

7 participants