Skip to content

K8SPS-867 version service and smart update e2e tests should use the actual latest and minimum supported mysql versions - #1522

Open
gkech wants to merge 3 commits into
mainfrom
K8SPS-867
Open

K8SPS-867 version service and smart update e2e tests should use the actual latest and minimum supported mysql versions#1522
gkech wants to merge 3 commits into
mainfrom
K8SPS-867

Conversation

@gkech

@gkech gkech commented Sep 1, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

CHANGE DESCRIPTION

Jira: https://perconadev.atlassian.net/browse/K8SPS-867

Problem:

we have e2e tests that are using mysql version 8.0.28

On our release notes we mention:

Percona Operator for MySQL based on Percona Server for MySQL - Percona Operator for MySQL 1.2.0 (2026-07-03)

Percona Server for MySQL 8.4.10-10.1
Percona Server for MySQL 8.0.46-37.1

smart update:

| yq eval '.spec.mysql.image="percona/percona-server:8.0.28-19"' - \

version service:

| yq eval '.spec.mysql.image="percona/percona-server:8.0.28-19"' - \

Update the tests to use the versions we are currently support, this work most probably should be redone once what we support changes.

Cause:
Short explanation of the root cause of the issue if applicable.

Solution:

With this pr, we are adjusting the aforementioned test to use the versions mentioned on the latest operator release.

CHECKLIST

Jira

  • Is the Jira ticket created and referenced properly?
  • Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
  • Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

  • Is an E2E test/test case added for the new feature/change?
  • Are unit tests added where appropriate?

Config/Logging/Testability

  • Are all needed new/changed options added to default YAML files?
  • Are all needed new/changed options added to the Helm Chart?
  • Did we add proper logging messages for operator actions?
  • Did we ensure compatibility with the previous version or cluster upgrade process?
  • Does the change support oldest and newest supported PS version?
  • Does the change support oldest and newest supported Kubernetes version?

@gkech
K8SPS-867 version service and smart update e2e tests should use the a…
6e66caf 
…ctual latest and minimum supported mysql versions
@pull-request-size pull-request-size Bot added the size/M 30-99 lines label Sep 1, 2026
@github-actions github-actions Bot added the tests label Sep 1, 2026
@gkech
gkech marked this pull request as ready for review September 1, 2026 10:39
@gkech
gkech requested review from egegunes, hors, mayankshah1607, nmarukovich, oksana-grishchenko and pooknull and a balanced review from Copilot September 1, 2026 10:40
Copilot AI reviewed Sep 1, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The version substitutions are internally consistent across test inputs, assertions, and version-service fixtures.

Pull request overview

Updates E2E fixtures to exercise currently supported minimum and latest Percona Server versions.

Changes:

  • Updates version-service scenarios for MySQL 8.0 and 8.4.
  • Updates SmartUpdate coverage from 8.4.8 to 8.4.10.
  • Refreshes mocked version-service metadata and hashes.
File summaries
File Description
e2e-tests/tests/version-service/11-create-cluster-80-latest.yaml Uses minimum supported 8.0 image.
e2e-tests/tests/version-service/11-assert.yaml Expects latest 8.0 version.
e2e-tests/tests/version-service/09-create-cluster-80-recommended.yaml Uses minimum supported 8.0 image.
e2e-tests/tests/version-service/09-assert.yaml Expects recommended 8.0 version.
e2e-tests/tests/version-service/07-create-cluster-exact.yaml Updates exact-version scenario.
e2e-tests/tests/version-service/07-assert.yaml Expects exact 8.4.8 version.
e2e-tests/tests/version-service/05-create-cluster-latest.yaml Starts latest-policy scenario from 8.4.8.
e2e-tests/tests/version-service/05-assert.yaml Expects latest 8.4.10 version.
e2e-tests/tests/version-service/03-assert.yaml Refreshes disabled-policy expectations.
e2e-tests/tests/version-service/02-assert.yaml Refreshes never-policy expectations.
e2e-tests/tests/version-service/01-create-cluster-recommended.yaml Starts recommended-policy scenario from 8.4.8.
e2e-tests/tests/version-service/01-assert.yaml Expects recommended 8.4.10 version.
e2e-tests/tests/smart-update/02-check-smart-update.yaml Targets 8.4.10 during SmartUpdate.
e2e-tests/tests/smart-update/01-create-cluster.yaml Starts SmartUpdate from 8.4.8.
e2e-tests/conf/operator.9.9.9.ps-operator.json Refreshes mocked versions, images, and hashes.
Review details
  • Files reviewed: 15/15 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

egegunes
egegunes approved these changes Sep 2, 2026
View reviewed changes

@egegunes egegunes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is okay for unblocking ourselves but we need to rework these tests to remove the need to update versions

@JNKPercona

JNKPercona commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator
Test Name Result Time
async-ignore-annotations-8-4 passed 00:09:38
async-global-metadata-8-4 passed 00:14:50
async-upgrade-8-0 passed 00:12:17
async-upgrade-8-4 passed 00:13:46
auto-config-8-4 passed 00:18:32
config-8-4 passed 00:27:03
config-router-8-0 passed 00:08:58
config-router-8-4 passed 00:08:28
custom-users-8-4 passed 00:06:36
demand-backup-8-0 passed 00:20:52
demand-backup-8-4 passed 00:21:28
gr-pitr-minio-8-4 passed 00:27:50
gr-pitr-encrypted-minio-8-4 passed 00:17:05
gr-pitr-one-pod-8-4 passed 00:12:44
async-pitr-minio-8-4 passed 00:25:04
demand-backup-cloud-8-4 passed 00:22:00
demand-backup-retry-8-4 passed 00:17:30
demand-backup-incremental-8-0 passed 00:35:01
demand-backup-incremental-8-4 passed 00:36:09
async-data-at-rest-encryption-8-0 passed 00:14:25
async-data-at-rest-encryption-8-4 passed 00:14:37
gr-cross-cluster-8-0 passed 00:32:39
gr-cross-cluster-8-4 passed 00:33:04
gr-cross-cluster-backup-8-0 passed 00:27:43
gr-cross-cluster-backup-8-4 passed 00:26:49
gr-global-metadata-8-4 passed 00:15:34
gr-data-at-rest-encryption-8-0 passed 00:15:59
gr-data-at-rest-encryption-8-4 passed 00:14:53
gr-demand-backup-8-4 passed 00:13:02
gr-demand-backup-cloud-8-4 passed 00:23:04
gr-demand-backup-haproxy-8-4 passed 00:10:46
gr-demand-backup-incremental-8-0 passed 00:26:49
gr-demand-backup-incremental-8-4 passed 00:25:11
gr-demand-backup-incremental-compressed-8-0 passed 00:12:53
gr-demand-backup-incremental-compressed-8-4 passed 00:12:09
gr-demand-backup-incremental-encrypted-8-0 passed 00:19:27
gr-demand-backup-incremental-encrypted-8-4 passed 00:17:51
gr-finalizer-8-4 passed 00:05:54
gr-haproxy-8-0 passed 00:04:57
gr-haproxy-8-4 passed 00:04:35
gr-ignore-annotations-8-4 passed 00:05:10
gr-init-deploy-8-0 passed 00:10:11
gr-init-deploy-8-4 passed 00:09:33
gr-one-pod-8-4 passed 00:05:51
gr-recreate-8-4 passed 00:17:30
gr-scaling-8-4 failure 00:07:56
gr-scheduled-backup-8-4 passed 00:26:34
gr-scheduled-backup-incremental-8-4 passed 00:34:36
gr-security-context-8-4 passed 00:10:24
gr-self-healing-8-4 passed 00:21:47
gr-tls-cert-manager-8-4 passed 00:11:32
gr-users-8-4 passed 00:07:27
gr-upgrade-8-0 passed 00:09:54
gr-upgrade-8-4 passed 00:11:03
haproxy-8-0 passed 00:09:24
haproxy-8-4 passed 00:08:49
init-deploy-8-0 passed 00:05:48
init-deploy-8-4 passed 00:06:00
limits-8-4 passed 00:04:20
monitoring-8-4 passed 00:19:04
one-pod-8-0 passed 00:06:14
one-pod-8-4 passed 00:06:21
operator-self-healing-8-4 passed 00:11:59
pvc-auto-resize-8-4 passed 00:04:40
pvc-resize-8-4 passed 00:13:32
recreate-8-4 passed 00:12:47
scaling-8-4 passed 00:10:41
scheduled-backup-8-0 passed 00:25:12
scheduled-backup-8-4 passed 00:25:18
scheduled-backup-incremental-8-0 passed 00:37:53
scheduled-backup-incremental-8-4 passed 00:36:48
service-per-pod-8-4 passed 00:05:58
sidecars-8-4 passed 00:04:57
smart-update-8-4 passed 00:09:51
storage-8-4 passed 00:04:10
switch-cluster-type-8-4 passed 00:11:35
telemetry-8-4 passed 00:06:19
tls-cert-manager-8-4 passed 00:12:13
users-8-0 passed 00:07:46
users-8-4 passed 00:08:18
version-service-8-4 passed 00:25:48
Summary Value
Tests Run 81/81
Job Duration 02:35:14
Total Test Time 21:00:00

commit: bf9a611
image: perconalab/percona-server-mysql-operator:PR-1522-bf9a611d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort, defaults to Balanced
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@egegunes egegunes egegunes approved these changes
@pooknull pooknull pooknull approved these changes
@nmarukovich nmarukovich nmarukovich approved these changes
@eleo007 eleo007 eleo007 approved these changes
@jvpasinatto jvpasinatto Awaiting requested review from jvpasinatto jvpasinatto is a code owner
@valmiranogueira valmiranogueira Awaiting requested review from valmiranogueira valmiranogueira is a code owner
@hors hors Awaiting requested review from hors
@mayankshah1607 mayankshah1607 Awaiting requested review from mayankshah1607
@oksana-grishchenko oksana-grishchenko Awaiting requested review from oksana-grishchenko

Assignees

No one assigned

Labels

size/M 30-99 lines tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@gkech @JNKPercona @egegunes @pooknull @nmarukovich @eleo007