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

Server side defaults not diffing correctly #555

Closed
Dhouti opened this issue May 6, 2019 · 3 comments
Closed

Server side defaults not diffing correctly #555

Dhouti opened this issue May 6, 2019 · 3 comments
Assignees
Labels

Comments

@Dhouti
Copy link

Dhouti commented May 6, 2019

This issue presented itself when switching kubernetes versions.

Kubernetes Server Version: v1.13.5

level=debug msg="spec diff between old and new statefulsets:
Template.Spec.Containers[0].TerminationMessagePath: "/dev/termination-log" != ""
Template.Spec.Containers[0].TerminationMessagePolicy: "File" != ""
Template.Spec.RestartPolicy: "Always" != ""
Template.Spec.DNSPolicy: "ClusterFirst" != ""
Template.Spec.DeprecatedServiceAccount: "operator" != ""
Template.Spec.SecurityContext: &v1.PodSecurityContext{} != nil
Template.Spec.SchedulerName: "default-scheduler" != ""
Template.Spec.Tolerations: []v1.Toleration(nil) != []v1.Toleration{}
VolumeClaimTemplates[0].Spec.VolumeMode: &"Filesystem" != nil
VolumeClaimTemplates[0].Status.Phase: "Pending" != ""
PodManagementPolicy: "OrderedReady" != ""
UpdateStrategy.Type: "OnDelete" != ""
RevisionHistoryLimit: &int32(10) != nil"

The "old" values in question are defaulted by the kubernetes server.
This is causing the statefulset to be recreated every time it is synced.

@knan-nrk
Copy link
Contributor

Also hits us on every sync. Kubernetes version 1.13.3.

`time="2019-05-10T08:14:53Z" level=info msg="statefulset "mediakvern-experimental/mediakvernexperimental-db" is not in the desired state and needs to be updated" cluster-name=mediakvern-experimental/mediakvern-experimental-db pkg=cluster worker=0

time="2019-05-10T08:14:53Z" level=debug msg="spec diff between old and new statefulsets: \nTemplate.Spec.Containers[0].TerminationMessagePath: "/dev/termination-log" != ""\nTemplate.Spec.Containers[0].TerminationMessagePolicy: "File" != ""\nTemplate.Spec.RestartPolicy: "Always" != ""\nTemplate.Spec.DNSPolicy: "ClusterFirst" != ""\nTemplate.Spec.DeprecatedServiceAccount: "zalando-postgres-operator" != ""\nTemplate.Spec.SchedulerName: "default-scheduler" != ""\nTemplate.Spec.Tolerations: []v1.Toleration(nil) != []v1.Toleration{}\nVolumeClaimTemplates[0].Spec.VolumeMode: &"Filesy
stem" != nil\nVolumeClaimTemplates[0].Status.Phase: "Pending" != ""\nRevisionHistoryLimit: &int32(10) != nil\n" cluster-name=mediakvern-experimental/mediakvern-experimental-db pkg=cluster worker=0`

@Jan-M
Copy link
Member

Jan-M commented May 13, 2019

Thanks for reporting this issue and reraising it. We are seeing this in 1.13 test environments too and are looking into the resolution.

@sdudoladov
Copy link
Member

closed by #563

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants