Skip to content

Extend Qualys integration timeout/XSUP 67858#44246

Merged
RosenbergYehuda merged 8 commits into
masterfrom
YR/Extend-Qualys-integration-timeout/XSUP-67858
May 12, 2026
Merged

Extend Qualys integration timeout/XSUP 67858#44246
RosenbergYehuda merged 8 commits into
masterfrom
YR/Extend-Qualys-integration-timeout/XSUP-67858

Conversation

@RosenbergYehuda
Copy link
Copy Markdown
Contributor

@RosenbergYehuda RosenbergYehuda commented May 10, 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

relates:https://jira-dc.paloaltonetworks.com/browse/XSUP-67858

Description

Increased timeout.

Must have

  • Tests
  • Documentation

@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

@RosenbergYehuda RosenbergYehuda self-assigned this May 10, 2026
@RosenbergYehuda RosenbergYehuda added docs-approved 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. labels May 10, 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 May 10, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 10, 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.

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 Qualys pack. I've reviewed the updates and left a few notes to help polish the changes. The main items involve adding unit tests for the new exception handling, improving the error logging, and making a couple of quick formatting tweaks to the metadata and release notes. Thanks again for your hard work!

Additionally, please address the following file-level notes:

  • Packs/qualys/pack_metadata.json: The vendor name (Qualys) must be included in the keywords list.

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

Comment thread Packs/qualys/Integrations/Qualysv2/Qualysv2.py
Comment thread Packs/qualys/Integrations/Qualysv2/Qualysv2.py Outdated
Comment thread Packs/qualys/ReleaseNotes/3_3_22.md Outdated
@content-bot

This comment has been minimized.

@content-bot
Copy link
Copy Markdown
Contributor

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: success
Report ID: e035f703cd83664b

📋 Triage Report

⚠️ AI-generated triage. Validate before acting.

Comment thread Packs/qualys/Integrations/Qualysv2/Qualysv2.py Outdated
@RosenbergYehuda RosenbergYehuda enabled auto-merge (squash) May 12, 2026 14:32
@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? ✅

@RosenbergYehuda RosenbergYehuda merged commit 1d9bc33 into master May 12, 2026
21 of 23 checks passed
@RosenbergYehuda RosenbergYehuda deleted the YR/Extend-Qualys-integration-timeout/XSUP-67858 branch May 12, 2026 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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

Development

Successfully merging this pull request may close these issues.

4 participants