Skip to content
This repository has been archived by the owner on May 18, 2020. It is now read-only.

fix(all-in-one): support for Kubernetes v1.16+ #132

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

subnetmarco
Copy link

@subnetmarco subnetmarco commented Apr 2, 2020

The extensions/v1beta1 API version is deprecated and no longer supported in Kubernetes v1.16+ and fails with the following error:

unable to recognize no matches for kind "Deployment" in version "extensions/v1beta1"

This PR makes the jaeger-all-in-one-template.yml compatible with newer version of Kubernetes.

@pavolloffay
Copy link
Member

@subnetmarco thanks for the PR.

To merge this we should probably bump the minikube version to pass the CI. Also the commits have to be signed (DCO).

@objectiser
Copy link
Contributor

@subnetmarco could you also apply the same fix to the production template?

@subnetmarco
Copy link
Author

@objectiser done with my last commit ca16ed2

@objectiser
Copy link
Contributor

Hi @subnetmarco can you also address the points raised by Pavol in #132 (comment)

@pavolloffay
Copy link
Member

We are going to deprecate the repository. Please have a look at https://github.com/jaegertracing/jaeger-operator/#experimental-generate-kubernetes-manifest-file and share your feedback.

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

Successfully merging this pull request may close these issues.

3 participants