Skip to content

Virtualization Story Update - new_vm and old_vm xdm field to vm.hostname#42937

Merged
akshotiamit-pa merged 7 commits intomasterfrom
CRTX-223734-SupportOldNewVM
Feb 5, 2026
Merged

Virtualization Story Update - new_vm and old_vm xdm field to vm.hostname#42937
akshotiamit-pa merged 7 commits intomasterfrom
CRTX-223734-SupportOldNewVM

Conversation

@akshotiamit-pa
Copy link
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

@content-bot
Copy link
Contributor

🤖 AI-Powered Code Review Available

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

@akshotiamit-pa akshotiamit-pa added docs-approved ForceMerge Forcing the merge of the PR despite the build status bypass.url Whether to create build bucket, add this label for marketplace.bootstrap.bypass.url 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 Feb 5, 2026
@akshotiamit-pa
Copy link
Contributor Author

@content-bot start review

@akshotiamit-pa akshotiamit-pa removed the ForceMerge Forcing the merge of the PR despite the build status label Feb 5, 2026
@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 the contribution! I've reviewed the changes and have a few suggestions regarding the release notes and modeling rule. Please ensure the release note descriptions accurately match the variable names used in the code and apply proper formatting to field names. There is also a suggestion to improve the regex robustness in the XIF file.

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

@content-bot
Copy link
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? ✅

@akshotiamit-pa akshotiamit-pa merged commit 43f58a4 into master Feb 5, 2026
21 checks passed
@akshotiamit-pa akshotiamit-pa deleted the CRTX-223734-SupportOldNewVM branch February 5, 2026 16:41
inbalapt1 pushed a commit that referenced this pull request Feb 8, 2026
…ame (#42937)

* update new_vm and old_vm xdm field to vm.hostname instead of resorce name and before_name

* update rn

* Trigger AI Reviewer

* Removed 2.9 from release note headline

* update rn

* add 2.9 to rn

---------

Co-authored-by: Content Bot <Content-Bot@users.noreply.github.com>
Co-authored-by: oatias <oatias@paloaltonetworks.com>
sdaniel6 pushed a commit that referenced this pull request Feb 9, 2026
…ame (#42937)

* update new_vm and old_vm xdm field to vm.hostname instead of resorce name and before_name

* update rn

* Trigger AI Reviewer

* Removed 2.9 from release note headline

* update rn

* add 2.9 to rn

---------

Co-authored-by: Content Bot <Content-Bot@users.noreply.github.com>
Co-authored-by: oatias <oatias@paloaltonetworks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bypass.url Whether to create build bucket, add this label for marketplace.bootstrap.bypass.url 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