Skip to content

Use Git LFS credentials in more tests - #1341

Merged
MarkEWaite merged 2 commits into
jenkinsci:masterfrom
MarkEWaite:use-lfs-credentials-in-more-tests
Sep 15, 2025
Merged

Use Git LFS credentials in more tests#1341
MarkEWaite merged 2 commits into
jenkinsci:masterfrom
MarkEWaite:use-lfs-credentials-in-more-tests

Conversation

@MarkEWaite

Copy link
Copy Markdown
Contributor

Use Git LFS credentials in more tests

Tests are now failing LFS checkout more frequently. Only run the LFS checkout tests if an LFS credential is defined.

Remove the checks for sparse checkout support with LFS, since all command line git versions that are supported by the plugin also support Git LFS and all reasonably modern Git LFS implementations support sparse checkout.

Testing done

Confirmed that tests fail without these changes and pass with the changes (when an LFS credential is available locally).

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests that demonstrate the feature works or the issue is fixed

Tests are now failing LFS checkout more frequently.  Only run the LFS
checkout tests if an LFS credential is defined.

Remove the checks for sparse checkout support with LFS, since all command
line git versions that are supported by the plugin also support Git LFS
and all reasonably modern Git LFS implementations support sparse checkout.
@MarkEWaite MarkEWaite added the test Automated test addition or improvement label Sep 15, 2025
@MarkEWaite
MarkEWaite requested a review from a team as a code owner September 15, 2025 04:48
@MarkEWaite MarkEWaite added the test Automated test addition or improvement label Sep 15, 2025
@github-actions github-actions Bot added the tests Automated test addition or improvement label Sep 15, 2025
@MarkEWaite MarkEWaite removed the tests Automated test addition or improvement label Sep 15, 2025
Fail fast reduces the amount of information available from the build
because it aborts all parallel stages when a test fails on one of the
stages.
@github-actions github-actions Bot added the tests Automated test addition or improvement label Sep 15, 2025
@MarkEWaite
MarkEWaite merged commit 80afc17 into jenkinsci:master Sep 15, 2025
18 checks passed
@MarkEWaite
MarkEWaite deleted the use-lfs-credentials-in-more-tests branch September 15, 2025 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test Automated test addition or improvement tests Automated test addition or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant