[sled-agent] Integrate self-assembling Propolis#3456
Merged
Conversation
smklein
added a commit
to oxidecomputer/propolis
that referenced
this pull request
Jul 3, 2023
Uses a method script in the Propolis zone to make it self-assembling. This should, along with a corresponding Omicron change, avoid the need to `zlogin` to the Propolis zone at any point. Related to: oxidecomputer/omicron#3456
davepacheco
approved these changes
Jul 3, 2023
davepacheco
left a comment
Collaborator
There was a problem hiding this comment.
Love the SMF profile approach over zlogin!
This was referenced Jul 3, 2023
jmpesp
added a commit
that referenced
this pull request
Jul 6, 2023
Bump crucible rev to pick up: - fix build break caused by merging #801 - Issue extent flushes in parallel - Add Logger Option to Volume construct method - Update Rust crate libc to 0.2.147 - Update Rust crate percent-encoding to 2.3 - Retry jobs until they succeed - Reorder select arms so pings can't be starved out - Treat a skipped IO like an error IO for ACK results. - Retry pantry requests - Remove panics and asserts in dummy tests - Update Rust crate csv to 1.2.2 - Update Rust crate reedline to 0.21.0 - Set open file resource limit to the max - Update Rust crate ringbuffer to 0.14 - DTrace meet cmon - Widen assert values to u128 to deal with u64::MAX - Change size_to_validate from usize to u64 - Turn on live-repair test in CI - Increase flush_timeout for some tests, collect cores - Update to latest dropshot Bump propolis rev to pick up: - Bump crucible rev to latest - Make the propolis zone self-assembling - Flesh out more PIIX3-PM to suppress log gripes - Bump crucible rev to latest - Restructure PM-timer under PIIX3 device - Fix inventory handling for nested child entities - Centralize vmm-data interface into bhyve_api - Clean up PCI device classes - Update openssl dep to 0.10.55 - Allow propolis-standalone to use VMM reservoir - only allow one request to reboot to be enqueued at a time Note that #3456 bumped package-manifest, but this commit bumps both that and Cargo.toml. All Propolis commits between 04a27573 and 25111a88 are listed above.
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Integrates oxidecomputer/propolis#454
This avoid using
zloginto modify the propolis zone after launch, instead preferring to write a profile file to the zone ahead-of-time. This matches the pattern for other self-assembling zones, where configuration information is supplied ahead-of-time through theProfileBuilder, which places an auxiliary SMF manifest into/var/svc/profile/site.smlto be imported when the zone starts.Relies on: oxidecomputer/propolis#454
Part of the fix for #3454