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

Feature: Implement dry_run to easily list images to delete #34

Merged
merged 4 commits into from
Jan 25, 2022

Conversation

nhuray
Copy link
Contributor

@nhuray nhuray commented Jan 19, 2022

Hi @mirakl-admin, @anouarchattouna, @PascalBourdier and @patricklubach

Thanks for the terraform module to easily cleanup images from Google Cloud Registry, it's really super useful.

I would like to use it but because I'm kind of scare to remove too much images I would like to run gcr-cleaner in dry-run mode.

It looks like there's a dry-run option since this commit : GoogleCloudPlatform/gcr-cleaner@1ac1af8

So in that PR, I added dry_run assuming we are using the latest gcr-cleanerimage.

Nicolas

@anouarchattouna
Copy link
Contributor

Hi @nhuray, thank you for this PR, we will give it a try shortly!

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.

Could you please update the examples accordingly and run pre-commit run -a again?

locals.tf Outdated Show resolved Hide resolved
locals.tf Outdated Show resolved Hide resolved
@anouarchattouna
Copy link
Contributor

@nhuray could please run pre-commit run -a again and update this PR with any changes?

@nhuray
Copy link
Contributor Author

nhuray commented Jan 24, 2022

@anouarchattouna I did it on local but there's no changes to commit !

@anouarchattouna anouarchattouna merged commit b5bb78a into mirakl:main Jan 25, 2022
@anouarchattouna anouarchattouna changed the title Implement dry_run to easily list images to delete Feature: Implement dry_run to easily list images to delete Jan 25, 2022
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)
@nhuray nhuray deleted the dry-run branch February 6, 2022 16:33
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