Skip to content

ref(cells): migrate org provisioning code from region -> cell#110696

Merged
lynnagara merged 3 commits intomasterfrom
org-provisioning
Mar 16, 2026
Merged

ref(cells): migrate org provisioning code from region -> cell#110696
lynnagara merged 3 commits intomasterfrom
org-provisioning

Conversation

@lynnagara
Copy link
Member

Add rpc method:

  • provision_organization_in_cell which will replace provision_organization_in_region

Removes unused (and unimplemented) rpc methods:

  • OrganizationProvisioningService.idempotent_provision_organization_in_region
  • ControlOrganizationProvisioningRpcService.idempotent_provision_organization

- add provision_organization_in_cell which will replace provision_organization_in_region

Removes unused (and unimplemented) rpc methods:
- OrganizationProvisioningService.idempotent_provision_organization_in_region
- ControlOrganizationProvisioningRpcService.idempotent_provision_organization
@lynnagara lynnagara requested a review from a team as a code owner March 13, 2026 22:34
@lynnagara lynnagara requested a review from a team March 13, 2026 22:34
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 13, 2026
Comment on lines 85 to 87
destination_region_name = self._validate_or_default_region(region_name=cell_name)
return self._control_based_provisioning(
provisioning_options=provisioning_options, region_name=destination_region_name
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
destination_region_name = self._validate_or_default_region(region_name=cell_name)
return self._control_based_provisioning(
provisioning_options=provisioning_options, region_name=destination_region_name
destination_cell_name = self._validate_or_default_region(region_name=cell_name)
return self._control_based_provisioning(
provisioning_options=provisioning_options, region_name=destination_cell_name

Should we rename these usages of region as well?


@abstractmethod
@rpc_method
def idempotent_provision_organization(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 This is unused.

@lynnagara lynnagara merged commit 9309c1e into master Mar 16, 2026
77 of 78 checks passed
@lynnagara lynnagara deleted the org-provisioning branch March 16, 2026 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants