fix: Making retention_policy is_locked as optional field (#394)

Author: amrkk123

examples/simple_bucket/main.tf

@@ -55,4 +55,7 @@ module "bucket" {
   }]
 
   autoclass = true
+  retention_policy = {
+    retention_period = 2
+  }
 }

modules/simple_bucket/README.md

@@ -54,7 +54,7 @@ Functional examples are included in the
 | name | The name of the bucket. | `string` | n/a | yes |
 | project\_id | The ID of the project to create the bucket in. | `string` | n/a | yes |
 | public\_access\_prevention | Prevents public access to a bucket. Acceptable values are inherited or enforced. If inherited, the bucket uses public access prevention, only if the bucket is subject to the public access prevention organization policy constraint. | `string` | `"inherited"` | no |
-| retention\_policy | Configuration of the bucket's data retention policy for how long objects in the bucket should be retained. | <pre>object({<br>    is_locked        = bool<br>    retention_period = number<br>  })</pre> | `null` | no |
+| retention\_policy | Configuration of the bucket's data retention policy for how long objects in the bucket should be retained. | <pre>object({<br>    is_locked        = optional(bool)<br>    retention_period = number<br>  })</pre> | `null` | no |
 | soft\_delete\_policy | Soft delete policies to apply. Format is the same as described in provider documentation https://www.terraform.io/docs/providers/google/r/storage_bucket.html#nested_soft_delete_policy | <pre>object({<br>    retention_duration_seconds = optional(number)<br>  })</pre> | `{}` | no |
 | storage\_class | The Storage Class of the new bucket. | `string` | `null` | no |
 | versioning | While set to true, versioning is fully enabled for this bucket. | `bool` | `true` | no |

modules/simple_bucket/metadata.yaml

@@ -100,7 +100,7 @@ spec:
         description: Configuration of the bucket's data retention policy for how long objects in the bucket should be retained.
         varType: |-
           object({
-              is_locked        = bool
+              is_locked        = optional(bool)
               retention_period = number
             })
       - name: custom_placement_config

modules/simple_bucket/variables.tf

@@ -78,7 +78,7 @@ variable "autoclass" {
 variable "retention_policy" {
   description = "Configuration of the bucket's data retention policy for how long objects in the bucket should be retained."
   type = object({
-    is_locked        = bool
+    is_locked        = optional(bool)
     retention_period = number
   })
   default = null