Skip to content

Junos: add some ipsec crash protection#9152

Merged
dhalperi merged 1 commit intomasterfrom
spr/master/74ad995c
Aug 9, 2024
Merged

Junos: add some ipsec crash protection#9152
dhalperi merged 1 commit intomasterfrom
spr/master/74ad995c

Conversation

@dhalperi
Copy link
Copy Markdown
Member

@dhalperi dhalperi commented Aug 9, 2024

No description provided.

@batfish-bot
Copy link
Copy Markdown

This change is Reviewable

@codecov
Copy link
Copy Markdown

codecov bot commented Aug 9, 2024

Codecov Report

Attention: Patch coverage is 90.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 72.70%. Comparing base (12d3018) to head (85d0ea1).
Report is 73 commits behind head on master.

Files with missing lines Patch % Lines
...h/representation/juniper/JuniperConfiguration.java 90.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #9152   +/-   ##
=======================================
  Coverage   72.69%   72.70%           
=======================================
  Files        3314     3314           
  Lines      169832   169843   +11     
  Branches    19995    19999    +4     
=======================================
+ Hits       123467   123477   +10     
- Misses      37202    37205    +3     
+ Partials     9163     9161    -2     
Files with missing lines Coverage Δ
...h/representation/juniper/JuniperConfiguration.java 86.09% <90.00%> (+0.34%) ⬆️

... and 2 files with indirect coverage changes

Copy link
Copy Markdown
Contributor

@progwriter progwriter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @anothermattbrown)

@dhalperi dhalperi merged commit 687112a into master Aug 9, 2024
@dhalperi dhalperi deleted the spr/master/74ad995c branch August 9, 2024 16:34
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