-
Notifications
You must be signed in to change notification settings - Fork 227
add custom setup.py to install mpijob module #579
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vandanavk Thanks for the great contribution!
sdk/python/v2beta1/setup.py
Outdated
""" | ||
setuptools.setup( | ||
name="kubeflow-mpijob", | ||
version="0.0.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version="0.0.1", | |
version="0.4.0", |
Should we use the release version?
sdk/python/v2beta1/setup.py
Outdated
name="kubeflow-mpijob", | ||
version="0.0.1", | ||
author="Kubeflow Authors", | ||
author_email="wg-batch@kubernetes.io", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
author_email="wg-batch@kubernetes.io", | |
author_email="kubeflow-discuss@googlegroups.com", |
Kubernetes WG batch doesn't maintain mpi-operator. So we should set another one to here.
@terrytangyuan @alculquicondor What do you think about using kubeflow-discuss@googlegroups.com
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It’s better not to spam that mailing list with bug reports
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I usually put a faux GitHub email here - the <name>@users.noreply.github.com
but if someone has an actual email, that is likely ideal. I'm not sure how often these emails get used, but I would say if the group is protected by someone needing to join, that would get around bots that scrape the emails (and spam) and maybe even be a good approach? Someone that wants help with the package should maybe reach out to this list?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay I updated to kubeflow@users.noreply.github.com
. it is definitely useless and fake, but will satisfy the pypi requirements if you ever decide to release there (you just need the contact field defined with something resembling an email).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No objection.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sgtm
Problem: we cannot currently install the python mpijob module Solution: add a setup.py proper that is ignored by the generator Signed-off-by: vsoch <vsoch@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: terrytangyuan The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Hi @vsoch , what's the timeline like to release this package to pypi? |
hey @cliu-nuro - there is no timeline, it's more that we could do it when it's requested. It isn't super hard, a maintainer/owner would need to create the first (dummy) deployment locally, usually you just do: python setup.py sdist
twine upload dist/<path-to-tar-gz> or create the package manually in the UI. Then get a pypi token / username for the package, explicitly, and then you just need name: release mpijob
on:
release:
types: [created]
jobs:
deploy:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- name: Install
run: conda create --quiet --name mpijob twine
- name: Install dependencies
run: |
export PATH="/usr/share/miniconda/bin:$PATH"
source activate mpijob
cd /path/to/where/mpijob/setup.py/is
pip install -e .[all]
pip install setuptools wheel twine
- name: Build and publish
env:
TWINE_USERNAME: ${{ secrets.PYPI_USER }}
TWINE_PASSWORD: ${{ secrets.PYPI_PASS }}
run: |
export PATH="/usr/share/miniconda/bin:$PATH"
source activate mpijob
python setup.py sdist bdist_wheel
twine upload dist/* And that should upload on releases. I'm not a maintainer here so I can't do most of that, but happy to provide guidance! |
@terrytangyuan what was the process to release to pypi? |
I don't think we have a process yet. It's not published to PyPI. |
We can instruct users to install directly from GitHub. |
Problem: we cannot currently install the python mpijob module
Solution: add a setup.py proper that is ignored by the generator
Notes:
pip install .
Future improvements:
This will close #578