Repository navigation
Conversation
sigstore-e2e is making CLI_CGW_VERSION mandatory for CLI_STRATEGY=openshift, with no default and no fallback (SECURESIGN-6181, sigstore-e2e PRs #103 to main and #108 to release-1.5, not yet merged). These two Tekton tasks clone sigstore-e2e at $(params.releaseBranch) (default main) and run with CLI_STRATEGY=openshift but never set CLI_CGW_VERSION, so they would start failing outright as soon as that change merges. Until now they were silently falling back to a hardcoded RHTAS 1.4.2 and validating the wrong CLI binaries. Setting CLI_CGW_VERSION=1.5.0 explicitly fixes that and pins them to the newest version confirmed published on the Red Hat content gateway. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
PR Summary by QodoSet the content gateway CLI version for OpenShift E2E tasks
AI Description
Diagram
High-Level Assessment
Files changed (2)
|
|
Closing this without merging: sigstore-e2e's Separately, hardcoding a single Reverted the added |
Problem
securesign/sigstore-e2eis changing so thatCLI_STRATEGY=openshiftrequires aCLI_CGW_VERSIONenvironment variable. There is no default and no fallback: if the variable is missing, the strategy now returns an error and the run fails outright. That change is SECURESIGN-6181, in sigstore-e2e PRs #103 (→main) and #108 (→release-1.5), not yet merged.Two Tekton tasks in this repo run sigstore-e2e with
CLI_STRATEGY: "openshift"and do not setCLI_CGW_VERSION:tasks/integration-test/sigstore-e2e.yamltasks/integration-test/sigstore-e2e-pkcs11.yamlBoth clone sigstore-e2e at
$(params.releaseBranch), which defaults tomain, so they have no pin to shield them and will pick up the change the moment it merges, breaking these tasks.Until now, they were silently falling back to a hardcoded RHTAS 1.4.2 and validating the wrong CLI binaries. Setting
CLI_CGW_VERSION=1.5.0explicitly fixes that too, pinning them to the newest version confirmed published on the Red Hat content gateway.Note:
tasks/integration-test/ansible/run-sigstore-e2e-tests.yamlis deliberately not touched — it usesCLI_STRATEGY: cli_server, which is unaffected by this change.Merge ordering
This PR should land before sigstore-e2e PRs #103 and #108, so these tasks never run against a sigstore-e2e build that requires
CLI_CGW_VERSIONwithout it being set.Jira
https://redhat.atlassian.net/browse/SECURESIGN-6181
Test plan
git diffshows only the intended 3-line additions per file, nothing removed🤖 Generated with Claude Code