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.

Sign up for GitHub

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

Jump to bottom

Migrate from old module #34

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Migrate from old module #34

wants to merge 10 commits into from

Conversation

duckhawk
Copy link
Member

@duckhawk duckhawk commented Feb 22, 2025 •
edited
Loading

Description

This PR implements hooks to migrate from deprecated internal module ceph-csi to current module

What is the expected result?

Automatic migration from old module

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.

@AleksZimin
fix
d5a24c4 
Signed-off-by: Aleksandr Zimin <alexandr.zimin@flant.com>
@AleksZimin
fix2
f719af3 
Signed-off-by: Aleksandr Zimin <alexandr.zimin@flant.com>
@AleksZimin
fix tests
31a7150 
Signed-off-by: Aleksandr Zimin <alexandr.zimin@flant.com>
@AleksZimin
fix english
c7f87b5 
Signed-off-by: Aleksandr Zimin <alexandr.zimin@flant.com>
@AleksZimin
fix in storageclass and add deprication label to CephClusterAutentuca…
8963fee 
…tion resources

Signed-off-by: Aleksandr Zimin <alexandr.zimin@flant.com>
@AleksZimin
some fixes
7023ad9 
Signed-off-by: Aleksandr Zimin <alexandr.zimin@flant.com>
@AleksZimin
Merge branch 'main' into migrate-to-actual-adr
83cb6a2 
Signed-off-by: Aleksandr Zimin <alexandr.zimin@flant.com>
@duckhawk duckhawk added the enhancement New feature or request label Feb 22, 2025
@duckhawk duckhawk requested review from astef and AleksZimin February 22, 2025 05:52
@duckhawk duckhawk self-assigned this Feb 22, 2025
@duckhawk duckhawk changed the base branch from main to migrate-to-actual-adr February 22, 2025 05:53
@duckhawk duckhawk marked this pull request as ready for review February 22, 2025 15:49
@duckhawk duckhawk force-pushed the migrate-from-old-module branch 6 times, most recently from d3718a0 to d62a322 Compare February 22, 2025 19:41
@duckhawk duckhawk requested a review from astef February 23, 2025 09:20
astef
astef previously approved these changes Feb 23, 2025
View reviewed changes
Base automatically changed from migrate-to-actual-adr to main February 28, 2025 13:11
@AleksZimin AleksZimin dismissed astef’s stale review February 28, 2025 13:11

The base branch was changed.

@duckhawk duckhawk force-pushed the migrate-from-old-module branch 5 times, most recently from d06f961 to 97f556a Compare March 3, 2025 10:25
@duckhawk
Fix
d33a52c 
Signed-off-by: v.oleynikov <vasily.oleynikov@flant.com>
@duckhawk duckhawk force-pushed the migrate-from-old-module branch from 97f556a to d33a52c Compare March 3, 2025 10:28
@duckhawk duckhawk marked this pull request as draft March 3, 2025 17:19
@duckhawk duckhawk requested a review from astef March 3, 2025 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AleksZimin AleksZimin Awaiting requested review from AleksZimin

@astef astef Awaiting requested review from astef

At least 1 approving review is required to merge this pull request.

Assignees

@duckhawk duckhawk

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@duckhawk @astef @AleksZimin