Skip to content

Adds shared-filesystem feature documentation#99

Merged
hemanthnakkina merged 1 commit into
canonical:mainfrom
claudiubelu:adds-shared-filesystem
Oct 23, 2025
Merged

Adds shared-filesystem feature documentation#99
hemanthnakkina merged 1 commit into
canonical:mainfrom
claudiubelu:adds-shared-filesystem

Conversation

@claudiubelu
Copy link
Copy Markdown
Contributor

@claudiubelu claudiubelu commented Aug 18, 2025

The shared-filesystem feature enables the Manila OpenStack service, allowing users to create Ceph NFS shares.

@claudiubelu claudiubelu force-pushed the adds-shared-filesystem branch 5 times, most recently from fd555f7 to 2c561e3 Compare August 30, 2025 09:27
@claudiubelu claudiubelu force-pushed the adds-shared-filesystem branch from 2c561e3 to 51e0c54 Compare September 3, 2025 07:48
@claudiubelu claudiubelu force-pushed the adds-shared-filesystem branch 2 times, most recently from 9068951 to 4b7b7d3 Compare September 25, 2025 05:49
Comment thread how-to/features/shared-filesystem.rst Outdated
Comment thread how-to/features/shared-filesystem.rst
+--------------------------------------+------------------------------------------------------------------------------------------------------------+-----------+
| ID | Path | Preferred |
+--------------------------------------+------------------------------------------------------------------------------------------------------------+-----------+
| 96f2ae5a-7fdf-4457-a65d-6b8579635dd0 | 192.168.137.10:/volumes/_nogroup/5692f246-d3d0-4567-81a6-3f590d1957a4/aa0c7383-b5e1-48ab-984c-15b6219c48e7 | True |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is the share exposed on public space?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, the ganesha service's bind address is taken from the peers relation's public-address, which is set to model.get_binding(binding_key="public").network.bind_address.

@claudiubelu claudiubelu force-pushed the adds-shared-filesystem branch from 7dfa0de to edb0a70 Compare October 8, 2025 09:39
Copy link
Copy Markdown
Contributor

@hemanthnakkina hemanthnakkina left a comment

Choose a reason for hiding this comment

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

@wolsen Can you take a look at the doc PR, thanks!

@claudiubelu claudiubelu force-pushed the adds-shared-filesystem branch from edb0a70 to b6678ff Compare October 9, 2025 19:58
Comment thread how-to/features/shared-filesystem.rst Outdated
Comment on lines +19 to +27
The following commands would have to be run on the storage nodes:

::

sudo microceph.ceph mgr module enable microceph
sudo microceph.ceph orch set backend microceph

The openstack CLI can now be used to create and manage CephFS NFS Shared
Filesystems. See the upstream `Manila CLI`_ documentation for details.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This feels like a miss that the user has to enable the mgr module and enable the orchestrator backend manually. I don't think this should be required.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@UtkarshBhatthere is working on canonical/microceph#611 and until the PR is merged and published to squid/stable, the manual steps are required.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Indeed. This would be addressed by this PR: canonical/microceph#611 . So, once that would get merged and promoted, that can be removed from here.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

From @UtkarshBhatthere comment, this should not be needed, as this should be performed by the charm automatically when the NFS relation is present?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Being resolved by: canonical/charm-microceph#209

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Lets wait for PR to get merged and based on microceph promotion cycle lets see if this can be released to squid/beta or squid/candidate by end of pulse and add a note accordingly before merging this PR.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@claudiubelu The fix is available on charm microceph squid/beta, can you add a note about it

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

@claudiubelu claudiubelu force-pushed the adds-shared-filesystem branch 2 times, most recently from bc8fd12 to 9193709 Compare October 15, 2025 13:33
The shared-filesystem feature enables the Manila OpenStack
service, allowing users to create Ceph NFS shares.

Signed-off-by: Claudiu Belu <cbelu@cloudbasesolutions.com>
@claudiubelu claudiubelu force-pushed the adds-shared-filesystem branch from 9193709 to 0297dfb Compare October 23, 2025 08:57
Copy link
Copy Markdown
Contributor

@hemanthnakkina hemanthnakkina left a comment

Choose a reason for hiding this comment

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

The Note on charm microceph squid/beta should be removed once canonical/charm-microceph#209 is released to squid/stable

@hemanthnakkina hemanthnakkina merged commit efd677a into canonical:main Oct 23, 2025
2 checks passed
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.

4 participants