Upgrade Prometheus to master #2658
Conversation
Signed-off-by: Marco Pracucci <[email protected]>
| testutil.Ok(t, err) | ||
| defer testutil.Ok(t, h.Close()) |
There was a problem hiding this comment.
I wonder how did this not cause an issue before 😮
Signed-off-by: Marco Pracucci <[email protected]>
Signed-off-by: Marco Pracucci <[email protected]>
| @@ -17,7 +17,7 @@ jobs: | |||
| # cores (> 30) while the CPU and RAM resources are throttled. If we | |||
| # don't limit this to the number of allocated cores, the job is | |||
| # likely to get OOMed and killed. | |||
| GOOPTS: "-p 2" | |||
| GOOPTS: "-p 1" | |||
There was a problem hiding this comment.
I further reduced it because make lint was failing this way:
https://circleci.com/gh/thanos-io/thanos/9391
Signed-off-by: Ganesh Vernekar <[email protected]>
Delete mmapped chunks after creating block from head
pkg/store/postings_codec_test.go
Outdated
| @@ -14,7 +14,7 @@ import ( | |||
| ) | |||
|
|
|||
| func TestDiffVarintCodec(t *testing.T) { | |||
| h, err := tsdb.NewHead(nil, nil, nil, 1000, tsdb.DefaultStripeSize) | |||
| h, err := tsdb.NewHead(nil, nil, nil, 1000, "", nil, tsdb.DefaultStripeSize, nil) | |||
There was a problem hiding this comment.
the chunk directory should be a temp dir, will push a fix
Signed-off-by: Ganesh Vernekar <[email protected]>
Update CHANGELOG and unit test
|
Updated the changelog, linter seems to get killed arbitrarily but with a higher chance of getting killed |
|
Well @codesome is an OOM. I have an idea, we could make a linting in GH action instead, those might be more capable, but let's looks on that next PR. |
|
I will look on that today |
|
Can we re-run this and hope for this to be green to unblock this PR? :) I saw that it was green a while ago |
|
I can merge even and fix master then, happy with that. |
Changes
In Cortex we're currently blocked by Prometheus update. In this PR I'm attempting an update like @krasi-georgiev correctly did in in #2633, without reworking the
FlushableStorageto hopefully get it in a mergeable state quicker 🤞Verification
Tests.