Extend worker logging ports list

Added new worker logging ports (5595 to 5599) to both `tests/test_config.py` and `config/nowcast.yaml`. This is to prevent intermittent worker failures due to all logging ports in use during collecting of obs in the early morning. Additionally, fixed a comment in `nowcast.yaml` to include double quotes around a colon character in the zmq section.
SalishSeaCast · Jul 30, 2024 · 3a2b11f · 3a2b11f
1 parent cc30816
commit 3a2b11f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/config/nowcast.yaml b/config/nowcast.yaml
@@ -1203,9 +1203,9 @@ zmq:
       workers: [
         5562, 5563, 5564, 5565, 5566, 5567, 5568, 5569,
         5570, 5571, 5572, 5573, 5574, 5575, 5576, 5577, 5578, 5579,
-        5590, 5591, 5592, 5593, 5594
+        5590, 5591, 5592, 5593, 5594, 5595, 5596, 5597, 5598, 5599,
       ]
-    # **host:port pairs in lists must be quoted to protect : characters**
+    # **host:port pairs in lists must be quoted to protect ":" characters**
       run_NEMO: ['salish.eos.ubc.ca:5556', '206.12.100.213:5556']
       watch_NEMO: ['salish.eos.ubc.ca:5557', '206.12.100.213:5557']
       make_ww3_wind_file: '206.12.100.213:5570'

diff --git a/tests/test_config.py b/tests/test_config.py
@@ -257,6 +257,11 @@ def test_log_aggregator_ports(self, prod_config):
                 5592,
                 5593,
                 5594,
+                5595,
+                5596,
+                5597,
+                5598,
+                5599,
             ],
             "run_NEMO": ["salish.eos.ubc.ca:5556", "206.12.100.213:5556"],
             "watch_NEMO": ["salish.eos.ubc.ca:5557", "206.12.100.213:5557"],
-Original file line number
+Diff line change
@@ Expand Up / @@ -257,6 +257,11 @@ def test_log_aggregator_ports(self, prod_config): @@
 ,
 ,
 ,
+,
+,
+,
+,
+,
                 ],
                 "run_NEMO": ["salish.eos.ubc.ca:5556", "206.12.100.213:5556"],
                 "watch_NEMO": ["salish.eos.ubc.ca:5557", "206.12.100.213:5557"],
@@ Expand Down @@