Repository navigation
docs(rhdh): add migration guide warnings, defaults inventory, and examples - #603
Conversation
…mples Add practical guidance that was missing from the migration guide: - Image digest/tag interaction warning: custom tag + chart's default digest can produce an unresolvable image reference - Air-gapped plugin limitation: global.imageRegistry does not apply to dynamic plugin oci:// references - Chart-managed defaults inventory: which extra* entries (volumes, mounts, env vars, init containers) can be safely removed, split into unconditional vs conditional - Before/after YAML examples for ingress, extraEnvFrom, and orchestrator structural transformations Assisted-by: Claude
Code Review by Qodo🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)
Great, no issues found!Qodo reviewed your code and found no material issues that require reviewTip of the day💡 Did you know, you can show, collapse, or hide each part of a finding: code, evidence, and all |
PR Summary by QodoClarify RHDH 1.y-to-2.y migration pitfalls and value mappings
AI Description
Diagram
High-Level Assessment
Files changed (1)
|
|
Important The |
… guide
Users migrating manually also need to update Go template expressions
in string values (e.g., {{ .Values.global.host }} → {{ .Values.host }}).
Add a table of common reference updates and a note about decomposed
fields that cannot be find-and-replaced.
Assisted-by: Claude
|
/cherrypick release-2.1 |
|
@rm3l: once the present PR merges, I will cherry-pick it on top of DetailsIn response to this:
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. |
…tion guide - Add note that mapping tables reflect chart version 2.1 specifically - Clarify that PostgreSQL 15 is still supported, just no longer the default Assisted-by: Claude
|
|
Merging - docs only. |
c1dfa53
into
redhat-developer:main
|
@rm3l: #603 failed to apply on top of branch "release-2.1": DetailsIn response to this:
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. |
…stream migration guide Remove duplicated mapping tables, behavioral changes, removed values, and new features from the skill reference — all now maintained in the upstream rhdh-chart migration guide (redhat-developer/rhdh-chart#603). The skill reference is now a thin delta containing only what the upstream guide does not: ambiguous area transformation algorithms, parent-path .Values.* rewriting entries, and the decomposed fields list. 670 → 332 lines. Co-authored-by: Tomas Kral <tkral@redhat.com> Co-authored-by: Nick Boldt <nboldt@redhat.com> Assisted-by: Claude



Description of the change
Adds practical guidance to the 1.y→2.y migration guide that was missing and surfaced during work on the AI-assisted migration skill (RHIDP-17323):
tag, the chart's defaultdigestis merged by Helm, producing an unresolvabletag@digestimage reference. Documents the workaround (digest: ""or set the correct digest).global.imageRegistryandglobal.imagePullSecretsdo not apply to dynamic pluginoci:///ref://references. Plugin images must be mirrored and referenced individually.extra*entries (volumes, volume mounts, env vars, init containers) can be safely removed after migration, split into unconditional (always chart-managed) vs conditional (keep only if customized).All additions are in the "Important behavioral changes" and "Values mapping reference" sections. No chart code or schema changes.
Which issue(s) does this PR fix or relate to
How to test changes / Special notes to the reviewer
Documentation-only change. Review the rendered markdown for accuracy and readability.
The content was derived from building the AI-assisted migration script and testing it against real customer values files — these were the gaps that caused confusion or migration failures during testing.
Checklist
Assisted-by: Claude