Skip to content

Drop "GaiResolverWithAbsolute"#486

Merged
seanses merged 1 commit intomainfrom
di/remove-custom-dns
Sep 9, 2025
Merged

Drop "GaiResolverWithAbsolute"#486
seanses merged 1 commit intomainfrom
di/remove-custom-dns

Conversation

@seanses
Copy link
Collaborator

@seanses seanses commented Sep 8, 2025

Removes this custom DNS resolver that forces domain names to be absolute. Using relative domain names is a valid practice in Kubernetes clusters to configure proxy servers, e.g. huggingface/huggingface_hub#3323

The original PR that added this resolver was trying to resolve huggingface/huggingface_hub#3155 later appeared to be a problem of user local misconfiguration.

@seanses seanses marked this pull request as ready for review September 8, 2025 22:00
@seanses seanses marked this pull request as draft September 8, 2025 22:19
@seanses seanses marked this pull request as ready for review September 9, 2025 15:50
Copy link
Member

@Hugoch Hugoch left a comment

Choose a reason for hiding this comment

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

🪦 RIP

@seanses seanses merged commit e2f7861 into main Sep 9, 2025
6 checks passed
@seanses seanses deleted the di/remove-custom-dns branch September 9, 2025 17:26
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.

DNS error while download file with xet storage backend

3 participants