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

[201911][component]: Introduce new firmware management API #87

Merged
merged 1 commit into from
May 22, 2020

Conversation

nazariig
Copy link
Collaborator

@nazariig nazariig commented May 21, 2020

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

Summary:
The purpose of this PR is to propagate some new component firmware management APIs which are intended to make the process more versatile. These new API's will be used mostly to extend fwutil functionality in order to support new platform components (e.g, SSD).

Motivation:

  1. Inform user in advance about required component firmware update actions when operating in manual mode
  2. Allow fwutil to read component firmware version directly from image instead of configuration file if not provided

@jleveque
Copy link
Contributor

@nazariig: Should this be cherry-picked into 201911, also?

@nazariig
Copy link
Collaborator Author

@nazariig: Should this be cherry-picked into 201911, also?

@jleveque This PR is for 201911. For master it is already merged.
@abdosi asked to open the same PR to 201911.

@jleveque
Copy link
Contributor

@nazariig: Thanks. Somehow I swore I saw "master" as the target. :P

@jleveque jleveque changed the title [component]: Introduce new firmware management API [201911][component]: Introduce new firmware management API May 22, 2020
@jleveque jleveque merged commit ee60f54 into sonic-net:201911 May 22, 2020
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-platform-common that referenced this pull request Oct 25, 2024
…onic-net#87)

* Updated based on sonic-swss-common changes to get the namespace (PR#378)
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.

3 participants