Skip to content

Conversation

mhofstetter
Copy link
Member

As a follow up of #29327, the slim k8s model for Ingress & IngressClass can be removed

This commit removes the unused IngressClass resource.

Signed-off-by: Marco Hofstetter <marco.hofstetter@isovalent.com>
Remove unused Ingress status converter function.

Signed-off-by: Marco Hofstetter <marco.hofstetter@isovalent.com>
This commit removes the unused slim k8s model for Ingress &
IngressClass.

Signed-off-by: Marco Hofstetter <marco.hofstetter@isovalent.com>
@mhofstetter mhofstetter added kind/cleanup This includes no functional changes. release-note/misc This PR makes changes that have no direct user impact. area/servicemesh GH issues or PRs regarding servicemesh feature/k8s-ingress labels Nov 30, 2023
@mhofstetter mhofstetter requested review from aanm and sayboras November 30, 2023 17:27
@mhofstetter mhofstetter requested review from a team as code owners November 30, 2023 17:27
Copy link
Member

@sayboras sayboras left a comment

Choose a reason for hiding this comment

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

Thanks ✅

@aanm aanm enabled auto-merge December 1, 2023 09:16
@aanm
Copy link
Member

aanm commented Dec 1, 2023

/test

@aanm aanm added this pull request to the merge queue Dec 1, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Dec 1, 2023
Merged via the queue into cilium:main with commit 98dfc7d Dec 1, 2023
@mhofstetter mhofstetter deleted the pr/mhofstetter/del-k8s-slim-ingress branch December 1, 2023 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/servicemesh GH issues or PRs regarding servicemesh feature/k8s-ingress kind/cleanup This includes no functional changes. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants