Skip to content
This repository was archived by the owner on Nov 25, 2022. It is now read-only.

Use the redis server in the sytest image #86

Merged
merged 1 commit into from
Aug 11, 2020
Merged

Use the redis server in the sytest image #86

merged 1 commit into from
Aug 11, 2020

Conversation

richvdh
Copy link
Member

@richvdh richvdh commented Aug 6, 2020

As of matrix-org/sytest#928, the sytest-synapse images
support a redis server, so we no longer need to docker-compose.

@richvdh
Use the redis server in the sytest image
8186d3a 
As of matrix-org/sytest#928, the sytest-synapse images
support a redis server, so we no longer need to docker-compose.
@richvdh richvdh requested review from erikjohnston and removed request for erikjohnston August 6, 2020 15:34
erikjohnston
erikjohnston approved these changes Aug 11, 2020
View reviewed changes
Copy link
Member

@erikjohnston erikjohnston left a comment

Choose a reason for hiding this comment

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

I think this works. After merge we should double check the test output logs to make sure it is connecting to redis

@richvdh richvdh merged commit c3a315f into master Aug 11, 2020
@richvdh richvdh deleted the rav/update_redis_pipeline branch August 11, 2020 13:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
1 more reviewer

@erikjohnston erikjohnston erikjohnston approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@richvdh @erikjohnston