Skip to content

Draft: CI: ensure Git-LFS files are checked out in all jobs working with the Git repo

Karl Wetterequested to merge
ANU-CGA/lalsuite:ensure-git-lfs-checkout into master

Detailed Description

The igwn/base:conda Docker image does not have git-lfs installed, leading to LALSuite repositories not being checked out correctly by GitLab runners (https://git.ligo.org/computing/helpdesk/-/issues/8089). This should be addressed in https://git.ligo.org/computing/helpdesk/-/work_items/8103. The manylinux images used to build the wheels are also missing git-lfs, this should be addressed in lalsuite-manylinux#4 (closed).

This MR adds a sanity check to the LALSuite CI: if the repository checkout is needed (i.e. if GIT_STRATEGY is not none), run git lfs ls-files. This will ensure git-lfs is installed in all CI images. The output of git lfs ls-files may be helpful when debugging LFS-related issues.

API Changes

Please tick one of the following options:

  • These changes do not modify the API.
  • These changes do modify the API, and are backwards compatible.
  • These changes do modify the API, and are backwards incompatible.

For examples of changes that do not modify the API and/or are considered backwards (in)compatible, please see the contributing guide.

Justification for Backwards Incompatible Changes

n/a

Review Status

n/a

Edited by Karl Wette

Merge request reports