Skip to content

Conversation

@chadcrum
Copy link
Contributor

Summary

  • Add missing selectGreetingWorkflowItem method to Orchestrator class
  • Add wait and timeout to orchestrator greeting workflow helm install
  • Increase dynamic-plugins-root volume to 5Gi for RHDH operator deployments
  • Increase Keycloak login timeout to 30 seconds for stability

Test plan

  • Verify orchestrator helm e2e tests pass
  • Verify RHDH operator deployments have sufficient volume size
  • Verify Keycloak login stability with increased timeout

🤖 Generated with Claude Code

chadcrum and others added 4 commits December 19, 2025 14:14
… install

Add --wait --timeout=5m flags to the greeting workflow helm install command
to ensure workflow pods are ready before tests execute.

Without --wait, the helm command returns immediately while pods are still
initializing, which can cause:
- Tests to run before workflows are available
- Race conditions between workflow deployment and test execution
- Pods experiencing CreateContainerConfigError during startup

With --wait, helm monitors the release and only returns success when all
pods are Running and pass readiness probes. The 5-minute timeout provides
ample time for the pods to start (observed ready time: ~90 seconds).

This ensures tests only run against fully-initialized infrastructure and
provides clearer failure messages if pods cannot start.

Related: FLPATH-2798
The default 2Gi ephemeral volume for dynamic-plugins-root is
insufficient when many plugins are enabled (orchestrator, kubernetes,
tekton, techdocs, keycloak, etc.). The init container fails with
"No space left on device" error during plugin extraction.

Increase the volume size to 5Gi for both showcase and RBAC namespaces
using the deployment.patch field in the Backstage CR.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
The default 10-second actionTimeout was being exceeded when the
Keycloak popup was slow to render, causing orchestrator RBAC tests
to fail during authentication setup.

Add explicit waitFor with 30-second timeout before interacting with
the Keycloak login form to handle slow responses.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@openshift-ci
Copy link

openshift-ci bot commented Dec 19, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign josephca for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sonarqubecloud
Copy link

@chadcrum
Copy link
Contributor Author

/ok-to-test

@github-actions
Copy link
Contributor

The image is available at:

/test e2e-ocp-helm

@openshift-ci
Copy link

openshift-ci bot commented Dec 19, 2025

@chadcrum: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-ocp-helm ceeea8b link true /test e2e-ocp-helm

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant