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.

Sign up for GitHub

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

Jump to bottom

Add session watch command like gh run watch #395

Closed
achimnol opened this issue Mar 21, 2022 · 2 comments · Fixed by #440
Closed

Add session watch command like gh run watch #395

achimnol opened this issue Mar 21, 2022 · 2 comments · Fixed by #440
Assignees
@rapsealk
Labels
comp:cli Related to CLI component effort:easy Need to understand only a specific region of codes (good first issue, easy).
Milestone
22.09

Comments

@achimnol
Copy link
Member

For better CLI scriptability, let's add backend.ai session watch command to let it display the event stream of the target session and terminates with a proper exit code (e.g., 0 for task-success and 1 for others for batch-type sessions) when it finishes.

We could implement this feature entirely in the client-side, using the existing codebase of backend.ai events.

Reference: https://cli.github.com/manual/gh_run_watch
@achimnol achimnol added type:feature Add new features comp:cli Related to CLI component labels Mar 21, 2022
@achimnol achimnol added this to the 22.09 milestone Mar 21, 2022
@achimnol achimnol added the effort:easy Need to understand only a specific region of codes (good first issue, easy). label Mar 21, 2022
@rapsealk
Copy link
Member

rapsealk commented May 23, 2022 •
edited
Loading

For better understanding, gh run watch works like this:

스크린샷 2022-05-23 오후 2 44 41

스크린샷 2022-05-23 오후 2 44 50

2022-05-24.11.41.26.mov

@rapsealk
Copy link
Member

rapsealk commented May 24, 2022 •
edited
Loading

2022-05-25.5.48.38.mov

Is this what it expected to be?

@rapsealk rapsealk self-assigned this May 25, 2022
@rapsealk rapsealk mentioned this issue May 27, 2022
Closed
This was referenced Jun 14, 2022
Merged
Closed
@achimnol achimnol removed the type:feature Add new features label Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rapsealk rapsealk

Labels
comp:cli Related to CLI component effort:easy Need to understand only a specific region of codes (good first issue, easy).
Projects
None yet
Milestone
22.09
Development

Successfully merging a pull request may close this issue.

feat: Add session watch command to client-py lablup/backend.ai
2 participants
@achimnol @rapsealk