Bug #2976
closed
Reports page not showing when an Error was returned
Added by Alex Leonhardt over 11 years ago.
Updated about 10 years ago.
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
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?
- 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.
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
- Related to Bug #3851: Reports with errors on the puppet master side are not shown as failed added
- Status changed from Feedback to New
how is this different to #3851?
- 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.
Also available in: Atom
PDF