Skip to content

CRTX-220668-CyberArkIdentity-StoryMapping#43649

Merged
akshotiamit-pa merged 15 commits into
masterfrom
CRTX-220668-CyberArkIdentity
May 1, 2026
Merged

CRTX-220668-CyberArkIdentity-StoryMapping#43649
akshotiamit-pa merged 15 commits into
masterfrom
CRTX-220668-CyberArkIdentity

Conversation

@akshotiamit-pa
Copy link
Copy Markdown
Contributor

@akshotiamit-pa akshotiamit-pa commented Mar 25, 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

Description

Must have

  • Tests
  • Documentation

@content-bot
Copy link
Copy Markdown
Contributor

content-bot commented Mar 25, 2026

🤖 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

content-bot commented Apr 25, 2026

This PR is marked as 'Stale' because it has been open for 30 days with no activity, it will be automatically closed in 15 days if no activity will be done. To reset the counter just remove the 'Stale' label or make changes to update this PR. If you wish this PR will never be marked as 'Stale' add the 'Ignore Stale'

@akshotiamit-pa akshotiamit-pa force-pushed the CRTX-220668-CyberArkIdentity branch from 56042e9 to 7feeff6 Compare April 29, 2026 07:29
@akshotiamit-pa akshotiamit-pa 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 ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. and removed Stale labels Apr 29, 2026
eepstain
eepstain reviewed Apr 29, 2026
View reviewed changes
Comment thread Packs/CyberArkPAS/ModelingRules/CyberArkISP/CyberArkISP.xif
eepstain
eepstain reviewed Apr 29, 2026
View reviewed changes
Comment thread Packs/CyberArkPAS/ModelingRules/CyberArkISP/CyberArkISP.xif
@marketplace-ai-reviewer marketplace-ai-reviewer removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Apr 29, 2026
@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

marketplace-ai-reviewer commented Apr 29, 2026

🤖 Analysis started. Please wait for results...

@akshotiamit-pa
Copy link
Copy Markdown
Contributor Author

akshotiamit-pa commented Apr 29, 2026

@content-bot start review

@akshotiamit-pa akshotiamit-pa added the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Apr 29, 2026
@marketplace-ai-reviewer marketplace-ai-reviewer removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Apr 29, 2026
@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

marketplace-ai-reviewer commented Apr 29, 2026

🤖 Analysis started. Please wait for results...

@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

marketplace-ai-reviewer commented Apr 29, 2026

🤖 AI 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.

marketplace-ai-reviewer
marketplace-ai-reviewer reviewed Apr 29, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@marketplace-ai-reviewer marketplace-ai-reviewer 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 your contribution to the CyberArkPAS pack! I've reviewed the PR and found a few areas that need minor adjustments before we can merge.

Mainly, please ensure all fields used in the XIF logic are properly defined in the schema and remember to include the missing test data file for the modeling rule. There are also a few standard formatting tweaks needed for the README and Release Notes.

Let me know if you have any questions!

Additionally, please address the following file-level notes:

  • Packs/CyberArkPAS/ModelingRules/CyberArkISP/CyberArkISP.yml: Missing test data file CyberArkISP_test.json.

@eepstain, @richardbluestone, @akshotiamit-pa please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

Comment thread Packs/CyberArkPAS/ModelingRules/CyberArkISP/CyberArkISP.xif
Comment thread Packs/CyberArkPAS/ModelingRules/CyberArkISP/CyberArkISP.xif
Comment thread Packs/CyberArkPAS/ModelingRules/CyberArkISP/CyberArkISP.xif Outdated
Comment thread Packs/CyberArkPAS/ModelingRules/CyberArkISP/CyberArkISP.xif
Comment thread Packs/CyberArkPAS/ModelingRules/CyberArkISP/CyberArkISP.xif
Comment thread Packs/CyberArkPAS/README.md
Comment thread Packs/CyberArkPAS/ReleaseNotes/1_1_6.md Outdated
@content-bot

This comment has been minimized.

Sign in to view
@content-bot

This comment has been minimized.

Sign in to view
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 30, 2026

Your contributed GenericAPICall pack has been modified on files:

Packs/GenericAPICall/Integrations/GenericAPICall/GenericAPICall.yml
Packs/GenericAPICall/Integrations/GenericAPICall/GenericAPICall.py
Please review the changes here

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 30, 2026

Your contributed DevSecOps pack has been modified on files:

Packs/DevSecOps/.secrets-ignore
Please review the changes here

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 30, 2026

Your contributed AnythingLLM pack has been modified on files:

Packs/AnythingLLM/Integrations/AnythingLLM/AnythingLLM_test.py
Packs/AnythingLLM/pack_metadata.json
Packs/AnythingLLM/Integrations/AnythingLLM/AnythingLLM.yml
Packs/AnythingLLM/Integrations/AnythingLLM/AnythingLLM.py
Packs/AnythingLLM/ReleaseNotes/2_1_4.md
Please review the changes here

@akshotiamit-pa akshotiamit-pa force-pushed the CRTX-220668-CyberArkIdentity branch 2 times, most recently from aaa63ae to 862e49d Compare April 30, 2026 07:33
@content-bot

This comment has been minimized.

Sign in to view
@content-bot

This comment has been minimized.

Sign in to view
@content-bot

This comment has been minimized.

Sign in to view
@content-bot
Copy link
Copy Markdown
Contributor

content-bot commented Apr 30, 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? ✅

@content-bot
Copy link
Copy Markdown
Contributor

content-bot commented Apr 30, 2026

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: success
Report ID: 6c2395c9f10f1533

📋 Triage Report

⚠️ AI-generated triage. Validate before acting.

@akshotiamit-pa akshotiamit-pa merged commit e129e8f into master May 1, 2026
22 checks passed
@akshotiamit-pa akshotiamit-pa deleted the CRTX-220668-CyberArkIdentity branch May 1, 2026 05:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@richardbluestone richardbluestone richardbluestone left review comments

@marketplace-ai-reviewer marketplace-ai-reviewer marketplace-ai-reviewer left review comments

@eepstain eepstain eepstain approved these changes

Assignees

@akshotiamit-pa akshotiamit-pa

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 supported-modules-approved

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@akshotiamit-pa @content-bot @marketplace-ai-reviewer @richardbluestone @eepstain