Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove data paths from lower level scripts #167

Closed
KateFriedman-NOAA opened this issue Nov 2, 2020 · 3 comments
Closed

Remove data paths from lower level scripts #167

KateFriedman-NOAA opened this issue Nov 2, 2020 · 3 comments
Labels
production update Processing update in production

Comments

@KateFriedman-NOAA
Copy link
Member

Look through lower level scripts (e.g. ush) and remove path settings that should be in upper level scripts (e.g. JJOBs).
@KateFriedman-NOAA KateFriedman-NOAA added production update Processing update in production and removed coupled labels Aug 11, 2021
lgannoaa pushed a commit to lgannoaa/global-workflow that referenced this issue Feb 9, 2022
@DusanJovic-NOAA
Update CMakeModules submodule (NOAA-EMC#167)
4b531bc
@WalterKolczynski-NOAA WalterKolczynski-NOAA added this to the GFSv17 milestone Sep 8, 2022
@WalterKolczynski-NOAA
Copy link
Contributor

If these settings absolutely must not be changed, we should mark them readonly in the j-job. That will actually prevent any downstream script from altering them.

@WalterKolczynski-NOAA
Copy link
Contributor

I think there is some overlap between this and #293. This might even be strictly a subset of that issue. Close as duplicate?

@KateFriedman-NOAA
Copy link
Member Author

Yeah this overlaps with #293. Closing as duplicate.

@KateFriedman-NOAA KateFriedman-NOAA closed this as not planned Won't fix, can't repro, duplicate, stale Oct 4, 2022
zhanglikate pushed a commit to zhanglikate/global-workflow that referenced this issue Oct 6, 2022
@aerorahul
Add PR, Issues for bugfix and feature request templates. (NOAA-EMC#167)
2833858 
* add PR templates for feature and bugfix
Change-Id: Iad15dd1a584f4bd2fbaadf88359a55efd7b1925c
* add codeowners for automatic review requests
Change-Id: If6050e6411519223f514435801e09a63ed696a02
* Update CODEOWNERS
kayeekayee pushed a commit to kayeekayee/global-workflow that referenced this issue May 30, 2024
@jderrico-noaa
Merge pull request NOAA-EMC#167 from kayeekayee/global-workflow
c9563ed 
Change paths to run under Rocky
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
production update Processing update in production
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@WalterKolczynski-NOAA @KateFriedman-NOAA @JianKuang-Intelsat