Todd Lipcon has posted comments on this change. ( )

Change subject: build: enable sharding within cmake/ctest

Patch Set 2:

Commit Message:
PS2, Line 26: * Flaky-test tracking is currently still done by the test binary 
            :   and not the specific shard, though we could easily switch that 
            :   the future.
> This gets weird though because the number of shards may change over time, w
yea, that's why I didn't change for now.. on the other hand it can be useful to 
know which sub-shard is actually flaky. I suppose longest term it woudl be nice 
to even track each test-case separately but that woudl require more mechanics
File CMakeLists.txt:
PS2, Line 678: If a test suite is long enough
             : #       to require a bumped timeout, consider enabling sharding 
of the
             : #       test by adding it to the NUM_SHARDS_BY_TEST dictionary 
> Update.
File build-support/
PS2, Line 56: TEST_COMMAND_RE = re.compile('Test command: (.+)$')
            : TEST_ENV_RE = re.compile('^\d+:  (\S+)=(.+)')
            : LDD_RE = re.compile(r'^\s+.+? => (\S+) \(0x.+\)')
> Would be nice to include an example string for each of these.
PS2, Line 322:                       name=execution.test_name,
> Note that prior to your change the name of the test also included the total
it seems like this 'name' actually never ends up displayed anywhere best I can 
tell. It probably corresponds to some ancient version of isolate. I'll just 
remove it.
PS2, Line 442:   execution = dict(argv=(["", options.cmd] + 
options.args), env={})
> This isn't right, I think it should be:
yea, though with looping a test you specify the whole command line yourself and 
typically filter to a single test. In other words, I almost always used the 
loop mode along with --disable-sharding. I couldn't figure out the best way to 
modify this for the new scheme. But you're right I also broke the existing 

To view, visit
To unsubscribe, visit

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I20ddbdd73a64fda3fe32fca98ee541aa4cead4b3
Gerrit-Change-Number: 9470
Gerrit-PatchSet: 2
Gerrit-Owner: Todd Lipcon <>
Gerrit-Reviewer: Adar Dembo <>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <>
Gerrit-Comment-Date: Sat, 03 Mar 2018 00:32:50 +0000
Gerrit-HasComments: Yes

Reply via email to