Uploaded image for project: 'RHEL Conversions'
  1. RHEL Conversions
  2. RHELC-1094

Expand pre-conversion report message attributes

XMLWordPrintable

    • 1.4 GA
    • Testable

      Messages need to be usable by third party tools, not just convert2rhel.  Specifically, insights for the Q3 mvp and  satellite will need the ability to construct messages from what convert2rhel returns.  To do that, we need convert2rhel to provide four message fields in the assessment json.

      Doing this means that we need to construct a complete message from the four fields inside of convert2rhel as well.

      Acceptance Criteria

      • In the action framework:
        • In ActionMessageBase split the message parameter into four new attributes, title, description, diagnosis, and remediation which are set from new parameters to {}init{}.  These variables are strings holding those four parts of any message that we return.
        • These attributes are included in the dictionary representation of ActionMessage and ActionResult.
        • The Action.set_result() and Action.add_message() methods each take new parameters to set these instead of the current single message parameter..
      • In convert2rhel.action.report.summary()
        • construct the message from those values.
          • We will combine the four strings to make a message in the report similar to what is currently displayed.
          • the string should be valid markdown but we will just display the plain text.  It's insights and other external tools that should be able to rely on it being markdown.
      • In the action plugins:
        • For each result and message that  the Actions currently emit, we need to split the message element into four parts for title, description, diagnosis, and remediation.
        • title and description should be static data (nothing determined at runtime).  diagnosis and remediation can use variables determined at runtime to complete their message.
      • Deprecate and remove (when we can) the message field from the Action return data.  Ask at cabal, we may be able to remove this for the upcoming release (since we did not have json output before AND assessment as a whole is considered experimental)

       

            prwatson@redhat.com Preston Watson
            tkuratom@redhat.com Toshio Kuratomi
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: