Skip to content

Update docker - #45588

Open
BEAdi wants to merge 12 commits into
masterfrom
AUD-demisto/xsup-71787
Open

Update docker#45588
BEAdi wants to merge 12 commits into
masterfrom
AUD-demisto/xsup-71787

Conversation

@BEAdi

@BEAdi BEAdi commented Aug 20, 2026

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

Description

Must have

  • Tests
  • Documentation

@BEAdi BEAdi self-assigned this Aug 20, 2026
@BEAdi
BEAdi requested a review from Ni-Knight as a code owner August 20, 2026 12:17
@BEAdi BEAdi added the ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines label Aug 20, 2026
@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

This comment has been minimized.

@BEAdi BEAdi added the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Aug 23, 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 Aug 23, 2026
@marketplace-ai-reviewer

Copy link
Copy Markdown
Contributor

🤖 Analysis started. Please wait for results...

@marketplace-ai-reviewer

Copy link
Copy Markdown
Contributor
🤖 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 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for your contribution! I've reviewed your updates and noticed a few missing fields in the pack metadata files. Specifically, the created key is missing from a couple of packs, and the vendor name needs to be added to the keywords list for the Smokescreen pack. Let me know if you have any questions while fixing these up!

Additionally, please address the following file-level notes:

  • Packs/Smokescreen_IllusionBLACK/pack_metadata.json: The vendor name (e.g., Smokescreen) must be included in the keywords list.
  • Packs/Core/pack_metadata.json: The mandatory key created is missing from the pack metadata.
  • Packs/FiltersAndTransformers/pack_metadata.json: The mandatory key created is missing from the pack metadata.

@jlevypaloalto, @Ni-Knight, @Shellyber, @BEAdi please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

@content-bot

This comment has been minimized.

@content-bot

Copy link
Copy Markdown
Contributor

This PR was automatically updated by a GitHub Action

  • CommonScripts pack version was bumped to 1.22.52.
  • Core pack version was bumped to 3.5.87.

To stop automatic version bumps, add the ignore-auto-bump-version label to the github PR.

@content-bot

This comment has been minimized.

@content-bot

Copy link
Copy Markdown
Contributor

This PR was automatically updated by a GitHub Action

  • Core pack version was bumped to 3.5.88.

To stop automatic version bumps, add the ignore-auto-bump-version label to the github PR.

@content-bot

This comment has been minimized.

@BEAdi

BEAdi commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

Force merge-
The "Create Phishing Classifier V2 ML Test" TPB is flaky.
The "ExifReadTest" TPB also failed without my changes #45628.

@BEAdi BEAdi added the ForceMerge Forcing the merge of the PR despite the build status label Aug 25, 2026
@content-bot

This comment has been minimized.

@content-bot

Copy link
Copy Markdown
Contributor

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

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: b308ad2dd859ead7

📋 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

auto-update-docker docs-approved ForceMerge Forcing the merge of the PR despite the build status ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants