-
Notifications
You must be signed in to change notification settings - Fork 538
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
[BUG] flyte-core helm chart missing ServiceAccounts and chart has warning messages #5361
Comments
Thank you for opening your first issue here! 🛠 |
To address the issues with the
flyteadmin:
serviceAccount:
create: true
name: flyteadmin
Would you like more detailed steps or code snippets for these configurations? Sources: This is an AI-generated response and your feedback is appreciated! Please leave a 👍 if this is helpful and 👎 if it is not. Mention @RunLLM to ask follow-up questions. See runllm.com for more! |
I have drafted proposed changes here: |
Describe the bug
flyte-core helm chart is missing some configuration functionality
Expected behavior
All Deployments should have the ability to create a ServiceAccount
Helm charts should not throw errors
Should have the ability to overwrite the deployment names (same as other charts, e.g., flyteagent)
Additional context to reproduce
No response
Screenshots
No response
Are you sure this issue hasn't been raised already?
Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: