Pass disk size with SI suffix to control plane #287

New issue

Closed

opened 2025-11-13 14:04:02 +00:00 by tobru · 2 comments

tobru commented

2025-11-13 14:04:02 +00:00

Owner

Stories

As a control plan, I want to get the disk size parameter in the correct form with SI suffix

Implementation Notes

The Kubernetes API wants the storage size with an SI suffix, not just a number. The API server doesn't currently validate that.

Update the frontend to show "Gi" in the form field. Keep it a number field, so that we can set min and max values.
Pass the number value with a "Gi" suffix as string to the control plane. No space between number and suffix, e.g. "10Gi"
The control plane returns it as string as well, so a conversion from string to number will be required
This currently only applies to the spec.parameters.size.disk field

Refs:

## Stories _As a control plan, I want to get the disk size parameter in the correct form with SI suffix_ ## Implementation Notes The Kubernetes API wants the storage size with an SI suffix, not just a number. The API server doesn't currently validate that. * Update the frontend to show "Gi" in the form field. Keep it a number field, so that we can set min and max values. * Pass the number value with a "Gi" suffix as string to the control plane. No space between number and suffix, e.g. "10Gi" * The control plane returns it as string as well, so a conversion from string to number will be required * This currently only applies to the `spec.parameters.size.disk` field Refs: * https://kubernetes.io/docs/concepts/storage/persistent-volumes/#capacity * https://kubernetes.io/docs/reference/glossary/?all=true#term-quantity

tobru added the

enhancement

label 2025-11-13 14:04:02 +00:00

tobru added this to the Development Planning project 2025-11-13 14:04:03 +00:00

rixx commented

2025-11-14 15:26:56 +00:00

Member

Is this only the spec.parameters.size.disk field or anything else aswell?
Does the format permit (or require?) a space between the number and the unit?

- Is this only the `spec.parameters.size.disk` field or anything else aswell? - Does the format permit (or require?) a space between the number and the unit?

tobru changed title from ~~Form validation for disk size~~ to Pass disk size with SI suffix to control plane

2025-11-17 10:30:20 +00:00

tobru commented

2025-11-17 10:31:08 +00:00

Author

Owner

The description is updated accordingly