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

Fix all-in-one start failed at k8s 1.16 #121

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix all-in-one start failed at k8s 1.16 #121

wants to merge 1 commit into from

Conversation

yuya894
Copy link

@yuya894 yuya894 commented Oct 11, 2019

Which problem is this PR solving?

Resolves #120

Short description of the changes

Change jaeger-all-in-one-template.yml according to kubernates 1.16 api

Signed-off-by: Yuya Yoshida <boyoyon894@gmail.com>
@yuya894
Copy link
Author

yuya894 commented Oct 11, 2019

plz update minikube&kubectl on travis ci.

curl -Lo minikube https://storage.googleapis.com/minikube/releases/v0.25.0/minikube-linux-amd64 && chmod +x minikube
# now v1.4.0
curl -Lo kubectl  https://storage.googleapis.com/kubernetes-release/release/v1.9.0/bin/linux/amd64/kubectl && chmod +x kubectl
# now v1.16.0

I think it should be done with another pull request, because the problem is big.....

@@ -15,7 +15,7 @@
apiVersion: v1
kind: List
items:
- apiVersion: extensions/v1beta1
- apiVersion: apps/v1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would this cause failing in older k8s versions? Is extensions/v1beta1 supported in k8s 1.16?

@pavolloffay
Copy link
Member

plz update minikube&kubectl on travis ci.

This repository is in maintenance mode. However, if you submit a PR we most likely will accept it.

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.

The Deployment "jaeger" is invalid
2 participants