Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Print the status of finished jobs #2801

Merged
merged 6 commits into from
Jul 29, 2022
Merged

Conversation

serhiy-storchaka
Copy link
Contributor

  • neuro run, neuro attach, neuro logs, neuro exec in non-quiet
    mode now print a line with the status of finished job. It helps to notice
    jobs terminated by the OOM Killer and cancelled jobs.
  • neuro attach to finished job now prints its status instead of
    "Job is running ..."
  • Job start progress now correctly handles statuses SUSPENDED,
    CANCELLED, UNKNOWN.

* `neuro run`, `neuro attach`, `neuro logs`, `neuro exec` in non-quiet
  mode print now a line with the status of finished job. It helps to notice
  jobs terminated by the OOM Killer and cancelled jobs.
* `neuro attach` to finished job now prints its status instead of
  "Job is running ..."
* Job start progress now correctly handles statuses SUSPENDED,
  CANCELLED, UNKNOWN.
@serhiy-storchaka serhiy-storchaka added the enhancement New feature or request label Jul 28, 2022
@serhiy-storchaka serhiy-storchaka linked an issue Jul 28, 2022 that may be closed by this pull request
5 tasks
@codecov
Copy link

codecov bot commented Jul 28, 2022

Codecov Report

Merging #2801 (ae02d65) into master (d61909e) will decrease coverage by 11.18%.
The diff coverage is 33.62%.

@@             Coverage Diff             @@
##           master    #2801       +/-   ##
===========================================
- Coverage   84.65%   73.47%   -11.19%     
===========================================
  Files          72       70        -2     
  Lines       12429    12258      -171     
  Branches     2123     2119        -4     
===========================================
- Hits        10522     9006     -1516     
- Misses       1493     2941     +1448     
+ Partials      414      311      -103     
Impacted Files Coverage Δ
neuro-cli/src/neuro_cli/ael.py 16.06% <7.69%> (-33.58%) ⬇️
neuro-cli/src/neuro_cli/job.py 42.94% <8.33%> (-37.53%) ⬇️
neuro-cli/src/neuro_cli/formatters/jobs.py 82.67% <88.88%> (+4.39%) ⬆️
neuro-cli/src/neuro_cli/storage.py 34.22% <0.00%> (-46.14%) ⬇️
neuro-cli/src/neuro_cli/disks.py 51.35% <0.00%> (-40.55%) ⬇️
neuro-cli/src/neuro_cli/image.py 40.00% <0.00%> (-34.82%) ⬇️
neuro-sdk/src/neuro_sdk/_buckets.py 54.59% <0.00%> (-31.09%) ⬇️
neuro-cli/src/neuro_cli/secrets.py 61.53% <0.00%> (-26.93%) ⬇️
... and 37 more

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 d61909e...ae02d65. Read the comment docs.

@serhiy-storchaka serhiy-storchaka merged commit 05a4d12 into master Jul 29, 2022
@serhiy-storchaka serhiy-storchaka deleted the ss/print-job-result branch July 29, 2022 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Indicate the status of finished jobs
2 participants