Skip to content

ci(release): skip python rpm in gateway smoke test#1559

Merged
pimlock merged 1 commit into
mainfrom
no-issue-skip-python-rpm-smoke/pml
May 25, 2026
Merged

ci(release): skip python rpm in gateway smoke test#1559
pimlock merged 1 commit into
mainfrom
no-issue-skip-python-rpm-smoke/pml

Conversation

@pimlock
Copy link
Copy Markdown
Collaborator

@pimlock pimlock commented May 25, 2026

Summary

Update the Fedora RPM smoke test to install only the CLI and gateway RPMs before running openshell-gateway --version. The previous *.rpm glob also installed the optional Python SDK RPM, which is not needed for this gateway smoke check.

Fedora fixed the earlier Rocky-specific Python ABI issue, but the smoke still failed because python3-openshell-0.0.48-1.fc44 requires python3.14dist(grpcio) >= 1.60 and python3.14dist(protobuf) >= 4.25, which Fedora 44 does not provide. Since this smoke validates the gateway package path, skipping the Python subpackage avoids blocking the release on unrelated SDK RPM dependencies.

Related Issue

None. Follow-up from failed release run https://github.com/NVIDIA/OpenShell/actions/runs/26415139181/job/77759471779.

Changes

  • Replace ./package-input/*.rpm with explicit CLI and gateway RPM globs in the release-tag RPM smoke step.
  • Leave python3-openshell out of the gateway smoke install path.

Testing

  • mise run pre-commit passes.
  • Unit tests added/updated - not applicable; workflow-only change.
  • E2E tests added/updated (if applicable) - not applicable.
  • YAML parsed with ruby -e "require 'yaml'; YAML.load_file('.github/workflows/release-tag.yml'); puts 'ok'".
  • Local Fedora x86_64 smoke passed against run 26415139181 artifacts: docker run --rm --platform linux/amd64 -v ... fedora:latest bash -lc 'set -euo pipefail; dnf install -y /work/package-input/openshell-[0-9]*.rpm /work/package-input/openshell-gateway-*.rpm; openshell-gateway --version' returned openshell-gateway 0.0.48.

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable) - not applicable.

Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
@github-actions
Copy link
Copy Markdown

@pimlock pimlock enabled auto-merge (squash) May 25, 2026 19:36
@pimlock pimlock disabled auto-merge May 25, 2026 20:40
@pimlock pimlock merged commit 286ce7c into main May 25, 2026
27 checks passed
@pimlock pimlock deleted the no-issue-skip-python-rpm-smoke/pml branch May 25, 2026 20:41
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