cloudfoundry-incubator/quarks-operator

qsts controller doesn't update sts volumeClaimTemplates

f0rmiga opened this issue · 4 comments

f0rmiga commented

Describe the bug

KubeCF recently changed the backing storage for the Diego ephemeral data from PVCs to emptyDir, which uses the nodes' local storage. Practically, when we do an upgrade, the controller should get rid of the spec.volumeClaimTemplates of the statefulset.

There are other implications in KubeCF while doing this, as I understand, the statefulset needs to be recreated, ultimately causing app downtime. This is not a concern of quarks-operator as it stands.

To Reproduce

Switch from PVC-backed ephemeral storage to the emptyDir-backed one, check that the pod respects the statefulset, which disrespects the quarks-statefulset.

Expected behavior

Even if it's not desirable (in the Diego cell's case - causing app downtime), it should do what the spec says.

Environment

quarks-operator version 6.1.8+0.g2821133a.

cf-gitbot commented

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/175135568

The labels on this github issue will be updated when the story is started.

manno commented

@viovanov couldn't reproduce this. Maybe it was fixed by another bug fix.

f0rmiga commented

Reopening as this was not fixed. Ping @mook-as as he's also seeing the same behaviour.

cf-gitbot commented

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/175194087

The labels on this github issue will be updated when the story is started.