Merge pull request #5 from Perun-Engineering/feat/buckets-versioning

feat: Buckets versioning

Author: babychm

README.md

@@ -61,6 +61,7 @@ In the above diagram, you can see the components and their relations (PostgreSQL
 |------|-------------|------|---------|:--------:|
 | <a name="input_bucket_prefix"></a> [bucket\_prefix](#input\_bucket\_prefix) | Prefix used for S3 buckets | `string` | `""` | no |
 | <a name="input_buckets_lifecycles"></a> [buckets\_lifecycles](#input\_buckets\_lifecycles) | Lifecycle rules for buckets | `map(string)` | `{}` | no |
+| <a name="input_buckets_versioning"></a> [buckets\_versioning](#input\_buckets\_versioning) | Versioning for buckets | `map(bool)` | `{}` | no |
 | <a name="input_cluster_name"></a> [cluster\_name](#input\_cluster\_name) | EKS cluster name where you want to deploy the release | `string` | n/a | yes |
 | <a name="input_database_password"></a> [database\_password](#input\_database\_password) | Password to access PostgreSQL database | `string` | n/a | yes |
 | <a name="input_gitlab_chart_version"></a> [gitlab\_chart\_version](#input\_gitlab\_chart\_version) | Version of the gitlab chart | `string` | `"7.8.1"` | no |

examples/main.tf

@@ -65,6 +65,10 @@ EOF
 }
 EOF
   }
+  # Default is `false`, determine for each of backup, lfs, artifacts, packages, uploads etc
+  buckets_versioning = {
+    backup = true
+  }
 
   values = [
     templatefile("values.yaml", {

main.tf

@@ -230,7 +230,7 @@ module "s3_bucket" {
   force_destroy = false
 
   versioning = {
-    enabled = false
+    enabled = try(var.buckets_versioning[each.key], false)
   }
 
   policy        = data.aws_iam_policy_document.s3_bucket_policy[each.key].json

variables.tf

@@ -95,6 +95,12 @@ variable "buckets_lifecycles" {
   default     = {}
 }
 
+variable "buckets_versioning" {
+  description = "Versioning for buckets"
+  type        = map(bool)
+  default     = {}
+}
+
 variable "tags" {
   description = "A map of tags to add to all resources"
   type        = map(string)