-
Notifications
You must be signed in to change notification settings - Fork 202
Add ServiceAccount enricher and configuration #931
Comments
This could be easily done by a specific enricher, which
We could also add a config option to the |
Agreed. It does feel like we should just have an enricher flag we could enable in the pom.xml to enable the SA stuff. I wonder if by default if you add an |
This should be also no problem by adding to the However it makes more sense to allow the configuration of an SA directly in the config like in <enricher>
<config>
<fmp-controller>
<service-account>....</service-account>
</fmp-controller>
</config>
</enricher> or by setting a property
or (when we finally give our XML configuration some love ;-) in XML config <resources>
....
<service-accounts>
<sa>...</sa>
<sa>...</sa>
<service-accounts>
</resources> Not yet sure about the proper XML syntax has we could combine it directly with allowing role bindings in the declaration, so that we might have a top-level XML config |
This issue has been automatically marked as stale because it has not had any activity since 90 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions! |
The idea for the service account enricher would be that if it is configured with a name, its added to the generated deployment / deployment config |
This issue has been automatically marked as stale because it has not had any activity since 90 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions! |
Right now its a little bit involved to add a ServiceAccount; you need to
sa.yml
filedeployment.yml
file if you have not already and add thespec.serviceAccountName: xyz
entryFor folks who have a
deployment.yml
already its not a biggie; but maybe it should just be flag thats enabled which would then generate an SA if its not already created (for the same name as the deployment) along with adding theserviceAccountName
to the deployment specThe text was updated successfully, but these errors were encountered: