Cleanup NICs when migrate from openstack MCM by dergeberl · Pull Request #89 · stackitcloud/machine-controller-manager-provider-stackit

dergeberl · 2026-01-22T09:55:35Z

How to categorize this PR?

/kind enhancement

/hold
wip
What this PR does / why we need it:

This PR involves migration of machines from OpenStack to STACKIT provider. There are mainly two controller involved in the change:

Creation
The newly created machine is created with STACKIT provider ID (stackit://project-id/server-id), and for the migrated machines with stackit.cloud/migrated-machine: true annotaion the creation will not work, to be sure not to mess with half created machines.

Delete
While deleting machine it needs to list and delete all ports if a machine with the stackit.cloud/migrated-machine annotation gets deleted.

NOTE: There is a fallback to get the server by name in case there is no providerID during deletion. Normally this is done with a label containing the machine name and a label selector. In case of a migrated machine with the stackit.cloud/migrated-machine annotation the deletion needs to get all servers and filters internally.

Special notes for your reviewer:

Breaking changes:

ske-prow · 2026-01-22T09:55:41Z

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign nschad for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

stackit-ske · 2026-02-06T16:52:41Z

Our project currently lacks enough active contributors to adequately respond to all PRs.
This bot triages PRs according to the following rules:

After 15d of inactivity, lifecycle/stale is applied
After 15d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
After 7d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

Mark this PR as fresh with /remove-lifecycle stale
Mark this PR as rotten with /lifecycle rotten
Close this PR with /close

/lifecycle stale

stackit-ske · 2026-02-21T17:44:48Z

Our project currently lacks enough active contributors to adequately respond to all PRs.
This bot triages PRs according to the following rules:

After 15d of inactivity, lifecycle/stale is applied
After 15d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
After 7d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

Mark this PR as fresh with /remove-lifecycle rotten
Close this PR with /close

/lifecycle rotten

stackit-ske · 2026-02-28T18:05:41Z

Our project currently lacks enough active contributors to adequately respond to all PRs.
This bot triages PRs according to the following rules:

After 15d of inactivity, lifecycle/stale is applied
After 15d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
After 7d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

Reopen this PR with /reopen
Mark this PR as fresh with /remove-lifecycle rotten

/close

ske-prow · 2026-02-28T18:05:45Z

@stackit-ske: Closed this PR.

Details

In response to this:

Our project currently lacks enough active contributors to adequately respond to all PRs.
This bot triages PRs according to the following rules:

After 15d of inactivity, lifecycle/stale is applied

After 15d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied

After 7d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

Reopen this PR with /reopen

Mark this PR as fresh with /remove-lifecycle rotten

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

dergeberl · 2026-03-02T07:35:23Z

/reopen

ske-prow · 2026-03-02T07:35:27Z

@dergeberl: Reopened this PR.

Details

In response to this:

/reopen

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

dergeberl · 2026-03-02T07:35:39Z

/remove-lifecycle rotten

stackit-ske · 2026-03-17T08:56:02Z

Our project currently lacks enough active contributors to adequately respond to all PRs.
This bot triages PRs according to the following rules:

After 15d of inactivity, lifecycle/stale is applied
After 15d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
After 7d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

Mark this PR as fresh with /remove-lifecycle stale
Mark this PR as rotten with /lifecycle rotten
Close this PR with /close

/lifecycle stale

dergeberl · 2026-03-20T10:56:39Z

/remove-lifecycle stale

# Conflicts: # pkg/client/sdk.go # pkg/client/stackit.go # pkg/provider/core.go

nschad · 2026-09-07T19:04:54Z

@aniruddha2000 Please add A LOT of documentation explaining this PR and the individual functions

nschad · 2026-09-11T11:31:32Z

 	}
-	return false, nil
+
+	return deleted, nil


simply set deleted to true when allErrors is nil

i have removed the deleted var, its less complex to just retun the true & false :)

ske-prow Bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. kind/enhancement Enhancement, improvement, extension do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels Jan 22, 2026

ske-prow Bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jan 22, 2026

dergeberl force-pushed the removeNicsOnDeletion branch 3 times, most recently from d542196 to cf02b5c Compare January 22, 2026 11:31

ske-prow Bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jan 22, 2026

ske-prow Bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Feb 6, 2026

ske-prow Bot added lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Feb 21, 2026

ske-prow Bot closed this Feb 28, 2026

ske-prow Bot reopened this Mar 2, 2026

ske-prow Bot removed the lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. label Mar 2, 2026

nschad reviewed Mar 2, 2026

View reviewed changes

Comment thread pkg/client/sdk.go Outdated

ske-prow Bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Mar 17, 2026

ske-prow Bot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Mar 20, 2026

dergeberl added the lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. label Mar 20, 2026

aniruddha2000 force-pushed the removeNicsOnDeletion branch from cf02b5c to d47b553 Compare August 27, 2026 09:25

ske-prow Bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Aug 27, 2026

aniruddha2000 force-pushed the removeNicsOnDeletion branch from 55364f4 to 1e553c4 Compare August 27, 2026 11:01

ske-prow Bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Aug 27, 2026

aniruddha2000 changed the title ~~WIP: Cleanup NICs when migrate from openstack MCM~~ Cleanup NICs when migrate from openstack MCM Sep 7, 2026

ske-prow Bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 7, 2026

nschad requested a review from stackit-ske-bot September 7, 2026 11:43

dergeberl and others added 5 commits September 7, 2026 13:52

WIP

7e0a67c

# Conflicts: # pkg/client/sdk.go # pkg/client/stackit.go # pkg/provider/core.go

fix core.go rebase isssues

6be6d0a

add tests for the create and delete machine

6999aa0

fix make verify

fe038e0

fix make verify

17def4f

aniruddha2000 force-pushed the removeNicsOnDeletion branch from 5b85955 to 17def4f Compare September 7, 2026 12:16

nschad requested changes Sep 7, 2026

View reviewed changes

aniruddha2000 added 3 commits September 8, 2026 14:29

resolve partial review

95cc8ce

fix make check

297978b

remove error check to have OS compatibilty

bd1f1ff

nschad reviewed Sep 10, 2026

View reviewed changes

Comment thread pkg/provider/delete.go Outdated

aniruddha2000 added 2 commits September 10, 2026 14:54

remove error shadow

5e8df25

add docstring

b47d91d

nschad requested changes Sep 11, 2026

View reviewed changes

Comment thread pkg/provider/delete.go

Comment thread pkg/provider/delete.go

aniruddha2000 added 2 commits September 11, 2026 11:48

collect errors and return one time for servers and nics

e776b6d

fix test

54cba9a

nschad reviewed Sep 11, 2026

View reviewed changes

remove deleted var

b8fb2b0

Uh oh!

Conversation

dergeberl commented Jan 22, 2026 • edited by aniruddha2000 Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ske-prow Bot commented Jan 22, 2026

Uh oh!

stackit-ske commented Feb 6, 2026

Uh oh!

stackit-ske commented Feb 21, 2026

Uh oh!

stackit-ske commented Feb 28, 2026

Uh oh!

ske-prow Bot commented Feb 28, 2026

Uh oh!

dergeberl commented Mar 2, 2026

Uh oh!

ske-prow Bot commented Mar 2, 2026

Uh oh!

dergeberl commented Mar 2, 2026

Uh oh!

Uh oh!

stackit-ske commented Mar 17, 2026

Uh oh!

dergeberl commented Mar 20, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

nschad commented Sep 7, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

nschad Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

aniruddha2000 Sep 11, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

dergeberl commented Jan 22, 2026 •

edited by aniruddha2000

Loading