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

Etcd watch response order #30

Closed
styleex opened this issue Aug 3, 2017 · 4 comments
Closed

Etcd watch response order #30

styleex opened this issue Aug 3, 2017 · 4 comments
Assignees
Labels

Comments

@styleex
Copy link

styleex commented Aug 3, 2017

WatchManager has potencial bug: etcd does not give guarantees about WatchResponse
(etcd-io/etcd#7036)

@connor4312
Copy link
Member

Thanks for opening an issue! To clarify, you're referring to the problem discussed in this comment etcd-io/etcd#7036 (comment)?

@styleex
Copy link
Author

styleex commented Aug 3, 2017

yes. In my mini lib on python I consistently initialize each observer. Those. First I wait for an answer to the first request, then I send the second one and wait for an answer, and so on. This works fine

@connor4312 connor4312 mentioned this issue Aug 9, 2017
@connor4312 connor4312 self-assigned this Aug 9, 2017
@connor4312 connor4312 added the bug label Aug 9, 2017
@connor4312
Copy link
Member

Thanks for the issue! Fix submitted in the linked PR

@connor4312
Copy link
Member

Fixed in #33

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

No branches or pull requests

2 participants