Merge ~valentinviennot/ubuntu-docker-images:update-redis-readme into ~ubuntu-docker-images/ubuntu-docker-images/+git/redis:edge

Proposed by Valentin Viennot
Status: Merged
Merge reported by: Sergio Durigan Junior
Merged at revision: da99fe9a7720d2b7ac07b23904af7d85db61bd21
Proposed branch: ~valentinviennot/ubuntu-docker-images:update-redis-readme
Merge into: ~ubuntu-docker-images/ubuntu-docker-images/+git/redis:edge
Diff against target: 18 lines (+2/-2)
1 file modified
README.md (+2/-2)
Reviewer Review Type Date Requested Status
Sergio Durigan Junior Approve
Richard Harding Pending
Review via email: mp+393898@code.launchpad.net

Description of the change

Fix typo in debug run command (missing "-it")

To post a comment you must log in.
Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

LGTM.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/README.md b/README.md
2index 879dc33..2cee474 100644
3--- a/README.md
4+++ b/README.md
5@@ -42,11 +42,11 @@ In case you need to debug what it is happening with the container you can run `d
6 $ docker exec -it <name_of_the_container> /bin/bash
7 ```
8
9-With this same image, you can launch another container with the `redis-cli` program,
10+With this same image, you can launch an interactive container with the `redis-cli` program,
11 and connect to the `redis-server` that is running in the first container.
12
13 ```
14-$ docker run --network redis-network --rm squeakywheel/redis:edge redis-cli -h redis-container
15+$ docker run -it --network redis-network --rm squeakywheel/redis:edge redis-cli -h redis-container
16 redis:6379> AUTH mypassword
17 OK
18 redis:6379> PING

Subscribers

People subscribed via source and target branches