Skip to content

feat: make tekton optional#3085

Merged
ElderMatt merged 13 commits intomainfrom
APL-1614
Mar 31, 2026
Merged

feat: make tekton optional#3085
ElderMatt merged 13 commits intomainfrom
APL-1614

Conversation

@ElderMatt
Copy link
Copy Markdown
Contributor

@ElderMatt ElderMatt commented Mar 27, 2026

📌 Summary

This PR makes tekton an optional app.

Newly created cluster will have tekton disabled by default.
Upgraded clusters will have a migration where the enabled property is added to tekton and set to true.

Tekton pipelines also has a added priorityClass so that tekton pipeline managed pods have a lower priority.

image

@svcAPLBot
Copy link
Copy Markdown
Contributor

svcAPLBot commented Mar 27, 2026

Comparison of Helm chart templating output:

@@ data.VERSIONS @@
! ± value change in multiline text (one insert, one deletion)
  
- {"api":"main","aplCharts":"main","console":"main","consoleLogin":"main","core":"main","specVersion":57,"tasks":"main","tools":"main"}
+ {"api":"main","aplCharts":"main","console":"main","consoleLogin":"main","core":"main","specVersion":58,"tasks":"main","tools":"main"}




@@ spec.resourcetemplates.0.spec.taskRunTemplate.podTemplate @@
# triggers.tekton.dev/v1alpha1/TriggerTemplate/trigger-template-demo-ext-repo-buildpacks-public-trigger
! + one map entry added:
+ priorityClassName: tekton-low


@@ spec.resourcetemplates.0.spec.taskRunTemplate.podTemplate @@
# triggers.tekton.dev/v1alpha1/TriggerTemplate/trigger-template-demo-ext-repo-private-trigger
! + one map entry added:
+ priorityClassName: tekton-low

@@ spec.resourcetemplates.0.spec.taskRunTemplate.podTemplate @@
# triggers.tekton.dev/v1alpha1/TriggerTemplate/trigger-template-demo-ext-repo-public-trigger
! + one map entry added:
+ priorityClassName: tekton-low

@@ spec.resourcetemplates.0.spec.taskRunTemplate.podTemplate @@
# triggers.tekton.dev/v1alpha1/TriggerTemplate/trigger-template-demo-java3-v0-0-1
! + one map entry added:
+ priorityClassName: tekton-low


@@ data @@
! + one map entry added:
+ default-pod-template: |
+   priorityClassName: tekton-low


@@ apps.tekton @@
! + one map entry added:
+ enabled: true

@@ versions.specVersion @@
! ± value change
- 57
+ 58

Copy link
Copy Markdown
Contributor

@CasLubbers CasLubbers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on fresh cluster and works ✅

@ElderMatt ElderMatt merged commit 823e9b2 into main Mar 31, 2026
14 checks passed
@ElderMatt ElderMatt deleted the APL-1614 branch March 31, 2026 13:51
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.

4 participants