Skip to content

Comments

Add trusted REST mappings fallback for CRD resources#852

Merged
terassyi merged 3 commits intomainfrom
add-trusted-rest-mappings
Feb 20, 2026
Merged

Add trusted REST mappings fallback for CRD resources#852
terassyi merged 3 commits intomainfrom
add-trusted-rest-mappings

Conversation

@terassyi
Copy link
Contributor

@terassyi terassyi commented Feb 12, 2026

This PR allows pre-registering REST mappings in the cluster config so that CKE
can manage custom resources whose CRDs may not yet be available via
API discovery. When the REST mapper fails, CKE falls back to these
trusted mappings.

Signed-off-by: terashima tomoya-terashima@cybozu.co.jp

@terassyi terassyi self-assigned this Feb 12, 2026
@terassyi terassyi marked this pull request as ready for review February 12, 2026 05:29
@terassyi terassyi force-pushed the add-trusted-rest-mappings branch from 5c2cec0 to 7431487 Compare February 12, 2026 05:33
Signed-off-by: terashima <tomoya-terashima@cybozu.co.jp>
@terassyi terassyi force-pushed the add-trusted-rest-mappings branch from 7431487 to c7d2d1a Compare February 12, 2026 07:47
Signed-off-by: terashima <tomoya-terashima@cybozu.co.jp>
@terassyi terassyi requested a review from YZ775 February 17, 2026 08:43
Copy link
Contributor

@morimoto-cybozu morimoto-cybozu left a comment

Choose a reason for hiding this comment

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

こちらでも見ていたのでコメントします。

  • docs/cluster.md, CHANGELOG.mdへの追記が必要です。
  • docs/user-resources.mdにも追記があると良いと思います。

Signed-off-by: terashima <tomoya-terashima@cybozu.co.jp>
@terassyi terassyi force-pushed the add-trusted-rest-mappings branch from deb1436 to 310539b Compare February 19, 2026 01:51
Copy link
Contributor

@YZ775 YZ775 left a comment

Choose a reason for hiding this comment

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

LGTM
Thank you for fixing.

@terassyi terassyi merged commit 030a90b into main Feb 20, 2026
7 checks passed
@terassyi terassyi deleted the add-trusted-rest-mappings branch February 20, 2026 00:32
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.

3 participants