aspnet / Hosting

[Archived] Code for hosting and starting up an ASP.NET Core application. Project moved to https://github.com/aspnet/Extensions and https://github.com/aspnet/AspNetCore
Apache License 2.0
552 stars 312 forks source link

IWebHost breaking change #1124

Closed grahamehorner closed 7 years ago

grahamehorner commented 7 years ago

the change 62f74d5be0b1f8091742d2d8a3e88f3fdbd05ab2 to IWebHost interface is breaking change; including service-fabric-aspnetcore, please consider re-adding Start() and Stop() to the interface to remain backwards compatible with service-fabric and other code; marking them as obsolete but with out error.

These could simply call the StartAsync, StopAsync under the hood
https://github.com/Azure/service-fabric-aspnetcore/issues/21

Tratcher commented 7 years ago

@DamianEdwards

davidfowl commented 7 years ago

We've decided to fix this.

davidfowl commented 7 years ago

@grahamehorner Have you tried service fabric with the latest build? Please re-open if you see any more issues.

grahamehorner commented 7 years ago

I have tried with

Service Fabric SDK 2.6.220 Service Fabric Runtime 5.6.220

but I'm still having issues, even the template generated code wont run inside service fabric when targeting ASP.NET Core 2.0

however the expectation with regard IWebHost.Start() is no longer shown?

I'll investigate more

grahamehorner commented 7 years ago

Ok, now tried with both Service Fabric Stable nugget 2.6.220 & ASPNETCORE 2.0 preview 3 and Service Fabric 2.7.0-preview5718 and the service fails to start :'( service shows the following

Error event: SourceId='System.FM', Property='State'. Partition is below target replica or instance count. fabric:/Application1/Web2 1 1 8b4e8ca4-a8ca-4f1b-9d49-6d7f102a79d1 IB _Node_1 Up 131435483798573745 (Showing 1 out of 1 replicas. Total available replicas: 0.)

grahamehorner commented 7 years ago

still having issues, so I've created a GitHub repo so you can see my experience of hosting ASPNETCORE 2.0 preview3 inside of Service Fabric; not sure where the root issue is yet;

https://github.com/grahamehorner/ServiceFabric.ASPNETCORE2