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 process should handle 401 error #30

Closed
nati opened this issue Nov 9, 2015 · 0 comments
Closed

etcd watch process should handle 401 error #30

nati opened this issue Nov 9, 2015 · 0 comments

Comments

@nati
Copy link
Contributor

nati commented Nov 9, 2015

etcd watch process should handle 401 error.

What's happening in our env.

(1) resources more than 500 rows
(2) boot gohan
(3) state watch process will lock for each
(4) etcd stores up to 1000 history, so we lost some history and got 401 error

nati added a commit that referenced this issue Nov 9, 2015
Closes: issue #30

Etcd can hold up to 1000 history, so we need to handle this case
correctly.
We have following issues.

(1) We should clone env for each extension run by sync
(2) sync.Watch API keep blocking even if we get 401.
(3) Stop lock per resource
  In case we have more than 1000 resources, sync process do infinite
  loop because it increases etcd index by lock.
(4) sync packages is using thread-unsafe map

In this commit, we fix these issues,  and we added simple integration
test configuraion file.
nati added a commit that referenced this issue Nov 9, 2015
Closes: issue #30

Etcd can hold up to 1000 history, so we need to handle this case
correctly.
We have following issues.

(1) We should clone env for each extension run by sync
(2) sync.Watch API keep blocking even if we get 401.
(3) Stop lock per resource
  In case we have more than 1000 resources, sync process do infinite
  loop because it increases etcd index by lock.
(4) sync packages is using thread-unsafe map

In this commit, we fix these issues,  and we added simple integration
test configuraion file.
@nati nati mentioned this issue Nov 11, 2015
@nati nati closed this as completed Nov 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant