Skip to content

Conversation

@jessesimpson36
Copy link
Contributor

@jessesimpson36 jessesimpson36 commented Jan 5, 2026

Which problem does the PR fix?

In my latest conversation with @mlowe-camunda , we decided upon the heuristic of allowing processing of {{ }} expressions anywhere where a URL is defined. And this includes the extraConfiguration and configuration sections of values.yaml.

What's in this PR?

To test this, I saved the following as k.yaml

jesse: jesseRules

orchestration:
  configuration: |
      {{ .Values.jesse | quote }}

  extraConfiguration:
    jesse: {{ .Values.jesse | quote }}

and then ran

helm template cpt charts/camunda-platform-8.9 -f k.yaml --show-only templates/orchestration/configmap-unified.yaml

Checklist

Please make sure to follow our Contributing Guide.

Before opening the PR:

  • In the repo's root dir, run make go.update-golden-only.
  • There is no other open pull request for the same update/change.
  • Tests for charts are added (if needed).
  • In-repo documentation are updated (if needed).

After opening the PR:

  • Did you sign our CLA (Contributor License Agreement)? It will show once you open the PR.
  • Did all checks/tests pass in the PR?

Signed-off-by: Jesse Simpson <jesse.simpson@camunda.com>
Signed-off-by: Jesse Simpson <jesse.simpson@camunda.com>
@jessesimpson36 jessesimpson36 marked this pull request as ready for review January 5, 2026 19:10
@jessesimpson36 jessesimpson36 requested a review from a team as a code owner January 5, 2026 19:10
@jessesimpson36 jessesimpson36 requested review from bkenez and removed request for a team January 5, 2026 19:10
@jessesimpson36 jessesimpson36 changed the title fix: call TPL inside extraConfiguration fix: call TPL inside extraConfiguration and configuration Jan 5, 2026
@mlowe-camunda
Copy link

Nice initiative 👍
I should be having a new epic coming out this week which includes this type of functionality. It might be good to park this PR until you see the epic.

@jessesimpson36 jessesimpson36 added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/connectors component/identity component/operate component/optimize component/orchestration component/tasklist component/web-modeler component/zeebe do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. version/8.6 Camunda applications/cycle version version/8.7 Camunda applications/cycle version version/8.8 Camunda applications/cycle version version/8.9 Camunda applications/cycle version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants