Skip to content

refactor: share multi-disk sdrs placement helper#771

Merged
tenthirtyam merged 1 commit into
mainfrom
refactor/share-multi-disk-sdrs-placement-helper
Jul 10, 2026
Merged

refactor: share multi-disk sdrs placement helper#771
tenthirtyam merged 1 commit into
mainfrom
refactor/share-multi-disk-sdrs-placement-helper

Conversation

@tenthirtyam

Copy link
Copy Markdown
Collaborator

Summary

Move shared Storage DRS datastore selection logic into common.ResolveMultiDiskDatastoreRefs and use it from both clone and ISO create flows. This keeps multi-disk placement behavior consistent while preserving the existing fallback to the primary datastore when recommendations fail.

Type

  • fix: Bug Fix
  • feat: Feature or Enhancement
  • docs: Documentation
  • refactor: Refactoring
  • chore: Build, Dependencies, Workflows, etc.
  • other: Other (Please describe.)

Breaking Changes?

  • Yes, there are breaking changes.
  • No, there are no breaking changes.

Tests

  • Tests have been added or updated.
  • Tests have been completed.

Output:

Documentation

  • Documentation has been added or updated.

Issue References

Release Note

Additional Information

Move shared Storage DRS datastore selection logic into `common.ResolveMultiDiskDatastoreRefs` and use it from both clone and ISO create flows. This keeps multi-disk placement behavior consistent while preserving the existing fallback to the primary datastore when recommendations fail.

Signed-off-by: Ryan Johnson <ryan@tenthirtyam.org>
@tenthirtyam tenthirtyam added this to the v2.3.0 milestone Jul 10, 2026
@tenthirtyam tenthirtyam self-assigned this Jul 10, 2026
@tenthirtyam tenthirtyam requested a review from a team as a code owner July 10, 2026 16:34
@tenthirtyam tenthirtyam added the refactor Refactor label Jul 10, 2026
@github-actions github-actions Bot added needs-review Needs Review size/m Relative Sizing: Medium labels Jul 10, 2026
@tenthirtyam tenthirtyam merged commit 15a023c into main Jul 10, 2026
10 checks passed
@tenthirtyam tenthirtyam deleted the refactor/share-multi-disk-sdrs-placement-helper branch July 10, 2026 16:38
@tenthirtyam tenthirtyam removed the needs-review Needs Review label Jul 10, 2026
tenthirtyam added a commit that referenced this pull request Jul 10, 2026
Move shared Storage DRS datastore selection logic into `common.ResolveMultiDiskDatastoreRefs` and use it from both clone and ISO create flows. This keeps multi-disk placement behavior consistent while preserving the existing fallback to the primary datastore when recommendations fail.

Signed-off-by: Ryan Johnson <ryan@tenthirtyam.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Refactor size/m Relative Sizing: Medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant