Skip to content

Commit

Permalink
bazci: fix output path computation
Browse files Browse the repository at this point in the history
These updates were happening in-place so `bazci` was constructing big,
silly paths like `backupccl_test/shard_6_of_16/shard_7_of_16/shard_13_of_16/...`
We just need to copy the variable here.

Release justification: Non-production code changes
Release note: None
  • Loading branch information
rickystewart committed Aug 30, 2022
1 parent 995950d commit 90b139c
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions pkg/cmd/bazci/bazci.go
Original file line number Diff line number Diff line change
Expand Up @@ -185,6 +185,7 @@ func processBuildEventProtocolLog(action, bepLoc string) error {
outputDir = filepath.Join("bazel-testlogs", outputDir)
summary := event.GetTestSummary()
for _, testResult := range testResults[label] {
outputDir := outputDir
if testResult.run > 1 {
outputDir = filepath.Join(outputDir, fmt.Sprintf("run_%d", testResult.run))
}
Expand Down

0 comments on commit 90b139c

Please sign in to comment.