Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add InstanceProfile CRD #1064

Closed
mariob1626 opened this issue Jan 14, 2022 · 4 comments · Fixed by #1159
Closed

Add InstanceProfile CRD #1064

mariob1626 opened this issue Jan 14, 2022 · 4 comments · Fixed by #1159
Assignees

Comments

@mariob1626
Copy link
Contributor

What resource do you need?

AWS cli parameter is:
create-instance-profile and add-role-to-instance-profile

What is your use case?

We are building VM which runs multiple commands against multiple endpoints where credentials are needed. Instead of passing these secrets directly into userdata field we can use Instance Profile to read credentials from AWS SecretManager.

Would you be willing to contribute it using code generator?

Sure, but might take some time at this point as we just started to use crossplane.

@haarchri
Copy link
Member

we also interested in instanceProfile since karpenter needs one ;)

  KarpenterNodeInstanceProfile:
    Type: "AWS::IAM::InstanceProfile"
    Properties:
      InstanceProfileName: !Sub "KarpenterNodeInstanceProfile-${ClusterName}"
      Path: "/"
      Roles:
        - Ref: "KarpenterNodeRole"

@jbw976
Copy link
Member

jbw976 commented Feb 3, 2022

@ytsarev is going to take this issue on! I'll assign him once he's a member of the Crossplane org 😊

@haarchri
Copy link
Member

haarchri commented Feb 3, 2022

@ytsarev take care that code-generator for iam is possible but region is injected check aws-controllers-k8s/community#1150

@ytsarev
Copy link
Contributor

ytsarev commented Feb 3, 2022

@haarchri great, thanks for the heads up 👍

tektondeploy pushed a commit to gtn3010/provider-aws that referenced this issue Mar 12, 2024
…vate-makefile-tools

feat(renovate): add uptest, up and golangcilint for renovate pr
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants