Skip to content

docs: Updates documentation to include information about enabling NFS#583

Merged
UtkarshBhatthere merged 1 commit intocanonical:mainfrom
claudiubelu:adds-nfs-docs
Aug 28, 2025
Merged

docs: Updates documentation to include information about enabling NFS#583
UtkarshBhatthere merged 1 commit intocanonical:mainfrom
claudiubelu:adds-nfs-docs

Conversation

@claudiubelu
Copy link
Copy Markdown
Contributor

Description

Adds information about how to enable nfs in MicroCeph.

Depends-On: #574

Type of change

  • Documentation update (change to documentation only)

How has this been tested?

N/A

Contributor checklist

Please check that you have:

  • self-reviewed the code in this PR
  • added code comments, particularly in less straightforward areas
  • checked and added or updated relevant documentation
  • checked and added or updated relevant release notes
  • added tests to verify effectiveness of this change

@claudiubelu claudiubelu mentioned this pull request Jul 16, 2025
5 tasks
@claudiubelu claudiubelu force-pushed the adds-nfs-docs branch 2 times, most recently from 691ffa2 to b168e9b Compare July 16, 2025 12:32
@UtkarshBhatthere UtkarshBhatthere requested a review from skoech July 17, 2025 07:59
@UtkarshBhatthere UtkarshBhatthere added the documentation Improvements or additions to documentation label Jul 17, 2025
@UtkarshBhatthere
Copy link
Copy Markdown
Contributor

@skoech please take a look.

@claudiubelu claudiubelu marked this pull request as draft July 17, 2025 12:14
Comment thread docs/how-to/enable-service-instances.rst Outdated
Comment thread docs/how-to/enable-service-instances.rst Outdated
Copy link
Copy Markdown
Collaborator

@skoech skoech left a comment

Choose a reason for hiding this comment

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

@UtkarshBhatthere, thanks for the great changes. I've left some comments and suggestions for you to consider.

@claudiubelu claudiubelu force-pushed the adds-nfs-docs branch 2 times, most recently from 691ee06 to 8928634 Compare August 14, 2025 07:03
@claudiubelu claudiubelu marked this pull request as ready for review August 14, 2025 07:07
@claudiubelu
Copy link
Copy Markdown
Contributor Author

@UtkarshBhatthere, thanks for the great changes. I've left some comments and suggestions for you to consider.

Done

@UtkarshBhatthere
Copy link
Copy Markdown
Contributor

@skoech ptal ?

Copy link
Copy Markdown
Collaborator

@skoech skoech left a comment

Choose a reason for hiding this comment

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

Signed-off-by: Claudiu Belu <cbelu@cloudbasesolutions.com>
Copy link
Copy Markdown
Collaborator

@skoech skoech left a comment

Choose a reason for hiding this comment

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

Thanks, @claudiubelu!

@claudiubelu
Copy link
Copy Markdown
Contributor Author

@UtkarshBhatthere, anything else needed for this PR?

@UtkarshBhatthere UtkarshBhatthere merged commit 71a7e05 into canonical:main Aug 28, 2025
23 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants