Skip to content

Add ManagedResourceDefinition one-pager#6577

Merged
negz merged 5 commits intocrossplane:mainfrom
negz:managerial
Jul 10, 2025
Merged

Add ManagedResourceDefinition one-pager#6577
negz merged 5 commits intocrossplane:mainfrom
negz:managerial

Conversation

@negz
Copy link
Copy Markdown
Member

@negz negz commented Jun 30, 2025

Description of your changes

If implemented this design would address #2262, #1143, and #4192.

This one-pager proposes we add a new ManagedResourceDefinition (MRD) type. An MRD is a managed-resource-specific abstraction on a CRD.

MRDs unlock two things:

  1. They allow filtering which MRs are enabled
  2. They give us somewhere to document MR connection details

I have:

Need help with this checklist? See the cheat sheet.

@negz negz requested a review from a team as a code owner June 30, 2025 23:42
@negz negz requested a review from jbw976 June 30, 2025 23:42
@negz negz requested a review from phisco July 3, 2025 00:05
Copy link
Copy Markdown
Contributor

@phisco phisco left a comment

Choose a reason for hiding this comment

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

Looking good; the devil is going to be in the details for sure. However, for a one-pager, I think this is more than enough to move to the implementation phase.

negz added 5 commits July 10, 2025 12:29
This one-pager proposes we add a new ManagedResourceDefinition (MRD)
type. An MRD is a managed-resource-specific abstraction on a CRD.

MRDs unlock two things:

1. They allow filtering which MRs are enabled
1. They give us somewhere to document MR connection details

Signed-off-by: Nic Cope <nicc@rk0n.org>
Also mark phisco as reviewer, fix formatting.

Signed-off-by: Nic Cope <nicc@rk0n.org>
Desired is implied by spec

Signed-off-by: Nic Cope <nicc@rk0n.org>
Signed-off-by: Nic Cope <nicc@rk0n.org>
Signed-off-by: Nic Cope <nicc@rk0n.org>
@negz negz merged commit f5f6aac into crossplane:main Jul 10, 2025
21 checks passed
@negz negz deleted the managerial branch July 10, 2025 19:43
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