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

create_experiment HPC_ACCOUNT not working #3124

Closed
CoryMartin-NOAA opened this issue Nov 25, 2024 · 2 comments · Fixed by #3127
Closed

create_experiment HPC_ACCOUNT not working #3124

CoryMartin-NOAA opened this issue Nov 25, 2024 · 2 comments · Fixed by #3127
Assignees
Labels
bug Something isn't working

Comments

@CoryMartin-NOAA
Copy link
Contributor

What is wrong?

One used to be able to do something like:

HPC_ACCOUNT=da-cpu ./create_experiment.py --yaml ci/cases/pr/C96C48_hybatmaerosnowDA.yaml

and it would replace all the ACCOUNT information with da-cpu that is no longer the case, it's instead fv3-cpu

What should have happened?

HPC_ACCOUNT=da-cpu should fill in da-cpu where appropriate

What machines are impacted?

All or N/A

What global-workflow hash are you using?

1ba8985

Steps to reproduce

See above

Additional information

No response

Do you have a proposed solution?

No response

@CoryMartin-NOAA CoryMartin-NOAA added bug Something isn't working triage Issues that are triage labels Nov 25, 2024
@AndrewEichmann-NOAA
Copy link
Contributor

Seconded, and same for the queue settings. At least they could be overridden in the yamls.

@DavidHuber-NOAA DavidHuber-NOAA removed the triage Issues that are triage label Nov 26, 2024
@DavidHuber-NOAA DavidHuber-NOAA self-assigned this Nov 26, 2024
@DavidHuber-NOAA
Copy link
Contributor

Yes, this does stem from #3053. In that PR, I made it so whatever was in the CI YAML would be overridden by the workflow/hosts machine YAML. This allowed for features that were not supported on a machine to be disabled, even if it is explicitly enabled in the CI test (such as METplus or the TC tracker).

I will need to think about how to reconcile this to allow users to influence assignments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants