Uploaded image for project: 'OpenShift Virtualization'
  1. OpenShift Virtualization
  2. CNV-39047

Storage: Webhooks should not depend on foreign data

XMLWordPrintable

    • no-remote-validation-storage
    • 77
      • no-ux
      • no-doc
      • QE scope is regression testing only
      • Storage owned webhooks do not rely on foreign data
    • Green
    • To Do
    • CNV-38613 - 2024 Code Quality
    • CNV-386132024 Code Quality
    • 25% To Do, 50% In Progress, 25% Done
    • dev-ready, doc-ready, po-ready, qe-ready, ux-ready
    • Hide

      Work to continue in 4.18...

      Show
      Work to continue in 4.18...

      Goal

      Avoid races in KubeVirt and ensure it is eventually consistent.

      Requirements

      • List all admission webhooks in KubeVirt, and review each.
      • Ensure that it does not attempt to peek outside of the modified value.
      • There may be exceptions, such as:
        • Clear, real-life, documented customer case report requesting early input validation.
        • Validation of non-KubeVirt resources
      • Remaining validation must be commented on and explained e.g.
        // This code is raceful. We keep it here in order to provide a quick feedback to an actual end user who enters a wrong input. Cf. bug report URL
        or
        // This code is raceful. We keep it because fixing it properly is going to be complex and we plan to delete this code anyway.
      • Remaining web-hook validation should have a matching reconciliation-loop validation.

      Non-Requirements

      •  

      Notes

      • Any additional details or decisions made/needed

            skagan@redhat.com Shelly Kagan
            dkenigsb@redhat.com Dan Kenigsberg
            Ying Cui Ying Cui
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: