Merge pull request #989 from gesiscss/testing

fix cleaning pods up after testing
jupyterhub · Oct 29, 2019 · 76b9d19 · 76b9d19
2 parents 29e7546 + b3c9128
commit 76b9d19
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/binderhub/tests/conftest.py b/binderhub/tests/conftest.py
@@ -222,9 +222,9 @@ def get_pods():
         print(f"deleting pod {pod.metadata.name}")
         try:
             kube.delete_namespaced_pod(
-                pod.metadata.name,
-                namespace,
-                kubernetes.client.V1DeleteOptions(grace_period_seconds=0),
+                name=pod.metadata.name,
+                namespace=namespace,
+                body=kubernetes.client.V1DeleteOptions(grace_period_seconds=0),
             )
         except kubernetes.client.rest.ApiException as e:
             # ignore 404, 409: already gone

diff --git a/doc/doc-requirements.txt b/doc/doc-requirements.txt
@@ -9,7 +9,7 @@ ruamel.yaml
 # install BinderHub dependencies. We manually list them here because some
 # dependencies (like pycurl) can't be installed on ReadTheDocs and aren't
 # needed to build the docs.
-kubernetes>=4.*
+kubernetes==9.0.*
 escapism
 tornado
 traitlets

diff --git a/helm-chart/images/binderhub/requirements.txt b/helm-chart/images/binderhub/requirements.txt
@@ -1,6 +1,6 @@
 pycurl==7.43.0.1
 tornado==5.0.*
-kubernetes==7.0.*
+kubernetes==9.0.*
 jupyterhub==1.0.0
 jsonschema==2.6.0
 # Logging sinks to send eventlogging events to

diff --git a/requirements.txt b/requirements.txt
@@ -1,6 +1,6 @@
 # When you change this file please also update `doc/doc-requirements.txt`
 # which needs to be kept in sync manually.
-kubernetes>=4.*
+kubernetes==9.0.*
 escapism
 tornado
 traitlets