fix(hc): Truncates organization names that exceed the max field length (#87136)

Author: GabeVillalobos

src/sentry/hybridcloud/services/region_organization_provisioning/impl.py

@@ -13,7 +13,7 @@
     RegionOrganizationProvisioningRpcService,
 )
 from sentry.issues.streamline import apply_streamline_rollout_group
-from sentry.models.organization import Organization
+from sentry.models.organization import ORGANIZATION_NAME_MAX_LENGTH, Organization
 from sentry.models.organizationmember import OrganizationMember
 from sentry.models.organizationmemberteam import OrganizationMemberTeam
 from sentry.models.organizationslugreservation import OrganizationSlugReservationType
@@ -52,8 +52,9 @@ def _create_organization_and_team(
         assert (user_id is None and email) or (
             user_id and email is None
         ), "Must set either user_id or email"
+        truncated_name = organization_name[:ORGANIZATION_NAME_MAX_LENGTH]
         org = Organization.objects.create(
-            id=organization_id, name=organization_name, slug=slug, is_test=is_test
+            id=organization_id, name=truncated_name, slug=slug, is_test=is_test
         )
 
         apply_streamline_rollout_group(organization=org)

src/sentry/models/organization.py

@@ -45,6 +45,7 @@
 
 SENTRY_USE_SNOWFLAKE = getattr(settings, "SENTRY_USE_SNOWFLAKE", False)
 NON_MEMBER_SCOPES = frozenset(["org:write", "project:write", "team:write"])
+ORGANIZATION_NAME_MAX_LENGTH = 64
 
 
 class OrganizationStatus(IntEnum):
@@ -154,7 +155,7 @@ class Organization(ReplicatedRegionModel):
     replication_version = 4
 
     __relocation_scope__ = RelocationScope.Organization
-    name = models.CharField(max_length=64)
+    name = models.CharField(max_length=ORGANIZATION_NAME_MAX_LENGTH)
     slug: models.Field[str, str] = SentryOrgSlugField(unique=True)
     status = BoundedPositiveIntegerField(
         choices=OrganizationStatus.as_choices(), default=OrganizationStatus.ACTIVE.value

tests/sentry/hybridcloud/services/test_region_organization_provisioning.py

@@ -139,6 +139,19 @@ def test_does_not_provision_and_returns_false_when_multiple_orgs_conflict(self)
             )
         assert not provisioning_user_memberships.exists()
 
+    def test_truncates_name_when_too_long(self) -> None:
+        user = self.create_user()
+        provision_options = self.get_provisioning_args(user)
+        provision_options.provision_options.name = "a" * 128
+        result = region_organization_provisioning_rpc_service.create_organization_in_region(
+            organization_id=42, provision_payload=provision_options, region_name="us"
+        )
+
+        assert result is True
+        with assume_test_silo_mode(SiloMode.REGION):
+            org: Organization = Organization.objects.get(id=42)
+            assert org.name == "a" * 64
+
     def test_does_not_provision_and_returns_false_when_conflicting_org_with_different_owner(
         self,
     ) -> None: