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

Upgrade to dotnet core 2.1 #4

Merged
merged 1 commit into from Nov 13, 2018

Conversation

ElanHasson
Copy link

No description provided.

@ElanHasson
Copy link
Author

Some of the tests seem flaky and fail due to concurrency issues with file IO access. This happened on master too. Refactoring the tests to use randomized files or in memory packages to avoid this should be considered.

@tomzo
Copy link
Member

tomzo commented Oct 10, 2018

Thanks. I haven't seen those issues, but this is possible. LiGet implementation of V2 is ported from old NuGet server, It can have issues.
As I said in other issue, my intentions are to make a BaGet image which is a drop-in replacement for LiGet. I am porting tests from LiGet to BaGet and writing new ones.
Please have a look at the fork and post your suggestions there - https://github.com/ai-traders/baget
I should have V2 implemented today.

@ElanHasson
Copy link
Author

Cool! Wasn't aware of BaGet.
The dockerfile still uses the ask image to deploy versus using a production dotnet image. Is there a reason for this?

@tomzo
Copy link
Member

tomzo commented Oct 10, 2018

The dockerfile still uses the ask image to deploy versus using a production dotnet image

Which dockerfile are you talking about?

@ElanHasson
Copy link
Author

The BaGet one.

It uses the stretch image. I couldn't readily find information on it, but appears to have the sdk included in it, which isn't required at runtime.

What is the benefit of that image over the runtime containers?

@tomzo
Copy link
Member

tomzo commented Oct 10, 2018

Ah, I see, you meant FROM microsoft/dotnet:2.1.402-sdk-stretch in my fork.
It's a mistake, should be just the runtime.
I will switch it shortly.
Thanks.

@alexanderfast
Copy link

I need this PR to build, I dont know if the dependent pre-release packages went away or something.

/home/afast/liget/src/LiGet.App/LiGet.App.csproj : error NU1102: Unable to find package Nancy with version (>= 2.0.0-Pre1885) [/home/afast/liget/LiGet.sln]
/home/afast/liget/src/LiGet.App/LiGet.App.csproj : error NU1102:   - Found 47 version(s) in nuget.org [ Nearest version: 2.0.0-clinteastwood ] [/home/afast/liget/LiGet.sln]
/home/afast/liget/tests/LiGet.Tests/LiGet.Tests.csproj : error NU1102: Unable to find package Nancy with version (>= 2.0.0-Pre1885) [/home/afast/liget/LiGet.sln]
/home/afast/liget/tests/LiGet.Tests/LiGet.Tests.csproj : error NU1102:   - Found 47 version(s) in nuget.org [ Nearest version: 2.0.0-clinteastwood ] [/home/afast/liget/LiGet.sln]
/home/afast/liget/src/LiGet.App/LiGet.App.csproj : error NU1102: Unable to find package Nancy.Bootstrappers.Autofac with version (>= 2.0.0-Pre486) [/home/afast/liget/LiGet.sln]
/home/afast/liget/src/LiGet.App/LiGet.App.csproj : error NU1102:   - Found 39 version(s) in nuget.org [ Nearest version: 2.0.0-clinteastwood ] [/home/afast/liget/LiGet.sln]
/home/afast/liget/tests/LiGet.Tests/LiGet.Tests.csproj : error NU1102: Unable to find package Nancy.Bootstrappers.Autofac with version (>= 2.0.0-Pre486) [/home/afast/liget/LiGet.sln]
/home/afast/liget/tests/LiGet.Tests/LiGet.Tests.csproj : error NU1102:   - Found 39 version(s) in nuget.org [ Nearest version: 2.0.0-clinteastwood ] [/home/afast/liget/LiGet.sln]
/home/afast/liget/src/LiGet/LiGet.csproj : error NU1102: Unable to find package Nancy with version (>= 2.0.0-Pre1885) [/home/afast/liget/LiGet.sln]
/home/afast/liget/src/LiGet/LiGet.csproj : error NU1102:   - Found 47 version(s) in nuget.org [ Nearest version: 2.0.0-clinteastwood ] [/home/afast/liget/LiGet.sln]
/home/afast/liget/src/LiGet/LiGet.csproj : error NU1102: Unable to find package Nancy.Bootstrappers.Autofac with version (>= 2.0.0-Pre486) [/home/afast/liget/LiGet.sln]
/home/afast/liget/src/LiGet/LiGet.csproj : error NU1102:   - Found 39 version(s) in nuget.org [ Nearest version: 2.0.0-clinteastwood ] [/home/afast/liget/LiGet.sln]

@tomzo
Copy link
Member

tomzo commented Nov 13, 2018

Yes, they were probably removed.
I was lazy on merging this due to work on BaGet migration.

@tomzo tomzo merged commit 8ba9c40 into ai-traders:master Nov 13, 2018
@alexanderfast
Copy link

What's the BaGet migration? Are you replacing Liget with BaGet, obsoleting this project?

@tomzo
Copy link
Member

tomzo commented Nov 13, 2018

@Mizipzor I already have a BaGet fork which is a drop-in replacement for LiGet.
My initial plan was to obsolete this project and provide migration scripts for all LiGet users to move to BaGet.
But recently it turns out that upstream author of BaGet will not merge my e2e tests and release cycle. So probably I will merge parts of BaGet fork into LiGet. I haven't had time to work on this yet.

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.

None yet

3 participants