[sled-agent] Decouple RSS from Bootstrap Agent, run in distinct task #876
Conversation
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
Collaborator
Author
|
Not exactly #820 - it's not a distinct service - but close enough for the moment. |
bnaecker
reviewed
Apr 5, 2022
bnaecker
reviewed
Apr 5, 2022
bnaecker
reviewed
Apr 5, 2022
bnaecker
approved these changes
Apr 5, 2022
Collaborator
bnaecker
left a comment
bnaecker
left a comment
There was a problem hiding this comment.
A couple of clarifying questions, but the PR itself looks good, thanks!
leftwo
pushed a commit
that referenced
this pull request
Aug 24, 2023
Updated sled-agent to no longer expect a propolis_client::instance_spec::VolumeConstructionRequest when it is now just a Crucible VolumeConstructionRequest. Changes in Crucible: eliminate spurious metadb-related syncs (#881) ACK the write after adding it to the work queue (#874) Use arc4random_buf or getrandom instead of ChaCha20Rng (#878) Fix crutest in README (#879) Show client_id in panic messages (#843) Reduce sqlite page cache size to 64KiB (#876) Only flush extents that might actually be dirty. (#875) Changes in Propolis: Clean up and restructure CQE handling in NVMe Improve error message when /dev/vmmctl not present (#506) new API endpoint for propolis-server to replace a crucible downstairs (#495) Update rustls-webpki for GHSA-fh2r-99q2-6mmg Add MemAsync block backend try reenabling PHD jobs (#484) Define versioned instance specs (#472) Update toml dependency to 0.7.x Add more USDT probes to NVMe emulation Add more to standalone-with-crucible (#490) Update propolis with new Crucible Volume change (#485) Minor polish to standalone-crucible doc Clean up bits for crucible in propolis-standalone doc iteration for crucible and propolis-standalone Skeleton docs for using propolis with crucible disks in isolation propolis-standalone: Update expected crucible opts (#488) Split up README content for server and standalone Add crucible config to propolis-standalone Use libstd-provided OnceCell equivalent Allow 64-vCPU instances on Helios (stlouis) Elide test (and doctest) steps where not required Clean up NVMe PRP parsing and add tests
leftwo
added a commit
that referenced
this pull request
Aug 24, 2023
Updated sled-agent to no longer expect a propolis_client::instance_spec::VolumeConstructionRequest when it is now just a Crucible VolumeConstructionRequest. Changes in Crucible: eliminate spurious metadb-related syncs (#881) ACK the write after adding it to the work queue (#874) Use arc4random_buf or getrandom instead of ChaCha20Rng (#878) Fix crutest in README (#879) Show client_id in panic messages (#843) Reduce sqlite page cache size to 64KiB (#876) Only flush extents that might actually be dirty. (#875) Changes in Propolis: Clean up and restructure CQE handling in NVMe Improve error message when /dev/vmmctl not present (#506) new API endpoint for propolis-server to replace a crucible downstairs (#495) Update rustls-webpki for GHSA-fh2r-99q2-6mmg Add MemAsync block backend try reenabling PHD jobs (#484) Define versioned instance specs (#472) Update toml dependency to 0.7.x Add more USDT probes to NVMe emulation Add more to standalone-with-crucible (#490) Update propolis with new Crucible Volume change (#485) Minor polish to standalone-crucible doc Clean up bits for crucible in propolis-standalone doc iteration for crucible and propolis-standalone Skeleton docs for using propolis with crucible disks in isolation propolis-standalone: Update expected crucible opts (#488) Split up README content for server and standalone Add crucible config to propolis-standalone Use libstd-provided OnceCell equivalent Allow 64-vCPU instances on Helios (stlouis) Elide test (and doctest) steps where not required Clean up NVMe PRP parsing and add tests Co-authored-by: Alan Hanson <alan@oxide.computer>
gjcolombo
added a commit
that referenced
this pull request
Mar 5, 2025
Changes since the last update: - lib: use correct MAXCPU value in CPUID specializer (#876) - phd: wait for source to resume before asking to migrate again (#874) - phd: add smoke test for VCR replacement (#872) - lib: implement reference TSC enlightenment (#856) - Update package deps for GHSAs - Wire up viona for illumos#17032 - mock-server: add single-step API (#869) - propolis-server should not crash when failing to start a VM (#866) - propolis-cli: check for duplicate spec keys when parsing toml (#865) - various new 1.85 clippy lints (#864) - mock: attempt realistic state transitions (#860) - lib: tidy up overlay page migration & reduce memory usage (#861) - server: add state machine docs (#862) - DTrace script to inspect VM exit reasons (#859) - lib: add better management of Hyper-V overlay pages (#851) - lib: emulate Hyper-V enlightenment stack (#849)
Merged
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.
This is a "minimal-functionality, mostly refactor" change.