Bug #12340
closed
Resend button should refresh the status screen correctly
Description
I'm not sure what is expected in this case.. but when I selected 'Resend' from discovery status screen, console refreshes and status changed from 'Success' to 'N/A (use Status to update). I believe 'Resend' tries to re-register the host with server ?
And when I selected 'Status' button to update the status then again nothing happens. In any case, when we are mentioning "(use Status to update)" then status should be updated to 'Success' as per my understanding.
Files
Updated by Sachin Ghai over 8 years ago
- Related to Tracker #10294: PXEless discovery feature added
Updated by Lukas Zapletal over 8 years ago
- Subject changed from On selecting 'Resend' button from Discovery status screen for a PXE based discovered host, change the Status to N/A to Resend button should refresh the status screen correctly
This is a little bit ackward, but Resend button only sends notification to discovery-register service to send facts. We need to wait for it to complete and after few seconds you need to refresh the screen. For technical reasons, I can't refresh the screen, therefore user must do this via Refresh button.
I might try to find a better way of IPC - maybe waiting for the result file. Let's see.
Also note that in PXE-less the Resend button does not even work (error), I disabled it for PXE-less in this patch: #12322. Nicer error will appear there.
Updated by Lukas Zapletal over 8 years ago
- Status changed from New to Ready For Testing
- Assignee set to Lukas Zapletal
- Pull request https://github.com/theforeman/foreman-discovery-image/pull/43 added
Updated by The Foreman Bot over 8 years ago
- Pull request https://github.com/theforeman/foreman-discovery-image/pull/45 added
Updated by Anonymous over 8 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset foreman-discovery-image|0febbb02fea80d24b3229be88ae5ad087a193eb2.