FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix error when updating managedFields failed by iakmc · Pull Request #476 · kbind-dev/kbind · GitHub

Fix error when updating managedFields failed - #476

Merged
mjudeikis merged 1 commit into
kbind-dev:mainfrom
iakmc:fix-failing-update-managedfields
Mar 4, 2026
Merged

Fix error when updating managedFields failed#476
mjudeikis merged 1 commit into
kbind-dev:mainfrom
iakmc:fix-failing-update-managedfields

Conversation

iakmc commented Feb 23, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

BoundSchema templates were created from CRDs and carried over CRD managedFields.
This caused a [SHOULD NOT HAPPEN] failed to update managedFields error in KCP logs.

What Type of PR Is This?

/kind bug

Related Issue(s)

Fixes #3788

Release Notes

NONE

iakmc requested a review from a team as a code owner February 23, 2026 13:02
iakmc force-pushed the fix-failing-update-managedfields branch from 3698d2b to 90bb63b Compare February 23, 2026 13:51

Copy link
Copy Markdown
Contributor

Looks like there's an error with the way you've linked the issue for this PR

iakmc force-pushed the fix-failing-update-managedfields branch from 90bb63b to 0444f68 Compare March 3, 2026 09:11
On-behalf-of: SAP <iskren.pertov@sap.com>
Signed-off-by: Iskren Petrov <iskren@kubermatic.com>

reset ObjectMeta when creating BoundSchema from CRD template

add boundschema test

fix copyright date
iakmc force-pushed the fix-failing-update-managedfields branch from 0444f68 to 6b438de Compare March 3, 2026 09:56
mjudeikis enabled auto-merge (squash) March 3, 2026 10:14
mjudeikis merged commit 8e4325e into kbind-dev:main Mar 4, 2026
7 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

4 participants


Back | FazBrowse Home | New Git URL