Skip to content

Commit

Permalink
Merge pull request #4 from sparkfabrik/fix/role_name_default
Browse files Browse the repository at this point in the history
refs #000: fix default role name
  • Loading branch information
Monska85 authored May 19, 2023
2 parents 642c526 + f95bc27 commit f9376ee
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 1 deletion.
10 changes: 10 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,18 @@ Following semver, any non backwards compatible feature implies that the next rel

## [Unreleased]

## [0.2.1] - 2023-05-19

[Compare with previous version](https://github.com/sparkfabrik/terraform-google-gcp-artifact-registry/compare/0.2.0...0.2.1)

### Changed

- FIX: the `artifact_registry_listers_custom_role_name` variable is configured as `custom.artifactRegistryLister` by default. The previous default value was `custom.artifactRegistryListers`.

## [0.2.0] - 2023-05-18

[Compare with previous version](https://github.com/sparkfabrik/terraform-google-gcp-artifact-registry/compare/0.1.0...0.2.0)

### Added

- FEATURE: if the `artifact_registry_listers` is provided, a new custom IAM role is created and assigned to the `artifact_registry_listers` principals, allowing them to list the artifacts repositories.
Expand Down
2 changes: 1 addition & 1 deletion variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ variable "repositories" {
variable "artifact_registry_listers_custom_role_name" {
type = string
description = "Name of the custom role for Artifact Registry listers."
default = "custom.artifactRegistryListers"
default = "custom.artifactRegistryLister"
}

variable "artifact_registry_listers" {
Expand Down

0 comments on commit f9376ee

Please sign in to comment.