Resolve "Deploy jobs do not pull files from LFS"

Closes #62 (closed)

assigned to @jon.azpiazu

added 1 commit

• d5beb168 - Avoid skipping LFS for deploy jobs

Compare with previous version

Test pipeline (WIP) in https://git.code.tecnalia.com/tecnalia_robotics/converging/converging_noether_demo/-/pipelines/141393

added 2 commits

• 8e3526b9 - Revert "Avoid skipping LFS for deploy jobs"
• 7cceb01a - Change LFS strategy to explicetely pull

Compare with previous version

added 2 commits

• b39ee1bf - Avoid skipping LFS for deploy jobs
• 5b9270e1 - Try alternate solution using image with git-lfs preinstalled

Compare with previous version

added 1 commit

• 9fce94ca - Adapt command to busybox

Compare with previous version

added 1 commit

• ba79c38f - Remove the smudge variable from the manual git lfs pull version

Compare with previous version

Testing in:

• https://git.code.tecnalia.com/tecnalia_robotics/converging/converging_noether_demo/-/commit/54de8622e16003c27a1ea9906d99d3b2125ede02
• https://git.code.tecnalia.com/tecnalia_robotics/converging/converging_noether_demo/-/commit/b548faea2c370ef134a84b60312ed4efd6bbdd25

added 2 commits

• c8573242 - Add debug message
• 61361a9c - Avoid smudge when cloning via ddeploy

Compare with previous version

added 1 commit

• 1b86072b - Add some additional debug output

Compare with previous version

added 1 commit

• 862fdf2b - Revert "Add some additional debug output"

Compare with previous version

requested review from @miguel.prada

This seems to be working correctly with my latest test job at https://git.code.tecnalia.com/tecnalia_robotics/converging/converging_noether_demo/-/jobs/759188

By the way, a good candidate for squashing before merging.

marked this merge request as ready

I don't use LFS so I'm a bit confused with all of this. Sorry if these questions are stupid:

• Why were there some # setup LFS authentication lines before if this wasn't supported?
• Why are both the pull (and ls-files?) command(s) in before_script and the GIT_LFS_SKIP_SMUDGE required? Is it to pull files from LFS in target repository and all dependencies listed in ddeploy.yaml as well?

resolved all threads

approved this merge request

Since I don't have an LFS-ready project with which to test and you seem to have tested it already, I think this can be merged already.

merged

mentioned in commit 9e280897