107079: grpcinterceptor: Use ctxutil to detect context cancellation r=miretskiy a=miretskiy Previous commit #76306 arranged for interceptors to run for all rpcs (local or otherwise). In doing so, it added an additional go routine for each RPC whose purpose was to detect context completion, and to arrange for the cleanup function to run when that happened. `ctxutil.WhenDone` (added by #103866/commits/dfe4ef82810aaf81ae21afecce83cbc1fb6783dd) allows for the same functionality without the additional goroutine cost. Informs #107053 Release note: None 107267: kvcoord: make rangefeed connection class `TenantReadOnly` r=erikgrinaker a=erikgrinaker Otherwise, the setting won't have any effect for tenants, always falling through to the default. Resolves #106895. Epic: none Release note: None 107273: build: use absolute path to pnpm-lock.yaml r=maryliag a=sjbarag Previously, actions/setup-node@v3 was used with a relative path for `cache-dependency-path`. This had an unexpected interaction with the `.defaults.run.working-directory: "pkg/ui/workspaces/cluster-ui"` declaration, which caused setup-node's cleanup action to look for pkg/ui/workspaces/cluster-ui/pkg/ui/pnpm-lock.yaml. Naturally, this file didn't exist. Use an absolute path (thanks, `github.workspace`!) to pnpm-lock.yaml instead. Release note: None Epic: none Co-authored-by: Yevgeniy Miretskiy <yevgeniy@cockroachlabs.com> Co-authored-by: Erik Grinaker <grinaker@cockroachlabs.com> Co-authored-by: Sean Barag <barag@cockroachlabs.com>

kvcoord: kv.rangefeed.use_dedicated_connection_class.enabled must be TenantReadOnly #106895

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions

kvcoord: `kv.rangefeed.use_dedicated_connection_class.enabled` must be `TenantReadOnly` #106895