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

vdk-server: logs url in quickstart vdk #2018

Merged
merged 4 commits into from
May 10, 2023

Conversation

murphp15
Copy link
Collaborator

@murphp15 murphp15 commented May 5, 2023

Why

In quickstart vdk we want to be able to see the logs of a job.

What

Read the logs from the api server

How was this tested

Ran it locally.

The logs endpoint in the api is not brilliant. It doesn't return anything unless the job is complete

images showing links in UI to links redirect link:
Screenshot 2023-05-10 at 09 46 55
Screenshot 2023-05-10 at 09 47 01

image showing logs for job not yet started:
Screenshot 2023-05-10 at 09 44 39

image showing logs for completed job:
Screenshot 2023-05-10 at 09 44 45

Signed-off-by: murphp15 <murphp15@tcd.ie>
Copy link
Contributor

@ivakoleva ivakoleva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can provide screenshot of how the link gets visualized, and how it looks like in the browser, to provide a quick context of the change

Copy link
Contributor

@DeltaMichael DeltaMichael left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regarding the endpoint for logs, I'm not sure if streaming live logs was scoped in the past. This might be a different issue entirely. We need further discussion here.

Signed-off-by: murphp15 <murphp15@tcd.ie>
@murphp15
Copy link
Collaborator Author

@ivakoleva @DeltaMichael
I updated the VEP and created the following two tickets: #2041, #2042.
I am going to merge this now.

@murphp15 murphp15 enabled auto-merge (squash) May 10, 2023 13:51
@murphp15 murphp15 merged commit 79edece into main May 10, 2023
@murphp15 murphp15 deleted the person/murphp15/logs_url_in_quickstart_vdk branch May 10, 2023 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants