Skip to content

Rename TektonScheduler to TektonKueue - #3809

Open
waveywaves wants to merge 1 commit into
tektoncd:mainfrom
waveywaves:rename/tekton-kueue
Open

Rename TektonScheduler to TektonKueue#3809
waveywaves wants to merge 1 commit into
tektoncd:mainfrom
waveywaves:rename/tekton-kueue

Conversation

@waveywaves

@waveywaves waveywaves commented Jul 24, 2026

Copy link
Copy Markdown
Member

Changes

Rename the Operator component API from TektonScheduler to TektonKueue,
including TektonConfig.spec.kueue, controller wiring, generated clients and
CRDs, RBAC, Helm and OperatorHub manifests, tests, and documentation.

Retain the deprecated TektonScheduler CRD and TektonConfig.spec.scheduler
field during a compatibility window. Pre-upgrade reconciliation migrates
existing configuration and legacy resources to TektonKueue; explicitly
configured spec.kueue values take precedence.

A local kind upgrade from the pre-rename API verified that queue configuration
is preserved, legacy InstallerSets and resources are removed, both CRDs remain
served, pre-upgrade completes, and the migrated TektonKueue reaches Ready.

Repository and release-fetch references use tektoncd/tekton-kueue. Go imports
retain github.com/konflux-ci/tekton-kueue because release v0.3.1 still
declares that module path.

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

See the contribution guide for more details.

Release Notes

`TektonScheduler` is deprecated in favor of `TektonKueue`, and
`TektonConfig.spec.scheduler` is deprecated in favor of
`TektonConfig.spec.kueue`. Existing configuration is migrated automatically
during pre-upgrade reconciliation.

Action required: rename scheduler image overrides to `IMAGE_KUEUE_MANAGER` and
`IMAGE_KUEUE_WEBHOOK`.

@tekton-robot tekton-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. release-note-action-required Denotes a PR that introduces potentially breaking changes that require user action. labels Jul 24, 2026
@tekton-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign savitaashture after the PR has been reviewed.
You can assign the PR to them by writing /assign @savitaashture in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tekton-robot tekton-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jul 24, 2026
@codecov-commenter

codecov-commenter commented Jul 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 26.06%. Comparing base (b3aa987) to head (96026cb).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3809      +/-   ##
==========================================
+ Coverage   25.78%   26.06%   +0.28%     
==========================================
  Files         463      471       +8     
  Lines       24530    24768     +238     
==========================================
+ Hits         6325     6457     +132     
- Misses      17507    17602      +95     
- Partials      698      709      +11     
Flag Coverage Δ
unit-tests 26.06% <ø> (+0.28%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@waveywaves
waveywaves marked this pull request as ready for review July 27, 2026 08:40
@tekton-robot tekton-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 27, 2026
@tekton-robot
tekton-robot requested a review from khrm July 27, 2026 08:40
@pramodbindal

Copy link
Copy Markdown
Member

As you mentioned this is breaking change. we need to provide some backward compatibility.

Something like. If you can keep the dummy TektonScheduler for some releases so existing TektonScheduler field does not create any issue and if present we need to move the config to TektonKueue.
This can be done in pre-upgrade Remove the Scheduler so after one release all customers will be moved to TektonScheduler from TektonKueue

@waveywaves
waveywaves marked this pull request as draft July 27, 2026 10:29
@tekton-robot tekton-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 27, 2026
@waveywaves
waveywaves marked this pull request as ready for review July 28, 2026 08:35
@tekton-robot tekton-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 28, 2026
@tekton-robot
tekton-robot requested a review from divyansh42 July 28, 2026 08:36
@anithapriyanatarajan

Copy link
Copy Markdown
Contributor

@waveywaves - do we wait until the tekton-kueue project is moved to tektoncd org?

@vdemeester

Copy link
Copy Markdown
Member

@waveywaves - do we wait until the tekton-kueue project is moved to tektoncd org?

for what is worth, it has moved in tektoncd org today 😛

@anithapriyanatarajan

Copy link
Copy Markdown
Contributor

@waveywaves - do we wait until the tekton-kueue project is moved to tektoncd org?

for what is worth, it has moved in tektoncd org today 😛

Thank you 😄 @waveywaves could you update the import pkg references and components.yaml references. Thank you

@tekton-robot tekton-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Aug 3, 2026
@waveywaves
waveywaves force-pushed the rename/tekton-kueue branch from 5441998 to aad0734 Compare August 6, 2026 13:52
@tekton-robot tekton-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 6, 2026
Rename the component API, controllers, clients, CRDs, manifests, and documentation to match Tekton Kueue.

Retain the deprecated TektonScheduler API and TektonConfig scheduler field for upgrade compatibility. Pre-upgrade reconciliation migrates legacy configuration and resources to TektonKueue and removes obsolete InstallerSets.

Use tektoncd/tekton-kueue for repository and release references. Keep Go imports on github.com/konflux-ci/tekton-kueue because v0.3.1 still declares that module path.

BREAKING CHANGE: TektonScheduler and TektonConfig.spec.scheduler are deprecated in favor of TektonKueue and TektonConfig.spec.kueue. Image overrides now use IMAGE_KUEUE_MANAGER and IMAGE_KUEUE_WEBHOOK.

Signed-off-by: waveywaves <11972233+waveywaves@users.noreply.github.com>
@waveywaves
waveywaves force-pushed the rename/tekton-kueue branch from aad0734 to 96026cb Compare August 6, 2026 14:33
@tekton-robot tekton-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note-action-required Denotes a PR that introduces potentially breaking changes that require user action. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants