Uploaded image for project: 'Red Hat Process Automation Manager'
  1. Red Hat Process Automation Manager
  2. RHPAM-4044

BA Operator UI generate startup strategy in wrong place

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Critical Critical
    • 7.12.0.GA
    • 7.12.0.GA
    • Cloud
    • None
    • BA Operator UI 7.12.0

    • False
    • False
    • CR1
    • CR2
    • Hide

      For generated KieApp yaml file manually change startup strategy parameter

      Show
      For generated KieApp yaml file manually change startup strategy parameter
      1. Deploy Operator to a new OCP project
      2. Open Operator console UI
      3. Add application name and selecet startup strategy and click view yaml button

      When is generated KieApp yamls from the BA Operator UI page, the startupStrategy is not under commonConfig and there is a typo in name. Generated yaml from UI looks like this:

      apiVersion: app.kiegroup.org/v2
      kind: KieApp
      metadata: 
        name: asdfasdf
      spec: 
        environment: rhpam-authoring
        commonConfig: 
          adminUser: admin
          adminPassword: admin
        startupStrategyartupStrategy: 
          strategyName: OpenShiftStartupStrategy
          controllerTemplateCacheTTL: 6000
      
      

      Generated KieApp with wrongly set startup strategy will not deploy the application.

      It is expected that startup strategy is placed in commonConfig and genereated yaml from UI should look like this:

      apiVersion: app.kiegroup.org/v2
      kind: KieApp
      metadata: 
        name: asdfasdf
      spec: 
        environment: rhpam-authoring
        commonConfig: 
          adminUser: admin
          adminPassword: admin
          startupStrategy: 
            strategyName: OpenShiftStartupStrategy
            controllerTemplateCacheTTL: 6000
      

      On wrong position in generated KieApp yaml is also placed parameter `disableSsl: true` bring in RHPAM-3475 - see change in PR. Right now it is placed into spec, but according to the KieApp CRD it should be placed under commonConfig. Can you please also include this change.

            mdessi-1 Massimiliano Dessi
            jakubschwan Jakub Schwan
            Jiri Simek Jiri Simek
            Jiri Simek Jiri Simek
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved: