Skip to content

Commit d655486

Browse files
committed
typo/aws/type: correct service name for AWS Wavelentgh Zones
The commit[1] introduced the AWS Wavelength Zones type, which has a typo in the correct service name[2]. [1] openshift@5bc9c3b [2] https://aws.amazon.com/wavelength/
1 parent 455dd37 commit d655486

File tree

4 files changed

+8
-6
lines changed

4 files changed

+8
-6
lines changed

pkg/asset/installconfig/aws/availabilityzones.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ func edgeZones(ctx context.Context, session *session.Session, region string) ([]
103103
return nil, fmt.Errorf("unable to retrieve Local Zone names: %w", err)
104104
}
105105

106-
wavelengthZones, err := filterZonesByType(ctx, session, region, typesaws.WavelengtyZoneType)
106+
wavelengthZones, err := filterZonesByType(ctx, session, region, typesaws.WavelengthZoneType)
107107
if err != nil {
108108
return nil, fmt.Errorf("unable to retrieve Wavelength Zone names: %w", err)
109109
}

pkg/asset/installconfig/aws/validation.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -423,7 +423,7 @@ func validateZoneLocal(ctx context.Context, meta *Metadata, fldPath *field.Path,
423423
for _, zone := range zones {
424424
if aws.StringValue(zone.ZoneName) == zoneName {
425425
switch aws.StringValue(zone.ZoneType) {
426-
case awstypes.LocalZoneType, awstypes.WavelengtyZoneType:
426+
case awstypes.LocalZoneType, awstypes.WavelengthZoneType:
427427
default:
428428
return field.Invalid(fldPath, zoneName, fmt.Sprintf("only zone type local-zone or wavelength-zone are valid in the edge machine pool: %s", aws.StringValue(zone.ZoneType)))
429429
}

pkg/tfvars/aws/aws.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,8 @@ func TFVars(sources TFVarsSources) ([]byte, error) {
112112
if _, ok := sources.AvailabilityZones[zoneName]; !ok {
113113
return nil, errors.New(fmt.Sprintf("unable to find the zone when generating terraform vars: %s", zoneName))
114114
}
115-
if sources.AvailabilityZones[zoneName].Type == typesaws.LocalZoneType || sources.AvailabilityZones[zoneName].Type == typesaws.WavelengtyZoneType {
115+
if sources.AvailabilityZones[zoneName].Type == typesaws.LocalZoneType ||
116+
sources.AvailabilityZones[zoneName].Type == typesaws.WavelengthZoneType {
116117
edgeLocalZoneMap[zoneName] = exists
117118
continue
118119
}

pkg/types/aws/availabilityzones.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,11 @@ package aws
33
const (
44
// AvailabilityZoneType is the type of regular zone placed on the region.
55
AvailabilityZoneType = "availability-zone"
6-
// LocalZoneType is the type of Local zone placed on the metropolitan areas.
6+
// LocalZoneType is the type of AWS Local Zones placed on the metropolitan area.
77
LocalZoneType = "local-zone"
8-
// WavelengtyZoneType is the type of Wavelength zone placed in the Carrier infrastructure closer to areas.
9-
WavelengtyZoneType = "wavelength-zone"
8+
// WavelengthZoneType is the type of AWS Wavelength Zones placed on the telecommunications
9+
// providers’ data centers at the edge of the 5G network.
10+
WavelengthZoneType = "wavelength-zone"
1011
// ZoneOptInStatusOptedIn is the opt-in status of the zone.
1112
// For Availability Zones, this parameter always has the value of opt-in-not-required.
1213
// For Local Zones and Wavelength Zones, this parameter is the opt-in status.

0 commit comments

Comments
 (0)