Skip to content

Commit a78fb5d

Browse files
authored
Merge pull request #190 from WengLab-InformaticsResearch/jaeger
Remove mysql query from /health check
2 parents 9ba4acb + b4961df commit a78fb5d

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

cohd/cohd.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -333,7 +333,10 @@ def api_call(service=None, meta=None, query=None, version=None):
333333
result = 'meta not recognized', 400
334334
elif service == 'health':
335335
# elastic load balancing health check
336-
mysql_health = query_cohd_mysql.health()
336+
# Disabling the MySQL health check since it's overloading OTEL and I don't see a way to exclude
337+
# it from OTEL collection, and currently these health checks don't do much for COHD.
338+
# mysql_health = query_cohd_mysql.health()
339+
mysql_health = True
337340
if not mysql_health:
338341
result = 'unhealthy MySQL server', 503
339342
else:

0 commit comments

Comments
 (0)