Skip to content

Remove unnecessary type cast#8702

Open
tisonkun wants to merge 1 commit intoeksctl-io:mainfrom
tisonkun:tidy
Open

Remove unnecessary type cast#8702
tisonkun wants to merge 1 commit intoeksctl-io:mainfrom
tisonkun:tidy

Conversation

@tisonkun
Copy link
Copy Markdown

Trivial as is.

BTW, I touch this file because I'm understanding:

RoleArn:                 gfnt.NewString(c.capability.RoleARN),

and https://docs.aws.amazon.com/AWSCloudFormation/latest/TemplateReference/aws-resource-eks-capability.html#cfn-eks-capability-rolearn

I still don't know whether this RoleArn is our chosen name for the to-be-created ArgoCD role, or something we're referring to use and that should be exist beforehand (and then, what do we use it for).

Signed-off-by: tison <wander4096@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant