-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Metrics for compaction and downsampling process #4801
Merged
Merged
Changes from 56 commits
Commits
Show all changes
75 commits
Select commit
Hold shift + click to select a range
6fb9752
fixed template; export metadata
metonymic-smokey f403fcc
added some error handling
metonymic-smokey 02d7037
remove planned blocks from original
metonymic-smokey ea83250
moved to pkg/compact
metonymic-smokey ff2e36a
draft: export metrics to Prom
metonymic-smokey 5961552
update metrics after entire plan sim; change func name
metonymic-smokey a2b2b97
added method to delete metas from group
metonymic-smokey 8f13acc
preallocate meta slice
metonymic-smokey f52e056
changed func signature; started adding metrics to Group
metonymic-smokey ad639c1
added termination condition
metonymic-smokey 845d497
metrics struct added
metonymic-smokey cc4efe4
draft: updating progress metric
metonymic-smokey 322340b
added feature flag for compact
metonymic-smokey a4642f6
use tsdb planner as default
metonymic-smokey 3092479
initialised maps with size; changed metrics update map
metonymic-smokey 7f91580
removed extra vars; changed prom init location
metonymic-smokey 5fd74c6
added labels and downsample resolution to metadata
metonymic-smokey 6f0b74a
added termination condition; reverted planner type
metonymic-smokey ec3e767
Merge branch 'thanos-io:main' into metrics
metonymic-smokey a3dfae1
draft: downsampling metrics overview
metonymic-smokey 772041a
Merge branch 'metrics' of https://github.com/metonymic-smokey/thanos …
metonymic-smokey ad4c704
removed extra bkt; processing individual blocks
metonymic-smokey 3c7b8ac
Merge branch 'thanos-io:main' into metrics
metonymic-smokey e6739d7
changed downsample sim logic; changed plan metric names
metonymic-smokey b4a662b
re-used interface; refactoring
metonymic-smokey cdeb0d7
draft: added Prom metrics
metonymic-smokey 7a53935
re-used planner; added flag; fixed interface
metonymic-smokey 3b90846
increment exporters; renamed funcs/structs
metonymic-smokey da04db8
check if downsampling is disabled
metonymic-smokey 8922b33
iterate through groups instead of meta
metonymic-smokey 81a5066
run both simulations repeatedly
metonymic-smokey 63ab795
made naming more consistent
metonymic-smokey 235528d
added unit test for planning simulation
metonymic-smokey dc301ab
Merge branch 'thanos-io:main' into metrics
metonymic-smokey 1810ba4
removed extra comments
metonymic-smokey a355886
fixed flag bug
metonymic-smokey 40fa1fc
added docs for new compact flag
metonymic-smokey 13bec69
lint fix
metonymic-smokey cec1c58
added sync meta to repeat loop
metonymic-smokey d8a1ab8
draft: downsampling unit test
metonymic-smokey 8b23ef3
Merge branch 'thanos-io:main' into compact-metrics
metonymic-smokey fc71280
new group for downsample; re-used context
metonymic-smokey 0460638
Merge branch 'compact-metrics' of https://github.com/metonymic-smokey…
metonymic-smokey 5ea12ef
added test names and comments for downsample unit test
metonymic-smokey 3629986
fixed copy error - changed to deep copy
metonymic-smokey fa40551
changed metric names
metonymic-smokey 825a721
changed to regular flag; recreate simulator obj every run
metonymic-smokey 6b9b326
added explanatory comments
metonymic-smokey adba129
removed extra var
metonymic-smokey 87e9667
Merge branch 'thanos-io:main' into compact-metrics
metonymic-smokey 4d7da02
re-register simulators
metonymic-smokey a0ea1d4
fixed names and comments
metonymic-smokey 5c5d7b6
fix simulator obj registration
metonymic-smokey d788f99
added some more test cases; table test for plan simulate
metonymic-smokey daea946
added comments for some public methods
metonymic-smokey 60bbe9e
lint fixes
metonymic-smokey 9c3d11c
fixed names, logs and comments
metonymic-smokey afd48bf
refactored test
metonymic-smokey 54b412f
Merge branch 'thanos-io:main' into compact-metrics
metonymic-smokey 8f1fff7
added test cases with two groups
metonymic-smokey ff3ea04
added test case with non consecutive blocks
metonymic-smokey fa5f03f
fixed some nits
metonymic-smokey 3c5cbce
fixed two_groups_test
metonymic-smokey 35c4ff7
updated docs
metonymic-smokey 639b964
draft: refactor compact progress test
metonymic-smokey c0caf42
draft: refactor downsample test
metonymic-smokey a58b63f
lint fixes and code clean up
metonymic-smokey 8ec72e6
draft: refactored compact progress test
metonymic-smokey e10c845
changed func signature
metonymic-smokey 3d0d375
draft: refactored downsample unit test to use helper func
metonymic-smokey 6999f7f
improved test cases; CircleCI fix
metonymic-smokey 37848f1
use reset instead of unregister; minor nits
metonymic-smokey e366232
improved downsample test case
metonymic-smokey 50d1be4
E2E test fix
metonymic-smokey 02bb607
added CHANGELOG
metonymic-smokey File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't need to call it
orgininal
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mean in the log unnecessary to call it original. "Could not group metadata" is good enough
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is still not solved @metonymic-smokey