Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add default value nginx to ingressClassName for the virtualservers CRD #6893

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

hanyouqing
Copy link
Contributor

@hanyouqing hanyouqing commented Dec 3, 2024

Proposed changes

Fix #6891

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

@hanyouqing hanyouqing requested a review from a team as a code owner December 3, 2024 08:44
Copy link

github-actions bot commented Dec 3, 2024

✅ All required contributors have signed the F5 CLA for this PR. Thank you!
Posted by the CLA Assistant Lite bot.

@hanyouqing
Copy link
Contributor Author

I have hereby read the F5 CLA and agree to its terms

@hanyouqing
Copy link
Contributor Author

recheck

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Status: Todo ☑
Development

Successfully merging this pull request may close these issues.

[Bug]: Multiple NGINX Ingress Controllers doesn't work well
1 participant