Replace node-zookeeper-client with zookeeper (#6049)

microsoft · May 5, 2021 · 5897efc · 5897efc
1 parent ba7b9a3
commit 5897efc
Show file tree

Hide file tree

Showing 8 changed files with 213 additions and 73 deletions.
diff --git a/server/routerlicious/README.md b/server/routerlicious/README.md
@@ -84,6 +84,11 @@ An example developer flow would be to:
 Then use another command window to deliver the changes:
 * `npm run build` - build
 * `docker-compose restart {modified service}` - allow the container to pick up the changes stored on the local machine
+
+> **Note:** Some dependencies are required to be installed and built on the OS they are running on.
+> When you update one of these dependencies (e.g. change versions) you will need to fully restart the service using `npm stop && npm start`.
+> Current dependencies with this requirement: zookeeper
+
 or
 
 ### Standalone

diff --git a/server/routerlicious/docker-compose.dev.yml b/server/routerlicious/docker-compose.dev.yml
@@ -3,21 +3,28 @@ services:
     alfred:
         volumes:
             - .:/usr/src/server
+            - /usr/src/server/node_modules/zookeeper
     deli:
         volumes:
             - .:/usr/src/server
+            - /usr/src/server/node_modules/zookeeper
     scriptorium:
         volumes:
             - .:/usr/src/server
+            - /usr/src/server/node_modules/zookeeper
     copier:
         volumes:
             - .:/usr/src/server
+            - /usr/src/server/node_modules/zookeeper
     scribe:
         volumes:
             - .:/usr/src/server
+            - /usr/src/server/node_modules/zookeeper
     foreman:
         volumes:
             - .:/usr/src/server
+            - /usr/src/server/node_modules/zookeeper
     riddler:
         volumes:
             - .:/usr/src/server
+            - /usr/src/server/node_modules/zookeeper