-
Notifications
You must be signed in to change notification settings - Fork 753
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
fix: ingressclass name should be customizable #864
Conversation
@kupnu4x: Thanks for this PR Would you please detail why you add a new value instead of using Or, to ask this question in a different way, do you think we should use this new value also on |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WDYT about handling it that way to avoid breaking changes ?
Co-authored-by: Rémi BUISSON <remi-buisson@orange.fr>
Co-authored-by: Rémi BUISSON <remi-buisson@orange.fr>
Co-authored-by: Rémi BUISSON <remi-buisson@orange.fr>
0c5ac66
to
4070408
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Co-authored-by: Michel Loiseleur <97035654+mloiseleur@users.noreply.github.com> Co-authored-by: Rémi BUISSON <remi-buisson@orange.fr>
What does this PR do?
Fixes issue #855
More
make test
and all the tests passed