Skip to content

fix: stabilize e2e auth setup for CI - #89

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift:masterfrom
pmeida:fix/e2e-ci-stability
Aug 5, 2026
Merged

fix: stabilize e2e auth setup for CI#89
openshift-merge-bot[bot] merged 1 commit into
openshift:masterfrom
pmeida:fix/e2e-ci-stability

Conversation

@pmeida

@pmeida pmeida commented Aug 5, 2026

Copy link
Copy Markdown

Summary

  • Wait for login form or username indicator after page.goto('/') to prevent Execution context was destroyed error during OAuth redirects
  • Use evaluate click for the skip tour button to bypass PF6 modal backdrop intercepting pointer events

Problem

The e2e auth setup crashes in CI (AWS) because page.evaluate() runs while the console is still redirecting through OAuth. Locally (CRC) the redirect completes fast enough to avoid the race.

Error from CI:

page.evaluate: Execution context was destroyed, most likely because of a navigation

After fixing that, the skip tour button click times out because a PF6 modal backdrop intercepts pointer events.

- Wait for login form or username indicator after goto('/') to avoid
  'Execution context was destroyed' during OAuth redirects
- Use evaluate click for skip tour button to bypass PF6 modal backdrop
  intercepting pointer events

Signed-off-by: Pedro Almeida <pealmeid@redhat.com>
@openshift-ci
openshift-ci Bot requested review from dsimansk and twoGiants August 5, 2026 07:56
@openshift-ci

openshift-ci Bot commented Aug 5, 2026

Copy link
Copy Markdown

@pmeida: 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.

pmeida added a commit to pmeida/release that referenced this pull request Aug 5, 2026
Points rehearsal at pmeida/faas-console-plugin fix/e2e-ci-stability
while openshift/faas-console-plugin#89 is being validated.
@dsimansk

dsimansk commented Aug 5, 2026

Copy link
Copy Markdown

/approve
/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Aug 5, 2026
@openshift-ci

openshift-ci Bot commented Aug 5, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dsimansk, pmeida

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 Aug 5, 2026
@openshift-merge-bot
openshift-merge-bot Bot merged commit e5e4348 into openshift:master Aug 5, 2026
6 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants