Skip to content

(DO NOT MERGE) enable mDNS verbose logging and add extra logs#12139

Closed
abtink wants to merge 1 commit intoopenthread:mainfrom
abtink:mdns/extra-logs
Closed

(DO NOT MERGE) enable mDNS verbose logging and add extra logs#12139
abtink wants to merge 1 commit intoopenthread:mainfrom
abtink:mdns/extra-logs

Conversation

@abtink
Copy link
Member

@abtink abtink commented Nov 11, 2025

This commit contains two changes:

  • It always enables MULTICAST_DNS_VERBOSE_LOGGING_ENABLE regardless of any other configs/options.
  • It also adds new logs to indicate when a conflict is detected, helping to identify the message that caused the conflict and what it contained.

@github-actions
Copy link

github-actions bot commented Nov 11, 2025 •
edited
Loading

Merging #12139 into main

name branch text data bss total
ot-cli-ftd 61e43cf 481024 860 66580 548464
45a0fed 481024 860 66580 548464
+/- +0 +0 +0 +0
ot-ncp-ftd 61e43cf 446940 764 61824 509528
45a0fed 446940 764 61824 509528
+/- +0 +0 +0 +0
ot-cli-mtd 61e43cf 373232 764 51052 425048
45a0fed 373232 764 51052 425048
+/- +0 +0 +0 +0
ot-ncp-mtd 61e43cf 354196 764 46320 401280
45a0fed 354196 764 46320 401280
+/- +0 +0 +0 +0
ot-cli-ftd-br 61e43cf 585048 868 136444 722360
45a0fed 587048 868 136452 724368
+/- +2000 +0 +8 +2008
ot-rcp 61e43cf 63200 568 20804 84572
45a0fed 63200 568 20804 84572
+/- +0 +0 +0 +0
Library files
name branch text data bss total
libopenthread-ftd.a 61e43cf 247949 95 40358 288402
45a0fed 247949 95 40358 288402
+/- +0 +0 +0 +0
libopenthread-cli-ftd.a 61e43cf 61459 0 8083 69542
45a0fed 61459 0 8083 69542
+/- +0 +0 +0 +0
libopenthread-ncp-ftd.a 61e43cf 33581 0 5948 39529
45a0fed 33581 0 5948 39529
+/- +0 +0 +0 +0
libopenthread-mtd.a 61e43cf 166067 0 24854 190921
45a0fed 166067 0 24854 190921
+/- +0 +0 +0 +0
libopenthread-cli-mtd.a 61e43cf 41414 0 8059 49473
45a0fed 41414 0 8059 49473
+/- +0 +0 +0 +0
libopenthread-ncp-mtd.a 61e43cf 25805 0 5948 31753
45a0fed 25805 0 5948 31753
+/- +0 +0 +0 +0
libopenthread-ftd-br.a 61e43cf 354067 100 110190 464357
45a0fed 356456 100 110198 466754
+/- +2389 +0 +8 +2397
libopenthread-cli-ftd-br.a 61e43cf 79303 0 8115 87418
45a0fed 79303 0 8115 87418
+/- +0 +0 +0 +0
libopenthread-rcp.a 61e43cf 9944 0 5060 15004
45a0fed 9944 0 5060 15004
+/- +0 +0 +0 +0
libopenthread-radio.a 61e43cf 19479 0 238 19717
45a0fed 19479 0 238 19717
+/- +0 +0 +0 +0

@abtink
(DO NOT MERGE) enable mDNS verbose logging and add extra logs
e3441c3 
This commit contains two changes:

- It always enables `MULTICAST_DNS_VERBOSE_LOGGING_ENABLE` regardless
  of any other configs/options.
- It also adds new logs to indicate when a conflict is detected,
  helping to identify the message that caused the conflict and what
  it contained.
@codecov
Copy link

codecov bot commented Nov 12, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.90%. Comparing base (01aba92) to head (e3441c3).
⚠️ Report is 7 commits behind head on main.

Files with missing lines Patch % Lines
src/core/net/mdns.cpp 0.00% 3 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main   #12139      +/-   ##
==========================================
+ Coverage   73.61%   75.90%   +2.28%     
==========================================
  Files         671      667       -4     
  Lines       95050    98359    +3309     
==========================================
+ Hits        69969    74656    +4687     
+ Misses      25081    23703    -1378
Files with missing lines Coverage Δ
src/core/net/mdns.cpp 0.97% <0.00%> (-0.01%) ⬇️

... and 231 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@abtink abtink closed this Nov 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@abtink