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

Update deprecated APIs in Docs #613

Merged
merged 3 commits into from
Jul 20, 2020
Merged

Conversation

rikatz
Copy link
Contributor

@rikatz rikatz commented Jul 5, 2020

Signed-off-by: Ricardo Pchevuzinske Katz [email protected]

This PR intends to start using the newer Ingress APIs instead of the deprecated ones.

Instead of using extensions/v1beta1 it will use the newer networking.k8s.io/v1beta1

Right now I'm changing only the docs, there's a need of decision whether the older API will still be supported (as Ingress NGINX) or be fully deprecated in further versions.

Creating a Ingress resource with networking.k8s.io works fine in the current API and Ingress Controller (tested with v1.17.0) and the extensions/v1beta1 will be fully removed from Kubernetes API in version v1.22

TODO

  • Decide if this PR is going also to change the behavior of Haproxy Ingress to support only the newer API version, otherwise check how to deal with both versions and print a warning to the user.

@rikatz rikatz changed the title [WIP] - Update deprecated APIs Update deprecated APIs in Docs Jul 5, 2020
@rikatz rikatz mentioned this pull request Jul 5, 2020
3 tasks
Signed-off-by: Ricardo Pchevuzinske Katz <[email protected]>
Signed-off-by: Ricardo Pchevuzinske Katz <[email protected]>
@jcmoraisjr
Copy link
Owner

Hi, thank you! It seems pretty good btw. Perhaps examples/rbac/ingress-controller-rbac.yml could also be updated?

@jcmoraisjr
Copy link
Owner

About the todo - lets make this version a k8s v1.14+. I'll add this backward compatibility item in the changelog.

Signed-off-by: Ricardo Pchevuzinske Katz <[email protected]>
@rikatz
Copy link
Contributor Author

rikatz commented Jul 13, 2020

🤦‍♂️ forgot this one

Did a new commit

@jcmoraisjr
Copy link
Owner

Thanks! Merging and sorry about the long delay. I'm aware we already have v1, let's update it in another step.

@jcmoraisjr jcmoraisjr merged commit 8fc9ca4 into jcmoraisjr:master Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants