Skip to content
This repository has been archived by the owner on Dec 19, 2018. It is now read-only.

IWebHost breaking change #1124

Closed
grahamehorner opened this issue Jun 30, 2017 · 6 comments
Closed

IWebHost breaking change #1124

grahamehorner opened this issue Jun 30, 2017 · 6 comments
Assignees
Milestone

Comments

@grahamehorner
Copy link

grahamehorner commented Jun 30, 2017

the change 62f74d5 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
microsoft/service-fabric-aspnetcore#21

@Tratcher
Copy link
Member

@DamianEdwards

@davidfowl davidfowl self-assigned this Jun 30, 2017
@davidfowl davidfowl added this to the 2.0.0 milestone Jun 30, 2017
@davidfowl
Copy link
Member

We've decided to fix this.

@davidfowl
Copy link
Member

davidfowl commented Jul 3, 2017

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

@grahamehorner
Copy link
Author

grahamehorner commented Jul 3, 2017

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
Copy link
Author

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
Copy link
Author

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants