Skip to content
Snippets Groups Projects

[ci] Update ci config

Closed [ci] Update ci config
feature/update-ci into master
2 unresolved threads
Closed Carsten Gräser requested to merge feature/update-ci into master
2 unresolved threads

This uses the standard ci config that pulls the current master of other modules like e.g. dune-functions.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Carsten Gräser added 1 commit

    added 1 commit

    Compare with previous version

  • Carsten Gräser added 1 commit

    added 1 commit

    Compare with previous version

    • Oliver Sander
      Oliver Sander @oliver.sander ·
      Owner

      I am not convinced. Running the full CI tests already takes quite a while. You changes will make it take even longer.

      I do see your point: If I need to fix something upstream then right know I have to wait for those changes to percolate into the CI Docker images. But does that really happen all that frequently?

      Edited by Ghost User
    • Carsten Gräser
      Carsten Gräser @carsten.graeser ·
      Author Owner

      I did some testing: Changing this increases the runtime from 56 min to 74 min. However runtimes will reduce significantly once we move to dune-project.org because the runners have more computational power there.

      On dune-project.org I those runtimes reduce to 5 min and 18 min. If we update the image to dune:git-debian-11-gcc-10-20 we have 8 min. Those numbers are not strictly comparable, because the jobs have been executed on different runners, but it still gives an impression.

      In view of the significant run-time increase I agree, that we should stay with the prepared images. In the worst case these can be updated manually by restarting the corresponding jobs in https://gitlab.dune-project.org/docker/ci. For dune:git-debian-11-gcc-10-20 this takes about 10 min.

    • Please register or sign in to reply
  • Oliver Sander
    Oliver Sander @oliver.sander started a thread on the diff
.gitlab-ci.yml
3 3 before_script:
4 - . /duneci/bin/duneci-init-job
5 - duneci-install-module https://gitlab.dune-project.org/core/dune-common.git
6 - duneci-install-module https://gitlab.dune-project.org/core/dune-geometry.git
7 - duneci-install-module https://gitlab.dune-project.org/core/dune-localfunctions.git
8 - duneci-install-module https://gitlab.dune-project.org/staging/dune-uggrid.git
9 - duneci-install-module https://gitlab.dune-project.org/core/dune-grid.git
10 - duneci-install-module https://gitlab.dune-project.org/core/dune-istl.git
4 11 - duneci-install-module https://gitlab.dune-project.org/extensions/dune-alugrid.git
5 12 - duneci-install-module https://gitlab.dune-project.org/extensions/dune-subgrid.git
13 - duneci-install-module https://gitlab.dune-project.org/staging/dune-typetree.git
14 - duneci-install-module https://gitlab.dune-project.org/staging/dune-functions.git
6 15 - duneci-install-module https://git.imp.fu-berlin.de/agnumpde/dune-matrix-vector.git
7 16
8 dune:2.8 gcc:
9 variables:
  • Carsten Gräser added 1 commit

    added 1 commit

    • c5585d52 - [ci] Add CI config for 2.9 release branch

    Compare with previous version

  • Carsten Gräser resolved all threads

    resolved all threads

  • Carsten Gräser
    Carsten Gräser @carsten.graeser ·
    Author Owner

    Any objection to merging this?

  • Carsten Gräser added 18 commits

    added 18 commits

    Compare with previous version

  • Carsten Gräser closed

    closed

  • Carsten Gräser mentioned in merge request !133 (merged)

    mentioned in merge request !133 (merged)

    • Please register or sign in to reply