Skip to content

operator; align labels/annotations with deployment templates #561

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

Merged
merged 1 commit into from
Apr 11, 2025

Conversation

zolug
Copy link
Collaborator

@zolug zolug commented Apr 10, 2025

Description

Operator updates metadata labels and annotations (including spec.template) for proxy daemonset, stateless-lb deployment, nsp statefulset, and ipam statefulset workloads according to their deployment templates.

While for nsp service and ipam service their metadata labels and annotations are updated.

The update merges the existing labels and annotations with the ones found in the deployment templates so that values from templates overwrite existing entries. Labels and annotations not found in templates are left intact.

Note:
With the changes introduced by this PR, in case spec.template labels or annotations have to be aligned in accordance with the new deployment templates, then it will result in workload update even if there were no other changes.

Issue link

NA

Checklist

  • Purpose
    • Bug fix
    • New functionality
    • Documentation
    • Refactoring
    • CI
  • Test
    • Unit test
    • E2E Test
    • Tested manually
  • Introduce a breaking change
    • Yes (description required)
    • No

Operator updates metadata labels and annotations (including spec.template)
for proxy daemonset, stateless-lb deployment, nsp statefulset, and ipam
statefulset workloads according to their deployment templates.

While for nsp service and ipam service their metadata labels and annotations
are updated.

The update merges the existing labels and annotations with the ones found in
the deployment templates so that values from templates overwrite
existing entries. Labels and annotations not found in templates are left intact.
@zolug zolug force-pushed the operator-update-labels-annotations branch from 0257a58 to e1bbd25 Compare April 10, 2025 13:27
@zolug zolug merged commit 97d139e into master Apr 11, 2025
13 checks passed
@zolug zolug moved this to ✅ Done in Meridio Apr 11, 2025
@zolug zolug deleted the operator-update-labels-annotations branch April 11, 2025 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant