pantsbuild / pants / 5428 / 2
0%
main: 93%

Ran 23 Jun 2015 10:04PM UTC

Files 331

Run time 15s

Badge

Committed 23 Jun 2015 10:03PM UTC coverage: 67.268% (-0.008%) from 67.276%

Job # CI_FLAGS="-fkmsrcn -u 0/2 'Unit tests for pants and pants-plugins - shard 1'"

Build Type

push

travis-ci

Committed by Benjy

Commit Message

An artifact cache subsystem.

This was no small endeavor, as the artifact cache code had
quite a few implicit tentacles out into the codebase.

This creates a CacheSetup subsystem, which in turn gives tasks
a CacheFactory, which is the thing that actually creates caches, based
on option values. The CacheFactory is necessary in order to encapsulate
non-option arguments such as the logger and the stable_name.

The resulting code is simpler and easier to grok (although the diffs
may not be). For example, tasks no longer need to explicitly set up
the cache; It gets set up as needed. Also, code that needs the read
cache accesses it directly, and ditto the write cache. The confusing
ReadWriteArtifactCache class is no longer needed, and has been deleted.

Also simplifies names of options and variables: 'artifact_cache'
is a mouthful, when there's only one kind of cache that 99% of users
care about - namely this one.

This is the first case we have of task-specific Subsystems, so there
are a few tweaks here and there to get that working properly.

This pecial-cases migration of the old cache-related options, which
required a little refactoring in migrate_config.py.

Testing Done:
Lots of manual testing to verify that the cache ends up used as expected with the new config locations. Also checked that the use of the cache to optimize tool bootstrapping in tests still works.

CI went green here: https://travis-ci.org/pantsbuild/pants/builds/68066786

Reviewed at https://rbcommons.com/s/twitter/r/2405/

Coverage Stats

13352 of 19849 relevant lines covered (67.27%)

0.67 hits per line

Source Files on job 5428.2 (CI_FLAGS="-fkmsrcn -u 0/2 'Unit tests for pants and pants-plugins - shard 1'")