Skip to content

feat: Override cluster and workers egress CIDRs#1237

Merged
barryib merged 1 commit into
terraform-aws-modules:masterfrom
aspectcapital:issue1236
Apr 20, 2021
Merged

feat: Override cluster and workers egress CIDRs#1237
barryib merged 1 commit into
terraform-aws-modules:masterfrom
aspectcapital:issue1236

Conversation

@bodgit

@bodgit bodgit commented Feb 11, 2021

Copy link
Copy Markdown
Contributor

PR o'clock

Description

Add two new variables to allow the destination CIDR blocks used in egress rules to be overridden. Default values match the previously hardcoded values.

Fixes #1236

Checklist

@bodgit

bodgit commented Feb 11, 2021

Copy link
Copy Markdown
Contributor Author

I don't believe the Lint / Docs failure is due to anything I've changed.

@wmene

wmene commented Apr 12, 2021

Copy link
Copy Markdown

@barryib Any idea when this will get merged?

@barryib

barryib commented Apr 19, 2021

Copy link
Copy Markdown
Member

@bodgit Can you please update your branch and resolve conflict.

@bodgit

bodgit commented Apr 20, 2021

Copy link
Copy Markdown
Contributor Author

@barryib Done

Add two new variables to allow the destination CIDR blocks used in
egress rules to be overridden.

Fixes terraform-aws-modules#1236
@barryib barryib merged commit 2a78efd into terraform-aws-modules:master Apr 20, 2021
barryib pushed a commit to barryib/terraform-aws-eks that referenced this pull request May 20, 2021
@github-actions

Copy link
Copy Markdown

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Nov 15, 2022
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.

Add ability to override CIDR blocks on egress security rules

3 participants