Newly added sparse checkout functionality is not cleaning up after itself

[markjm]

Thanks @dscho for contributing #1369 . We have started using it and noticed the following issue

repo-a / .github/workflows/workflow-a.yml

jobs:
  job-1:
    runs-on: internal-runner

- uses: actions/checkout@v3
  with:
    sparse-checkout: |
      .github
      src

running just the above works as expected. But consider another workflow in the same repo

repo-a / .github/workflows/workflow-b.yml

jobs:
  job-12:
    runs-on: internal-runner

- uses: actions/checkout@v3
- run: ls ./not-src

When running workflow-a, then workflow workflow-b, workflow-b will not find any files in ./not-src.

This is because workflow workflow-b runs in the same _work directory as workflow-a. On the first run of workflow-a, we have polluted the repo in a way this action doesn't know about (and thus can't clean up).

Notice that, even in workflow-b:

git config --list will show core.sparseCheckout true
git sparse-checkout list will still include the listed paths
and a simple
ls will only show the sparse directories for workflow-a

It seems that - similar to the auth cleanup (and other pre-run steps) this action does - we need to disable sparse checkout in the pre steps of this action

Note: this is likely only an issue for self-hosted runners which dont get the same cleaning/re-init process github-hosted runners get

Similar-ish issue to recently reported 47fbe2d (for submodules, instead of sparse checkout). @megamanics

[dscho]

this is likely only an issue for self-hosted runners

Indeed.

The working directory's path typically contains the workflow name, e.g. D:\a\_work\repo-a\repo-a, but not the workflow name.

A solid work-around would be to scorch the worktree (if it exists), right before the actions/checkout step.

A more surgical work-around would be to use a shell script like this:

- shell: bash
  run: |
    if test true = "$(git config --type bool core.sparsecheckout)"
    then
      rm "$(git rev-parse --git-path info/sparse-checkout)" &&
      git sparse-checkout disable
    fi

[markjm]

We are looking to use such a workaround in the interim, but it seems this is something that should be fixed here within actions/checkout, no?

[jinhyoo-mp]

This caused CI issues for us so we had to remove sparse checkout

[vibro]

I also ran into issues with this, two different workflows, one which uses sparse and one which doesn't. I ended up removing the contents of the directory prior to checkout to force a clean checkout every time.

[aamagda]

@dscho Hi! Do you have plans to fix this isuue?

[dscho]

@aamagda time constraints force me to say "no".

Having said that, if you can start implementing that (probably by specifically resetting the sparse checkout if none is desired), and verify that it works, I can take it from there.

[justin-newman]

Assuming v4.1.2 works when explicitly setting it, I am still having the same issue with v4.1.2 on a self-hosted server

[dscho]

Assuming v4.1.2 works when explicitly setting it, I am still having the same issue with v4.1.2 on a self-hosted server

@justin-newman could you verify in the logs that the git sparse-checkout disable command was run, like so (line 51)?

[justin-newman]

Assuming v4.1.2 works when explicitly setting it, I am still having the same issue with v4.1.2 on a self-hosted server

@justin-newman could you verify in the logs that the git sparse-checkout disable command was run, like so (line 51)?

Yes, it is doing that

[dscho]

could you verify in the logs that the git sparse-checkout disable command was run, like so (line 51)?

Yes, it is doing that

@justin-newman can you find out whether that command simply does not work in your setup? I am somewhat surprised to read that it was called yet did not fix the problem.