[Bug] Update user resource to accept the new finadmin and account owner roles

[jlacefie]

What are you really trying to do?

Temporal introduced 2 new roles: Account Owner and Finance Admin. The Terraform User Resource currently only supports admin, dev, read. This is a request to update the resource to accept the new role types.

Minimal Reproduction

To reproduce, create a .tf file with a user who has either account owner or finance admin for account access. Terraform plan will fail with

Attribute account_access value must be one of: ["admin" "developer" "read"], got: "Owner"

[jlacefie]

Note - Account Owner can only be assigned by Temporal. Adding it as an acceptable value may not make sense.

[wangzhihao0629]

Adding it as an acceptable value may not make sense.

Import might make sense. Ideally, we want to keep all the information in the format of codes including Account Owner. In this case, I'm okay with not being able to apply.

[jroof88]

@jlacefie This is breaking our terraform setup. I was going to put together a PR for owner to to be added to the values for account_access but before I do...will it be accepted?

[captainbeardo]

Owner does make sense as a value on an imported user, but not for a user being created through terraform. A PR would be welcome and we are actively watching PRs on this repo and will work with you to get it in.

[jroof88]

@captainbeardo That feels kind of backwards to me. For example, in the future let's say I want to transfer Account Owner to someone else on my team. I would have to file a support ticket with Temporal to change it and then import the user back into my tfstate? It would then still be broken cause my tf files would still say admin?

PR: #135