Skip to content

[rhoso-gitops] Docs overhaul: diagrams, app table, cluster sizing, Secret zero - #716

Merged
gaurav-nelson merged 1 commit into
validatedpatterns:mainfrom
cjeanner:rhoso-gitops/update-diagrams
Aug 4, 2026
Merged

[rhoso-gitops] Docs overhaul: diagrams, app table, cluster sizing, Secret zero#716
gaurav-nelson merged 1 commit into
validatedpatterns:mainfrom
cjeanner:rhoso-gitops/update-diagrams

Conversation

@cjeanner

@cjeanner cjeanner commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Update diagrams — replace SVG with PNG for reliable rendering across all browsers
  • Deduplicate app tables — move the full applications table (with RHOSO doc links, enabled/sync columns) to the Configuration page; keep a compact name+purpose table on the Architecture page
  • Baremetal requirement — replace the cluster sizing intro with the shared cluster-sizing-template include and clarify that RHOSO requires baremetal (no cloud instances)
  • Remove "Next steps" footers — they conflicted with the built-in Previous/Next navigation links
  • Add warning about example overlays — note in Configuration that upstream example/ paths are reference samples only, with direct links to the GitHub directories
  • Correct Secret zero workflow — rewrite bootstrap credential steps: create namespace first, inject the secret out of band, then wire up Kustomize overlay and enable openstack-secrets

Files changed (13)

  • 5 content pages (_index, cluster-sizing, configuration, getting-started, troubleshooting)
  • 3 modules (rhoso-gitops-about, rhoso-gitops-architecture, rhoso-gitops-configuration, rhoso-gitops-deploying)
  • 2 new PNG diagrams, 2 removed SVGs

@openshift-ci

openshift-ci Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Hi @cjeanner. Thanks for your PR.

I'm waiting for a validatedpatterns member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-ci openshift-ci Bot added the size/XL label Jul 31, 2026
@cjeanner
cjeanner force-pushed the rhoso-gitops/update-diagrams branch from f8b3cc1 to 19a086f Compare August 3, 2026 08:24
@cjeanner
cjeanner marked this pull request as ready for review August 3, 2026 08:24
@openshift-ci
openshift-ci Bot requested review from beekhof and gaurav-nelson August 3, 2026 08:25
@gaurav-nelson

Copy link
Copy Markdown
Collaborator

/ok-to-test

@ocpdocs-previewbot

ocpdocs-previewbot commented Aug 4, 2026

Copy link
Copy Markdown

🤖 Tue Aug 04 01:10:38 - The preview is ready at:
https://716--patternsdocs-pr.netlify.app

@gaurav-nelson
gaurav-nelson force-pushed the rhoso-gitops/update-diagrams branch from 19a086f to e02c878 Compare August 4, 2026 01:08

@gaurav-nelson gaurav-nelson left a comment

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.

LGTM 🚀

@openshift-ci openshift-ci Bot added the lgtm label Aug 4, 2026
@gaurav-nelson
gaurav-nelson merged commit b2e0a2a into validatedpatterns:main Aug 4, 2026
2 checks passed
@cjeanner
cjeanner deleted the rhoso-gitops/update-diagrams branch August 4, 2026 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants