Skip to content

18.0 use treehash phase 2 xdo #1127

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 2 commits into
base: 18.0
Choose a base branch
from
Draft

Conversation

Xavier-Do
Copy link
Contributor

No description provided.

It is quite frequent to have multiple builds with the same tree hash,
most likely when rewording a commit or stashing multiple ones. This can
lead to useless load since pushing squashed commit should have the same
result as pushing the original commit in terms of test (This is not the
case for config checking the diff, the commit messages ... aka batch
dependant commits)

This commit modifies the way the identifier is computed to use the
tree_hash instead of the commit_id. This way, we can link
builds with the same tree_hash together, and have faster results

Note that this commit still have a problem to solve, how to ensure to
send status on the new commits.
@Xavier-Do Xavier-Do force-pushed the 18.0-use-treehash-phase-xdo branch from f254b91 to 45a3b5a Compare April 16, 2025 07:10
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.

1 participant