⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Add tutorials with DuMux as solver into system tests#696

Draft
Fujikawas wants to merge 31 commits intodevelopfrom
dumuxsystemtest
Draft

Add tutorials with DuMux as solver into system tests#696
Fujikawas wants to merge 31 commits intodevelopfrom
dumuxsystemtest

Conversation

@Fujikawas
Copy link
Member

Checklist:

  • I added a summary of any user-facing changes (compared to the last release) in the changelog-entries/<PRnumber>.md.
  • I will remember to squash-and-merge, providing a useful summary of the changes of this PR.

@IshaanDesai IshaanDesai marked this pull request as draft January 16, 2026 14:55
@MakisH
Copy link
Member

MakisH commented Jan 21, 2026

To test: trigger the run testsuite manual workflow with:

  • use workflow from: dumuxsystemtest (not needed yet, maybe if you change the workflows)
  • test suite dumux_test
  • no arguments
  • branch to take the system tests from: dumuxsystemtest
  • logging: DEBUG

@IshaanDesai
Copy link
Member

I got further in the debugging and now see the error:

#21 0.167 Cloning into 'dumux-adapter'...
#21 0.678 error: pathspec 'main' did not match any file(s) known to git
#21 ERROR: process "/bin/bash -c git clone --depth 1 https://github.com/precice/dumux-adapter.git &&    cd dumux-adapter &&     if [ -n \"${DUMUX_ADAPTER_PR}\" ]; then git fetch origin pull/${DUMUX_ADAPTER_PR}/head; fi &&     git checkout ${DUMUX_ADAPTER_REF} &&     cd .. &&    dunecontrol --opts=cmake-test.opts --only=dumux-adapter all" did not complete successfully: exit code: 1
------
 > [micro-dumux dumux_adapter 6/6] RUN git clone --depth 1 https://github.com/precice/dumux-adapter.git &&    cd dumux-adapter &&     if [ -n "${DUMUX_ADAPTER_PR}" ]; then git fetch origin pull/${DUMUX_ADAPTER_PR}/head; fi &&     git checkout main &&     cd .. &&    dunecontrol --opts=cmake-test.opts --only=dumux-adapter all:
0.167 Cloning into 'dumux-adapter'...
0.678 error: pathspec 'main' did not match any file(s) known to git
------

It seems like git checkout main for the dumux-adapter did not work, although the main branch does exist.

@MakisH
Copy link
Member

MakisH commented Jan 21, 2026

You are pulling with --depth 1, asking for the branch main, but the default branch is develop. There is no need for --depth 1 in this case: you want to change to other references, for which you need the full repository.

@Fujikawas
Copy link
Member Author

With the last manual run of the test suite dumux_test, it takes about 6min51s to build the docker image(at least the DuMux part is newly built) and run the case. Next step would be to generate the reference solution.

The manual run before the change to the docker file failed for timeout. Possible reasons are: two times complete compilation of DUNE modules took double the time, or one of this compilations failed build of the adapter, as I built and tested locally to find that the solver can't find the adapter among the other DUNE modules.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants