Skip to content
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

add helm chart #60

Merged
merged 11 commits into from
Nov 17, 2023
Merged

add helm chart #60

merged 11 commits into from
Nov 17, 2023

Conversation

DavidKellyonGitHub
Copy link

This helm chart will deploy a pod running a containerized version of the chatgpt-bot image. The pod connects to a running mattermost pod's service via the cluster service DNS url (e.g. http://mattermost-team-edition.mattermost.svc.clusterlocal). A mattermost instance deployed with helm will have such a service running.

This chart uses environment variables
MATTERMOST_TOKEN
OPENAI_API_KEY

and defaults for mattermost-team-edition, and the 'mattermost' namespace

@yGuy
Copy link
Owner

yGuy commented Nov 13, 2023

Sorry, I have absolutely no capabilities to verify/review and merge this. Never used nor even heard of helm charts before... Can you find someone to review this?

@m00nyONE
Copy link
Contributor

@yGuy I'm able to review this. i already created my own working helm chart for the bot.
@DavidKellyonGitHub it would be awesome if you could accept my PR to your fork. There are some issues with your current implementation that need to be worked out before merging. DavidKellyonGitHub#1

@yGuy
Copy link
Owner

yGuy commented Nov 17, 2023

Awesome! Thanks @m00nyONE and @DavidKellyonGitHub - I think this pull request needs to be updated as the changes by @m00nyONE are now part of your main branch in your fork.

Copy link
Contributor

@m00nyONE m00nyONE left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be good to go :-)

@yGuy yGuy merged commit c6e5471 into yGuy:master Nov 17, 2023
@yGuy
Copy link
Owner

yGuy commented Nov 17, 2023

Thanks, guys!

@yGuy yGuy added the enhancement New feature or request label Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants