convert Instance-related lookups to new lookup API #843
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
|
@bnaecker would you mind taking a look at this one? As part of this I changed the datastore functions related to network interfaces to look more consistent with other resources. (More specifically: |
71 tasks
bnaecker
reviewed
Mar 31, 2022
davepacheco
changed the title
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>
leftwo
added a commit
that referenced
this pull request
Jan 29, 2025
Crucible changes are: Add early rejection of IOs if too many Downstairs are inactive (#1565) Fix missing write stats in Oximeter. (#1617) Shrink replay buffer (#1616) Update tokio to 1.40 (#1611) Propolis changes are: crates: move host CPUID queries from cpuid-gen to cpuid-utils (#843) --------- Co-authored-by: Alan Hanson <alan@oxide.computer>
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.
See #798. This is converting the Instance-related lookups and network interface-related lookups.