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: Fix IPSec tun protect + add IPSec e2e test #1654

Merged
merged 3 commits into from
Apr 28, 2020

Conversation

rastislavs
Copy link
Contributor

Signed-off-by: Rastislav Szabo raszabo@cisco.com

Signed-off-by: Rastislav Szabo <raszabo@cisco.com>
Signed-off-by: Rastislav Szabo <raszabo@cisco.com>
Signed-off-by: Rastislav Szabo <raszabo@cisco.com>
@codecov
Copy link

codecov bot commented Apr 27, 2020

Codecov Report

Merging #1654 into master will increase coverage by 0.46%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1654      +/-   ##
==========================================
+ Coverage   57.65%   58.11%   +0.46%     
==========================================
  Files         495      293     -202     
  Lines       40641    23850   -16791     
==========================================
- Hits        23431    13861    -9570     
+ Misses      14754     8858    -5896     
+ Partials     2456     1131    -1325     
Flag Coverage Δ
#e2e ?
#unittests 58.11% <ø> (ø)

@sonarcloud
Copy link

sonarcloud bot commented Apr 28, 2020

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability B 2 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 2 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@rastislavs rastislavs changed the title Add IPSec e2e test fix: Fix IPSec tun protect + add IPSec e2e test Apr 28, 2020
@ondrej-fabry ondrej-fabry merged commit a8f52a6 into ligato:master Apr 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants