Skip to content

Commit 4059c58

Browse files
committed
storage capacity: clarify autoscaler design
As discussed during PR review, the section was focused on local storage without explicitly saying that. Network attached storage needs to be dealt with differently and has a different problem (scale from zero and lacking capacity information).
1 parent 991e4e9 commit 4059c58

File tree

1 file changed

+9
-1
lines changed
  • keps/sig-storage/1472-storage-capacity-tracking

1 file changed

+9
-1
lines changed

keps/sig-storage/1472-storage-capacity-tracking/README.md

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1200,7 +1200,7 @@ This problem can be solved by the cluster administrator. They can find out how
12001200
much storage will be made available by new nodes, for example by running
12011201
experiments, and then configure the cluster so that this information is
12021202
available to the autoscaler. This can be done with the existing
1203-
CSIStorageCapacity API as follows:
1203+
CSIStorageCapacity API for node-local storage as follows:
12041204

12051205
- When creating a fictional Node object from an existing Node in
12061206
a node group, autoscaler must modify the topology labels of the CSI
@@ -1238,6 +1238,14 @@ components. Simpler solutions may be possible by adding support for specific
12381238
CSI drivers into custom autoscaler binaries or into operators that control the
12391239
cluster setup.
12401240

1241+
Network attached storage doesn't need renaming of labels when cloning an
1242+
existing Node. The information published for that Node is also valid for the
1243+
fictional one. Scale up from zero however is problematic: the CSI specification
1244+
does not support listing topology segments that don't have some actual Nodes
1245+
with a running CSI driver on them. Either a CSI specification change or manual
1246+
configuration of the external-provisioner sidecar will be needed to close this
1247+
gap.
1248+
12411249
### Alternative solutions
12421250

12431251
At the moment, storage vendor can already achieve the same goals

0 commit comments

Comments
 (0)