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

taskTimeout was increased to 180,000 ms #2456

Merged
merged 2 commits into from
Jun 25, 2017
Merged

taskTimeout was increased to 180,000 ms #2456

merged 2 commits into from
Jun 25, 2017

Conversation

vik-borisov
Copy link
Contributor

Restoring the package was unsuccessful with a low-speed network connection

@matthid
Copy link
Member

matthid commented Jun 22, 2017

Not sure if this is the correct approach here... 30s is already quite a lot of time for a request to finish. Even with a slow connection.
Maybe this is related to our new request preloading... Maybe we start just too many requests in parallel for the same server?

@forki
Copy link
Member

forki commented Jun 25, 2017

@vik-borisov is this against nuget.org?

@vik-borisov
Copy link
Contributor Author

no, it is myget

@forki forki merged commit d98a974 into fsprojects:master Jun 25, 2017
@forki
Copy link
Member

forki commented Jun 25, 2017

and it was restore that failed - the download. right?

@matthid
Copy link
Member

matthid commented Jun 25, 2017

To be honest this change is completely unrelated to restore. And even if it would help - I still think it's the wrong way :)

@forki
Copy link
Member

forki commented Jun 25, 2017

that's why I asked. I have no problem with increasing an already arbitray timeout. but wonder if the reported issue is actually restore

@vik-borisov
Copy link
Contributor Author

Yes, the restore will fail. But then I increased the environment variable PAKET_RESOLVER_TASK_TIMEOUT to 180,000, then the problem was solved

@vik-borisov
Copy link
Contributor Author

If i use package local cache or i use version 4, i have no problem

@matthid
Copy link
Member

matthid commented Jun 25, 2017

But restore/download is actually repeated 3 times, was that reported by paket?
That the environment variable helped probably was just an accident?

@vik-borisov
Copy link
Contributor Author

vik-borisov commented Jun 25, 2017

The environment variable is a workaround, but it works for me and my colleagues

@matthid
Copy link
Member

matthid commented Jun 25, 2017

Can you show the failing output?

@vik-borisov
Copy link
Contributor Author

Well, I'll try to reproduce this error

@vik-borisov
Copy link
Contributor Author

vik-borisov commented Jun 25, 2017

Paket failed with:
-> There was a version conflict during package resolution. Resolved packages:
- StimulReport.Base 2015.1.0.0 Conflict detected:
- Dependencies file requested package StimulReport.Web: 2015.1 - Available versions:
- (2016.2.0.0, [https://myget.org]) - (2016.1.0.1, [https://myget.org])
- (2016.1.0.0, [https://myget.org]) - (2015.1.0.0, [https://myget.org])> Please try to relax some conditions or resolve the conflict manually (see http://fsprojects.github.io/Paket/nuget-dependencies.html#Use-exactly-this-version-constraint).
StackTrace: в Paket.PackageResolver.ResolutionModule.getModelOrFail(Resolution res)
в [email protected](GroupName groupName, DependenciesGroup dependenciesGroup) в Microsoft.FSharp.Collections.MapTreeModule.mapiOpt[a,b,c](FSharpFunc3 f, MapTree2 m)
в Microsoft.FSharp.Collections.FSharpMap`2.Map[b](FSharpFunc`2 f) в Paket.UpdateProcess.selectiveUpdate[a](Boolean force, FSharpFunc2 getSha1, FSharpFunc2 getVersionsF, FSharpF
unc`2 getPackageDetailsF, FSharpFunc`2 getRuntimeGraphFromPackage, LockFile lockFile, DependenciesFile dependenciesFile, UpdateMode updateMode, SemVerUpdateMode semVerUpdateMode)
в Paket.UpdateProcess.SelectiveUpdate(DependenciesFile dependenciesFile, FSharpOption`1 alternativeProjectRoot,UpdateMode updateMode, SemVerUpdateMode semVerUpdateMode, Boolean force)
в Paket.UpdateProcess.SmartInstall(DependenciesFile dependenciesFile, UpdateMode updateMode, UpdaterOptions options)
в <StartupCode$Paket-Core>[email protected](Unit unitVar0) в Paket.Utils.RunInLockedAccessMode[a](String rootFolder, FSharpFunc2 action) `` в [email protected](ParseResults1 results)
в Paket.Program.processWithValidation[T](Boolean silent, FSharpFunc`2 validateF, FSharpFunc`2 commandF, ParseResults1 result) `` в Paket.Program.main$cont@453(ParseResults1 results, Boolean silent, Boolean fromBootstrapper, Unit unitVar)
в Paket.Program.main()-> Unable to retrieve package details for 'StimulReport.Web'-2015.1.0.0
StackTrace: в [email protected](FSharpList1 sources, GroupName groupName, PackageName `` packageName, SemVerInfo semVer) `` в Paket.PackageResolver.explorePackageConfig(FSharpFunc2 getPackageDetailsBlock, PackageConfig pkgConfig)
-> TimeoutException: Waited 30 seconds for a request to finish. Check the following sources, they might be rate limiting and stopped responding:
- https://myget.org - https://myget.org
- https://myget.org - https://myget.org

@vik-borisov
Copy link
Contributor Author

This error is reproduced for version 5.1.7. For version 5.2.1 works fine

@matthid
Copy link
Member

matthid commented Jun 25, 2017

But this means you are doing paket install or paket update and not paket restore...

@vik-borisov
Copy link
Contributor Author

yes, now i was used paket update.

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.

3 participants