AutoGPT icon indicating copy to clipboard operation
AutoGPT copied to clipboard

Fixed issues with running redis command due to the use of the redis-s…

Open V0nMis3s opened this issue 1 year ago • 3 comments

…tack-server image instead of the redis-stack one. Added explicit redis arguments in order to define the password using the --requirepass argument. Added example redis password in the docker-compose and .env.template files.

Background

Fixed issues with running redis command due to the use of the redis-stack-server image instead of the redis-stack one.

Changes

  • [x] Modified the docker-compose.yml file to update the redis image.
  • [x] Added explicit redis arguments in order to define the password using the --requirepass argument.
  • [x] Added example redis password in the docker-compose and .env.template files.

Documentation

No need for further documentation. The changes are already covered by the existing one.

Test Plan

n/a

PR Quality Checklist

  • [x] My pull request is atomic and focuses on a single change.
  • [x] I have thoroughly tested my changes with multiple different prompts.
  • [x] I have considered potential risks and mitigations for my changes.
  • [x] I have documented my changes clearly and comprehensively.
  • [x] I have not snuck in any "extra" small tweaks changes

V0nMis3s avatar Apr 17 '23 15:04 V0nMis3s

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

github-actions[bot] avatar Apr 19 '23 17:04 github-actions[bot]

This is a mass message from the AutoGPT core team. Our apologies for the ongoing delay in processing PRs. This is because we are re-architecting the AutoGPT core!

For more details (and for infor on joining our Discord), please refer to: https://github.com/Significant-Gravitas/Auto-GPT/wiki/Architecting

p-i- avatar May 05 '23 00:05 p-i-

Sorry if this sounds a little weird, but maybe it's worth trying: would the folks with open redis PRs consider mutually reviewing their PRs to gather some feedback from people actually using redis ?

Boostrix avatar May 16 '23 15:05 Boostrix

Redis has been removed from the default docker config

Pwuts avatar Jun 07 '23 00:06 Pwuts