Skip to content

Conversation

@graebm
Copy link
Contributor

@graebm graebm commented Jan 8, 2021

BACKGROUND:
Our release pipeline needs to wait until a newly uploaded version is available in pypi, before installing the new version for testing.

ISSUE:
We were using pip search to determine that the new version was available, but pip search has been disabled at a server level (https://status.python.org/incidents/grk0k7sz6zkp). We've had issues with this approach in the past anyway, where it looked like it was globally available, but then didn't install on some specific platform.

CHANGES:
New strategy is to just retry install attempts until they eventually work (or we cap out).

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Our release pipeline needs to wait until a newly uploaded version is available in pypi, before installing the new version for testing. We were using `pip search` to determine that the new version was available, but `pip search` has been disabled at a server level (https://status.python.org/incidents/grk0k7sz6zkp). We've had issues with this approach in the past anyway, where it looked like it was globally available, but then didn't install on some specific platform.

New strategy is to just retry install attempts until they work (or cap out).
@graebm graebm merged commit 56aa08e into main Jan 8, 2021
@graebm graebm deleted the new-pypi-wait-strat branch January 8, 2021 17:42
graebm added a commit to aws/aws-iot-device-sdk-python-v2 that referenced this pull request Feb 22, 2021
This technique has been used in aws-crt-python (see PR awslabs/aws-crt-python#221) for weeks with great success. Use it here now too.
graebm added a commit to aws/aws-iot-device-sdk-python-v2 that referenced this pull request Feb 24, 2021
The release pipeline would occasionally fail, due to the flakiness of our wait-for-pypi.py script.

This technique has been used in aws-crt-python (see PR awslabs/aws-crt-python#221) for weeks with great success. Use it here now too.
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