Skip to content

Add kubernetes-client/python-base as submodule to this repo#222

Merged
mbohlool merged 3 commits into
kubernetes-client:masterfrom
mbohlool:sub
May 19, 2017
Merged

Add kubernetes-client/python-base as submodule to this repo#222
mbohlool merged 3 commits into
kubernetes-client:masterfrom
mbohlool:sub

Conversation

@mbohlool

Copy link
Copy Markdown
Contributor

We need to restructure python client according to https://github.com/kubernetes-client/community/blob/master/design-docs/clients-library-structure.md

for that purpose, base folder is moved to the other repo.

@mbohlool mbohlool requested a review from sebgoa May 19, 2017 07:35

@sebgoa sebgoa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems good to me. I did not pay attention to the change, but it looks good. I suppose the move is not finished yet.

FWIW the python setup.py install gave me issues on 2.7.

I installed with pip install -e . after having initialized the submodule with git submodule update --init --recursive. The README might need some adjusting.

@mbohlool

Copy link
Copy Markdown
Contributor Author

I see python setup.py install is failing on master, so it is not related to this PR. I will look at it. Would you file an issue for it?

@mbohlool mbohlool merged commit 87e16bb into kubernetes-client:master May 19, 2017
yliaog pushed a commit to yliaog/client-python that referenced this pull request Jan 8, 2022
…labels

configmap e2e: use labels and label selector
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants