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

ceph_volume: support encrypted volumes for lvm new-db/new-wal/migrate commands #50427

Merged
merged 6 commits into from Aug 8, 2023

Conversation

ifed01
Copy link
Contributor

@ifed01 ifed01 commented Mar 7, 2023

@ifed01 ifed01 requested review from a team as code owners March 7, 2023 18:09
@ifed01 ifed01 force-pushed the wip-ifed-encrypted-ceph-volume branch 3 times, most recently from e258642 to 77b4c86 Compare March 7, 2023 18:18
@ifed01 ifed01 requested a review from guits March 7, 2023 18:20
@ifed01 ifed01 force-pushed the wip-ifed-encrypted-ceph-volume branch from 77b4c86 to 4ca126e Compare March 9, 2023 13:32
@ifed01 ifed01 changed the title ceph_volume: support encrypted volumes for lvm new-db/new-wal commands ceph_volume: support encrypted volumes for lvm new-db/new-wal/migrate commands Mar 9, 2023
@ifed01 ifed01 force-pushed the wip-ifed-encrypted-ceph-volume branch 2 times, most recently from 744b52e to 5510f64 Compare March 9, 2023 15:08
@ifed01
Copy link
Contributor Author

ifed01 commented Apr 5, 2023

@guits - ping.

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
Fixes: https://tracker.ceph.com/issues/58943

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
Just a tiny cleanup.

Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
@guits
Copy link
Contributor

guits commented Jul 27, 2023

some new comments, can be addressed in a separate PR though (there are existing tests that have already the same @patch() approach)

@guits guits removed the request for review from a team July 31, 2023 11:26
Signed-off-by: Igor Fedotov <igor.fedotov@croit.io>
@ifed01
Copy link
Contributor Author

ifed01 commented Aug 7, 2023

jenkins test make check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants