Skip to content

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