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

kubernetes api: avoid overriding content-type header in kubernetes-asyncio, pass in via arg instead #1602

Merged
merged 1 commit into from Mar 16, 2024

Conversation

ikreymer
Copy link
Member

  • instead of overriding the content-type header globally, pass 'application/merge-patch+json' to self.custom_api.patch_namespaced_custom_object() directly
  • bump kubernetes-asyncio to 29.0.0
  • fixes potential issues with global override of the header in kubernetes-asyncio

- instead of overriding the content-type header globally, pass 'application/merge-patch+json' to self.custom_api.patch_namespaced_custom_object()
directly
- bump kubernetes-asyncio to 29.0.0
- fixes potential issues with global override of the header in kubernetes-asyncio
@ikreymer ikreymer changed the title kubernetes api: kubernetes api: avoid overriding content-type header in kubernetes-asyncio, pass in via arg instead Mar 16, 2024
@ikreymer
Copy link
Member Author

Tested profiles dev and locally - profile ping calls the _patch_job() function every few seconds

@ikreymer ikreymer merged commit ac69e6a into v1.9.4-release Mar 16, 2024
5 checks passed
@ikreymer ikreymer deleted the patch-job-set-content-type branch March 16, 2024 08:18
ikreymer added a commit that referenced this pull request Mar 18, 2024
…yncio, pass in via arg instead (main) (#1605)

- instead of overriding the content-type header globally, pass
'application/merge-patch+json' to
self.custom_api.patch_namespaced_custom_object() directly
- bump kubernetes-asyncio to 29.0.0
- fixes potential issues with global override of the header in
kubernetes-asyncio
- copy of #1602 for main
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

Successfully merging this pull request may close these issues.

None yet

1 participant