Bug #2976
closedReports page not showing when an Error was returned
Description
The Hosts view shows an [E] status of the last host's run status, however, the reports view/page doesn't show that error (only inside the actual report itself).
Files
Updated by Greg Sutcliffe over 11 years ago
We actually have a specific unit test for this type of report, to check that Foreman sets the error state on the Host correctly. This is because Puppet will send a report with no metrics in it, so the numbers are all zeros (i.e. there are no errors because no resources were in error state).
I'm just in the middle of moving the report handling logic to the report processor (i.e. building a JSON hash for Foreman to import) - there's no reason we couldn't extend it to try to detect this case and inject a single error metric. However, that feels a little bit clunky and fragile. Thoughts?
Updated by Dominic Cleal over 11 years ago
- Status changed from New to Feedback
Digging into this a little, I can't see why this wouldn't work as-is. The error status of the host in Foreman comes from either "failed" or "failed_restarts" being non-zero, which means it should show up in the appropriate columns on the reports list.
Are you sure you're looking at reports from the right host?
A host on our own Foreman instance has a catalog build failure at the moment, shows up in an error state and shows a single "failure" event on the reports list. It's running Puppet 2.7.21 for reference.
Updated by Alex Leonhardt about 11 years ago
Hi, yes, I'm 100% sure :) ... for obvious reasons I cant just post the whole page / screen here, but it definitely does not show up as Error in the reports if the catalog wasnt able to compile. It does show in the hosts list though. We're running puppet 3.0.2 at the moment and foreman 1.1 if that helps ?
Alex
Updated by Dominic Cleal almost 11 years ago
- Related to Bug #3851: Reports with errors on the puppet master side are not shown as failed added
Updated by Benjamin Papillon almost 11 years ago
- Status changed from Feedback to New
Updated by Dominic Cleal about 10 years ago
- Status changed from New to Duplicate
I think it probably is the same, so was only recently fixed. Please let us know if it recurs with the current report processor.