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

Cache timeout tests often fail. #1361

Closed
wants to merge 1 commit into from
Closed

Conversation

garenchan
Copy link
Contributor

  1. We currently use virtual machine as CI environment. It runs too
    slowly, causing timeout-related tests to fail easily. will we consider
    using docker as CI environment?
  2. We can set the cache item's timeout to a large value.Since we have
    fast_sleep, don't worry about waiting for a long time until cache item
    expired.

1. We currently use virtual machine as CI environment. It runs too
slowly, causing timeout-related tests to fail easily. will we consider
using docker as CI environment?
2. We can set the cache item's timeout to a large value.Since we have
`fast_sleep`, don't worry about waiting for a long time until cache item
expired.
@garenchan
Copy link
Contributor Author

image

image

@garenchan garenchan closed this Sep 12, 2018
@davidism
Copy link
Member

We're aware of this. We've tried to fix this multiple times. I have some plans but for now I just rerun the tests when I notice it.

@garenchan garenchan reopened this Sep 12, 2018
@davidism
Copy link
Member

There are some cache tests that can't use fast_sleep, so bumping the sleep time makes the tests take a very long time.

@lepture
Copy link
Contributor

lepture commented Sep 13, 2018

The good solution could be #1249

@davidism
Copy link
Member

Closing this since we're not going to be maintaining the cache contrib. I'm going to disable tests for now so they don't cause problems with other PRs.

@davidism davidism closed this Nov 12, 2018
@garenchan garenchan deleted the dev2 branch December 3, 2018 06:16
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 13, 2020
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.

None yet

3 participants