Skip to content

feat(tls): inject centrally managed TLS config into pruner webhook#3453

Open
infernus01 wants to merge 1 commit into
tektoncd:mainfrom
infernus01:pruner-tls
Open

feat(tls): inject centrally managed TLS config into pruner webhook#3453
infernus01 wants to merge 1 commit into
tektoncd:mainfrom
infernus01:pruner-tls

Conversation

@infernus01
Copy link
Copy Markdown
Member

Changes

Wire the OpenShift APIServer TLS profile into the tekton-pruner-webhook deployment so that the pruner webhook applies the cluster-wide TLS version and cipher suite policy (PQC readiness).

The pruner webhook uses the Knative webhook framework, which WEBHOOK_TLS_MIN_VERSION, WEBHOOK_TLS_CIPHER_SUITES, and WEBHOOK_TLS_CURVE_PREFERENCES environment variables at startup via
knative.dev/pkg/network/tls.DefaultConfigFromEnv("WEBHOOK_").

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

Central TLS configuration is now injected into the tekton-pruner-webhook on OpenShift, aligning it with the cluster-wide TLS security profile for PQC readiness

Signed-off-by: Shubham Bhardwaj <shubbhar@redhat.com>
@tekton-robot tekton-robot added the release-note Denotes a PR that will be considered when it comes time to generate release notes. label May 29, 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 vdemeester after the PR has been reviewed.
You can assign the PR to them by writing /assign @vdemeester 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 requested a review from pramodbindal May 29, 2026 10:32
@tekton-robot tekton-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label May 29, 2026
@infernus01
Copy link
Copy Markdown
Member Author

/hold

@tekton-robot tekton-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants