Skip to content

feat: handle parent refs generically#4187

Merged
pmalek merged 1 commit intomainfrom
crd-from-oas-generic-parentref-type-name-handling
May 8, 2026
Merged

feat: handle parent refs generically#4187
pmalek merged 1 commit intomainfrom
crd-from-oas-generic-parentref-type-name-handling

Conversation

@pmalek
Copy link
Copy Markdown
Member

@pmalek pmalek commented May 8, 2026

What this PR does / why we need it:

Fix generic parent ref handling by adding necessary functions to CRD types:

  • HasParent() bool returning true for types that have parents
  • GetParentGVK() schema.GroupVersionKind which are attached to all non root objects

Which issue this PR fixes

Fixes #

Special notes for your reviewer:

PR Readiness Checklist:

Complete these before marking the PR as ready to review:

  • the CHANGELOG.md release notes have been updated to reflect significant changes

@pmalek pmalek self-assigned this May 8, 2026
@pmalek pmalek added this to the KO v2.2.x milestone May 8, 2026
@pmalek pmalek marked this pull request as ready for review May 8, 2026 10:55
@pmalek pmalek requested a review from a team as a code owner May 8, 2026 10:55
@pmalek pmalek requested a review from alacuku May 8, 2026 10:55
Comment thread controller/konnect/reconciler_generic_handle_generated_refs.go
@pmalek pmalek force-pushed the crd-from-oas-generic-parentref-type-name-handling branch from 210c373 to bb092c9 Compare May 8, 2026 12:17
@pmalek pmalek force-pushed the crd-from-oas-generic-parentref-type-name-handling branch from bb092c9 to 57bc406 Compare May 8, 2026 12:19
@pmalek pmalek requested a review from tao12345666333 May 8, 2026 12:21
@pmalek pmalek merged commit 8b1a6d8 into main May 8, 2026
88 of 90 checks passed
@pmalek pmalek deleted the crd-from-oas-generic-parentref-type-name-handling branch May 8, 2026 13:33
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.

2 participants