Refactor: Consolidate log streaming logic into centralized `log_utils.follow_logs()`

skypilot-org / skypilot

SkyPilot: Run AI and batch jobs on any infra (Kubernetes or 12+ clouds). Get unified execution, cost savings, and high GPU availability via a simple interface.

https://skypilot.readthedocs.io

Apache License 2.0

6.81k stars 513 forks source link

Refactor: Consolidate log streaming logic into centralized `log_utils.follow_logs()` #4323

Closed andylizf closed 1 week ago

andylizf commented 1 week ago

Continue #4292

Tested (run the relevant ones):

[ ] Code formatting: bash format.sh
[ ] Any manual or new tests for this PR (please specify below)
[ ] All smoke tests: pytest tests/test_smoke.py
[ ] Relevant individual smoke tests: pytest tests/test_smoke.py::test_fill_in_the_name
[ ] Backward compatibility tests: conda deactivate; bash -i tests/backward_compatibility_tests.sh

andylizf commented 1 week ago

@cblmemo PTAL, thanks!