Conversation
|
Pipeline controller notification For optional jobs, comment This repository is configured in: LGTM mode |
|
@lkladnit: This pull request references OCPNETUI-56 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "5.1.0" version, but no target version was set. DetailsIn response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Repository: openshift/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: Comment |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lkladnit The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
3cbb726 to
1868fec
Compare
Skip e2e.yml when CONSOLE_URL secret is not configured Set Xvfb screen to 1920x1080 for full-size screenshots Grant cluster-admin to CI console SA for full API access The console SA had insufficient permissions - 403 on operator.openshift.io and user.openshift.io APIs, preventing the networking plugin from loading. Use cluster-admin for CI (this is a test-only SA in an ephemeral namespace). Fix login flow for no-auth mode (hot-cluster bearer-token console) In hot-cluster CI, the console runs with BRIDGE_USER_AUTH=disabled (bearer-token auth). There is no OAuth login page. Detect this by checking if the URL lands on the dashboard directly, and skip the cy.origin() OAuth flow. Also fix visit-pages to wait for sidebar nav instead of relying on quickstart data-test IDs. Install Xvfb and GTK deps for Cypress headless on ARC runner Fix Node.js install - use tar.gz instead of tar.xz (xz not on runner) Install Node.js on ARC runner (not pre-installed) Replace ci-env-controller with direct helm install from runner The ci-env-controller pod lacks helm and the ci-test-stack chart, so it cannot provision environments. Instead, install the test stack directly from the runner using helm install with the local chart. Also install helm on the runner alongside oc CLI. Finalize Hot-cluster CI. Add hot-cluster workflows, update docs, add cluster setup guide. - Add GitHub Actions workflows (hot-cluster-e2e.yml, hot-cluster-e2e-run.yml) - Add composite actions (ci-env-request, ci-env-release) - Add Dockerfile.ci and .dockerignore for GitHub Actions builds - Add ui-tests-cy/CLUSTER.md with step-by-step RHOS cluster preparation - Update MIGRATION.md and PLAN.md to reflect current state Co-authored-by: Cursor <cursoragent@cursor.com>
cf8e206 to
5c94f43
Compare
|
@lkladnit: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. |
Summary
Adds hot-cluster CI infrastructure for running Cypress E2E tests on a persistent OpenShift cluster via GitHub Actions + ARC self-hosted runners. Also fixes login flow for no-auth mode and updates test suite for OCP 5.0.
What is included
hot-cluster-e2e.yml(entry + health check) andhot-cluster-e2e-run.yml(build, provision, test, cleanup)ci-env-request/ci-env-releaseDockerfile.ciwith public UBI9 base imagesFIPS cluster support
OPENSSL_FORCE_FIPS_MODE=0for runner,GOLANG_FIPS=0for oc CLICluster setup docs
See
ui-tests-cy/CLUSTER.mdfor step-by-step RHOS cluster preparation.Depends on #489, #490, #491, #497.
Jira: OCPNETUI-56
Made with Cursor