-
Notifications
You must be signed in to change notification settings - Fork 4.1k
testutils: do not require CCL import or enterprise license to test with secondary tenants #108761
Copy link
Copy link
Closed
Labels
A-multitenancyRelated to multi-tenancyRelated to multi-tenancyA-testingTesting tools and infrastructureTesting tools and infrastructureC-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)T-multitenantIssues owned by the multi-tenant virtual teamIssues owned by the multi-tenant virtual team
Description
(Discussed with @yuzefovich)
Prior to this patch, for test tenant servers to be automatically used two conditions needed to hold:
- the CCL package(s) needed to be imported.
- an enterprise license needed to be set (usually via
ccl.TestingEnableEnterprise())
While we want these conditions to hold to start separate-process SQL servers in production, they do not need to be verified in unit tests. Also ever since #98203 was merged, the CCL import is not necessary at all.
Epic: CRDB-18499
Jira issue: CRDB-30626
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-multitenancyRelated to multi-tenancyRelated to multi-tenancyA-testingTesting tools and infrastructureTesting tools and infrastructureC-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)T-multitenantIssues owned by the multi-tenant virtual teamIssues owned by the multi-tenant virtual team