-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mega Feature: Add support for external references #72
Comments
Great idea! This approach can also streamline creating roles and policies. Instead of duplicating the same IAM policy or role, we can use a centralized ConfigMap and reference it across kro applications object and only override the values that are distinct. |
This would be a very beneficial feature that would greatly improve the usability of kro. |
Definitely agree - option 2 is my preferred approach if that is feasible. I can see cases where we would want a cluster owner would want to onboard a tenant and reference a resource created before running through the kro-orchestration tenant onboarding flow |
+1 |
Good suggestion. Definitely need this feature. Just as long as you have the correct permissions (RBAC) to read from the external reference, then it will be quite a secure solution. |
Add ability to reference fields from external to the ResourceGroups resources that exist in the cluster.
For example, reference
data
from an existing ConfigMap:Sample ConfigMap:
Option 1: Import an external resource explicitly without taking over control of it and reference values from other resources. Import by one of three options 1/
name
, 2/labels
, 3/annotations
Option 2: just reference it inline.
Option 1 contains more lines of yaml but is readable and easier to implement hence it is the preferred option.
The text was updated successfully, but these errors were encountered: