search

cybozu-go / coil

CNI plugin for Kubernetes designed for scalability and extensibility
Apache License 2.0
158 stars 18 forks source link

Fix Egress NAT Deployment does not rollout restart #225

Closed terassyi closed 1 year ago

terassyi commented 1 year ago

I changed that taking over annotations when reconciling deployments to complete rollout restart. When Egress NAT executes rollout restart, kubectl adds the annotation kubectl.kubernetes.io/restartedAt to PodTemplateSpec. However coil egress controller initialized all annotations when reconciling deployments. Therefore I fixed as described above.

terassyi commented 1 year ago

I fixed not to use tmpMap and to take over depl.Spec.Template.Annotations as target.Annotations. And when this map is nil, initialize new map.