Skip to content

fix: load raw values from tekton app values#3261

Merged
j-zimnowoda merged 13 commits into
mainfrom
APL-1886
May 26, 2026
Merged

fix: load raw values from tekton app values#3261
j-zimnowoda merged 13 commits into
mainfrom
APL-1886

Conversation

@merll
Copy link
Copy Markdown
Collaborator

@merll merll commented May 19, 2026

📌 Summary

This PR provides a fix for an issue reported by a user of App Platform, that setting _rawValues for Tekton does not have any effect.

🔍 Reviewer Notes

🧹 Checklist

  • Code is readable, maintainable, and robust.
  • Unit tests added/updated

@merll merll requested review from Ani1357 and j-zimnowoda as code owners May 19, 2026 12:53
@svcAPLBot
Copy link
Copy Markdown
Contributor

svcAPLBot commented May 19, 2026

Comparison of Helm chart templating output:

# rabbitmq-cluster-operator/templates/messaging-topology-operator/validating-webhook-configuration.yaml

# tekton-pipeline/templates/tekton-bot-sa.yaml

@@ metadata @@
! + one map entry added:
+ annotations:
+   test: v1

# values-repo.yaml

@@ apps.tekton._rawValues @@
! + one map entry added:
+ serviceaccount:
+   annotations:
+     test: v1

@j-zimnowoda j-zimnowoda self-assigned this May 26, 2026
@j-zimnowoda j-zimnowoda enabled auto-merge (squash) May 26, 2026 12:58
@merll merll disabled auto-merge May 26, 2026 14:11
@j-zimnowoda j-zimnowoda merged commit 62e7b62 into main May 26, 2026
14 checks passed
@j-zimnowoda j-zimnowoda deleted the APL-1886 branch May 26, 2026 14:52
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.

3 participants