[jenniferubah]

Summary

• Fix Helm README examples for multi-value environmentAgent.embeddedSps so commas survive bash and reach Helm’s --set parser.
• Replace single-backslash escapes (,) with double-backslash (\,) in the guide and install/upgrade examples (container\,vm, container\,cluster).

Fixes: https://redhat.atlassian.net/browse/FLPATH-4946

Assisted-By: Cursor AI

(cherry picked from commit 96b82b1)

[qodo-code-review]

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 review

Tip of the day

💡 Did you know, you can hide the parts of a finding you never read, like the evidence or the agent prompt

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

[qodo-code-review]

PR Summary by Qodo

Fix comma escaping in Helm Environment Agent deployment examples

📝 Documentation 🐞 Bug fix 🕐 Less than 5 minutes

AI Description

• Correct comma escaping so multi-provider values reach Helm’s --set parser intact.
• Fix the container,vm and container,cluster upgrade examples.

Diagram

graph TD
  A["README examples"] --> B["Shell parsing"] --> C["Helm set parser"] --> D["Embedded SP value"]

Loading

High-Level Assessment

Keep the corrected unquoted examples: the extra backslash accounts for shell parsing while retaining the guide’s existing command style. Quoting each --set argument would also work but offers no clear advantage here.

Files changed (1) +3 / -3

Documentation (1) +3 / -3

README.mdCorrect escaping in multi-provider Helm examples +3/-3

Correct escaping in multi-provider Helm examples

• Changes the comma-escaping guidance and the 'container,vm' and 'container,cluster' upgrade commands from '\,' to '\\,'. This preserves the escape through shell parsing so Helm receives each provider list as a single value.

deploy/helm/dcm/README.md