Skip to content

docs(rhoso-apps): expand readme for vault secret zero#51

Open
cjeanner wants to merge 2 commits into
mainfrom
docs/rhoso-apps-secret-zero
Open

docs(rhoso-apps): expand readme for vault secret zero#51
cjeanner wants to merge 2 commits into
mainfrom
docs/rhoso-apps-secret-zero

Conversation

@cjeanner
Copy link
Copy Markdown
Collaborator

Add a TOC, reorganize sections, and document the out-of-Git bootstrap credential (secret zero) for Vault or ESO on OpenShift. Clarify openstack-secrets defaults, wrap prose for readability, use reference-style links where helpful, and pin the in-repo VSO example URL to v0.2.0.

AI-Assist: Cursor; model=Composer-2; mode=agent; origin=cursor

@cjeanner cjeanner self-assigned this May 13, 2026
Add a TOC, reorganize sections, and document the out-of-Git bootstrap credential (secret zero) for Vault or ESO on OpenShift. Clarify openstack-secrets defaults, wrap prose for readability, use reference-style links where helpful, and pin the in-repo VSO example URL to v0.2.0.

AI-Assist: Cursor; model=Composer-2; mode=agent; origin=cursor
@cjeanner cjeanner force-pushed the docs/rhoso-apps-secret-zero branch from 21d9d2f to 9dbad1f Compare May 13, 2026 12:25
@pinikomarov
Copy link
Copy Markdown
Contributor

Line 387 - Patch operation type:

  • op: replace
    path: /spec/appRole/secretRef
    value: "your-vault-approle-secret-name"
    ❌ Problem: replace fails if /spec/appRole/secretRef doesn't already exist in the base manifest.

✅ Suggested change to add:

  • op: add
    path: /spec/appRole/secretRef
    value: "your-vault-approle-secret-name"

@cjeanner
Copy link
Copy Markdown
Collaborator Author

cjeanner commented Jun 2, 2026

Line 387 - Patch operation type:

* op: replace
  path: /spec/appRole/secretRef
  value: "your-vault-approle-secret-name"
  ❌ Problem: replace fails if /spec/appRole/secretRef doesn't already exist in the base manifest.

✅ Suggested change to add:

* op: add
  path: /spec/appRole/secretRef
  value: "your-vault-approle-secret-name"

Ah, interesting, the add operation is actually more an add or replace. While this is a "dumb" example not really made to copy-paste as-is, I'll update it, since it won't break anything.

`add` acts more as an "add or replace", leading to a closer working code
example - even if that example isn't meant to be copy-pasted as-is
(placeholders, etc).
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.

2 participants