Skip to content

Doc/12187 v1 add MODULES.md - a description of how modules work - #11

Closed
KEIAHNY wants to merge 2 commits into
mainfrom
12187-doc-suriconf-pipeline-v1
Closed

Doc/12187 v1 add MODULES.md - a description of how modules work #11
KEIAHNY wants to merge 2 commits into
mainfrom
12187-doc-suriconf-pipeline-v1

Conversation

@KEIAHNY

@KEIAHNY KEIAHNY commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

PR Type

Documentation


Description

  • Added comprehensive MODULES.md documentation file

  • Documented Suriconf pipeline stages with diagrams

  • Detailed all 4 configuration modules with formulas

  • Updated README.md with MODULES.md reference link


Diagram Walkthrough

flowchart LR
  User["User provides config"] --> Preconfig["Preconfiguration stage"]
  Preconfig --> Query["Query and Execute stage"]
  Query --> Output["suricata_result.yaml + nic_setup.sh"]
Loading

File Walkthrough

Relevant files
Documentation
MODULES.md
Add comprehensive Suriconf modules documentation                 

MODULES.md

  • Created new documentation file with 272 lines
  • Documented Suriconf pipeline with 2 stages diagram
  • Detailed CPU affinity, Memory usage, Flow, Flow threads modules
  • Included calculation formulas and configuration parameters
+272/-0 
README.md
Link MODULES.md from README modules section                           

README.md

  • Added reference link to MODULES.md in modules section
  • Improved navigation to detailed module descriptions
+1/-1     

@KEIAHNY KEIAHNY self-assigned this Aug 13, 2026
@KEIAHNY KEIAHNY added the documentation Improvements or additions to documentation label Aug 13, 2026
@CESNET CESNET deleted a comment from github-actions Bot Aug 13, 2026
@KEIAHNY

KEIAHNY commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator Author

/review

@github-actions

github-actions Bot commented Aug 13, 2026

Copy link
Copy Markdown

PR Reviewer Guide 🔍

(Review updated until commit f47c73a)

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@KEIAHNY

KEIAHNY commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator Author

/review

@github-actions

Copy link
Copy Markdown

Persistent review updated to latest commit f47c73a

@KEIAHNY

KEIAHNY commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator Author

/review

@github-actions

Copy link
Copy Markdown

Persistent review updated to latest commit f47c73a

@KEIAHNY

KEIAHNY commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator Author

Closing: #12.

@KEIAHNY KEIAHNY closed this Aug 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation Review effort 1/5

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant