search

NOAA-EMC / RDASApp

Regional DAS
GNU Lesser General Public License v2.1
0 stars 5 forks source link

Add basic on-prem CI to RDASApp #17

Closed CoryMartin-NOAA closed 1 month ago

CoryMartin-NOAA commented 1 month ago

Adding the capability to run some basic CI on Hera and Orion like what GDASApp can do.

NOTE: this runs as my user on these machines. We should find someone else to (eventually) take over these crons.

CoryMartin-NOAA commented 1 month ago

Do not approve/merge this yet, seems that the missing piece is the GitHub CLI that I forgot was needed and is not in the module env

emcbot commented 1 month ago

Automated RDASApp Testing Results: Machine: hera

Start: Fri Mar 29 20:19:52 UTC 2024 on hfe11
---------------------------------------------------
Build:                                 *SUCCESS*
Build: Completed at Fri Mar 29 21:00:27 UTC 2024
---------------------------------------------------
Tests:                                 *SUCCESS*
Tests: Completed at Fri Mar 29 21:00:50 UTC 2024
Tests:
emcbot commented 1 month ago

Automated RDASApp Testing Results: Machine: orion

Start: Fri Mar 29 15:32:29 CDT 2024 on Orion-login-1.HPC.MsState.Edu
---------------------------------------------------
Build:                                 *SUCCESS*
Build: Completed at Fri Mar 29 16:10:35 CDT 2024
---------------------------------------------------
Tests:                                 *SUCCESS*
Tests: Completed at Fri Mar 29 16:10:55 CDT 2024
Tests:
delippi commented 1 month ago

Hi @CoryMartin-NOAA, what are the cron entries you are using to run this? I want to give it a try.

CoryMartin-NOAA commented 1 month ago

@delippi 

*/15 * * * * /scratch1/NCEPDEV/da/Cory.R.Martin/CI/RDASApp/repo/ci/driver.sh -t hera
CoryMartin-NOAA commented 1 month ago

@delippi thanks, and you are correct!

TingLei-NOAA commented 1 month ago

I haven't seen the disadvantage building both (except for computing time) . The gdasapp/rdasapp seems to be able to manage both well. For developers , they can also use existing -m option to select what they want. Maybe, in the future, when more submodules are added depending on one of mpas or fv3 , this combined option should be looked at .