Improve DNS verification by exposing port 53 to host#107
Merged
Conversation
- Expose CoreDNS port 53/udp to localhost:53 for host accessibility - Query localhost instead of container IP during DNS verification - Add comprehensive network and port diagnostics on verification failure - Improve error messages with common causes and debugging guidance - Add better exception handling with specific diagnostics for: - Port already in use (permission denied errors) - Connection timeouts - Connection refused errors - Log container network configuration and port mappings when verification fails This fixes the issue where DNS verification fails on the host because the host cannot reach container IPs on isolated Docker networks. The port binding allows the orchestrator to verify DNS is working by querying the standard port-mapped address. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01J3G1MWM7E94xwcYRKZgKuL
- Remove unused root_ip variable - Fix f-string that was missing placeholders - Break long lines to meet 100-character limit Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01J3G1MWM7E94xwcYRKZgKuL
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
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.
Summary
This PR improves DNS service verification and diagnostics by exposing the CoreDNS container's port 53/UDP to the host via localhost, enabling the orchestrator to verify DNS functionality from the host machine rather than requiring access to the internal core network.
Key Changes
127.0.0.1:53on the host, allowing DNS queries from the orchestrator without requiring core network access10.0.0.2→10.10.0.2) to127.0.0.1:53(localhost), improving accessibility and reliability_query_soa()with:_query_soa()method explaining parameters and return valueImplementation Details
127.0.0.1to restrict DNS access to the host machine only, maintaining securitylsofornetstatcommands to check port usage)debugfor transient failures anderrorfor persistent issueshttps://claude.ai/code/session_01J3G1MWM7E94xwcYRKZgKuL