Skip to content

fix: update vmuser MCS propagation when spec changes#963

Merged
AndrejsPon00 merged 4 commits into
mainfrom
fix-vmuser-mcs-upgrade
Apr 27, 2026
Merged

fix: update vmuser MCS propagation when spec changes#963
AndrejsPon00 merged 4 commits into
mainfrom
fix-vmuser-mcs-upgrade

Conversation

@AndrejsPon00
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings April 24, 2026 10:48
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the VMUser controller’s MultiClusterService (MCS) propagation logic so that when the desired propagation spec changes, the existing MCS is updated instead of being left stale.

Changes:

  • Switch VMUser creation flow to createOrUpdatePropagationMCS when MCSConfig is set.
  • Add logic to fetch an existing propagation MultiClusterService and update its Spec when it differs.
  • Add event recording for propagation MCS update success/failure.

Comment thread kof-operator/internal/controller/vmuser/victoriametrics_user.go Outdated
Comment thread kof-operator/internal/controller/vmuser/victoriametrics_user.go
@AndrejsPon00 AndrejsPon00 marked this pull request as ready for review April 24, 2026 12:07
@AndrejsPon00 AndrejsPon00 force-pushed the fix-vmuser-mcs-upgrade branch from 1e70d01 to 2d7a765 Compare April 24, 2026 14:14
denis-ryzhkov
denis-ryzhkov previously approved these changes Apr 24, 2026
Copy link
Copy Markdown
Collaborator

@gmlexx gmlexx left a comment

Choose a reason for hiding this comment

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

Please cover the fix with unit test

@AndrejsPon00 AndrejsPon00 merged commit 6744099 into main Apr 27, 2026
20 of 21 checks passed
@AndrejsPon00 AndrejsPon00 deleted the fix-vmuser-mcs-upgrade branch April 27, 2026 09:13
@github-project-automation github-project-automation Bot moved this to Done in k0rdent Apr 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants