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

spidermultusconfig: validate customcniconfig if is valid #2153

Merged
merged 1 commit into from Aug 8, 2023

Conversation

cyclinder
Copy link
Collaborator


name: Pull request
about: Tell us about your contribution
labels: ["pr/release/none-required"]


Thanks for contributing !

Before submitting a pull request, make sure you read about our Contribution notice here: https://spidernet-io.github.io/spiderpool/contributing/development/pullrequest/#need-review

What this PR does / why we need it:

spidermultusconfig: validate customcniconfig if is valid

Which issue(s) this PR fixes:

#2151

Special notes for your reviewer:

make sure your commit is signed off

@codecov
Copy link

codecov bot commented Aug 8, 2023

Codecov Report

Merging #2153 (c767310) into main (71c15e0) will decrease coverage by 0.52%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2153      +/-   ##
==========================================
- Coverage   81.68%   81.17%   -0.52%     
==========================================
  Files          49       49              
  Lines        5210     5210              
==========================================
- Hits         4256     4229      -27     
- Misses        792      823      +31     
+ Partials      162      158       -4     
Flag Coverage Δ
unittests 81.17% <ø> (-0.52%) ⬇️

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

see 1 file with indirect coverage changes

@weizhoublue
Copy link
Collaborator

update the test/doc

@weizhoublue weizhoublue added the cherrypick-release-v0.6 Cherry-pick the PR to branch release-v0.6. label Aug 8, 2023
@weizhoublue weizhoublue merged commit 14a02fe into spidernet-io:main Aug 8, 2023
35 of 36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherrypick-release-v0.6 Cherry-pick the PR to branch release-v0.6.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants