Skip to content

Conversation

@mandre
Copy link
Collaborator

@mandre mandre commented Sep 12, 2025

Add a controller for managing volumes, using the new controller generator (#506).

This replaces #474.

For #473.

@mandre mandre mentioned this pull request Sep 12, 2025
@github-actions github-actions bot added the semver:major Breaking change label Sep 12, 2025
@mandre mandre force-pushed the volume-controller-v2 branch 3 times, most recently from 6cb32df to 522e201 Compare September 15, 2025 12:37
@mandre mandre added this to the Release 2.3 milestone Sep 16, 2025
@mandre mandre force-pushed the volume-controller-v2 branch 2 times, most recently from fdaa487 to 18affab Compare September 17, 2025 08:31
@mandre mandre force-pushed the volume-controller-v2 branch 2 times, most recently from 0df0e90 to 3fe09d7 Compare September 26, 2025 12:00
@mandre mandre added this pull request to the merge queue Sep 30, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to invalid changes in the merge commit Sep 30, 2025
Scaffold the volume controller using the new controller generator:

    ❯ go run ./cmd/scaffold-controller -interactive=false \
        -kind Volume \
	-gophercloud-client NewBlockStorageV3 \
	-gophercloud-module github.com/gophercloud/gophercloud/v2/openstack/blockstorage/v3/volumes \
	-available-polling-period 15 \
	-deleting-polling-period 15 \
	-optional-create-depencency VolumeType
Add a new mutable `volumes` fields in the server resource spec, allowing
to control volumes attached to the server.
@mandre mandre force-pushed the volume-controller-v2 branch from 3fe09d7 to 32c6b30 Compare September 30, 2025 06:47
@mandre mandre enabled auto-merge September 30, 2025 06:47
@mandre mandre added this pull request to the merge queue Sep 30, 2025
Merged via the queue into k-orc:main with commit 26cb101 Sep 30, 2025
8 checks passed
@mandre mandre deleted the volume-controller-v2 branch September 30, 2025 07:26
mandre added a commit to shiftstack/openstack-resource-controller that referenced this pull request Sep 30, 2025
This should have been part of
k-orc#508.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver:major Breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants