Conversation
…ow) + code formating
Feat/aspire
|
For getting this working, what did you do to ignore the rowversion columns, so it would work? I would like to add that to my PR (https://github.com/Valour-Software/Valour/pull/1380/changes), so I can finally test aspire. |
|
I think you may need to add a fallback, as when I tested aspire out, trying to run the server with docker containers (without using aspire/apphost) led to an exception regarding connecting to postgres. |
I just removed it entirely on these two lines:
I noticed there was a hard coded API url. I've updated that now |
This PR configures aspire for Valour primarily setup for development as to not disturb production, but long term it could be a good idea to look into implementing aspire for deployment as well (https://aspire.dev/deployment/overview/). There are also support for kubernetes deployment (https://aspire.dev/integrations/compute/kubernetes/)
There's a few benefits with having aspire with development alone such as not having to install other services locally other than a docker service (standard docker or other containerd services like podman). In addition aspire can take care of local secret management which would move it out of appsettings in a secrets storage locally on the device (https://aspire.dev/fundamentals/external-parameters/#secret-values). Then there's traces, default otel configuration etc
Developer onboarding could also be easier. I do recommend using the aspire cli for the best experience tho (https://aspire.dev/reference/cli/overview/)