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.

Sign up for GitHub

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

Jump to bottom

Allow for aws:skipMetadataApiCheck #291

Closed
poobah opened this issue Jun 17, 2022 · 3 comments
Closed

Allow for aws:skipMetadataApiCheck #291

poobah opened this issue Jun 17, 2022 · 3 comments
Assignees
@t0yv0
Labels
kind/enhancement Improvements or new features resolution/fixed This issue was fixed

Comments

@poobah
Copy link

poobah commented Jun 17, 2022 •
edited
Loading

Hello!

  • Vote on this issue by adding a 👍 reaction
  • If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

If you would like to use IAM creds for the Operator rather than pass AWS creds around it seems the Operator is running into this issue pulumi/pulumi-aws#1692

Setting the AWS_EC2_METADATA_DISABLED does not appear to not work (as stated in the issue)
however adding aws:skipMetadataApiCheck to your provider works.

Might be worth adding this point your documentation as the examples all use AWS creds in the examples.
@poobah poobah added kind/enhancement Improvements or new features needs-triage Needs attention from the triage team labels Jun 17, 2022
@poobah poobah closed this as completed Jun 20, 2022
@poobah poobah reopened this Jun 20, 2022
@stack72
Copy link
Contributor

stack72 commented Jun 20, 2022

Hi @poobah

I am sorry for the confusion here :/ Are you saying that you'd like to be able to set this via an EnvVar or are you asking for it to be made more clear in our docs?

Paul

@stack72 stack72 added awaiting-feedback Blocked on input from the author and removed needs-triage Needs attention from the triage team labels Jun 20, 2022
@stack72 stack72 self-assigned this Jun 20, 2022
@poobah
Copy link
Author

poobah commented Jun 20, 2022

Hi @stack72,

Looks like the EnvVar does not work in pulumi-asp so no point yet in setting it but the provider setting does work. I suggested that the docs could do with stating that it is set in the provider (maybe adding to troubleshooting) if folks want to use IAM roles over AWS creds.

@stack72 stack72 removed their assignment Sep 30, 2022
@stack72 stack72 mentioned this issue Oct 4, 2022
Merged
@mikhailshilkov mikhailshilkov removed the awaiting-feedback Blocked on input from the author label Aug 2, 2023
@EronWright EronWright added the resolution/fixed This issue was fixed label Oct 29, 2024
@EronWright
Copy link
Contributor

This is fixed (thanks @t0yv0!) by pulumi/pulumi-aws#3960.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@t0yv0 t0yv0

Labels
kind/enhancement Improvements or new features resolution/fixed This issue was fixed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@poobah @t0yv0 @stack72 @mikhailshilkov @EronWright