Skip to content

Integrate fetching latest components into Systemtest.py#842

Merged
MakisH merged 18 commits into
developfrom
latest-components
Jun 15, 2026
Merged

Integrate fetching latest components into Systemtest.py#842
MakisH merged 18 commits into
developfrom
latest-components

Conversation

@MakisH

@MakisH MakisH commented Jun 15, 2026

Copy link
Copy Markdown
Member

This PR largely simplifies the GitHub Actions workflows by integrating the resolution of the latest commits of each repository directly into the systemtests.py.

The algorithm now is generally:

  • Collect all needed parameters (as before)
  • Replace any not given needed parameters with their defaults (as before)
  • For all parameters ending in _REF, resolve the respective commit from the given repository
    • If this fails, just continue with the ref as-is, and throw a warning.

To achieve that, besides the changes in systemtests.py:

  • In the components.yaml, I moved the repository: from the component level to the level of each build argument. This is a bit ugly, but it was the easiest access to the repository. Note that this has to be the same as the remote in the respective Dockerfile (currently not enforced).
  • In the components.yaml, I replaced all the default git references to the respective develop branches. The master/main were anyway not really useful. I also removed the description, since these were also not used anywhere.

For the GitHub Actions workflows:

  • I removed the nmbgeek/github-action-get-latest-commit action:
  • For similar reasons, I removed the recently introduced VeyronSakai/display-inputs-action, replacing it with a JSON dump of all given inputs.

The workflows can be further simplified and cleaned-up, but that's good enough for now.

I have ran the system tests locally and on GitHub Actions, using branches and tags as given refs, as well as omitting or giving empty arguments. Empty arguments are now assumed to be replaced by the default. Example run: https://github.com/precice/tutorials/actions/runs/27567474586 (the inputs will only be shown once merged to develop, since it reuses the run_testsuite workflow.

Closes #841.

Checklist

  • I added a summary of any user-facing changes (compared to the last release) in the changelog-entries/<PRnumber>.md.

@MakisH MakisH self-assigned this Jun 15, 2026
@MakisH MakisH changed the title Integrate fetching of latest components into systemtests.py Integrate fetching latest components into Systemtest.py Jun 15, 2026
@MakisH MakisH marked this pull request as ready for review June 15, 2026 21:18
@MakisH MakisH merged commit 7fccdc3 into develop Jun 15, 2026
1 check passed
@MakisH MakisH deleted the latest-components branch June 15, 2026 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Resolve latest commit of each branch

1 participant