Skip to content

Conversation

@ronething
Copy link
Contributor

@ronething ronething commented Jan 12, 2026

Type of change:

  • Bugfix
  • New feature provided
  • Improve performance
  • Backport patches
  • Documentation
  • Refactor
  • Chore
  • CI/CD or Tests

What this PR does / why we need it:

close #2681

IngressClass does not support directly specifying parameters.namespace in the earlier versions.
To maintain compatibility with earlier versions of IngressClass, we provide this via annotations.

Pre-submission checklist:

  • Did you explain what problem does this PR solve? Or what new features have been added?
  • Have you added corresponding test cases?
  • Have you modified the corresponding document?
  • Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

Signed-off-by: Ashing Zheng <axingfly@gmail.com>
Signed-off-by: Ashing Zheng <axingfly@gmail.com>
@ronething ronething merged commit b52c26c into master Jan 12, 2026
26 checks passed
@ronething ronething deleted the feat/ingressannotaions branch January 12, 2026 06:26
ronething added a commit that referenced this pull request Jan 12, 2026
Signed-off-by: Ashing Zheng <axingfly@gmail.com>
(cherry picked from commit b52c26c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

request help: upgrade 2.0 failed! ingressClass does not match GatewayProxy

3 participants