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

Allow halting on certificate errors #2761

Merged
merged 1 commit into from
Oct 25, 2023

Conversation

skitt
Copy link
Member

@skitt skitt commented Oct 23, 2023

When certificate errors are encountered, the fix is usually to restart the affected pod. To allow this to happen automatically, add a configuration setting for the gateway agent.

The setting is disabled by default; it will be enabled by default by the operator.

Depends on submariner-io/admiral#767

@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr2761/skitt/fatal-certificate-errors
🚀 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.

@skitt
Copy link
Member Author

skitt commented Oct 23, 2023

Let’s wait for submariner-io/admiral#767 instead (if deemed appropriate).

@skitt skitt force-pushed the fatal-certificate-errors branch from 274b08d to 0480578 Compare October 23, 2023 15:26
@skitt skitt added the backport This change requires a backport to eligible release branches label Oct 23, 2023
@skitt skitt force-pushed the fatal-certificate-errors branch 3 times, most recently from 970ef4c to 70123a9 Compare October 24, 2023 07:59
@github-actions
Copy link
Contributor

This PR/issue depends on:

When certificate errors are encountered, the fix is usually to restart
the affected pod. To allow this to happen automatically, add a
configuration setting for the gateway agent; extend the mechanism to
the route agent.

The setting is disabled by default; it will be enabled by default by
the operator.

Signed-off-by: Stephen Kitt <skitt@redhat.com>
@skitt skitt force-pushed the fatal-certificate-errors branch from 0b70cfc to f68ec6f Compare October 25, 2023 08:36
@submariner-bot submariner-bot added the ready-to-test When a PR is ready for full E2E testing label Oct 25, 2023
@tpantelis tpantelis enabled auto-merge (rebase) October 25, 2023 11:51
@tpantelis tpantelis merged commit 12ab267 into submariner-io:devel Oct 25, 2023
41 checks passed
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr2761/skitt/fatal-certificate-errors]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport This change requires a backport to eligible release branches backport-handled ready-to-test When a PR is ready for full E2E testing
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants