Skip to content

V3.0.2 remove logs#42941

Open
viktor-orca wants to merge 4 commits intodemisto:contrib/viktor-orca_v3.0.2-remove-logsfrom
viktor-orca:v3.0.2-remove-logs
Open

V3.0.2 remove logs#42941
viktor-orca wants to merge 4 commits intodemisto:contrib/viktor-orca_v3.0.2-remove-logsfrom
viktor-orca:v3.0.2-remove-logs

Conversation

@viktor-orca
Copy link
Contributor

@viktor-orca viktor-orca commented Feb 5, 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

Must have

  • Tests
  • Documentation

relates: https://jira-dc.paloaltonetworks.com/browse/CIAC-15714

@content-bot content-bot added Contribution Thank you! Contributions are always welcome! External PR Partner Support Level Indicates that the contribution is for Partner supported pack labels Feb 5, 2026
@content-bot content-bot changed the base branch from master to contrib/viktor-orca_v3.0.2-remove-logs February 5, 2026 11:40
@content-bot content-bot requested a review from kamalq97 February 5, 2026 11:40
@content-bot
Copy link
Contributor

Thank you for your contribution. Your generosity and caring are unrivaled! Make sure to register your contribution by filling the Contribution Registration form, so our content wizard @kamalq97 will know the proposed changes are ready to be reviewed.
For your convenience, here is a link to the contributions SLAs document.

@content-bot
Copy link
Contributor

Hi @viktor-orca, thanks for contributing to the XSOAR marketplace. To receive credit for your generous contribution please follow this link.

@content-bot
Copy link
Contributor

🤖 AI-Powered Code Review Available

Hi @kamalq97, 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

@kamalq97 kamalq97 added the ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR. label Feb 5, 2026
@kamalq97
Copy link
Contributor

kamalq97 commented Feb 5, 2026

@content-bot start review

@content-bot
Copy link
Contributor

For the Reviewer: Trigger build request has been accepted for this contribution PR.

@content-bot
Copy link
Contributor

For the Reviewer: Successfully created a pipeline in GitLab with url: https://gitlab.xdr.pan.local/xdr/cortex-content/content/-/pipelines/7025903

@content-bot content-bot added ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. and removed ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR. labels Feb 5, 2026
@content-bot content-bot added Partner-Approved Contribution Form Filled Whether contribution form filled or not. Partner labels Feb 5, 2026
@content-bot
Copy link
Contributor

Validate summary
The following errors were reported as warnings: DO106.
The following errors were thrown as a part of this pr: RN106.
The following errors cannot be ignored: RN106.
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: RN106.

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

@content-bot content-bot removed the ready-for-ai-review The PR is ready for reviewing the PR with the AI Reviewer. label Feb 5, 2026
@content-bot
Copy link
Contributor

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

@content-bot
Copy link
Contributor

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

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 submitting your PR!
Before moving on with the detailed review, please take a moment to address the following general comments.

  • The PR title mentions removing logs, but the code changes actually remove the logic for storing imported_ids in the last_run context (state management). Please update the PR title and description to accurately reflect that you are modifying how incident IDs are stored/tracked, as this is a functional change rather than just removing log statements.

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

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

Labels

Contribution Form Filled Whether contribution form filled or not. Contribution Thank you! Contributions are always welcome! External PR Partner Support Level Indicates that the contribution is for Partner supported pack Partner Partner-Approved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants