docs: reconcile service list with cortex -modules
#4586
Conversation
docs/architecture.md
Outdated
There was a problem hiding this comment.
Just checking: is this strictly required for running blocks storage, or is it only required if using blocks storage and the bucket index?
There was a problem hiding this comment.
The compactor also dedupes if you have replication across ingesters, and bulks up the blocks from 2 hours to 24 hours which reduces the cost of historic queries substantially. So you can technically run without it, but nobody should do.
jmcarp
force-pushed
the
jmcarp/services-docs
branch
2 times, most recently
from
1ca0c74 to
f76558d
Compare
There was a problem hiding this comment.
I've marked it as approved to speed up getting to 2 approvals, but really I don't want to add the Flusher and Purger.
Table Manager is debatable; it's quite likely someone reads about it in an older blog so we could justify mentioning it. Or equally we can say "don't add any new stuff about chunks".
docs/architecture.md
Outdated
There was a problem hiding this comment.
Chunks storage is deprecated; I think we should be taking it out of docs not adding more references.
There was a problem hiding this comment.
Makes sense. I've dropped references to services used with chunks storage.
Note: services used with the chunks storage are excluded. Signed-off-by: Josh Carp <[email protected]>
jmcarp
force-pushed
the
jmcarp/services-docs
branch
from
f76558d to
31c8b1f
Compare
3 participants
What this PR does:
Docs-only update to reconcile the list of services in architecture.md with the output of
cortex -modules. Sorry if I missed this elsewhere, but I hadn't seen a list of all cortex services in one place in the docs.Which issue(s) this PR fixes:
Fixes #
Checklist
CHANGELOG.mdupdated - the order of entries should be[CHANGE],[FEATURE],[ENHANCEMENT],[BUGFIX]