Skip to content

fix(vd): fix data volume watcher#2292

Merged
Isteb4k merged 1 commit intomainfrom
fix/vd/dv-watcher
Apr 28, 2026
Merged

fix(vd): fix data volume watcher#2292
Isteb4k merged 1 commit intomainfrom
fix/vd/dv-watcher

Conversation

@Isteb4k
Copy link
Copy Markdown
Contributor

@Isteb4k Isteb4k commented Apr 28, 2026

Description

Fix data volume watcher.

The data volume entered the WFFC phase, but there was no watcher this event. As a result, the controller didn’t know that the data volume was ready and didn’t transition the disk into the WFFC phase.

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: vd
type: fix
summary: Fixed a potential issue where a virtual disk could get stuck in the Provisioning state when using a storage class with the WaitForFirstConsumer mode.

Signed-off-by: Dmitry Rakitin <dmitry.rakitin@flant.com>
@Isteb4k Isteb4k force-pushed the fix/vd/dv-watcher branch from fabc283 to ad5008e Compare April 28, 2026 12:45
@Isteb4k Isteb4k added this to the v1.8.1 milestone Apr 28, 2026
@Isteb4k Isteb4k marked this pull request as ready for review April 28, 2026 13:36
@Isteb4k Isteb4k requested a review from yaroslavborbat as a code owner April 28, 2026 13:36
@Isteb4k Isteb4k requested a review from danilrwx April 28, 2026 13:37
@Isteb4k Isteb4k merged commit fe92284 into main Apr 28, 2026
33 of 37 checks passed
@Isteb4k Isteb4k deleted the fix/vd/dv-watcher branch April 28, 2026 15:01
Isteb4k added a commit that referenced this pull request Apr 28, 2026
Fix data volume watcher.

The data volume entered the WFFC phase, but there was no watcher this event. As a result, the controller didn’t know that the data volume was ready and didn’t transition the disk into the WFFC phase.

Signed-off-by: Dmitry Rakitin <dmitry.rakitin@flant.com>
(cherry picked from commit fe92284)
This was referenced Apr 28, 2026
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