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

Add trace logging for the DNS server #10809

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

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.

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.

None yet

1 participant