Add smoketest to ensure swarming named caches are persistent |
||||||
Issue descriptionAI: Modify luci-py/appengine/swarming/local_smoke_test.py to add a new test case that shall do: - trigger a task that requests a named cache and put something in there - trigger a second task and ensure the cache is still populated. - update tools/stat_bot.py wipe_cache() to wipe any named cache.
,
Nov 22 2016
Will try to do it in a not too distant future.
,
Jan 17 2017
,
Jan 17 2017
,
Jan 17 2017
,
Jan 18 2017
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/83e5bed8895f64f184dd994d721420835c9cbb64 commit 83e5bed8895f64f184dd994d721420835c9cbb64 Author: maruel <maruel@chromium.org> Date: Wed Jan 18 00:56:03 2017 swarming: Fix named cache support. - Add support for named cache in swarming.py so it can be used via the CLI. - Reshuffled the swarming.py trigger option grouping to differentiate between properties and request metadata. - Add integration smoke test to ensure it actually work. - Fix named cache bug, it didn't work: free_space wasn't initialized so all the named caches were deleted after use if min_free_space wasn't None. - made min_free_space == 0 use the normal path. - Add logging to help diagnose problems. R=vadimsh@chromium.org BUG= 667290 Review-Url: https://codereview.chromium.org/2636993002 [modify] https://crrev.com/83e5bed8895f64f184dd994d721420835c9cbb64/appengine/swarming/local_smoke_test.py [modify] https://crrev.com/83e5bed8895f64f184dd994d721420835c9cbb64/appengine/swarming/tools/start_bot.py [modify] https://crrev.com/83e5bed8895f64f184dd994d721420835c9cbb64/client/named_cache.py [modify] https://crrev.com/83e5bed8895f64f184dd994d721420835c9cbb64/client/swarming.py [modify] https://crrev.com/83e5bed8895f64f184dd994d721420835c9cbb64/client/tests/swarming_test.py
,
Jan 18 2017
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/83e5bed8895f64f184dd994d721420835c9cbb64 commit 83e5bed8895f64f184dd994d721420835c9cbb64 Author: maruel <maruel@chromium.org> Date: Wed Jan 18 00:56:03 2017 swarming: Fix named cache support. - Add support for named cache in swarming.py so it can be used via the CLI. - Reshuffled the swarming.py trigger option grouping to differentiate between properties and request metadata. - Add integration smoke test to ensure it actually work. - Fix named cache bug, it didn't work: free_space wasn't initialized so all the named caches were deleted after use if min_free_space wasn't None. - made min_free_space == 0 use the normal path. - Add logging to help diagnose problems. R=vadimsh@chromium.org BUG= 667290 Review-Url: https://codereview.chromium.org/2636993002 [modify] https://crrev.com/83e5bed8895f64f184dd994d721420835c9cbb64/appengine/swarming/local_smoke_test.py [modify] https://crrev.com/83e5bed8895f64f184dd994d721420835c9cbb64/appengine/swarming/tools/start_bot.py [modify] https://crrev.com/83e5bed8895f64f184dd994d721420835c9cbb64/client/named_cache.py [modify] https://crrev.com/83e5bed8895f64f184dd994d721420835c9cbb64/client/swarming.py [modify] https://crrev.com/83e5bed8895f64f184dd994d721420835c9cbb64/client/tests/swarming_test.py
,
Jan 18 2017
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by estaab@chromium.org
, Nov 22 2016