Skip to content

RT updates: remove compiler from labels#129

Draft
mkavulich wants to merge 4 commits into
mainfrom
feature/update_auto_rt
Draft

RT updates: remove compiler from labels#129
mkavulich wants to merge 4 commits into
mainfrom
feature/update_auto_rt

Conversation

@mkavulich

@mkavulich mkavulich commented Feb 7, 2024

Copy link
Copy Markdown
Collaborator

PR Author Checklist:

  • I have linked PR's from all sub-components involved in section below.
  • I am confirming reviews are completed in ALL sub-component PR's.
  • I have run the full RT suite on either Hera/Cheyenne AND have attached the log to this PR below this line:
    • LOG:
  • I have added the list of all failed regression tests to "Anticipated changes" section.
  • I have filled out all sections of the template.

Description

Updating Regression tests so that we no longer have compiler-specific labels, as in the main repository (now all compilers are run by default). Will be adding more to this PR as I fix various issues, needs more testing first.

Linked Issues and Pull Requests

Associated UFSWM Issue to close

None

Subcomponent Pull Requests

None

Blocking Dependencies

None

Subcomponents involved:

  • AQM
  • CDEPS
  • CICE
  • CMEPS
  • CMakeModules
  • FV3
  • GOCART
  • HYCOM
  • MOM6
  • NOAHMP
  • WW3
  • stochastic_physics
  • none

Anticipated Changes

Input data

  • No changes are expected to input data.

Regression Tests:

  • No changes are expected to any regression test.
  • Changes are expected to the following tests:
Tests effected by changes in this PR:

Libraries

  • Not Needed
Code Managers Log
  • 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.
  • Move new/updated input data on RDHPCS Hera and propagate input data changes to all supported systems.
    • N/A

Testing Log:

  • RDHPCS
    • Hera
    • Orion
    • Hercules
    • Jet
    • Gaea
    • Cheyenne
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
    • Completed
  • opnReqTest
    • N/A
    • Log attached to comment

@mkavulich mkavulich added hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 7, 2024
@mkavulich

mkavulich commented Feb 7, 2024

Copy link
Copy Markdown
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: RT
[RT] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240207215510/ufs-weather-model
[RT] Log file shows failures.
[RT] Please obtain logs from /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240207215510/ufs-weather-model

@mkavulich mkavulich added hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich

mkavulich commented Feb 8, 2024

Copy link
Copy Markdown
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: RT
[RT] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240208154010/ufs-weather-model
[RT] Log file shows failures.
[RT] Please obtain logs from /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240208154010/ufs-weather-model

@mkavulich mkavulich added hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich
Remove call to delete_pr_dirs() which is messing up test compilation …
1ab2a72 
…somehow, finally clean up the unparseable wall of text logs into something more readable
@mkavulich mkavulich force-pushed the feature/update_auto_rt branch from 8a882c5 to 1ab2a72 Compare February 8, 2024 19:17
@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich

mkavulich commented Feb 8, 2024

Copy link
Copy Markdown
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240208192114/ufs-weather-model
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich

mkavulich commented Feb 8, 2024

Copy link
Copy Markdown
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240208202113/ufs-weather-model
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich

mkavulich commented Feb 8, 2024

Copy link
Copy Markdown
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240208205712/ufs-weather-model
[BL] ERROR: Baseline location exists before creation:
/scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/new/
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich

mkavulich commented Feb 8, 2024

Copy link
Copy Markdown
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added the hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET label Feb 8, 2024
@mkavulich

mkavulich commented Feb 8, 2024

Copy link
Copy Markdown
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich

mkavulich commented Feb 8, 2024

Copy link
Copy Markdown
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added the hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET label Feb 8, 2024
@mkavulich

mkavulich commented Feb 8, 2024

Copy link
Copy Markdown
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240208213710/ufs-weather-model
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 8, 2024
@mkavulich

mkavulich commented Feb 9, 2024

Copy link
Copy Markdown
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209001213/ufs-weather-model
[BL] Baseline creation successful
Please make changes and add the following label back: hera-BL

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 9, 2024
@mkavulich

mkavulich commented Feb 9, 2024

Copy link
Copy Markdown
Collaborator Author

Automated RT Failure Notification
Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209020916/ufs-weather-model
[BL] Baseline creation successful
[RT] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209023026/ufs-weather-model
Automated RT Failure Notification
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209020916/ufs-weather-model
[BL] Baseline creation successful
[RT] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209023026/ufs-weather-model
[RT] {job_obj.clargs.machine} Job Completed

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 9, 2024
@mkavulich

mkavulich commented Feb 9, 2024

Copy link
Copy Markdown
Collaborator Author

Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209030912/ufs-weather-model
[BL] Baseline creation successful
[RT] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209034711/ufs-weather-model
[RT] hera Job Completed

@mkavulich mkavulich added hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-BL Create new baselines on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 9, 2024
@mkavulich

mkavulich commented Feb 9, 2024

Copy link
Copy Markdown
Collaborator Author

Machine: hera
Job: BL
[BL] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209184012/ufs-weather-model
Baseline creation successful on hera
[RT] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240209185950/ufs-weather-model
***Regression test successful on hera!

@grantfirl

grantfirl commented Feb 23, 2024

Copy link
Copy Markdown
Collaborator

@mkavulich Did you want to commit this next?

@mkavulich

mkavulich commented Feb 23, 2024

Copy link
Copy Markdown
Collaborator Author

@grantfirl I will need to bring it up-to-date with the latest code, but if it's okay with you I think that would be a good idea to get this one in next. Once #120 is merged I can update this branch and run a final set of tests/open the PR for review.

@mkavulich

mkavulich commented Feb 23, 2024

Copy link
Copy Markdown
Collaborator Author

Okay since tests are running for #121, this will wait until after that one is merged.

@mkavulich mkavulich added hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET and removed hera-RT Run regression test on Hera machine. TESTING ONLY, NOT FOR GENERAL PRS YET labels Feb 23, 2024
@mkavulich

mkavulich commented Feb 23, 2024

Copy link
Copy Markdown
Collaborator Author

Machine: hera
Job: RT
[RT] Repo location: /scratch1/BMC/gmtb/CCPP_regression_testing/NCAR_ufs-weather-model/beta/run//1716477654/20240223233521/ufs-weather-model
Regression test successful on hera!

@grantfirl

grantfirl commented May 23, 2024

Copy link
Copy Markdown
Collaborator

@mkavulich Can we revive this and get it merged in preparation for running RTs, creating new baselines and catching up with ufs/dev?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mkavulich @grantfirl