-
Notifications
You must be signed in to change notification settings - Fork 25
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
Jupyter Scheduler 2.10.0 Source Distribution tar built too large causing PyPI upload failure #558
Comments
For reference, here is the relevant log excerpt:
|
Jupyter Scheduler 2.10.0 npm package is now available at https://www.npmjs.com/package/@jupyterlab/scheduler, Source Distribution is now available at PyPI https://pypi.org/project/jupyter-scheduler/2.10.0/#files. Let's use this issue to track the need to understand why Jupyter Scheduler 2.10.0 Source Distribution tar was built too large causing PyPI upload failure and to prevent it happening in the next release. jupyter_releaser issue on the topic: jupyter-server/jupyter_releaser#592 |
Out of curiosity, do you know why it produces so big a distribution? From a quick look it seems that you might be missing: [tool.jupyter-releaser.hooks]
before-build-python = ["jlpm clean:all"] in the |
@krassowski no. I've created jupyter-server/jupyter_releaser#592 in jupyter_releaser repo to surface the problem and hopefully get some insight from jupyter_releaser contributors. Thank you for the suggestion and generally for looking into this. |
Do you have the contents of the package built locally with |
It looks like it includes I think in addition
so That said, I already see So why does it include things from the git repo? In the logs of check-release action (https://github.com/jupyter-server/jupyter-scheduler/actions/runs/11809051201/job/32898683727) I see that the releaser is reading configuration from
Also, that one does include the clean hook: jupyter-scheduler/package.json Lines 132 to 143 in 14c4451
Interesting. It looks like it did not use hatch at all? |
None of that helps yet: #561 I went ahead and triggered a new |
I tried aligning the scheduler config with other repos using releaser in #561 but nothing helped. The thing is that jupyter-releaser does not do anything bespoke, it just runs
|
Running |
Description
Jupyter Scheduler 2.10.0 Source Distribution initial upload failed due to PyPI source distributions size limits (~150 Mb) due to
tar
build being drastically larger in size vs before jupyter-releaser introduction.Built distribution upload went through, npm upload did not as it's later in the script.
How to reproduce
jupyter-releaser build-python
, see its size (>100 Mb)Expected behavior
The text was updated successfully, but these errors were encountered: