search

neondatabase / neon

Neon: Serverless Postgres. We separated storage and compute to offer autoscaling, code-like database branching, and scale to zero.
https://neon.tech
Apache License 2.0
14.48k stars 420 forks source link

tenant_id no longer logged for config updates since 5227 #7340

Open koivunej opened 5 months ago

koivunej commented 5 months ago 
INFO request{method=PUT path=/v1/tenant/config request_id=9bc81d88-e2f6-4898-b066-63816b621a9f}: Handling request
INFO request{method=PUT path=/v1/tenant/config request_id=9bc81d88-e2f6-4898-b066-63816b621a9f}:tenant_config{tenant_id=X}: configuring tenant X
INFO request{method=PUT path=/v1/tenant/config request_id=9bc81d88-e2f6-4898-b066-63816b621a9f}: Request handled, status: 200 OK

INFO version: git-env:82027e22dd42107aaedffaafde9a0f5737e88c19 failpoints: true, features: [] launch_timestamp: 2024-04-03 12:30:24.713685917 UTC build_tag: build_tag-env:release-5227

INFO request{method=PUT path=/v1/tenant/config request_id=00aefa58ef24aaa6f5abed7936c20561}: Handling request
INFO request{method=PUT path=/v1/tenant/config request_id=00aefa58ef24aaa6f5abed7936c20561}: Request handled, status: 200 OK

Now logging is completely useless for the config update.

jcsp commented 4 months ago

Need to add a span to the request handler that includes tenant_id, since it's not in the request URL