Skip to content

Fix missing return and variable shadowing in RemoteMachine controller - #1369

Open
kahirokunn wants to merge 2 commits into
k0sproject:mainfrom
kahirokunn:fix/remote-machine-controller-bugs
Open

Fix missing return and variable shadowing in RemoteMachine controller#1369
kahirokunn wants to merge 2 commits into
k0sproject:mainfrom
kahirokunn:fix/remote-machine-controller-bugs

Conversation

@kahirokunn

@kahirokunn kahirokunn commented Feb 25, 2026

Copy link
Copy Markdown
Contributor

The RemoteMachine controller has several issues:

  • A redundant defer that patches with a potentially cancelled context
  • A missing return after the paused cluster check, causing fall-through into provisioning logic
  • Context and err variable shadowing leading to unintended behavior
  • Ready status not being set when ProviderID is already present

@kahirokunn
kahirokunn requested a review from a team as a code owner February 25, 2026 04:28
@kahirokunn kahirokunn changed the title Fix critical bugs in RemoteMachine controller reconciliation Fix missing return and variable shadowing in RemoteMachine controller Feb 25, 2026
@kahirokunn
kahirokunn force-pushed the fix/remote-machine-controller-bugs branch 2 times, most recently from 62d67b0 to e68ff3d Compare February 25, 2026 05:26
Signed-off-by: kahirokunn <okinakahiro@gmail.com>
Signed-off-by: kahirokunn <okinakahiro@gmail.com>
@kahirokunn
kahirokunn force-pushed the fix/remote-machine-controller-bugs branch from e68ff3d to 0f78267 Compare February 25, 2026 05:27
@github-actions

Copy link
Copy Markdown

The PR is marked as stale since no activity has been recorded in 30 days

@github-actions github-actions Bot added Stale and removed Stale labels Mar 27, 2026
@github-actions

Copy link
Copy Markdown

The PR is marked as stale since no activity has been recorded in 30 days

@github-actions github-actions Bot added Stale and removed Stale labels Apr 29, 2026
@github-actions

Copy link
Copy Markdown

The PR is marked as stale since no activity has been recorded in 30 days

@github-actions github-actions Bot added Stale and removed Stale labels May 31, 2026
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

The PR is marked as stale since no activity has been recorded in 30 days

@github-actions

Copy link
Copy Markdown

This pull request has merge conflicts that need to be resolved.

@github-actions

Copy link
Copy Markdown

The PR is marked as stale since no activity has been recorded in 30 days

@github-actions github-actions Bot added Stale and removed Stale labels Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant