Fixed a missing RegisterOrDie call and removed references to EKS Serv…

…ice account in Makefile

Makefile

@@ -48,7 +48,6 @@ apply: ## Deploy the controller into your ~/.kube/config cluster
 		$(HELM_OPTS) \
 		--set controller.image=ko://github.com/awslabs/karpenter/cmd/controller \
 		--set webhook.image=ko://github.com/awslabs/karpenter/cmd/webhook \
-		--set serviceAccount.annotations.'eks\.amazonaws\.com/role-arn'=\"$$(kubectl get configmaps aws-auth -n kube-system -ojsonpath='{.data.mapRoles}' | grep KarpenterNodeRole | head -1 | cut -d ' ' -f3)\" \
 		| $(WITH_GOFLAGS) ko apply -B -f -
 
 delete: ## Delete the controller from your ~/.kube/config cluster

pkg/cloudprovider/registry/aws.go

@@ -21,6 +21,6 @@ import (
 	"github.com/awslabs/karpenter/pkg/cloudprovider/aws"
 )
 
-func NewCloudProvider(options cloudprovider.Options) cloudprovider.CloudProvider {
+func newCloudProvider(options cloudprovider.Options) cloudprovider.CloudProvider {
 	return aws.NewCloudProvider(options)
 }

pkg/cloudprovider/registry/fake.go

@@ -21,6 +21,6 @@ import (
 	"github.com/awslabs/karpenter/pkg/cloudprovider/fake"
 )
 
-func NewCloudProvider(cloudprovider.Options) cloudprovider.CloudProvider {
+func newCloudProvider(cloudprovider.Options) cloudprovider.CloudProvider {
 	return &fake.CloudProvider{}
 }

pkg/cloudprovider/registry/registry.go

@@ -23,6 +23,12 @@ import (
 	"knative.dev/pkg/apis"
 )
 
+func NewCloudProvider(options cloudprovider.Options) cloudprovider.CloudProvider {
+	cloudProvider := newCloudProvider(options)
+	RegisterOrDie(cloudProvider)
+	return cloudProvider
+}
+
 // RegisterOrDie populates supported instance types, zones, operating systems,
 // architectures, and validation logic. This operation should only be called
 // once at startup time. Typically, this call is made by NewCloudProvider(), but