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

Increase transientTimeout of boltdb to 2mimute #135

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

Conversation

coolljt0725
Copy link

During docker stress test, there are a lot of
timeout errors on update libnetwork object to store.
ERRO[0137] docker: Error response from daemon: failed to update store for object type *libnetwork.endpoint: timeout. error=exit status 125

I think we should increase the timeout to 2m because 2m is also the
timeout of container starting timeout of docker.

Signed-off-by: Lei Jitang leijitang@huawei.com

During docker stress test, there are a lot of
timeout errors on update libnetwork object to store.
`ERRO[0137] docker: Error response from daemon: failed
to update store for object type *libnetwork.endpoint: timeout.
  error=exit status 125`

I think we should increase the timeout to 2m because 2m is also the
timeout of container starting timeout of docker.

Signed-off-by: Lei Jitang <leijitang@huawei.com>
@coolljt0725 coolljt0725 changed the title Increase transientTimeout of boltdb to 2m Increase transientTimeout of boltdb to 2mimute Aug 4, 2016
@thaJeztah
Copy link
Member

ping @abronan PTAL

@abronan
Copy link
Contributor

abronan commented Aug 11, 2016

I don't know if this is the right approach to bump the default transient timeout to fix the tests (hitting the error mostly because of limited resources on the CI), imho the field should be configurable with libnetwork and using a custom value for testing. But I'm not familiar with how libnetwork configures boltdb.

ping @sanimej as this impacts boltdb and libnetwork.

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

Successfully merging this pull request may close these issues.

3 participants