thanos icon indicating copy to clipboard operation
thanos copied to clipboard

Proposal: Granular Endpoint Configuration

Open SrushtiSapkale opened this issue 2 years ago • 3 comments

Signed-off-by: Srushti Sapkale [email protected]

  • [ ] I added CHANGELOG entry for this change.
  • [x] Change is not relevant to the end user.

SrushtiSapkale avatar Jul 15 '22 10:07 SrushtiSapkale

Having something like this would be great for using Queriers with Prometheus sidecars. However, I think it can be a big pitfall when federating multiple queriers through a single query endpoint.

A common use case with Thanos to have cluster-local or regional Queriers which are connected globally through one central Querier. The advantage of using Envoy in this case is that it can load-balance gRPC connections, something we currently don't support natively in Thanos. If people used the endpoints config to connect the global to the regional queriers, they will not be able to scale regional queriers due to the persistent nature of gRPC connections.

Because of that, I think this feature should be very clear on when the endpoint config can be useful versus using something like Envoy.

fpetkovski avatar Jul 16 '22 12:07 fpetkovski

Yes, thanks! we need to keep in mind that if the endpoint config is implemented we need a solution for scaling the querier. Also as @saswatamcode suggested, I think per store endpt configuration can a better option for distributed querier :+1:

SrushtiSapkale avatar Jul 21 '22 13:07 SrushtiSapkale

@saswatamcode can you review the changes in the proposal once again?:slightly_smiling_face:

SrushtiSapkale avatar Jul 25 '22 08:07 SrushtiSapkale

Hello 👋 Looks like there was no activity on this amazing PR for the last 30 days. Do you mind updating us on the status? Is there anything we can help with? If you plan to still work on it, just comment on this PR or push a commit. Thanks! 🤗 If there will be no activity in the next week, this issue will be closed (we can always reopen a PR if you get back to this!). Alternatively, use remind command if you wish to be reminded at some point in future.

stale[bot] avatar Oct 30 '22 00:10 stale[bot]

What's the status of this proposal and the implementation @SrushtiSapkale @saswatamcode @bwplotka? Looks like this was marked stale now.

matej-g avatar Oct 31 '22 09:10 matej-g

Yup, so the proposal has some bits that need improvement! Will TAL. 🙂

saswatamcode avatar Oct 31 '22 10:10 saswatamcode

This proposal was a part of my lfx mentorship about implementing the endpoint configuration. Need to add some more points, would be happy to take this forward whenever it gets reviewed again 😅

SrushtiSapkale avatar Oct 31 '22 13:10 SrushtiSapkale

Hello 👋 Looks like there was no activity on this amazing PR for the last 30 days. Do you mind updating us on the status? Is there anything we can help with? If you plan to still work on it, just comment on this PR or push a commit. Thanks! 🤗 If there will be no activity in the next week, this issue will be closed (we can always reopen a PR if you get back to this!). Alternatively, use remind command if you wish to be reminded at some point in future.

stale[bot] avatar Jan 07 '23 21:01 stale[bot]

Closing for now as promised, let us know if you need this to be reopened! 🤗

stale[bot] avatar May 21 '23 19:05 stale[bot]