Skip to content

Prepare Multi Provider Setup - #483

Merged
sven-rosenzweig merged 1 commit into
mainfrom
feat/introduce_device_provider_field
Aug 6, 2026
Merged

Prepare Multi Provider Setup#483
sven-rosenzweig merged 1 commit into
mainfrom
feat/introduce_device_provider_field

Conversation

@sven-rosenzweig

Copy link
Copy Markdown
Contributor

The network-operator currently requires a single provider to be selected at startup via the --provider flag. Every controller receives the same ProviderFunc and calls it unconditionally.

The goal is to move provider selection from a startup flag to the Device resource, so a single operator deployment can manage devices running different operating systems. Therefore, a new field provider is introduced.

After migration this field will become required, once rolled out and introduced everywhere. Provider flag will be removed.

@sven-rosenzweig
sven-rosenzweig force-pushed the feat/introduce_device_provider_field branch 2 times, most recently from b7790bb to a97603d Compare August 6, 2026 12:32
@github-actions github-actions Bot added size/S and removed size/XS labels Aug 6, 2026
@sven-rosenzweig
sven-rosenzweig marked this pull request as ready for review August 6, 2026 13:19
@sven-rosenzweig sven-rosenzweig changed the title feat: Prepare Multi Provider Setup Prepare Multi Provider Setup Aug 6, 2026
The network-operator currently requires a single provider
to be selected at startup via the --provider flag.
Every controller receives the same ProviderFunc and calls
it unconditionally.

The goal is to move provider selection from a startup flag
to the Device resource, so a single operator deployment can
manage devices running different operating systems.
Therefore, a new field `provider` is introduced.

After migration this field will become required, once
rolled out and introduced everywhere. Provider flag will
be removed.

Signed-off-by: Sven Rosenzweig <sven.rosenzweig@sap.com>
@sven-rosenzweig
sven-rosenzweig force-pushed the feat/introduce_device_provider_field branch from a97603d to ea3bf8e Compare August 6, 2026 13:50
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

Merging this branch will not change overall coverage

Impacted Packages Coverage Δ 🤖
github.com/ironcore-dev/network-operator/api/core/v1alpha1 1.72% (ø)

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/ironcore-dev/network-operator/api/core/v1alpha1/device_types.go 48.48% (ø) 33 16 17

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

@sven-rosenzweig
sven-rosenzweig merged commit 82d64f5 into main Aug 6, 2026
24 checks passed
@sven-rosenzweig
sven-rosenzweig deleted the feat/introduce_device_provider_field branch August 6, 2026 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants