Skip to content

ci(kubernetes): expose e2e test selection#2260

Draft
elezar wants to merge 1 commit into
mainfrom
codex/kubernetes-e2e-selection
Draft

ci(kubernetes): expose e2e test selection#2260
elezar wants to merge 1 commit into
mainfrom
codex/kubernetes-e2e-selection

Conversation

@elezar

@elezar elezar commented Jul 14, 2026

Copy link
Copy Markdown
Member

Summary

Expose optional Rust test-target and Cargo feature selection through the reusable Kubernetes e2e workflow. Empty inputs preserve the current test suite and the existing OPENSHELL_E2E_KUBERNETES_FEATURES developer-facing contract.

This remains a draft because main has no independent caller that consumes the new inputs yet. The intended first consumer is currently part of #1868; this PR deliberately does not add HA tests or callers.

Related Issue

Extracted from #1868 for independent review.

Changes

  • Add optional test-name and kubernetes-features workflow_call inputs.
  • Forward the inputs through OPENSHELL_E2E_KUBE_TEST and the existing OPENSHELL_E2E_KUBERNETES_FEATURES variable.
  • Keep empty values behavior-neutral by relying on the existing shell defaults.

Testing

  • mise run pre-commit passes
    • Attempted inside the Nix environment after pre-syncing Python dependencies. The five-minute run timed out during parallel cold Rust builds; remaining Rust processes were resource-killed with SIGKILL, not failed assertions. Completed format, license, Helm, Markdown, Python lint/typecheck/tests, packaging, and install-script checks passed.
  • Parsed the workflow YAML and asserted both input defaults and environment wiring.
  • bash -n e2e/rust/e2e-kubernetes.sh
  • Verified unset, empty, and explicit OPENSHELL_E2E_KUBERNETES_FEATURES expansion.
  • E2E tests run (no independent caller on main yet)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)

Signed-off-by: Evan Lezar <elezar@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

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