From bb4a6433859e89940d0db50fb03b33611d75370e Mon Sep 17 00:00:00 2001 From: Melissa Kilby Date: Wed, 24 Jan 2024 16:19:26 +0000 Subject: [PATCH] update(config): soft deprecation of old stats add CHANGE NOTICE wrt syscall_event_drops Co-authored-by: Leonardo Grasso Signed-off-by: Melissa Kilby --- falco.yaml | 4 ++-- userspace/falco/app/actions/load_config.cpp | 6 ------ 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/falco.yaml b/falco.yaml index 7ff645be557..61ee59f2399 100644 --- a/falco.yaml +++ b/falco.yaml @@ -61,7 +61,7 @@ # Falco logging / alerting / metrics related to software functioning (advanced) # output_timeout # syscall_event_timeouts -# syscall_event_drops [DEPRECATED] -> Use `metrics` instead, `syscall_event_drops` will be removed in Falco 0.38! +# syscall_event_drops -> [CHANGE NOTICE] Automatic notifications will be simplified in Falco 0.38! If you depend on the detailed drop counters payload, use 'metrics.output_rule' along with 'metrics.kernel_event_counters_enabled' instead # metrics # Falco performance tuning (advanced) # syscall_buf_size_preset [DEPRECATED] -> Replaced by `engine..buf_size_preset` starting Falco 0.38! @@ -793,7 +793,7 @@ output_timeout: 2000 syscall_event_timeouts: max_consecutives: 1000 -# [Stable] `syscall_event_drops` -> Use `metrics` instead, `syscall_event_drops` will be removed in Falco 0.38! +# [Stable] `syscall_event_drops` -> [CHANGE NOTICE] Automatic notifications will be simplified in Falco 0.38! If you depend on the detailed drop counters payload, use 'metrics.output_rule' along with 'metrics.kernel_event_counters_enabled' instead # # Generates "Falco internal: syscall event drop" rule output when `priority=debug` at minimum # diff --git a/userspace/falco/app/actions/load_config.cpp b/userspace/falco/app/actions/load_config.cpp index 903b1763750..8dac82f5bd2 100644 --- a/userspace/falco/app/actions/load_config.cpp +++ b/userspace/falco/app/actions/load_config.cpp @@ -41,12 +41,6 @@ static falco::app::run_result apply_deprecated_options(falco::app::state& s) return run_result::fatal("You can not specify more than one of -e, -g (--gvisor-config), --modern-bpf, --nodriver, and the FALCO_BPF_PROBE env var"); } - if(s.config->m_min_priority == falco_common::PRIORITY_DEBUG) - { - falco_logger::log(falco_logger::level::WARNING, - "DEPRECATION NOTICE: 'syscall_event_drops' config is deprecated and will be removed in Falco 0.38! If you rely on this config, use 'metrics.output_rule' along with 'metrics.kernel_event_counters_enabled' to monitor the number of drops. Note that the 'syscall_event_drops' config is enabled by default when the 'priority' is set to 'debug'. You can turn it off by setting the 'priority' to any higher level\n"); - } - // Please note: is not possible to mix command line options and configs to obtain a configuration // we need to use only one method. For example, is not possible to set the gvisor-config through // the command line and the gvisor-root through the config file. For this reason, if we detect