Skip to content

Spurious CI failures when cloning #3415

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.

Sign up for GitHub

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

Jump to bottom
Closed
allanrenucci opened this issue Oct 31, 2017 · 4 comments
Closed

Spurious CI failures when cloning #3415

allanrenucci opened this issue Oct 31, 2017 · 4 comments
Labels
area:infrastructure exp:novice itype:bug

Comments

@allanrenucci
Copy link
Contributor

Drone sometime fails to clone on PR because of a bug with GitHub.

A workaround is described in this discussion
@CucumisSativus
Copy link
Contributor

@allanrenucci on my way with fixing it

CucumisSativus added a commit to CucumisSativus/dotty that referenced this issue Oct 31, 2017
@CucumisSativus
Fix scala#3415 Spurious CI failures when cloning
7038246 
Due to concurency issue in drone I applied workaround propose in
https://discourse.drone.io/t/fix-git-clone-with-custom-extra-hosts/217/2

the image present in the yaml is suggested by http://plugins.drone.io/drone-plugins/drone-git/
CucumisSativus added a commit to CucumisSativus/dotty that referenced this issue Oct 31, 2017
@CucumisSativus
Fix scala#3415 Spurious CI failures when cloning
7929f21 
Due to concurency issue in drone I applied workaround propose in
https://discourse.drone.io/t/solved-issues-when-cloning-github-pr/482/11

the image present in the yaml is suggested by http://plugins.drone.io/drone-plugins/drone-git/
CucumisSativus added a commit to CucumisSativus/dotty that referenced this issue Oct 31, 2017
@CucumisSativus
Fix scala#3415 Spurious CI failures when cloning
e83e3b3 
Due to concurency issue in drone I applied workaround propose in
https://discourse.drone.io/t/solved-issues-when-cloning-github-pr/482/11

the image present in the yaml is suggested by http://plugins.drone.io/drone-plugins/drone-git/
@CucumisSativus
Copy link
Contributor

@allanrenucci according to https://discourse.drone.io/t/solved-issues-when-cloning-github-pr/482/15 looks like the issue is solved on plugin level, probably after plugin update the issue does not need a workaround

@allanrenucci
Copy link
Contributor Author

I have restarted the CI, hopefully it will pick up the latest version of the git plugin

@nicolasstucki
Copy link
Contributor

nicolasstucki commented Nov 2, 2017 •
edited
Loading

Currently it is failing on every push I make :(

allanrenucci added a commit to allanrenucci/dotty that referenced this issue Nov 2, 2017
@allanrenucci
Fix scala#3415: spurious clone failures in CI
8d8f6a1
allanrenucci added a commit to allanrenucci/dotty that referenced this issue Nov 2, 2017
@allanrenucci
Fix scala#3415: spurious clone failures in CI
57de3b0
allanrenucci added a commit to allanrenucci/dotty that referenced this issue Nov 2, 2017
@allanrenucci
Fix scala#3415: spurious clone failures in CI
773e728
allanrenucci added a commit to allanrenucci/dotty that referenced this issue Nov 2, 2017
@allanrenucci
Fix scala#3415: spurious clone failures in CI
fecea24
allanrenucci added a commit to allanrenucci/dotty that referenced this issue Nov 2, 2017
@allanrenucci
Fix scala#3415: spurious clone failures in CI
dfd8c4e
allanrenucci added a commit to allanrenucci/dotty that referenced this issue Nov 2, 2017
@allanrenucci
Fix scala#3415: spurious clone failures in CI
3e21131
allanrenucci added a commit to allanrenucci/dotty that referenced this issue Nov 2, 2017
@allanrenucci
Fix scala#3415: Spurious clone failures in CI
cfc7ecd 
This is a workaround for a bug where GitHub asks Drone to build a branch
that does not exist yet. We simply sleep for 5s before cloning to make
sure the branch is created
allanrenucci added a commit that referenced this issue Nov 3, 2017
@allanrenucci
Merge pull request #3432 from allanrenucci/fix-#3415
22ff8a4 
Fix #3415: Spurious clone failures in CI
allanrenucci added a commit to allanrenucci/dotty that referenced this issue Jan 12, 2018
@allanrenucci
Mitigate scala#2387: Improve cloning logic
de26b52 
- Clone submodules in parallel
- Improve workaround to scala#3415. Don't rely on GitHub to perform the merge
  on PRs
@allanrenucci allanrenucci mentioned this issue Jan 12, 2018
Closed
allanrenucci added a commit to dotty-staging/dotty that referenced this issue Jan 15, 2018
@allanrenucci
Mitigate scala#2387: Improve cloning logic
13d2cd4 
- Clone submodules in parallel
- Improve workaround to scala#3415. Don't rely on GitHub to perform the merge
  on PRs
allanrenucci added a commit to allanrenucci/dotty that referenced this issue Jan 16, 2018
@allanrenucci
Mitigate scala#2387: Improve cloning logic
6d94b0f 
- Clone submodules in parallel
- Improve workaround to scala#3415. Don't rely on GitHub to perform the merge
  on PRs
allanrenucci added a commit to allanrenucci/dotty that referenced this issue Jan 16, 2018
@allanrenucci
Mitigate scala#2387: Improve cloning logic
dc8a2bd 
- Clone submodules in parallel
- Improve workaround to scala#3415. Don't rely on GitHub to perform the merge
  on PRs
@allanrenucci allanrenucci mentioned this issue Jan 16, 2018
Merged
allanrenucci added a commit to allanrenucci/dotty that referenced this issue Jan 16, 2018
@allanrenucci
Mitigate scala#2387: Improve cloning logic
30df7ac 
- Clone submodules in parallel
- Improve workaround to scala#3415. Don't rely on GitHub to perform the merge
  on PRs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:infrastructure exp:novice itype:bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@CucumisSativus @nicolasstucki @allanrenucci