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

Honor network_mode of "none" #51

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Honor network_mode of "none" #51

wants to merge 5 commits into from

Conversation

socketbox
Copy link
Collaborator

@socketbox socketbox commented Mar 22, 2024

@nlicitra reported that he could not create a taskdef with no open ports and networking disabled. This PR would honor such a request on the part of the client programmer.

To create a non-networked taskdef, the user would pass in network_mode = none and requires_compatibilites to a value that does not include "FARGATE" (the default). See module foo in the attached file for an example of this.

To test, download the attached file and rename it to main.tf. Apply it and check that it creates three task definitions: one with no networking and no open ports, the other with network mode set to awsvpc and an open port of 80, and the last with a network mode set to awsvpc and an open port of 8080.

@socketbox socketbox requested a review from ehirsch-3pg March 22, 2024 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants