Skip to content

Retry release smoke package install#1608

Open
tconley1428 wants to merge 1 commit into
mainfrom
retry-release-smoke-install
Open

Retry release smoke package install#1608
tconley1428 wants to merge 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
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.

1 participant