Skip to content

#1694 : Extract only first IPv4 address when displaying and editing VM IP addresses#1695

Merged
sarika-pf9 merged 2 commits intomainfrom
1692-preserve-ip-for-a-powered-on-vm-is-not-getting-ips-for-multi-ip-interfaces
Mar 16, 2026
Merged

#1694 : Extract only first IPv4 address when displaying and editing VM IP addresses#1695
sarika-pf9 merged 2 commits intomainfrom
1692-preserve-ip-for-a-powered-on-vm-is-not-getting-ips-for-multi-ip-interfaces

Conversation

@AbhijeetThakur
Copy link
Collaborator

@AbhijeetThakur AbhijeetThakur commented Mar 16, 2026

What this PR does / why we need it

In the case of multiple IPs, the additional field for all IPs was added unnecessarily, which created confusion. I have now removed it.

Which issue(s) this PR fixes

fixes #

Special notes for your reviewer

For your reference, the migration-system/vmwaremachines API is sending separate entries for each network interface.
image
image

Testing done

Screencast.from.16-03-26.02.11.34.PM.IST.webm

Copy link
Contributor

@windsurf-bot windsurf-bot bot left a comment

Choose a reason for hiding this comment

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

💡 To request another review, post a new comment with "/windsurf-review".

@sarika-pf9 sarika-pf9 merged commit ca28128 into main Mar 16, 2026
12 checks passed
@sarika-pf9 sarika-pf9 deleted the 1692-preserve-ip-for-a-powered-on-vm-is-not-getting-ips-for-multi-ip-interfaces branch March 16, 2026 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Preserve IP for a powered on VM is not getting IPs for multi IP interfaces

2 participants