Skip to content
This repository has been archived by the owner on Aug 17, 2023. It is now read-only.

No module named 'kubernetes.client.api' in CI tests #421

Closed
jinchihe opened this issue Nov 21, 2019 · 2 comments · Fixed by #423
Closed

No module named 'kubernetes.client.api' in CI tests #421

jinchihe opened this issue Nov 21, 2019 · 2 comments · Fixed by #423
Labels
kind/bug

Comments

@jinchihe
Copy link
Member

Kubeflow fairing CI failed due to

../../kubeflow/fairing/builders/base_builder.py:3: in <module>
    from kubernetes import client
/home/travis/virtualenv/python3.7.1/lib/python3.7/site-packages/kubernetes-11.0.0b1-py3.7.egg/kubernetes/__init__.py:19: in <module>
    import kubernetes.client
/home/travis/virtualenv/python3.7.1/lib/python3.7/site-packages/kubernetes-11.0.0b1-py3.7.egg/kubernetes/client/__init__.py:20: in <module>
    from kubernetes.client.api.admissionregistration_api import AdmissionregistrationApi
E   ModuleNotFoundError: No module named 'kubernetes.client.api'
@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the label kind/bug to this issue, with a confidence of 0.95. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

@jinchihe
Copy link
Member Author

This is caused by kubernetes python Client issue: kubernetes-client/python#1008

@jinchihe jinchihe mentioned this issue Nov 21, 2019
Closed
consideRatio added a commit to consideRatio/fairing that referenced this issue Nov 21, 2019
@consideRatio
Test PyPI pre-releases but allow for failures
a6f8a0c 
Using `python setup.py install` will install PyPI pre-releases of
packages while `pip install -r requirements.txt` won't do that. This
lead to CI tests failing as reported in kubeflow#421. This commit updates the CI
build setup to run two parallell tests, one where we don't use
pre-releases and one where we do but allow them to fail and won't wait
on the run to complete either.

ref: jupyterhub/kubespawner#370 (comment)
Fixes kubeflow#421
@consideRatio consideRatio mentioned this issue Nov 21, 2019
Merged
k8s-ci-robot pushed a commit that referenced this issue Nov 21, 2019
@consideRatio @k8s-ci-robot
Test PyPI pre-releases but allow for failures (#423)
fd86fc7 
Using `python setup.py install` will install PyPI pre-releases of
packages while `pip install -r requirements.txt` won't do that. This
lead to CI tests failing as reported in #421. This commit updates the CI
build setup to run two parallell tests, one where we don't use
pre-releases and one where we do but allow them to fail and won't wait
on the run to complete either.

ref: jupyterhub/kubespawner#370 (comment)
Fixes #421
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
kind/bug
Projects
None yet
Milestone
No milestone
1 participant
@jinchihe