-
Notifications
You must be signed in to change notification settings - Fork 383
T7489: Fix output state of ipsec passthrough child #4616
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
T7489: Fix output state of ipsec passthrough child #4616
Conversation
All contributors have signed the CLA ✍️ ✅ |
👍 |
69dfbb9
to
afc4389
Compare
❌ Conflicts Found. This pull request has conflicts. Please resolve them before we can evaluate the pull request. |
I have read the CLA Document and I hereby sign the CLA |
afc4389
to
6d6b441
Compare
Show state of passthrough tunnels as always up. Passthrough children of connection have PASS mode but have no sa and are not shown in vici list_sas. Fix by passing mode from vici list_connections to _get_child_sa_state and always return 'up' for child with PASS mode.
6d6b441
to
6060c19
Compare
✅ Conflicts Resolved. Conflicts have been resolved. A maintainer will review the pull request shortly. |
recheck |
CI integration 👍 passed! Details
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The passthrough mode is always up
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Passthrough indeed has no state in the same sense as normal tunnels, no objections.
Fix output state of ipsec passthrough child
Show state of passthrough tunnels as always up.
Passthrough children of connection have PASS mode but have no sa and are not shown in vici list_sas.
Fix by passing mode from vici list_connections to _get_child_sa_state and always return 'up' for child with PASS mode.
Types of changes
Related Task(s)
Related PR(s)
How to test / Smoketest result
Setup VPN IPSEC with one network being subnetwork of another e.g.
Commands to setup these machines:
VyOS-central:
VyOS-remote
Run on vyos-remote
show vpn ipsec connections
:Before the fix
PEERCENTRAL-tunnel-0-passthrough
wasdown
. After fix it isup
even if VPN is down as passthrough connection is available anyway.Checklist: