Skip to content
This repository was archived by the owner on Jul 3, 2023. It is now read-only.

enable tasks to use IAM role#115

Merged
achille-roussel merged 1 commit intosegmentio:masterfrom
nathanielks:enable-task-role
Jun 22, 2017
Merged

enable tasks to use IAM role#115
achille-roussel merged 1 commit intosegmentio:masterfrom
nathanielks:enable-task-role

Conversation

@nathanielks
Copy link
Copy Markdown
Contributor

Enable the task module to receive a role property. Depends on #114

@achille-roussel
Copy link
Copy Markdown
Contributor

This change is not gonna be enough, the ecs-agent requies some iptables rules to be setup in order to support task roles => https://github.com/aws/amazon-ecs-agent

I believe these should be configured at the AMI level, do you think you can tackle this?

@nathanielks
Copy link
Copy Markdown
Contributor Author

@achille-roussel aye!

@nathanielks
Copy link
Copy Markdown
Contributor Author

@achille-roussel I believe I'll work on that over in #114

@achille-roussel
Copy link
Copy Markdown
Contributor

Can you fix the conflict? Than I'll be happy to merge this PR.

@nathanielks
Copy link
Copy Markdown
Contributor Author

@achille-roussel resolved!

@achille-roussel achille-roussel merged commit 49dfeb1 into segmentio:master Jun 22, 2017
@achille-roussel
Copy link
Copy Markdown
Contributor

Sorry for the delay merging this, thanks for your contribution!

@nathanielks
Copy link
Copy Markdown
Contributor Author

No worries at all, @achille-roussel!

@nathanielks nathanielks deleted the enable-task-role branch June 23, 2017 20:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants