Skip to content

Conversation

@sandeepgopi
Copy link

Description

Updated tags variable for the aws_ecr_repository repo as below.

Resource (In the 'terraform-aws-lambda -> modules -> docker-build -> main.tf' file): "aws_ecr_repository"
Resource name: "this"

Motivation and Context

When we began using the repo, we found that one of the resources is yet to be tag enabled in the standard way. Hence when our internal projects were calling this repo, our custom tags were not being added in AWS.

Breaking Changes

None

How Has This Been Tested?

This is a basic feature to add tags with no functional change introduced . It has been done as per https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/ecr_repository

@sandeepgopi sandeepgopi changed the title Updated tags for aws_ecr_repository feat: Updated tags for aws_ecr_repository Apr 11, 2022
Copy link
Member

@antonbabenko antonbabenko left a comment

Choose a reason for hiding this comment

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

docker-build is a separate submodule and it uses var.ecr_repo_tags to define tags. There is nothing wrong with this.

@github-actions
Copy link

github-actions bot commented Nov 8, 2022

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 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants