kruise
kruise copied to clipboard
add ephemeraljob validating webhook
Ⅰ. Describe what this PR does
fix #1614
Ⅱ. Does this pull request fix one issue?
Ⅲ. Describe how to verify it
Invalid ejob must be rejected by webhook.
For example:
Ⅳ. Special notes for reviews
Codecov Report
Attention: Patch coverage is 70.33898%
with 35 lines
in your changes missing coverage. Please review.
Project coverage is 49.16%. Comparing base (
0d0031a
) to head (7b495c6
). Report is 40 commits behind head on master.
Additional details and impacted files
@@ Coverage Diff @@
## master #1615 +/- ##
==========================================
+ Coverage 47.91% 49.16% +1.24%
==========================================
Files 162 183 +21
Lines 23491 18921 -4570
==========================================
- Hits 11256 9302 -1954
+ Misses 11014 8398 -2616
Partials 1221 1221
Flag | Coverage Δ | |
---|---|---|
unittests | 49.16% <70.33%> (+1.24%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: furykerry
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [furykerry]
Approvers can indicate their approval by writing /approve
in a comment
Approvers can cancel approval by writing /approve cancel
in a comment