Uploaded image for project: 'OpenShift Bugs'
  1. OpenShift Bugs
  2. OCPBUGS-30567

Remove wrong arguments in K8sCreate method instance for Create YAML editor

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done-Errata
    • Icon: Major Major
    • 4.16.0
    • 4.16.0
    • Management Console
    • None
    • No
    • False
    • Hide

      None

      Show
      None
    • Hide
      Previously an incorrect field was sent to the API server which was not noticeable. But with the implementation of Admission Webhook display warning the same action would trigger a warning notification. A fix was provided to resolve the issue.
      Show
      Previously an incorrect field was sent to the API server which was not noticeable. But with the implementation of Admission Webhook display warning the same action would trigger a warning notification. A fix was provided to resolve the issue.
    • Release Note Not Required
    • In Progress

      Description of problem:

      - Investigate why  `name` and `namespcae` properties are passed as arguments in `k8sCreate` instance for Create YAML editor function
      
      - Remove the `name` and `namespcae` arguments in `k8sCreate` instance for Create YAML editor function if it does not require a big change. 
      
      Problem:
      If consoleFetchCommon takes an additional option(argument) and return response based on the option as proposed in "[Add support for returning response.header in consoleFetchCommon function|https://issues.redhat.com/browse/CONSOLE-3949]" story, the wrong and unused arguments in k8sCreate would cause the consoleFetchCommon method arguments to return entire response instead of response body which would break the Create Resource YAML functionality.
      
      Code: https://github.com/openshift/console/blob/master/frontend/public/components/edit-yaml.jsx#L334
          

      Version-Release number of selected component (if applicable):

      
          

      How reproducible:

      
          

      Steps to Reproduce:

          1.
          2.
          3.
          

      Actual results:

      
          

      Expected results:

      
          

      Additional info:

      
          

            cajieh Cyril Ajieh
            cajieh Cyril Ajieh
            YaDan Pei YaDan Pei
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved: