Refactor cleanup task and add checks for gempak files#4258
Refactor cleanup task and add checks for gempak files#4258DavidHuber-NOAA merged 42 commits intoNOAA-EMC:developfrom
Conversation
|
FYI @CatherineThomas-NOAA @JessicaMeixner-NOAA @aerorahul. I will test this on an extended case with spoofed older COM structures. |
…/global-workflow into feature/save_gempak_1p00
|
C48mx500_3DVarAOWCDA FAILED on Gaeac6 (pipeline ID: 6066) In directory: Error Log Files: View Error Logs: (gfs_arch_tar_gfs_flux_1p00.log) (gfs_arch_tar_gfs_netcdfb.log) (gfs_arch_tar_gfsa.log) (gfs_arch_tar_ocean_grib2.log) This failure was detected automatically by global-workflow's CI/CD Pipeline |
This test is covered by C48_S2SW_extended
|
@CatherineThomas-NOAA @aerorahul Could you please review this PR? |
There was a problem hiding this comment.
The latest test of the workflow - specifically modifying GEMPAK data file cleanup through case C96_atm3DVar_extended_gempak3 - provided the desired reduction in the retention of past cycle GEMPAK data files while retaining those GEMPAK files needed by JGFS_ATMOS_GEMPAK_META, thereby providing resolution to #4166.
With DO_GEMPAK=YES, DO_BUFRSND=YES, and DO_ARCHCOM=YES each set in config.base with the test case, and with no errors resulting after changes to gfs_bfr2gpk.sh and gfs_sndp.sh, there appears to be resolution to #4271.
aerorahul
left a comment
There was a problem hiding this comment.
thanks for addressing the comments. lgtm.
|
Launching CI on WCOSS2. |
CatherineThomas-NOAA
left a comment
There was a problem hiding this comment.
Looks great. Thanks @DavidHuber-NOAA!
|
All tests passed on WCOSS2 including archiving for |
Description
This adds more specific checks for gempak files and makes an attempt at making the cleanup job less disk-intensive by running a single
find -fcommand.Resolves #4271
Resolves #4166
Type of change
Change characteristics
How has this been tested?
Will test on an extended test.
Checklist