-
Notifications
You must be signed in to change notification settings - Fork 4k
Closed
Labels
A-cluster-observabilityRelated to cluster observabilityRelated to cluster observabilityA-multitenancyRelated to multi-tenancyRelated to multi-tenancyC-bugCode not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.T-db-serverv25.3.0-prerelease
Description
Found while investigating #110001.
Describe the problem
When TestStatusEngineStatsJson in server/storage_api is modified to point to a secondary tenant (e.g. via #110008), the test starts failing when accessing the /_status/enginestats endpoint:
engine_test.go:51: condition failed to evaluate within 45s: from http.go:190: status: 501 Not Implemented, content-type: application/json, body: {
"error": "method EngineStats not implemented",
"code": 12,
"message": "method EngineStats not implemented",
"details": [
]
}, error: <nil>
How to reproduce
from #110008 reset DefaultTestTenant in TestStatusEngineStatsJson, run the test with a test tenant and observe the failure.
Expected behavior
Once the tenant is granted a suitable capability, the endpoint should work properly. It should retrieve the engine stats over the KV tenant connector over the network.
Epic CRDB-38968
Jira issue: CRDB-31226
Metadata
Metadata
Assignees
Labels
A-cluster-observabilityRelated to cluster observabilityRelated to cluster observabilityA-multitenancyRelated to multi-tenancyRelated to multi-tenancyC-bugCode not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.T-db-serverv25.3.0-prerelease