Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add volume size also to Volume status #241

Open
arttor opened this issue Dec 10, 2021 · 1 comment
Open

Add volume size also to Volume status #241

arttor opened this issue Dec 10, 2021 · 1 comment
Labels
enhancement New feature or request later Items that should be tracked / kept in mind and shouldn't be implemented now but later.

Comments

@arttor
Copy link
Contributor

arttor commented Dec 10, 2021

Summary

Add size parameter to Volume status indicating the actual resources of the underlying volume. Size parameter in the spec will reflect requested resources.

Basic example

apiVersion: storage.onmetal.de/v1alpha1
kind: Volume
...
status:
  resources:
    storage: 100Gi

Motivation

Api user should know the actual size of the underlying volume. It may be different from requested size during volume resize operation. This can happen if the storage cluster doesn't have enough capacity to fulfill the new size.
@arttor arttor added the enhancement New feature or request label Dec 10, 2021
@adracus adracus added the later Items that should be tracked / kept in mind and shouldn't be implemented now but later. label Dec 14, 2021
@adracus
Copy link
Contributor

adracus commented Feb 17, 2022

We'll do this later when we handle storage resize correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request later Items that should be tracked / kept in mind and shouldn't be implemented now but later.
Projects
Core
Status: Todo
Milestone
No milestone
Development

No branches or pull requests
2 participants
@adracus @arttor