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

Ambassador ingress setup documentation no longer valid #2856

Closed
DmytryEmery opened this issue Aug 2, 2022 · 2 comments
Closed

Ambassador ingress setup documentation no longer valid #2856

DmytryEmery opened this issue Aug 2, 2022 · 2 comments
Labels
kind/documentation Categorizes issue or PR as related to documentation.

Comments

@DmytryEmery
Copy link

What would you like to be documented:
Removal or updating of Ambassdor ingress docs

Why is this needed:

Following the documentation for ingress setup for Ambassdor , it fails with current version of Kind.

error: resource mapping not found for name: "ambassadorinstallations.getambassador.io" namespace: "" from "https://github.com/datawire/ambassador-operator/releases/latest/download/ambassador-operator-crds.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"

Digging around it appears: apiextensions.k8s.io/v1beta1 is depreciated in new versions of Kubernetes.

As per: datawire/ambassador-operator#73

I was able to get it to work by using an older version of the nodes
e.g.
kind create cluster --config .\kind-config.yaml --image kindest/node:v1.21.12​

This could be mentioned in the docs, or alternately, move to emissary-ingress which seems to be the successor project?

@DmytryEmery DmytryEmery added the kind/documentation Categorizes issue or PR as related to documentation. label Aug 2, 2022
@BenTheElder
Copy link
Member

Removal or updating of Ambassdor ingress docs
[...]
This could be mentioned in the docs, or alternately, move to emissary-ingress which seems to be the successor project?

In this case I would lean towards remove, these are contributed and maintained by folks from or interested in the specific ingress typically.

While we've tried to permit all open options folks are willing to contribute, having more of these only has made the guide more confusing, and the kind maintainers do not intend to support N ingresses directly. Within the Kubernetes project we have ingress-nginx which works well enough for this purpose.

Given that nobody else has reported this previously since May when we last released kind and when v1.24.0 Kubernetes came out, it seems interest is minimal and we should probably drop this from the guide.

@DmytryEmery
Copy link
Author

Seems like a reasonable approach to me, if someone wants to use the others they can add those as needed. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/documentation Categorizes issue or PR as related to documentation.
Projects
None yet
Development

No branches or pull requests

2 participants