Travis Nielsen
Travis Nielsen
@jhoblitt Can you clarify which destruction and explosions you expected? :) The CSI driver to be deleted? I would expect those pods to all go away when the Rook operator...
@mate4st Setting the per-deviceclass resources currently expects that the main `osd` resources are not set. So if only the deviceclass-specific resources are set (such as `osd-ssd`), then it it is...
Ah, I was missing that you're trying to specify resources both per-deviceclass and per-node. I believe this feature was only tested when using `spec.resources..resources`. The design around this is a...
One thing to check... Do you see the correct labels on the OSD pods for the device class? For example, you should see a label such as `device-class: hdd`. If...
@mate4st Did you see [this question](https://github.com/rook/rook/pull/15427#issuecomment-2715432764) about what `deviceclass` label you are seeing on the OSDs?
From the [last comment](https://github.com/rook/rook/pull/15427#issuecomment-2804469843), it seems the correct device classes are being assigned to the osd pods. @zeelax Do you also see the correct labels? Just to be clear that...
Ok, I was now about to repro this and find a fix with a new PR #16614. @mate4st I added you as a coauthor since you helped raise the issue...
Ok, found a clue... Any panics were being swallowed during a reconcile. After every panic, the reconcile was just restarting again. So the mons reconcile started again and again, always...
> good catch and indeed above fixes the error Great! Any unit tests that could be added to harden up that method?
The upgrade test is succeeding all the tests, but then during test teardown, the csi driver is not running, so it it not able to unmount the volumes and cleanup...