Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: terraform-aws-modules/terraform-aws-eks
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v18.2.6
Choose a base ref
...
head repository: terraform-aws-modules/terraform-aws-eks
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v18.2.7
Choose a head ref
  • 2 commits
  • 2 files changed
  • 2 contributors

Commits on Feb 2, 2022

  1. fix: Don't tag self managed node security group with kubernetes.io/cl… 

    …uster tag (#1774)
    Devin Young authored Feb 2, 2022

    Verified

    This commit was created on github.com and signed with GitHub’s verified signature. The key has expired.
    GPG key ID: 4AEE18F83AFDEB23
    Expired
    Verified
    Learn about vigilant mode
    Copy the full SHA
    a638e4a View commit details

  2. chore(release): version 18.2.7 [skip ci] 

    ### [18.2.7](v18.2.6...v18.2.7) (2022-02-02)

### Bug Fixes

* Don't tag self managed node security group with kubernetes.io/cluster tag ([#1774](#1774)) ([a638e4a](a638e4a))
    @semantic-release-bot
    semantic-release-bot committed Feb 2, 2022
    Copy the full SHA
    d81ff0d View commit details
Showing with 8 additions and 2 deletions.
  1. +7 −0 CHANGELOG.md
  2. +1 −2 modules/self-managed-node-group/main.tf
7 changes: 7 additions & 0 deletions CHANGELOG.md
View file
Original file line number Diff line number Diff line change
@@ -2,6 +2,13 @@

All notable changes to this project will be documented in this file.

### [18.2.7](https://github.com/terraform-aws-modules/terraform-aws-eks/compare/v18.2.6...v18.2.7) (2022-02-02)


### Bug Fixes

* Don't tag self managed node security group with kubernetes.io/cluster tag ([#1774](https://github.com/terraform-aws-modules/terraform-aws-eks/issues/1774)) ([a638e4a](https://github.com/terraform-aws-modules/terraform-aws-eks/commit/a638e4a754c15ab230cfb0e91de026e038ca4e26))

### [18.2.6](https://github.com/terraform-aws-modules/terraform-aws-eks/compare/v18.2.5...v18.2.6) (2022-02-01)


3 changes: 1 addition & 2 deletions modules/self-managed-node-group/main.tf
View file
Original file line number Diff line number Diff line change
@@ -460,8 +460,7 @@ resource "aws_security_group" "this" {
tags = merge(
var.tags,
{
"Name" = local.security_group_name
"kubernetes.io/cluster/${var.cluster_name}" = "owned"
"Name" = local.security_group_name
},
var.security_group_tags
)