Skip to content

start#44535

Open
YaelShamai wants to merge 1 commit into
masterfrom
ys-securiti-swcuriti
Open

start#44535
YaelShamai wants to merge 1 commit into
masterfrom
ys-securiti-swcuriti

Conversation

@YaelShamai
Copy link
Copy Markdown
Contributor

Contributing to Cortex XSOAR Content

Make sure to register your contribution by filling the contribution registration form

The Pull Request will be reviewed only after the contribution registration form is filled.

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

fixes: https://jira-dc.paloaltonetworks.com/browse/CIAC-16669

Description

Securiti Securiti Event Collector

Must have

  • Tests
  • Documentation

@content-bot
Copy link
Copy Markdown
Contributor

🤖 AI-Powered Code Review Available

You can leverage AI-powered code review to assist with this PR!

Available Commands:

  • @marketplace-ai-reviewer start review - Initiate a full AI code review
  • @marketplace-ai-reviewer re-review - Incremental review for new commits

@content-bot
Copy link
Copy Markdown
Contributor

⚠️ The PR is missing the ready-for-pipeline-running label. Please add the label when the PR is ready in order to proceed.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 7, 2026

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/Securiti/Integrations/SecuritiEventCollector
   SecuritiEventCollector.py1271242%4–5, 8, 12–13, 15–20, 25, 32, 41, 47, 49, 65, 84–85, 91–93, 99, 108, 111, 119–122, 124, 127, 140–141, 143–148, 151, 189–192, 194, 196–198, 200, 206–208, 212–213, 215, 217–222, 224–225, 228–230, 233, 235, 237–238, 240, 245, 250–252, 256–260, 262–263, 266–268, 270, 272, 279–280, 286, 292, 298, 308, 310, 315–319, 321, 324, 334–335, 337–339, 341, 343, 351, 357, 360, 377–378, 380–384, 386, 391, 399–401, 403
TOTAL1271242% 

Tests Skipped Failures Errors Time
1 0 💤 0 ❌ 1 🔥 3.405s ⏱️

@content-bot
Copy link
Copy Markdown
Contributor

Validate summary
The following errors were reported as warnings: BC117.
BC117 validation requires PM review due to changes to supportedModules.
The following errors were thrown as a part of this pr: IN145, RN108, IM100, DO106, PA128.
The following errors can be ignored: IN145.
The following errors cannot be ignored: RN108, IM100, DO106, PA128.
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.
The following errors don't run as part of the nightly flow and therefore can be force merged: RN108, IM100, DO106, PA128.

Verdict: PR can be force merged from validate perspective? ❌

@content-bot
Copy link
Copy Markdown
Contributor

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: 0b2e4eeb9e41f798

📋 Triage Report
💡 Resolutions are available in the full report.

⚠️ AI-generated triage. Validate before acting.

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.

2 participants