Uploaded image for project: 'Couchbase Kubernetes'
  1. Couchbase Kubernetes
  2. K8S-107

Improve readability of spec update msg

    XMLWordPrintable

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Done
    • None
    • 1.0.0
    • operator

    Description

      Current logging is an escaped json string.

      time="2018-01-09T21:24:24Z" level=info msg="spec update: Old Spec:" cluster-name=cb-example module=cluster time="2018-01-09T21:24:24Z" level=info msg="{" cluster-name=cb-example module=cluster time="2018-01-09T21:24:24Z" level=info msg="    \"baseImage\": \"couchbase/server\"," cluster-name=cb-example module=cluster

      This makes debugging what changed a harder than necessary.  If possible, let's try to print out as yaml or as a formatted way.

       

       

       

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          Ascending order - Click to sort in descending order
          tommie Tommie McAfee created issue -
          mikew Mike Wiederhold [X] (Inactive) made changes -
          Field Original Value New Value
          Rank Ranked higher
          mikew Mike Wiederhold [X] (Inactive) made changes -
          Fix Version/s feature-backlog [ 14923 ]
          Fix Version/s 1.0.0 [ 15036 ]
          mikew Mike Wiederhold [X] (Inactive) made changes -
          Rank Ranked lower
          mikew Mike Wiederhold [X] (Inactive) added a comment -

          We are printing a formatted, spaced, multi-line string for config updates.

          mikew Mike Wiederhold [X] (Inactive) added a comment - We are printing a formatted, spaced, multi-line string for config updates.
          mikew Mike Wiederhold [X] (Inactive) made changes -
          Fix Version/s 1.0.0 [ 15036 ]
          Fix Version/s feature-backlog [ 14923 ]
          Resolution Done [ 6 ]
          Status Open [ 1 ] Resolved [ 5 ]

          People

            mikew Mike Wiederhold [X] (Inactive)
            tommie Tommie McAfee
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty