Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(peerseeds/autoupdate): hickory-dns upgrade, default to system DNS #6600

Merged
merged 3 commits into from
Oct 3, 2024

Conversation

sdbondi
Copy link
Member

@sdbondi sdbondi commented Oct 3, 2024

Description

Upgrade from trust dns to hickory DNS
Allow system default DNS to be used for DNS resolution

Motivation and Context

Cloudflare 1.1.1.1 was used by default and this can be blocked. Better UX to use the system default.

How Has This Been Tested?

Existing tests updated

What process can a PR reviewer use to test or verify this change?

Seed peers still work

Breaking Changes

  • None
  • Requires data directory on base node to be deleted
  • Requires hard fork
  • Other - Please specify

@sdbondi sdbondi force-pushed the system-default-dns branch 2 times, most recently from 1e7a4b7 to ffeb16a Compare October 3, 2024 12:41
@sdbondi sdbondi force-pushed the system-default-dns branch from 5ac92ec to 51c3bf5 Compare October 3, 2024 12:46
Copy link

github-actions bot commented Oct 3, 2024

Test Results (Integration tests)

 2 files  11 suites   32m 27s ⏱️
36 tests 35 ✅ 0 💤 1 ❌
38 runs  36 ✅ 0 💤 2 ❌

For more details on these failures, see this check.

Results for commit 6f60db1.

♻️ This comment has been updated with latest results.

@ghpbot-tari-project ghpbot-tari-project added P-acks_required Process - Requires more ACKs or utACKs P-reviews_required Process - Requires a review from a lead maintainer to be merged labels Oct 3, 2024
Copy link

github-actions bot commented Oct 3, 2024

Test Results (CI)

    3 files    129 suites   35m 45s ⏱️
1 325 tests 1 325 ✅ 0 💤 0 ❌
3 973 runs  3 973 ✅ 0 💤 0 ❌

Results for commit 6f60db1.

♻️ This comment has been updated with latest results.

SWvheerden
SWvheerden previously approved these changes Oct 3, 2024
Copy link
Collaborator

@SWvheerden SWvheerden left a comment

Choose a reason for hiding this comment

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

pending ci

@sdbondi
Copy link
Member Author

sdbondi commented Oct 3, 2024

Looks like we're using an old version of gcc CI. I've added that to the CI deps install script so let's see.

rustls/rustls#1967

@sdbondi sdbondi force-pushed the system-default-dns branch from be6dd94 to 6f60db1 Compare October 3, 2024 13:48
@sdbondi
Copy link
Member Author

sdbondi commented Oct 3, 2024

Guess not - I've removed the aws-ls-rs feature from rusttls. Looks like that is working

@SWvheerden SWvheerden merged commit 5bcddf4 into tari-project:development Oct 3, 2024
16 of 17 checks passed
@sdbondi sdbondi deleted the system-default-dns branch October 4, 2024 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P-acks_required Process - Requires more ACKs or utACKs P-reviews_required Process - Requires a review from a lead maintainer to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants