search

beacon-biosignals / K8sClusterManagers.jl

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

Redirect worker output via `kubectl attach` #58

Open omus opened 3 years ago

omus commented 3 years ago

Alternative fix for #56 which currently is non-functional. The main issue seems to be that I can't seem to get stdin/stdout to work correctly with kubectl attach. There is one commit on this branch that works due but it only uses stdout and requires us to slow the worker down such that the output of the worker IP/port can be captured by the manager.

codecov[bot] commented 3 years ago

Codecov Report

Merging #58 (7736f41) into main (6df57b4) will decrease coverage by 19.63%. The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##             main      #58       +/-   ##
===========================================
- Coverage   67.63%   48.00%   -19.64%     
===========================================
  Files           4        4               
  Lines         173      175        +2     
===========================================
- Hits          117       84       -33     
- Misses         56       91       +35
Impacted Files Coverage Δ
src/K8sClusterManagers.jl 100.00% <ø> (ø)
src/native_driver.jl 6.55% <0.00%> (-0.47%) :arrow_down:
src/pod.jl 63.82% <ø> (-33.05%) :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 6df57b4...7736f41. Read the comment docs.