Skip to content

Remove unnecessary branch retrieval loop (JENKINS-65796) #29

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

Merged
merged 3 commits into from
Oct 27, 2021

Conversation

justusbunsi
Copy link
Member

The mentioned Gitea bug go-gitea/gitea#2088 was fixed years ago. I always use branch patterns like <bug|feature|task>/<summarized-content> which is exactly the kind of branches for which all branches are fetched to get the only one that is needed. In my environment this takes about 20 seconds, speaking of repositories with more than 400 branches.

With this fix, this part of the build takes only 200ms.

Fixes: https://issues.jenkins.io/browse/JENKINS-65796

@justusbunsi
Copy link
Member Author

To be honest, I don't know how to provide tests for this. Any help or hint is appreciated.

@justusbunsi justusbunsi force-pushed the jenkins-65796 branch 2 times, most recently from 9cec1ca to ca8e07a Compare June 13, 2021 16:53
@justusbunsi justusbunsi changed the title Remove unnecessary branch retrieval loop on build startup phase Remove unnecessary branch retrieval loop (JENKINS-65796) Oct 15, 2021
@lafriks lafriks added this to the 1.3.0 milestone Oct 27, 2021
@lafriks lafriks merged commit baf58a0 into jenkinsci:master Oct 27, 2021
@justusbunsi justusbunsi deleted the jenkins-65796 branch October 27, 2021 11:23
justusbunsi added a commit to justusbunsi/gitea-plugin that referenced this pull request Nov 22, 2021
As a result of PR jenkinsci#29 it happens that some users cannot fetch branches
with slashes in their names anymore.

Regression from: JENKINS-65796
lafriks pushed a commit that referenced this pull request Nov 24, 2021
As a result of PR #29 it happens that some users cannot fetch branches
with slashes in their names anymore.

Regression from: JENKINS-65796
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants