Fix non-enum CSR condition field, omit enums from static openapi snapshot

[liggitt]

What type of PR is this?

/kind bug
/kind feature

What this PR does / why we need it:

Follow-up to #108639

1. Fixes an incorrectly marked enum field on CSR conditions
2. Omits enums from the static openapi snapshot being used to generate clients

xref #109177

Does this PR introduce a user-facing change?

Omit enum declarations from the static openapi file captured at https://git.k8s.io/kubernetes/api/openapi-spec. This file is used to generate API clients, and use of enums in those generated clients (rather than strings) can break forward compatibility with additional future values in those fields. See https://issue.k8s.io/109177 for details.

/sig api-machinery
for openapi snapshot

/sig auth
for CSR API

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: liggitt

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• api/OWNERS [liggitt]
• hack/OWNERS [liggitt]
• pkg/generated/openapi/OWNERS [liggitt]
• staging/src/k8s.io/api/OWNERS [liggitt]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[liggitt]

/cc @jiahuif

[k8s-triage-robot]

This PR may require API review.

If so, when the changes are ready, complete the pre-review checklist and request an API review.

Status of requested reviews is tracked in the API Review project.

[liggitt]

#109182 flake

/retest

[aojea]

k8s.io/kubernetes/test/integration/apiserver/apply: TestDroppingSubresourceField expand_more	2m26s
k8s.io/kubernetes/test/integration/auth: TestRBAC expand_more	0s
k8s.io/kubernetes/test/integration/client: TestApplyWithApplyConfiguration expand_more	0s
k8s.io/kubernetes/test/integration/controlplane: TestOpenAPIDelegationChainPlumbing expand_more	2m24s
k8s.io/kubernetes/test/integration/controlplane/transformation: TestSecretsShouldBeTransformed expand_more	2m24s
k8s.io/kubernetes/test/integration/cronjob: TestCronJobLaunchesPodAndCleansUp expand_more

/retest

[dims]

/priority critical-urgent
/lgtm

[k8s-ci-robot]

New changes are detected. LGTM label has been removed.

[liggitt]

/retest

[liggitt]

rebased after #109195 merged, readding tag

[sftim]

This PR seems to have removed some enum explanations from the generated API docs.

Could I advocate for a changelog entry that is more clear about what has happened to the OpenAPI spec as a result of this PR? (I don't think I'd have understood that entry without context).

[jiahuif]

Currently, we do not have support for types that can be either enum or any value. Some client generators break when encountering an enum type, especially Java and Python that has built-in enum types. As a result, we have to undo the generated comments from enum types generator.

[liggitt]

Could I advocate for a changelog entry that is more clear about what has happened to the OpenAPI spec as a result of this PR? (I don't think I'd have understood that entry without context).

updated the changelog

[sftim]

I also filed #109463 to cover the new gap in our API docs