From f1fff738b3da1dd20b1fa2f9a508882d72989871 Mon Sep 17 00:00:00 2001 From: seph Date: Thu, 1 Aug 2024 11:53:54 -0400 Subject: [PATCH] Update osquery checkup I don't think an existing osquery is an `Error`. I think it is, at best, a warning. --- ee/debug/checkups/osq_config_conflicts.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ee/debug/checkups/osq_config_conflicts.go b/ee/debug/checkups/osq_config_conflicts.go index 494d40129..fc9893aec 100644 --- a/ee/debug/checkups/osq_config_conflicts.go +++ b/ee/debug/checkups/osq_config_conflicts.go @@ -62,13 +62,13 @@ func (occ *osqConfigConflictCheckup) Run(ctx context.Context, extraFH io.Writer) } if dirNotEmpty { - occ.status = Failing + occ.status = Warning occ.summary = "At least one notable directory is present and non-empty" } else if dirHasError { occ.status = Erroring occ.summary = "At least one notable directory is present and could not be read" } else if dirExists { - occ.status = Warning + occ.status = Informational occ.summary = "At least one notable directory is present, but empty" } else { occ.status = Passing