[GitHub] [incubator-inlong] dockerzhang commented on a change in pull request #2511: [INLONG-2509][Feature][Audit][DockerCompose] Add support for Audit on Docker Compose

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

dockerzhang commented on a change in pull request #2511:
URL: https://github.com/apache/incubator-inlong/pull/2511#discussion_r806853698



##########
File path: docker/docker-compose/docker-compose.yml
##########
@@ -123,4 +136,20 @@ services:
     ports:
       - "8008:8008"
     volumes:
-      - ./collect-data:/data/collect-data
\ No newline at end of file
+      - ./collect-data:/data/collect-data
+
+  audit:
+    image: inlong/audit:latest
+    container_name: audit
+    depends_on:
+      mysql:
+        condition: service_healthy

Review comment:
       it's better to use another PR to adapt the 3.0 docker-compose after this issue is fixed.

##########
File path: docker/docker-compose/docker-compose.yml
##########
@@ -33,6 +33,19 @@ services:
       - ./mysql:/var/lib/mysql
       - ./sql:/docker-entrypoint-initdb.d
 
+  pulsar:
+    image: apachepulsar/pulsar
+    container_name: pulsar
+    environment:
+      - PULSAR_MEM="-Xms256m -Xmx512m -XX:MaxDirectMemorySize=1g"
+    ports:
+      - "6650:6650"
+      - "8181:8080"

Review comment:
       the 8080 port for pulsar admin url is not used by audit ,we can remove this port mapping.




-- 
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@inlong.apache.org

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