-
Notifications
You must be signed in to change notification settings - Fork 24
Open
Labels
dxkind/cleanupCleanup/refactor an existing component/codeCleanup/refactor an existing component/codetriage/acceptedThe issue was reviewed and is complete enough to start working on itThe issue was reviewed and is complete enough to start working on it
Milestone
Description
Description
For ease these were initially added to resources which is is more used for an "abstract" resource i.e. other resources probably inherit/use this.
MeshIdentity/MeshTrusts ideally don't belong in here and should be in their own module or modules, either:
mesh-identitiesandmesh-trusts; ormtlscontaining both potentially subfolder'd intomtls/meshidentities,mtls/meshidentities
I kinda like having an mtls module containing all mtls stuff which can be enabled disabled in one shot. But we currently have a pattern of one module per resource and then we sometimes wrap those into a parent module (as is the case with service-mesh.
All in all, I guess we should stick with the "one module per resource" pattern for now at least
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
dxkind/cleanupCleanup/refactor an existing component/codeCleanup/refactor an existing component/codetriage/acceptedThe issue was reviewed and is complete enough to start working on itThe issue was reviewed and is complete enough to start working on it