-
Notifications
You must be signed in to change notification settings - Fork 353
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 1.22 api removed beta ingress #1824
Comments
Documentation examples should be updated as well |
I am updating to Kubernetes I am now trying to run skipper, so far I've modified
Unfortunately I am getting the:
Any idea why this might be? Skipper.yaml
I smell something related to RBAC, but don't have enough experience to nail it down quickly. |
For RBAC you need to replace this part: - apiGroups: ["extensions"]
resources: ["ingresses", ]
verbs: ["get", "list"] with: - apiGroups: ["networking.k8s.io"]
resources: ["ingresses", ]
verbs: ["get", "list"] |
Thanks @mikkeloscar.
currently it responds with 403, whilst previously it was
|
Ok, then the issue is skipper is looking for the old api So the real solution includes changes to skipper to use the newer API, similar to what was done for kube-ingress-aws-controller. |
Yes we need to change the API path, but also change the service port, because spec changed there. |
Yes an example of previous
and
Is it something you will have chance working anytime soon? Can you give any non-binding ETA? I am happy at least to test those changes on my cluster. |
@tomekit right now I can't give an ETA, because I have no time, but next week one colleague is back and maybe it changes soon. |
@tomekit I implemented kubernetes ingress V1 and you can test report if it works great. |
IngressesClusterURI for example is using beta and our tests should also be changed to return v1 networking objects.
Related #1477 #1478
according to https://www.civo.com/learn/migrating-your-ingresses-in-k3s-1-20 spec change is also in service that now has
see also https://kubernetes.io/docs/reference/using-api/deprecation-guide/#ingress-v122
The text was updated successfully, but these errors were encountered: