Uploaded image for project: 'Puppet'
  1. Puppet
  2. PUP-9093

Make Reported Event objects stringify rich data

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Normal
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: PUP 6.0.0
    • Component/s: None
    • Labels:
      None
    • Template:
    • Team:
      Platform Core
    • Release Notes:
      Bug Fix
    • Release Notes Summary:
      Hide
      With rich-data turned on for a catalog (now the default) a Report could contain rich data in reported events, and that was not good because nothing downstream from the agent is prepared to handle rich data. This is now fixed so that data in reported events are stringified when needed.
      Show
      With rich-data turned on for a catalog (now the default) a Report could contain rich data in reported events, and that was not good because nothing downstream from the agent is prepared to handle rich data. This is now fixed so that data in reported events are stringified when needed.
    • QA Risk Assessment:
      Needs Assessment

      Description

      ORIGINAL


      An option is needed in the ToDataConverter so that warnings for stringification of rich data is suppressed. This is needed because there is legitimate use cases where stringification is required.

      UPDATE


      The legitimate use case was in Event objects in reports. There, rich-data cannot be allowed because nothing downstream from the report is prepared to handle rich data. The specification for reports even say that all values in Events (before/after) are supposed to be strings, but that is a lie since it is any JSON data type (but not any additional "rich data" encoding.

        Attachments

          Issue Links

            Activity

              jsd-sla-details-panel

                People

                • Assignee:
                  henrik.lindberg Henrik Lindberg
                  Reporter:
                  henrik.lindberg Henrik Lindberg
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: