address review comments

Signed-off-by: Pankaj Walke <[email protected]>

Author: punkwalker

pkg/cloud/converters/eks.go

@@ -217,7 +217,7 @@ func NodegroupUpdateconfigFromSDK(ngUpdateConfig *ekstypes.NodegroupUpdateConfig
 	return converted
 }
 
-// AMITypeToSDK is used to CAPA ManagedMachineAMIType to AWS SDK AMIType.
+// AMITypeToSDK converts a CAPA ManagedMachineAMIType to AWS SDK AMIType.
 func AMITypeToSDK(amiType expinfrav1.ManagedMachineAMIType) ekstypes.AMITypes {
 	switch amiType {
 	case expinfrav1.Al2x86_64:

test/e2e/shared/suite.go

@@ -157,7 +157,7 @@ func Node1BeforeSuite(e2eCtx *E2EContext) []byte {
 	ensureSSHKeyPair(e2eCtx.AWSSession, DefaultSSHKeyPairName)
 	e2eCtx.Environment.BootstrapAccessKey = newUserAccessKey(context.TODO(), e2eCtx.AWSConfig, bootstrapTemplate.Spec.BootstrapUser.UserName)
 	e2eCtx.BootstrapUserAWSSession = NewAWSSessionWithKey(e2eCtx.Environment.BootstrapAccessKey)
-	// v2AccessKey can be removed after AWS SDK V2 Migration
+	//TODO: v2AccessKey can be removed after AWS SDK V2 Migration
 	v2AccessKey := &iamtypes.AccessKey{
 		AccessKeyId:     e2eCtx.Environment.BootstrapAccessKey.AccessKeyId,
 		SecretAccessKey: e2eCtx.Environment.BootstrapAccessKey.SecretAccessKey,
@@ -231,7 +231,7 @@ func AllNodesBeforeSuite(e2eCtx *E2EContext, data []byte) {
 	e2eCtx.Environment.BootstrapClusterProxy = framework.NewClusterProxy("bootstrap", conf.KubeconfigPath, e2eCtx.Environment.Scheme)
 	e2eCtx.E2EConfig = &conf.E2EConfig
 	e2eCtx.BootstrapUserAWSSession = NewAWSSessionWithKey(conf.BootstrapAccessKey)
-	// v2AccessKey can be removed after AWS SDK V2 Migration
+	//TODO: v2AccessKey can be removed after AWS SDK V2 Migration
 	v2AccessKey := &iamtypes.AccessKey{
 		AccessKeyId:     e2eCtx.Environment.BootstrapAccessKey.AccessKeyId,
 		SecretAccessKey: e2eCtx.Environment.BootstrapAccessKey.SecretAccessKey,