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

feat: Document how to deploy with ceph-csi against multiple ceph clusters #1127

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

addyess
Copy link
Contributor

@addyess addyess commented Feb 27, 2025

Feature addition of documenting how to relate to multiple ceph clusters with the ceph-csi charm

@addyess addyess requested a review from a team as a code owner February 27, 2025 19:48
@addyess addyess force-pushed the KU-2648/ceph-mulit-cluster-deployment branch from b46ef67 to 2d726b0 Compare February 27, 2025 19:48
Copy link
Contributor

@nhennigan nhennigan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work 😸 I left a few comments but most of them are small typos or nits. Well done!

@addyess
Copy link
Contributor Author

addyess commented Feb 28, 2025

@nhennigan thanks so much for your very insightful and prompt review

Copy link
Contributor

@nhennigan nhennigan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small typo on Juju vs juju but other than that thanks for implementing the changes I suggested. Looks good 😸

@addyess addyess force-pushed the KU-2648/ceph-mulit-cluster-deployment branch from 7f95ce2 to 799b0bc Compare February 28, 2025 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants