-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat: add optional ServiceAccount resource #4
Conversation
Alternatively... if the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Matt, this looks good, thanks for the contribution!
@wrenhunter Thanks! I did forget to update the README. Is there another process to update the helm repo with the updated chart? |
Hi @MattMencel, we plan to update the README with some other stuff, and can add a note about the ServiceAccount. Is that what you meant by updating the repo further, or something else? |
@wrenhunter I don't see the new chart yet in the Helm Repo. So just wondered when it would be published. helm search repo strongdm
NAME CHART VERSION APP VERSION DESCRIPTION
strongdm/sdm-client 0.1.0 1.0.0 strongDM Client Container
strongdm/sdm-relay 0.1.1 1.0.0 strongDM Relay |
@MattMencel I see what you mean. I'll need to track down a colleague with access to Artifact, so that we can push the changes. I'll let you know when that happens, probably next week. |
Hi @MattMencel, the Artifact repo is updated. We're seeing an issue with the repo icons there, but no effect on your update, we'll get that sorted this week. |
No description provided.