Skip to content

Commit

Permalink
update to support iam
Browse files Browse the repository at this point in the history
  • Loading branch information
catalinpan committed Aug 14, 2017
1 parent 70acaa7 commit e61d0d5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 36 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,11 @@ REGION
RENEW_TOKEN - default 6h
```

##### AWS instance with IAM role
### AWS instance with IAM role

For AWS instances if the region is not declared it will be auto discovered from IAM as long as the instance supports that. [pull request](https://github.com/catalinpan/aws-ecr-proxy/pull/1/commits/899ef1a80a7fa141f66e500a76f6ed86f8d19f4e), [commit](https://github.com/catalinpan/aws-ecr-proxy/commit/d8a709bf043cfd14b88defae738833e93c946f4b).

The AWS key and secret can be also configured using a IAM role (without mounting them secrets or specifying them as variables). A sample IAM role config can be found in examples folder. More details on the [AWS official documentation](http://docs.aws.amazon.com/IAM/latest/UserGuide/id_roles.html).
The AWS key and secret can be also configured using a IAM role (without mounting them secrets or specifying them as variables). A sample IAM role config can be found in the examples folder. More details on the [AWS official documentation](http://docs.aws.amazon.com/IAM/latest/UserGuide/id_roles.html).

The configs will be checked in the following order:

Expand Down
34 changes: 0 additions & 34 deletions examples/aws-instance-role-policy.json
Original file line number Diff line number Diff line change
@@ -1,31 +1,6 @@
{
"Version": "2012-10-17",
"Statement": [
{
"Action": "s3:*",
"Resource": [
"arn:aws:s3:::ecr-pull-*"
],
"Effect": "Allow"
},
{
"Action": [
"ec2:*"
],
"Resource": [
"*"
],
"Effect": "Allow"
},
{
"Action": [
"route53:*"
],
"Resource": [
"*"
],
"Effect": "Allow"
},
{
"Action": [
"ecr:GetAuthorizationToken",
Expand All @@ -38,15 +13,6 @@
],
"Resource": "*",
"Effect": "Allow"
},
{
"Action": [
"elasticloadbalancing:*"
],
"Resource": [
"*"
],
"Effect": "Allow"
}
]
}

0 comments on commit e61d0d5

Please sign in to comment.