Skip to content

Microsoft defender for cloud - CRTX 228905#43186

Open
yasta5 wants to merge 5 commits intomasterfrom
Microsoft_Defender_for_Cloud_CRTX-228905_YA
Open

Microsoft defender for cloud - CRTX 228905#43186
yasta5 wants to merge 5 commits intomasterfrom
Microsoft_Defender_for_Cloud_CRTX-228905_YA

Conversation

@yasta5
Copy link
Contributor

@yasta5 yasta5 commented Feb 18, 2026

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

https://jira-dc.paloaltonetworks.com/browse/CRTX-123374

Description

Modified the modeling rule's filters based on the different schemas.

@yasta5 yasta5 requested a review from Moish-Gilboa as a code owner February 18, 2026 14:45
@content-bot
Copy link
Contributor

content-bot commented Feb 18, 2026

🤖 AI-Powered Code Review Available

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

Available Commands:

  • @content-bot start review - Initiate a full AI code review
  • @content-bot re-review - Incremental review for new commits

@yasta5 yasta5 added bypass.url Whether to create build bucket, add this label for marketplace.bootstrap.bypass.url ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines labels Feb 18, 2026
@yasta5 yasta5 requested a review from OBavly February 18, 2026 14:59
@yasta5 yasta5 added docs-approved ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. labels Feb 18, 2026
@yasta5 yasta5 removed the request for review from Moish-Gilboa February 18, 2026 15:03
@content-bot content-bot removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Feb 18, 2026
@content-bot
Copy link
Contributor

content-bot commented Feb 18, 2026

🤖 Content AI Reviewer: Analysis started. Please wait for results...

@content-bot
Copy link
Contributor

content-bot commented Feb 18, 2026

🤖 Content-bot Review Disclaimer

This review was generated by an AI-powered tool and may contain inaccuracies. Please be advised, and we extend our sincere apologies for any inconvenience this may cause.

content-bot
content-bot reviewed Feb 18, 2026
View reviewed changes
Copy link
Contributor

@content-bot content-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution! I noticed a few formatting inconsistencies in the Modeling Rule that need adjustment. Please switch to using 4 spaces for indentation and remove the extra whitespace around the dataset definition and filter keywords. Great work otherwise!

@OBavly please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

@content-bot
Copy link
Contributor

content-bot commented Feb 18, 2026

Validate summary
The following errors were thrown as a part of this pr: .
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@content-bot content-bot content-bot left review comments

@eepstain eepstain eepstain approved these changes

@OBavly OBavly Awaiting requested review from OBavly

Assignees

No one assigned

Labels

bypass.url Whether to create build bucket, add this label for marketplace.bootstrap.bypass.url docs-approved ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@yasta5 @content-bot @eepstain

Comments