Uploads broken

[srittau]

Currently, it seems that PyPI uploads are broken. See for example these two runs:

• https://github.com/typeshed-internal/stub_uploader/actions/runs/3228142124/jobs/5283851894
• https://github.com/typeshed-internal/stub_uploader/actions/runs/3247524346/jobs/5327604962

But none of the stubs (regex, colorama, paho-mqtt, redis) have recent uploads.

[srittau]

Manual updates are also broken: https://github.com/typeshed-internal/stub_uploader/actions/runs/3249872314/jobs/5332782779

[JelleZijlstra]

I suspect there's something wrong with the sort_by_dependency() function; that would explain why nothing happens after it prints what packages it would upload. I can't reproduce the problem locally though.

[JelleZijlstra]

I just triggered a manual run and it worked: https://pypi.org/project/types-regex/#history got uploaded. Not sure what changed.

[hauntsaninja]

Sorry for the trouble! I just merged something to typeshed, so hopefully we'll have another datapoint soon. One more note is I'll have limited availability this weekend. Feel free to revert #61 if need be (looks to me like things were working before that, at least).

@JelleZijlstra I might be confused, but didn't we print the Building and uploading stubs for: redis, colorama, paho-mqtt message after sort_by_dependency?

[JelleZijlstra]

@JelleZijlstra I might be confused, but didn't we print the Building and uploading stubs for: redis, colorama, paho-mqtt message after sort_by_dependency?

We do, but the printed output is the input to sort_by_dependency:

stub_uploader/stub_uploader/upload.py

Line 25 in f3e24f6

print("Building and uploading stubs for:", ", ".join(distributions))

[hauntsaninja]

So https://github.com/typeshed-internal/stub_uploader/actions/runs/3252136124/jobs/5338000226 seems to have worked

[srittau]

I also (successfully) triggered the rebuild of redis, colorama, and paho-mqtt. I would still like to keep this open for a bit. I'm wary of a bug that fixes itself.

[hauntsaninja]

Looks like things have continued to work, as far as I can tell.

We do, but the printed output is the input to sort_by_dependency

I think that's true on main, but false on the commit we saw the issue (got changed in #69), see:

stub_uploader/stub_uploader/upload_changed.py

Line 35 in 0c5e1d0

print("Building and uploading stubs for:", ", ".join(to_upload))

I'm really a little mystified. Assuming there was an issue that is now fixed, #69 seems most likely to have fixed it.

[srittau]

Well, this hasn't happened since, so closing this for now. I blame weird caching issues or the phase of the moon.