Skip to content

fix(tests): handle root-owned Falco logs in stack tests and increase trigger wait - #114

Merged
JJediny merged 1 commit into
mainfrom
fix/stack-verification-and-test-robustness
Oct 11, 2026
Merged

JJediny merged 1 commit into
mainfrom
fix/stack-verification-and-test-robustness

Conversation

@JJediny

@JJediny JJediny commented Oct 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • Stack Integration Tests: In tests/test_stack_integration.py, guard FALCO_FILE_PATH.touch(exist_ok=True) so that when .data/falco/events.jsonl is already present and owned by root (via container or host Falco daemon execution), non-root test runners don't fail with PermissionError: [Errno 13].
  • Alert Trigger Runner: In tools/trigger_alerts.py, increase the ingestion wait window from 3s to 6s so that OpenTelemetry Collector's filelog receiver polling cycle reliably flushes newly appended Falco and ClamAV lines before verification queries execute.

Verification

  • Ran full pytest suite: 369 passed, 40 skipped.
  • Ran live stack integration tests against running stack (test_stack_integration.py, test_alert_receiver_integration.py, test_rsigma_alerts.py, test_agent_session_e2e.py): all passed cleanly.
  • Executed python3 tools/trigger_alerts.py --all: 100% hits verified across falco, clamav, agent_logs, system_logs, and osquery streams.

@JJediny
JJediny merged commit 04d846e into main Oct 11, 2026
8 checks passed
@JJediny
JJediny deleted the fix/stack-verification-and-test-robustness branch October 11, 2026 01:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant