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

watcher: Register Exception Handler before watch request #1287

Merged
merged 1 commit into from
Dec 14, 2023

Conversation

giri-vsr
Copy link
Contributor

Registers Exception handler before watch request so that if exception happens watch can resume.

Fixes #1261

Signed-off-by: Giri Ramasamy <46867712+giri-vsr@users.noreply.github.com>
@giri-vsr
Copy link
Contributor Author

@lburgazzoli can you please check this?

@lburgazzoli
Copy link
Collaborator

is there a chance to add a test case for this ?

@giri-vsr
Copy link
Contributor Author

is there a chance to add a test case for this ?

I couldn't replicate the issue with test containers.But the issue happens when we have this in k8s

@lburgazzoli
Copy link
Collaborator

have you validated the change on k8s ?

@giri-vsr
Copy link
Contributor Author

have you validated the change on k8s ?

Haven't validated yet.Will validate and confirm.Is it possible to get a SNAPSHOT version for this PR?

@lburgazzoli lburgazzoli merged commit 3d66295 into etcd-io:main Dec 14, 2023
9 checks passed
@lburgazzoli
Copy link
Collaborator

once the main build ends, a snapshot should be published to the snapshot repo

@giri-vsr
Copy link
Contributor Author

giri-vsr commented Dec 15, 2023

@lburgazzoli I have tested with snapshot from main in k8s and It is working now.

@giri-vsr
Copy link
Contributor Author

@lburgazzoli when can I expect next release?

@lburgazzoli
Copy link
Collaborator

I can try to do a release today or beginning next week

@lburgazzoli
Copy link
Collaborator

0.7.7 should be available to maven central soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

watch without waitForReady sometimes not reschedule when all servers are down
2 participants