Bug #12065
closed
Refresh host tab in job invocation detail together with overall status
Added by Ohad Levy about 9 years ago.
Updated over 6 years ago.
Description
when reviewing a job status, I could see multiple requests in the same second for refresh.
this is caused due to how the refresh is triggered (every time you get a response from the server, you immediately start to ask for another refresh).
IMHO this logic should be extracted from the rails response code, and managed once within the initial js that is loaded.
related, the host tab is not updated, and requires a manual update.
I don't follow, with every response there's 1 second timeout so next refresh is in interval = (response time + 1 second). If the refresh is triggered more frequently, it might be bug in JS but not caused by the fact that JS is sent in response?
Yes you are right, was reading it wrong.
then in this case, we can convert this issue to the host tab not updating? (maybe we can update it on click vs in the background)? and refresh it if its active?
- Subject changed from job review refresh interval is too high to Refresh host tab in job invocation detail together with overall status
- Status changed from New to Assigned
- Assignee set to Marek Hulán
- Target version set to 87
I think it will be more efficient to do the update together in same request for which we do the chart update. We won't refresh the whole table, just hosts that hasn't finish the run yet. Otherwise we'd have to deal with pagination, also is user would select some text it would remove the selection etc...
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman_remote_execution/pull/52 added
- Pull request deleted (
- Blocks Feature #11865: Job invocation results host list improvements added
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
- Translation missing: en.field_release set to 99
- Translation missing: en.field_release changed from 99 to 101
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by 
Updated by