Skip to content

[platform/broadcom]: Makefile changes for arista-fwutil#21600

Merged
rlhui merged 1 commit intosonic-net:masterfrom
liamkearney-msft:pub-dev/add-arista-fwutil
Feb 4, 2025
Merged

[platform/broadcom]: Makefile changes for arista-fwutil#21600
rlhui merged 1 commit intosonic-net:masterfrom
liamkearney-msft:pub-dev/add-arista-fwutil

Conversation

@liamkearney-msft
Copy link
Copy Markdown
Contributor

Description for the changelog

Add facilities to include arista fwutil into image if deb is in repo. Arista fwutil is proprietary and will be included in private repos

Why I did it

This utility is required for updating optics firmware on eos->sonic conversion for chassis

Work item tracking
  • Microsoft ADO (number only):

How I did it

Add makefile entries to copy and install the deb if it exists

How to verify it

Build image with the deb in platform/broadcom/extra-debs and see if it is in the image

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

  • [ x] 202405

Tested branch (Please provide the tested image version)

* Add facilities to include arista fwutil into image if deb is in repo

Signed-off-by: Liam Kearney <[email protected]>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Contributor

@prgeor prgeor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. @Staphylo can you review?

Copy link
Copy Markdown
Collaborator

@Staphylo Staphylo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to msft-202405: Azure/sonic-buildimage-msft#582

@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to msft-202503: Azure/sonic-buildimage-msft#1834

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.

6 participants