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

Tested and verified a method for pushing the bot onto heroku using docker. #473

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

shim1998
Copy link

Benefits

  • Doesn't require the addition of add ons or config variables
  • Seems an alternative method of deploying the bot.
  • Similar method can be used to deploy the bot on AWS or GCP machines.

Please mail me if you want proof or have doubts about how this works.
Changes are welcomed in making this method more efficient.

@iampjeetsingh
Copy link

What is the use of Procfile, heroku.yml, and changes you made in Dockerfile?

I mean running these commands in the root directory of the bot deploys it to Heroku without any issue

sudo heroku container:login
sudo heroku container:push worker -a <heroku app name>
sudo heroku container:release worker -a <heroku app name>

@shim1998
Copy link
Author

  • docker-compose.yml file is there for a way to create and run the docker image locally via docker-compose engine. (It's optional to push that on your branch)
  • heroku.yml file was for testing out another method to deploy docker image on heroku, it works as well.
  • However, DB fails to load at times because I am using free version of heroku so I have limited cpu and ram which results in sharding problems and restart the bot which results in losing the stored users on it.

@fishy15
Copy link

fishy15 commented Aug 3, 2021

Are you able to find a way to a way to keep the data even through restarts? I was working on a fork of TLE that would use the PostgreSQL database given for free on Heroku, but I have been lazy so I haven't finished testing to make sure everything is working right. However, if this fork isn't necessary, I don't see a point continuing to work on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants