Skip to content

Commit b57ea31

Browse files
craig[bot]herkolategan
andcommitted
Merge #143790
143790: microbench-ci: fix marker file name r=DarrylWong a=herkolategan Previously, the marker file started with a dot, which made it invisible in the directory listing and copy operation ran during the GitHub actions to copy to GCS. It's possible to update the copy operation to include the dot, but it's better to standardize the name to start with an underscore. Epic: None Release note: None Co-authored-by: Herko Lategan <[email protected]>
2 parents 02912b5 + 4c99f0c commit b57ea31

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

pkg/cmd/microbench-ci/run.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ func (b *Benchmark) run() error {
124124
// Write change marker file if the benchmark changed.
125125
if status != NoChange {
126126
marker := strings.ToUpper(status.String())
127-
err := os.WriteFile(path.Join(suite.artifactsDir(New), "."+marker), nil, 0644)
127+
err := os.WriteFile(path.Join(suite.artifactsDir(New), "_"+marker), nil, 0644)
128128
if err != nil {
129129
return err
130130
}

pkg/cmd/microbench-ci/testdata/regression.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ tree
157157
/github-summary.md
158158
/qwerty456
159159
/qwerty456/artifacts
160-
/qwerty456/artifacts/.REGRESSED
160+
/qwerty456/artifacts/_REGRESSED
161161
/qwerty456/artifacts/cleaned_Sysbench_SQL_3node_oltp_read_write.log
162162
/qwerty456/artifacts/cpu_Sysbench_SQL_3node_oltp_read_write_merged.prof
163163
/qwerty456/artifacts/memory_Sysbench_SQL_3node_oltp_read_write_merged.prof

0 commit comments

Comments
 (0)