Skip to content

Avoid duplicate kube-controllers tolerations#4746

Merged
caseydavenport merged 1 commit intotigera:masterfrom
officialasishkumar:issue-3783-dedupe-kube-controller-tolerations
Apr 23, 2026
Merged

Avoid duplicate kube-controllers tolerations#4746
caseydavenport merged 1 commit intotigera:masterfrom
officialasishkumar:issue-3783-dedupe-kube-controller-tolerations

Conversation

@officialasishkumar
Copy link
Copy Markdown
Contributor

Description

Bug fix for #3783. The calico-kube-controllers renderer now appends the operator's default tolerations only when they are not already present in Installation.spec.controlPlaneTolerations. This prevents duplicate CriticalAddonsOnly tolerations while preserving the default control-plane tolerations and GKE Arm64 toleration behavior.

Testing done:

  • go test ./pkg/render/kubecontrollers

Components affected:

  • calico-kube-controllers rendering

Closes #3783

Release Note

Avoid rendering duplicate tolerations on calico-kube-controllers when matching default tolerations are already configured through Installation controlPlaneTolerations.

For PR author

  • Tests for change.
  • If changing pkg/apis/, run make gen-files
  • If changing versions, run make gen-versions

For PR reviewers

A note for code reviewers - all pull requests must have the following:

  • Milestone set according to targeted release.
  • Appropriate labels:
    • kind/bug if this is a bugfix.
    • kind/enhancement if this is a a new feature.
    • enterprise if this PR applies to Calico Enterprise only.

When rendering calico-kube-controllers, append the operator's default tolerations only if they are not already present in Installation controlPlaneTolerations. This prevents duplicate CriticalAddonsOnly tolerations while preserving the default tolerations that kube-controllers needs.\n\nAdds a render test that covers the duplicate CriticalAddonsOnly case reported in tigera#3783.

Signed-off-by: Asish Kumar <officialasishkumar@gmail.com>
@caseydavenport
Copy link
Copy Markdown
Member

/sem-approve

@caseydavenport caseydavenport merged commit 48b4f54 into tigera:master Apr 23, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Duplicate tolerations in kube-controllers deployment

3 participants