-
Notifications
You must be signed in to change notification settings - Fork 413
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Skip *heavy* validation on deletion 馃檭 #1085
Skip *heavy* validation on deletion 馃檭 #1085
Conversation
ffdf2cc
to
2aa784d
Compare
The following is the coverage report on the affected files.
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dibyom The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This is great! Thanks! (Could we add a unit test?) |
Yeah, I think we can 馃檭. Let me look 馃槈 |
2aa784d
to
9d0200b
Compare
The following is the coverage report on the affected files.
|
When deleting an object, we don't need to pursue all the validation that we do at creation. It reduces the work to be done as part of the validation *and* allows invalid version of the resource (from previous versions for example) to be deleted safely. Signed-off-by: Vincent Demeester <vdemeest@redhat.com>
9d0200b
to
afd21f7
Compare
The following is the coverage report on the affected files.
|
/lgtm |
Changes
When deleting an object, we don't need to pursue all the validation
that we do at creation. It reduces the work to be done as part of the
validation and allows invalid version of the resource (from previous
versions for example) to be deleted safely.
Signed-off-by: Vincent Demeester vdemeest@redhat.com
The main reasoning for this change is to not face issues to reduce the situation where an upgrade makes object not deletable (because of newly invalid field). The example that happen upgrading from 0.10.x to 0.12.x make
EventListener
not deletable at all./kind bug
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:
See the contribution guide for more details.
Release Notes