Skip to content

Conversation

@octo-sts
Copy link
Contributor

@octo-sts octo-sts bot commented Jul 10, 2025

@octo-sts octo-sts bot added request-version-update request for a newer version of a package automated pr drupal-11 labels Jul 10, 2025
@octo-sts
Copy link
Contributor Author

octo-sts bot commented Jul 10, 2025

🔄 Build Failed: Git Checkout Error

FAIL Expected commit 4a1db0db3b144a42daa0887e904f942a55fe5b6b for 11.2.2, found 965123445745f50c7fc001c72a96650518c267d2

Build Details

Category Details
Build System git
Failure Point git checkout

Root Cause Analysis 🔍

The build failed because the actual commit hash for Drupal tag 11.2.2 (965123445745f50c7fc001c72a96650518c267d2) does not match the expected commit hash specified in the build configuration (4a1db0db3b144a42daa0887e904f942a55fe5b6b). This indicates that either the tag has been updated in the repository since the build configuration was created, or there's an incorrect commit hash specified in the configuration.


🔍 Build failure fix suggestions

Found similar build failures that have been fixed in the past and analyzed them to suggest a fix:

Similar PRs with fixes

Suggested Changes

File: drupal-11.yaml

      expected-commit: 4a1db0db3b144a42daa0887e904f942a55fe5b6b

Replacement:

      expected-commit: 965123445745f50c7fc001c72a96650518c267d2

Content:

Update the expected-commit hash to match the actual commit hash for Drupal 11.2.2
Click to expand fix analysis

Analysis

After examining the similar fixed build failures, a clear pattern emerges: all three examples involved fixing a Git commit hash mismatch in the expected-commit parameter of the git-checkout action. In each case, the build failed because the actual commit hash for a specific tag didn't match what was specified in the configuration. The fixes consistently involved updating the expected-commit value to match the actual commit hash that the tag points to in the upstream repository. This suggests that tags in Git repositories sometimes get updated to point to different commits, particularly when tags are annotated objects rather than direct references to commits.

Click to expand fix explanation

Explanation

The build is failing because the expected commit hash for Drupal tag 11.2.2 (4a1db0db3b144a42daa0887e904f942a55fe5b6b) doesn't match the actual commit hash (965123445745f50c7fc001c72a96650518c267d2) that the tag points to in the repository. This could happen for several reasons:

  1. The tag may have been updated in the upstream repository since the build configuration was created
  2. The tag might be an annotated tag object rather than a direct reference to a commit
  3. The repository structure may have changed or been reorganized

By updating the expected-commit value in the git-checkout action to match the actual commit hash (965123445745f50c7fc001c72a96650518c267d2), we're telling the build system to expect that specific commit when checking out the 11.2.2 tag, which should resolve the mismatch error.

This solution directly addresses the error message "FAIL Expected commit 4a1db0db3b144a42daa0887e904f942a55fe5b6b for 11.2.2, found 965123445745f50c7fc001c72a96650518c267d2" by updating our expectation to match reality.

Click to expand alternative approaches

Alternative Approaches

  • Rather than updating the expected commit hash, we could consider using a different Git checkout option that doesn't validate the commit hash. However, this would reduce security as the hash validation helps ensure the code integrity.
  • Another approach would be to check out the specific commit hash directly rather than using the tag. However, this would make future version updates more complex and less automated.
  • We might need to also update the expected commit for the recommended-project repository checkout as it's using the same hash. If both repositories should be in sync, updating both expected-commit values would be necessary.

Was this comment helpful? Please use 👍 or 👎 reactions on this comment.

@octo-sts octo-sts bot added the ai/skip-comment Stop AI from commenting on PR label Jul 10, 2025
@vishal-chdhry vishal-chdhry self-assigned this Jul 11, 2025
@octo-sts octo-sts bot added the bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. label Jul 11, 2025
Copy link
Member

@aborrero aborrero left a comment

Choose a reason for hiding this comment

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

LGTM.

@aborrero aborrero merged commit 1ef1f40 into main Jul 11, 2025
17 of 18 checks passed
@aborrero aborrero deleted the wolfictl-06de3d77-32af-4738-a290-0a0e76c1f739 branch July 11, 2025 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai/skip-comment Stop AI from commenting on PR automated pr bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. drupal-11 request-version-update request for a newer version of a package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants