Skip to content

[rhoso-gitops] singleArgoCD: true by default - #725

Draft
cjeanner wants to merge 2 commits into
validatedpatterns:mainfrom
cjeanner:rhoso-gitops/singleArgoCD
Draft

[rhoso-gitops] singleArgoCD: true by default#725
cjeanner wants to merge 2 commits into
validatedpatterns:mainfrom
cjeanner:rhoso-gitops/singleArgoCD

Conversation

@cjeanner

Copy link
Copy Markdown
Contributor
  • Update all rhoso-gitops documentation to reflect singleArgoCD: true as the default: replace every rhoso-gitops-standalone namespace reference with vp-gitops, rewrite the "Dual Argo CD namespaces" architecture section to
    describe a single namespace, remove duplicate oc get applications verification steps, and update the values-global.yaml table row to mention singleArgoCD mode and custom health checks

  • Apply Red Hat/IBM style guide fixes: replace "via" with "by using", use {rh-ocp} attribute instead of hard-coded "OpenShift", remove redundant paragraph, split a 34-word sentence, add missing short description to the
    cluster-sizing page, add cross-reference link for the new overrides configuration step, and use "about" instead of "on" in link intros

Depends-on: validatedpatterns-sandbox/rhoso-gitops#30

cjeanner and others added 2 commits August 13, 2026 14:03
The pattern now sets global.singleArgoCD: true by default. Update all
documentation to reflect a single vp-gitops namespace instead of the
former dual-namespace layout (vp-gitops + rhoso-gitops-standalone).
Update the values-global.yaml table row to mention singleArgoCD mode
and custom ArgoCD health checks.

Depends-On: validatedpatterns-sandbox/rhoso-gitops#30

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Apply Red Hat/IBM style guide fixes: replace "via" with "by using",
use product name attributes instead of hard-coded strings, remove
redundant paragraph, split long sentence, add short description to
cluster-sizing page, add cross-reference for overrides step, and
use "about" instead of "on" in link intros.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@openshift-ci

openshift-ci Bot commented Aug 13, 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.

Tip

We noticed you've done this a few times! Consider joining the org to skip this step and gain /lgtm and other bot rights. We recommend asking approvers on your previous PRs to sponsor you.

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/M label Aug 13, 2026

@pinikomarov pinikomarov left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

lgtm

@pinikomarov

Copy link
Copy Markdown

/ok-to-test

@openshift-ci

openshift-ci Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

@pinikomarov: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test message.

Details

In response to this:

/ok-to-test

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.

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.

2 participants