Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lack of content types in patchNamespacedDeploymentCall #3189

Open
kozjan opened this issue Mar 11, 2024 · 2 comments
Open

Lack of content types in patchNamespacedDeploymentCall #3189

kozjan opened this issue Mar 11, 2024 · 2 comments

Comments

@kozjan
Copy link
Contributor

kozjan commented Mar 11, 2024

Describe the bug
I am migrating from client version v19.0.0.
When using AppsV1Api::patchNamespacedDeploymentCall, I get an error:

io.kubernetes.client.openapi.ApiException: Message: 
HTTP response code: 415
HTTP response body: 
{
    "kind": "Status",
    "apiVersion": "v1",
    "metadata": {},
    "status": "Failure",
    "message": "415: Unsupported Media Type",
    "reason": "UnsupportedMediaType",
    "details": {},
    "code": 415
}

I see that v19.0.0 defines more content types:

final String[] localVarContentTypes = {
    "application/json-patch+json", "application/merge-patch+json", "application/strategic-merge-patch+json", "application/apply-patch+yaml"
};

compared to v20.0.0:

final String[] localVarContentTypes = {
    "application/json"
};

Is there any reason for this?

Client Version
v20.0.0

Kubernetes Version
1.24

Java Version
Java 8

To Reproduce
Steps to reproduce the behavior:

  • execute an APIpatchNamespacedDeploymentRequest

Expected behavior
K8s accepts the request without problems.

KubeConfig
If applicable, add a KubeConfig file with secrets redacted.

Server (please complete the following information):

  • OS: [e.g. Linux]
  • Environment [e.g. container]
  • Cloud [e.g. Azure]

Additional context
Add any other context about the problem here.

@kozjan kozjan changed the title Lacking content types in patchNamespacedDeploymentCall Lack of content types in patchNamespacedDeploymentCall Mar 11, 2024
@brendandburns
Copy link
Contributor

you should use PatchUtils

@oreissig
Copy link

oreissig commented Mar 13, 2024

you should use PatchUtils

This works as a workaround, but in fact this really is a bug in the client.

Looks related to #3106

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants