Skip to content

feat (Tekton Scheduler) : Onboard Tekton Scheduler to operator#2934

Merged
tekton-robot merged 1 commit intotektoncd:mainfrom
tektoncd-pb:tekton-kueue
Feb 3, 2026
Merged

feat (Tekton Scheduler) : Onboard Tekton Scheduler to operator#2934
tekton-robot merged 1 commit intotektoncd:mainfrom
tektoncd-pb:tekton-kueue

Conversation

@pramodbindal
Copy link
Copy Markdown
Member

@pramodbindal pramodbindal commented Oct 30, 2025 •
edited
Loading

Changes

Onboarding Tekton Scheduler to operator.

Tekton Scheduler enables the pipeline scheduling on Single or multi cluster environment. But default this feature remains disabled.

This feature is dependent on upstream Kueue so that must be installed first before scheduler can be enabled. It also uses cert-manager so that also must be installed before head.

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

Tekton Scheduler component  can be installed by Pipelines Operator.
   Scheduler component Supports Scheduling of pipelines on Single or Multiple clusters.
  For multi-cluster environment,  You can define the role of cluster in Scheduler configiuration

For more details refer docs/TektonScheduler.md

@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 Denotes a PR that will be considered when it comes time to generate release notes. labels Oct 30, 2025
@tekton-robot tekton-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Oct 30, 2025
@tekton-robot tekton-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Nov 10, 2025
@tekton-robot tekton-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 17, 2025
@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 Nov 18, 2025
@tekton-robot tekton-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 4, 2025
@tekton-robot tekton-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 5, 2025
@tekton-robot tekton-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 10, 2025
@pramodbindal pramodbindal force-pushed the tekton-kueue branch 2 times, most recently from cfa0d55 to 149d551 Compare December 10, 2025 13:42
@pramodbindal
Copy link
Copy Markdown
Member Author

pramodbindal commented Dec 11, 2025

/retest

@pramodbindal pramodbindal force-pushed the tekton-kueue branch 5 times, most recently from bdf5304 to 39467c2 Compare December 16, 2025 05:56
@tekton-robot tekton-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 13, 2026
@tekton-robot tekton-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 16, 2026
@tekton-robot tekton-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 16, 2026
@pramodbindal pramodbindal force-pushed the tekton-kueue branch 3 times, most recently from a4bbf4c to 7dee5fd Compare January 16, 2026 07:48
@tekton-robot tekton-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 27, 2026
@savitaashture
Copy link
Copy Markdown
Contributor

savitaashture commented Jan 28, 2026 •
edited
Loading

@pramodbindal Please add the description for the change

  • Reason for this change
  • How this PR helps in single and multicluster env
  • and upgrade test screenshot
  • Is this works for both K8s and other distributions

Thank you

@savitaashture
Copy link
Copy Markdown
Contributor

savitaashture commented Jan 28, 2026

@pramodbindal can we also add a document on what pre-requisite required for the scheduler to be enabled

jkhelil
jkhelil reviewed Jan 30, 2026
View reviewed changes
Comment thread pkg/apis/operator/v1alpha1/tektonscheduler_types.go Outdated
Comment thread pkg/reconciler/shared/tektonconfig/scheduler/scheduler_test.go Outdated
package scheduler

//
//import (
Copy link
Copy Markdown
Member

@jkhelil jkhelil Jan 30, 2026

Choose a reason for hiding this comment

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

Any reason why testss are commented out here ?
We need to add these tests

Comment thread pkg/apis/operator/v1alpha1/tektonscheduler_types.go Outdated
Comment thread pkg/apis/operator/v1alpha1/tektonconfig_defaults.go
Comment thread pkg/reconciler/kubernetes/tektonscheduler/finalize.go Outdated
Comment thread pkg/apis/operator/v1alpha1/tektonscheduler_types.go
Comment thread docs/AirGapImageConfiguration.md Outdated
@jkhelil
Copy link
Copy Markdown
Member

jkhelil commented Jan 30, 2026 •
edited
Loading

@pramodbindal Can we add e2e tests under /test dir
Overwall, we dont have much test for the feature

jkhelil
jkhelil reviewed Jan 30, 2026
View reviewed changes
Comment thread docs/AirGapImageConfiguration.md
jkhelil
jkhelil reviewed Jan 30, 2026
View reviewed changes
Comment thread pkg/apis/operator/v1alpha1/tektonscheduler_validation.go
Comment thread pkg/reconciler/kubernetes/tektonscheduler/installerset.go Outdated
@pramodbindal
Copy link
Copy Markdown
Member Author

pramodbindal commented Feb 2, 2026

@pramodbindal Can we add e2e tests under /test dir Overwall, we dont have much test for the feature

This will require complete multi-cluster setup.
We are planning to add multicluster e2e in different repo

@jkhelil
Copy link
Copy Markdown
Member

jkhelil commented Feb 2, 2026

/approve

@tekton-robot
Copy link
Copy Markdown
Contributor

tekton-robot commented Feb 2, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jkhelil

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

The pull request process is described 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

@mbpavan
Copy link
Copy Markdown
Contributor

mbpavan commented Feb 3, 2026

/kind feature

@mbpavan
Copy link
Copy Markdown
Contributor

mbpavan commented Feb 3, 2026

@pramodbindal please add release notes

@mbpavan
Copy link
Copy Markdown
Contributor

mbpavan commented Feb 3, 2026

@pramodbindal please add info for docs/TektonConfig.md and also please create docs/TektonScheduler.md where we can mention prerequiste steps and other information regarding this

@mbpavan
Copy link
Copy Markdown
Contributor

mbpavan commented Feb 3, 2026

can you please also validate if upgrade is working fine both on kubernetes and openshift?

@pramodbindal
Copy link
Copy Markdown
Member Author

pramodbindal commented Feb 3, 2026

@pramodbindal please add info for docs/TektonConfig.md and also please create docs/TektonScheduler.md where we can mention prerequiste steps and other information regarding this

Added

@pramodbindal
Copy link
Copy Markdown
Member Author

pramodbindal commented Feb 3, 2026

can you please also validate if upgrade is working fine both on kubernetes and openshift?

Yes it validated

mbpavan
mbpavan reviewed Feb 3, 2026
View reviewed changes
Comment thread ...ift/manifests/bases/openshift-pipelines-operator-rh.clusterserviceversion.template.yaml.tmpl Outdated
Comment thread ...penshift/manifests/bases/openshift-pipelines-operator-rh.clusterserviceversion.template.yaml
@pramodbindal
Add MultiCluster Scheduler config
c7158b8 
Signed-off-by: Pramod Bindal <prbindal@redhat.com>

Feat:Onboard Tekton-Kueue

Fix: change go package in tektonpruner

Fix: change go package in tektonpruner

Add Entry in CSV template
@mbpavan
Copy link
Copy Markdown
Contributor

mbpavan commented Feb 3, 2026

/lgtm

@mbpavan mbpavan mentioned this pull request Feb 6, 2026
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jkhelil jkhelil jkhelil left review comments

@mbpavan mbpavan mbpavan left review comments

@PuneetPunamiya PuneetPunamiya Awaiting requested review from PuneetPunamiya

@anithapriyanatarajan anithapriyanatarajan Awaiting requested review from anithapriyanatarajan

Assignees

@mbpavan mbpavan

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. kind/feature Categorizes issue or PR as related to a new feature. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@pramodbindal @savitaashture @jkhelil @tekton-robot @mbpavan