-
Notifications
You must be signed in to change notification settings - Fork 253
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.
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
Minor bugfixes for CCPP v6 #1231
Minor bugfixes for CCPP v6 #1231
Conversation
@grantfirl We will work on your PR today, are your branches up to date? |
Yes, this is ready to work on. |
Automated RT Failure Notification |
@BrianCurtis-NOAA It looks to me that the tests are still running on jet . Not sure why we got this message. |
on-behalf-of @ufs-community <brian.curtis@noaa.gov>
@junwang-noaa maybe a python try/except hiccup with slurm pending warning... But yes jobs still running |
on-behalf-of @ufs-community <brian.curtis@noaa.gov>
on-behalf-of @ufs-community <brian.curtis@noaa.gov>
on-behalf-of @ufs-community <brian.curtis@noaa.gov>
Automated RT Failure Notification |
Automated RT Failure Notification |
@junwang-noaa Jet is notoriously error prone it seems to me. If jobs are still running on Jet but the error message appeared on here, that means Jet killed the AutoRT but ecflow is still running things. There will be no logs automatically pushed and i'm not confident all jobs will run. |
@grantfirl The cray/dell log files are under /scratch1/NCEPDEV/stmp2/Jun.Wang/wcosslog |
gaea.intel failure is
probably just needs to be re-run. |
@BrianCurtis-NOAA yeah, slurm pending issues dumped fail_tests: cpld_debug_p8 006, hafs_regional_atm_thompson_gfdlsf 088, hafs_regional_datm_cdeps 094. I will see if I can run these three cases manually. |
The control_c384gdas_wav failed for me the first time (time-out) on hera.intel too. It passed while running separately. |
Thanks. They've been added/pushed. |
…ther-model into 20220523_ccpp_v6_needs
The control_c384gdas_wav was reran and it finished successfully on hera_intel. |
Automated RT Failure Notification |
…ther-model into 20220523_ccpp_v6_needs
@BrianCurtis-NOAA @junwang-noaa if I run manually on jet, hafs_regional_atm_thompson_gfdlsf ran ok but all others seem to have slurm unknown status and pending issues. |
…ther-model into 20220523_ccpp_v6_needs
@grantfirl RT passed, please merge CCPP PRs. Thanks |
@junwang-noaa The FV3 submodule pointer has been updated and .gitmodules reverted. |
PR Checklist
This PR is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR. Please consult the ufs-weather-model wiki if you are unsure how to do this.
This PR has been tested using a branch which is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR
An Issue describing the work contained in this PR has been created either in the subcomponent(s) or in the ufs-weather-model. The Issue should be created in the repository that is most relevant to the changes in contained in the PR. The Issue and the dependent sub-component PR
are specified below.
Results for one or more of the regression tests change and the reasons for the changes are understood and explained below.
New or updated input data is required by this PR. If checked, please work with the code managers to update input data sets on all platforms.
Description
This PR only contains minor bugfixes for CCPP-framework and CCPP-physics and should not affect UFS RTs at all (will test to confirm)
Issue(s) addressed
NCAR/ccpp-physics#927
NCAR/ccpp-framework#450
Testing
All tests in rt.conf pass with existing baselines.
Dependencies
NCAR/ccpp-physics#924
NCAR/ccpp-framework#451
NOAA-EMC/fv3atm#541