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

Commit 7854a6b

Browse files
committed
Warn instead of error on unadvised instance types
1 parent ba29a08 commit 7854a6b

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
@@ -537,17 +537,14 @@ func (a *Amazon) VerifyInstanceTypes(instancePools []interfaces.InstancePool) er
537537
}
538538

539539
if found {
540-
err := fmt.Errorf("type '%s' is not supported for master instance", instanceType)
541-
result = multierror.Append(result, err)
540+
a.tarmak.Log().Warnf("Type '%s' is not advised for master instance", instanceType)
542541
}
543542
break
544543

545544
case "etcd", "vault":
546545
if a.awsInstanceBurstable(instanceType) {
547-
err := fmt.Errorf("instance '%s' does not support burstable type (%s)", instance.Name(), instanceType)
548-
result = multierror.Append(result, err)
546+
a.tarmak.Log().Warnf("Burstable type '%s' is not advised for instance '%s'", instanceType, instance.Name())
549547
}
550-
551548
break
552549

553550
}

0 commit comments

Comments
 (0)