Skip to content
This repository has been archived by the owner on Sep 16, 2024. It is now read-only.

add support for gcr buckets with uniform_bucket_level_access = true #30

Merged
merged 4 commits into from
Jan 18, 2022

Conversation

patricklubach
Copy link
Contributor

As requested by @anouarchattouna I create another PR with the same changes from @andyroyle because @andyroyle does not respond anymore.

Copy link
Contributor

@anouarchattouna anouarchattouna left a comment

Choose a reason for hiding this comment

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

Can you also add this to data.tf please ?

data "google_storage_bucket" "bucket" {
  for_each = toset(local.buckets)

  name = each.value
} 

locals.tf Show resolved Hide resolved
locals.tf Outdated Show resolved Hide resolved
iam.tf Outdated Show resolved Hide resolved
@patricklubach
Copy link
Contributor Author

@anouarchattouna I applied your requested changes.

Copy link
Contributor

@anouarchattouna anouarchattouna left a comment

Choose a reason for hiding this comment

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

Last minor change, otherwise LGTM!

data.tf Show resolved Hide resolved
requested by @anouarchattouna

Co-authored-by: Anouar Chattouna <9251649+anouarchattouna@users.noreply.github.com>
Copy link

@madhifallah madhifallah left a comment

Choose a reason for hiding this comment

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

LGTM

@madhifallah madhifallah merged commit b882a21 into mirakl:main Jan 18, 2022
anouarchattouna added a commit that referenced this pull request Jan 18, 2022
anouarchattouna added a commit that referenced this pull request Jan 18, 2022
@patricklubach patricklubach deleted the uniform-bucket-access branch January 19, 2022 11:41
mirakl-admin pushed a commit that referenced this pull request Feb 3, 2022
## 1.0.0 (2022-02-03)

### Features

* Add support for GCR buckets with uniform_bucket_level_access = true ([#32](#32)) ([16d6232](16d6232))
* First Implementation of GCR Cleaner ([#1](#1)) ([5ad9957](5ad9957))
* Implement dry_run to easily list images to delete ([#34](#34)) ([b5bb78a](b5bb78a))
* Implementing all payload parameters ([#24](#24)) ([dde3658](dde3658))
* Implementing get all repositories of a given project ([#3](#3)) ([f043971](f043971))
* Introduce new payload parameters ([#29](#29)) ([0ea8b25](0ea8b25))

### Bug Fixes

* Adding repos parameter to payload ([#27](#27)) ([0b62c57](0b62c57)), closes [#24](#24)

### Reverts

* Revert "add support for gcr buckets with uniform_bucket_level_access = true (#30)" (#31) ([9eb0fde](9eb0fde)), closes [#30](#30) [#31](#31)
mirakl-admin pushed a commit that referenced this pull request Feb 3, 2022
## 1.0.0 (2022-02-03)

### Features

* Add basic Google Artifact Registry support ([#39](#39)) ([8d6ba49](8d6ba49))
* Add support for GCR buckets with uniform_bucket_level_access = true ([#32](#32)) ([16d6232](16d6232))
* First Implementation of GCR Cleaner ([#1](#1)) ([5ad9957](5ad9957))
* Implement dry_run to easily list images to delete ([#34](#34)) ([b5bb78a](b5bb78a))
* Implementing all payload parameters ([#24](#24)) ([dde3658](dde3658))
* Implementing get all repositories of a given project ([#3](#3)) ([f043971](f043971))
* Introduce new payload parameters ([#29](#29)) ([0ea8b25](0ea8b25))

### Bug Fixes

* Adding repos parameter to payload ([#27](#27)) ([0b62c57](0b62c57)), closes [#24](#24)

### Reverts

* Revert "add support for gcr buckets with uniform_bucket_level_access = true (#30)" (#31) ([9eb0fde](9eb0fde)), closes [#30](#30) [#31](#31)
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.

3 participants