Skip to content

Add PVC capacity validation - #55

Open
AdyCohen1 wants to merge 1 commit into
kiagnose:mainfrom
AdyCohen1:test/add-pvc-capacity-validation
Open

Add PVC capacity validation#55
AdyCohen1 wants to merge 1 commit into
kiagnose:mainfrom
AdyCohen1:test/add-pvc-capacity-validation

Conversation

@AdyCohen1

@AdyCohen1 AdyCohen1 commented Jul 29, 2026

Copy link
Copy Markdown

What this PR does

Before this PR:
The storage checkup does not verify whether the storage provisioner delivers the correct amount of raw storage when a PVC is requested.

After this PR:
The checkup creates a 12345Mi Block-mode PVC and verifies the provisioner delivers at least the requested capacity. If the provisioned capacity is less than requested, the checkup fails. Handles WaitForFirstConsumer StorageClasses via selected-node annotation.

Test plan

  • Unit tests pass (pvcCapacityMismatch, happy path, skip when no default SC)

  • Run checkup against Immediate and WFFC StorageClasses; confirm bind + capacity result

  • Confirm under-provisioned capacity fails the checkup and is reported in status.result.pvcCapacity

    References
    https://redhat.atlassian.net/browse/CNV-94240

@AdyCohen1
AdyCohen1 marked this pull request as draft August 3, 2026 08:38
@AdyCohen1
AdyCohen1 force-pushed the test/add-pvc-capacity-validation branch from 316eebc to 77ffb1a Compare August 3, 2026 09:33
@AdyCohen1
AdyCohen1 marked this pull request as ready for review August 3, 2026 09:34
Comment thread manifests/storage_checkup_permissions.yaml Outdated
Comment thread pkg/internal/checkup/checkup.go Outdated
@AdyCohen1
AdyCohen1 force-pushed the test/add-pvc-capacity-validation branch from 77ffb1a to f5be44d Compare August 6, 2026 15:47
@AdyCohen1
AdyCohen1 force-pushed the test/add-pvc-capacity-validation branch from f5be44d to d0becb0 Compare August 18, 2026 14:18
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