PR#68: Add flask-http-requests info to the metrics - cts

Ozan Unsal • 2 years ago

59080a4

cts/__init__.py

file modified

		`@@ -33,6 +33,7 @@`
		`from cts.proxy import ReverseProxy`
		`from cts.errors import NotFound, Forbidden`

		`+ from prometheus_flask_exporter.multiprocess import GunicornInternalPrometheusMetrics`
		`import pkg_resources`

		`try:`
		`@@ -59,6 +60,11 @@`

		`init_auth(login_manager, conf.auth_backend)`

		`+ metrics = GunicornInternalPrometheusMetrics.for_app_factory(`
		`+ group_by="endpoint", default_prefix="cachito_flask"`
lsedlar commented 2 years ago The prefix should probably be changed :)
		`+ )`
		`+ metrics.init_app(app)`
		`+`

		`def json_error(status, error, message):`
		`response = jsonify({"status": status, "error": error, "message": message})`

cts/metrics.py

file modified

+7 -5

		`@@ -37,13 +37,15 @@`

		`# This environment variable should be set if deployment uses multiple`
		`# processes.`
		`- if not os.environ.get("prometheus_multiproc_dir"):`
		`- os.environ.setdefault("prometheus_multiproc_dir", tempfile.mkdtemp())`
		`-`
		`-`
		`+ multiproc_temp_dir = os.environ.get("PROMETHEUS_MULTIPROC_DIR")`
lsedlar commented 2 years ago With uppercase var name I'm getting errors. If I change it to lowercase, the application starts fine.
		`+ if not multiproc_temp_dir:`
		`+ multiproc_temp_dir = tempfile.mkdtemp()`
		`+ if not os.path.isdir(multiproc_temp_dir):`
		`+ os.makedirs(multiproc_temp_dir)`
		`+ os.environ["PROMETHEUS_MULTIPROC_DIR"] = multiproc_temp_dir`
		`registry = CollectorRegistry()`
		`ProcessCollector(registry=registry)`
		`- multiprocess.MultiProcessCollector(registry)`
		`+ multiprocess.MultiProcessCollector(registry, path=multiproc_temp_dir)`


		`class ComposesCollector(object):`

requirements.txt

file modified

start_cts_from_here

file modified

		`@@ -2,5 +2,6 @@`

		`export PYTHONPATH=.`
		`export CTS_DEVELOPER_ENV=1`
		`+ export PROMETHEUS_MULTIPROC_DIR=/tmp/prometheus_multiproc_dir`

		`python cts/manage.py runssl`