Skip to content

Commit

Permalink
Use ec2.DescribeInstanceTypes for machine type info
Browse files Browse the repository at this point in the history
  • Loading branch information
rifelpet committed Apr 7, 2020
1 parent a06cebc commit 6982c99
Show file tree
Hide file tree
Showing 13 changed files with 77 additions and 2,895 deletions.
7 changes: 0 additions & 7 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -802,13 +802,6 @@ build-docs-netlify:
pip install -r ${KOPS_ROOT}/images/mkdocs/requirements.txt
mkdocs build

# Update machine_types.go
.PHONY: update-machine-types
update-machine-types:
go build -o hack/machine_types/machine_types ${KOPS_ROOT}/hack/machine_types/
hack/machine_types/machine_types --out upup/pkg/fi/cloudup/awsup/machine_types.go
go fmt upup/pkg/fi/cloudup/awsup/machine_types.go

#-----------------------------------------------------------
# development targets

Expand Down
5 changes: 5 additions & 0 deletions cloudmock/aws/mockec2/instances.go
Original file line number Diff line number Diff line change
Expand Up @@ -49,3 +49,8 @@ func (m *MockEC2) DescribeInstancesPages(request *ec2.DescribeInstancesInput, ca
func (m *MockEC2) DescribeInstancesPagesWithContext(aws.Context, *ec2.DescribeInstancesInput, func(*ec2.DescribeInstancesOutput, bool) bool, ...request.Option) error {
panic("Not implemented")
}

func (m *MockEC2) DescribeInstanceTypes(*ec2.DescribeInstanceTypesInput) (*ec2.DescribeInstanceTypesOutput, error) {
klog.Warningf("MockEc2::DescribeInstanceTypes is stub-implemented")
return &ec2.DescribeInstanceTypesOutput{}, nil
}
1 change: 0 additions & 1 deletion hack/machine_types/.gitignore

This file was deleted.

21 changes: 0 additions & 21 deletions hack/machine_types/BUILD.bazel

This file was deleted.

19 changes: 0 additions & 19 deletions hack/machine_types/README.md

This file was deleted.

328 changes: 0 additions & 328 deletions hack/machine_types/machine_types.go

This file was deleted.

Loading

0 comments on commit 6982c99

Please sign in to comment.