Skip to content

HDDS-14536. Storage Containers shown in menu as storage-containers#317

Merged
sarvekshayr merged 4 commits intoapache:HDDS-9225-website-v2from
Russole:HDDS-14536
Feb 4, 2026
Merged

HDDS-14536. Storage Containers shown in menu as storage-containers#317
sarvekshayr merged 4 commits intoapache:HDDS-9225-website-v2from
Russole:HDDS-14536

Conversation

@Russole
Copy link
Copy Markdown
Contributor

@Russole Russole commented Feb 1, 2026

What changes were proposed in this pull request?

https://ozone-site-v2.staged.apache.org/docs/core-concepts/replication/storage-containers
Core Concepts > Replication > storage-containers
Shown in the menu and breadcrumbs as "storage-containers" instead of "Storage Containers".

What is the link to the Apache Jira?

https://issues.apache.org/jira/browse/HDDS-14536

How was this patch tested?

The page was rendered locally to verify that it displays correctly.

Copy link
Copy Markdown
Contributor

@echonesis echonesis left a comment

Choose a reason for hiding this comment

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

Thanks @Russole for the patch.
Overall looks good. Just one minor nit inline about Apache license header removal.

Other than that, LGTM!

Comment on lines 5 to 20
@@ -14,6 +18,7 @@
See the License for the specific language governing permissions and
limitations under the License.
-->
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.

According to HDDS-14419, we could remove this Apache license header.

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.

+1

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.

@Russole The entire license section needs to be removed.

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.

@sarvekshayr, The entire license section has been completely removed.

Comment on lines +41 to +42
| mutable | immutable |
| replicated with RAFT (Ratis) | Replicated with async container copy |
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.

Use capitalisation here.

Suggested change
| mutable | immutable |
| replicated with RAFT (Ratis) | Replicated with async container copy |
| Mutable | Immutable |
| Replicated with RAFT (Ratis) | Replicated with async container copy |

@Russole
Copy link
Copy Markdown
Contributor Author

Russole commented Feb 2, 2026

Thanks @echonesis, @sarvekshayr for the review. I’ve updated the patch accordingly.

@sarvekshayr
Copy link
Copy Markdown
Contributor

sarvekshayr commented Feb 3, 2026

#322 has fixed the CI failure seen here. Please merge the latest changes to get a green CI.

Copy link
Copy Markdown
Contributor

@echonesis echonesis left a comment

Choose a reason for hiding this comment

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

Thanks @Russole for the update.
LGTM!

@sarvekshayr sarvekshayr merged commit 8bd2e51 into apache:HDDS-9225-website-v2 Feb 4, 2026
11 checks passed
@sarvekshayr
Copy link
Copy Markdown
Contributor

Thanks @Russole for the patch and @echonesis for the review,

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