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

Multipart HTTP update support #341

Open
ofaurax opened this issue Aug 7, 2023 · 7 comments
Open

Multipart HTTP update support #341

ofaurax opened this issue Aug 7, 2023 · 7 comments

Comments

@ofaurax
Copy link
Collaborator

ofaurax commented Aug 7, 2023

When trying to update a firmware with bmclib (using install-firmware example):

$ ./main --host REDACTED --user root --password REDACTED --component bios --firmware update.tar.gz
FATA[0007] 3 errors occurred:
	* provider: gofish: Multipart HTTP push updates not supported: redfish update service error
	* Multipart HTTP push updates not supported: redfish update service error
	* failure in FirmwareInstall
@ofaurax ofaurax changed the title Multipart HTTP support Basic update support Aug 8, 2023
@ofaurax
Copy link
Collaborator Author

ofaurax commented Aug 8, 2023

It seems only multipart update is supported in bmclib. OpenBMC only supports basic update, as of now.

@ofaurax ofaurax changed the title Basic update support Multipart HTTP update support Aug 8, 2023
@ofaurax
Copy link
Collaborator Author

ofaurax commented Aug 8, 2023

OpenBMC only supports basic update, as of now.

In fact, OpenBMC implements "Unstructured HTTP push update", which is deprecated, not "basic update".

@davidallendj
Copy link

I'm having this issue as well. Did you ever find a solution or try rolling your own?

@ofaurax
Copy link
Collaborator Author

ofaurax commented Oct 5, 2023

We implemented "Unstructured HTTP push update" in bmclib so that OpenBMC can be updated

@ofaurax
Copy link
Collaborator Author

ofaurax commented Oct 16, 2023

Also, support for multipart update has been merged in openbmc:
https://gerrit.openbmc.org/c/openbmc/bmcweb/+/32174

@davidallendj
Copy link

Awesome! That means this should now work with bmclib if using the latest openbmc since bmclib already supports this, correct?

@ofaurax
Copy link
Collaborator Author

ofaurax commented Oct 18, 2023

Awesome! That means this should now work with bmclib if using the latest openbmc since bmclib already supports this, correct?

Yes, and if both are supported, bmclib should use multipart update. Please give feedback if you test it

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

No branches or pull requests

2 participants