4.19: OLS-3155: adding cluster claim

[JoaoFula]

and changing getting kubeconfig, base url and password

Summary by CodeRabbit

• Refactor
  • Updated cluster provisioning workflow to use ephemeral cluster provisioning instead of standard provisioning
  • Changed authentication credential handling to use Kubernetes secrets directly
  • Added automatic cluster deprovisioning at the end of test runs
  • Streamlined kubeconfig and credential sourcing logic

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR updates a Tekton pipeline to replace static cluster provisioning with ephemeral cluster provisioning. The provision-ephemeral-cluster task is introduced with clusterClaim metadata, credentials are passed via Kubernetes secret reference to E2E tests, kubeconfig and password are extracted from that secret in the test execution, and cleanup is automated in the finally section.

Changes

Ephemeral Cluster Credential Flow

Layer / File(s)	Summary
Ephemeral cluster provisioning task .tekton/integration-tests/lightspeed-console-pre-commit.yaml	Replaces previous cluster provisioning with new provision-ephemeral-cluster task, configured with clusterClaim metadata (architecture, cloud, region, variant, owner, product, timeout, version) and wired to run after eaas-provision-space and lint.
E2E test credential parameter wiring .tekton/integration-tests/lightspeed-console-pre-commit.yaml	Updates ols-e2e-tests task to accept clusterCredentialsSecretRef parameter sourced from provision-ephemeral-cluster task result, and declares the parameter in taskSpec to enable subsequent steps to consume it.
E2E test credential usage and BASE_URL derivation .tekton/integration-tests/lightspeed-console-pre-commit.yaml	Updates E2E test step to extract kubeconfig from clusterCredentialsSecretRef secret, writes kubeconfig file, derives LOGIN_PASSWORD from the same secret, computes BASE_URL using oc command against the provisioned cluster, and removes prior password-path extraction mechanism.
Cluster cleanup and downstream task updates .tekton/integration-tests/lightspeed-console-pre-commit.yaml	Adds deprovision-ephemeral-cluster task in finally section parameterized from provision-ephemeral-cluster claim metadata, and updates gather-cluster-resources task to reference kubeconfig file path directly.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 From static clusters to ephemeral flights,
Credentials flow through secrets so bright,
Each test now provisions its own little space,
Then cleans up the queue at a reasonable pace—
Tekton pipelines dancing with grace!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title references adding cluster claim and the issue OLS-3155, which aligns with the main change in the changeset—replacing provision-cluster with provision-ephemeral-cluster and updating cluster credential handling.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[openshift-ci]

[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 joaofula 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:

• .tekton/OWNERS

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

[openshift-ci-robot]

@JoaoFula: This pull request references OLS-3155 which is a valid jira issue.

Details

In response to this:

and changing getting kubeconfig, base url and password

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.

[JoaoFula]

/retest

[JoaoFula]

/retest

[JoaoFula]

/retest

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.tekton/integration-tests/lightspeed-console-pre-commit.yaml:
- Around line 55-58: Remove the undeclared PipelineTask parameters
prEventPayload and prEventHeaders from the Pipeline spec in
.tekton/integration-tests/lightspeed-console-pre-commit.yaml so the
provision-ephemeral-cluster task (openshift/konflux-tasks
tasks/provision-ephemeral-cluster/0.1) matches its declared contract; locate the
entries named prEventPayload and prEventHeaders and delete them. Also add or
ensure a step that runs Playwright browser installation (e.g., run npx
playwright install) before the E2E test step in the same pipeline so browsers
are available at test runtime.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: adf3bdb7-01e4-4145-97d2-8cff9f418b03

📥 Commits

Reviewing files that changed from the base of the PR and between c362249 and 11c3ea2.

📒 Files selected for processing (1)

• .tekton/integration-tests/lightspeed-console-pre-commit.yaml

[JoaoFula]

/retest