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

Update LNW + IPsec documentation for 1.8 #28

Merged
merged 3 commits into from
Aug 12, 2024
Merged

Update LNW + IPsec documentation for 1.8 #28

merged 3 commits into from
Aug 12, 2024

Conversation

5abeel
Copy link
Collaborator

@5abeel 5abeel commented Aug 8, 2024

Changes in this PR

  • Updated all load_custom_config.sh to make it generic, with a comment to indicate two fields are for LAG and IPsec usecase
  • Update the IPsec doc to point that we now use the LNW combined recipe
  • Remove the second half of the IPsec doc which simply gives some design details -- something that the user does not have to handle manually. All this is done by the ipsec strongSwan plugin code
  • LAG/LACP has seen some issues when bulk programming rules -- adding a delay avoids this problem. Added a note in documentation to point this out

Copy link
Collaborator

@ffoulkes ffoulkes left a comment

Choose a reason for hiding this comment

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

Note corrections.

Otherwise, LGTM.

docs/apps/ipsec-offload.md Show resolved Hide resolved
docs/apps/lnw/es2k/es2k-lnw-underlay-lag.md Outdated Show resolved Hide resolved
docs/apps/lnw/es2k/es2k-lnw-underlay-lag.md Outdated Show resolved Hide resolved
Signed-off-by: Sabeel Ansari <[email protected]>
@ffoulkes ffoulkes merged commit 6a59f5b into main Aug 12, 2024
5 checks passed
@ffoulkes ffoulkes deleted the doc_update_1.8 branch August 12, 2024 15:42
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.

2 participants