Skip to content

Commit

Permalink
Add Python 3.11 to the CI matrix (#864)
Browse files Browse the repository at this point in the history
* Bump min py

* Upgrade versioneer for python 3.12

* Update badges

* Exit tests after 3 failures and reduce reruns

* Bumping Python in container to see if that fixes failures

* Build container image for the right Python version

* Drop 3.12 for now
  • Loading branch information
jacobtomlinson authored Feb 20, 2024
1 parent 726f0f8 commit 84846b1
Show file tree
Hide file tree
Showing 12 changed files with 1,144 additions and 581 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/helmcluster.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ jobs:
strategy:
fail-fast: false
matrix:
python-version: ["3.9", "3.10"]
python-version: ["3.9", "3.10", "3.11"]
kubernetes-version: ["1.29.2"]
include:
- python-version: '3.10'
Expand All @@ -51,7 +51,7 @@ jobs:
- name: Run tests
env:
KUBERNETES_VERSION: ${{ matrix.kubernetes-version }}
run: pytest dask_kubernetes/common/tests dask_kubernetes/helm/tests
run: pytest --reruns=5 dask_kubernetes/common/tests dask_kubernetes/helm/tests
- name: Debug k8s resources
if: success() || failure()
run: kubectl get all -A
4 changes: 2 additions & 2 deletions .github/workflows/kubecluster.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ jobs:
strategy:
fail-fast: false
matrix:
python-version: ["3.9", "3.10"]
python-version: ["3.9", "3.10", "3.11"]
kubernetes-version: ["1.29.2"]
include:
- python-version: '3.10'
Expand All @@ -51,7 +51,7 @@ jobs:
- name: Run tests
env:
KUBERNETES_VERSION: ${{ matrix.kubernetes-version }}
run: pytest dask_kubernetes/common/tests dask_kubernetes/classic/tests
run: pytest --reruns=5 dask_kubernetes/common/tests dask_kubernetes/classic/tests
- name: Debug k8s resources
if: success() || failure()
run: kubectl get all -A
4 changes: 2 additions & 2 deletions .github/workflows/operator.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ jobs:
strategy:
fail-fast: false
matrix:
python-version: ["3.9", "3.10"]
python-version: ["3.9", "3.10", "3.11"]
kubernetes-version: ["1.29.2"]
include:
- python-version: '3.10'
Expand All @@ -64,7 +64,7 @@ jobs:
KUBERNETES_VERSION: ${{ matrix.kubernetes-version }}
TEST_ISTIO: "true"
TEST_DASK_GATEWAY: "true"
run: pytest dask_kubernetes/common dask_kubernetes/operator
run: pytest --maxfail=3 --reruns=5 dask_kubernetes/common dask_kubernetes/operator
- name: Debug kubernetes resources
if: always()
run: kubectl get all -A
2 changes: 1 addition & 1 deletion README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ Dask Kubernetes
:target: https://anaconda.org/conda-forge/dask-kubernetes
:alt: Conda Forge

.. image:: https://img.shields.io/badge/python%20support-3.9%7C3.10-blue
.. image:: https://img.shields.io/badge/python%20support-3.9%7C3.10%7C3.11%7C3.12-blue
:target: https://kubernetes.dask.org/en/latest/installing.html#supported-versions
:alt: Python Support

Expand Down
4 changes: 3 additions & 1 deletion ci/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
FROM ghcr.io/dask/dask:latest-py3.10
ARG PYTHON=3.11

FROM ghcr.io/dask/dask:latest-py${PYTHON}

# Install latest dev builds of Dask and Distributed
RUN pip install git+https://github.com/dask/distributed@main
Expand Down
7 changes: 2 additions & 5 deletions dask_kubernetes/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
from warnings import warn

from . import config
from . import _version
from .common.auth import (
AutoRefreshConfiguration,
AutoRefreshKubeConfigLoader,
Expand All @@ -14,11 +15,7 @@
from .helm import HelmCluster

__all__ = ["HelmCluster", "KubeCluster"]

from ._version import get_versions

__version__ = get_versions()["version"]
del get_versions
__version__ = _version.get_versions()["version"]


def __getattr__(name):
Expand Down
Loading

0 comments on commit 84846b1

Please sign in to comment.