Skip to content

fix: stabilize Traefik reconciliation and instance identity - #24

Open
StefanSa wants to merge 3 commits into
fosrl:mainfrom
StefanSa:fix/traefik-instance-label-ssa
Open

fix: stabilize Traefik reconciliation and instance identity#24
StefanSa wants to merge 3 commits into
fosrl:mainfrom
StefanSa:fix/traefik-instance-label-ssa

Conversation

@StefanSa

@StefanSa StefanSa commented Sep 2, 2026

Copy link
Copy Markdown

Summary

  • reconcile external HTTP load-balancer targets through generated Kubernetes Services and EndpointSlices instead of unsupported TraefikService loadBalancer fields
  • use server-side apply from initial creation with the stable pangolin-kube-controller field manager
  • safely adopt legacy controller Update spec ownership only for controller-managed resources
  • propagate configured Traefik instance identity through generic and specialized IngressRoute, TCP, and UDP paths
  • preserve unrelated metadata and repair missing controller-owned identity in place

Production compatibility

This complete set is required for current Traefik CRD schemas: external HTTP targets must be converted before SSA, while generated Traefik resources retain stable ownership and the configured production instance identity.

Validation

  • go test ./...
  • go vet ./...
  • production validation covers controller readiness, clean reconciliation, external HTTP targets, SSA ownership, instance-label persistence, CrowdSec, Bitwarden, Forgejo, and codex-lb

@StefanSa StefanSa changed the title fix: preserve Traefik instance labels during apply fix: stabilize SSA ownership and Traefik instance labels Sep 3, 2026
@StefanSa StefanSa changed the title fix: stabilize SSA ownership and Traefik instance labels fix: stabilize Traefik reconciliation and instance identity Sep 3, 2026
@sonarqubecloud

sonarqubecloud Bot commented Sep 3, 2026

Copy link
Copy Markdown

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.

1 participant