Skip to content

chore(e2e): select the last matched droute pod - #3075

Merged
openshift-merge-bot[bot] merged 3 commits into
redhat-developer:mainfrom
PatAKnight:fix-droute
Jul 4, 2025
Merged

chore(e2e): select the last matched droute pod#3075
openshift-merge-bot[bot] merged 3 commits into
redhat-developer:mainfrom
PatAKnight:fix-droute

Conversation

@PatAKnight

@PatAKnight PatAKnight commented Jul 4, 2025

Copy link
Copy Markdown
Member

Description

We are starting to see a wave of failing PR tests where the rbac showcase tests are not starting up. Looking at the logs, it seems like there are now two droute pods that are showing up where originally we only had one.

This is PR mainly aims to confirm my suspensions by selecting the first matched droute pod and checking if that fixes the problem.

Which issue(s) does this PR fix

  • Fixes #?

PR acceptance criteria

Please make sure that the following steps are complete:

  • GitHub Actions are completed and successful
  • Unit Tests are updated and passing
  • E2E Tests are updated and passing
  • Documentation is updated if necessary (requirement for new features)
  • Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

Signed-off-by: Patrick <pknight@redhat.com>
@github-actions

github-actions Bot commented Jul 4, 2025

Copy link
Copy Markdown
Contributor

Signed-off-by: Patrick <pknight@redhat.com>
Signed-off-by: Patrick <pknight@redhat.com>
@PatAKnight PatAKnight changed the title chore(e2e): select the first matched droute pod chore(e2e): select the last matched droute pod Jul 4, 2025
@albarbaro

Copy link
Copy Markdown
Member

/lgtm
/approve

@openshift-ci

openshift-ci Bot commented Jul 4, 2025

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: albarbaro, dzemanov

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 label Jul 4, 2025
@openshift-merge-bot
openshift-merge-bot Bot merged commit 0453215 into redhat-developer:main Jul 4, 2025
14 checks passed
chadcrum pushed a commit to chadcrum/rhdh that referenced this pull request Aug 3, 2025
* chore(e2e): select the first matched droute pod

Signed-off-by: Patrick <pknight@redhat.com>

* chore(e2e): check to see if the last pod works

Signed-off-by: Patrick <pknight@redhat.com>

* chore(e2e): reenable skipped tekton pipelines test

Signed-off-by: Patrick <pknight@redhat.com>

---------

Signed-off-by: Patrick <pknight@redhat.com>
guyoron1 pushed a commit to guyoron1/rhdh that referenced this pull request Sep 29, 2025
* chore(e2e): select the first matched droute pod

Signed-off-by: Patrick <pknight@redhat.com>

* chore(e2e): check to see if the last pod works

Signed-off-by: Patrick <pknight@redhat.com>

* chore(e2e): reenable skipped tekton pipelines test

Signed-off-by: Patrick <pknight@redhat.com>

---------

Signed-off-by: Patrick <pknight@redhat.com>
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.

3 participants