Skip to content

Junos: flesh out more tunnel grammar, specifically destination-udp-port#9238

Merged
dhalperi merged 1 commit intomasterfrom
spr/master/4c71b233
Oct 8, 2024
Merged

Junos: flesh out more tunnel grammar, specifically destination-udp-port#9238
dhalperi merged 1 commit intomasterfrom
spr/master/4c71b233

Conversation

@dhalperi
Copy link
Copy Markdown
Member

@dhalperi dhalperi commented Oct 8, 2024

Stack:

⚠️ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

@batfish-bot
Copy link
Copy Markdown

This change is Reviewable

Base automatically changed from spr/master/433a04fa to master October 8, 2024 16:07
@dhalperi dhalperi force-pushed the spr/master/4c71b233 branch from ca15d66 to e8574ba Compare October 8, 2024 17:38
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.72%. Comparing base (e3a110c) to head (e8574ba).
Report is 54 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9238      +/-   ##
==========================================
- Coverage   72.73%   72.72%   -0.01%     
==========================================
  Files        3314     3314              
  Lines      170017   170025       +8     
  Branches    20046    20046              
==========================================
- Hits       123658   123655       -3     
- Misses      37197    37207      +10     
- Partials     9162     9163       +1     
Files with missing lines Coverage Δ
...fish/grammar/flatjuniper/ConfigurationBuilder.java 69.46% <100.00%> (+0.06%) ⬆️

... and 4 files with indirect coverage changes

@dhalperi dhalperi enabled auto-merge (squash) October 8, 2024 17:57
Copy link
Copy Markdown
Contributor

@SLarkworthy SLarkworthy left a comment

Choose a reason for hiding this comment

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

:lgtm:

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

@dhalperi dhalperi merged commit 50fc45d into master Oct 8, 2024
@dhalperi dhalperi deleted the spr/master/4c71b233 branch October 8, 2024 18:18
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