-
Notifications
You must be signed in to change notification settings - Fork 162
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
Make programmable deny of wildcard hostnames #417
Conversation
8943549
to
945ad2e
Compare
b5054e7
to
945ad2e
Compare
b1f2707
to
25524ae
Compare
@alegrey91 @prometherion Hey! And why not to add |
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.
Looking at changes everything seems good, however we got failing tests on ingressClass check, still investigating why this happens.
25524ae
to
074cb23
Compare
That would require a CRD change, requiring a new minor release: idea is to use the annotations, and then prepare the struct change for the new API version. Does it sound good to you, @MaxFedotov ? |
074cb23
to
cdb3723
Compare
cdb3723
to
d1736bb
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.
Tests are flaky but not related to the following changes.
Closes #219