Skip to content

Conversation

@skoya76
Copy link
Contributor

@skoya76 skoya76 commented Dec 5, 2025

No description provided.

@skoya76 skoya76 added the do-not-merge Prohibit to merge PR label Dec 5, 2025
@skoya76 skoya76 force-pushed the e2e-test-controller branch from c07d075 to f427a19 Compare December 5, 2025 05:37
Integrate the job that detects data corruption via checksum verification into
the controller. Apply minor fixes to keep existing tests passing. New tests
will be added in the next commit.

Signed-off-by: Kohya Shiozaki <[email protected]>
previously ENABLE_CHECKSUM_VERIFY was set as a literal env var in each job, so
changing skip-checksum-verify later did nothing because the pod env was fixed
at creation. Now each backup/verification/deletion job reads
ENABLE_CHECKSUM_VERIFY from a ConfigMap, and the ConfigMap is removed when the
FinBackup is deleted. As a result, any new pods started after adding
skip-checksum-verify will honor the new value.

Signed-off-by: Kohya Shiozaki <[email protected]>
@skoya76 skoya76 force-pushed the e2e-test-controller branch from f427a19 to d6c63dc Compare December 5, 2025 06:28
Signed-off-by: Kohya Shiozaki <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge Prohibit to merge PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants