Skip to content
This repository was archived by the owner on Jan 9, 2023. It is now read-only.

Commit 2c7f991

Browse files
committed
Warn instead of error on unadvised instance types
1 parent ad58629 commit 2c7f991

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

pkg/tarmak/provider/amazon/amazon.go

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -558,17 +558,14 @@ func (a *Amazon) VerifyInstanceTypes(instancePools []interfaces.InstancePool) er
558558
}
559559

560560
if found {
561-
err := fmt.Errorf("type '%s' is not supported for master instance", instanceType)
562-
result = multierror.Append(result, err)
561+
a.tarmak.Log().Warnf("Type '%s' is not advised for master instance", instanceType)
563562
}
564563
break
565564

566565
case "etcd", "vault":
567566
if a.awsInstanceBurstable(instanceType) {
568-
err := fmt.Errorf("instance '%s' does not support burstable type (%s)", instance.Name(), instanceType)
569-
result = multierror.Append(result, err)
567+
a.tarmak.Log().Warnf("Burstable type '%s' is not advised for instance '%s'", instanceType, instance.Name())
570568
}
571-
572569
break
573570

574571
}

0 commit comments

Comments
 (0)