Bug #12065
closed
Refresh host tab in job invocation detail together with overall status
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.
Updated by Marek Hulán over 8 years ago
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?
Updated by Ohad Levy over 8 years ago
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?
Updated by Marek Hulán over 8 years ago
- 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...
Updated by The Foreman Bot over 8 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman_remote_execution/pull/52 added
- Pull request deleted (
)
Updated by Marek Hulán over 8 years ago
- Blocks Feature #11865: Job invocation results host list improvements added
Updated by Marek Hulán over 8 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
- translation missing: en.field_release set to 99
Updated by Ivan Necas over 8 years ago
- translation missing: en.field_release changed from 99 to 101