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

health-checker should get started during NodeGetVolumeStats call if not running yet #4219

Closed
nixpanic opened this issue Oct 30, 2023 · 0 comments
Assignees
Labels
component/cephfs Issues related to CephFS

Comments

@nixpanic
Copy link
Member

As mentioned in #4200, the health-checker may not be running when NodeGetVolumeStats gets called. It is possible that the CephFS-nodeplugin was restarted (updated) while a volume was already mounted and used by an application.

A difficulty here is that the NodeGetVolumeStats call only passes the publish-path for the volume, and the health-checker is using the staging-path (to prevent running multiple checkers for the same volume).

@nixpanic nixpanic added the component/cephfs Issues related to CephFS label Oct 30, 2023
nixpanic added a commit to nixpanic/ceph-csi that referenced this issue Nov 2, 2023
The HealthChecker is configured to use the Staging path pf the volume,
with a `.csi/` subdirectory. In the future this directory could be a
directory that is not under the Published directory.

Fixes: ceph#4219
Signed-off-by: Niels de Vos <[email protected]>
@nixpanic nixpanic self-assigned this Nov 2, 2023
nixpanic added a commit to nixpanic/ceph-csi that referenced this issue Nov 3, 2023
The HealthChecker is configured to use the Staging path pf the volume,
with a `.csi/` subdirectory. In the future this directory could be a
directory that is not under the Published directory.

Fixes: ceph#4219
Signed-off-by: Niels de Vos <[email protected]>
nixpanic added a commit to nixpanic/ceph-csi that referenced this issue Nov 3, 2023
The HealthChecker is configured to use the Staging path pf the volume,
with a `.csi/` subdirectory. In the future this directory could be a
directory that is not under the Published directory.

Fixes: ceph#4219
Signed-off-by: Niels de Vos <[email protected]>
@mergify mergify bot closed this as completed in 7d96caf Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/cephfs Issues related to CephFS
Projects
None yet
Development

No branches or pull requests

1 participant