Skip to content

Tz resolve xsup 75189 - #45615

Merged
talzich merged 4 commits into
masterfrom
tz-resolve-xsup-75189
Aug 25, 2026
Merged

Tz resolve xsup 75189#45615
talzich merged 4 commits into
masterfrom
tz-resolve-xsup-75189

Conversation

@talzich

@talzich talzich commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

Fixes: https://jira-dc.paloaltonetworks.com/browse/XSUP-75189

Description

This PR resolve xsup-75189, an issue where the update-issue command did not actually updated the issue on the XDR side.

@talzich
talzich requested a review from AradCarmi August 23, 2026 11:13
@talzich
talzich requested a review from Shellyber as a code owner August 23, 2026 11:13
@talzich talzich 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 Aug 23, 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

@github-actions

github-actions Bot commented Aug 23, 2026

Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/CortexXDR/Integrations/CortexXDRIR
   CortexXDRIR.py137117887%90, 98, 102–104, 192–193, 276–280, 282, 284, 287, 292, 319–323, 325, 332–335, 354, 357, 363, 366, 399–403, 405, 413, 415, 417–420, 423–427, 429, 550, 563–566, 568, 613, 618, 621, 626, 629, 634, 637, 642, 645, 650, 653, 658, 661–662, 666–668, 671, 676, 679, 683, 686, 690, 693, 698, 701, 707, 712, 715, 726, 731, 738, 742, 749, 754, 761, 766, 779, 784, 797, 807, 824, 841, 856, 863, 868, 871, 876, 880, 883, 888, 891, 896, 899, 975, 1022–1023, 1042, 1052, 1117, 1120, 1153, 1160, 1162, 1226–1227, 1229, 1280, 1361, 1482–1483, 1486, 1536–1537, 1558, 1599–1602, 1658, 1675–1677, 1682, 1687, 1715–1718, 1750, 1766, 1879–1880, 1982–1983, 2062–2063, 2156–2157, 2182–2183, 2311–2312, 2443–2444, 2492–2495, 2507, 2509, 2561–2562, 2919–2920, 2928, 2930, 3052, 3060, 3199–3200, 3211–3212, 3345
TOTAL137117887% 

Tests Skipped Failures Errors Time
225 0 💤 0 ❌ 0 🔥 16.027s ⏱️

@talzich
talzich force-pushed the tz-resolve-xsup-75189 branch from df62745 to e34d836 Compare August 23, 2026 11:19
@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.

Thank you for your contribution! No issues found. Great work! 👍

@Shellyber, @AradCarmi, @talzich please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

@AradCarmi AradCarmi 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.

Nice!

Comment thread Packs/CortexXDR/ReleaseNotes/6_3_34.md Outdated
Comment thread Packs/CortexXDR/Integrations/CortexXDRIR/CortexXDRIR.yml
Co-authored-by: Arad Carmi <62752352+AradCarmi@users.noreply.github.com>
@talzich
talzich requested a review from AradCarmi August 23, 2026 12:31
@content-bot

This comment has been minimized.

@content-bot

Copy link
Copy Markdown
Contributor

Validate summary
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: success
Report ID: ec3582d8c6e470df

📋 Triage Report

⚠️ AI-generated triage. Validate before acting.

@AradCarmi AradCarmi 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.

Great success!

@talzich
talzich merged commit 3d6442d into master Aug 25, 2026
25 checks passed
@talzich
talzich deleted the tz-resolve-xsup-75189 branch August 25, 2026 07:53
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