-
Notifications
You must be signed in to change notification settings - Fork 40
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
rearrange buildomat jobs; rewrite releng process in rust and aggressively parallelize #5744
Merged
Merged
Changes from 14 commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
ee79d02
wip
iliana d584262
bug fixes and performance improvements
iliana e948b3c
i did not know you could do this!
iliana eab1717
various logging tweaks
iliana 3b00902
hubris bits, stamping, job limits
iliana 5dcbb21
tuf repo!
iliana e6a0129
will it blend?
iliana 12d1cf0
move logfiles to end of output_rules
iliana 0219f87
attempt to fix deploy job
iliana 2ae7fc5
don't separately upload OS images (it's slow)
iliana 712aef1
[shakes fist at tokio]
iliana aa7506d
try to get to host-image faster
iliana 9bb1c17
correctness: put recovery artifacts in a new dir
iliana 6dbca3c
actually use the stamped artifacts
iliana f9186f9
delete caboose-util
iliana 35ec9e6
we no longer use tools/hubris_{checksums,version}
iliana 2e0916e
write releng.adoc
iliana 84be01c
oops
iliana c3a0537
num_cpus -> std::thread::available_parallelism
iliana b720b63
spawn job work onto tasks
iliana 8b3c76d
fix incorrect sha256
iliana 6b6fc8c
asciidoc :(
iliana 86003af
not sure why these fell out but sure
iliana 18ac798
explain the hubris structs
iliana fe6adf4
refactor out the `os_image_jobs!` macro
iliana eddff63
Merge remote-tracking branch 'origin/main' into iliana/releng
iliana e457190
job runner comments
iliana 61400a3
use the new-ish actually line tables only setting
iliana a2168a2
honor $GIT/$OMICRON_PACKAGE
iliana f491f2f
cancel safety in spawn_with_output
iliana bf93834
handle alternate target directories
iliana 3b6c54b
Merge remote-tracking branch 'origin/main' into iliana/releng
iliana 0453fd1
workspace dep cargo_metadata
iliana aef238c
replace the terrible std::mem::replace hack with more indirection
iliana b7664dd
explain $CARGO
iliana 9e341e8
respect $CARGO/$CARGO_HOME
iliana 1f7320d
Merge remote-tracking branch 'origin/main' into iliana/releng
iliana d5341bb
doc tweaks
iliana f3c1fa8
Merge remote-tracking branch 'origin/main' into iliana/releng
iliana File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Is this supposed to stay in sync with
rust-toolchain.toml
? Can we read from that file directly instead? To be honest I didn't know this existed.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.
Yes:
omicron/rust-toolchain.toml
Lines 1 to 7 in d2ed452
If it's not in sync it doesn't actually break anything, but the job runs a bit slower (Buildomat downloads 1.72.1 and then rustup's Cargo proxy sees you actually wanted 1.77.2 and downloads that).
Buildomat feature request: oxidecomputer/buildomat#55