Skip to content
This repository has been archived by the owner on Jul 20, 2023. It is now read-only.

fix(fl-operator): reconnect on reconcile to handle failures with envoy #42

Open
wants to merge 1 commit into
base: beta
Choose a base branch
from

Conversation

nfnt
Copy link
Member

@nfnt nfnt commented Jan 2, 2023

In early cluster stage, envoy doesn't yet work as expected and needs a new gRPC connection once it does. This causes a problem with this controller as it caches the (falsy) connection once a first one is established. To fix this, we create a new connection every time the reconcile loop is run.

@nfnt nfnt requested a review from a team January 2, 2023 14:40
In early cluster stage, envoy doesn't yet work as expected and needs a new gRPC
connection once it does. This causes a problem with this controller as it caches
the (falsy) connection once a first one is established. To fix this, we create a
new connection every time the reconcile loop is run.
@nfnt nfnt force-pushed the jan/fix-falsy-grpc-connection branch from 73a21c4 to 376e8df Compare January 2, 2023 14:47
@nfnt nfnt changed the title fix: reconnect on reconcile to handle failures with envoy fix(fl-operator): reconnect on reconcile to handle failures with envoy Jan 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant