8000 ua: remove usage of `roachpb.MustMakeTenantID(2)` as an app tenant from tests · Issue #138801 · cockroachdb/cockroach · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
ua: remove usage of roachpb.MustMakeTenantID(2) as an app tenant from tests #138801
@shubhamdhama

Description

@shubhamdhama

Part of #136667

There are around 31 usages of roachpb.MustMakeTenantID(2) in tests used for application tenant. We should be replacing them with tenant id 3 and disallow creating secondary tenants with id 2. This would align with real world scenario where tenant with id 2 doesn't exist, which is enforced in demo cluster and normal operations. And if a cluster goes through UA migration, tenant 2 would be a system tenant.

Jira issue: CRDB-46349

Metadata

Metadata

Assignees

Labels

C-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)T-db-server

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions

    0