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

[Marvell] platform renaming for innovium #1408

Merged
merged 1 commit into from
Nov 21, 2024

Conversation

krismarvell
Copy link
Contributor

@krismarvell krismarvell commented Aug 6, 2024

What i did

=======
this is one of the dependent set of PRs being raised for changing the innovium asic type to marvell-teralynx

How i did

=======
syncd based logging enablement inside syncd docker and its related asic type checks are modified as per the renaming

How to verify

==========
Verfify syncd docker configs are applied properly in teralynx platform

@krismarvell
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@krismarvell
Copy link
Contributor Author

@kcudnik Can you pls help review ?

saiarcot895
saiarcot895 previously approved these changes Sep 25, 2024
kcudnik
kcudnik previously approved these changes Oct 25, 2024
kcudnik
kcudnik previously approved these changes Nov 18, 2024
@lguohan
Copy link
Contributor

lguohan commented Nov 18, 2024

@saiarcot895 , are you okay with this one?

@saiarcot895
Copy link
Contributor

@krismarvell can you do another merge?

@krismarvell
Copy link
Contributor Author

@krismarvell can you do another merge?

Just done the rebase now

@saiarcot895 saiarcot895 merged commit d772864 into sonic-net:master Nov 21, 2024
18 checks passed
shiraez pushed a commit to Marvell-switching/sonic-sairedis that referenced this pull request Dec 12, 2024
shiraez pushed a commit to Marvell-switching/sonic-sairedis that referenced this pull request Dec 12, 2024
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.

5 participants