Unverified Commit f87938a3 authored by Noir's avatar Noir Committed by GitHub

Merge pull request #1239

Add metrics filters to the list of CLI parameters
parents 1b29bf10 21aea94b
...@@ -26,6 +26,10 @@ is based on [Keep a Changelog](https://keepachangelog.com). ...@@ -26,6 +26,10 @@ is based on [Keep a Changelog](https://keepachangelog.com).
- Passing a function reference to the constructor of an actor caused a compiler - Passing a function reference to the constructor of an actor caused a compiler
error when building with logging enabled. CAF now properly handles this edge error when building with logging enabled. CAF now properly handles this edge
case and logs such constructor arguments as `<unprintable>` (#1229). case and logs such constructor arguments as `<unprintable>` (#1229).
- The CLI parser did not recognize metrics filters. Hence, passing
`--caf.metrics-filters.actors.includes=...` to a CAF application resulted in
an error. The `includes` and `excludes` filters are now consistently handled
and accepted in config files as well as on the command line (#1238).
## [0.18.0] - 2021-01-25 ## [0.18.0] - 2021-01-25
......
...@@ -101,6 +101,9 @@ actor_system_config::actor_system_config() ...@@ -101,6 +101,9 @@ actor_system_config::actor_system_config()
.add<string>("format", "format for printed console lines") .add<string>("format", "format for printed console lines")
.add<string>("verbosity", "minimum severity level for console output") .add<string>("verbosity", "minimum severity level for console output")
.add<string_list>("excluded-components", "excluded components on console"); .add<string_list>("excluded-components", "excluded components on console");
opt_group{custom_options_, "caf.metrics-filters.actors"}
.add<string_list>("includes", "selects actors for run-time metrics")
.add<string_list>("excludes", "excludes actors from run-time metrics");
} }
settings actor_system_config::dump_content() const { settings actor_system_config::dump_content() const {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment