Skip to content

[sled-agent] Integrate self-assembling Propolis#3456

Merged
smklein merged 3 commits into
mainfrom
self-assembling-propolis
Jul 3, 2023
Merged

[sled-agent] Integrate self-assembling Propolis#3456
smklein merged 3 commits into
mainfrom
self-assembling-propolis

Conversation

@smklein

@smklein smklein commented Jun 29, 2023

Copy link
Copy Markdown
Collaborator

Integrates oxidecomputer/propolis#454

This avoid using zlogin to 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 the ProfileBuilder, which places an auxiliary SMF manifest into /var/svc/profile/site.sml to be imported when the zone starts.

Relies on: oxidecomputer/propolis#454
Part of the fix for #3454

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 davepacheco left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Love the SMF profile approach over zlogin!

@smklein smklein merged commit 2d46f8c into main Jul 3, 2023
@smklein smklein deleted the self-assembling-propolis branch July 3, 2023 17:15
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.
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.

2 participants