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 notes on maintaining Docker based instances #660

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

Conversation

kobeBigs
Copy link

@kobeBigs kobeBigs commented Dec 8, 2022

Added instructions on how to;

  1. check running processes,
  2. restart processes/containers
  3. check and flush redis
  4. change number of workers

@justinclift
Copy link
Member

This seems like it would be useful, though it'd need a bit of updating to match the current commands. 😄

@kobeBigs
Copy link
Author

kobeBigs commented Aug 5, 2024

This seems like it would be useful, though it'd need a bit of updating to match the current commands. 😄

@justinclift current commands? 😄 let me look into that

@justinclift
Copy link
Member

Yeah, things like:

  • Use docker compose (ie docker version 2 syntax) rather than docker-compose (docker version 1 syntax).

  • Celery was replaced with RQ, so would need updating to match

  • For the command to access PG, it's probably better to use the compose command instead:

    $ docker compose exec -u postgres postgres psql
    

Note that I'm kind of expecting the content of this page would have changed a bit since the PR was created. If so, then this PR would need updating to be based upon the current contents of the page (etc). Hopefully that makes sense. 😄

@kobeBigs
Copy link
Author

kobeBigs commented Aug 5, 2024

@justinclift you are right. I will bring in the changes in master, try out the changes and revise.
Also, for the sake of folks still using Celery and docker-compose, I will bring in the current commands instead of replacing. What do you think?

@justinclift
Copy link
Member

Good point. Those commands are likely still useful for people running older Redash releases. 😄

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.

2 participants