Skip to content

Commit

Permalink
ref: use absolute imports for BaseManager
Browse files Browse the repository at this point in the history
fixing an __init__.py cycle one step at a time
  • Loading branch information
asottile-sentry committed Jun 18, 2024
1 parent 4cdd6f9 commit 230bf79
Show file tree
Hide file tree
Showing 55 changed files with 69 additions and 86 deletions.
1 change: 0 additions & 1 deletion src/sentry/db/models/__init__.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
from .base import * # NOQA
from .fields import * # NOQA
from .manager import * # NOQA
from .paranoia import * # NOQA
from .query import * # NOQA
3 changes: 1 addition & 2 deletions src/sentry/db/models/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,10 @@
from sentry.backup.sanitize import SanitizableField, Sanitizer
from sentry.backup.scopes import ImportScope, RelocationScope
from sentry.db.models.fields.uuid import UUIDField
from sentry.db.models.manager.base import BaseManager, create_silo_limited_copy
from sentry.silo.base import SiloLimit, SiloMode

from .fields.bounded import BoundedBigAutoField
from .manager import BaseManager
from .manager.base import create_silo_limited_copy
from .query import update

__all__ = (
Expand Down
5 changes: 0 additions & 5 deletions src/sentry/db/models/manager/__init__.py
Original file line number Diff line number Diff line change
@@ -1,5 +0,0 @@
from __future__ import annotations

from .base import BaseManager

__all__ = ("BaseManager",)
3 changes: 2 additions & 1 deletion src/sentry/db/models/outboxes.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,8 @@
from django.dispatch import receiver
from sentry_sdk.api import capture_exception

from sentry.db.models import BaseManager, Model
from sentry.db.models import Model
from sentry.db.models.manager.base import BaseManager
from sentry.signals import post_upgrade
from sentry.silo.base import SiloMode
from sentry.types.region import find_regions_for_orgs, find_regions_for_user
Expand Down
3 changes: 2 additions & 1 deletion src/sentry/db/models/paranoia.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@
from django.db import models
from django.utils import timezone

from sentry.db.models import BaseManager, BaseModel, sane_repr
from sentry.db.models import BaseModel, sane_repr
from sentry.db.models.manager.base import BaseManager
from sentry.db.models.manager.base_query_set import BaseQuerySet
from sentry.db.models.manager.types import M

Expand Down
3 changes: 2 additions & 1 deletion src/sentry/discover/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,11 @@

from sentry import features
from sentry.backup.scopes import RelocationScope
from sentry.db.models import BaseManager, FlexibleForeignKey, Model, region_silo_model, sane_repr
from sentry.db.models import FlexibleForeignKey, Model, region_silo_model, sane_repr
from sentry.db.models.fields import JSONField
from sentry.db.models.fields.bounded import BoundedBigIntegerField, BoundedPositiveIntegerField
from sentry.db.models.fields.hybrid_cloud_foreign_key import HybridCloudForeignKey
from sentry.db.models.manager.base import BaseManager
from sentry.models.dashboard_widget import TypesClass
from sentry.models.projectteam import ProjectTeam
from sentry.tasks.relay import schedule_invalidate_project_config
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/incidents/models/alert_rule.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
sane_repr,
)
from sentry.db.models.fields.hybrid_cloud_foreign_key import HybridCloudForeignKey
from sentry.db.models.manager import BaseManager
from sentry.db.models.manager.base import BaseManager
from sentry.incidents.models.alert_rule_activations import AlertRuleActivations
from sentry.incidents.models.incident import IncidentTrigger
from sentry.incidents.utils.constants import INCIDENTS_SNUBA_SUBSCRIPTION_TYPE
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/incidents/models/alert_rule_activations.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@

from sentry.backup.scopes import RelocationScope
from sentry.db.models import FlexibleForeignKey, Model, region_silo_model
from sentry.db.models.manager import BaseManager
from sentry.db.models.manager.base import BaseManager
from sentry.models.releases.constants import DB_VERSION_LENGTH

if TYPE_CHECKING:
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/incidents/models/incident.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
sane_repr,
)
from sentry.db.models.fields.hybrid_cloud_foreign_key import HybridCloudForeignKey
from sentry.db.models.manager import BaseManager
from sentry.db.models.manager.base import BaseManager
from sentry.models.organization import Organization
from sentry.utils.retries import TimedRetryPolicy

Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/activity.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@
from sentry import features, options
from sentry.backup.scopes import RelocationScope
from sentry.db.models import (
BaseManager,
BoundedPositiveIntegerField,
FlexibleForeignKey,
GzippedDictField,
Expand All @@ -23,6 +22,7 @@
sane_repr,
)
from sentry.db.models.fields.hybrid_cloud_foreign_key import HybridCloudForeignKey
from sentry.db.models.manager.base import BaseManager
from sentry.tasks import activity
from sentry.types.activity import CHOICES, ActivityType
from sentry.types.group import PriorityLevel
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/apiapplication.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,13 +11,13 @@
from sentry.backup.sanitize import SanitizableField, Sanitizer
from sentry.backup.scopes import RelocationScope
from sentry.db.models import (
BaseManager,
BoundedPositiveIntegerField,
FlexibleForeignKey,
Model,
control_silo_model,
sane_repr,
)
from sentry.db.models.manager.base import BaseManager
from sentry.models.outbox import ControlOutbox, OutboxCategory, OutboxScope, outbox_context
from sentry.types.region import find_all_region_names

Expand Down
3 changes: 2 additions & 1 deletion src/sentry/models/apikey.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,9 @@
from sentry.backup.dependencies import NormalizedModelName, get_model_name
from sentry.backup.sanitize import SanitizableField, Sanitizer
from sentry.backup.scopes import RelocationScope
from sentry.db.models import BaseManager, BoundedPositiveIntegerField, control_silo_model, sane_repr
from sentry.db.models import BoundedPositiveIntegerField, control_silo_model, sane_repr
from sentry.db.models.fields.hybrid_cloud_foreign_key import HybridCloudForeignKey
from sentry.db.models.manager.base import BaseManager
from sentry.db.models.outboxes import ReplicatedControlModel
from sentry.models.apiscopes import HasApiScopes
from sentry.models.outbox import OutboxCategory
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/authenticator.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,13 +21,13 @@
from sentry.backup.sanitize import SanitizableField, Sanitizer
from sentry.backup.scopes import RelocationScope
from sentry.db.models import (
BaseManager,
BoundedAutoField,
BoundedPositiveIntegerField,
FlexibleForeignKey,
control_silo_model,
)
from sentry.db.models.fields.picklefield import PickledObjectField
from sentry.db.models.manager.base import BaseManager
from sentry.db.models.outboxes import ControlOutboxProducingModel
from sentry.models.outbox import ControlOutboxBase, OutboxCategory
from sentry.types.region import find_regions_for_user
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/avatars/sentry_app_avatar.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
from django.db import models

from sentry.db.models import FlexibleForeignKey, control_silo_model
from sentry.db.models.manager import BaseManager
from sentry.db.models.manager.base import BaseManager

from . import ControlAvatarBase

Expand Down
3 changes: 2 additions & 1 deletion src/sentry/models/avatars/user_avatar.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,8 @@

from django.db import models, router, transaction

from sentry.db.models import BaseManager, FlexibleForeignKey, control_silo_model
from sentry.db.models import FlexibleForeignKey, control_silo_model
from sentry.db.models.manager.base import BaseManager

from ...types.region import find_regions_for_user
from ..outbox import ControlOutboxBase, OutboxCategory
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/commit.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,14 +9,14 @@

from sentry.backup.scopes import RelocationScope
from sentry.db.models import (
BaseManager,
BoundedBigIntegerField,
BoundedPositiveIntegerField,
FlexibleForeignKey,
Model,
region_silo_model,
sane_repr,
)
from sentry.db.models.manager.base import BaseManager
from sentry.utils.groupreference import find_referenced_groups

if TYPE_CHECKING:
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/commitauthor.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

from sentry.backup.scopes import RelocationScope
from sentry.db.models import BoundedBigIntegerField, Model, region_silo_model, sane_repr
from sentry.db.models.manager import BaseManager
from sentry.db.models.manager.base import BaseManager

if TYPE_CHECKING:
from sentry.services.hybrid_cloud.user import RpcUser
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/commitfilechange.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,13 @@

from sentry.backup.scopes import RelocationScope
from sentry.db.models import (
BaseManager,
BoundedBigIntegerField,
FlexibleForeignKey,
Model,
region_silo_model,
sane_repr,
)
from sentry.db.models.manager.base import BaseManager

COMMIT_FILE_CHANGE_TYPES = frozenset(("A", "D", "M"))

Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/debugfile.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,14 +25,14 @@
from sentry.backup.scopes import RelocationScope
from sentry.constants import KNOWN_DIF_FORMATS
from sentry.db.models import (
BaseManager,
BoundedBigIntegerField,
FlexibleForeignKey,
JSONField,
Model,
region_silo_model,
sane_repr,
)
from sentry.db.models.manager.base import BaseManager
from sentry.models.files.file import File
from sentry.models.files.utils import clear_cached_files
from sentry.reprocessing import resolve_processing_issue
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/environment.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,13 @@
from sentry.backup.scopes import RelocationScope
from sentry.constants import ENVIRONMENT_NAME_MAX_LENGTH, ENVIRONMENT_NAME_PATTERN
from sentry.db.models import (
BaseManager,
BoundedBigIntegerField,
FlexibleForeignKey,
Model,
region_silo_model,
sane_repr,
)
from sentry.db.models.manager.base import BaseManager
from sentry.utils import metrics
from sentry.utils.cache import cache
from sentry.utils.hashlib import md5_text
Expand Down
10 changes: 2 additions & 8 deletions src/sentry/models/featureadoption.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,14 +10,8 @@
from sentry.adoption import manager
from sentry.adoption.manager import UnknownFeature
from sentry.backup.scopes import RelocationScope
from sentry.db.models import (
BaseManager,
FlexibleForeignKey,
JSONField,
Model,
region_silo_model,
sane_repr,
)
from sentry.db.models import FlexibleForeignKey, JSONField, Model, region_silo_model, sane_repr
from sentry.db.models.manager.base import BaseManager
from sentry.utils.redis import (
get_dynamic_cluster_from_options,
is_instance_rb_cluster,
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/group.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@
from sentry.backup.scopes import RelocationScope
from sentry.constants import DEFAULT_LOGGER_NAME, LOG_LEVELS, MAX_CULPRIT_LENGTH
from sentry.db.models import (
BaseManager,
BoundedBigIntegerField,
BoundedIntegerField,
BoundedPositiveIntegerField,
Expand All @@ -36,6 +35,7 @@
region_silo_model,
sane_repr,
)
from sentry.db.models.manager.base import BaseManager
from sentry.eventstore.models import GroupEvent
from sentry.issues.grouptype import ErrorGroupType, GroupCategory, get_group_type_by_type_id
from sentry.issues.priority import (
Expand Down
3 changes: 2 additions & 1 deletion src/sentry/models/groupassignee.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,9 @@

from sentry import features
from sentry.backup.scopes import RelocationScope
from sentry.db.models import BaseManager, FlexibleForeignKey, Model, region_silo_model, sane_repr
from sentry.db.models import FlexibleForeignKey, Model, region_silo_model, sane_repr
from sentry.db.models.fields.hybrid_cloud_foreign_key import HybridCloudForeignKey
from sentry.db.models.manager.base import BaseManager
from sentry.models.grouphistory import GroupHistoryStatus, record_group_history
from sentry.models.groupowner import GroupOwner
from sentry.models.groupsubscription import GroupSubscription
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/grouphistory.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,14 +8,14 @@

from sentry.backup.scopes import RelocationScope
from sentry.db.models import (
BaseManager,
BoundedPositiveIntegerField,
FlexibleForeignKey,
Model,
region_silo_model,
sane_repr,
)
from sentry.db.models.fields.hybrid_cloud_foreign_key import HybridCloudForeignKey
from sentry.db.models.manager.base import BaseManager
from sentry.types.activity import ActivityType
from sentry.types.actor import Actor
from sentry.types.group import GROUP_SUBSTATUS_TO_GROUP_HISTORY_STATUS
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/grouplink.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@

from sentry.backup.scopes import RelocationScope
from sentry.db.models import (
BaseManager,
BoundedBigIntegerField,
BoundedPositiveIntegerField,
FlexibleForeignKey,
Expand All @@ -18,6 +17,7 @@
region_silo_model,
sane_repr,
)
from sentry.db.models.manager.base import BaseManager

if TYPE_CHECKING:
from sentry.models.group import Group
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/groupmeta.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

from sentry.backup.scopes import RelocationScope
from sentry.db.models import FlexibleForeignKey, Model, region_silo_model, sane_repr
from sentry.db.models.manager import BaseManager
from sentry.db.models.manager.base import BaseManager

ERR_CACHE_MISSING = "Cache not populated for instance id=%s"

Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/groupsnooze.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,14 +10,14 @@
from sentry import tsdb
from sentry.backup.scopes import RelocationScope
from sentry.db.models import (
BaseManager,
BoundedPositiveIntegerField,
FlexibleForeignKey,
JSONField,
Model,
region_silo_model,
sane_repr,
)
from sentry.db.models.manager.base import BaseManager
from sentry.issues.constants import get_issue_tsdb_group_model, get_issue_tsdb_user_group_model
from sentry.snuba.referrer import Referrer
from sentry.utils import metrics
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/groupsubscription.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,14 +9,14 @@

from sentry.backup.scopes import RelocationScope
from sentry.db.models import (
BaseManager,
BoundedPositiveIntegerField,
FlexibleForeignKey,
Model,
region_silo_model,
sane_repr,
)
from sentry.db.models.fields.hybrid_cloud_foreign_key import HybridCloudForeignKey
from sentry.db.models.manager.base import BaseManager
from sentry.integrations.types import ExternalProviders
from sentry.notifications.types import (
GroupSubscriptionReason,
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/identity.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,13 +13,13 @@
from sentry.backup.scopes import RelocationScope
from sentry.db.models import (
ArrayField,
BaseManager,
BoundedPositiveIntegerField,
FlexibleForeignKey,
Model,
control_silo_model,
)
from sentry.db.models.fields.jsonfield import JSONField
from sentry.db.models.manager.base import BaseManager
from sentry.integrations.types import ExternalProviders
from sentry.services.hybrid_cloud.user import RpcUser

Expand Down
10 changes: 2 additions & 8 deletions src/sentry/models/integrations/external_issue.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,15 +7,9 @@
from django.utils import timezone

from sentry.backup.scopes import RelocationScope
from sentry.db.models import (
BaseManager,
FlexibleForeignKey,
JSONField,
Model,
region_silo_model,
sane_repr,
)
from sentry.db.models import FlexibleForeignKey, JSONField, Model, region_silo_model, sane_repr
from sentry.db.models.fields.hybrid_cloud_foreign_key import HybridCloudForeignKey
from sentry.db.models.manager.base import BaseManager
from sentry.eventstore.models import Event

if TYPE_CHECKING:
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/integrations/integration.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
from sentry.constants import ObjectStatus
from sentry.db.models import BoundedPositiveIntegerField, DefaultFieldsModel, control_silo_model
from sentry.db.models.fields.jsonfield import JSONField
from sentry.db.models.manager import BaseManager
from sentry.db.models.manager.base import BaseManager
from sentry.models.integrations.organization_integration import OrganizationIntegration
from sentry.models.outbox import ControlOutbox, OutboxCategory, OutboxScope, outbox_context
from sentry.services.hybrid_cloud.organization import RpcOrganization, organization_service
Expand Down
2 changes: 1 addition & 1 deletion src/sentry/models/integrations/integration_feature.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
Model,
control_silo_model,
)
from sentry.db.models.manager import BaseManager
from sentry.db.models.manager.base import BaseManager
from sentry.models.integrations.doc_integration import DocIntegration
from sentry.models.integrations.sentry_app import SentryApp

Expand Down
Loading

0 comments on commit 230bf79

Please sign in to comment.