fix: versioning_enabled is working now

Author: cichy1173

README.md

@@ -48,7 +48,7 @@ module "my_bucket" {
 | <a name="input_region"></a> [region](#input_region) | Region in which the bucket should be created. Ressource will be created in the region set at the provider level if null. | `string` | `null` | no |
 | <a name="input_tags"></a> [tags](#input_tags) | A list of tags for the bucket. As the Scaleway console does not support key/value tags, tags are written with the format value/value. | `list(string)` | `[]` | no |
 | <a name="input_versioning_enabled"></a> [versioning_enabled](#input_versioning_enabled) | Enable versioning. Once you version-enable a bucket, it can never return to an unversioned state. You can, however, suspend versioning on that bucket. | `bool` | `false` | no |
-| <a name="input_versioning_lock_configuration"></a> [versioning_lock_configuration](#input_versioning_lock_configuration) | Specifies the Object Lock rule for the bucket. Requires versioning. | ```object({ mode = string, days = optional(number), years = optional(number), })``` | ```{ "days": null, "mode": "GOVERNANCE", "years": null }``` | no |
+| <a name="input_versioning_lock_configuration"></a> [versioning_lock_configuration](#input_versioning_lock_configuration) | Specifies the Object Lock rule for the bucket. Requires versioning. | ```object({ mode = optional(string, "GOVERNANCE"), days = optional(number), years = optional(number), })``` | ```{ "days": null, "years": null }``` | no |
 | <a name="input_website_index"></a> [website_index](#input_website_index) | Website Configuration. | `string` | `null` | no |
 
 ## Outputs

main.tf

@@ -54,9 +54,16 @@ resource "scaleway_object_bucket_lock_configuration" "this" {
 
   rule {
     default_retention {
-      mode  = var.versioning_lock_configuration["mode"]
-      days  = var.versioning_lock_configuration["days"]
-      years = var.versioning_lock_configuration["years"]
+      mode  = var.versioning_lock_configuration.mode
+      days  = var.versioning_lock_configuration.days
+      years = var.versioning_lock_configuration.years
+    }
+  }
+
+  lifecycle {
+    precondition {
+      condition     = var.versioning_enabled ? (var.versioning_lock_configuration.days != null) != (var.versioning_lock_configuration.years != null) : true
+      error_message = "Exactly one of 'days' or 'years' must be provided when versioning is enabled."
     }
   }
 }

variables.tf

@@ -77,12 +77,11 @@ variable "versioning_enabled" {
 variable "versioning_lock_configuration" {
   description = "Specifies the Object Lock rule for the bucket. Requires versioning."
   type = object({
-    mode  = string,
+    mode  = optional(string, "GOVERNANCE"),
     days  = optional(number),
     years = optional(number),
   })
   default = {
-    mode  = "GOVERNANCE"
     days  = null,
     years = null,
   }