AWS Profile in kubeconfig template

[mschuchard]

I'm submitting a

• feature request

For my current delivery, the customer has credentials with multiple profiles and not only needs to specify different profiles per cluster, but has no default profile.

It would be great if the kubeconfig.tpl could be modified:

...
users:
- name: aws
  user:
    exec:
      apiVersion: client.authentication.k8s.io/v1alpha1
      command: heptio-authenticator-aws
      args:
        - "token"
        - "-i"
        - "${cluster_name}"
      env:
        - name: AWS_PROFILE
          value: ${aws_profile}

where the default Terraform value to populate the template would be default to ensure no regression.

I wanted to start a discussion before a PR to ensure best path forward on this. Thanks!

[brandonjbjelland]

Hi @mschuchard

This sounds good but I think this is a duplicate request. We just started discussing a similar (if not the same) issue here a day ago. Let me know if your use case can be accommodated by implementing what's proposed there.

[mschuchard]

I do not see any mention of implementing AWS profiles into the kubeconfig in that issue. Version 1.1.0 is currently failing for the customer because EKS cannot be reached due to the kubeconfig not using the correct AWs profile since that is not customizable in this module.

Having separate kubeconfigs is fine, but in reality people are likely going to be merging their kubeconfigs and then differentiating between them via context.

[sdavids13]

I submitted PR #49 which provides the ability to specify a generic map of environment variables to be appended to the kubeconfig.

[brandonjbjelland]

This is addressed having just merged #49 . Closing this up.

[github-actions]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.