Skip to content

tests: create idempotency tests#1880

Merged
AndrewChubatiuk merged 1 commit intomasterfrom
e2e-indempotent-reconciliation
Feb 24, 2026
Merged

tests: create idempotency tests#1880
AndrewChubatiuk merged 1 commit intomasterfrom
e2e-indempotent-reconciliation

Conversation

@vrutkovs
Copy link
Collaborator

@vrutkovs vrutkovs commented Feb 24, 2026

Ensure that reconciliation doesn't change object resources when no changes to the spec applied

Fixes #1799


Summary by cubic

Added e2e idempotency tests to ensure reconciliation doesn’t change Kubernetes resources when the spec hasn’t changed. Covers VMAgent, VMAlert, VMCluster (VMStorage/VMSelect StatefulSets and VMInsert Deployment), and VMSingle; asserts ResourceVersion stays the same across repeated CreateOrUpdate calls. Fixes #1799.

Written for commit 5f38cda. Summary will update on new commits.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 4 files

@vrutkovs vrutkovs force-pushed the e2e-indempotent-reconciliation branch from 0fc1ce3 to ab840d1 Compare February 24, 2026 13:07
Ensure that reconciliation doesn't change object resources when no changes to the spec applied
@vrutkovs vrutkovs force-pushed the e2e-indempotent-reconciliation branch from ab840d1 to 5f38cda Compare February 24, 2026 15:25
@AndrewChubatiuk AndrewChubatiuk merged commit 7759501 into master Feb 24, 2026
6 checks passed
@AndrewChubatiuk AndrewChubatiuk deleted the e2e-indempotent-reconciliation branch February 24, 2026 17:05
vrutkovs added a commit that referenced this pull request Mar 3, 2026
Ensure that reconciliation doesn't change object resources when no changes to the spec applied
AndrewChubatiuk pushed a commit that referenced this pull request Mar 3, 2026
Ensure that reconciliation doesn't change object resources when no changes to the spec applied
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tests: add tests verifying that reconcile doesn't change resource without spec changes

2 participants