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

Add HubrisArchiveBuilder to create hubris archives from raw images or thin air #19

Merged
merged 3 commits into from
Aug 17, 2023

Conversation

jgallagher
Copy link
Contributor

My immediate goal with this is to be able to generate fake hubris archives for integration tests in omicron, and as of this PR HubrisArchiveBuilder is probably only useful for that. I tried not to close the door on it potentially being useful to generate a real hubris archive from a standalone image, but it will need some additions for that to be useful (e.g., adding an app.toml for humility).

Copy link
Contributor

@labbott labbott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I think this will be a good starting point for future work.

@jgallagher jgallagher merged commit 0c642f6 into main Aug 17, 2023
@jgallagher jgallagher deleted the archive-builder branch August 17, 2023 15:29
jgallagher added a commit to oxidecomputer/omicron that referenced this pull request Aug 17, 2023
…t hardware revisions) (#3887)

This PR also teaches tufaceous-lib how to generate fake hubris artifacts
that look like real hubris archives (at least up to having a valid
caboose), via the not-yet-merged `HubrisArchiveBuilder` from `hubtools`.

There are a couple things I need to do before merging this PR:

- [x] Update `hubtools` in Cargo.toml once
oxidecomputer/hubtools#19 lands
- [x] Test on madrid with a manually-built repo with multiple SP images

but I think it's close enough it can be reviewed.

Fixes #3394.
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.

3 participants