Skip to content

Test updates for 1.22.0 release - #1323

Open
trdoyle81 wants to merge 9 commits into
redhat-developer:v1.22from
trdoyle81:v1.22-test-updates-rc4
Open

trdoyle81 wants to merge 9 commits into
redhat-developer:v1.22from
trdoyle81:v1.22-test-updates-rc4

Conversation

@trdoyle81

@trdoyle81 trdoyle81 commented Sep 28, 2026 •

Copy link
Copy Markdown
Member

/kind failing-test

What does this PR do / why we need it:

Fixes 5 failing e2e tests on s390x with GitOps 1.22.0 RC4. Updates test expectations to match RC4 component versions and addresses infrastructure limitations.

Changes include:

  • Updated toolchain version expectations (Argo CD v3.5.3, Redis 7.2.16, Dex v2.45.1, Helm v4.2.4, Kustomize v5.8.1)
  • Fixed CSV permission test to use v1.22.0 expectations instead of v1.16.0
  • Replaced guestbook image with multi-arch nginx for s390x support
  • Added s390x Skip checks for Gitea tests with TODO for building custom image

Have you updated the necessary documentation?

  • Documentation has been updated.

Which issue(s) this PR fixes:

Fixes #GITOPS-9446

Test acceptance criteria:

  • E2E Test

How to test changes / Special notes to the reviewer:

Verified on cluster with GitOps 1.22.0 RC4:

  • 1-031_validate_toolchain:
  • 1-090_validate_permissions:
  • 1-121_validate_image_updater:
  • 1-141_source_hydrator (Gitea tests): Skip on s390x (upstream limitation)

Run tests:
ginkgo -v -focus "1-031|1-090|1-121" ./test/openshift/e2e/ginkgo/parallel

Signed-off-by: Triona Doyle <trdoyle@redhat.com>
@openshift-ci openshift-ci Bot added the kind/failing-test Categorizes issue or PR as related to a frequently failing test. label Sep 28, 2026
@coderabbitai

coderabbitai Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Repository YAML (base), Organization UI (inherited)
  • Review profile: CHILL
  • Plan: Enterprise
  • Run ID: dc6f9267-59e5-4598-81e5-133ef2e9080c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

Signed-off-by: Triona Doyle <trdoyle@redhat.com>
@trdoyle81

Copy link
Copy Markdown
Member Author

/retest

Signed-off-by: Triona Doyle <trdoyle@redhat.com>
Comment thread test/openshift/e2e/ginkgo/parallel/1-141_source_hydrator.go
Comment thread test/openshift/e2e/ginkgo/parallel/1-121_validate_image_updater_test.go Outdated
Signed-off-by: Triona Doyle <trdoyle@redhat.com>
Signed-off-by: Triona Doyle <trdoyle@redhat.com>
@trdoyle81
trdoyle81 force-pushed the v1.22-test-updates-rc4 branch from 7011b25 to 2071064 Compare September 29, 2026 09:36
Signed-off-by: Triona Doyle <trdoyle@redhat.com>
@trdoyle81

Copy link
Copy Markdown
Member Author

/retest-required

@trdoyle81

Copy link
Copy Markdown
Member Author

/test v4.22-kuttl-parallel

1 similar comment
@trdoyle81

Copy link
Copy Markdown
Member Author

/test v4.22-kuttl-parallel

@olivergondza

Copy link
Copy Markdown
Collaborator

/lgtm
/approve

@openshift-ci

openshift-ci Bot commented Oct 2, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: olivergondza

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 Oct 2, 2026
@trdoyle81

Copy link
Copy Markdown
Member Author

/retest

@chengfang

Copy link
Copy Markdown
Contributor

/test v4.22-kuttl-parallel

@ranakan19

Copy link
Copy Markdown
Contributor

is this still planned to be merged, given 1.22.0 release is out?

@svghadi

svghadi commented Oct 7, 2026

Copy link
Copy Markdown
Member

/retest

Signed-off-by: Triona Doyle <trdoyle@redhat.com>
@trdoyle81
trdoyle81 force-pushed the v1.22-test-updates-rc4 branch from 517859a to 47fcfb6 Compare October 7, 2026 11:13
@openshift-ci openshift-ci Bot removed the lgtm label Oct 7, 2026
@openshift-ci

openshift-ci Bot commented Oct 7, 2026

Copy link
Copy Markdown

New changes are detected. LGTM label has been removed.

Signed-off-by: Triona Doyle <trdoyle@redhat.com>
@varshab1210

Copy link
Copy Markdown
Member

/test v4.22-kuttl-parallel

Signed-off-by: Triona Doyle <trdoyle@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved kind/failing-test Categorizes issue or PR as related to a frequently failing test.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants