Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add trace logging for the DNS server #10809

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

Closed
wants to merge 7 commits into from
Closed

Conversation

simonrw
Copy link
Contributor

@simonrw simonrw commented May 13, 2024

Motivation

Building on #10808, we want to enable debug logging for the DNS server.

Changes

  • Set default DNS logging level to INFO in debug mode, so DEBUG=1 does not enable this debug logging
  • Add debug logging for DNS which can be enabled with the configuration LOGGING_OVERRIDE=localstack.dns=DEBUG

Requires #10808

@simonrw simonrw added the semver: patch Non-breaking changes which can be included in patch releases label May 13, 2024
@simonrw simonrw self-assigned this May 13, 2024
Copy link

github-actions bot commented May 13, 2024

S3 Image Test Results (AMD64 / ARM64)

  2 files    2 suites   3m 11s ⏱️
398 tests 346 ✅  52 💤 0 ❌
796 runs  692 ✅ 104 💤 0 ❌

Results for commit 7c9abb9.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented May 13, 2024

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 36m 52s ⏱️ +2s
2 950 tests ±0  2 648 ✅ ±0  302 💤 ±0  0 ❌ ±0 
2 952 runs  ±0  2 648 ✅ ±0  304 💤 ±0  0 ❌ ±0 

Results for commit 7c9abb9. ± Comparison against base commit 9d80628.

♻️ This comment has been updated with latest results.

@simonrw
Copy link
Contributor Author

simonrw commented Oct 29, 2024

Closing as this is no longer required.

@simonrw simonrw closed this Oct 29, 2024
@alexrashed alexrashed deleted the dns/trace-logging branch October 30, 2024 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants