Skip to content

vlcluster,vtcluster: do not ignore extraStorageNode, when default storage is not enabled#1911

Open
AndrewChubatiuk wants to merge 1 commit intomasterfrom
fix-extra-storage-node-when-no-storage
Open

vlcluster,vtcluster: do not ignore extraStorageNode, when default storage is not enabled#1911
AndrewChubatiuk wants to merge 1 commit intomasterfrom
fix-extra-storage-node-when-no-storage

Conversation

@AndrewChubatiuk
Copy link
Contributor

@AndrewChubatiuk AndrewChubatiuk commented Mar 2, 2026

fixes #1910


Summary by cubic

Fixes VLSelect and VTSelect ignoring ExtraStorageNodes when default storage is disabled, so select pods connect to the provided storage nodes. Also adds validation to prevent duplicate storageNode addresses. Addresses issue #1910.

  • Bug Fixes
    • Build the -storageNode flag and collect storage node IDs outside the default-storage check.
    • Append -storageNode args only when at least one node is present; include ExtraStorageNodes even without default storage.
    • Add CR validation to reject duplicate storageNode addresses from ExtraArgs and ExtraStorageNodes.

Written for commit f0dfa99. Summary will update on new commits.

Loading
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.

VLCluster: extraStorageNodes silently ignored without in-cluster vlstorage

3 participants