Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(core): hide target pod during migration via cilium label #609

Merged

Conversation

Isteb4k
Copy link
Contributor

@Isteb4k Isteb4k commented Dec 25, 2024

Description

During the VM migration process, two pods with the same address are created and packets are randomly delivered to them.
To force delivery of packages to only one VM pod, the special label network.deckhouse.io/hidden-pod for target pod were added.
When the migration completes, the label is removed and the target pod becomes accessible via network.

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

@Isteb4k Isteb4k self-assigned this Dec 25, 2024
@Isteb4k Isteb4k force-pushed the fix/core/hide-target-pod-during-migration-via-cilium-label branch 11 times, most recently from 9bfd49e to 6d393da Compare December 27, 2024 18:09
@Isteb4k Isteb4k marked this pull request as ready for review December 27, 2024 18:10
@Isteb4k Isteb4k force-pushed the fix/core/hide-target-pod-during-migration-via-cilium-label branch 3 times, most recently from fa6b351 to e67fdcd Compare January 13, 2025 16:25
yaroslavborbat
yaroslavborbat previously approved these changes Jan 14, 2025
@Isteb4k Isteb4k force-pushed the fix/core/hide-target-pod-during-migration-via-cilium-label branch from e67fdcd to 8668a29 Compare January 14, 2025 13:41
@Isteb4k Isteb4k merged commit b185f6e into main Jan 14, 2025
12 checks passed
@Isteb4k Isteb4k deleted the fix/core/hide-target-pod-during-migration-via-cilium-label branch January 14, 2025 13:45
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.

2 participants