kcp icon indicating copy to clipboard operation
kcp copied to clipboard

feat(workspacetype): Add defaultAPIBinding lifecylce

Open MisterMX opened this issue 10 months ago • 5 comments

Summary

Add a controller to automatically keep defaultAPIBindings defined in a workspacetype up to date in all workspaces that derive from it.

on-behalf-of: @eon-se [email protected]

What Type of PR Is This?

/kind feature /kind api-change

Related Issue(s)

  • Fixes https://github.com/kcp-dev/kcp/issues/3248
  • Replaces: https://github.com/kcp-dev/kcp/pull/3279

Release Notes

Add a controller to automatically keep defaultAPIBindings defined in a workspacetype up to date in all workspaces that derive from it.

MisterMX avatar Apr 16 '25 13:04 MisterMX

Hi @MisterMX. Thanks for your PR.

I'm waiting for a kcp-dev member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

kcp-ci-bot avatar Apr 16 '25 13:04 kcp-ci-bot

/ok-to-test

embik avatar Apr 17 '25 06:04 embik

/retest

flakes?

embik avatar Apr 17 '25 09:04 embik

/retest

embik avatar Apr 28 '25 10:04 embik

/retest

flakes due to CI infrastructure issues, I think?

embik avatar Jun 02 '25 14:06 embik

/retest

embik avatar Jun 05 '25 08:06 embik

/retest

embik avatar Jun 06 '25 08:06 embik

LGTM label has been added.

Git tree hash: 32b1dfd5dcc46e4813c57f798ada113430aa6160

kcp-ci-bot avatar Jun 06 '25 08:06 kcp-ci-bot

/approve

Given that https://github.com/kcp-dev/kcp/pull/3381#discussion_r2131842826 was a minor concern and I think it's fine, let's move this PR forward. It's been long in the making. Thank you so much for the contribution, @MisterMX.

embik avatar Jun 11 '25 14:06 embik

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: embik

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

The pull request process is described here

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 avatar Jun 11 '25 14:06 kcp-ci-bot

/lgtm

oops.

embik avatar Jun 11 '25 15:06 embik

LGTM label has been added.

Git tree hash: 3002525af2c597ffe7be76ab20a9213be47d8b2b

kcp-ci-bot avatar Jun 11 '25 15:06 kcp-ci-bot