Skip to content

Commit

Permalink
Merge pull request #182 from lreciomelero/fix/aws_standard_sc_type
Browse files Browse the repository at this point in the history
changed to standard sc by default
  • Loading branch information
lreciomelero authored Jun 27, 2023
2 parents b9a8661 + 7e3df6c commit f5c1cf5
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions pkg/cluster/internal/create/actions/createworker/aws.go
Original file line number Diff line number Diff line change
Expand Up @@ -55,11 +55,11 @@ var storageClassAWSTemplate = StorageClassDef{
}

var standardAWSParameters = commons.SCParameters{
Type: "st1",
Type: "gp3",
}

var premiumAWSParameters = commons.SCParameters{
Type: "gp3",
Type: "io2",
}

type AWSBuilder struct {
Expand Down Expand Up @@ -364,6 +364,6 @@ func (b *AWSBuilder) getParameters(sc commons.StorageClass) commons.SCParameters
case "premium":
return mergeSCParameters(sc.Parameters, premiumAWSParameters)
default:
return mergeSCParameters(sc.Parameters, premiumAWSParameters)
return mergeSCParameters(sc.Parameters, standardAWSParameters)
}
}
2 changes: 1 addition & 1 deletion pkg/cluster/internal/create/actions/createworker/azure.go
Original file line number Diff line number Diff line change
Expand Up @@ -274,7 +274,7 @@ func (b *AzureBuilder) getParameters(sc commons.StorageClass) commons.SCParamete
case "premium":
return mergeSCParameters(sc.Parameters, premiumAZParameters)
default:
return mergeSCParameters(sc.Parameters, premiumAZParameters)
return mergeSCParameters(sc.Parameters, standardAZParameters)
}
}

Expand Down
2 changes: 1 addition & 1 deletion pkg/cluster/internal/create/actions/createworker/gcp.go
Original file line number Diff line number Diff line change
Expand Up @@ -223,7 +223,7 @@ func (b *GCPBuilder) getParameters(sc commons.StorageClass) commons.SCParameters
case "premium":
return mergeSCParameters(sc.Parameters, premiumGCPParameters)
default:
return mergeSCParameters(sc.Parameters, premiumGCPParameters)
return mergeSCParameters(sc.Parameters, standardGCPParameters)

}
}
Expand Down

0 comments on commit f5c1cf5

Please sign in to comment.