Skip to content

Fix: VJailbreakNode CR status was stuck at "CreatingVM" for L2 networks even after the VM joined the K8s cluster. #1693

Merged
spai-p9 merged 1 commit intomainfrom
private/main/ghi-1680
Mar 16, 2026
Merged

Fix: VJailbreakNode CR status was stuck at "CreatingVM" for L2 networks even after the VM joined the K8s cluster. #1693
spai-p9 merged 1 commit intomainfrom
private/main/ghi-1680

Conversation

@spai-p9
Copy link
Collaborator

@spai-p9 spai-p9 commented Mar 16, 2026

What this PR does / why we need it

For L2 network, IP is assigned by the user and this won't be reflected on the port. hence we won't get the ip from pcd, hence once the agent joins the cluster get the ip from k3s apis and update the vjailbreaknode CR's.

Which issue(s) this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged)

fixes #1680

Special notes for your reviewer

image

Testing done

please add testing details (logs, screenshots, etc.)

@spai-p9 spai-p9 marked this pull request as ready for review March 16, 2026 16:13
@spai-p9 spai-p9 requested a review from OmkarDeshpande7 March 16, 2026 16:13
@spai-p9 spai-p9 force-pushed the private/main/ghi-1680 branch from 261766e to 540c62e Compare March 16, 2026 16:13
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.

Looks good to me 🤙

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

Copy link
Collaborator

@OmkarDeshpande7 OmkarDeshpande7 left a comment

Choose a reason for hiding this comment

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

lgtm

@spai-p9 spai-p9 merged commit caefd3d into main Mar 16, 2026
12 checks passed
@spai-p9 spai-p9 deleted the private/main/ghi-1680 branch March 16, 2026 17:29
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.

[L2] Agent VM status is not updated to Ready/Created and remains stuck at CreatingVM

2 participants