Skip to content

Consolidate charm-microceph with MicroCeph docs#707

Open
skoech wants to merge 13 commits intomainfrom
consolidate-charm-microceph
Open

Consolidate charm-microceph with MicroCeph docs#707
skoech wants to merge 13 commits intomainfrom
consolidate-charm-microceph

Conversation

@skoech
Copy link
Copy Markdown
Collaborator

@skoech skoech commented Apr 3, 2026

Description

This PR consolidates charm-microceph docs with the MicroCeph docs in this repository, inspired by the style in the Canonical Kubernetes docs, like this:

  1. Creates two sub-directories, under microceph/docs/, and name then snap/ and charm/.
  • This is where the actual content for MicroCeph and charm-microceph, respectively, will live.
  1. Each section will have its own index.rst file, with a hidden toctree.
  2. The navigation structure is defined in this index.rst’s toctree, like this:
.. toctree::
   :hidden:
   :titlesonly:

   Deploy from Snap package <snap/index>
   Deploy with Juju <charm/index>
  1. Then, there are be Diataxis components under each sub-directory. For now, we’ll only have the tutorial and how-to components in the charm sub-directory, because that is the only documentation we have on the repository.

Type of change

  • Documentation update (change to documentation only)

How has this been tested?

I have built and served the changes locally, and verified that the documentation builds and renders as expected:

image

Contributor checklist

  • 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

@skoech skoech force-pushed the consolidate-charm-microceph branch from 5333b07 to ac9c8d8 Compare April 9, 2026 11:21
Comment thread docs/charm/how-to/index.rst Outdated
=============

Our how-to guides give directions on how to perform key operations and processes
in charm-microCeph.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
in charm-microCeph.
in charm-microceph.

Comment thread docs/charm/how-to/index.rst Outdated
Our how-to guides give directions on how to perform key operations and processes
in charm-microCeph.

Install and deploy charm-microCeph
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
Install and deploy charm-microCeph
Install and deploy charm-microceph

Comment thread docs/charm/how-to/index.rst Outdated
Install and deploy charm-microCeph
----------------------------------

Deploy a charm-microCeph cluster with Juju.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
Deploy a charm-microCeph cluster with Juju.
Deploy a charm-microceph cluster with Juju.

@johnramsden
Copy link
Copy Markdown
Member

Regarding versioning, I think the way that would mirror how we would do something if it were a software release would probably be a git tag that corresponds to a release - https://docs.readthedocs.com/platform/latest/versions.html.

One complexity here is this might tie the two documentation types together (c-mc & mc would be on one tag)

And then other than that the idea seems good to me. Looks like some work still needs to be done on syntax and the rst so I won't comment there

@skoech skoech changed the title Consolidate charm-microceph with MicroCeph docs - Approach 1 Consolidate charm-microceph with MicroCeph docs Apr 12, 2026
skoech added 3 commits April 15, 2026 09:59
Signed-off-by: Sharon Koech <sharon.koech@canonical.com>
Signed-off-by: Sharon Koech <sharon.koech@canonical.com>
Signed-off-by: Sharon Koech <sharon.koech@canonical.com>
@skoech skoech force-pushed the consolidate-charm-microceph branch from ac9c8d8 to 4256556 Compare April 15, 2026 07:18
@skoech
Copy link
Copy Markdown
Collaborator Author

skoech commented Apr 15, 2026

Good catch on charm-microceph naming, thanks @johnramsden. I've done away with that and now sticking to just MicroCeph as the product name(with different ways of deploying).
As for versioning, using git tags corresponding to releases seems like the way to go. I think it's OK to have the same tag for both mc & c-mc since the risk channels correspond to each other, and we can just have a stable version for both of them. But I'm happy to discuss about this some more.

skoech added 8 commits April 15, 2026 19:30
Signed-off-by: Sharon Koech <sharon.koech@canonical.com>
Signed-off-by: Sharon Koech <sharon.koech@canonical.com>
Signed-off-by: Sharon Koech <sharon.koech@canonical.com>
Signed-off-by: Sharon Koech <sharon.koech@canonical.com>
Signed-off-by: Sharon Koech <sharon.koech@canonical.com>
Signed-off-by: Sharon Koech <sharon.koech@canonical.com>
Signed-off-by: Sharon Koech <sharon.koech@canonical.com>
Signed-off-by: Sharon Koech <sharon.koech@canonical.com>
@skoech skoech marked this pull request as ready for review April 17, 2026 19:42
@skoech skoech requested a review from sabaini April 17, 2026 19:43
Copy link
Copy Markdown
Collaborator

@sabaini sabaini left a comment

Choose a reason for hiding this comment

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

Hey @skoech in general I'm +1 on the plan, one thing I wonder about is committing the charm-microceph docs into this repo though.

Aiui it's still the plan that the canonical source of truth for the charm-microceph docs still is the charm-microceph repo, is that correct? If that is the case I'd prefer not to duplicate them into this repo but download / pull them on demand only, i.e. when docs are built. Otherwise we'd risk overwriting our own changes. Hope I'm not missing sth

Signed-off-by: Sharon Koech <sharon.koech@canonical.com>
@skoech
Copy link
Copy Markdown
Collaborator Author

skoech commented Apr 21, 2026

@sabaini, you're right. I've removed them and modified the script to fetch them from the charm-microceph repo at RTD build time.

Signed-off-by: Sharon Koech <sharon.koech@canonical.com>
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.

3 participants