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

Adding IAM roles to the cred chain #85

Closed
wants to merge 23 commits into from
Closed

Adding IAM roles to the cred chain #85

wants to merge 23 commits into from

Conversation

jduv
Copy link

@jduv jduv commented Jan 3, 2019

Similar to our other PRs in other resources, we'd like to add the default IAM cred chain to resources with no static keys defined. This implementation is very close to what I did in the s3-resource, so the code should look familiar.

concourse/s3-resource#115

All tests pass, and we've been running this in production for a week now.

https://cloud.docker.com/u/7factor/repository/docker/7factor/semver-resource

@vito
Copy link
Member

vito commented Jan 10, 2019

Thanks for submitting this! As mentioned in concourse/concourse#3023 though I'm going to put this on hold until we figure out how we want to support IAM roles in the context of concourse/concourse#2386.

@vito vito added the blocked label Jan 10, 2019
@vito vito self-assigned this Apr 20, 2020
@vito
Copy link
Member

vito commented Jun 4, 2020

Closing this one too - see concourse/s3-resource#115 (comment). Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants