Uploaded image for project: 'Red Hat OpenShift AI Engineering'
  1. Red Hat OpenShift AI Engineering
  2. RHOAIENG-4747 Simplify Service Mesh setup
  3. RHOAIENG-4749

Promote Ingress Gateway config to be part of ServiceMesh infra component

    XMLWordPrintable

Details

    • Sub-task
    • Resolution: Unresolved
    • Major
    • None
    • None
    • Platform
    • False
    • Hide

      None

      Show
      None
    • False
    • RHOAISTRAT-41 - Support SSO for all RHOAI components
    • No
    • Testable

    Description

      Ingress Gateway configuration allows to define domain and certificate attached to the gateway. As the platform is handling the creation and configuration of this key resource, it should be promoted to a top-level configuration component.

      While doing so we should:

      • add IngressGatewaySpec to DSCI.ServiceMesh
      • deprecate KServe.ServingSpec.IngressGatewaySpec in the API
      • for the existing installation, during the upgrade process (in operator bootstrap - main.go), migrate the values from DSC to DSCI
      • handle "conversion paths" (captured below)

      Possible scenarios

      For the fresh install of the operator these are potential scenarios of applying DSCI/DSC resources:

      • only DSCI.ServiceMesh.IngressGateway is set - features related to it but no KServe specific, are applied there (though I think there should not be any IGW specific kserve bit config, it's shared thing in the platform)
      • both DSCI.ServiceMesh.IngressGateway and DSC.KServe.Serverless are set. We use the former config, the latter is ignored and logged (WARN).
      • using old DSC definition, where this configuration is in the deprecated place, but DSCI does not have it yet.

      Q: how the last case should be handled? DSC updating DSCI?

      Attachments

        Activity

          People

            Unassigned Unassigned
            bartosz-1 Bartosz Majsak
            Aslak Knutsen, Bartosz Majsak, Cameron Garrison
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:

              PagerDuty