[GitHub] [pulsar] xiaogang-sn commented on a diff in pull request #17853: [fix][doc] Rewrite the steps of docker-deploy

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

xiaogang-sn commented on code in PR #17853:
URL: https://github.com/apache/pulsar/pull/17853#discussion_r992131738


##########
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.
+—

Review Comment:
   Shouldn't there be a sanity check CI job for any documentation change in the repo if full pipeline is too expensive?



-- 
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