From ca76bf4a12708c29877467033c625d58eb833968 Mon Sep 17 00:00:00 2001 From: Todd Neal Date: Thu, 30 Jun 2022 09:34:48 -0500 Subject: [PATCH] docs: correct information regarding default instance types (#2028) * doc: correct information regarding default instance types Fixes #2027 * test: fix race with clearing maps --- pkg/cloudprovider/aws/fake/ec2api.go | 10 ++++++++-- website/content/en/preview/provisioner.md | 7 +------ 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/pkg/cloudprovider/aws/fake/ec2api.go b/pkg/cloudprovider/aws/fake/ec2api.go index aa2db4dc8a4b..e826a19a9fc5 100644 --- a/pkg/cloudprovider/aws/fake/ec2api.go +++ b/pkg/cloudprovider/aws/fake/ec2api.go @@ -78,8 +78,14 @@ func (e *EC2API) Reset() { e.CalledWithCreateFleetInput.Reset() e.CalledWithCreateLaunchTemplateInput.Reset() e.DescribeSpotPriceHistoryOutput.Reset() - e.Instances = sync.Map{} - e.LaunchTemplates = sync.Map{} + e.Instances.Range(func(k, v any) bool { + e.Instances.Delete(k) + return true + }) + e.LaunchTemplates.Range(func(k, v any) bool { + e.LaunchTemplates.Delete(k) + return true + }) e.InsufficientCapacityPools.Reset() e.NextError.Reset() } diff --git a/website/content/en/preview/provisioner.md b/website/content/en/preview/provisioner.md index 809dcade6f20..c41f557ccb73 100644 --- a/website/content/en/preview/provisioner.md +++ b/website/content/en/preview/provisioner.md @@ -109,12 +109,7 @@ Generally, instance types should be a list and not a single value. Leaving this Review [AWS instance types](https://aws.amazon.com/ec2/instance-types/). -The default value includes all instance types with the exclusion of metal -(non-virtualized), -[non-HVM](https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/virtualization_types.html), -and GPU instances. - -View the full list of instance types with `aws ec2 describe-instance-types`. +The default value includes most instance types with the exclusion of [non-HVM](https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/virtualization_types.html). The full list of supported instance types can be seen [here](../aws/instance-types/) **Example**