Skip to content

ci: checkout step for scout job#2628

Merged
thompson-shaun merged 2 commits into
docker:masterfrom
crazy-max:ci-skip-scout
Jul 25, 2024
Merged

ci: checkout step for scout job#2628
thompson-shaun merged 2 commits into
docker:masterfrom
crazy-max:ci-skip-scout

Conversation

@crazy-max
Copy link
Copy Markdown
Member

@crazy-max crazy-max commented Jul 25, 2024

checkout seems suggested when uploading the SARIF report: https://github.com/docker/buildx/actions/runs/10093571745/job/27910395571#step:4:25

  Processing sarif files: ["/tmp/docker-scout-action-72Brf6/result.txt"]
  Validating /tmp/docker-scout-action-72Brf6/result.txt
  Combining SARIF files using the CodeQL CLI
  Adding fingerprints to SARIF file. See https://docs.github.com/en/enterprise-cloud@latest/code-security/code-scanning/integrating-with-code-scanning/sarif-support-for-code-scanning#providing-data-to-track-code-scanning-alerts-across-runs for more information.
  Could not determine current commit SHA using git. Continuing with data from user input or environment. The checkout path provided to the action does not appear to be a git repository.
  Uploading results
  Successfully uploaded results

I guess this is to determine the origin. That's smth we might need for moby/buildkit#5184 (comment)

Last commit is related to #1715

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@crazy-max crazy-max changed the title ci: skip scout job on forked repo ci: checkout step for scout job Jul 25, 2024
@crazy-max crazy-max requested a review from thompson-shaun July 25, 2024 12:18
@crazy-max crazy-max marked this pull request as ready for review July 25, 2024 12:18
Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@thompson-shaun thompson-shaun merged commit d130f8e into docker:master Jul 25, 2024
@crazy-max crazy-max deleted the ci-skip-scout branch July 25, 2024 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants