Skip to content

Conversation

@vitodb
Copy link
Contributor

@vitodb vitodb commented Apr 27, 2022

This PR updates the Docker entrypoint script used by the CI to allow all unit test for to run.
Specifically this PR unset PROMETHEUS_MULTIPROC_DIR env variable.
This var is set in the framework Dockerfile, and this is causing test_check_metrics_env_var_unset test to fail.

@codecov
Copy link

codecov bot commented Apr 27, 2022

Codecov Report

Merging #646 (fafd699) into master (e10fe5a) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #646      +/-   ##
==========================================
+ Coverage   97.51%   97.55%   +0.03%     
==========================================
  Files          49       49              
  Lines        2941     2941              
  Branches      469      469              
==========================================
+ Hits         2868     2869       +1     
  Misses         49       49              
+ Partials       24       23       -1     
Flag Coverage Δ
python-3.10 97.31% <ø> (ø)
python-3.6 97.09% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
.../decisionengine/framework/engine/DecisionEngine.py 93.62% <0.00%> (+0.17%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e10fe5a...fafd699. Read the comment docs.

@vitodb vitodb changed the title Update Docker entrypoint script Update Docker CI entrypoint script Apr 28, 2022
@mambelli mambelli merged commit cbffa99 into HEPCloud:master May 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants