Uploaded image for project: 'Pegasus'
  1. Pegasus
  2. PM-1680

kickstart needs to quote argument vector when outputting it to yaml

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: master
    • Fix Version/s: master, 5.0.0
    • Component/s: pegasus-kickstart
    • Labels:
      None

      Description

      it seems yaml record generated by kickstart is malformed when they are quotes in an argument value. for one of our user's workflow kickstart generated the following snippet, that is breaking the yaml parsers

      argument_vector:
            - "/home/scitech/shared-data/sample-geoedf-wf/mcd15-viz.yml"
            - "1"
            - "Input"
            - "NASAInput"
            - "/home/scitech/shared-data/sample-geoedf-wf/1600793629/stage-1-Input.yml"
            - "/data/1600793629"
            - "/home/scitech/shared-data/sample-geoedf-wf/1600793629"
            - "None"
            - "None"
            - "None"
            - "{"password": "xxxxx"}"
            - "None"
            - "condorpool"

      The above is invalid yaml because of the quotes in the - "{"password": "NASAservice123"}"

      instead it should have been
      argument_vector:
            - "/home/scitech/shared-data/sample-geoedf-wf/mcd15-viz.yml"
            - "1"
            - "Input"
            - "NASAInput"
            - "/home/scitech/shared-data/sample-geoedf-wf/1600793629/stage-1-Input.yml"
            - "/data/1600793629"
            - "/home/scitech/shared-data/sample-geoedf-wf/1600793629"
            - "None"
            - "None"
            - "None"
            - "{\"password\": \"xxxxx\"}"
            - "None"
            - "condorpool"

      am attaching the kickstart output and also the planner generated invocation where the strings were escaped correctly

        Attachments

          Activity

            People

            • Assignee:
              rynge Mats Rynge
              Reporter:
              vahi Karan Vahi
            • Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: