search

beacon-biosignals / K8sClusterManagers.jl

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

Refactor `Distributed.launch` #47

Closed omus closed 3 years ago

omus commented 3 years ago

Since we've removed Kuber.jl we no longer need to have special logic to handle API throttling exceptions. This overall has allowed the Distributed.launch method to be much cleaner.

Also, I fixed an issue with the reporting in the "test-multi-addprocs" test.

codecov[bot] commented 3 years ago

Codecov Report

Merging #47 (8054462) into main (6d8f969) will increase coverage by 16.24%. The diff coverage is 40.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##             main      #47       +/-   ##
===========================================
+ Coverage   52.85%   69.10%   +16.24%     
===========================================
  Files           4        4               
  Lines         140      123       -17     
===========================================
+ Hits           74       85       +11     
+ Misses         66       38       -28
Impacted Files Coverage Δ
src/native_driver.jl 10.25% <0.00%> (+5.77%) :arrow_up:
src/pod.jl 95.31% <90.90%> (-0.92%) :arrow_down:

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 6d8f969...8054462. Read the comment docs.