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

Fix panic when create spidermultusconfig with nil podRPFilter in validate webhook #4078

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

weizhoublue
Copy link
Collaborator

robot cherry pick PR #4062 from cyclinder,to branch release-v0.9, action https://github.com/spidernet-io/spiderpool/actions/runs/10933291536 , commits 603c57e

Fix panic when create spidermultusconfig with nil podRPFilter in validate webhook

Signed-off-by: robot <[email protected]>
Copy link

codecov bot commented Sep 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.60%. Comparing base (8389d30) to head (6771a9b).
Report is 2 commits behind head on release-v0.9.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff                @@
##           release-v0.9    #4078      +/-   ##
================================================
+ Coverage         81.56%   81.60%   +0.04%     
================================================
  Files                50       50              
  Lines              4355     4355              
================================================
+ Hits               3552     3554       +2     
+ Misses              641      640       -1     
+ Partials            162      161       -1     
Flag Coverage Δ
unittests 81.60% <ø> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1 file with indirect coverage changes

@cyclinder cyclinder merged commit 1637b83 into release-v0.9 Sep 20, 2024
53 checks passed
@cyclinder cyclinder deleted the robot/cherrypick/pr4062/release-v0.9 branch September 20, 2024 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants