Skip to content

Conversation

@universal-itengineer
Copy link
Member

@universal-itengineer universal-itengineer commented Jul 17, 2025

Description

The virt-operator must be placed on the system nodes if they are present in the cluster, otherwise on the master nodes.

Why do we need it, and what problem does it solve?

The previously used template did not produce the expected result.

What is the expected result?

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

Changelog entries

section: core
type: fix
summary: Fixed the placement of the virtualization management component on system nodes when they are present in the cluster. If there are no system nodes in the cluster, it will be placed on master nodes.

@universal-itengineer universal-itengineer added this to the v0.23.0 milestone Jul 17, 2025
@universal-itengineer universal-itengineer force-pushed the chore/core/fix-virt-operator-placement branch 4 times, most recently from fe484a4 to 65e3b4c Compare July 18, 2025 15:04
@universal-itengineer
Copy link
Member Author

image

@universal-itengineer universal-itengineer force-pushed the chore/core/fix-virt-operator-placement branch from b5e2076 to fc988ae Compare July 18, 2025 16:17
@universal-itengineer universal-itengineer marked this pull request as ready for review July 18, 2025 16:17
@universal-itengineer universal-itengineer force-pushed the chore/core/fix-virt-operator-placement branch from fc988ae to 2b2e970 Compare July 18, 2025 16:25
@universal-itengineer universal-itengineer force-pushed the chore/core/fix-virt-operator-placement branch 2 times, most recently from 922671f to 5dd03db Compare July 21, 2025 12:50
Signed-off-by: Nikita Korolev <[email protected]>

add custom template

Signed-off-by: Nikita Korolev <[email protected]>

move virt_helper_node_selector to templates

Signed-off-by: Nikita Korolev <[email protected]>

tst tmpl 1

Signed-off-by: Nikita Korolev <[email protected]>

move tmpl

Signed-off-by: Nikita Korolev <[email protected]>

work

Signed-off-by: Nikita Korolev <[email protected]>

++

Signed-off-by: Nikita Korolev <[email protected]>

format tmpl

Signed-off-by: Nikita Korolev <[email protected]>
Signed-off-by: Nikita Korolev <[email protected]>

fix virt tmpl

Signed-off-by: Nikita Korolev <[email protected]>

fix tmplate

Signed-off-by: Nikita Korolev <[email protected]>

fix template

Signed-off-by: Nikita Korolev <[email protected]>
@universal-itengineer universal-itengineer force-pushed the chore/core/fix-virt-operator-placement branch from 5dd03db to 89e0183 Compare July 21, 2025 12:52
Signed-off-by: Nikita Korolev <[email protected]>
@universal-itengineer universal-itengineer merged commit a47ee4d into main Jul 22, 2025
26 of 27 checks passed
@universal-itengineer universal-itengineer deleted the chore/core/fix-virt-operator-placement branch July 22, 2025 13:48
This was referenced Jul 22, 2025
yachmenevas pushed a commit that referenced this pull request Oct 15, 2025
Description
The virt-operator must be placed on the system nodes if they are present in the cluster, otherwise on the master nodes.

Why do we need it, and what problem does it solve?
The previously used template did not produce the expected result.


---------

Signed-off-by: Nikita Korolev <[email protected]>
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.

4 participants