Skip to content

Commit

Permalink
Revert "Remove lazy imports of kubernetes"
Browse files Browse the repository at this point in the history
This reverts commit 208b97d.
  • Loading branch information
abrookins committed May 10, 2024
1 parent 208b97d commit cd72b7e
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,15 @@
from typing import TYPE_CHECKING, Dict, List, Optional

from prefect.events import Event, RelatedResource, emit_event
from prefect.utilities.importtools import lazy_import

if TYPE_CHECKING:
import kubernetes
import kubernetes.client
import kubernetes.watch
from kubernetes.client import ApiClient, V1Pod
else:
import kubernetes
kubernetes = lazy_import("kubernetes")

EVICTED_REASONS = {
"OOMKilled",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,7 @@
from prefect.server.schemas.responses import DeploymentResponse
from prefect.utilities.asyncutils import run_sync_in_worker_thread
from prefect.utilities.dockerutils import get_prefect_image_name
from prefect.utilities.importtools import lazy_import
from prefect.utilities.pydantic import JsonPatch
from prefect.utilities.templating import find_placeholders
from prefect.workers.base import (
Expand Down Expand Up @@ -163,7 +164,7 @@

from prefect.client.schemas import FlowRun
else:
import kubernetes
kubernetes = lazy_import("kubernetes")

MAX_ATTEMPTS = 3
RETRY_MIN_DELAY_SECONDS = 1
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,14 @@
import time
from unittest.mock import MagicMock, call, patch

import kubernetes
import pytest
from kubernetes.client import V1Pod
from prefect_kubernetes.events import EVICTED_REASONS, KubernetesEventsReplicator

from prefect.events import RelatedResource
from prefect.utilities.importtools import lazy_import

kubernetes = lazy_import("kubernetes")


@pytest.fixture
Expand Down
3 changes: 2 additions & 1 deletion src/prefect/blocks/kubernetes.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,12 +16,13 @@

from prefect.blocks.core import Block
from prefect.utilities.collections import listrepr
from prefect.utilities.importtools import lazy_import

if TYPE_CHECKING:
import kubernetes
from kubernetes.client.api_client import ApiClient
else:
import kubernetes
kubernetes = lazy_import("kubernetes")


@deprecated_class(
Expand Down
3 changes: 2 additions & 1 deletion src/prefect/infrastructure/kubernetes.py
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@
from prefect.infrastructure.base import Infrastructure, InfrastructureResult
from prefect.utilities.asyncutils import run_sync_in_worker_thread, sync_compatible
from prefect.utilities.hashing import stable_hash
from prefect.utilities.importtools import lazy_import
from prefect.utilities.pydantic import JsonPatch
from prefect.utilities.slugify import slugify

Expand All @@ -52,7 +53,7 @@
import kubernetes.config
from kubernetes.client import BatchV1Api, CoreV1Api, V1Job, V1Pod
else:
import kubernetes
kubernetes = lazy_import("kubernetes")


class KubernetesImagePullPolicy(enum.Enum):
Expand Down

0 comments on commit cd72b7e

Please sign in to comment.