Skip to content

Version calculation performance on tfs build #1065

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

Closed
IAfanasov opened this issue Oct 10, 2016 · 3 comments
Closed

Version calculation performance on tfs build #1065

IAfanasov opened this issue Oct 10, 2016 · 3 comments

Comments

@IAfanasov
Copy link

IAfanasov commented Oct 10, 2016

Hi guys!

tnx for the tool!

I use GitVersionTask nuget package and have problem with really slow version calculation on the build sever. In total it (version calc) takes up to 10 minutes. For some reason GitVersion firstly get all commits and then filter them by commit date. It adds up several minutes. Also it branching same thing 3 times. I didn't get why it needs even one brunching. Tried to find some configuration tweeks in documentation without any success. Is it possible to optimize performance of GitVersion?

Extract from VSO log attached

@normanhh3
Copy link

I"m seeing similarly long times and the one section that seems to be taking up the most time shows up as this in the log:

`2018-01-11T13:36:06.9090877Z INFO [01/11/18 8:36:06:90] Begin: Calculating base versions

2018-01-11T13:37:50.1101969Z INFO [01/11/18 8:37:50:11] Fallback base version: 0.1.0 with commit count source 11812048b0b32dfac4f661d54bbe31ab6428ed87

2018-01-11T13:40:54.6097161Z INFO [01/11/18 8:40:54:60] Base version used: Fallback base version: 0.1.0 with commit count source 11812048b0b32dfac4f661d54bbe31ab6428ed87

2018-01-11T13:40:54.6097161Z INFO [01/11/18 8:40:54:60] End: Calculating base versions (Took: 287,700.63ms)`

Any ideas why this might be taking a long time?

I'm running on a Windows 8-based Build Agent right now.

@normanhh3
Copy link

Turns out that this PR seems to have resolved the issue for us. #1100

@asbjornu
Copy link
Member

Ok, great. Closing.

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

No branches or pull requests

4 participants