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

Suggested improvements for R&D CI runs #950

Closed
climbfuji opened this issue Jan 16, 2024 · 1 comment · Fixed by #955
Closed

Suggested improvements for R&D CI runs #950

climbfuji opened this issue Jan 16, 2024 · 1 comment · Fixed by #955
Assignees
Labels
INFRA JEDI Infrastructure

Comments

@climbfuji
Copy link
Collaborator

Is your feature request related to a problem? Please describe.

See #940 (review):

Would it make sense to use strategy:matrix:... to avoid all the indentation, and also allow gcc and intel to run in parallel? I have no problem with the current approach though.

Describe the solution you'd like
See above

Additional context
n/a

@climbfuji climbfuji self-assigned this Jan 16, 2024
@climbfuji climbfuji added the INFRA JEDI Infrastructure label Jan 16, 2024
@climbfuji
Copy link
Collaborator Author

I thought about this a bit more. Because that's running on a self-hosted runner in the same directory, we'd have to make a lot more efforts to make sure we can build two separate environments (append compiler to env name?) using the same temporary directories etc. I want to avoid this. My hope is that we can revert all this when the problem goes away in the future (either by itself, same as it showed up first, or because the spack developers implement the hierarchical matrix approach I mentioned earlier). So I am going to fix the bug that I introduced in the R&D CI in a PR shortly, but leave the logic as is if that's ok.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INFRA JEDI Infrastructure
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

1 participant