[GitHub] [pulsar] momo-jun commented on a diff in pull request #17853: [fxi][doc] Rewrite the steps of docker-deploy

[GitBox <gi...@apache.org>]

momo-jun commented on code in PR #17853:
URL: https://github.com/apache/pulsar/pull/17853#discussion_r991184207


##########
site2/docs/deploy-docker.md:
##########
@@ -2,51 +2,163 @@
 id: deploy-docker
 title: Deploy a cluster on Docker
 sidebar_label: "Docker"
----
-
-To deploy a Pulsar cluster on Docker, complete the following steps:
-1. Deploy a ZooKeeper cluster (optional)
-2. Initialize cluster metadata
-3. Deploy a BookKeeper cluster
-4. Deploy one or more Pulsar brokers
-
-## Prepare
-
-To run Pulsar on Docker, you need to create a container for each Pulsar component: ZooKeeper, BookKeeper and broker. You can pull the images of ZooKeeper and BookKeeper separately on [Docker Hub](https://hub.docker.com/), and pull a [Pulsar image](https://hub.docker.com/r/apachepulsar/pulsar-all/tags) for the broker. You can also pull only one [Pulsar image](https://hub.docker.com/r/apachepulsar/pulsar-all/tags) and create three containers with this image. This tutorial takes the second option as an example.
+—
 
+You can use two kinds of methods to deploy a Pulsar cluster on Docker.
+The first uses Docker commands, while the second uses a `docker-compose.yaml` file.
+## Deploy a cluster using Docker commands
+To deploy a Pulsar cluster on Docker, you need to complete the following steps:
+Pull a Pulsar Docker image.

Review Comment:
   ```suggestion
   1. Pull a Pulsar Docker image.
   ```
   The three steps missed the numbering. Can you pls add them all?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org