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.

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

can not run single test using rt.sh #2192

Closed
WChen-NOAA opened this issue Mar 14, 2024 · 5 comments
Closed

can not run single test using rt.sh #2192

WChen-NOAA opened this issue Mar 14, 2024 · 5 comments
Assignees
Labels
bug Something isn't working EPIC Support Requested

Comments

@WChen-NOAA
Copy link
Contributor

Description

can not run single test using rt.sh

To Reproduce:

  1. /rt.sh -l rt.conf -n control_p8 intel -a $ACCNR
  2. then failed with err.msg:
    'The -n option needs AND in quotes, i.e. -n "control_p8 intel"'
  3. then tried /rt.sh -l rt.conf -n "control_p8 intel" -a $ACCNR
    run all regression tests.
@WChen-NOAA WChen-NOAA added the bug Something isn't working label Mar 14, 2024
@BrianCurtis-NOAA
Copy link
Collaborator

Thanks for this issue Wen. I believe without the -l rt.conf it will run only that one test. But, the issue is that when the user specifies -l and -n it should run the test using that specified conf file, which is a use case that I know I haven't tested for.

@uturuncoglu
Copy link
Collaborator

@BrianCurtis-NOAA JFYI, we are always using -l and -n together under ufs-coastal without any issue. But of course, we did not sync ufs-coastal recently and I think that "control_p8 intel" usage is new. So, maybe the update in RT system break that capability. If so we might need to fix it since ufs-coastal has its own test file called as rt_coastal.conf at this point. Anyway, once i synced ufs-coastal again, I'll check and report back.

@FernandoAndrade-NOAA
Copy link
Collaborator

Leaving a note here that the current develop works fine running a single test with -n and without -l in a recent test on gaea at
/gpfs/f5/epic/scratch/Fernando.Andrade-maldonado/reg-test/wm/dev-test/ufs-weather-model/tests/logs/RegressionTests_gaea.log

Please ignore the error itself in the log, that's a system issue.
test was run with ./rt.sh -a epic -e -c -n "regional_spp_sppt_shum_skeb intel"

@zach1221
Copy link
Collaborator

I think a fix for this has been found, as seen in issue 2244, so this can be closed once the commit is in develop.

@zach1221
Copy link
Collaborator

Closing as UFS-WM PR #2241 has been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working EPIC Support Requested
Projects
Status: Done
Development

No branches or pull requests

6 participants