Skip to content

tools: add host side tool for string filtering#5185

Merged
LalitMaganti merged 13 commits intomainfrom
dev/lalitm/filter-tool-v2
Mar 19, 2026
Merged

tools: add host side tool for string filtering#5185
LalitMaganti merged 13 commits intomainfrom
dev/lalitm/filter-tool-v2

Conversation

@LalitMaganti
Copy link
Member

@LalitMaganti LalitMaganti commented Mar 18, 2026

Simple tool which allows easy testing of whether string filtering is working correctly.

This was a very very dumb issue where we weren't actually using the
overlay code from the config even though we had it just fine.
Extract ConvertPolicy, ConvertSemanticTypeMask, and the full
bytecode+string-filter loading logic from tracing_service_impl.cc
into a new shared helper: LoadMessageFilterConfig(). This avoids
duplicating this logic across tools that need to load a MessageFilter
from a TraceConfig's trace_filter field.
Add -i/--trace_in and -o/--trace_out flags to apply bytecode-based
proto field filtering plus string filtering to a binary trace file.
Uses the shared LoadMessageFilterConfig helper to load the filter
configuration from a TraceConfig textproto.

The existing string-only mode (positional argument) is unchanged.
@LalitMaganti LalitMaganti requested a review from a team as a code owner March 18, 2026 20:48
@LalitMaganti LalitMaganti changed the title tools: add trace filtering mode to string_filter_tool tools: add host side tool for string filtering Mar 18, 2026
Base automatically changed from dev/lalitm/refactor-filter-config to main March 19, 2026 13:28
@LalitMaganti LalitMaganti enabled auto-merge (squash) March 19, 2026 13:32
@LalitMaganti LalitMaganti merged commit cc7b51d into main Mar 19, 2026
23 checks passed
@LalitMaganti LalitMaganti deleted the dev/lalitm/filter-tool-v2 branch March 19, 2026 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants