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.

Sign up for GitHub

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

Jump to bottom

MMS testing trigger mpi runs on serial test suite runs #22635

Closed
GiudGiud opened this issue Nov 9, 2022 · 0 comments
Closed

MMS testing trigger mpi runs on serial test suite runs #22635

GiudGiud opened this issue Nov 9, 2022 · 0 comments
Labels
C: TestHarness P: normal A defect affecting operation with a low possibility of significantly affects. T: defect An anomaly, which is anything that deviates from expectations.

Comments

@GiudGiud
Copy link
Contributor

GiudGiud commented Nov 9, 2022

Bug Description

missing a min_parallel or some logic to make sure we dont call mpi on a serial test suite run

Steps to Reproduce

run test suite and observe the FV mms testing trigger mpi calls

Impact

users find out they have mpi issues while running mms testing
instead of dedicated MPI tests
@GiudGiud GiudGiud added C: TestHarness T: defect An anomaly, which is anything that deviates from expectations. P: normal A defect affecting operation with a low possibility of significantly affects. labels Nov 9, 2022
GiudGiud added a commit to GiudGiud/moose that referenced this issue Dec 30, 2022
@GiudGiud
No need to drum up MPI for one rank
bf14a25 
closes idaholab#22635
GiudGiud added a commit to GiudGiud/moose that referenced this issue Dec 30, 2022
@GiudGiud
Add a basic MPI test to point users to their broken MPI refs idaholab…
08e388b 
…#22635
GiudGiud added a commit to GiudGiud/moose that referenced this issue Dec 30, 2022
@GiudGiud
Add a basic MPI test to point users to their broken MPI refs idaholab…
4b4eb17 
…#22635
GiudGiud added a commit to GiudGiud/moose that referenced this issue Dec 30, 2022
@GiudGiud
Add a basic MPI test to point users to their broken MPI refs idaholab…
13b804b 
…#22635
GiudGiud added a commit to GiudGiud/moose that referenced this issue Dec 30, 2022
@GiudGiud
Add a basic MPI test to point users to their broken MPI refs idaholab…
9dac1b1 
…#22635
GiudGiud added a commit to GiudGiud/moose that referenced this issue Jan 1, 2023
@GiudGiud
Add a basic MPI test to point users to their broken MPI refs idaholab…
577a993 
…#22635
GiudGiud added a commit to GiudGiud/moose that referenced this issue Jan 1, 2023
@GiudGiud
Avoid running 1 rank MPI for mms studies. It fails MPI tests
97152d2 
I initially made the underlying mooseUtil not use MPI for one rank but this would remove any sort of MPI overhead and could be confusing for leading scalability studies
Closes idaholab#22635
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C: TestHarness P: normal A defect affecting operation with a low possibility of significantly affects. T: defect An anomaly, which is anything that deviates from expectations.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@GiudGiud