Skip to content

feat: faster integrated tests - #4115

Closed
victorapm wants to merge 24 commits into
developfrom
feature/paludettomag1/faster-integrated_tests
Closed

feat: faster integrated tests#4115
victorapm wants to merge 24 commits into
developfrom
feature/paludettomag1/faster-integrated_tests

Conversation

@victorapm

@victorapm victorapm commented Aug 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • Add reduced, integrated-test-specific inputs and meshes without changing tutorial problems.
  • Shorten simulated durations and event/checkpoint schedules for the long-running smoke tests.
  • Preserve the existing solver choices and restart/curve-check coverage.
  • Total ATS wall time. Before: 55 min After: 35 min

These are not full ATS cases (no second restart run or restart/curve checks). Aggregate runtime for the 25 tests on a local workstation as the runner (not streak2): 9:55 → 1:56 (5.1x speedup).

Test develop this branch Speedup
PoroElastic hybridHexPrism co2 fim 3d 01 1:38 0:06 16x
LagrangeContactBubbleStab FixedSlip smoke 01 0:13 0:01 19x
pknViscosityDominated smoke 01 0:44 0:01 43x
PoroElastic hybridHexPrism co2 sequential 3d 01 1:11 0:04 17x
deadOilEgg smoke 3d 01 0:05 0:02 2.4x
validationCase 01 1:12 0:09 8.5x
FieldCaseTutorial3 Thermal smoke 01 0:08 0:01 14x
multiphasePoromechanics FaultModel well fim smoke 01 0:58 0:02 24x
multiphasePoromechanics FaultModel smoke 01 0:45 0:01 31x
dead oil wells hybrid 2d 01 0:18 0:05 3.9x
immiscibleTwoPhase SPE10 layer84 benchmark iterative 01 0:21 0:08 2.6x
staged open perf 2 01 0:02 0:02 1.3x
dome soreide whitson smoke 01 0:04 0:02 1.9x
staged open perf 1 01 0:02 0:02 1.3x
ALM multiphasePoromechanics curvedFrac smoke 01 0:19 0:03 6.2x
deadOil fractureMatrixFlow edfm inclinedFrac smoke 01 0:25 0:25 1.0x
staged close perf 2 01 0:02 0:02 1.3x
staged close perf 1 01 0:02 0:02 1.3x
ALM TFrac smoke 01 0:18 0:18 1.0x
ALM singlephasePoromechanics curvedFrac smoke 01 0:15 0:01 11x
acouselas3D Q2 abc smoke 01 0:13 0:05 2.9x
dome kvalue smoke 01 0:03 0:02 1.8x
staircase co2 wells hybrid 3d 01 0:17 0:08 2.1x
kgdToughness Storage Dominated smoke 01 0:12 0:02 7.7x
PoroElastic staircase co2 3d sequential 01 0:08 0:04 2.1x

Open question: for some reason, many single rank tests on streak2 take minutes to execute, while in a 16-cores workstation they take seconds

Closes #4049

joshua-white and others added 24 commits August 8, 2026 19:07
@joshua-white
Attempt 2: try migrating ATS python setup to docker setup
cf2a09f
@joshua-white
disable pip upgrades
1fcbb6b
@joshua-white
Add crypto diagnostics
3f2f8a2
@joshua-white
Attempt to bypass fips.so requirement
bf9a014
@joshua-white
Merge branch 'develop' into ci-ubuntu-FIPS-issue
a05db3e
@joshua-white
Make GCP upload optional; Try to re-enable local sccache optimization
e422d85
@joshua-white
Fix sccache max size
3dbf9b7
@joshua-white
Set RUNS_ON: streak2 for defaulting sccache configurations
bafde66
@joshua-white
Address concurrency issue by fixing NPROC and memory args
293a90c
@joshua-white
Try fixing CTEST_PARALLEL args for slow unit tests
17ace4d
@joshua-white
Move unit tests from streak2 back to github
26b5b90
@joshua-white
Re-enable GCS sccache for gitub-runners
e5c035a
@victorapm
Fix python issue
746ac1f
@victorapm
Merge branch 'ci-ubuntu-FIPS-issue' of github.com:GEOS-DEV/GEOS into …
286c864 
…ci-ubuntu-FIPS-issue
@joshua-white
Disable isothm_mass_inj_table_02 tests temporarily
eef1913
@joshua-white
Merge branch 'ci-ubuntu-FIPS-issue' of github.com:GEOS-DEV/GEOS into …
c2f3d5a 
…ci-ubuntu-FIPS-issue
@joshua-white
Use host networking for streak2 Docker CI jobs to avoid bridge DNS fa…
39be927 
…ilures
@joshua-white
Restore Docker bridge networking for streak2 CI jobs; Fixes outbound …
42934e7 
…issue, but breaks sccache with collisions
@joshua-white
Cleanup to remove debugging code / verbosity
d82138a
@joshua-white
Update HPCReach submodule
9bef5e5
@victorapm
Reduce integrated test runtimes
787c29d 
- Add reduced smoke-test inputs and meshes
- Shorten long-running schedules and checkpoint cadence
- Preserve tutorial inputs, solvers, and validation coverage
@victorapm
Merge branch 'ci-ubuntu-FIPS-issue' into feature/paludettomag1/faster…
d97daf3 
…-integrated_tests
@joshua-white
Aggressively trim PR; disable two super-slow tests
81ebad5
@victorapm
Merge branch 'ci-ubuntu-FIPS-issue' into feature/paludettomag1/faster…
bcdae56 
…-integrated_tests
@victorapm victorapm self-assigned this Aug 13, 2026
@victorapm victorapm added ci: run CUDA builds flag: ready for review flag: requires rebaseline Requires rebaseline branch in integratedTests ci: run integrated tests Allows to run the integrated tests in GEOS CI ci: run code coverage enables running of the code coverage CI jobs labels Aug 13, 2026
@victorapm
victorapm changed the base branch from develop to ci-ubuntu-FIPS-issue August 13, 2026 01:14
Base automatically changed from ci-ubuntu-FIPS-issue to develop August 13, 2026 04:45
@victorapm

victorapm commented Aug 15, 2026

Copy link
Copy Markdown
Contributor Author

Closing in favor of #4118

@victorapm victorapm closed this Aug 15, 2026
@victorapm
victorapm deleted the feature/paludettomag1/faster-integrated_tests branch August 15, 2026 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rrsettgast rrsettgast Awaiting requested review from rrsettgast rrsettgast is a code owner
@dkachuma dkachuma Awaiting requested review from dkachuma dkachuma is a code owner
@OmarDuran OmarDuran Awaiting requested review from OmarDuran OmarDuran is a code owner
@joshua-white joshua-white Awaiting requested review from joshua-white joshua-white is a code owner
@jhuang2601 jhuang2601 Awaiting requested review from jhuang2601 jhuang2601 is a code owner
@acitrain acitrain Awaiting requested review from acitrain acitrain is a code owner
@sframba sframba Awaiting requested review from sframba sframba is a code owner
@bd713 bd713 Awaiting requested review from bd713 bd713 is a code owner
@jafranc jafranc Awaiting requested review from jafranc jafranc is a code owner
@frankfeifan frankfeifan Awaiting requested review from frankfeifan frankfeifan is a code owner
@cssherman cssherman Awaiting requested review from cssherman cssherman is a code owner
@kimtaeho07 kimtaeho07 Awaiting requested review from kimtaeho07 kimtaeho07 is a code owner
@tjb-ltk tjb-ltk Awaiting requested review from tjb-ltk tjb-ltk is a code owner
@MelReyCG MelReyCG Awaiting requested review from MelReyCG MelReyCG is a code owner

Assignees

@victorapm victorapm

Labels

ci: run code coverage enables running of the code coverage CI jobs ci: run integrated tests Allows to run the integrated tests in GEOS CI flag: ready for review flag: requires rebaseline Requires rebaseline branch in integratedTests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Long application/integrated tests.

2 participants

@victorapm @joshua-white