Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • C chipathlon
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 4
    • Issues 4
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • Holland Computing Center
  • chipathlon
  • Issues
  • #35

Closed
Open
Created May 24, 2017 by aknecht2@aknecht2Maintainer

Workflow Job Parsing Update

Somehow, we never hit the case where we needed to optionally pass entire arguments to scripts. Since all arguments are always included, there is currently no way to handle #34 (closed). Some small updates to the workflow job parsing should be able to handle this, but will require changes across several spots.

Change the input structure to be a dictionary not a list. Inputs will be passed by name instead of in a list. If an argument has a value that is None, we will skip adding it entirely. This will require updating the workflow_module yaml files to similarly be dictionaries instead of lists, as well as the internals of the workflow_module class to correctly pass information to the workflow_job classes.

Assignee
Assign to
Time tracking