Skip to content

feat(vm): add webhook to validate disk node availability before attach#2132

Open
loktev-d wants to merge 6 commits intofix/vm/pv-node-affinity-schedulingfrom
feat/vm/disk-node-availability-webhook
Open

feat(vm): add webhook to validate disk node availability before attach#2132
loktev-d wants to merge 6 commits intofix/vm/pv-node-affinity-schedulingfrom
feat/vm/disk-node-availability-webhook

Conversation

@loktev-d
Copy link
Copy Markdown
Contributor

@loktev-d loktev-d commented Mar 23, 2026

Description

Add validating webhooks for VM and VMBDA that reject disk attachment if the disk's PV nodeAffinity doesn't match the node where the VM is running.

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

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: vm
type: feature
summary: add webhook to validate disk node availability before attach

@loktev-d loktev-d marked this pull request as draft March 23, 2026 14:38
@loktev-d loktev-d changed the title wip feat(vm): add webhook to validate disk node availability before attach Mar 24, 2026
@loktev-d loktev-d added this to the v1.6.2 milestone Mar 24, 2026
@loktev-d loktev-d marked this pull request as ready for review March 24, 2026 12:21
@Isteb4k Isteb4k changed the base branch from feat/vm/hotplug-block-devices-via-spec to fix/vm/pv-node-affinity-scheduling March 24, 2026 13:05
Signed-off-by: Daniil Loktev <lokt.daniil@gmail.com>
Signed-off-by: Daniil Loktev <lokt.daniil@gmail.com>
Signed-off-by: Daniil Loktev <lokt.daniil@gmail.com>
Signed-off-by: Daniil Loktev <lokt.daniil@gmail.com>
@Isteb4k Isteb4k force-pushed the feat/vm/disk-node-availability-webhook branch from f7f57c2 to 9fb758b Compare March 24, 2026 13:13
@nevermarine nevermarine modified the milestones: v1.6.2, v1.7.0 Mar 25, 2026
Signed-off-by: Daniil Loktev <lokt.daniil@gmail.com>
Signed-off-by: Daniil Loktev <lokt.daniil@gmail.com>
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.

2 participants