Skip to content

Commit

Permalink
Merge pull request prometheus#12199 from colega/fix-test-cancel-compa…
Browse files Browse the repository at this point in the history
…ctions-on-windows

Fix TestCancelCompactions on windows
  • Loading branch information
codesome authored Mar 30, 2023
2 parents ae42dd4 + 3ded84e commit b8a9ced
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions tsdb/compact_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1171,12 +1171,9 @@ func TestCancelCompactions(t *testing.T) {
createBlock(t, tmpdir, genSeries(1, 1, 2000, 2001)) // The most recent block is ignored so can be e small one.

// Copy the db so we have an exact copy to compare compaction times.
tmpdirCopy := tmpdir + "Copy"
tmpdirCopy := t.TempDir()
err := fileutil.CopyDirs(tmpdir, tmpdirCopy)
require.NoError(t, err)
defer func() {
require.NoError(t, os.RemoveAll(tmpdirCopy))
}()

// Measure the compaction time without interrupting it.
var timeCompactionUninterrupted time.Duration
Expand Down Expand Up @@ -1228,6 +1225,7 @@ func TestCancelCompactions(t *testing.T) {
for i, b := range blocks {
require.Falsef(t, b.Meta().Compaction.Failed, "block %d (%s) should not be marked as compaction failed", i, b.Meta().ULID)
}
require.NoError(t, readOnlyDB.Close())
}
}

Expand Down

0 comments on commit b8a9ced

Please sign in to comment.