Skip to content
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

Expose configuration for tolerateNonExistentBackendService and tolerateNonExistentBackendAction #3330

Closed
kc9ddi opened this issue Aug 10, 2023 · 1 comment · Fixed by #3342
Labels
good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines.

Comments

@kc9ddi
Copy link
Contributor

kc9ddi commented Aug 10, 2023

Is your feature request related to a problem?
When deploying an ingress that is misconfigured with a missing service or action, the ingress will deploy successfully. However, when serving requests destined for the missing service/action, those requests will fail with an HTTP 503 error. This behavior was introduced intentionally in #1967 - in order to ensure that a single misconfiguration doesn't block all ingress reconciliation. However, it also means that a category of misconfiguration is more difficult to discover prior to deploying it and running traffic against the misconfigured ingress.

Describe the solution you'd like
The above PR mentions that this behavior should be configurable, to allow user to opt-in to the behavior as it was prior to the PR making that change. It would be nice if this configuration were exposed.

Describe alternatives you've considered
Manual validation of ingress configuration prior to deployment using custom / 3rd party tools.

@M00nF1sh
Copy link
Collaborator

M00nF1sh commented Aug 11, 2023

I think it’s reasonable to allow configuration of this,maybe a feature flag?

@M00nF1sh M00nF1sh added the good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. label Aug 11, 2023
@oliviassss oliviassss mentioned this issue Oct 16, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants