Skip to content

Allow checkout from a specific hash - #8528

Merged
torcolvin merged 2 commits into
mainfrom
fix-e2e-tests
Aug 5, 2026
Merged

Allow checkout from a specific hash#8528
torcolvin merged 2 commits into
mainfrom
fix-e2e-tests

Conversation

@torcolvin

Copy link
Copy Markdown
Collaborator

Allow checkout from a specific hash.

When Couchbase Lite E2E is run, it gets a hash from a checkout like abc123 which was not able to be checked out:

Added https://jenkins.sgwdev.com/job/Couchbase%20Lite%20E2E%20jenkins%20testing/29 and https://jenkins.sgwdev.com/job/Couchbase%20Lite%20E2E%20jenkins%20testing/28 for fixing the checkout.

need to use $class: 'GitSCM' to have access to refspec.

@torcolvin
torcolvin requested review from a team and a lite review from Copilot August 4, 2026 15:40
@factory-droid

factory-droid Bot commented Aug 4, 2026

Copy link
Copy Markdown

Droid finished @torcolvin's task —— View job


Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Couchbase Lite E2E Jenkins pipeline to allow checking out Sync Gateway by a specific revision string (branch name or commit hash), addressing failures when the provided revision is not resolvable via the default Jenkins git step.

Changes:

  • Replaces the git(...) step with a checkout([$class: 'GitSCM', ...]) configuration to allow custom refspec usage.
  • Attempts to fetch and checkout params.SG_COMMIT even when it’s a commit SHA rather than a branch name.

Comment thread integration-test/e2e/Jenkinsfile
Comment thread integration-test/e2e/Jenkinsfile

@factory-droid factory-droid Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Jenkins checkout change should work for full 40-character SHAs, but it still won’t handle abbreviated commit SHAs (which the PR description indicates are common). Tighten the parameter contract to full SHAs or adjust the fetch/checkout strategy so short SHAs can be resolved after normal refs are fetched.

bbrks
bbrks previously approved these changes Aug 4, 2026
@torcolvin
torcolvin merged commit 7aea6e3 into main Aug 5, 2026
29 checks passed
@torcolvin
torcolvin deleted the fix-e2e-tests branch August 5, 2026 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants