Skip to content

Conversation

GenosseOtt
Copy link
Contributor

image

@andreaskienle
Copy link
Contributor

Fixed the merge conflicts and moved texts to en.json. We can keep the styling like this until we decided on a styling strategy.

@andreaskienle
Copy link
Contributor

@GenosseOtt So, the generic term is Managed Resource, and Providers, ProviderConfigs, and Resources are all (parts of) Managed Resources?

For #2 , I had thought it was the other way around, so I had named the component (and everything related) ManagedResource. In this case, would it make sense to rename it to just Resource?

@enrico-kaack-comp
Copy link
Contributor

I would disagree with the naming. Managed Resources are the resources (K8s resources) that are managed by a crossplane provider (as in that are translated into API requests by the provider and represents a managed service- https://docs.crossplane.io/latest/concepts/managed-resources/).
Providers are a resource defined by crossplane. A specific ProviderConfig is defined by every provider but their meaning is to configure the Provider (like describing with which API the provider should work).

Therefore I suggest the naming:
Crossplane (Top Level)

  • Table Providers
  • Table Provider Configs
  • Table Managed Resources

@GenosseOtt
Copy link
Contributor Author

Hey, no strong opinion. In the current POC we also called it Crossplane. No complains...

Long term I was hoping to use rather a term reflecting purpose (e.g. Policies) rather than products (e.g. Kyverno).
For Crossplane, given the 3 tables it summarizes, this cannot be fully translated.

No concern with the naming suggested by @enrico-kaack-comp

Therefore I suggest the naming:
Crossplane (Top Level)

Table Providers
Table Provider Configs
Table Managed Resources

@andreaskienle
Copy link
Contributor

@enrico-kaack-comp, @GenosseOtt I changed it to this:
image

@GenosseOtt
Copy link
Contributor Author

Thanks. I cannot approve my own request here. But 👍 from my side.

@enrico-kaack-comp enrico-kaack-comp merged commit a19d434 into main Mar 28, 2025
4 checks passed
@enrico-kaack-comp enrico-kaack-comp deleted the improve-mcp-detail-spacing branch March 28, 2025 15:04
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