rustest: added limits to docker compose

HackerDom · Nov 4, 2023 · 4953eca · 4953eca
1 parent 2b89ec3
commit 4953eca
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 5 deletions.
diff --git a/services/rustest/docker-compose.yml b/services/rustest/docker-compose.yml
@@ -16,17 +16,37 @@ services:
       - '--initial-cluster-token=etcd-cluster-token'
     volumes:
       - etcd1:/etcd_data
-    ports:
-      - 2379:2379
-    restart: always
+    restart: unless-stopped
+    ulimits:
+      nofile:
+        soft: 32768
+        hard: 32768
+    logging:
+      options:
+        max-file: "2"
+        max-size: "5m"
+    mem_limit: 1G
+    memswap_limit: 1G
+    cpu_count: 1
 
   rustest:
     build: "./"
     ports:
       - 13337:3000
-    restart: always
+    restart: unless-stopped
     environment:
       - ETCD_HOST=http://etcd:2379
+    ulimits:
+      nofile:
+        soft: 32768
+        hard: 32768
+    logging:
+      options:
+        max-file: "2"
+        max-size: "5m"
+    mem_limit: 1G
+    memswap_limit: 1G
+    cpu_count: 1
 
 volumes:
   etcd1:
diff --git a/services/rustest/src/storage.rs b/services/rustest/src/storage.rs
@@ -112,7 +112,7 @@ impl ETCDRustestStorage {
             current_min_revision = resp.kvs.last().unwrap().create_revision + 1;
 
             let mut kvs: &[KeyValue] = &resp.kvs;
-            while kvs.len() > 0 {
+            while !kvs.is_empty() {
                 let offset_bound_revision = kvs.first().unwrap().create_revision;
 
                 offset_to_revision.insert(current_offset as u64, offset_bound_revision);