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

[contoller] Add a possibility to edit a LocalStorageClass spec.lvm.LVMVolumeGroups to configure an appropriate Storage Class. #17

Merged
merged 1 commit into from
Mar 7, 2024

Conversation

ViktorKram
Copy link
Contributor

Description

Add a possibility to edit a LocalStorageClass spec.lvm.LVMVolumeGroups resources to configure an appropriate Storage Class.
Now if the resource validation fails, the controller will add a request to requeue until the resource is fixed or deleted.

Why do we need it, and what problem does it solve?

A user has a possibility to change LVMVolumeGroups using by a Storage Class.
If a user fixes the LocalStorageClass validation failure, the controller will continue the work automatically.

What is the expected result?

A LocalStorageClass spec.lvm.LVMVolumeGroups editing provides an appropriate Storage Class recreation with actual LVMVolumeGroups.
If the validation of the Local Storage Class resource fails, the controller will retry processing the resource until it succeeds or the problematic resource is deleted.

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

@ViktorKram ViktorKram added the enhancement New feature or request label Mar 6, 2024
@ViktorKram ViktorKram requested review from dxavx and AleksZimin March 6, 2024 12:25
@ViktorKram ViktorKram self-assigned this Mar 6, 2024
@ViktorKram ViktorKram force-pushed the v-kramarenko-adds-new-logic-to-lsc branch from d14a68a to 2ea1362 Compare March 7, 2024 14:17
… corresponding Storage Class

Signed-off-by: Viktor Kramarenko <viktor.kramarenko@flant.com>
@ViktorKram ViktorKram force-pushed the v-kramarenko-adds-new-logic-to-lsc branch from 2ea1362 to 5a3c4e2 Compare March 7, 2024 14:43
@AleksZimin AleksZimin merged commit 3124b2f into main Mar 7, 2024
3 checks passed
@AleksZimin AleksZimin deleted the v-kramarenko-adds-new-logic-to-lsc branch March 7, 2024 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants