Update unikernel with new build from builds.robur.coop

[PizieDust]

This PR is a continuation of the work started in #102 and is related to #54

In this PR, we :

• download the binary from builds.robur.coop,
• perform a manifest check to ensure that there's no changes in the bridge or block devices.
• check that a binary exists from the latest build before attempting to download it
• check if the vm should be compressed or not
• modify or update the args passed in the config to albatross
  if everything checks out, then we pass this binary to albatross to perform the update.

[reynir]

I think this looks fine. I have not tried to review the javascript changes, and I may be missing context.

[PizieDust]

@hannesm Please feel free to merge :)

[hannesm]

Thanks for your reminder, @PizieDust. I didn't look at this change at all. If you'd like to have this PR reviewed by me, it will need to wait some more time.

OTOH, Reynir reviewed it, so it should be good. I'd appreciate if we avoid a pin in config.ml and wait for the solo5-elftool 0.4 to be released (which, if I understand Reynir, will happen soon)!?

[PizieDust]

Thanks for your reminder, @PizieDust. I didn't look at this change at all. If you'd like to have this PR reviewed by me, it will need to wait some more time.
OTOH, Reynir reviewed it, so it should be good. I'd appreciate if we avoid a pin in config.ml and wait for the solo5-elftool 0.4 to be released (which, if I understand Reynir, will happen soon)!?

Yes we shouldn't merge with the pin. We'll wait for the release.

[reynir]

This will need more changes for ocaml-solo5-elftool then. I'll work on that and then merge (I didn't yet look at the new error message code). tirs. 4. feb. 2025 16.46 skrev Hannes Mehnert ***@***.***>:

…

***@***.**** commented on this pull request. ------------------------------ In albatross.ml <#105 (comment)>: > @@ -91,6 +95,98 @@ struct } | Error (`Msg err) -> Error err + let manifest_devices_match ~bridges ~block_devices binary = + let b = + Bigarray.Array1.create Bigarray.Int8_unsigned Bigarray.c_layout + (String.length binary) + in + for i = 0 to String.length binary - 1 do + b.{i} <- String.get_uint8 binary i + done; + let binary = b in can we name the binding b above just binary and then don't need this alias? — Reply to this email directly, view it on GitHub <#105 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AACVC324MDX7VDZ6GWX3XBT2ODOETAVCNFSM6AAAAABWCZG5HWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKOJTGI4DMMJQGY> . You are receiving this because you commented.Message ID: ***@***.***>

[reynir]

The code is now updated to ocaml-solo5-elftool.0.4.0 (pending release). With a temporary pin (now removed) it passes CI \o/

[PizieDust]

The code is now updated to ocaml-solo5-elftool.0.4.0 (pending release). With a temporary pin (now removed) it passes CI \o/

Awesome.
Thank you Reynir

[hannesm]

This is fine. I don't quite understand why it triggers the opam-monorepo failure, though.

I'll merge to clean up the queue, and will propose the step for albatross 2.4.0