Skip to content

Improve reconciled resources#149

Merged
kcp-ci-bot merged 7 commits intokcp-dev:mainfrom
xrstf:probes
Feb 9, 2026
Merged

Improve reconciled resources#149
kcp-ci-bot merged 7 commits intokcp-dev:mainfrom
xrstf:probes

Conversation

@xrstf
Copy link
Contributor

@xrstf xrstf commented Feb 6, 2026

Summary

This PR performs some general improvements to the reconciled resources:

  • make test-e2e will now also enable the feature gate for the bundles, which makes it possible again to run the e2e tests locally.
  • Missing readiness, liveness and startup probes were added to all Deployments, inspired by the values from the kcp Helm charts.
  • The front-proxy had mixed up liveness/readiness probes. Fixed.
  • Some missing labels are now being added.
  • A consistent security context is applied to all Deployments now.

What Type of PR Is This?

/kind feature

Release Notes

* Fixed missing labels in generated Kubernetes objects.
* Fixed wrong readiness/liveness probes on kcp-front-proxy Deployments.
* Add liveness/readiness/startup probes to the shard Deployments.
* All Deployments now specifiy a security context with a readonly filesystem, dropping all privileges and forbidding privilege escalations.

xrstf added 6 commits February 6, 2026 13:10
On-behalf-of: @SAP christoph.mewes@sap.com
On-behalf-of: @SAP christoph.mewes@sap.com
On-behalf-of: @SAP christoph.mewes@sap.com
On-behalf-of: @SAP christoph.mewes@sap.com
On-behalf-of: @SAP christoph.mewes@sap.com
@kcp-ci-bot kcp-ci-bot added release-note Denotes a PR that will be considered when it comes time to generate release notes. kind/feature Categorizes issue or PR as related to a new feature. dco-signoff: yes Indicates the PR's author has signed the DCO. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Feb 6, 2026
On-behalf-of: @SAP christoph.mewes@sap.com
Copy link
Member

@ntnn ntnn left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@kcp-ci-bot kcp-ci-bot added the lgtm Indicates that a PR is ready to be merged. label Feb 6, 2026
@kcp-ci-bot
Copy link
Contributor

LGTM label has been added.

DetailsGit tree hash: acd10e3c68b76057c5faedd410d9f9ac5aa6a947

@xrstf
Copy link
Contributor Author

xrstf commented Feb 9, 2026

/approve

@kcp-ci-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ntnn, xrstf

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

@kcp-ci-bot kcp-ci-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 9, 2026
@kcp-ci-bot kcp-ci-bot merged commit 829b625 into kcp-dev:main Feb 9, 2026
12 checks passed
xrstf added a commit to xrstf/kcp-operator that referenced this pull request Feb 11, 2026
On-behalf-of: @SAP christoph.mewes@sap.com
@xrstf xrstf deleted the probes branch February 12, 2026 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has signed the DCO. 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/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.

3 participants