[tamird]

What type of PR is this?

/kind bug
/kind api-change

What this PR does / why we need it:

Deleting a ConfigMap can return either the ConfigMap or a Kubernetes Status. Today both come back as dictionaries. Deleting a Pod has a different failure: a successful Status response can raise a validation error because the client tries to read it as a Pod.

Regenerate the synchronous and asyncio clients so individual built-in DELETEs return the model the server actually sent. For example, delete_namespaced_config_map() returns V1ConfigMap | V1Status:

result = api.delete_namespaced_config_map(name, namespace)
if isinstance(result, client.V1Status):
    print(result.message)
else:
    print(result.metadata.name)

For _with_http_info, the model is the first item in the synchronous client’s (data, status, headers) tuple and the .data field of the asyncio response. Collection deletes and generic custom-object methods keep their existing behavior.

Which issue(s) this PR fixes:

Fixes #2752

Special notes for your reviewer:

Depends on kubernetes-client/gen#313, where the generator change lives. The OpenAPI v3 migration plan preserves the return types shown here.

Does this PR introduce a user-facing change?

action required: Individual built-in DELETE methods now return the deleted resource model or V1Status. If you read these responses as dictionaries, switch to model attributes and use isinstance(result, V1Status) to distinguish a Status from the resource. For _with_http_info methods, read the model from the first tuple item in the synchronous client or from response.data in the asyncio client.

Prepared with Codex assistance.

[kubernetes-prow]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: tamird
Once this PR has been reviewed and has the lgtm label, please assign roycaihw for approval. For more information see the Code Review Process.

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

Details

Needs approval from an approver in each of these files:

• OWNERS

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

[tamird]

@yliaog, #2664 told 37.x callers to use dictionary access. Would you be comfortable switching these methods to return models in the next major release?

@tomplus, does returning a V1ConfigMap or V1Status directly from await api.delete_namespaced_config_map(...) work for asyncio callers?

— with Codex assistance

[yliaog]

@tamird let's wait for the next release 1.38 for the breaking change