-
Notifications
You must be signed in to change notification settings - Fork 264
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
Job Results UI should render log messages immediately #650
Comments
A few code references that should be useful:
We only attempt to render the job logs if the jobresult is completed - removing these checks should be part of the fix.
We only call A simple fix would possibly be to allow the page template to render logs from non-completed JobResults and also to refresh the page every N seconds until a terminal state is reached; a more ideal fix might be to move the job log rendering out of the page template and into the page JS to dynamically re-render the logs as the jobresult data is refreshed via AJAX. |
I will look into possible solutions to dynamically render the job logs while it's running. |
@Joezeppe hey Sammy, are you going to be able to look into this? |
@jedelman8 this could be included on the Jobs epic. |
As I understand it, the data model changes in #1030 make this much more feasible to implement. |
Should includes API for job log entry too |
#1152 is probably a pre-requisite for this. |
The simple fix is easily attainable. Unfortunately since we're using |
Environment
Steps to Reproduce
Expected Behavior
Log statements in the job should be rendered when they've been logged. Jobs can be complex and have several steps; it would be nice if the statements appeared while the user is waiting for the job to complete.
Observed Behavior
All log statements appear all at once, after the job completes.
The text was updated successfully, but these errors were encountered: