Skip to content

[internal-dns] Avoid 'picking ports'#1233

Merged
smklein merged 14 commits into
mainfrom
fix-internal-dns-api
Jun 21, 2022
Merged

[internal-dns] Avoid 'picking ports'#1233
smklein merged 14 commits into
mainfrom
fix-internal-dns-api

Conversation

@smklein

@smklein smklein commented Jun 20, 2022

Copy link
Copy Markdown
Collaborator

Fixes #1200

The internal-dns crate includes two servers: a "DNS server" and a "Dropshot server". This change refactors the launching of both to ensure that their ports are queryable, especially under tests conditions where they are now set to "port zero" (let the OS pick for us).

This PR also removes the dependency on the "port-picker" crate.

Comment thread internal-dns/src/dns_server.rs Outdated
Comment thread internal-dns/tests/basic_test.rs Outdated
Base automatically changed from nexus-argsplit to main June 21, 2022 17:22
@smklein smklein merged commit 54ab253 into main Jun 21, 2022
@smklein smklein deleted the fix-internal-dns-api branch June 21, 2022 17:39
leftwo pushed a commit that referenced this pull request Apr 1, 2024
Crucible
Increase max backpressure again (#1243)
Bump backpressure delay for byte-based backpressure (#1240)
`crutest` cleaning; adding rand-read/write tests (#1233)
Propolis, just the above crucible.
@leftwo leftwo mentioned this pull request Apr 1, 2024
leftwo added a commit that referenced this pull request Apr 2, 2024
Crucible
Increase max backpressure again (Crucible issue #1243)
Bump backpressure delay for byte-based backpressure (Crucible issue #1240) `crutest`
cleaning; adding rand-read/write tests (Crucible issue #1233) 
Propolis changes, just the above crucible bits.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[internal-dns] servfail test flake?

2 participants