last run summary is an inconsistent data structure
|Affected Puppet version:||Branch:||https://github.com/masterzen/puppet/tree/tickets/2.7.x/10676|
- If a run has a failed resource such that all the subsequent events don’t fire, we don’t write out ‘events 0’, we simply don’t put in the events key.
RI’s real world use of this file led him to need to create a hash with the following data and merge it in. We should be consistent so this isn’t needed.
- out of sync
We should just do something similar but with absolutely every top level item. Start with the base data structure, and merge into it, rather than building it up as it sounds like we’re doing.
#2 Updated by Brice Figureau about 2 years ago
- Branch set to https://github.com/masterzen/puppet/tree/tickets/2.7.x/10676
One of the reason we do it this way is that a report has no schema enforced. The last run summary is extracted from the current registered metrics, and those are registered depending on what happened during the run.
I’ve sent a pull request containing a step in the correct direction, at least we’ll now have all resources status defined. Check: https://github.com/puppetlabs/puppet/pull/212
#4 Updated by Jacob Helwig almost 2 years ago
- Status changed from In Topic Branch Pending Review to Merged - Pending Release
- Target version changed from 3.x to 2.7.10
Merged brices patch into 2.7.x in commit:04b027b8d4a4e13b4b9f2ee7cd4c01715ffd68e8
(#10676) Include all resource statuses in reports regardless of count Before, we would only include resource status states, and failure/success counts if there were resources that were in these various states. We now initialize the hashes used to gather the counts of these metrics so that they will include all known states, even if the count of resources that were in that state is zero. Signed-off-by: Brice Figureau <firstname.lastname@example.org> Signed-off-by: Jacob Helwig <email@example.com>