pantsbuild / pants / 5428
0%
main: 93%

LAST BUILD BRANCH: feat-configurable-nailgun-jar
DEFAULT BRANCH: main
Ran
Jobs 10
Files 331
Run time 9min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
5428

push

travis-ci

Benjy 
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/

14106 of 19849 relevant lines covered (71.07%)

1.34 hits per line

Jobs
ID Job ID Ran Files Coverage
1 5428.1 (CI_FLAGS="-cjlpn 'Various pants self checks'") 0
Travis Job 5428.1
2 5428.2 (CI_FLAGS="-fkmsrcn -u 0/2 'Unit tests for pants and pants-plugins - shard 1'") 0
67.27
 Travis Job 5428.2
3 5428.3 (CI_FLAGS="-fkmsrcn -u 1/2 'Unit tests for pants and pants-plugins - shard 2'") 0
66.27
 Travis Job 5428.3
4 5428.4 (CI_FLAGS="-fkmsrcjlp 'Python contrib tests'") 0
Travis Job 5428.4
5 5428.5 (CI_FLAGS="-fkmsrjlpn -i 0/6 'Python integration tests for pants - shard 1'") 0
Travis Job 5428.5
6 5428.6 (CI_FLAGS="-fkmsrjlpn -i 1/6 'Python integration tests for pants - shard 2'") 0
Travis Job 5428.6
7 5428.7 (CI_FLAGS="-fkmsrjlpn -i 2/6 'Python integration tests for pants - shard 3'") 0
Travis Job 5428.7
8 5428.8 (CI_FLAGS="-fkmsrjlpn -i 3/6 'Python integration tests for pants - shard 4'") 0
Travis Job 5428.8
9 5428.9 (CI_FLAGS="-fkmsrjlpn -i 4/6 'Python integration tests for pants - shard 5'") 0
Travis Job 5428.9
10 5428.10 (CI_FLAGS="-fkmsrjlpn -i 5/6 'Python integration tests for pants - shard 6'") 0
Travis Job 5428.10
Source Files on build 5428
Detailed source file information is not available for this build.