Update aspire hosting samples#8265
Conversation
…r/docs.particular.net into aspire-hosting-samples # Conflicts: # platform/aspire/index.md
| ASQN: NServiceBus.Transport.AzureStorageQueues | ||
| ASP: NServiceBus.Persistence.AzureStorage | ||
| ASTP: NServiceBus.Persistence.AzureTable | ||
| Aspire: Particular.Aspire.Hosting.ServicePlatform |
There was a problem hiding this comment.
When we have a client package down the track (which seems to be a valid assumption to make), should we name this so that's more explicit and allows for other packages? So perhaps:
| Aspire: Particular.Aspire.Hosting.ServicePlatform | |
| AspirePlatformHost: Particular.Aspire.Hosting.ServicePlatform |
There was a problem hiding this comment.
I'm all for this
I think I made it Aspire.Hostingin my first pass
|
|
||
| See [Usage reporting when using the Azure Service Bus transport](/servicecontrol/servicecontrol-instances/configuration.md#usage-reporting-when-using-the-azure-service-bus-transport) for what each value is used for in ServiceControl. | ||
|
|
||
| #### Options |
There was a problem hiding this comment.
Is it worth listing this here if these are the same as linked to above? Otherwise if those defaults\requirements change then we'll have to repeat the same here.
There was a problem hiding this comment.
I think its worth keeping. These document the API specifically. If these changed on the transport, we'd need to change it in the package API anyway. The table also gives a bit more info on the type of param for the API.
|
|
||
| #### Options | ||
|
|
||
| | Option | Default | |
There was a problem hiding this comment.
Is it worth listing this here if these are the same as linked to above? Otherwise if those defaults\requirements change then we'll have to repeat the same here.
| // }); | ||
| ``` | ||
|
|
||
| Refer to the following table for the default endpoint names and ports exposed by each component: |
There was a problem hiding this comment.
Again, if we mention this info already elsewhere, should we just point to it instead of repeating it?
There was a problem hiding this comment.
the endpoint name info isn't documented elsewhere. I think you need this info for WithEndpoint(endpointName, callback), and those names are internal const in the package
There was a problem hiding this comment.
In fact, maybe we need to make those consts public?
| Two integration-specific points to keep in mind for any publish target: | ||
|
|
||
| - The license parameter default is materialized into the manifest at publish time. Supply the license through Aspire parameter inputs in production so the value does not live in the published output. See [Supplying the license in production](#production-considerations-supplying-the-license-in-production). | ||
| - The Learning transport is blocked in publish mode unless explicitly opted in. See [Learning transport in publish mode](#production-considerations-learning-transport-in-publish-mode). |
There was a problem hiding this comment.
This refers to a section that no longer exists
| - The Learning transport is blocked in publish mode unless explicitly opted in. See [Learning transport in publish mode](#production-considerations-learning-transport-in-publish-mode). | |
| - The Learning transport is blocked in publish mode. |
There was a problem hiding this comment.
3 participants
samples/hosting/aspireusing Aspire hosting packagesamples/aspire/platformsamples/aspire/platform-asbplatform/aspire