Skip to content

Commit

Permalink
Update v3 flow diagram
Browse files Browse the repository at this point in the history
  • Loading branch information
sunnylovestiramisu committed Oct 25, 2023
1 parent 256bee5 commit 23fa960
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions keps/sig-storage/3751-volume-attributes-class/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,7 @@ type ModifyVolumeStatus struct {
// TargetVolumeAttributesClassName is the name of the VolumeAttributesClass the PVC currently being reconciled
TargetVolumeAttributesClassName string
// Status is the status of the ControllerModifyVolume operation
Status *PersistentVolumeClaimModifyVolumeStatus
Status PersistentVolumeClaimModifyVolumeStatus
}
// +enum
Expand All @@ -206,7 +206,7 @@ type PersistentVolumeClaimModifyVolumeStatus string
const (
// Pending indicates that the PersistentVolumeClaim cannot be modified due to requirements not being met, such as
// the PersistentVolumeClaim being in an invalid state or the specified VolumeAttributesClass is existing
// the specified VolumeAttributesClass doesn't exist
PersistentVolumeClaimControllerModifyVolumePending PersistentVolumeClaimModifyVolumeStatus = "ControllerModifyVolumePending"
// State set when modify volume controller starts modifying the volume
PersistentVolumeClaimControllerModifyVolumeInProgress PersistentVolumeClaimModifyVolumeStatus = "ControllerModifyVolumeInProgress"
Expand Down Expand Up @@ -581,7 +581,7 @@ spec:

ModifyVolume is only allowed on bound PVCs. Under the ModifyVolume call, it will pass in the mutable parameters and do the update operation based on the `VolumeAttributesClass` parameters.

![ModifyVolume Flow Diagram](./VolumeAttributesClass-ModifyVolume-Flow-v2.png)
![ModifyVolume Flow Diagram](./VolumeAttributesClass-ModifyVolume-Flow-v3.png)

### Implementation & Handling Failure

Expand Down
Binary file not shown.
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

0 comments on commit 23fa960

Please sign in to comment.