beacon-biosignals / K8sClusterManagers.jl

A Julia cluster manager for Kubernetes
Other
31 stars 5 forks source link

Update GitHub action dependencies #95

Closed omus closed 1 year ago

omus commented 1 year ago

Some package CI upkeep

codecov[bot] commented 1 year ago

Codecov Report

Merging #95 (a6999bc) into main (905c995) will decrease coverage by 17.96%. The diff coverage is n/a.

:exclamation: Current head a6999bc differs from pull request most recent head 9ffd932. Consider uploading reports for the commit 9ffd932 to get more accurate results

@@             Coverage Diff             @@
##             main      #95       +/-   ##
===========================================
- Coverage   67.05%   49.10%   -17.96%     
===========================================
  Files           4        4               
  Lines         170      167        -3     
===========================================
- Hits          114       82       -32     
- Misses         56       85       +29     

see 2 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

omus commented 1 year ago

Cluster tests are currently failing with:

 test-oom: Test Failed at /home/runner/work/K8sClusterManagers.jl/K8sClusterManagers.jl/test/cluster.jl:402
  Expression: pod_status(worker_pod) == ("terminated" => "OOMKilled")
   Evaluated: "terminated" => "Error" == "terminated" => "OOMKilled"

As #96 seems to need some of these changes I'll just proceed and fix this later.