-
Notifications
You must be signed in to change notification settings - Fork 114
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
Firmware Interface proposal #364
Conversation
109e71b
to
7206c85
Compare
/retitle Firmware Interface proposal |
7206c85
to
4bdfa84
Compare
986081e
to
8cd2a4b
Compare
@elfosardo @dtantsur if you think I need to update more things here, let me know. |
8cd2a4b
to
5bd082f
Compare
- Verify that updates can be executed on two type of hardware that supports | ||
Redfish. | ||
- Verify that the information about the desired version is available after | ||
cleaning. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BMO has new e2e tests (check the commit history). That allows us to test firmware upgrades against sushy-tools. WDYT?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hummm that could be a thing, but sushy-tools currently doesn't have the capability to execute firmware updates.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mquhuy @mboukhalfa This might be interesting feature for you to implement in susy tools or to anyone in our team @kashifest
/approve I think we're at the point where the details can be polished in the code (but I'd still appreciate my comments addressed). |
Signed-off-by: Iury Gregory Melo Ferreira <imelofer@redhat.com>
5bd082f
to
9da24dc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
I like this feature very much, from my POV it is a go!
- Verify that updates can be executed on two type of hardware that supports | ||
Redfish. | ||
- Verify that the information about the desired version is available after | ||
cleaning. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mquhuy @mboukhalfa This might be interesting feature for you to implement in susy tools or to anyone in our team @kashifest
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dtantsur, Rozzii The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Add a new CRD for the HostFirmwareComponents described in the Firmware Interface proposal - metal3-io/metal3-docs#364.
Add a new CRD for the HostFirmwareComponents described in the Firmware Interface proposal - metal3-io/metal3-docs#364. Signed-off-by: Iury Gregory Melo Ferreira <imelofer@redhat.com>
Add a new CRD for the HostFirmwareComponents described in the Firmware Interface proposal - metal3-io/metal3-docs#364. Signed-off-by: Iury Gregory Melo Ferreira <imelofer@redhat.com>
Add a new CRD for the HostFirmwareComponents described in the Firmware Interface proposal - metal3-io/metal3-docs#364. Signed-off-by: Iury Gregory Melo Ferreira <imelofer@redhat.com>
Add a new CRD for the HostFirmwareComponents described in the Firmware Interface proposal - metal3-io/metal3-docs#364. Signed-off-by: Iury Gregory Melo Ferreira <imelofer@redhat.com>
Add a new CRD for the HostFirmwareComponents described in the Firmware Interface proposal - metal3-io/metal3-docs#364. Signed-off-by: Iury Gregory Melo Ferreira <imelofer@redhat.com>
No description provided.