Issue #7967 has been updated by Daniel Pittman.
Jacob Helwig wrote: > Are we worried about syntactic errors, or semantic errors? Both? Both, though semantic errors dominate. > Might be worth checking how badly adding a simple "is the report > syntactically correct" (is it valid YAML) check before spooling the import > off into the background? That probably won't be in the first cut, because the time and memory cost of performing a YAML parse is so crazy-high. If that turns out to be a common error in practice we can revise the decision. Does that match your expectations and experience? Generally, bad YAML is only going to come out of really strange bugs in Puppet, or deliberate attack, where the instant report of the syntax problems won't really help, I think. ---------------------------------------- Feature #7967: Add way to view report import failures in Dashboard https://projects.puppetlabs.com/issues/7967 Author: Matt Robinson Status: Accepted Priority: Normal Assignee: Category: Target version: 1.x Keywords: Branch: Affected URL: Affected Dashboard version: Currently Dashboard receives report uploads and processes them all in one go, but with the work being done to take report submissions and process them in the background (#7938) errors in processing reports won't propagate back to the report submitter. This means if Puppet sends a bad report, it will never find out. There needs to be some way to find out when a bad report it processed, so there should be some UI in dashboard itself to give this feedback. -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
