Serve metrics for Prometheus
All threads resolved!
All threads resolved!
Closes #1618 (closed)
Merge request reports
Activity
changed milestone to %Arkindex 1.5.2
requested review from @babadie
assigned to @vrigal
- Resolved by Valentin Rigal
I tested with the Dockerfile build, but I did not managed with the binary build because of a license check error (even after upgrading the keygen policy):
[ERROR] License check failed: No fingerprint generated
.
added 8 commits
-
4079bbc2...018ecb1f - 4 commits from branch
master
- 067158ac - Update builds
- 06835b98 - Serve metrics
- 7087d20f - Update README
- 5bc6c363 - Add a simple test case
Toggle commit list-
4079bbc2...018ecb1f - 4 commits from branch
- Automatically resolved by Valentin Rigal
- Automatically resolved by Valentin Rigal
- Automatically resolved by Valentin Rigal
added 10 commits
-
557a505d - 1 commit from branch
master
- 79fe8e36 - Update builds
- 7aff19da - Serve metrics
- 931aa9d8 - Update README
- bd941fca - Add a simple test case
- a3f951d3 - Detect application port
- b5da35fe - Move middleware to the top
- 817355cd - Simply use a view check
- e5f529f6 - Suggestions
- 6df97eff - Update tests
Toggle commit list-
557a505d - 1 commit from branch
I noticed that
./arkindex/manage.py gunicorn
did not work, crashing on the int parse of None (default arg for metrics port).I decided to simplify the logic:
- configure the port from YAML config, so it's more standard
- always have the metrics port set, default to 3000
- use the same setting in view & gunicorn
- no
--metrics-port
cli arg
I also set the response content type to
text/plain
as it's expected by the prometheus client (I did not tell you do to it though).
Please register or sign in to reply