Skip to content
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

Set fetch-depth to 500 so we don't take forever to clone repo on ci #2694

Merged

Conversation

Christewart
Copy link
Contributor

I think this may be the culprit for some slow release times on CI. The latest publish took 19 minutes, and i've seen them take upwards of 40 minutes.

Here is the description for fetch-depth

    # Number of commits to fetch. 0 indicates all history for all branches and tags.
    # Default: 1

As talked about in #2674 a full git clone with our repo currently takes 8 minutes on my machine (!)

I believe this also is slowly down builds on CI.

This PR does this, but it does come with a tradeoff. If we don't fetch deep enough (specifically to our last official tag) we won't get correct versioning. I experienced this on #2684 and ended up with weird versions on docker hub that look like this

0.0.0-1-a24e2278-SNAPSHOT

note that we don't have the latest tag (0.5.0) so we revert to the default git tag (0.0.0). So we need to make sure strike a balance here.

@Christewart Christewart added this to the v0.6.0 milestone Feb 19, 2021
@Christewart Christewart added this to In Progress in v0.6 via automation Feb 19, 2021
@Christewart Christewart changed the title Set fetch-depth to 100 so we don't take forever to clone repo on ci Set fetch-depth to 500 so we don't take forever to clone repo on ci Feb 19, 2021
@Christewart Christewart merged commit 67303b1 into bitcoin-s:master Feb 19, 2021
v0.6 automation moved this from In Progress to Done Feb 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
v0.6
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants