Skip to content

Refactor existing APIs to use APIResourceSchema, BoundAPIResourceSchema for service exposure#261

Merged
mjudeikis merged 28 commits into
kbind-dev:mainfrom
cnvergence:v1alpha2-apiserviceexport
Jun 20, 2025
Merged

Refactor existing APIs to use APIResourceSchema, BoundAPIResourceSchema for service exposure#261
mjudeikis merged 28 commits into
kbind-dev:mainfrom
cnvergence:v1alpha2-apiserviceexport

Conversation

@cnvergence

@cnvergence cnvergence commented May 9, 2025

Copy link
Copy Markdown
Member

Summary

  • move existing apis to v1alpha2 and adapt them for new APIResourceSchema and BoundAPIResourceSchema.
  • adapt konnector code
  • adapt example-backend code
  • fix e2e tests

Related issue(s)

Fixes #255

Release Notes

Refactor existing APIs to use APIResourceSchema, BoundAPIResourceSchema for service exposure .

@cnvergence cnvergence requested a review from a team as a code owner May 9, 2025 15:19
@cnvergence cnvergence marked this pull request as draft May 9, 2025 15:30
@cnvergence cnvergence force-pushed the v1alpha2-apiserviceexport branch 3 times, most recently from 1798a1d to 3ff4507 Compare May 16, 2025 15:51
@cnvergence cnvergence force-pushed the v1alpha2-apiserviceexport branch 2 times, most recently from e03489a to 1c1b2c9 Compare May 20, 2025 14:12
@cnvergence cnvergence changed the title Refactor APIServiceExport and APIServiceExportRequest for v1alpha2 Refactor existing APIs to use APIResourceSchema, BoundAPIResourceSchema for service exposure May 22, 2025
@cnvergence cnvergence force-pushed the v1alpha2-apiserviceexport branch 4 times, most recently from 5e86fa4 to b8fcd7a Compare May 28, 2025 11:53
@cnvergence cnvergence force-pushed the v1alpha2-apiserviceexport branch 3 times, most recently from 32a7ed7 to d95c9af Compare June 5, 2025 16:31
@cnvergence cnvergence marked this pull request as ready for review June 6, 2025 10:17
cnvergence added 14 commits June 9, 2025 14:38
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
@cnvergence cnvergence force-pushed the v1alpha2-apiserviceexport branch from 48d1dce to 783b6c1 Compare June 9, 2025 14:56
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
@cnvergence cnvergence force-pushed the v1alpha2-apiserviceexport branch from 783b6c1 to a8bfcb8 Compare June 9, 2025 15:26
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
@cnvergence cnvergence force-pushed the v1alpha2-apiserviceexport branch from 627f0d2 to 8a210e6 Compare June 11, 2025 07:39

source "$(dirname "$0")/host-ip.sh"
get_host_ip
DEFAULT_KONNECTOR_IMAGE="ghcr.io/kube-bind/konnector:v0.4.6"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This will break as soon as we merge this. repo has "latest" image, should we do the un-holy thing and just use latest here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I wasn't aware of that, for now, I wanted a default for "sane" tested version

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

But test uses APISchemas already, but this implementation does not support it :D

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

No, I meant for running the example-backend kind setup with 0.4.8 version of images and kube-bind cli.
We can use the latest as the default, I don't mind

cnvergence and others added 12 commits June 11, 2025 17:18
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

@mjudeikis mjudeikis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Lets merge and see.

@mjudeikis mjudeikis merged commit 49200cc into kbind-dev:main Jun 20, 2025
9 checks passed
@cnvergence cnvergence deleted the v1alpha2-apiserviceexport branch June 20, 2025 12:19
mjudeikis added a commit to mjudeikis/kube-bind that referenced this pull request Aug 18, 2025
…ma for service exposure (kbind-dev#261)

* Refactor APIServiceExport and APIServiceExportRequest for v1alpha2

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
On-behalf-of: @SAP karol.szwaj@sap.com

* storage version

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

* add APIServiceBinding v1alpha2

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

* update konnector to v1alpha2

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

* update cmd and example-backend to v1alpha2

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

* update go.mod

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

* refactor konnector serviceexport and servicebinding controllers

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

* update service binding reconciler

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

* refactor example backend

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

* refactor example backend

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

* fix lint

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

* remove validation rules

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

* add crds to e2e

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

* add rbac e2e

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

* setup with local kind

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

* fix backend kind cluster

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

* make apiresourceschemas cluster scoped

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

* add version gate

* nit

* Move towards single kind cluster

* Add additional perms to read schemas

* Fix conditions

* fix race condition

* fix test

* update readme

* add short names to crds and prealloc hashes

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

* update readme, move kcp setup to quickstart page

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com

---------

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>
Co-authored-by: Mangirdas Judeikis <mangirdas@judeikis.lt>
On-behalf-of: @SAP mangirdas.judeikis@sap.com
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor existing APIs to use APIResourceSchema, BoundAPIResourceSchema for service exposure.

2 participants