Skip to content

Fix Issues discovered while stress testing#1858

Open
AlyaGomaa wants to merge 98 commits intodevelopfrom
alya/immune/stress_testing
Open

Fix Issues discovered while stress testing#1858
AlyaGomaa wants to merge 98 commits intodevelopfrom
alya/immune/stress_testing

Conversation

@AlyaGomaa
Copy link
Copy Markdown
Collaborator

Closes #1311, #1837, #1339, #1848

  • Speedup the processing of evidence by starting multiple EvidenceHandlerWorker processes in parallel.

  • Optimize and refactor localnet detection logic.

  • Reduced the default number of profiler workers started at launch from 5 to 3, with the existing scaling logic still available.

  • Changed network discovery to consume new_flow events instead of tw_modified, to avoid depending on the pub sub published msgs per flows for portscan checks. (main latency bottleneck)

  • Added a generate_performance_plots parameter to enable/disable performance plots.

  • Added performance instrumentation for input, profiler workers, and evidence handling to record throughput and latency data in output/performance_plots/csv/.

  • Added a Plotter to generate plots on shutdown and log throughput/latency metrics to metrics.txt.

  • Added a stress-testing docs documenting baseline, sudden spikes and soak testing.

  • Handle divide-by-zero case in progress reporting in slips/main.py.

…s to avoid "encoding with 'idna' codec failed" err
…s to avoid "encoding with 'idna' codec failed" err
… alya/immune/stress_testing

# Conflicts:
#	slips_files/common/abstracts/icore.py
#	slips_files/core/database/redis_db/constants.py
#	slips_files/core/database/redis_db/database.py
#	slips_files/core/input.py
…s to avoid "encoding with 'idna' codec failed" err
… alya/immune/stress_testing

# Conflicts:
#	slips_files/core/profiler.py
…pped, until the stop msg is received. to avoid shutting down profilers early as soon as input stops (usually input stops early and profilers cont. profiling until all flows are analyzed by them)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

1 participant