Skip to content

XSUP-68212: AWS GuardDuty#44131

Merged
RotemAmit merged 5 commits intomasterfrom
xsup-68212-aws-guardduty
May 3, 2026
Merged

XSUP-68212: AWS GuardDuty#44131
RotemAmit merged 5 commits intomasterfrom
xsup-68212-aws-guardduty

Conversation

@RotemAmit
Copy link
Copy Markdown
Contributor

@RotemAmit RotemAmit commented Apr 30, 2026 •
edited
Loading

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/XSUP-68212

Description

Fixed the type of the arguments feedback and comments of the command aws-gd-update-findings-feedback to be a string.

Must have

  • Tests
  • Documentation

@RotemAmit RotemAmit requested a review from Moish-Gilboa as a code owner April 30, 2026 12:51
@content-bot
Copy link
Copy Markdown
Contributor

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

@RotemAmit RotemAmit added the ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines label Apr 30, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 30, 2026 •
edited
Loading

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/AWS-GuardDuty/Integrations/AWSGuardDuty
   AWSGuardDuty.py3791695%13–14, 31–32, 34, 249, 332, 363, 365, 367, 375, 379, 404, 425, 429, 709
TOTAL3791695% 

Tests Skipped Failures Errors Time
56 0 💤 0 ❌ 0 🔥 4.208s ⏱️

@RotemAmit
Copy link
Copy Markdown
Contributor Author

RotemAmit commented Apr 30, 2026

@marketplace-ai-reviewer start review

@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

marketplace-ai-reviewer commented Apr 30, 2026

🤖 Analysis started. Please wait for results...

@marketplace-ai-reviewer
Copy link
Copy Markdown
Contributor

marketplace-ai-reviewer commented Apr 30, 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 30, 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.

Hi! Thanks for your contribution to the AWS GuardDuty pack. I've reviewed your PR, and everything looks mostly good, though there is a minor formatting detail to fix regarding the release notes. Thanks for your work on this!

@Moish-Gilboa, @RotemAmit please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

Comment thread Packs/AWS-GuardDuty/ReleaseNotes/1_3_70.md Outdated
yaakovpraisler
yaakovpraisler approved these changes Apr 30, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@yaakovpraisler yaakovpraisler left a comment

Choose a reason for hiding this comment

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

Looks good!

@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: cebde9bab4016c8a

📋 Triage Report

⚠️ AI-generated triage. Validate before acting.

@RotemAmit RotemAmit merged commit 3eb120f into master May 3, 2026
28 of 29 checks passed
@RotemAmit RotemAmit deleted the xsup-68212-aws-guardduty branch May 3, 2026 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

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

@yaakovpraisler yaakovpraisler yaakovpraisler approved these changes

@Moish-Gilboa Moish-Gilboa Awaiting requested review from Moish-Gilboa Moish-Gilboa is a code owner

Assignees

No one assigned

Labels

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.

4 participants

@RotemAmit @content-bot @marketplace-ai-reviewer @yaakovpraisler