Uploaded image for project: 'Red Hat Advanced Cluster Management'
  1. Red Hat Advanced Cluster Management
  2. ACM-11421

Store the detected diff in the ConfigurationPolicy status

XMLWordPrintable

    • 3
    • False
    • None
    • False
    • Hide

      Provide the required acceptance criteria using this template.
      * ...
      Show
      Provide the required acceptance criteria using this template. * ...
    • ACM-5892 - RFE Visualize difference between Policy desired state vs actual state
    • No
    • GRC Sprint 2024-09, GRC Sprint 2024-10

      Value Statement

      To show the recorded diff in the console, the diff needs to be stored somewhere. It is proposed to store it in the ConfigurationPolicy status.relatedObjects[*].properties.diff field.

      The diff will be stored by default if the object is not one of these kinds:

      • Secrets
      • Config maps
      • Routes
      • OAuth access tokens
      • OAuth authorize tokens

      In addition, any policy that looks up a secret or decrypts a value with templating will have it be disabled. This will require a change in go-template-utils.

      The existing `recordDiff` option will default to `Automatic` for this behavior. The user can explicitly set `None`, `Log`, and the new options of `InStatus` and `Automatic`.

      Definition of Done for Engineering Story Owner (Checklist)

      • ...

      Development Complete

      • The code is complete.
      • Functionality is working.
      • Any required downstream Docker file changes are made.

      Tests Automated

      • [ ] Unit/function tests have been automated and incorporated into the
        build.
      • [ ] 100% automated unit/function test coverage for new or changed APIs.

      Secure Design

      • [ ] Security has been assessed and incorporated into your threat model.

      Multidisciplinary Teams Readiness

      Support Readiness

      • [ ] The must-gather script has been updated.

            mprahl Matthew Prahl
            mprahl Matthew Prahl
            Derek Ho Derek Ho
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: