Retry release smoke package install by tconley1428 · Pull Request #1608 · temporalio/sdk-python · GitHub
Skip to content

Retry release smoke package install - #1608

Merged
tconley1428 merged 1 commit into
mainfrom
retry-release-smoke-install
Jun 18, 2026
Merged

tconley1428 merged 1 commit into
mainfrom
retry-release-smoke-install

Conversation

@tconley1428

Copy link
Copy Markdown
Contributor

What changed

  • Adds a bounded retry loop around the initial release package install in the shared smoke-test helper.
  • Uses --no-cache-dir for that install so retries re-check the package index.
  • Leaves dependency installation and pip check fail-fast after the release package resolves.

This covers both TestPyPI and PyPI smoke tests because both use .github/actions/release-smoke-package.

Why

The 1.29.0 release run uploaded successfully to TestPyPI, but the immediate smoke install saw a stale simple index that still ended at 1.28.0. A rerun several minutes later hit the same cached index from the runner. Retrying the package install handles package-index propagation without sleeping longer than needed.

Validation

  • uv run ruff check .github/scripts/install_release_package.py
  • uv run ruff format --check .github/scripts/install_release_package.py
  • python3 -m py_compile .github/scripts/install_release_package.py
  • Exercised _pip_install_with_retries with mocked transient failures
  • Installed temporalio==1.29.0 from TestPyPI in a temporary venv with .github/scripts/install_release_package.py

@tconley1428
tconley1428 requested a review from a team as a code owner June 17, 2026 21:19
@tconley1428
tconley1428 merged commit b51be68 into main Jun 18, 2026
24 of 28 checks passed
@tconley1428
tconley1428 deleted the retry-release-smoke-install branch June 18, 2026 15:40
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