Update ospatch's windows update classification field to be an array.

[nat-henderson]

Also updates docs to display enum options in lists.
Fixes hashicorp/terraform-provider-google#6909.
Supports #3820, because it allows arrays of enums to display values in docs.

Release Note Template for Downstream PRs (will be copied)

osconfig: `google_os_config_patch_deployment`'s `windows_update.classifications` field now works correctly, accepting multiple values.

all: Lists of enums now display the enum options in the documentation pages.

[modular-magician]

Hi! I'm the modular magician. Your PR generated some diffs in downstreams - here they are.

Diff report:

Terraform GA: Diff ( 6 files changed, 14 insertions(+), 6 deletions(-))
Terraform Beta: Diff ( 8 files changed, 16 insertions(+), 6 deletions(-))
TF OiCS: Diff ( 1 file changed, 1 insertion(+))

[modular-magician]

I have triggered VCR tests based on this PR's diffs. See the results here: "https://ci-oss.hashicorp.engineering/viewQueued.html?itemId=137777"

[slevenick]

Will this break backwards compatibility for users who are using this field for a single classification?

[nat-henderson]

Re: backwards compatibility, this never worked. In fact it actually crashed if you set these values server-side and tried to import it.

[modular-magician]

Hi! I'm the modular magician. Your PR generated some diffs in downstreams - here they are.

Diff report:

Terraform GA: Diff ( 9 files changed, 18 insertions(+), 10 deletions(-))
Terraform Beta: Diff ( 13 files changed, 22 insertions(+), 12 deletions(-))
TF OiCS: Diff ( 1 file changed, 1 insertion(+))

[modular-magician]

I have triggered VCR tests based on this PR's diffs. See the results here: "https://ci-oss.hashicorp.engineering/viewQueued.html?itemId=137820"

[modular-magician]

I have triggered VCR tests in RECORDING mode for the following tests that failed during VCR: TestAccCloudBuildTrigger_cloudbuildTriggerBuildExample|TestAccCloudBuildTrigger_basic|TestAccComputeInstanceTemplate_shieldedVmConfig1|TestAccComputeInstanceTemplate_shieldedVmConfig2|TestAccComputeInstance_shieldedVmConfig1|TestAccComputeInstance_shieldedVmConfig2|TestAccEndpointsService_basic|TestAccFolderIamAuditConfig_multiple|TestAccOSConfigPatchDeployment_osConfigPatchDeploymentFullExample|TestAccRedisInstance_redisInstanceFullExample|TestAccRedisInstance_redisInstancePrivateServiceExample You can view the result here: "https://ci-oss.hashicorp.engineering/viewQueued.html?itemId=137822"

[slevenick]

Looks like TestAccOSConfigPatchDeployment_osConfigPatchDeploymentFullExample is failing. Possibly related to these changes?

[nat-henderson]

Oh, interesting, looks like that at_least_one_of needs to be exactly_one_of. Fixed it.

[modular-magician]

Hi! I'm the modular magician. Your PR generated some diffs in downstreams - here they are.

Diff report:

Terraform GA: Diff ( 9 files changed, 21 insertions(+), 15 deletions(-))
Terraform Beta: Diff ( 13 files changed, 25 insertions(+), 17 deletions(-))
TF OiCS: Diff ( 1 file changed, 1 insertion(+), 1 deletion(-))

[modular-magician]

I have triggered VCR tests based on this PR's diffs. See the results here: "https://ci-oss.hashicorp.engineering/viewQueued.html?itemId=137848"

[modular-magician]

I have triggered VCR tests in RECORDING mode for the following tests that failed during VCR: TestAccFolderIamAuditConfig_multiple|TestAccOSConfigPatchDeployment_osConfigPatchDeploymentFullExample|TestAccSqlDatabaseInstance_dontDeleteDefaultUserOnReplica|TestAccSqlDatabaseInstance_settings_basic|TestAccSqlDatabaseInstance_replica|TestAccSqlDatabaseInstance_highAvailability|TestAccSqlDatabaseInstance_slave|TestAccSqlDatabaseInstance_settingsDowngrade|TestAccSqlDatabaseInstance_PointInTimeRecoveryEnabled|TestAccSqlDatabaseInstance_settings_upgrade|TestAccSqlDatabaseInstance_authNets|TestAccSqlDatabaseInstance_basic_with_user_labels You can view the result here: "https://ci-oss.hashicorp.engineering/viewQueued.html?itemId=137853"