Skip to content

[to_yaml] Update .cicd/scripts/wrapper_srw_ftest.sh to add third argument to call to load_modules_run_task.sh#5

Merged
christinaholtNOAA merged 1 commit into
christinaholtNOAA:to_yamlfrom
MichaelLueken:to_yaml
Jul 25, 2024
Merged

[to_yaml] Update .cicd/scripts/wrapper_srw_ftest.sh to add third argument to call to load_modules_run_task.sh#5
christinaholtNOAA merged 1 commit into
christinaholtNOAA:to_yamlfrom
MichaelLueken:to_yaml

Conversation

@MichaelLueken

Copy link
Copy Markdown

DESCRIPTION OF CHANGES:

In the last PR, I missed that the wrapper_srw_ftest.sh Jenkins script had specific changes to run the wrapper scripts on Gaea. For Gaea, load_modules_run_task.sh is called for the run_post script. The changes in to_yaml have added a third argument to the call to load_modules_run_task.sh, leading to the Functional WorkflowTaskTests on Gaea (all other machines are successfully running now).

This PR will add the third argument to the call, allowing the Functional WorkflowTaskTests to run successfully on Gaea.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

TESTS CONDUCTED:

  • gaea.intel - The Functional WorkflowTasksTest successfully passed with these changes.

CHECKLIST

  • My code follows the style guidelines in the Contributor's Guide
  • I have performed a self-review of my own code using the Code Reviewer's Guide
  • My changes generate no new warnings
  • New and existing tests pass with my changes

@christinaholtNOAA christinaholtNOAA merged commit 85070ad into christinaholtNOAA:to_yaml Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants