Skip to content

Commit

Permalink
Remove classic KubeCluster and HelmCluster (#890)
Browse files Browse the repository at this point in the history
* Remove classic KubeCluster and HelmCluster

* Be less specific in testing config

* Remove deprecation warning test

* Shift docs focus a little

* Refactor docs a little more

* Reinstate code example

* doc -> docs
  • Loading branch information
jacobtomlinson authored Apr 30, 2024
1 parent ef41984 commit 30c22a5
Show file tree
Hide file tree
Showing 52 changed files with 232 additions and 4,943 deletions.
57 changes: 0 additions & 57 deletions .github/workflows/helmcluster.yaml

This file was deleted.

57 changes: 0 additions & 57 deletions .github/workflows/kubecluster.yaml

This file was deleted.

16 changes: 1 addition & 15 deletions .github/workflows/operator.yaml
Original file line number Diff line number Diff line change
@@ -1,21 +1,7 @@
name: "Operator"
on:
pull_request:
paths:
- ".github/workflows/operator.yaml"
- "requirements*"
- "ci/**"
- "dask_kubernetes/operator/**"
- "dask_kubernetes/common/**"
- "dask_kubernetes/*"
push:
paths:
- ".github/workflows/operator.yaml"
- "requirements*"
- "ci/**"
- "dask_kubernetes/operator/**"
- "dask_kubernetes/common/**"
- "dask_kubernetes/*"

concurrency:
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
Expand Down Expand Up @@ -64,7 +50,7 @@ jobs:
KUBERNETES_VERSION: ${{ matrix.kubernetes-version }}
TEST_ISTIO: "true"
TEST_DASK_GATEWAY: "true"
run: pytest --maxfail=3 --reruns=5 dask_kubernetes/common dask_kubernetes/operator
run: pytest --maxfail=3 --reruns=5 dask_kubernetes
- name: Debug kubernetes resources
if: always()
run: kubectl get all -A
2 changes: 1 addition & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ instance/
.scrapy

# Sphinx documentation
doc/_build/
docs/_build/

# PyBuilder
target/
Expand Down
4 changes: 2 additions & 2 deletions .readthedocs.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ version: 2

# Set the version of Python and other tools you might need
sphinx:
configuration: doc/source/conf.py
configuration: docs/source/conf.py

formats: all

Expand All @@ -20,7 +20,7 @@ python:
install:
- method: pip
path: .
- requirements: doc/requirements-docs.txt
- requirements: docs/requirements-docs.txt

submodules:
include: all
24 changes: 1 addition & 23 deletions dask_kubernetes/__init__.py
Original file line number Diff line number Diff line change
@@ -1,32 +1,10 @@
from importlib import import_module
from warnings import warn

from . import config

from .common.auth import (
AutoRefreshConfiguration,
AutoRefreshKubeConfigLoader,
ClusterAuth,
InCluster,
KubeAuth,
KubeConfig,
)
from .common.objects import clean_pod_template, make_pod_from_dict, make_pod_spec
from .helm import HelmCluster

__all__ = ["HelmCluster", "KubeCluster"]
__all__ = []

try:
from ._version import version as __version__ # noqa
from ._version import version_tuple as __version_tuple__ # noqa
except ImportError:
__version__ = "0.0.0"
__version_tuple__ = (0, 0, 0)


def __getattr__(name):
if name == "KubeCluster":
new_module = import_module("dask_kubernetes.classic")
return getattr(new_module, name)

raise AttributeError(f"module {__name__} has no attribute {name}")
2 changes: 0 additions & 2 deletions dask_kubernetes/classic/__init__.py

This file was deleted.

Loading

0 comments on commit 30c22a5

Please sign in to comment.