Update defaults for variables of type list

[evansmurithi]

Terraform converts the given default value to the specified type. Set an
empty list [] default for variables of type list list()

Signed-off-by: Evans Murithi murithievans80@gmail.com

[davidkarlsen]

Terraform converts the given default value to the specified type. Set an empty list [] default for variables of type list list()

Signed-off-by: Evans Murithi murithievans80@gmail.com

Or maybe even use set(string) if they are used for loops, to avoid larger diffs when changing a value.

[lonegunmanb]

Terraform converts the given default value to the specified type. Set an empty list [] default for variables of type list list()
Signed-off-by: Evans Murithi murithievans80@gmail.com

Or maybe even use set(string) if they are used for loops, to avoid larger diffs when changing a value.

Great idea! But this time I think the list is ok since we didn't use it for loop, but yes this is a very important principal.

[lonegunmanb]

Terraform converts the given default value to the specified type. Set an empty list [] default for variables of type list list()

Signed-off-by: Evans Murithi murithievans80@gmail.com

Thanks @evansmurithi for opening this pr, would you please give me a hint about what we could gain from this change?

[evansmurithi]

@lonegunmanb prior to the change, I was getting an error Error: expected "....admin_group_object_ids.0" to be a valid UUID. The default value for admin_group_object_ids was null. Not sure how internally terraform parses null values, but changing the default from null to [] fixed my issue.

[github-actions]

MAIN BRANCH PUSH DETECTED DUE TO #241, THIS PR WILL BE UPDATED.

[github-actions]

MAIN BRANCH PUSH DETECTED DUE TO #, THIS PR WILL BE UPDATED.

[github-actions]

MAIN BRANCH PUSH DETECTED DUE TO #, THIS PR WILL BE UPDATED.

[github-actions]

MAIN BRANCH PUSH DETECTED DUE TO #, THIS PR NEED TO BE UPDATED TO TRIGGER CI.

[github-actions]

MAIN BRANCH PUSH DETECTED DUE TO #, THIS PR NEED TO BE UPDATED TO TRIGGER CI.