-
Notifications
You must be signed in to change notification settings - Fork 12
BoB L1: new tdx-init, LH 8.0.0 #43
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
base: main
Are you sure you want to change the base?
Changes from all commits
2f22972
14fcc2e
b21e753
a6dd9b5
ef5dd27
2cf8e72
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,5 @@ | ||
| [Build] | ||
| SandboxTrees=bob-l1/mkosi.sandbox | ||
| Environment=LIGHTHOUSE_BINARY KERNEL_CONFIG_SNIPPETS=bob-l1/kernel.config KERNEL_VERSION=6.13.12 | ||
| WithNetwork=true | ||
|
|
||
|
|
@@ -12,7 +13,8 @@ Packages=openntpd | |
| BuildPackages=build-essential | ||
| git | ||
| gcc | ||
| cargo | ||
| cargo/trixie-backports | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nice, i will look for other places we're using rustup in the repo to get newer version of rust and switch them out for this technique now that the backports version is new enough |
||
| rustc/trixie-backports | ||
| zlib1g-dev | ||
| libzstd-dev | ||
| libleveldb-dev | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| Types: deb deb-src | ||
| URIs: https://snapshot.debian.org/archive/debian/20251113T083151Z | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm slightly leaning towards a mkosi.* script so that we don't have to hard code the timestamp twice. What do you think? We could also just make this part of the base module. Everything should have backports, especially since it can help us avoid using rustup throughout the repo since it nontrivially slows down builds, depends on network access, and downloads untrusted binaries.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Unfortunately, adding backports repo via Also, we can't add it via scripts (e.g. systemd/mkosi#3439), because build packages installation happens in sandbox, not image itself. |
||
| Suites: trixie-backports | ||
| Components: main | ||
| Enabled: yes | ||
| Signed-By: /usr/share/keyrings/debian-archive-keyring.gpg | ||
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.
The lighthouse build process can be simplified starting with v8 like so: https://github.com/flashbots/flashbots-images/pull/38/files#diff-3c7f2c5d727d89202366ab0b86107555d64c34fed340037e8204c3f8959d80e7R28-R38
I'll make the necessary changes to this PR to get it working
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.
Thanks!