Skip to content

Commit 683df32

Browse files
LucaGuerrapoiana
authored andcommitted
fix(falco): allow disable_cri_async from both CLI and config
Signed-off-by: Luca Guerra <[email protected]>
1 parent 262aa9a commit 683df32

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

userspace/falco/app/actions/init_inspectors.cpp

+8-8
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,14 @@ static void init_syscall_inspector(falco::app::state& s, std::shared_ptr<sinsp>
6767
"Enabled container runtime socket at '" + p + "' via config file");
6868
}
6969
}
70-
inspector->set_cri_async(!s.config->m_container_engines_disable_cri_async);
70+
71+
bool disable_cri_async =
72+
s.config->m_container_engines_disable_cri_async || s.options.disable_cri_async;
73+
inspector->set_cri_async(!disable_cri_async);
74+
75+
if(disable_cri_async) {
76+
falco_logger::log(falco_logger::level::DEBUG, "Disabling async lookups for 'CRI'");
77+
}
7178

7279
// Container engines configs via CLI args
7380
// If required, set the CRI paths
@@ -79,13 +86,6 @@ static void init_syscall_inspector(falco::app::state& s, std::shared_ptr<sinsp>
7986
}
8087
}
8188

82-
// Decide whether to do sync or async for CRI metadata fetch
83-
inspector->set_cri_async(!s.options.disable_cri_async);
84-
85-
if(s.options.disable_cri_async || s.config->m_container_engines_disable_cri_async) {
86-
falco_logger::log(falco_logger::level::DEBUG, "Disabling async lookups for 'CRI'");
87-
}
88-
8989
//
9090
// If required, set the snaplen
9191
//

0 commit comments

Comments
 (0)