diff --git a/CHANGELOG.md b/CHANGELOG.md index 0f573b42ab..fa6032cdb7 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -33,6 +33,7 @@ New tracing span: ### Fixed - [#921](https://github.com/improbable-eng/thanos/pull/921) `thanos_objstore_bucket_last_successful_upload_time` now does not appear when no blocks have been uploaded so far +- [#966](https://github.com/improbable-eng/thanos/pull/966) Bucket: verify no longer warns about overlapping blocks, that overlap `0s` ## [v0.3.2](https://github.com/improbable-eng/thanos/releases/tag/v0.3.2) - 2019.03.04 diff --git a/pkg/verifier/overlapped_blocks.go b/pkg/verifier/overlapped_blocks.go index 675442ace7..072fe54aec 100644 --- a/pkg/verifier/overlapped_blocks.go +++ b/pkg/verifier/overlapped_blocks.go @@ -2,7 +2,6 @@ package verifier import ( "context" - "github.com/go-kit/kit/log" "github.com/go-kit/kit/log/level" "github.com/improbable-eng/thanos/pkg/block" @@ -11,6 +10,7 @@ import ( "github.com/oklog/ulid" "github.com/pkg/errors" "github.com/prometheus/tsdb" + "sort" ) const OverlappedBlocksIssueID = "overlapped_blocks" @@ -66,6 +66,11 @@ func fetchOverlaps(ctx context.Context, logger log.Logger, bkt objstore.Bucket) overlaps := map[string]tsdb.Overlaps{} for k, groupMetas := range metas { + + sort.Slice(groupMetas, func(i, j int) bool { + return groupMetas[i].MinTime < groupMetas[j].MinTime + }) + o := tsdb.OverlappingBlocks(groupMetas) if len(o) > 0 { overlaps[k] = o