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

Automated backport of #2710: Retry IPsec connections when there are any whack errors #2717

Conversation

sridhargaddam
Copy link
Member

Backport of #2710 on release-0.14.

#2710: Retry IPsec connections when there are any whack errors

For details on the backport process, see the backport requests page.

In the current code, when 'whack' encounters an error, we only log
a warning message without hanlding the error case. As a result, in
cases where there is an issue connecting to a remote endpoint, the
caller is left unaware that connection attempts have failed, and
no retries are initiated. This PR fixes it.

Signed-off-by: Sridhar Gaddam <[email protected]>
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr2717/sridhargaddam/automated-backport-of-#2710-origin-release-0.14
🚀 Full E2E won't run until the "ready-to-test" label is applied. I will add it automatically once the PR has 2 approvals, or you can add it manually.

@submariner-bot submariner-bot added the ready-to-test When a PR is ready for full E2E testing label Sep 21, 2023
@tpantelis tpantelis enabled auto-merge (rebase) September 21, 2023 12:57
@tpantelis tpantelis merged commit 4de4a96 into submariner-io:release-0.14 Sep 21, 2023
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr2717/sridhargaddam/automated-backport-of-#2710-origin-release-0.14]

@dfarrell07 dfarrell07 added the release-note-needed Should be mentioned in the release notes label Sep 26, 2023
@dfarrell07
Copy link
Member

This can be release-noted together with the other related IPSec change, #2708.

@sridhargaddam
Copy link
Member Author

Release note PR: submariner-io/submariner-website#1040

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automated-backport ready-to-test When a PR is ready for full E2E testing release-note-handled release-note-needed Should be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants