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

Limit ingress support to only networking.k8s.io/v1 #515

Merged
merged 7 commits into from
Jan 31, 2024

Conversation

katyanna
Copy link
Member

@katyanna katyanna commented Jun 22, 2022

Given the deprecation of extensions/v1beta1 and networking.k8s.io/v1beta1,
remove support for them.

katyanna added 2 commits June 22, 2022 11:33
Given the deprecation of extensions/v1beta1 and networking.k8s.io/v1beta1,
remove support for them.

Signed-off-by: Katyanna Moura <[email protected]>
@katyanna katyanna marked this pull request as ready for review June 22, 2022 10:33
@szuecs
Copy link
Member

szuecs commented Jun 22, 2022

@katyanna please change the VERSION file to create the next minor version and explain in the readme.md section what the update to >= the new version will do.

@katyanna
Copy link
Member Author

I've talked with @mikkeloscar and decided for freezing this for now so other open source users can use the older versions if they need to.

@szuecs
Copy link
Member

szuecs commented Jun 22, 2022

Yeah true I also thought about that and you can change the default in kubernetes-on-aws which we likely already did.

@szuecs
Copy link
Member

szuecs commented Dec 4, 2023

@katyanna I think we can now go on and drop the v1beta1 code

Signed-off-by: Katyanna Moura <[email protected]>
* master:
  Bump github.com/zalando/skipper from 0.17.32 to 0.18.38 (#648)
  Add test case for rolling in progress stacks
  Bump github.com/aws/aws-sdk-go from 1.47.0 to 1.49.13
  Bump github.com/google/uuid from 1.4.0 to 1.5.0 (#670)
  Bump github.com/prometheus/client_golang from 1.17.0 to 1.18.0 (#669)
  Require labels (#666)
  Bump golang.org/x/crypto from 0.14.0 to 0.17.0
@katyanna katyanna added dependencies Pull requests that update a dependency file minor labels Jan 18, 2024
Signed-off-by: Katyanna Moura <[email protected]>
@katyanna
Copy link
Member Author

👍

@katyanna
Copy link
Member Author

@katyanna please change the VERSION file to create the next minor version and explain in the readme.md section what the update to >= the new version will do.

@szuecs Is this still relevant or should we merge without changing the version?

@mikkeloscar
Copy link
Collaborator

@szuecs Is this still relevant or should we merge without changing the version?

We should still change the version to indicate to users that this is no longer supported.

@katyanna katyanna self-assigned this Jan 29, 2024
Signed-off-by: Katyanna Moura <[email protected]>
@katyanna
Copy link
Member Author

👍

1 similar comment
@zaklawrencea
Copy link
Member

👍

@katyanna katyanna merged commit c530fcd into master Jan 31, 2024
10 checks passed
@katyanna katyanna deleted the ingress-upgrade-v1 branch January 31, 2024 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file minor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants