Skip to content

feat(blockstorage): Cinder volume resource and volume/snapshot data sources - #7

Merged
PF9-pushkar merged 1 commit into
mainfrom
feat/blockstorage
Jul 12, 2026
Merged

feat(blockstorage): Cinder volume resource and volume/snapshot data sources#7
PF9-pushkar merged 1 commit into
mainfrom
feat/blockstorage

Conversation

@PF9-pushkar

Copy link
Copy Markdown
Collaborator

Phase 1 — block storage (Cinder v3)

Final Phase 1 family. Code-complete; acceptance is gated by a lab limitation (below), the same way pcd_compute_instance's boot is.

Stacked on feat/compute (#6). Review order: #3#4#5#6 → this.

Resource

  • pcd_blockstorage_volume — create (waits for available), in-place extend when size increases (shrink is rejected), name/description/metadata update, delete (waits), import. Sourceable from snapshot / source volume / image.

Data sources

  • pcd_blockstorage_volume, pcd_blockstorage_snapshot (by id or filters).

Lab caveat (not a provider issue)

The CE lab has no Cinder storage backend (storageBackends={} in the blueprint; only a __DEFAULT__ volume type with nothing behind it), so a created volume goes creating → error. The resource's create + status-waiter + error-detection path is verified — the waiter correctly reports the ERROR state. TestAccBlockStorageVolume_basic flips green on any cloud with a working storage backend.

Follow-ups

  • blockstorage_volume_type, blockstorage_qos, quotas (Phase 3); blockstorage_snapshot resource; compute_volume_attach (needs a booted instance + a real volume).

…a sources

Cinder v3. Code-complete; the volume create/status-waiter/error-detection path is
verified against the lab (the waiter correctly catches the backend ERROR). The full
acceptance test can't pass on the CE lab because it has no storage backend configured
(storageBackends={}), so volumes go creating -> error. It will pass on a cloud with a
working Cinder backend. Tracked in DECISIONS.md.
- pcd_blockstorage_volume: create (wait available), in-place extend on size increase,
  name/description/metadata update, import.
- pcd_blockstorage_volume / pcd_blockstorage_snapshot data sources.
- clients: BlockStorageV3Client (Cinder).
Base automatically changed from feat/compute to main July 12, 2026 01:08
@PF9-pushkar
PF9-pushkar merged commit 3990deb into main Jul 12, 2026
2 checks passed
PF9-pushkar added a commit that referenced this pull request Jul 14, 2026
feat(blockstorage): Cinder volume resource and volume/snapshot data sources
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.

1 participant