Skip to content

helios-rot: make the trait / interface async - #430

Open
flihp wants to merge 2 commits into
oxidecomputer:mainfrom
flihp:helios-rot-async
Open

helios-rot: make the trait / interface async#430
flihp wants to merge 2 commits into
oxidecomputer:mainfrom
flihp:helios-rot-async

Conversation

@flihp

@flihp flihp commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

This is required to integrate an impl of HeliosRot into verifier-cli, and will be required for any future integration w/ sled-agent

@flihp
flihp requested a review from papertigers September 9, 2026 17:29
@flihp

flihp commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator Author

CI failures don't look like they're caused by anything done in this PR: https://github.com/oxidecomputer/dice-util/actions/runs/34383199452/job/102572857725?pr=430#step:4:34. The internet says this is caused by a "network glitch" which I take to mean: just try again and hope github has sorted itself out ¯\_(ツ)_/¯

@flihp

flihp commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator Author

this is another instance of microsoft/linux-package-repositories#34 not actually being fixed

@flihp
flihp marked this pull request as draft September 9, 2026 17:51
@flihp

flihp commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator Author

flipped this back to Draft while I poke at this

@flihp

flihp commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator Author

not related to microsoft/linux-package-repositories#34, instead it looks like actions/runner-images#7048 and / or actions/runner-images#12949. The summary from Microsoft in these issues is that the Ubuntu mirrors that they host in azure are unstable and shouldn't be used. They give instructions for disabling them in the linked issues which I'll give a shot here.

@flihp
flihp force-pushed the helios-rot-async branch 3 times, most recently from 2a6c920 to 4d1aede Compare September 9, 2026 18:27
@flihp
flihp marked this pull request as ready for review September 9, 2026 18:50
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

Successfully merging this pull request may close these issues.

1 participant