-
Notifications
You must be signed in to change notification settings - Fork 281
Test failures on CentOS 8 #169
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
Comments
Hi @igchor, is there any possibility to check the same on the host machine (outside Docker), to eliminate the docker variable ? |
Hi, unfortunately, CacheLib dependencies do not compile on that host machine (fedora 34). I tried running Those dependencies problems are the reason why we created that docker image (to always use the same dependencies versions which we know work fine). |
Summary: This change fixes following flaky tests in NvmCacheTests. * NvmCacheTest.Delete * NvmCacheTest.NvmEvicted * NvmCacheTest.EvictToNvmGetCheckCtime The root cause of the failures are essentially the same as D42443647 (5e7ff9a) which fixed the problem for NvmCacheTest.EvictToNvmGet; we are inserting enough items that could be spilled to NVM cache, where the NvmCache::put() can be dropped and the item is evicted completely when the delete operations (and tombstones) issued as part of the insertion are still outstanding. In order to fix the problem, this change flushes the NVM cache periodically during the insertions. Also, since this could cause more regions are used, the size of NVM cache needs to be increased. This change bumps the default size of NVM cache to 200MB (previous 100MB). Also, the size of persist storage used in the test PersistenceCache has been bumped by 100MB accordingly, i.e., from 400MB to 500MB. This change addresses the github issue #169 Reviewed By: therealgymmy Differential Revision: D43592888 fbshipit-source-id: f0968884eb39fb5728b59129e98345df3240f01e
Describe the bug
List of tests failing (some randomly):
To Reproduce
Steps to reproduce the behavior:
I'm using a docker image with following steps (assuming CacheLib is in /root/CacheLib):
Expected behavior
No test failures
Desktop (please complete the following information):
Additional context
Some of the nvm tests seems to pass when I insert a sleep command after each operation.
The text was updated successfully, but these errors were encountered: