Skip to content

Commit 34fce2f

Browse files
committed
add support for 100gb regional
1 parent 08b7635 commit 34fce2f

File tree

2 files changed

+45
-4
lines changed

2 files changed

+45
-4
lines changed

pkg/csi_driver/controller.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ const (
6060
zonalSmallTierMaxSize = 9984 * util.Gb
6161
zonalLargeTierMinSize = 10 * util.Tb
6262
zonalLargeTierMaxSize = 100 * util.Tb
63-
regionalSmallTierMinSize = 1 * util.Tb
63+
regionalSmallTierMinSize = 100 * util.Gb
6464
regionalSmallTierMaxSize = 9984 * util.Gb
6565
regionalLargeTierMinSize = 10 * util.Tb
6666
regionalLargeTierMaxSize = 100 * util.Tb

pkg/csi_driver/controller_test.go

Lines changed: 44 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -766,7 +766,7 @@ func TestCreateVolume(t *testing.T) {
766766
},
767767
resp: &csi.CreateVolumeResponse{
768768
Volume: &csi.Volume{
769-
CapacityBytes: 1 * util.Tb,
769+
CapacityBytes: 100 * util.Gb,
770770
VolumeId: testVolumeID,
771771
VolumeContext: map[string]string{
772772
attrIP: testIP,
@@ -1422,9 +1422,50 @@ func TestGetRequestCapacity(t *testing.T) {
14221422
bytes: 10 * util.Tb,
14231423
},
14241424
{
1425-
name: "required less than small REGIONAL minimum capacity",
1425+
name: "required equals small REGIONAL minimum capacity",
14261426
capRange: &csi.CapacityRange{
1427-
RequiredBytes: 100 * util.Gb,
1427+
RequiredBytes: regionalSmallTierMinSize,
1428+
},
1429+
tier: regionalTier,
1430+
bytes: regionalSmallTierMinSize,
1431+
},
1432+
{
1433+
name: "required 50 GB below small REGIONAL minimum capacity",
1434+
capRange: &csi.CapacityRange{
1435+
RequiredBytes: 50 * util.Gb,
1436+
},
1437+
tier: regionalTier,
1438+
bytes: regionalSmallTierMinSize,
1439+
},
1440+
{
1441+
name: "required 1 GB above small REGIONAL minimum capacity",
1442+
capRange: &csi.CapacityRange{
1443+
RequiredBytes: 101 * util.Gb,
1444+
},
1445+
tier: regionalTier,
1446+
bytes: 101 * util.Gb,
1447+
},
1448+
{
1449+
name: "limit equals small REGIONAL minimum capacity",
1450+
capRange: &csi.CapacityRange{
1451+
LimitBytes: regionalSmallTierMinSize,
1452+
},
1453+
tier: regionalTier,
1454+
bytes: regionalSmallTierMinSize,
1455+
},
1456+
{
1457+
name: "limit below small REGIONAL minimum capacity",
1458+
capRange: &csi.CapacityRange{
1459+
LimitBytes: 50 * util.Gb,
1460+
},
1461+
tier: regionalTier,
1462+
errorExpected: true,
1463+
},
1464+
{
1465+
name: "required below min, limit equals min REGIONAL",
1466+
capRange: &csi.CapacityRange{
1467+
RequiredBytes: 50 * util.Gb,
1468+
LimitBytes: regionalSmallTierMinSize,
14281469
},
14291470
tier: regionalTier,
14301471
bytes: regionalSmallTierMinSize,

0 commit comments

Comments
 (0)