From 47efebfc6fcbce028c0f3df5f9d584119e8e8ffe Mon Sep 17 00:00:00 2001 From: Florian Klink Date: Fri, 18 Oct 2024 13:33:51 +0200 Subject: [PATCH] docs(tvix/TODO): add instance_name idea for stores Change-Id: If14f81ca7180109195f35352b0f43b8d997176cd Reviewed-on: https://cl.tvl.fyi/c/depot/+/12660 Autosubmit: flokli Reviewed-by: edef Tested-by: BuildkiteCI --- tvix/docs/src/TODO.md | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tvix/docs/src/TODO.md b/tvix/docs/src/TODO.md index b0fdc68a8..033a6bfc4 100644 --- a/tvix/docs/src/TODO.md +++ b/tvix/docs/src/TODO.md @@ -186,6 +186,10 @@ This requires some more designing. Some goals: - Maybe add a ?cache=$other_url parameter support to the URL syntax, to easily wrap a store with a caching frontend, using $other_url as the "near" store URL. + - Each store should get its instance name passed down, and add this as a + field in the instrumentation calls. This causes log messages and + per-instance store metrics to be traceable back to the specific instance + (if multiple backends of the same type are present). ### Store Config There's already serde for some store options (bigtable uses `serde_qs`).