Skip to content

feat(vpc): enable NAT gateway creation for svcs placed in private subnets#2111

Merged
mergify[bot] merged 7 commits intoaws:mainlinefrom
huanjani:add-conditional
Apr 3, 2021
Merged

feat(vpc): enable NAT gateway creation for svcs placed in private subnets#2111
mergify[bot] merged 7 commits intoaws:mainlinefrom
huanjani:add-conditional

Conversation

@huanjani
Copy link
Copy Markdown
Contributor

This uses the template from #2076 to create NAT Gateways when Copilot users deploy their services in private subnets and use a default Copilot-generated VPC (as opposed to importing their own).

Related: #1959.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@huanjani huanjani requested a review from a team as a code owner March 26, 2021 22:57
@huanjani huanjani requested a review from Lou1415926 March 26, 2021 22:57
Copy link
Copy Markdown
Contributor

@Lou1415926 Lou1415926 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good! just a few thoughts ⬇️

Comment thread internal/pkg/deploy/cloudformation/stack/lb_web_svc.go Outdated
Comment thread internal/pkg/template/workload.go Outdated
@huanjani huanjani marked this pull request as draft March 29, 2021 23:36
@huanjani huanjani marked this pull request as ready for review March 31, 2021 20:54
Copy link
Copy Markdown
Contributor

@efekarakus efekarakus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Just a few small things

Comment thread internal/pkg/deploy/cloudformation/stack/workload.go Outdated
Comment thread site/content/docs/manifest/backend-service.md Outdated
Comment thread templates/environment/versions/cf-v1.3.0.yml Outdated
Comment thread templates/workloads/partials/cf/env-controller-action-with-alb.yml Outdated
Comment thread templates/workloads/partials/cf/env-controller-action.yml Outdated
Comment thread templates/workloads/partials/cf/env-controller.yml Outdated
Copy link
Copy Markdown
Contributor

@efekarakus efekarakus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! We checked that this works fine right :D

Comment thread internal/pkg/template/workload.go Outdated
@efekarakus efekarakus added the do-not-merge Pull requests that mergify shouldn't merge until the requester allows it. label Apr 2, 2021
Copy link
Copy Markdown
Contributor

@iamhopaul123 iamhopaul123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

@huanjani
Copy link
Copy Markdown
Contributor Author

huanjani commented Apr 2, 2021

Nice! We checked that this works fine right :D

Yup.

@huanjani huanjani removed the do-not-merge Pull requests that mergify shouldn't merge until the requester allows it. label Apr 3, 2021
@mergify mergify Bot merged commit a9ab638 into aws:mainline Apr 3, 2021
mergify Bot pushed a commit that referenced this pull request Apr 8, 2021
These changes enable the EnvController to update parameters, specifically to remove workload names from parameters. 

Related: #2111, #1959.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
@huanjani huanjani deleted the add-conditional branch June 22, 2021 04:14
thrau pushed a commit to localstack/copilot-cli-local that referenced this pull request Dec 9, 2022
…nets (aws#2111)

This uses the template from aws#2076 to create NAT Gateways when Copilot users deploy their services in private subnets and use a default Copilot-generated VPC (as opposed to importing their own).

Related: aws#1959.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
thrau pushed a commit to localstack/copilot-cli-local that referenced this pull request Dec 9, 2022
These changes enable the EnvController to update parameters, specifically to remove workload names from parameters. 

Related: aws#2111, aws#1959.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
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.

5 participants