Skip to content

e2e: run the functional suite against a real OCP cluster - #559

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift:mainfrom
gauron99:push-txmuypkryoum
Oct 20, 2026
Merged

openshift-merge-bot[bot] merged 1 commit into
openshift:mainfrom
gauron99:push-txmuypkryoum

Conversation

@gauron99

@gauron99 gauron99 commented Oct 19, 2026 •

Copy link
Copy Markdown

Add hack/openshift/ to run the e2e suite against an existing OpenShift cluster an engineer is oc-login'd to:

  • prepare-ocp-cluster.sh: idempotent, env-gated setup. Installs Gitea (+ kube-public/gitea-endpoint ConfigMap), Red Hat OpenShift Pipelines, and (optionally) CMA for the keda deployer. No registry is deployed: the suite pushes to ttl.sh with on-cluster builds (FUNC_REMOTE=true).

  • run-e2e-tests.sh: sets REGISTRY to a fresh ttl.sh namespace, DEFAULT_DEPLOYER=raw, FUNC_REMOTE=true and FUNC_MANAGEMENT_DISABLED=true, and execs make test-e2e.

The default E2E_SKIP drops specs that cannot work under FUNC_REMOTE=true (currently -- details in run-e2e-tests.sh): the SSH-repository specs (need an exposed SSH endpoint), Middleware Update and registry-auth-secret (their stale-CLI seed premise breaks remotely), and the monorepo subdirectory spec (the remote seed deploy leaves the function name out of the committed func.yaml, so the operator's describe fails). The middleware-rebuild spec self-skips while STALE_FUNCTION_IMAGE is unset.

@openshift-ci openshift-ci Bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Oct 19, 2026
@openshift-ci
openshift-ci Bot requested a review from maschmid October 19, 2026 14:10
@openshift-ci

openshift-ci Bot commented Oct 19, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gauron99

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

The pull request process is described 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

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 19, 2026
Add hack/openshift/ to run the e2e suite against an existing OpenShift
cluster an engineer is oc-login'd to:

- prepare-ocp-cluster.sh: idempotent, env-gated setup. Installs Gitea
  (+ kube-public/gitea-endpoint ConfigMap), Red Hat OpenShift Pipelines,
  and (optionally) CMA for the keda deployer. No registry is deployed:
  the suite pushes to ttl.sh with on-cluster builds (FUNC_REMOTE=true).

- run-e2e-tests.sh: sets REGISTRY to a fresh ttl.sh namespace,
  DEFAULT_DEPLOYER=raw, FUNC_REMOTE=true and
  FUNC_MANAGEMENT_DISABLED=true, and execs make test-e2e.

- Makefile: pass E2E_SKIP and STALE_FUNCTION_IMAGE through to ginkgo.

The default E2E_SKIP drops specs that cannot work under
FUNC_REMOTE=true (details in run-e2e-tests.sh): the SSH-repository
specs (need an exposed SSH endpoint), Middleware Update and
registry-auth-secret (their stale-CLI seed premise breaks remotely),
and the monorepo subdirectory spec (the remote seed deploy leaves the
function name out of the committed func.yaml, so the operator's
describe fails). The middleware-rebuild spec self-skips while
STALE_FUNCTION_IMAGE is unset.
@gauron99

gauron99 commented Oct 19, 2026 •

Copy link
Copy Markdown
Author

/hold to merge after functions-dev#269 gets propagated downstream

@openshift-ci openshift-ci Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 19, 2026
@gauron99

Copy link
Copy Markdown
Author

the E2E_SKIP in makefile is coming in via #579

@maschmid

Copy link
Copy Markdown

/lgtm

@gauron99

Copy link
Copy Markdown
Author

/unhold

@openshift-ci openshift-ci Bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 20, 2026
@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Oct 20, 2026
@openshift-ci

openshift-ci Bot commented Oct 20, 2026

Copy link
Copy Markdown

@gauron99: all tests passed!

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.

@openshift-merge-bot
openshift-merge-bot Bot merged commit 7d35b3c into openshift:main Oct 20, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants