Skip to content

Commit 3dd71b8

Browse files
committed
Allow to not collect process arguments
1 parent 842c664 commit 3dd71b8

File tree

3 files changed

+10
-2
lines changed

3 files changed

+10
-2
lines changed

collector/lib/CollectorConfig.cpp

+3
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,8 @@ BoolEnvVar use_podman_ce("ROX_COLLECTOR_CE_USE_PODMAN", false);
5858

5959
BoolEnvVar enable_introspection("ROX_COLLECTOR_INTROSPECTION_ENABLE", false);
6060

61+
BoolEnvVar disable_process_arguments("ROX_COLLECTOR_NO_PROCESS_ARGUMENTS", false);
62+
6163
} // namespace
6264

6365
constexpr bool CollectorConfig::kTurnOffScrape;
@@ -87,6 +89,7 @@ void CollectorConfig::InitCollectorConfig(CollectorArgs* args) {
8789
use_docker_ce_ = use_docker_ce.value();
8890
use_podman_ce_ = use_podman_ce.value();
8991
enable_introspection_ = enable_introspection.value();
92+
disable_process_arguments_ = disable_process_arguments.value();
9093

9194
for (const auto& syscall : kSyscalls) {
9295
syscalls_.push_back(syscall);

collector/lib/CollectorConfig.h

+3
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,7 @@ class CollectorConfig {
8787
unsigned int GetSinspBufferSize() const;
8888
unsigned int GetSinspTotalBufferSize() const { return sinsp_total_buffer_size_; }
8989
unsigned int GetSinspThreadCacheSize() const { return sinsp_thread_cache_size_; }
90+
bool DisableProcessArguments() const { return disable_process_arguments_; }
9091

9192
std::shared_ptr<grpc::Channel> grpc_channel;
9293

@@ -122,6 +123,8 @@ class CollectorConfig {
122123
double connection_stats_error_;
123124
unsigned int connection_stats_window_;
124125

126+
bool disable_process_arguments_ = false;
127+
125128
// One ring buffer will be initialized for this many CPUs
126129
unsigned int sinsp_cpu_per_buffer_ = 0;
127130
// Size of one ring buffer, in bytes.

collector/lib/ProcessSignalFormatter.cpp

+4-2
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,10 @@ ProcessSignal* ProcessSignalFormatter::CreateProcessSignal(sinsp_evt* event) {
120120
signal->set_exec_file_path(*name);
121121
}
122122

123-
// set process arguments
124-
if (const char* args = event_extractor_->get_proc_args(event)) signal->set_args(args);
123+
// set process arguments, if not explicitely disabled
124+
if (!config_.DisableProcessArguments()) {
125+
if (const char* args = event_extractor_->get_proc_args(event)) signal->set_args(args);
126+
}
125127

126128
// set pid
127129
if (const int64_t* pid = event_extractor_->get_pid(event)) signal->set_pid(*pid);

0 commit comments

Comments
 (0)